Bug #16656
closedjQuery migrate helper can cause debug loops
0%
Description
I noticed when debugging #16655 that the front-end scripts for enable-jquery-migrate-helper were causing an infinite series of log statements. In this particular case, the errors are being triggered by the events-manager plugin upgrade from 5.12.1 to 6.0.1, but the infinite loop seems to be a bug in the enable-jquery-migrate-helper plugin.
Updated by Boone Gorges about 2 years ago
- Status changed from New to Resolved
In https://github.com/cuny-academic-commons/cac/commit/d6ada419fe7806d0652697df20bd3215a60af3bd I've added a clause to the enable-jquery-migrate-helper plugin that ensures that infinite log messages aren't written.
To be clear, I think that this points to a bug in events-manager, which will probably cause memory leakage and poor page performance. I don't have the bandwidth to do deeper debugging on that plugin at the moment. But at least my fix should eliminate the significant overhead that was resulting from the error logging.
Updated by Raymond Hoh about 2 years ago
To be clear, I think that this points to a bug in events-manager, which will probably cause memory leakage and poor page performance.
I took a look over on the wp.org forum for events-manager and a lot of people are complaining about performance issues in v6. For example, https://wordpress.org/support/topic/big-performance-issue-with-6-0-update/.
Also the changelog notes that v6.1 will have a "major architecture change for how tickets are stored" (their words), so we might have to look out for that when we bump Events Manager to v6.1.x and above.