KAZOO-1770:Correct if transaction = 0

2600hz · Jan 21, 2014 · 4f60b92 · 4f60b92
1 parent d8d48e6
commit 4f60b92
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 7 deletions.
diff --git a/applications/crossbar/src/modules_v2/cb_devices_v2.erl b/applications/crossbar/src/modules_v2/cb_devices_v2.erl
@@ -213,11 +213,15 @@ dry_run(#cb_context{account_id=AccountId, req_data=JObj}) ->
     UpdateServices = wh_service_devices:reconcile(Services, DeviceType),
 
     Charges = wh_services:activation_charges(<<"devices">>, DeviceType, Services),
-    Transaction = wh_transaction:debit(AccountId, wht_util:dollars_to_units(Charges)),
-    Desc = <<"activation charges for ", DeviceType/binary , " ", DeviceName/binary>>,
-    Transaction2 = wh_transaction:set_description(Desc, Transaction),
 
-    wh_services:calulate_charges(UpdateServices, [Transaction2]).
+    case Charges > 0 of
+        'false' -> wh_services:calulate_charges(UpdateServices, []);
+        'true' ->
+            Transaction = wh_transaction:debit(AccountId, wht_util:dollars_to_units(Charges)),
+            Desc = <<"activation charges for ", DeviceType/binary , " ", DeviceName/binary>>,
+            Transaction2 = wh_transaction:set_description(Desc, Transaction),
+            wh_services:calulate_charges(UpdateServices, [Transaction2])
+    end.
 
 
 

diff --git a/applications/crossbar/src/modules_v2/cb_users_v2.erl b/applications/crossbar/src/modules_v2/cb_users_v2.erl
@@ -175,9 +175,15 @@ dry_run(#cb_context{account_id=AccountId, req_data=JObj}) ->
     UpdateServices = wh_service_users:reconcile(Services, UserType),
 
     Charges = wh_services:activation_charges(<<"devices">>, UserType, Services),
-    Transaction = wh_transaction:debit(AccountId, wht_util:dollars_to_units(Charges)),
-    Desc = <<"activation charges for ", UserType/binary , " ", UserName/binary>>,
-    Transaction2 = wh_transaction:set_description(Desc, Transaction),
+
+    case Charges > 0 of
+        'false' -> wh_services:calulate_charges(UpdateServices, []);
+        'true' ->
+            Transaction = wh_transaction:debit(AccountId, wht_util:dollars_to_units(Charges)),
+            Desc = <<"activation charges for ", UserType/binary , " ", UserName/binary>>,
+            Transaction2 = wh_transaction:set_description(Desc, Transaction),
+            wh_services:calulate_charges(UpdateServices, [Transaction2])
+    end.
 
     wh_services:calulate_charges(UpdateServices, [Transaction2]).