This is a very minor one with low prio:

Update hooks should use a different syntax, because the documentation summary (first line) is displayed to someone running update.php to tell them which updates need to run.
See: https://www.drupal.org/docs/develop/coding-standards/api-documentation-a...

CommentFileSizeAuthor
#4 After_patch.png35.3 KBAanal.addweb
#4 Before_patch.png41.13 KBAanal.addweb
#2 search_api_acquia-2938494-2.patch408 bytesstijnstroobants

Comments

StijnStroobants created an issue. See original summary.

stijnstroobants’s picture

stijnstroobants
Primary language Dutch
Location Leuven
commented
StatusFileSize
new search_api_acquia-2938494-2.patch408 bytes
stijnstroobants’s picture

stijnstroobants
Primary language Dutch
Location Leuven
commented
Status: Active » Needs review
Aanal.addweb’s picture

Aanal.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new Before_patch.png41.13 KB
new After_patch.png35.3 KB

@StijnStroobants, Thanks for the patch, it works well for me as It changes the standard as per documentation. I verified it by applying it manually in the module file.

  • japerry committed e4ab8a8 on 7.x-2.x 
    Issue #2938494: Different syntax for update hooks
japerry’s picture

japerry
Location KOMK
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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