The error code E_RECOVERABLE_ERROR (4096) is missing from the list of possible error codes.
See http://www.php.net/manual/en/ref.errorfunc.php#e-recoverable-error

CommentFileSizeAuthor
#3 drupal-head.error-handler.junyor.patch1.1 KBJunyor
#1 error_handling_0.patch910 bytesthesaint_02
error_handling.patch892 bytesthesaint_02
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

thesaint_02’s picture

thesaint_02 CreditAttribution: thesaint_02 commented
Category: task » bug
FileSize
error_handling_0.patch910 bytes

now a patch that is made from root dir

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Version: 5.1 » 6.x-dev
Status: Reviewed & tested by the community » Needs work

This should be committed to 6-dev first, but

patching file includes/common.inc
Hunk #1 FAILED at 544.
1 out of 1 hunk FAILED -- saving rejects to file includes/common.inc.rej

Junyor’s picture

Junyor CreditAttribution: Junyor commented
Status: Needs work » Needs review
FileSize
drupal-head.error-handler.junyor.patch1.1 KB

Updated patch to HEAD. I found that I needed this for Drupal 5.x, FWIW.

jvandyk’s picture

jvandyk CreditAttribution: jvandyk commented
Status: Needs review » Reviewed & tested by the community

This looks good to me. E_RECOVERABLE_ERROR is available since PHP 5.2.0.

Dries’s picture

Dries CreditAttribution: Dries commented
Version: 6.x-dev » 5.x-dev

Thanks! I've committed this to CVS HEAD. Will need to get committed to DRUPAL-5 as well. Lowering the version.

Junyor’s picture

Junyor CreditAttribution: Junyor commented

I think the patch from #1 will apply to 5.x while the one for HEAD won't.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Reviewed & tested by the community » Fixed

Committed to 5.x.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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