Closed (fixed)
Project:
Drupal core
Version:
8.2.x-dev
Component:
documentation
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
26 Oct 2016 at 13:04 UTC
Updated:
30 Dec 2016 at 16:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
shashikant_chauhan commentedadding patch.
Comment #3
chi commentedComment #4
xjmI actually do not think we should use a shorthand here, because
\Drupalis a mapping for so many different classes in so many different namespaces. Elsewhere in/core/lib/Drupal.php, we always use the fully qualified namespace for clarity.However, I actually think the whole sentence is just redundant with the @see and the sentence before that says " the link generator service's generate() method". Let's just delete the sentence?
Also, @Chi, can you add a comment describing how you reviewed issues when you mark them RTBC? This will ensure you get credit for your reviews in the future, and also help committers know what points of review have been considered or not.
Thanks @Chi and @shashikant_chauhan!
Comment #5
shashikant_chauhan commented@xjm, I have removed the redundant statement in this patch.
The issue & patch 2822296-2.patch was based on alexpott's comment https://www.drupal.org/node/2819479#comment-11748024
Comment #6
jp.stacey commentedJust for clarity:
* I've reworded the description for this ticket to talk about removing the reference rather than rewording it, to match @xjm's request in #4 and @shashikant_chauhan's patch in #5.
* I'm also uploading an interdiff for patches #2 and #5, to get some idea of what's changed.
Otherwise, I can confirm the patch applies cleanly to 8.3.x HEAD and satisfies the new description.
Comment #7
jp.stacey commentedSorry: I did apply to 8.3.x HEAD rather than 8.2.x HEAD; but do we not want to be applying it to the most up-to-date 8.* HEAD, and only backporting it if we need to?
Comment #10
xjmThanks @jp.stacey and @shashikant_chauhan!
@jp.stacey, documentation improvements can always be backported, so this was moved to 8.2.x to indicate it would be committed to both active branches. Typically it's okay to create and test backportable patches with either branch, except for the rare cases where the branches have diverged and the patch doesn't apply to both. You can see the details of what patches we backport at: https://www.drupal.org/core/d8-allowed-changes#patch
I confirmed this is the only case of this stale reference in core:
Committed and pushed to 8.3.x and 8.2.x. Thanks!
Comment #11
jp.stacey commented@xjm thanks for the clarification!