The WATCHDOG_CRITICALI constant in core/modules/system/system.api.php documentation should be WATCHDOG_CRITICAL per the actual constant in bootstrap.inc.

CommentFileSizeAuthor
WATCHDOG_CRITICAL.patch588 bytesdroplet

Comments

jrreid’s picture

Status: Needs review » Reviewed & tested by the community

Looks pretty straight forward to me.

kristen pol’s picture

Title: no WATCHDOG_CRITICALI constant » Rename WATCHDOG_CRITICALI constant to WATCHDOG_CRITICAL
kristen pol’s picture

I also concur this is RTBC.

But, this does NOT need a backport for D7 as it is not an issue there. The documented code in D7 is as expected:

  $severity_list = array(
    WATCHDOG_EMERGENCY => t('Emergency'),
    WATCHDOG_ALERT     => t('Alert'),
    WATCHDOG_CRITICAL  => t('Critical'),
    WATCHDOG_ERROR     => t('Error'),
    WATCHDOG_WARNING   => t('Warning'),
    WATCHDOG_NOTICE    => t('Notice'),
    WATCHDOG_INFO      => t('Info'),
    WATCHDOG_DEBUG     => t('Debug'),
  );

Kristen

jrreid’s picture

Issue tags: -Needs backport to D7

Thanks for checking, I hadn't gotten around to checking 7.x yet.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

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

Anonymous’s picture

Issue summary: View changes

clean up issue description