Fixed an issue with adding a customer (2600hz#1878)

* Fixed an issue with adding a customer

When someone needed to add a customer without having their credit card braintree send an error that make_default(undefined) clause is not defined in braintree_card.erl. now we can add a new customer in braintree by just having using his info and adding a card is not needed.

* Updated specs and changed undefined to catch all method

* fixed braintree for use case where credit card info is not present

* changed get_binary_value to get_value

core/braintree/src/braintree_card.erl

@@ -299,7 +299,6 @@ record_to_xml(#bt_card{}=Card, ToString) ->
 %% @end
 %%--------------------------------------------------------------------
 -spec json_to_record(api_object()) -> bt_card().
-json_to_record('undefined') -> 'undefined';
 json_to_record(JObj) ->
     #bt_card{token = create_or_get_json_id(JObj)
              ,cardholder_name = kz_json:get_binary_value(<<"cardholder_name">>, JObj)

core/braintree/src/braintree_customer.erl

@@ -346,9 +346,16 @@ json_to_record(JObj) ->
                  ,phone = kz_json:get_binary_value(<<"phone">>, JObj)
                  ,fax = kz_json:get_binary_value(<<"fax">>, JObj)
                  ,website = kz_json:get_binary_value(<<"website">>, JObj)
-                 ,credit_cards = [braintree_card:json_to_record(kz_json:get_value(<<"credit_card">>, JObj))]
+                 ,credit_cards = maybe_add_credit_card(JObj)
                 }.
 
+-spec maybe_add_credit_card(api_object()) -> bt_cards().
+maybe_add_credit_card(JObj) ->
+  case kz_json:get_value(<<"credit_card">>, JObj) of
+      'undefined' -> [];
+      Card -> [braintree_card:json_to_record(Card)]
+  end.
+
 %%--------------------------------------------------------------------
 %% @public
 %% @doc