Problem/Motivation

The aggregator module might be removed from core but it has several dependencies that are in core that only it uses. We should move those dependencies to aggregator to make it easier to remove them from core when aggregator is removed.

Proposed resolution

Deprecate feed.bridge.reader in core.services and \Drupal\Component\Bridge\ZfExtensionManagerSfContainer. Move them both to the aggregator module.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 3258654-2.patch6.92 KBalexpott

Comments

alexpott created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review
StatusFileSize
new 3258654-2.patch6.92 KB

Here we go... this completely moves the Laminas\Feed dependency into the aggregator module for Drupal 10. So we could deprecate the Aggregator module in favour of feeds in Drupal 9.4.x and then we would not need Laminas\Feed in core.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Looks ready

  • catch committed fbb54d8 on 10.0.x 
    Issue #3258654 by alexpott: Deprecate feed.bridge.reader and move it to...

  • catch committed 2d8630f on 9.4.x 
    Issue #3258654 by alexpott: Deprecate feed.bridge.reader and move it to...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 10.0.x and cherry-picked to 9.4.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

quietone’s picture

quietone commented

asked in #contribute if someone could update the CR, it only has an '@todo' for content. longwave responded, updating the CR and publishing it.