[wp-trac] [WordPress Trac] #39338: class-wp-hook.php - apply_filters() infinite loop
WordPress Trac
noreply at wordpress.org
Thu Dec 29 00:17:47 UTC 2016
#39338: class-wp-hook.php - apply_filters() infinite loop
-------------------------------------------+--------------------
Reporter: frettled | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 4.7.1
Component: Plugins | Version: 4.7
Severity: normal | Resolution:
Keywords: reporter-feedback needs-patch | Focuses:
-------------------------------------------+--------------------
Changes (by dd32):
* milestone: Awaiting Review => 4.7.1
Comment:
Replying to [comment:15 frettled]:
> I've seen this dozens of times, for different WP installations. Please
don't mistake the lack of reports for a lack of a problem.
If you could provide any common details about installations which are
experiencing the issue, it'll help greatly.
> I apologize for not reporting this earlier, although please trust me
when I say that I could have reported this a long time ago, and
frequently, and kept nagging about it for months and years. :)
In 4.7 the code for `apply_filters()` changed significantly, it was almost
a rewrite in many respects - That's why we're interested here, the code in
question simply did not exist in it's current form in the past (although
previously it was a similar `next()` loop, but based on data that could
change or vanish - unlike in 4.7)
The change you've proposed will not be merged as-is, without a
reproducible test case or understanding the circumstances under which it
can be triggered.
I'm happy to help you debug a site which experiences the issue in order to
track down what's happening, just email me with details.
I've milestoned this for 4.7.1 for tracking purposes and investigation,
but will not be in a release until the above conditions are met.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/39338#comment:17>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list