Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#22425 closed defect (bug) (fixed)

Trunk Dashboard Throwing Fatal Error

Reported by: miqrogroove Owned by: nacin
Priority: normal Milestone: 3.5
Component: External Libraries Version: 3.5
Severity: major Keywords: has-patch
Cc: Focuses:

Description

Just installed trunk 3.5-beta2-22546 in debug mode. The dashboard Incoming Links widget displayed this:

Fatal error: Call to undefined method DOMElement::getLineNo() in wp-includes/SimplePie/Locator.php on line 280

Checked logs and found no other entries.

After navigating away and returning to the dashboard, error repeats.

Attachments (1)

22425.patch (692 bytes ) - added by SergeyBiryukov 14 years ago.
The mentioned fix

Download all attachments as: .zip

Change History (8)

#1 @nacin
14 years ago

  • Component GeneralExternal Libraries
  • Milestone Awaiting Review3.5

#2 @SergeyBiryukov
14 years ago

The method is only available since PHP 5.3.0.

There's a method_exists() check in two places, but not in that one.

Related: https://github.com/simplepie/simplepie/issues/223

#3 @georgestephanis
14 years ago

Yup, that line is

if ($this->base_location < $link->getLineNo())

Probably should be

if ($this->base_location < ( method_exists( $link, 'getLineNo' ) ? $link->getLineNo() : 1 ) )

@SergeyBiryukov
14 years ago

The mentioned fix

#5 follow-up: @SergeyBiryukov
14 years ago

  • Keywords has-patch added

Looks like the fix was only committed to the master branch and didn't get included in SimplePie 1.3.1 ([22366]).

#6 @nacin
14 years ago

  • Owner set to nacin
  • Resolutionfixed
  • Status newclosed

In 22555:

SimplePie: Ensure the DOMNode::getLineNo() method exists. props rmccue. fixes #22425.

#7 in reply to: ↑ 5 @rmccue
14 years ago

Replying to SergeyBiryukov:

Looks like the fix was only committed to the master branch and didn't get included in SimplePie 1.3.1 ([22366]).

Damn it, I'll backport that ASAP. Thanks.

Note: See TracTickets for help on using tickets.