Rule Condition: Check if order has an specific coupon.

+++ b/commerce_coupon.rules.inc
@@ -47,6 +47,35 @@ function commerce_coupon_rules_event_info() {
+ * Implements hook_rules_condition_info() ¶

Please fix the extra spaces

+++ b/commerce_coupon.rules.inc
@@ -47,6 +47,35 @@ function commerce_coupon_rules_event_info() {
+    'label' => t('Order has a specific coupon'),

Wording improvement suggestion: "Coupon has been applied to an order"

+++ b/commerce_coupon.rules.inc
@@ -47,6 +47,35 @@ function commerce_coupon_rules_event_info() {
+	'description' => 'The order to be checked for the coupon.'

Missing t()

+++ b/commerce_coupon.rules.inc
@@ -232,3 +261,13 @@ function commerce_coupon_action_remove_all_coupons_from_order($order) {
+  if (empty($order)) return true;

Not following drupal standards for ifs

+++ b/commerce_coupon.rules.inc
@@ -232,3 +261,13 @@ function commerce_coupon_action_remove_all_coupons_from_order($order) {
+  if (empty($coupon)) return true;

Same here

+++ b/commerce_coupon.rules.inc
@@ -232,3 +261,13 @@ function commerce_coupon_action_remove_all_coupons_from_order($order) {
+  return in_array($coupon->coupon_id, $order_wrapper->commerce_coupon_order_reference->raw());

Coupons may be disabled, status = 0, so we may want to create an api function with uses entity field query for this

Thanks for the patch

if (empty($order)) return true; What's the logic behind that? Shouldn't we return FALSE if either of the parameters is invalid?

Also be good to specify whether the coupon identifier is the coupon string or the raw ID.

I think this patch should be recreated for use wtin 2.x branch, also fixing the problems that @pcambra already said.

ok I do not understand why is necesarry this part in the patch

+/**
  * Implements hook_rules_action_info().
  */
 function commerce_coupon_rules_action_info() {
@@ -232,3 +261,13 @@ function commerce_coupon_action_remove_all_coupons_from_order($order) {
   }
   commerce_coupon_remove_all_coupons_from_order($order);
 }

Should be removed

and regarding the comment of @aindalis, yes it would be better idea to specify that the condition checks for the coupon raw id and not the coupon code itself, but would be better idea to change to be the "coupon code" because is more friendly for the users...