During update (and possibly during installation), the exception handler doesn't use the correct placeholder for the exception message, resulting in not-so-useful messages like:

Failed: PDOException: %message in db_change_field() (line 2895 of /var/www/drupal/includes/database/database.inc).

We broke that in #642160: Make debug() message work better (usable).

CommentFileSizeAuthor
#2 951486-exception-messages.patch2.46 KBDamien Tournoud
#1 951486-exception-messages.patch2.2 KBDamien Tournoud
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
Status: Active » Needs review
FileSize
951486-exception-messages.patch2.2 KB

This is an easy fix.

Damien Tournoud’s picture

Damien Tournoud CreditAttribution: Damien Tournoud commented
FileSize
951486-exception-messages.patch2.46 KB

With some additional comments.

chx’s picture

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

Good one! I like the extra comments (suggested by me ;) )

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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