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