fix(vouchers): rewrite convention payments

This commit fixes the voucher tool for convention payments. The previous version downloaded the entirety of the Debtors table and Invoices table. The new version only downloads data that it needs. It also fixes a few usability issues such as sorting by cost.
IMA-WorldHealth · Sep 16, 2017 · 2a92f9b · 2a92f9b
1 parent 8e30ff1
commit 2a92f9b
Show file tree

Hide file tree

Showing 7 changed files with 167 additions and 175 deletions.
diff --git a/client/src/i18n/en/vouchers.json b/client/src/i18n/en/vouchers.json
@@ -29,6 +29,7 @@
     "GLOBAL":{
       "CASHFLOW_OPTION":"Cashflow Option",
       "TRANSFER_DESCRIPTION": "Transfer: From {{ fromAccount }} to {{ toAccount }} for {{ amount }} {{ symbol }}",
+      "CONVENTION_PAYMENT_DESCRIPTION": "Convention Payment: Payment for {{ numInvoices }} invoice(s) by {{ debtorGroupName }}.",
       "CONVENTION_INVOICES":"Convention - Invoices payment",
       "SUPPORT_FORM":"Patients Support - Form",
       "SUPPORT_TEXT":"When the above two fields are filled in, we will look for open debts for the patient to assign to the \"Support Patient\" Account.",

diff --git a/client/src/i18n/fr/vouchers.json b/client/src/i18n/fr/vouchers.json
@@ -1,56 +1,70 @@
-{"VOUCHERS":{"COMPLEX":{"ADD_ITEMS":"Ajouter item(s)",
-"CASH_PAYMENT":"Paiement Caisse",
-"CREATE_SUCCESS":"Transaction postée avec succes",
-"CREATE_ERROR":"Une erreur s'est produite lors du post",
-"CREDITOR":"Crediteur (C)",
-"DEBTOR":"Debiteur (D)",
-"DEB_CRED":"Debtor/Creditor",
-"DESCRIPTION":"Ce module permet de passer des transactions avancees dans le journal",
-"ENTITY":"Entité (Debiteur/Crediteur)",
-"ERROR_ACCOUNT":"Les comptes selectionnés sont invalides",
-"ERROR_AMOUNT":"Les montants sont invalides",
-"ERROR_PRECISION":"Les quantités saisies ne peuvent pas avoir plus de 4 décimales dans la précision",
-"ERROR_TOTALS":"Les totaux ne sont pas balancés",
-"INVALID_VALUES":"Detection des valeurs invalides",
-"PATIENT_INVOICE":"Facture Patient",
-"REFERENCE":"Reference",
-"ROWS":"Ligne(s)",
-"SELECT_DOCUMENT_TYPE":"Veuillez selectionner le type de document",
-"TITLE":"Ajout des transactions",
-"VALID_TOTALS":"Les transactions sont valides (Transactions balancées)",
-"VOUCHER":"Bordereau de Transfert"},
-"TYPES":{"SUPPORT_PAYMENT_DESCRIPTION":"Prise en charge de {{ patientName }} ({{ patientReference}}) pour la facture {{ invoiceReference }}."},
-"GLOBAL":{"CASHFLOW_OPTION":"Option pour le flux de trésorerie physique",
-"CONVENTION_INVOICES":"Convention - Paiement factures",
-"TRANSFER_DESCRIPTION": "Transfert: De {{ fromAccount }} à {{ toAccount }} pour {{ amount }} {{ symbol }}",
-"SUPPORT_FORM":"Prise en Charge des patients",
-"SUPPORT_TEXT":"Lorsque les deux champs ci-dessus sont remplis, Les dettes du patient apparaissent en vue qu'une  \"Prise en Charge\" puissent être attribue à un Compte.",
-"ERROR_SAME_ACCOUNT":"Vous ne pouvez pas utiliser le même compte deux fois dans cette transaction. Veuillez sélectionner deux comptes différents.",
-"GENERIC_EXPENSE":"Depense Generique",
-"GENERIC_INCOME":"Recette Generique",
-"CASH_TRANSFER":"Transfert d'argent",
-"ORIGIN_ACCOUNT":"Compte de provenance",
-"DESTINATION_ACCOUNT":"Compte de destination",
-"TRANSFER_ACCOUNT":"Compte de transfert",
-"IMPORT_TRANSACTION":"Importer les transactions",
-"RECEIPT":{"SUCCESS":"Bon de paiement enregistré avec succès."},
-"REPORT":"Rapport des Quittances",
-"TITLE":"Quittance",
-"TRANSFER_TYPE_INFO":"Type de transaction selon qu'on a soit une recette ou une depense",
-"USE_FINANCIAL_ACCOUNT":"Utilisation de compte de caisses ou banques dans cette transaction"},
-"SIMPLE":{"CASH_PAYMENT":"Paiement des clients",
-"CASH_RETURN":"Remboursement",
-"CONVENTION_PAYMENT":"Paiement des Conventions",
-"CREDIT_NOTE":"Note de crédit",
-"EXPENSE":"Dépenses",
-"INCOME":"Récettes",
-"INVOICING":"Facturation",
-"SALARY_PAYMENT":"Paiement des Salaires",
-"GENERIC_EXPENSE":"Autres Depenses",
-"GENERIC_INCOME":"Autres Recettes",
-"PURCHASES":"Achats",
-"SUPPORT_INCOME":"Paiement Prise en charge",
-"TITLE":"Bordereau de transfert",
-"TRANSFER":"Transfert d'argent",
-"TRANSFER_PATIENT_INVOICE_AMOUNT":"Prise en Charge de Facture de Patient"},
-"REPORT":"Rapport"}}
+{
+  "VOUCHERS":{
+    "COMPLEX":{
+      "ADD_ITEMS":"Ajouter item(s)",
+      "CASH_PAYMENT":"Paiement Caisse",
+      "CREATE_SUCCESS":"Transaction postée avec succes",
+      "CREATE_ERROR":"Une erreur s'est produite lors du post",
+      "CREDITOR":"Crediteur (C)",
+      "DEBTOR":"Debiteur (D)",
+      "DEB_CRED":"Debtor/Creditor",
+      "DESCRIPTION":"Ce module permet de passer des transactions avancees dans le journal",
+      "ENTITY":"Entité (Debiteur/Crediteur)",
+      "ERROR_ACCOUNT":"Les comptes selectionnés sont invalides",
+      "ERROR_AMOUNT":"Les montants sont invalides",
+      "ERROR_PRECISION":"Les quantités saisies ne peuvent pas avoir plus de 4 décimales dans la précision",
+      "ERROR_TOTALS":"Les totaux ne sont pas balancés",
+      "INVALID_VALUES":"Detection des valeurs invalides",
+      "PATIENT_INVOICE":"Facture Patient",
+      "REFERENCE":"Reference",
+      "ROWS":"Ligne(s)",
+      "SELECT_DOCUMENT_TYPE":"Veuillez selectionner le type de document",
+      "TITLE":"Ajout des transactions",
+      "VALID_TOTALS":"Les transactions sont valides (Transactions balancées)",
+      "VOUCHER":"Bordereau de Transfert"
+    },
+    "TYPES":{
+      "SUPPORT_PAYMENT_DESCRIPTION":"Prise en charge de {{ patientName }} ({{ patientReference}}) pour la facture {{ invoiceReference }}."
+    },
+    "GLOBAL":{
+      "CASHFLOW_OPTION":"Option pour le flux de trésorerie physique",
+      "CONVENTION_INVOICES":"Convention - Paiement factures",
+      "CONVENTION_PAYMENT_DESCRIPTION": "Convention Paiement Factures: Paiement pour {{ numInvoices }} factures(s) par {{ debtorGroupName }}.",
+      "TRANSFER_DESCRIPTION": "Transfert: De {{ fromAccount }} à {{ toAccount }} pour {{ amount }} {{ symbol }}",
+      "SUPPORT_FORM":"Prise en Charge des patients",
+      "SUPPORT_TEXT":"Lorsque les deux champs ci-dessus sont remplis, Les dettes du patient apparaissent en vue qu'une  \"Prise en Charge\" puissent être attribue à un Compte.",
+      "ERROR_SAME_ACCOUNT":"Vous ne pouvez pas utiliser le même compte deux fois dans cette transaction. Veuillez sélectionner deux comptes différents.",
+      "GENERIC_EXPENSE":"Depense Generique",
+      "GENERIC_INCOME":"Recette Generique",
+      "CASH_TRANSFER":"Transfert d'argent",
+      "ORIGIN_ACCOUNT":"Compte de provenance",
+      "DESTINATION_ACCOUNT":"Compte de destination",
+      "TRANSFER_ACCOUNT":"Compte de transfert",
+      "IMPORT_TRANSACTION":"Importer les transactions",
+      "RECEIPT":{
+        "SUCCESS":"Bon de paiement enregistré avec succès."
+      },
+      "REPORT":"Rapport des Quittances",
+      "TITLE":"Quittance",
+      "TRANSFER_TYPE_INFO":"Type de transaction selon qu'on a soit une recette ou une depense",
+      "USE_FINANCIAL_ACCOUNT":"Utilisation de compte de caisses ou banques dans cette transaction"
+    },
+    "SIMPLE":{"CASH_PAYMENT":"Paiement des clients",
+    "CASH_RETURN":"Remboursement",
+    "CONVENTION_PAYMENT":"Paiement des Conventions",
+    "CREDIT_NOTE":"Note de crédit",
+    "EXPENSE":"Dépenses",
+    "INCOME":"Récettes",
+    "INVOICING":"Facturation",
+    "SALARY_PAYMENT":"Paiement des Salaires",
+    "GENERIC_EXPENSE":"Autres Depenses",
+    "GENERIC_INCOME":"Autres Recettes",
+    "PURCHASES":"Achats",
+    "SUPPORT_INCOME":"Paiement Prise en charge",
+    "TITLE":"Bordereau de transfert",
+    "TRANSFER":"Transfert d'argent",
+    "TRANSFER_PATIENT_INVOICE_AMOUNT":"Prise en Charge de Facture de Patient"
+    },
+    "REPORT":"Rapport"
+  }
+}
diff --git a/client/src/modules/debtors/groups.service.js b/client/src/modules/debtors/groups.service.js
@@ -1,5 +1,5 @@
 angular.module('bhima.services')
-.service('DebtorGroupService', DebtorGroupService);
+  .service('DebtorGroupService', DebtorGroupService);
 
 DebtorGroupService.$inject = ['$http', '$uibModal', 'util', 'SessionService'];
 
@@ -64,17 +64,17 @@ function DebtorGroupService($http, Modal, util, SessionService) {
     delete debtorGroup.uuid;
 
     return $http.put(url, debtorGroup)
-    .then(util.unwrapHttpResponse);
+      .then(util.unwrapHttpResponse);
   }
 
   /**
   * @method remove
   * @param {string} uuid The debtor group uuid
   * @description This function is responsible for deleting a debtor group
-  */  
+  */
   function remove(uuid) {
     return $http.delete(baseUrl.concat(uuid))
-    .then(util.unwrapHttpResponse);
+      .then(util.unwrapHttpResponse);
   }
 
 
@@ -109,13 +109,9 @@ function DebtorGroupService($http, Modal, util, SessionService) {
       controller : 'BillingServiceSubscriptions as SubCtrl',
       size : 'md',
       resolve : {
-        Subscriptions : function Subscriptions() {
-          return subscriptions;
-        },
-        DebtorGroup : function DebtorGroup() {
-          return debtorGroup;
-        }
-      }
+        Subscriptions : function Subscriptions() { return subscriptions; },
+        DebtorGroup : function DebtorGroup() { return debtorGroup; },
+      },
     });
   }
 
@@ -125,13 +121,9 @@ function DebtorGroupService($http, Modal, util, SessionService) {
       controller : 'SubsidySubscriptions as SubCtrl',
       size : 'md',
       resolve : {
-        Subscriptions : function Subscriptions() {
-          return subscriptions;
-        },
-        DebtorGroup : function DebtorGroup() {
-          return debtorGroup;
-        }
-      }
+        Subscriptions : function Subscriptions() { return subscriptions; },
+        DebtorGroup : function DebtorGroup() { return debtorGroup; },
+      },
     });
   }
 

diff --git a/client/src/modules/vouchers/toolkit/cash_transfer/cash_transfer.html b/client/src/modules/vouchers/toolkit/cash_transfer/cash_transfer.html
@@ -40,8 +40,8 @@
     <div
       class="form-group"
       ng-class="{ 'has-error' : ToolForm.$submitted && ToolForm.account.$invalid }">
-      <label class="control-label"  translate>VOUCHERS.GLOBAL.TRANSFER_ACCOUNT</label>
-      (<label class="control-label"  translate>FORM.LABELS.CREDIT</label>)
+      <label class="control-label" translate>VOUCHERS.GLOBAL.TRANSFER_ACCOUNT</label>
+      (<label class="control-label" translate>FORM.LABELS.CREDIT</label>)
 
       <!-- input typeahead -->
       <input