Bug in "UpdateTheSubscriptionCreditCard" method

kfrancis / chargify-dot-net

MIT License

20 stars 49 forks source link

Bug in "UpdateTheSubscriptionCreditCard" method #2

Closed crispwaters closed 9 years ago

crispwaters commented 9 years ago

In the private method for updating the subscription credit card, the CVV's length is checked if the CVV is required in Chargify. HOWEVER, it is possible to pass in a null value for the CVV if that value should not be updated. Because there is no check to see if the CVV is null, the call to this method throws a null object reference exception. This bug makes it impossible to update just the billing address from the library.

kfrancis commented 9 years ago

Currently there's a property of ChargifyConnect CvvRequired. Set CvvRequired = false for those cases for now.