CommentFileSizeAuthor
#1 fix_typo-2284883-1.patch794 bytesAohRveTPV
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

AohRveTPV’s picture

AohRveTPV CreditAttribution: AohRveTPV commented
Status: Active » Needs review
FileSize
fix_typo-2284883-1.patch794 bytes
jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs review » Needs work

Thanks! Would you care to fix the typo on the line above? "crypted" as far as I know isn't a word -- it should be "encrypted".

AohRveTPV’s picture

AohRveTPV CreditAttribution: AohRveTPV commented

"crypted" may refer to the crypt() function, so changing it might change the meaning. (And if it were changed, should crypt() also be changed to encrypt() for consistency?) Also I think it may not be accurate to refer to a hashed password as "encrypted".

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Status: Needs work » Reviewed & tested by the community

OK.

jhodgdon’s picture

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

Thanks again! Committed to 8.x.

  • Commit f742b6f on 8.x by jhodgdon: 
    Issue #2284883 by AohRveTPV: Fix typo in code comment in...

Status: Fixed » Closed (fixed)

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