|
Viewing Issue Simple Details
[ Jump to Notes ]
[ Wiki ]
|
[ View Advanced ]
[ Issue History ]
[ Print ]
|
|
ID |
Category |
Type |
Reproducibility |
Date Submitted |
Last Update |
|
0000612 |
[In-Portal CMS] Data Management |
bug report |
always |
2010-03-14 14:35 |
2010-06-10 13:44 |
|
|
Reporter |
alex |
View Status |
public |
Project Name |
In-Portal CMS |
|
Assigned To |
alex |
Developer |
|
|
Priority |
normal |
Resolution |
fixed |
Fixed in Version |
5.0.4-B1 |
|
Status |
closed |
Product Version |
5.0.2 |
Target Version |
5.0.4 |
| Time Estimate | No estimate | |
|
|
Summary |
0000612: Event "OnAfterItemUpdate" is called even, when nothing was changed in database |
|
Description |
Event "OnAfterItemUpdate" is called even, when nothing was changed in database. For example OnAfterItemDelete event is called only, when something was really deleted and OnAfterItemValidate is called only, when something was successfully validated. |
|
Additional Information |
|
| Tags |
No tags attached. |
|
|
Reference |
http://groups.google.com/group/in-portal-bugs/browse_thread/thread/16277c3680ccb85d |
|
Change Log Message |
|
|
Estimate Points |
0 |
|
|
Attached Files |
 onafteritemupdate_called_only_when_data_was_changed.patch [^] (678 bytes) 2010-03-14 14:35
[Show Content]
Index: db/dbitem.php
===================================================================
--- db/dbitem.php (revision 13130)
+++ db/dbitem.php (working copy)
@@ -513,13 +513,18 @@
return false;
}
- $affected = $this->Conn->getAffectedRows();
- if (!$system_update && $affected == 1) {
+ $affected_rows = $this->Conn->getAffectedRows();
+
+ if (!$system_update && ($affected_rows > 0)) {
$this->setModifiedFlag(clUPDATE);
}
$this->saveCustomFields();
- $this->raiseEvent('OnAfterItemUpdate');
+
+ if ($affected_rows > 0) {
+ $this->raiseEvent('OnAfterItemUpdate');
+ }
+
$this->Loaded = true;
if ($this->mode != 't') {
|
|
|
Relationships [ Relation Graph ]
[ Dependency Graph ]
Relationships 
(0001785)
(0002188)
