This patch updates the "Implementation of hook_NAME()." text in the code comments found in Drupal's core modules. Thanks to Morbus and chx for Drupal Tough Love, where I've learned that no coding style violation is too small to go unnoticed.
For the most part all I did here was add a period to end the line. "Implementation of hook_theme()" was the most common string missing a period.
Why do user.module and node.module sometimes include "Implementation of a Drupal action" in the code comments and not include the function name?
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | core_code_comment_periods.patch | 8.12 KB | dmitrig01 |
| core_code_comment_periods.patch | 5.95 KB | christefano |
Comments
Comment #1
morbus iffI'd like to point out that most of these errors are the direct result of new hooks inside Drupal 6. It's just something that wasn't caught before, but it does indicate that core has been pretty careful about following the suggested guideline of a period at the end.
Comment #2
catchNo longer applies.
Comment #3
dmitrig01 commentedRe-rolled
Comment #4
cburschkaLooks like that takes care of all of them.
Comment #5
dries commentedCommitted to CVS HEAD. Yay.
Comment #6
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.