Merge pull request #243 from recurly/correct-coupon-listing

Coupon lists require a parent object
recurly · Jul 1, 2016 · 88f948b · 88f948b
2 parents c5892ee + f86c24e
commit 88f948b
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 4 deletions.
diff --git a/lib/recurly/redemption_list.php b/lib/recurly/redemption_list.php
@@ -2,8 +2,18 @@
 
 class Recurly_CouponRedemptionList extends Recurly_Pager
 {
-  public static function get($params = null, $client = null) {
-    $uri = self::_uriWithParams(Recurly_Client::PATH_COUPON_REDEMPTIONS, $params);
+  public static function getForAccount($accountCode, $params = null, $client = null) {
+    $uri = self::_uriWithParams(Recurly_Client::PATH_ACCOUNTS . '/' . rawurlencode($accountCode) . Recurly_Client::PATH_COUPON_REDEMPTIONS, $params);
+    return new self($uri, $client);
+  }
+
+  public static function getForInvoice($invoiceNumber, $params = null, $client = null) {
+    $uri = self::_uriWithParams(Recurly_Client::PATH_INVOICES . '/' . rawurlencode($invoiceNumber) . Recurly_Client::PATH_COUPON_REDEMPTIONS, $params);
+    return new self($uri, $client);
+  }
+
+  public static function getForSubscription($subscriptionUuid, $params = null, $client = null) {
+    $uri = self::_uriWithParams(Recurly_Client::PATH_SUBSCRIPTIONS . '/' . rawurlencode($subscriptionUuid) . Recurly_Client::PATH_COUPON_REDEMPTIONS, $params);
     return new self($uri, $client);
   }
 

diff --git a/lib/recurly/unique_coupon_code_list.php b/lib/recurly/unique_coupon_code_list.php
@@ -3,8 +3,8 @@
 class Recurly_UniqueCouponCodeList extends Recurly_Pager
 {
 
-  public static function get($params = null, $client = null) {
-    $uri = self::_uriWithParams(Recurly_Client::PATH_UNIQUE_COUPONS, $params);
+  public static function get($couponCode, $params = null, $client = null) {
+    $uri = self::_uriWithParams(Recurly_Client::PATH_COUPONS . '/' . rawurlencode($couponCode) . Recurly_Client::PATH_UNIQUE_COUPONS, $params);
     return new self($uri, $client);
   }