I was adding test coverage today when I discovered a problem with the new author parsing that was added for 🐛 Author field is not saved on aggregator item Fixed . Here's the sequence of events:
I found this by adding an assertion for the empty <author> tag item that's in aggregator_test_rss091.xml. The author that should have been empty wasn't. It had the author from several items before it. Once a new author was parsed by our extension, then that replaced the previous one in getting copied to following items.
The Laminas reader has an internal cache when parsing items (entries). If a function is called again for the same item, then you don't have to re-query the DOM. But for whatever reason that cache is getting copied from from entry to entry. I don't know why this is happening, but an easy fix is to not write to or read from the cache in our extension.
Fixed
2.0
Code
The last submitted patch, 2: 3392510-2-test-only.patch, failed testing. View results →
Automatically closed - issue fixed for 2 weeks with no activity.