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 ]

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 Estimate

1 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

Attached Files

Relationships [ Relation Graph ] [ Dependency Graph ]

related to

0000106

needs work (Icebox)

alex

Ability to Change Price on Products "Advanced View" Grid

Notes
(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.

(0000598) alex (manager) 2009-09-30 14:29	Fix committed to 5.0.x branch. Commit Message: Fixes 0000104: Unify "Price" virtual field processing

(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