In-Portal Issue Tracker

Welcome to the In-Portal Open Source CMS Issue Tracker! This is a central management / tracking tool for all types of tasks / issues / bugs for the In-Portal Project. Before reporting any issues, please make sure to read the Guide into Issue Tracker and How to Properly Test and Report Bugs!

Viewing Issue Simple Details Jump to Notes ] Wiki ] View Advanced ] Issue History ] Print ]
ID Category Type Reproducibility Date Submitted Last Update
0000104 [In-Portal CMS] Optimization task always 2009-06-30 04:14 2010-01-11 22:05
Reporter alex View Status public Project Name In-Commerce
Assigned To alex Developer
Priority normal Resolution fixed Fixed in Version 5.0.2
Status closed Product Version 5.0.0 Target Version 5.0.2
Time Estimate1 hours total (1 hours remaining)
Summary 0000104: Unify "Price" virtual field processing
Description 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.
Additional Information
Tags No tags attached.
Reference
Change Log Message
Estimate Points 0
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to 0000106needs work (Icebox)alex Ability to Change Price on Products "Advanced View" Grid 

-  Notes
User avatar (0000523)
alex (manager)
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.
User avatar (0000598)
alex (manager)
2009-09-30 14:29

Fix committed to 5.0.x branch. Commit Message:

Fixes 0000104: Unify "Price" virtual field processing
User avatar (0001391)
Dmitry (manager)
2010-01-11 22:05

Closing completed tasks.

- Related Changesets
Modules :: In-Commerce: 5.0.x r12645
Timestamp: 2009-09-30 14:29:13
Author: alex
Details ] Diff ]
Fixes 0000104: Unify "Price" virtual field processing
mod - /in-commerce/branches/5.0.x/in-commerce/units/products/products_config.php Diff ] File ]
mod - /in-commerce/branches/5.0.x/in-commerce/units/products/products_event_handler.php Diff ] File ]

- Issue History
Date Modified Username Field Change
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



Web Development by Intechnic
In-Portal Open Source CMS
In-Portal Open Source CMS
Copyright © 2000 - 2009 MantisBT Group

Powered by Mantis Bugtracker