Multi-firing of BXGY promotion - consumption of qualifying order entries

Question

I have implemented BXGY custom promotion. To simplify my case let's say after you have X products in your cart you'll get Y free products.

If I have 2*X qualifying products in cart expected behavior is to get 2*Y free products.

My problem is multi-firing of that promotion. If I set maximum count of multifiring to 4, than action (get free products) is activated 4 times although I have 2*X products in cart.

So my conclusion is that I need to consume qualifying products (entries with that product) like free products that user gets are consumed so they cannot participate in qualification for another promotion.

Conclusion of my research is that I need to consume entries in (YFreeGift)RAOAction, but then I need to retrieve all conditional products and I don't feel that it's a right way (condition in action).

Do anybody knows how to consume qualifying entries?

Condition translator for BXGY (I have mandatory qualifying products too but that isn't important for this problem)

public class RuleHeinekenBXGYFQualifyingProductsConditionTranslator implements RuleConditionTranslator
{
    @Override
    public RuleIrCondition translate(RuleCompilerContext context, RuleConditionData ruleCondition, RuleConditionDefinitionData conditionDefinition) throws RuleCompilerException
    {
        List<String> mandatoryProducts = (List) getConditionParameterValue(ruleCondition, "mandatoryQualifyingProducts");
        List<String> alternativeProducts = (List) getConditionParameterValue(ruleCondition, "alternativeQualifyingProducts");
        Integer qualifyingCount = (Integer) getConditionParameterValue(ruleCondition, "qualifyingCount");

        if(isEmpty(mandatoryProducts) && isEmpty(alternativeProducts) || qualifyingCount == null || qualifyingCount <= 0)
            throw new PromotionConditionParametersValidationException();

        String cartRAO = context.generateVariable(CartRAO.class);
        List<RuleIrCondition> mandatoryProductsConditions = getMandatoryProductsConditions(context, mandatoryProducts, cartRAO);
        List<RuleIrCondition> qualifyingCountCondition = getQualifyingCountCondition(context, qualifyingCount, cartRAO);

        RuleIrGroupCondition qualifyingConditions = new RuleIrGroupCondition();
        qualifyingConditions.setOperator(RuleIrGroupOperator.AND);
        qualifyingConditions.setChildren(listUnion(mandatoryProductsConditions, qualifyingCountCondition));
        return qualifyingConditions;
    }

    private List<RuleIrCondition> getQualifyingCountCondition(RuleCompilerContext context, Integer qualifyingCount, String cartRAO)
    {
        String qualifyingCountRAO = context.generateVariable(QualifyingCountRAO.class);
        String promotionCode = context.getRule().getCode();

        return getListOfRuleConditions(
            aRuleCondition()
                .withModelRAO(qualifyingCountRAO)
                .withAttribute("promotionCode")
                .withOperator(EQUAL)
                .withValue(promotionCode)
                .buildAttributeCondition(),

            aRuleCondition()
                .withModelRAO(qualifyingCountRAO)
                .withAttribute("qualifyingCount")
                .withOperator(GREATER_THAN_OR_EQUAL)
                .withValue(qualifyingCount)
                .buildAttributeCondition(),

            aRuleCondition()
                .withModelRAO(cartRAO)
                .withAttribute("qualifyingCounts")
                .withOperator(CONTAINS)
                .withTargetVariable(qualifyingCountRAO)
                .buildAttributeRelationCondition());
    }

    private List<RuleIrCondition> getMandatoryProductsConditions(RuleCompilerContext context, List<String> mandatoryProducts, String cartRAO)
    {
        if(isEmpty(mandatoryProducts))
            return emptyList();

        return getMapOfQualifyingProductsWithQuantities(mandatoryProducts)
                 .entrySet().stream()
                 .map(entry -> getMandatoryProductCondition(context, cartRAO, entry.getKey(), entry.getValue()))
                 .collect(toList());
    }

    private RuleIrExistsCondition getMandatoryProductCondition(RuleCompilerContext context, String cartRAO, String product, int qualifyingCount)
    {
        RuleIrLocalVariablesContainer variablesContainer = context.createLocalContainer();
        String containsProductRAO = context.generateLocalVariable(variablesContainer, ProductRAO.class);
        String containsOrderEntryRAO = context.generateLocalVariable(variablesContainer, OrderEntryRAO.class);

        List<RuleIrCondition> listOfConditions = getListOfRuleConditions(
            aRuleCondition()
                .withModelRAO(containsProductRAO)
                .withAttribute("code")
                .withOperator(EQUAL)
                .withValue(product)
                .buildAttributeCondition(),

            aRuleCondition()
                .withModelRAO(containsOrderEntryRAO)
                .withAttribute("product")
                .withOperator(EQUAL)
                .withTargetVariable(containsProductRAO)
                .buildAttributeRelationCondition(),

            aRuleCondition()
                .withModelRAO(containsOrderEntryRAO)
                .withAttribute("quantity")
                .withOperator(GREATER_THAN_OR_EQUAL)
                .withValue(qualifyingCount)
                .buildAttributeCondition(),

            aRuleCondition()
                .withModelRAO(cartRAO)
                .withAttribute("entries")
                .withOperator(CONTAINS)
                .withTargetVariable(containsOrderEntryRAO)
                .buildAttributeRelationCondition());

        RuleIrExistsCondition mandatoryProductsExistCondition = new RuleIrExistsCondition();
        mandatoryProductsExistCondition.setVariablesContainer(variablesContainer);
        mandatoryProductsExistCondition.setChildren(listOfConditions);
        return mandatoryProductsExistCondition;
    }
}

score 1 · Answer 1 · answered May 29 '17 at 23:41

1

Rule Engine provides a mechanism for you to configure a limit on the number of times a rule can trigger Action Items.

The maximum rule executions allow you to control the maximum number of times the action for a rule can be executed, as long as the conditions are met. For all out-of-the-box promotions actions, the value of this attribute should be set to one.

Look at RuleConfigurationRRD attributes.

Taken from help.hybris.com.

answered May 29 '17 at 23:41

arsen_adzhiametov

696
5
17

from reading the question it looks like Zildyan is already aware of this property. – Sebastian May 31 '17 at 15:03

Sebastian · Answer 2 · 2017-05-31T15:39:51.297

1

what version are you on? There is a bug in the free gift action prior to 6.4 or 6.3 AFAIK. The free gift action is comprised of 2 actions internally (add product to cart and give 100% discount on added product). Due to the bug you might have to increase the maximum rule executions to double the amount you intend originally (as each action "add to cart" and "discount product" will count as one execution).

To answer your highlighted question about order entry consumption: The promotionengine doesn't support order entry consumption OOTB right now. This is a feature that was part of the legacy promotions but has been dropped in favor of stackability.

But more generally, the approach you seem to take is not how hybris intends the free gift action to be used. If you want a BOGOF (or BXGYF) you would normally define a promotion that checks if both your qualifying product X AND the to-be-discounted products Y are in the cart in sufficient quantity. The action would then simply discount the Y products. You can also add a potential promotion checking only for the product X that will trigger a message like Add n products Y to your cart to get them for free in order to alert the customer that s/he is eligible for a promotion.

If you look at the promotionenginesamplesaddon extension/addon you will find those two promotions that should do exactly what I mean above:

potential_product_buy_x_get_y_free
product_buy_x_get_y_free

Hope this helps,

Sebastian

edited May 31 '17 at 15:39

answered May 31 '17 at 15:02

Sebastian

1,743
1
16
35

Thank you for good answer, but - and there is always but. Client has some complex custom promotions. I succeed to implement them but I'm stuck on this. so basically if you have two BXGY promotion that have same or intersecting qualifying products, one with higher priority should disable those that she used so other BXGY promotion can have only leftovers, if it's qualified wtih them, great. – Zildyan May 31 '17 at 16:27
Example: Buy 4 beers get 1 Gin with higher priority then Buy 2 Beers get 1 Whisky. If there is 10 beers in a cart, customer should get 2 Gins and 1 Whisky. – Zildyan May 31 '17 at 16:29
yeah, that would be doable if there was order entry consumption, but for now its not easily possible to do that :( .. maybe get your client to request it from hybris? Sometimes they listen to their clients ;) – Sebastian May 31 '17 at 16:33
I'm afraid they need to go live in a few weeks. I will hack it, and do massive calculations on RuleConfigurationRrdPopulator to stop or allow and calculate multifire promotions, as well as on action side - because there are also all kinds of NotOOTB restrictions - yeah I have great solution architect... he knows how to say only - yes... – Zildyan May 31 '17 at 16:41
1

haha.. oh well.. we've all had those kind of supervisors I guess.. good luck for now then (y) – Sebastian May 31 '17 at 16:56

Multi-firing of BXGY promotion - consumption of qualifying order entries

2 Answers2