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)?

Jacques

On Tuesday, February 11, 2014 3:12 AM, [email protected] 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