Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
node system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
7 Sep 2015 at 17:44 UTC
Updated:
15 Nov 2015 at 14:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
olli commentedThis seems to work.
Comment #3
dawehner+1 for the fix, of course!
Comment #4
geertvd commentedAdded test coverage (not just for the issue described but for the whole nid argument handler)
Comment #5
geertvd commentedI had to borrow this line from #2492839: Views replacement token bc layer allows for Twig template injection via arguments to make my test pass.
Comment #10
geertvd commentedComment #11
dawehnerIMHO we should use a file, if possible.
Comment #16
geertvd commentedYeah, I guess it does make more sense to use a real views export there.
Also fixed the test since we shouldn't be using the
%1syntax anymore but rather{{ arguments.nid }}Comment #19
dawehnerYeah! As always I like if things are actually broken and we can expand our test coverage.
Comment #20
alexpottThis is bug with a non-disruptive fix - permitted in beta. Committed 7f11ec7 and pushed to 8.0.x. Thanks!
Comment #24
geertvd commented