Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Currently, there is no documentation for the difference between NOTICE, WARNING, and ERROR levels in watchdog. Probably because of that, they are used somewhat inconsistently throughout core. This patch attempts to provide some, which probably need to be agreed on and then a second patch can make the use of each consistent.
Comment | File | Size | Author |
---|---|---|---|
#8 | watchdog-document-constants-102603-8.patch | 3.49 KB | webchick |
#5 | watchdog-document-constants-102603-5.patch | 3.4 KB | webchick |
#4 | watchdog-document-constants-102603-4.patch | 2.98 KB | webchick |
#3 | watchdog-document-constants-102603-3.patch | 2.28 KB | webchick |
watchdog-doc.patch | 1.01 KB | webchick | |
Comments
Comment #1
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Thanks.
Comment #2
(not verified) CreditAttribution: commentedComment #3
webchickOops. These got un-documented when we switched to using standard error reporting. Let's try that again. ;)
Comment #4
webchickActually, this is better.
1. Moved the reference to the RFC URL to the PHPDoc of watchdog_severity_levels() which, as a side effect, documented that function. :)
2. Fixed some pluralization inconsistency (thanks, Morbus!)
3. Separate the "Log message" and the "Thing: Defintion" with "--" to make it a bit more clear (thanks, Morbus!)
4. Moved the @see references to the constants themselves, rather than in that weird comment block above which won't be displayed anywhere.
5. Removed said weird comment block above.
Comment #5
webchickOops. I misunderstood Morbus on #2. Setting back to the way things are pluralized in the RFC.
Note: "Notice" is inconsistent by being singular rather than plural, but we decided it's best to stick with what's in the RFC rather than deviate from it, even slightly. I'll go file a bug with IETF. ;)
Comment #6
Morbus IffTogether in IRC.
Comment #7
Senpai CreditAttribution: Senpai commentedSecond review, same as the first, a little bit louder, and a whole...
Never mind. RTBC via IRC chat.
Comment #8
webchickPatch is the same; only difference is it now references ietf.org rather than faqs.org for the RFC URL. I believe this is now every pedantic thing we could possibly fix about this documentation. Feels good. :)
Thanks, Morbus and Senpai! :)
Comment #9
webchickYay! This was committed as part of #101543: Document all constants.
Comment #10
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.