eva_plugin_display_entity implements get_path but does so in a way that consumers like render_more_link are guaranteed to be wrong, because views_plugin_display's logic never gets a chance to apply.

Our use case is setting the 'more link' to a page display that has arguments like 'node/%/events'. Current EVA behavior is to output NODE_PATH/NID which is wrong. eva_plugin_display_entity's get_path needs to check if a path has been overridden before applying its logic.

I will attach a patch that fixes the issue.

CommentFileSizeAuthor
#1 eva-get-path-semantics-2459633-1-7.x.patch777 bytescha0s

Comments

cha0s’s picture

cha0s commented
StatusFileSize
new eva-get-path-semantics-2459633-1-7.x.patch777 bytes
kevinsiji’s picture

kevinsiji commented
Status: Active » Reviewed & tested by the community

Tested. Working perfectly.

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented

Code looks reasonable.

  • axel.rutz committed 707ec01 on 7.x-1.x authored by cha0s 
    Issue #2459633 by cha0s: More link is broken due to get_path semantics
geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented

Fixed in 1.4-rc1, which will become 1.4 if enough people use it without bugs reported. See #2647732: Plan for EVA 7.x-1.4 release

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed

This was committed.

Status: Fixed » Closed (fixed)

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