Problem/Motivation

Core links to nodes by building hardcoded URL paths, instead of using entity_uri(). This means the node entity type's URI callbacks are bypassed completely, disallowing other modules from altering node URLs.

Proposed resolution

Replace hardcoded URL paths with calls to entity_uri().

Remaining tasks

None.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 drupal_2594585_2.patch1.04 KBxano

Comments

Xano created an issue. See original summary.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Status: Active » Needs review
StatusFileSize
new drupal_2594585_2.patch1.04 KB
dcam’s picture

dcam commented
Status: Needs review » Reviewed & tested by the community

Makes sense. I tested the patch. After applying the patch the node title links in admin/content were still linking to the correct pages.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

Thanks for the review!

See #2611906: Use entity_uri() when linking to terms for a similar issue.

David_Rothstein’s picture

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

Committed to 7.x - thanks!

  • David_Rothstein committed 193adfa on 7.x 
    Issue #2594585 by Xano, dcam: Use entity_uri() when linking to nodes on...

Status: Fixed » Closed (fixed)

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