From 6768e62d481bb3062752ac60f2a82b2ec92e1de7 Mon Sep 17 00:00:00 2001
From: efernandez <efernandez@focasoftware.com>
Date: Wed, 16 Jan 2019 18:21:08 -0300
Subject: [PATCH] refactor

---
 src/js/controller.js                   |  63 ++++++++------
 src/views/modal-detalle-hoja-ruta.html | 147 +++++++++++----------------------
 2 files changed, 85 insertions(+), 125 deletions(-)

diff --git a/src/js/controller.js b/src/js/controller.js
index e30717b..cb8b001 100644
--- a/src/js/controller.js
+++ b/src/js/controller.js
@@ -4,39 +4,54 @@ angular.module('focaModalDetalleHojaRuta')
             '$filter',
             '$scope',
             '$uibModalInstance',
-            'parametrosDetalleHojaRuta',
-            function($filter, $scope, $uibModalInstance, parametrosDetalleHojaRuta) {
-                $scope.paso = 0;
-                $scope.parametrosDetalleHojaRuta = parametrosDetalleHojaRuta;
+            'remito',
+            function($filter, $scope, $uibModalInstance, remito) {
+                $scope.remito = remito;
                 $scope.aceptar = function() {
-                    var parametrosDetalleHojaRuta = {
-                        remito: $scope.parametrosDetalleHojaRuta.remito,
-                        cliente: $scope.parametrosDetalleHojaRuta.cliente,
-                        domicilio: $scope.parametrosDetalleHojaRuta.domicilio,
-                        producto: $scope.parametrosDetalleHojaRuta.producto,
-                        litros: $scope.parametrosDetalleHojaRuta.litros,
-                        litrosDescargados: $scope.parametrosDetalleHojaRuta.litrosDescargados,
-                        numeroRecibo: $scope.parametrosDetalleHojaRuta.numeroRecibo,
-                        observaciones: $scope.parametrosDetalleHojaRuta.observaciones
-                    };
-                    $uibModalInstance.close(parametrosDetalleHojaRuta);
+                    var articulos = articulosDescargados();
+                    var cisternaMovimientos = [];
+                    var hojaRutaMovimientos = [];
+                    for(var i = 0; i < articulos.length; i++) {
+                        var cisternaMovimiento = {
+                            idRemito: $scope.remito.id,
+                            cantidad: articulos[i].aCargar,
+                            metodo: 'descarga'
+                        }
+                        var hojaRutaMovimiento = {
+                            idRemito: $scope.remito.id,
+                            reciboDescarga: articulos[i].numeroRecibo
+                        }
+                        cisternaMovimientos.push(cisternaMovimiento);
+                        hojaRutaMovimientos.push(hojaRutaMovimiento);
+                    }
+                    var result = {
+                        cisternaMovimientos: cisternaMovimientos,
+                        hojaRutaMovimientos: hojaRutaMovimientos,
+                        idHojaRuta: $scope.remito.idHojaRuta
+                    }
+                    $uibModalInstance.close(result);
                 };
 
                 $scope.cancel = function() {
                     $uibModalInstance.dismiss('cancel');
                 };
-                
+
                 $scope.validar = function() {
-                    return $scope.formDetalleHojaRuta.$pristine ||
-                        !$scope.parametrosDetalleHojaRuta.litrosDescargados ||
-                        $scope.parametrosDetalleHojaRuta.litrosDescargados < 1 ||
-                        !$scope.parametrosDetalleHojaRuta.numeroRecibo ||
-                        $scope.parametrosDetalleHojaRuta.numeroRecibo < 1;
+                    if(!articulosDescargados().length) {
+                        return true;
+                    }
+                    return false;
                 };
 
-                $scope.irPaso = function(paso) {
-                    $scope.paso = paso;
-                };
+                function articulosDescargados() {
+                    var articulosDescargados = $scope.remito.articulosRemito.filter(
+                        function(articulo) {
+                            if(articulo.aCargar && articulo.numeroRecibo) {
+                                return articulo;
+                            }
+                        });
+                    return articulosDescargados;
+                }
             }
         ]
     );
diff --git a/src/views/modal-detalle-hoja-ruta.html b/src/views/modal-detalle-hoja-ruta.html
index 1276067..d8425ad 100644
--- a/src/views/modal-detalle-hoja-ruta.html
+++ b/src/views/modal-detalle-hoja-ruta.html
@@ -1,120 +1,65 @@
 <div class="modal-header py-1">
-    <h5 class="modal-title">Detalle de hoja de ruta</h5>
+    <h5 class="modal-title">Detalle de descarga</h5>
 </div>
 <div class="modal-body" id="modal-body">
     <form name="formDetalleHojaRuta">
         <div class="form-group row">
-            <div class="col-6 px-2">
-                <label class="col-12 col-sm-6 px-0 mb-0">
-                    <b>Remito</b>
+            <div class="col-12 px-0">
+                <label class="form-control-sm">
+                    <b>Remito Nº</b>
+                    <span ng-bind="[remito.numeroRemito, remito.sucursal] | comprobante"/>
                 </label>
-                <div class="col-12 col-sm-6 px-0">
-                    <span ng-bind="parametrosDetalleHojaRuta.remito"/>
-                </div>
             </div>
-            <div class="col-6 px-2">
-                <label class="col-12 col-sm-6 px-0 mb-0">
+            <div class="col-12 px-0">
+                <label class="form-control-sm">
                     <b>Cliente</b>
                 </label>
-                <div class="col-12 col-sm-6 px-0">
-                    <span ng-bind="parametrosDetalleHojaRuta.cliente"/>
-                </div>
+                <span ng-bind="remito.nombreCliente"/>
             </div>
-            <div class="col-6 px-2">
-                <label class="col-12 col-sm-6 px-0 mb-0">
+            <div class="col-12 px-0">
+                <label class="form-control-sm">
                     <b>Domicilio</b>
+                    <span ng-bind="remito.domicilioStamp"/>
                 </label>
-                <div class="col-12 col-sm-6 px-0">
-                    <span ng-bind="parametrosDetalleHojaRuta.domicilio"/>
-                </div>
             </div>
-            <div class="col-6 px-2">
-                <label class="col-12 col-sm-6 px-0 mb-0">
-                    <b>Contacto</b>
-                </label>
-                <div class="col-12 col-sm-6 px-0">
-                    <span ng-bind="parametrosDetalleHojaRuta.contacto"/>
-                </div>
-            </div>
-            <div class="col-6 px-2">
-                <label class="col-12 col-sm-6 px-0 mb-0">
-                    <b>Teléfono</b>
-                </label>
-                <div class="col-12 col-sm-6 px-0">
-                    <span ng-bind="parametrosDetalleHojaRuta.telefonoContacto"/>
-                </div>
+            <div class="col-12 px-0">
+                <table class="table table-sm">
+                    <thead>
+                        <tr>
+                            <th>Articulo</th>
+                            <th>Total</th>
+                            <th>A descargar</th>
+                            <th>Nº Recibo</th>
+                        </tr>
+                    </thead>
+                    <tbody>
+                        <tr ng-repeat="(key, articulo) in remito.articulosRemito">
+                            <td ng-bind="articulo.descripcion"></td>
+                            <td ng-bind="articulo.cantidad"></td>
+                            <td><input
+                                class="form-control form-control-sm"
+                                type="number"
+                                ng-model="articulo.aCargar"
+                                foca-focus="key === 0"/></td>
+                            <td><input
+                                class="form-control form-control-sm"
+                                type="number"
+                                ng-model="articulo.numeroRecibo"/></td>
+                            </tr>
+                    </tbody>
+                </table>
             </div>
-            <div class="col-6 px-2">
-                <label class="col-12 col-sm-6 px-0 mb-0">
-                    <b>Producto</b>
+            <div class="col-12 px-0 py-0">
+                <label class="form-control-sm">
+                    <b>Observaciones</b>
                 </label>
-                <div class="col-12 col-sm-6 px-0">
-                    <span ng-bind="parametrosDetalleHojaRuta.producto"/>
-                </div>
-            </div>
-            <div class="col-6 px-2">
-                <label class="col-12 col-sm-6 px-0 mb-0">
-                    <b>Litros</b>
-                </label>
-                <div class="col-12 col-sm-6 px-0">
-                    <span ng-bind="parametrosDetalleHojaRuta.litros"/>
-                </div>
-            </div>
-            <div class="w-100"></div>
-            <div class="form-group col-6 px-0">
-                <label 
-                    class="col-12 col-sm-3 col-form-label col-form-label-sm px-2"
-                >Litros descargados</label>
-                <div class="col-12 col-sm-3 px-2">
-                    <input
-                        type="number"
-                        min="0"
-                        step="1"
-                        class="form-control form-control-sm"
-                        id="litrosDescargados"
-                        name="litrosDescargados"
-                        placeholder="Litros descargados"
-                        ng-model="parametrosDetalleHojaRuta.litrosDescargados"
-                        foca-focus="paso === 0"
-                        ng-keypress="$event.keyCode === 13 && irPaso(1)"
-                        foca-tipo-input
-                        teclado-virtual
-                    />
-                </div>
-            </div>
-            <div class="form-group col-6 px-0">
-                <label 
-                    class="col-12 col-sm-3 col-form-label col-form-label-sm px-2"
-                >Nº recibo</label>
-                <div class="col-12 col-sm-33 px-2">
-                    <input
-                        type="number"
-                        min="0"
-                        step="0.01"
-                        class="form-control form-control-sm"
-                        id="numeroRecibo"
-                        name="numeroRecibo"
-                        placeholder="Nº Recibo"
-                        ng-model="parametrosDetalleHojaRuta.numeroRecibo"
-                        foca-focus="paso === 1"
-                        ng-keypress="$event.keyCode == 13 && validar() && aceptar()"
-                        foca-tipo-input
-                        teclado-virtual
-                    />
-                </div>
             </div>
-            <div class="row">
-                <div class="form-group col-12 pr-0">
-                    <label 
-                        class="col-12 col-sm-3 col-form-label col-form-label-sm px-2"
-                    >Observaciones</label>
-                    <textarea
-                        ng-model="parametrosDetalleHojaRuta.observaciones"
-                        class="form-control mx-2"
-                        rows="5"
-                        teclado-virtual
-                    ></textarea>
-                </div>
+            <div class="col-12 pl-0">
+                <textarea
+                    ng-model="remito.observaciones"
+                    class="form-control mx-2"
+                    rows="5"
+                ></textarea>
             </div>
         </div>
     </form>
-- 
1.9.1