This topic has been brought up several times in the forums, and mentioned in other issues (#366962: fetch a shipping quote action doesn't work, #459316: Cannot add 'Fetch a shipping quote' action to Flate Rate Conditional Action, #627306: When "Trigger: Getting shipping quote via Flat Rate" it's run?), but essentially the problem is that uc_checkout_pane_quotes() and uc_order_pane_quotes() both use ca_evaluate_conditions() to evaluate the conditionals on the actions associated with shipping quotes instead of using ca_pull_trigger(). This is a bug, because it's an abuse of the conditional actions system -- users rightfully assume that the actions will actually be fired, but they aren't.

What's worse is that the "Fetch a shipping quote" action cannot be added to the "Actions" of any conditional action, despite the status message that it has been added successfully.

What needs to happen is that the code needs to switch over to using ca_pull_trigger(), and the "Fetch a shipping quote" action should stop being hidden from the actions page.

Comments

Volx’s picture

Volx commented

Same is true for uc_taxes and "calculate taxes" trigger. This trigger is actually never pulled.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Closed (works as designed)

This is by design; only the conditions are useful here, it does not make sense to perform an action every time a shipping quote or tax line item is calculated.

Volx’s picture

Volx commented
Status: Closed (works as designed) » Active

You actually mean to say, that it is by design that an existing and publicly available (meaning available to an admin) trigger is never actually being triggered? That makes the CA system completely absurd. It may not make sense to you, but it is exactly what I need to do. This is most definitely a bug, maybe not in code but in design. Moreover it should be an easy fix, just pull the trigger, nothing bad can come from it, if it has never been used before.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Closed (works as designed)

This is not going to change in 6.x now, unless someone else is willing to write and contribute a patch for it.