The docblock for hasLowerLanguageNegoitationWeight() has a typo. It says "TRUE if the the content entity language negotiator has higher priority"

CommentFileSizeAuthor
#4 typo-fix-2620266-4.diff1.02 KBAnonymous (not verified)
#2 typo-fix-2620266-2.diff921 bytesAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous at Ashday Interactive Systems commented

JKerschner created an issue. See original summary.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous at Ashday Interactive Systems commented
Status: Active » Needs review
FileSize
typo-fix-2620266-2.diff921 bytes
plach’s picture

plach
he/him
Primary language Italian
Location Venezia
CreditAttribution: plach as a volunteer commented
Status: Needs review » Needs work 
+++ b/core/modules/language/src/Plugin/LanguageNegotiation/LanguageNegotiationContentEntity.php
@@ -181,7 +181,7 @@ public function getLanguageSwitchLinks(Request $request, $type, Url $url) {
-   *   TRUE if the the content entity language negotiator has higher priority
+   *   TRUE if the content entity language negotiator has higher priority
    *   than the url language negotiator, FALSE otherwise.

Thanks, but now the comment needs to be adjusted to wrap as close as possible to column 80.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous at Ashday Interactive Systems commented
Status: Needs work » Needs review
FileSize
typo-fix-2620266-4.diff1.02 KB

Fixed to wrap comments correctly.

cilefen’s picture

cilefen CreditAttribution: cilefen commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Documentation

There is not much to say!

  • xjm committed df64790 on 8.1.x 
    Issue #2620266 by JKerschner, plach: Typo in...
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.1.x and 8.0.x. Thanks for the patch!

  • xjm committed e451b4b on 8.0.x 
    Issue #2620266 by JKerschner, plach: Typo in...

Status: Fixed » Closed (fixed)

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