Drupal 8.3 has moved the rest.link_manager service to the serialization module. The 'EntityReferenceRevisionsServiceProvider' still depends on the old res.link_manager which throws the error on 8.3:
The service "serializer.normalizer.entity_reference_revision_item" has a dependency on a non-existent service "rest.link_manager".

CommentFileSizeAuthor
#12 remove-rest-dependency-2862543-12.patch788 byteskmetz
#8 remove-rest-dependency-2862543-8.patch1013 bytesberdir
#2 dependency_on_rest-2862543-2.patch788 byteskalpaitch

Comments

kalpaitch created an issue. See original summary.

kalpaitch’s picture

kalpaitch commented
StatusFileSize
new dependency_on_rest-2862543-2.patch788 bytes

// @deprecated in Drupal 8.3.x and will be removed before Drupal 9.0.0.
// Use hal.link_manager instead.

Dependency should be on the hal link manager or force an additional dependency on hal.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Needs review
andypost’s picture

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

As 8.2 no longer supported it makes sense to commit

ayalon’s picture

ayalon commented

Please create a new version. A lot of peoploe will upgrade to 8.3.1 now and this module breaks every page that has paragraphs installed wihtout the patch.

vincejones’s picture

vincejones commented

Just did a fix for this issue locally using composer. You can add this to the "patches" section of your composer.json and run a composer install.

...
"extra": {
        "enable-patching": true,
        "patches": {
            "drupal/entity_reference_revisions": {
                "2862543-2 - Switch from using Rest LinkManager to Hal LinkManager": "https://www.drupal.org/files/issues/dependency_on_rest-2862543-2.patch"
            }
        },
}
...
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Title: Dependency on rest link_manager moved in 8.3 » Update link.manager dependency to use service by hal.module
Category: Bug report » Task
Priority: Normal » Major
Status: Reviewed & tested by the community » Needs work

#2849136: EntityReferenceRevisionsServiceProvider should check for rest module "fixed" this by ensuring that rest module is enabled and this was committed to the dev version, so that should work on 8.3.

This makes sense now, but it should remove the check that was added for rest, otherwise the result is that this doesn't run anymore at all when rest module is not enabled.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs work » Needs review
StatusFileSize
new remove-rest-dependency-2862543-8.patch1013 bytes

This should do the trick.

andypost’s picture

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

Yes, that's better

  • miro_dietiker committed 58ea6ad on 8.x-1.x authored by Berdir 
    Issue #2862543 by kalpaitch, Berdir: Update link.manager dependency to...
miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

Committed, thx.

kmetz’s picture

kmetz commented
StatusFileSize
new remove-rest-dependency-2862543-12.patch788 bytes

Adding remove-rest-dependency-2862543-12.patch, which works against 8.x-1.2.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented

That patch doesn't apply, we have committed it already to HEAD. It will be part of the next release.

Status: Fixed » Closed (fixed)

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

lpeabody’s picture

lpeabody commented

@kmetz thank you for the patch.