Add 'Delete' to ShippingAddress object

[joannasese]

• Add delete function to ShippingAddress object
• Add unit test

To delete a shipping address by id (id = 2922857578120715120 in this example):

$shipping_addresses = Recurly_ShippingAddressList::get($account->account_code);

foreach ($shipping_addresses as $address) {
    if ($address->id == 2922857578120715120){
      $address->delete();
    }
}

To delete all shipping addresses associated with account:

$shipping_addresses = Recurly_ShippingAddressList::get($account->account_code);

foreach ($shipping_addresses as $address) {
    $address->delete();
} 

[bhelx]

I think it's safe to just take the first one?

$shipping_addresses[0]->delete();

If you want to make it more future proof (optional). You can use the shipping address to setup the stub. Something like:

$address = $shipping_addresses[0];
$this->client->addResponse('DELETE', 'https://api.recurly.com/v2/accounts/abcdef1234567890/shipping_addresses/' . $address->id, 'shipping_addresses/destroy-204.xml');

[joannasese]

shipping_addresses[0] yields Cannot use object of type Recurly_ShippingAddressList as array, which may or may have not led to a StackOverflow wormhole. It might explain why there are other instances of foreach being used.

[bhelx]

👍