WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#29123 closed defect (bug) (fixed)

Undefined index within testcase.php _restore_hooks()

Reported by: kevinlangleyjr Owned by: SergeyBiryukov
Priority: normal Milestone: 4.0
Component: Build/Test Tools Version: 4.0
Severity: normal Keywords:
Cc: Focuses:

Description (last modified by SergeyBiryukov)

Due to the change made here, [29251] to backup and restore these global variables, all unit tests are failing for me with Undefined index: merged_filters . By adding a check on line 101 to check if the value exists within the self::$hooks_saved would prevent this error from occurring.

Here is an example of the phpunit output there. ​https://travis-ci.org/voceconnect/thermal-api/jobs/31176473

Attachments (1)

testcase.php.patch​ (179 bytes ) - added by kevinlangleyjr 12 years ago.

Download all attachments as: .zip

Change History (7)

@kevinlangleyjr
12 years ago

#1 @TobiasBg
12 years ago

Possible related: #28986

#2 @SergeyBiryukov
12 years ago

  • Description modified (diff)

#3 @helen
12 years ago

  • Milestone Awaiting Review
  • Resolution → invalid
  • Status new → closed

[29251] was reverted. Feel free to re-open if this is still an issue.

#4 @SergeyBiryukov
12 years ago

  • Milestone → Awaiting Review
  • Resolution invalid
  • Status closed → reopened

[29251] was not reverted, [29267] was (in [29274]).

Last edited 12 years ago by SergeyBiryukov (previous) (diff)

#5 @SergeyBiryukov
12 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status reopened → closed

In 29503:

Avoid undefined index notices in WP_UnitTestCase::_restore_hooks().

props kevinlangleyjr.
fixes #29123.

#6 @SergeyBiryukov
12 years ago

  • Milestone Awaiting Review → 4.0
Note: See TracTickets for help on using tickets.