On Feb 11, 2014, at 9:07 AM, Jacques Le Roux <jacques.le.r...@les7arts.com> 
wrote:

> I'd create a Jira. It looks like a miss to me, the check you do seems good. 
> I'd also prevent users to create such rules from UI
> Maybe we could add a properties, in case someone would like to use a such 
> scenario (could be used for temporarily give a percent on all products)?

I don't like the idea to prevent this in the ui because the user may really 
know what she is doing. I also don't like the idea of (another) property.
I would rather improve the ui to make it more evident to the user that the 
rules will be applied to all the products.

Jacopo


> 
> Jacques
> 
> On Tuesday, February 11, 2014 3:12 AM, s...@thedevers.org wrote
>> I recently had a user accidentially create a product price rule with no
>> conditions.  This resulted in all products sold afterwards getting the
>> discounted price (-5% price) set by the action.
>> 
>> After tracking this down, I added some code in PriceServices.java to check
>> if there were no price conditions and if so, set the local variable alltrue
>> to false.  This fixes it for me.
>> 
>> I am debating whether or not to add a Jira for this issue.  Was this
>> intended and can anyone see any value in this behavior?
>> 
>> Skip

Reply via email to