In-Portal Issue Tracker - In-Commerce
Viewing Issue Advanced Details
104 [In-Portal CMS] Optimization task always 2009-06-30 04:14 2010-01-11 22:05
alex  
alex  
normal  
closed 5.0.0  
fixed  
 
none 5.0.2  
0
0000104: Unify "Price" virtual field processing
For now, when user enters price in product editing general tab it is set to primary price bracket for "Everyone" group. This is good of course. I've expected, that such type of processing will also happen, when I manually load product object and try to set price like this:

$product =& $this->Application->recallObject('p', null, Array ('skip_autoload' => true));
/* @var $product kCatDBItem */

$product->Load(55);
$product->SetDBField('Price', 77.55);
$product->Update();

Unfortunately this doesn't work, because price processing code is located inside OnPreSave event only, and not in OnBeforeItemCreate/OnBeforeItemUpdate event pair.

I propose to change that to expected behavior.
related to 0000106needs work  (Icebox)alex Ability to Change Price on Products "Advanced View" Grid 
Issue History
2010-01-11 22:05 Dmitry Note Added: 0001391
2010-01-11 22:05 Dmitry Status resolved => closed
2009-10-06 14:25 alex Type bug report => task
2009-09-30 14:29 alex Note Added: 0000598
2009-09-30 14:29 alex Status reviewed and tested => resolved
2009-09-30 14:29 alex Fixed in Version => 5.0.2
2009-09-30 14:29 alex Resolution open => fixed
2009-09-30 14:29 alex Changeset attached 5.0.x r12645
2009-09-30 06:59 alex Assigned To andrew => alex
2009-09-30 06:58 alex Status active => reviewed and tested
2009-09-30 06:58 alex Assigned To => andrew
2009-09-23 13:01 alex Time Estimate Added 1
2009-09-23 13:01 alex Note Added: 0000523
2009-09-20 15:08 alex Target Version 5.1.0 => 5.0.2
2009-06-30 22:08 Dmitry Relationship added related to 0000106
2009-06-30 22:04 Dmitry version => 5.0.0
2009-06-30 22:04 Dmitry Target Version => 5.1.0
2009-06-30 04:14 alex New Issue

Notes
(0000523)
alex   
2009-09-23 13:01   
Plan:
1. Move code from OnPreSave event to OnAfterItemCreate/OnAfterItemUpdate events. Code only should be invoked, when Price virtual field is changed.
(0000598)
alex   
2009-09-30 14:29   
Fix committed to 5.0.x branch. Commit Message:

Fixes 0000104: Unify "Price" virtual field processing
(0001391)
Dmitry   
2010-01-11 22:05   
Closing completed tasks.