Just noticed this because of my aggressive highlighting of coding standards in my editor :)

Patch coming up.

CommentFileSizeAuthor
#2 missing_function-2845290-2.patch541 byteseiriksm

Comments

eiriksm created an issue. See original summary.

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Status: Active » Needs review
StatusFileSize
new missing_function-2845290-2.patch541 bytes
poker10’s picture

poker10 commented
Status: Needs review » Reviewed & tested by the community

This looks good, thanks! Moving to RTBC.

The similar comment is used in user_admin_role_validate() function.

  • poker10 committed c355b0b5 on 7.x 
    Issue #2845290 by eiriksm: Missing function comment doc for...

poker10 credited mcdruid.

poker10’s picture

poker10 commented
Assigned: eiriksm » Unassigned
Status: Reviewed & tested by the community » Fixed

This got +1 from @mcdruid on Slack, so committed. Thanks @eiriksm!

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
commented

Haha, nice random change to get committed after 6 years 😅🤓

poker10’s picture

poker10 commented

Better late than never :) I was doing some documentation issues cleanup and also run into this issue.

Status: Fixed » Closed (fixed)

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