FIX [ bug 1634 ] Error deleting a project when it had many linked objects

Projet::delete was throwing DB error messages directly to the user instead of logging them and showing a proper error message to the user
This commit is contained in:
Marcos García de La Fuente 2015-07-19 14:56:52 +02:00
parent c1445a88e1
commit 1be33e1dda
3 changed files with 3 additions and 2 deletions

View File

@ -13,6 +13,7 @@ FIX [ bug #2855 ] Wrong translation key in localtax report page
FIX [ bug #1852 ] JS error when editing a customer order line FIX [ bug #1852 ] JS error when editing a customer order line
FIX [ bug #2900 ] Courtesy title is not stored in create thirdparty form FIX [ bug #2900 ] Courtesy title is not stored in create thirdparty form
FIX [ bug #3055 ] Product image thumbnails were not deleted after deleting the image FIX [ bug #3055 ] Product image thumbnails were not deleted after deleting the image
FIX [ bug 1634 ] Error deleting a project when it had many linked objects
***** ChangeLog for 3.7.1 compared to 3.7.* ***** ***** ChangeLog for 3.7.1 compared to 3.7.* *****
FIX Bug in the new photo system FIX Bug in the new photo system

View File

@ -339,7 +339,7 @@ if (empty($reshook))
else else
{ {
dol_syslog($object->error,LOG_DEBUG); dol_syslog($object->error,LOG_DEBUG);
setEventMessage($object->error,'errors'); // setEventMessage($object->error,'errors');
setEventMessage($object->errors,'errors'); setEventMessage($object->errors,'errors');
} }
} }

View File

@ -533,7 +533,7 @@ class Project extends CommonObject
$resql = $this->db->query($sql); $resql = $this->db->query($sql);
if (!$resql) if (!$resql)
{ {
$this->errors[] = $this->db->lasterror(); $this->errors[] = $langs->trans("CantRemoveProject");
$error++; $error++;
} }
} }