While checking the jsonapi core module, I noticed that in the core/modules/jsonapi/jsonapi.api.php file there is a plural possessive typo: modules's (127:13).

Changing this will have no effect on any code, just a simple typo in a comment.

CommentFileSizeAuthor
#4 Screenshot 2022-07-26 at 3.11.54 PM.png23.48 KBchaubeyji
#2 3283929-fix-comment-with.patch642 byteseleonel
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

eleonel created an issue. See original summary.

eleonel’s picture

eleonel
he/him
Primary language Spanish
Location Itoshima 🇯🇵
CreditAttribution: eleonel at Morpht commented
FileSize
3283929-fix-comment-with.patch642 bytes
eleonel’s picture

eleonel
he/him
Primary language Spanish
Location Itoshima 🇯🇵
CreditAttribution: eleonel at Morpht commented
Status: Active » Needs review
chaubeyji’s picture

chaubeyji CreditAttribution: chaubeyji as a volunteer and at OpenSense Labs for DrupalFit commented
FileSize
Screenshot 2022-07-26 at 3.11.54 PM.png23.48 KB

Applied patch on #2. Working fine.

smustgrave’s picture

smustgrave CreditAttribution: smustgrave at Mobomo commented
Priority: Normal » Minor
Status: Needs review » Reviewed & tested by the community

Changing to a minor

But the change appears to be correct to me and as the description says no code was touched.

  • catch committed 0ac15a8 on 10.0.x 
    Issue #3283929 by eleonel, smustgrave: Fix comment with a plural...
  • catch committed 67e7431 on 10.1.x
    Issue #3283929 by eleonel, smustgrave: Fix comment with a plural...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 10.1.x and cherry-picked to 10.0.x, thanks!

Status: Fixed » Closed (fixed)

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