Closed (fixed)
Project:
EVA: Entity Views Attachment
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Jul 2019 at 13:16 UTC
Updated:
17 Jul 2019 at 11:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
saranya ashokkumar commentedAttached patch for this issue. Please Review!!!
Comment #3
saranya ashokkumar commentedComment #4
ahebrank commentedLooks OK to me. While you're at it, can you make the default toUrl() parameter explicit:
to hint why we're checking that link template?
Comment #5
saranya ashokkumar commentedahebrank,
Thanks for reviewing the patch. By default, toUrl() function has argument as "canonical". So I didn't add it explicitly. Anyway, I updated the patch.
Please Review!
Comment #7
ahebrank commentedThanks. Merged to dev with a couple code standards tweaks.