Provide a shipping method condition for orders

Works great

Working great

There are no tests for this.

This works great thanks. In my case I needed a payment method of cash on delivery but only for one shipping method so I had to create a new custom checkout flow for payment method and moved the billing there.

Updating to align with #3170419: Move all Shipping Conditions to the Shipping Category

The patches are not applying on latest commerce shipping rc2

Rerolled patch to work with the latest version.

#10 worked for me on rc2. Thank you.

Hello,

Thank you, I was looking for this but I'm not sure how to define this.

In my case I have the option to "come and get the order" like a take-away or deliver the order.
In case of take-away we accept cash or online payment
In case of delivery we accept online payment only

What I have done so far:

1. I made the sure the shipping is before the billing pane in the checkout flow
2. in the payment gateway cash, I defined this option is only available in case of shipping method delivery
3. in the payment gateway online, I defined this option is available in both take-away and delivery

After defining this I get the error:
There are no payment gateways available for this order. Please try again later.

When I remove the conditions on the payment gateways both payment options appear.
But online payment remains available although I selected to deliver the order...

Thing is that when switching between the delivery options nothing really happens.
I would expect a refresh or loading of options.
Like when choosing the payment options cash or online.

Any idea what I'm missing?

Kind regards,
Johnny

Unfortunately latest patch no longer works for current version.

Re-roll for the patch.

Added tests and schema fixes.

Removing leftover debug screenshot.

+++ b/src/EventSubscriber/FilterConditionsEventSubscriber.php
@@ -0,0 +1,34 @@
+use Symfony\Component\EventDispatcher\EventSubscriberInterface;
+
+class FilterConditionsEventSubscriber implements EventSubscriberInterface {
+
+  /**

Missing class docblock.

Tests and so on look good to me, so removing that tag.

Rerolled to apply to current dev.
Also added the class docblock.

Retitling for clarity.

order_shipping is too generic as Commerce core provides an "order_shipping_address" condition.

We need to rename it to order_shipping_method.
Additionally, we should change the category to "Shipment" (instead of Shipping, to be consistent with what Commerce shipping is currently doing.

Also, to be consistent with what Commerce is doing in general, the condition needs to store the shipping method UUIDS (similar to the Store & customer conditions for example).

Will upload a patch that addresses these soon.

Better with the condition itself :).

We're probably also missing a unit test for the new condition, similar to what we have for other conditions provided by commerce_shipping.

Went ahead and committed the patch from #24.