A "DB_ERROR" constant is defined by both PEAR DB and Drupal. This causes a conflict with for example CiviCRM, which uses PEAR DB. I would suggest that the constant be renamed something like "DRUPAL_DB_ERROR". I've attached patches for Drupal 6 and 7.

CommentFileSizeAuthor
#4 d6-dberror.patch2.55 KBmfb
d6-dberror.patch3.54 KBmfb
d7-dberror.patch1.49 KBmfb

Comments

damien tournoud’s picture

damien tournoud commented
Version: 7.x-dev » 6.x-dev

DB_ERROR is of no use for 7.x, and will be removed (by #304924: Extend drupal_error_handler to manage exceptions). Downgrading to 6.x.

dpearcefl’s picture

dpearcefl commented

Is this still an issue using current Drupal 6?

Status: Needs review » Needs work

The last submitted patch, d6-dberror.patch, failed testing.

mfb’s picture

mfb
they or he
commented
Status: Needs work » Needs review
StatusFileSize
new d6-dberror.patch2.55 KB

I don't hold out much hope of such old patches being committed, but still an issue..

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.