Remove deprecated methods

stripe · Aug 9, 2019 · 4c70d33 · 4c70d33
1 parent fda9f86
commit 4c70d33
Show file tree

Hide file tree

Showing 14 changed files with 1 addition and 461 deletions.
diff --git a/lib/ApplicationFee.php b/lib/ApplicationFee.php
@@ -32,19 +32,6 @@ class ApplicationFee extends ApiResource
 
     const PATH_REFUNDS = '/refunds';
 
-    /**
-     * @param array|null $params
-     * @param array|string|null $opts
-     *
-     * @return ApplicationFee The refunded application fee.
-     */
-    public function refund($params = null, $opts = null)
-    {
-        $this->refunds->create($params, $opts);
-        $this->refresh();
-        return $this;
-    }
-
     /**
      * @param string|null $id The ID of the application fee on which to create the refund.
      * @param array|null $params

diff --git a/lib/Charge.php b/lib/Charge.php
@@ -117,20 +117,6 @@ class Charge extends ApiResource
     const STATUS_PENDING   = 'pending';
     const STATUS_SUCCEEDED = 'succeeded';
 
-    /**
-     * @param array|null $params
-     * @param array|string|null $options
-     *
-     * @return Charge The refunded charge.
-     */
-    public function refund($params = null, $options = null)
-    {
-        $url = $this->instanceUrl() . '/refund';
-        list($response, $opts) = $this->_request('post', $url, $params, $options);
-        $this->refreshFrom($response, $opts);
-        return $this;
-    }
-
     /**
      * @param array|null $params
      * @param array|string|null $options
@@ -144,63 +130,4 @@ public function capture($params = null, $options = null)
         $this->refreshFrom($response, $opts);
         return $this;
     }
-
-    /**
-     * @param array|null $params
-     * @param array|string|null $options
-     *
-     * @deprecated Use the `save` method on the Dispute object
-     *
-     * @return array The updated dispute.
-     */
-    public function updateDispute($params = null, $options = null)
-    {
-        $url = $this->instanceUrl() . '/dispute';
-        list($response, $opts) = $this->_request('post', $url, $params, $options);
-        $this->refreshFrom(['dispute' => $response], $opts, true);
-        return $this->dispute;
-    }
-
-    /**
-     * @param array|string|null $options
-     *
-     * @deprecated Use the `close` method on the Dispute object
-     *
-     * @return Charge The updated charge.
-     */
-    public function closeDispute($options = null)
-    {
-        $url = $this->instanceUrl() . '/dispute/close';
-        list($response, $opts) = $this->_request('post', $url, null, $options);
-        $this->refreshFrom($response, $opts);
-        return $this;
-    }
-
-    /**
-     * @param array|string|null $opts
-     *
-     * @return Charge The updated charge.
-     */
-    public function markAsFraudulent($opts = null)
-    {
-        $params = ['fraud_details' => ['user_report' => 'fraudulent']];
-        $url = $this->instanceUrl();
-        list($response, $opts) = $this->_request('post', $url, $params, $opts);
-        $this->refreshFrom($response, $opts);
-        return $this;
-    }
-
-    /**
-     * @param array|string|null $opts
-     *
-     * @return Charge The updated charge.
-     */
-    public function markAsSafe($opts = null)
-    {
-        $params = ['fraud_details' => ['user_report' => 'safe']];
-        $url = $this->instanceUrl();
-        list($response, $opts) = $this->_request('post', $url, $params, $opts);
-        $this->refreshFrom($response, $opts);
-        return $this;
-    }
 }
diff --git a/lib/Customer.php b/lib/Customer.php
@@ -65,84 +65,6 @@ public static function getSavedNestedResources()
     const PATH_SOURCES = '/sources';
     const PATH_TAX_IDS = '/tax_ids';
 
-    /**
-     * @param array|null $params
-     *
-     * @return InvoiceItem The resulting invoice item.
-     */
-    public function addInvoiceItem($params = null)
-    {
-        $params = $params ?: [];
-        $params['customer'] = $this->id;
-        $ii = InvoiceItem::create($params, $this->_opts);
-        return $ii;
-    }
-
-    /**
-     * @param array|null $params
-     *
-     * @return array An array of the customer's Invoices.
-     */
-    public function invoices($params = null)
-    {
-        $params = $params ?: [];
-        $params['customer'] = $this->id;
-        $invoices = Invoice::all($params, $this->_opts);
-        return $invoices;
-    }
-
-    /**
-     * @param array|null $params
-     *
-     * @return array An array of the customer's InvoiceItems.
-     */
-    public function invoiceItems($params = null)
-    {
-        $params = $params ?: [];
-        $params['customer'] = $this->id;
-        $iis = InvoiceItem::all($params, $this->_opts);
-        return $iis;
-    }
-
-    /**
-     * @param array|null $params
-     *
-     * @return array An array of the customer's Charges.
-     */
-    public function charges($params = null)
-    {
-        $params = $params ?: [];
-        $params['customer'] = $this->id;
-        $charges = Charge::all($params, $this->_opts);
-        return $charges;
-    }
-
-    /**
-     * @param array|null $params
-     *
-     * @return Subscription The updated subscription.
-     */
-    public function updateSubscription($params = null)
-    {
-        $url = $this->instanceUrl() . '/subscription';
-        list($response, $opts) = $this->_request('post', $url, $params);
-        $this->refreshFrom(['subscription' => $response], $opts, true);
-        return $this->subscription;
-    }
-
-    /**
-     * @param array|null $params
-     *
-     * @return Subscription The cancelled subscription.
-     */
-    public function cancelSubscription($params = null)
-    {
-        $url = $this->instanceUrl() . '/subscription';
-        list($response, $opts) = $this->_request('delete', $url, $params);
-        $this->refreshFrom(['subscription' => $response], $opts, true);
-        return $this->subscription;
-    }
-
     /**
      * @return Customer The updated customer.
      */

diff --git a/lib/Recipient.php b/lib/Recipient.php
@@ -31,17 +31,4 @@ class Recipient extends ApiResource
     use ApiOperations\Delete;
     use ApiOperations\Retrieve;
     use ApiOperations\Update;
-
-    /**
-     * @param array|null $params
-     *
-     * @return Collection of the Recipient's Transfers
-     */
-    public function transfers($params = null)
-    {
-        $params = $params ?: [];
-        $params['recipient'] = $this->id;
-        $transfers = Transfer::all($params, $this->_opts);
-        return $transfers;
-    }
 }
diff --git a/lib/Source.php b/lib/Source.php
@@ -109,19 +109,6 @@ public function detach($params = null, $options = null)
         }
     }
 
-    /**
-     * @param array|null $params
-     * @param array|string|null $options
-     *
-     * @return Source The detached source.
-     *
-     * @deprecated Use the `detach` method instead.
-     */
-    public function delete($params = null, $options = null)
-    {
-        $this->detach($params, $options);
-    }
-
     /**
      * @param array|null $params
      * @param array|string|null $options

diff --git a/lib/Transfer.php b/lib/Transfer.php
@@ -46,17 +46,6 @@ class Transfer extends ApiResource
     const SOURCE_TYPE_CARD           = 'card';
     const SOURCE_TYPE_FINANCING      = 'financing';
 
-    /**
-     * @return TransferReversal The created transfer reversal.
-     */
-    public function reverse($params = null, $opts = null)
-    {
-        $url = $this->instanceUrl() . '/reversals';
-        list($response, $opts) = $this->_request('post', $url, $params, $opts);
-        $this->refreshFrom($response, $opts);
-        return $this;
-    }
-
     /**
      * @return Transfer The canceled transfer.
      */

diff --git a/lib/UsageRecord.php b/lib/UsageRecord.php
@@ -17,27 +17,4 @@
 class UsageRecord extends ApiResource
 {
     const OBJECT_NAME = "usage_record";
-
-    /**
-     * @param array|null $params
-     * @param array|string|null $options
-     *
-     * @return \Stripe\ApiResource The created resource.
-     */
-    public static function create($params = null, $options = null)
-    {
-        self::_validateParams($params);
-        if (!array_key_exists('subscription_item', $params)) {
-            throw new Error\InvalidRequest("Missing subscription_item param in request", null);
-        }
-        $subscription_item = $params['subscription_item'];
-        $url = "/v1/subscription_items/$subscription_item/usage_records";
-        $request_params = $params;
-        unset($request_params['subscription_item']);
-
-        list($response, $opts) = static::_staticRequest('post', $url, $request_params, $options);
-        $obj = \Stripe\Util\Util::convertToStripeObject($response->json, $opts);
-        $obj->setLastResponse($response);
-        return $obj;
-    }
 }
diff --git a/tests/Stripe/ApplicationFeeTest.php b/tests/Stripe/ApplicationFeeTest.php
@@ -28,18 +28,6 @@ public function testIsRetrievable()
         $this->assertInstanceOf(\Stripe\ApplicationFee::class, $resource);
     }
 
-    public function testIsRefundable()
-    {
-        $fee = ApplicationFee::retrieve(self::TEST_RESOURCE_ID);
-        $this->expectsRequest(
-            'post',
-            '/v1/application_fees/' . $fee->id . '/refunds'
-        );
-        $resource = $fee->refund();
-        $this->assertInstanceOf(\Stripe\ApplicationFee::class, $resource);
-        $this->assertSame($resource, $fee);
-    }
-
     public function testCanCreateRefund()
     {
         $this->expectsRequest(

diff --git a/tests/Stripe/ChargeTest.php b/tests/Stripe/ChargeTest.php
@@ -65,18 +65,6 @@ public function testIsUpdatable()
         $this->assertInstanceOf(\Stripe\Charge::class, $resource);
     }
 
-    public function testCanRefund()
-    {
-        $charge = Charge::retrieve(self::TEST_RESOURCE_ID);
-        $this->expectsRequest(
-            'post',
-            '/v1/charges/' . $charge->id . '/refund'
-        );
-        $resource = $charge->refund();
-        $this->assertInstanceOf(\Stripe\Charge::class, $resource);
-        $this->assertSame($resource, $charge);
-    }
-
     public function testCanCapture()
     {
         $charge = Charge::retrieve(self::TEST_RESOURCE_ID);
@@ -88,53 +76,4 @@ public function testCanCapture()
         $this->assertInstanceOf(\Stripe\Charge::class, $resource);
         $this->assertSame($resource, $charge);
     }
-
-    public function testCanUpdateDispute()
-    {
-        $charge = Charge::retrieve(self::TEST_RESOURCE_ID);
-        $this->expectsRequest(
-            'post',
-            '/v1/charges/' . $charge->id . '/dispute'
-        );
-        $resource = $charge->updateDispute();
-        $this->assertInstanceOf(\Stripe\Dispute::class, $resource);
-    }
-
-    public function testCanCloseDispute()
-    {
-        $charge = Charge::retrieve(self::TEST_RESOURCE_ID);
-        $this->expectsRequest(
-            'post',
-            '/v1/charges/' . $charge->id . '/dispute/close'
-        );
-        $resource = $charge->closeDispute();
-        $this->assertInstanceOf(\Stripe\Charge::class, $resource);
-        $this->assertSame($resource, $charge);
-    }
-
-    public function testCanMarkAsFraudulent()
-    {
-        $charge = Charge::retrieve(self::TEST_RESOURCE_ID);
-        $this->expectsRequest(
-            'post',
-            '/v1/charges/' . $charge->id,
-            ['fraud_details' => ['user_report' => 'fraudulent']]
-        );
-        $resource = $charge->markAsFraudulent();
-        $this->assertInstanceOf(\Stripe\Charge::class, $resource);
-        $this->assertSame($resource, $charge);
-    }
-
-    public function testCanMarkAsSafe()
-    {
-        $charge = Charge::retrieve(self::TEST_RESOURCE_ID);
-        $this->expectsRequest(
-            'post',
-            '/v1/charges/' . $charge->id,
-            ['fraud_details' => ['user_report' => 'safe']]
-        );
-        $resource = $charge->markAsSafe();
-        $this->assertInstanceOf(\Stripe\Charge::class, $resource);
-        $this->assertSame($resource, $charge);
-    }
 }