Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Feb 2014 at 12:36 UTC
Updated:
29 Jul 2014 at 23:22 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
paulh commentedUpdated documentation.
Comment #2
paulh commentedComment #4
jhodgdonThanks for filing this issue! I agree with the intent.
The patch, however, is not quite accurate:
The signature of url() does not include route_name or route_parameters, so this part of the documentation needs an update.
Comment #5
larowlanIt matches the signature of \Drupal::url not url(), perhaps we should reference the \Drupal::url() method instead?
Comment #6
larowlanThere is also whitespace on the end of 'keys', can that be removed during next patch re-roll?
Comment #7
paulh commentedUpdated the documentation to reference the correct method. Also stripped trailing white space.
Comment #8
paulh commentedComment #10
berdirComment #11
berdir7: documentation-2199369-1.patch queued for re-testing.
Comment #12
jhodgdonThanks, seems fine to me!
Comment #13
larowlan+1
Comment #14
jhodgdonThanks again everyone! Committed to 8.x.