See https://www.drupal.org/node/2758897

The existing code should be backwards compatible. However, it's not working properly during site-install. The easy fix is to upgrade the code to not rely on a deprecated service.

Would you like a dependency added to the .info file, or is this OK by itself?

After fixing this, could you please tag a release.

CommentFileSizeAuthor
#8 hal-link_manager-2875499-8.patch1016 bytespiggito
#4 hal-link_manager-2875499-3.patch2.2 KBdouggreen
hal-link_manager.txt2.2 KBdouggreen

Comments

douggreen created an issue. See original summary.

badjava’s picture

badjava commented
Status: Active » Needs review
badjava’s picture

badjava commented
Status: Needs review » Needs work

@douggreen Can you update the file to a .patch file?

douggreen’s picture

douggreen commented
StatusFileSize
new hal-link_manager-2875499-3.patch2.2 KB
badjava’s picture

badjava commented
Status: Needs work » Needs review

  • badjava committed 153165a on 8.x-1.x authored by douggreen 
    Issue #2875499 by douggreen: Change rest.link_manager to hal....
badjava’s picture

badjava commented
Status: Needs review » Fixed

Thanks @douggreen

piggito’s picture

piggito commented
StatusFileSize
new hal-link_manager-2875499-8.patch1016 bytes

Adding patch to change a remaining rest LinkManager dependency in TermEntityNormalizer.

piggito’s picture

piggito commented
Status: Fixed » Needs review
andypost’s picture

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

  • badjava committed 1de13a4 on 8.x-1.x authored by piggito 
    Issue #2875499 by piggito, andypost: Changed rest.link_manager to hal....
badjava’s picture

badjava commented
Status: Reviewed & tested by the community » Fixed

Good catch and thanks @piggito and @andypost. Committed and cut another release.

Status: Fixed » Closed (fixed)

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