We have a shiny new #1158322: Add backtrace to all errors and even #1118520: Add inclusion of a settings.local.php file in settings.php...

...but the error level constants cannot be used for $conf in settings.php. :(

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

Status: Active » Needs review
FileSize
1.55 KB

This should do it.

If I'm not mistaken, then we have a test somewhere that checks for exact line numbers in errors.inc (which was the original reason for splitting that code out of frequently changed files such as bootstrap.inc and common.inc), so this change might trigger a test failure.

sun’s picture

Issue tags: -Needs backport to D7

#1: drupal8.error-constants.1.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Needs backport to D7

The last submitted patch, drupal8.error-constants.1.patch, failed testing.

sun’s picture

Assigned: Unassigned » sun
Status: Needs work » Needs review

So this patch actually passed should be RTBC if it still comes back green.

sun’s picture

heh. :)

jbrown’s picture

Status: Needs review » Reviewed & tested by the community
webchick’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed and pushed to 8.x. Thanks!

For some reason this didn't apply to 7.x, so marking for backport.

dcam’s picture

Status: Patch (to be ported) » Needs review
FileSize
1.28 KB

Backported #5 to D7.

sun’s picture

Assigned: sun » Unassigned
Status: Needs review » Reviewed & tested by the community

Thanks!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 7.x. Thanks!

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