#22425 closed defect (bug) (fixed)
Trunk Dashboard Throwing Fatal Error
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.5 |
| Component: | External Libraries | Version: | 3.5 |
| Severity: | major | Keywords: | has-patch |
| Cc: |
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)
Change History (8)
- Component changed from General to External Libraries
- Milestone changed from Awaiting Review to 3.5
comment:2
SergeyBiryukov — 6 months ago
comment:3
georgestephanis — 6 months 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 ) )
comment:4
georgestephanis — 6 months ago
rmccue added a fix a month ago on the github repo --
https://github.com/simplepie/simplepie/commit/75baa9651ee04c8c7d57702a8d5f1b7b6eb58605
comment:5
follow-up:
↓ 7
SergeyBiryukov — 6 months 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]).
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In 22555:
Note: See
TracTickets for help on using
tickets.

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