Related #113 Sincronizar comprobantes huerfanos

php/datos/dao_sq_nucleo.php

@@ -652,6 +652,8 @@ class dao_sq_nucleo extends dao_pirapire
             return;
         }
 
+
+
         foreach ($ventas as $venta) {
             // Si no tiene el id de la venta de SQ se obtiene mediante el id_venta_origen
             if (is_null($venta['id_venta_sq'])){
@@ -754,12 +756,23 @@ class dao_sq_nucleo extends dao_pirapire
         }
     }
 
-    public function sincronizar_comprobantes_huerfanos(){
+    /**
+     * Obtiene los comprobantes huerfanos mediante el recurso de SQ afip/comprobantes_huerfanos
+     *
+     * Por cada combrobante huerfano se obtiene los datos del comprobante de pirapire y se busca los
+     * datos actuales en SQ y luego se actualiza los datos del comprobante
+     *
+     * @param null $id_punto_cobranza
+     * @return false
+     * @throws toba_error
+     * @throws toba_error_db
+     */
+    public function sincronizar_comprobantes_huerfanos($id_punto_cobranza = null){
         $pirapire_rest = new pirapire_rest();
 
         try {
-            if (isset($filtro['id'])) {
-                $resultado = $pirapire_rest->sanaviron()->get('afip/comprobantes_huerfanos/'.$filtro['id_punto_cobranza']);
+            if (!is_null($id_punto_cobranza)) {
+                $resultado = $pirapire_rest->sanaviron()->get('afip/comprobantes_huerfanos/'.$id_punto_cobranza);
             } else {
                 $resultado = $pirapire_rest->sanaviron()->get('afip/comprobantes_huerfanos');
             }
@@ -786,12 +799,12 @@ class dao_sq_nucleo extends dao_pirapire
             11 => 64,
         ];
 
+        $dao_ventas = new dao_ventas();
+
         foreach ($resultado as $comprobante){
             $punto_venta = str_pad($puntos_cobranzas_sq[$comprobante['id_punto_cobranza']], 5, 0, STR_PAD_LEFT);
             $numero_comprobante = str_pad($comprobante['numero_faltante'], 8, 0, STR_PAD_LEFT);
 
-            $dao_ventas = new dao_ventas();
-
             $filtro['nro_punto_venta'] = "nro_punto_venta = '{$punto_venta}'";
             $filtro['nro_comprobante'] = "nro_comprobante = '{$numero_comprobante}'";
 
@@ -799,6 +812,7 @@ class dao_sq_nucleo extends dao_pirapire
 
             if (count($datos_comprobantes) > 0 && isset($datos_comprobantes[0]['id'])){
                 $id_venta = $datos_comprobantes[0]['id'];
+
                 $this->sincronizar_ventas_datos_sq($id_venta);
             }
         }

php/extension_toba/pirapire_comando.php

@@ -113,8 +113,8 @@ class pirapire_comando extends toba_aplicacion_comando_base
             toba_nucleo::instancia()->iniciar_contexto_desde_consola('desarrollo', 'pirapire');
         }
 
-        $id = (array_key_exists('--id_punto_cobranza', $parametros)) ? $parametros['--id_punto_cobranza'] : null;
+        $id_punto_cobranza = (array_key_exists('--id_punto_cobranza', $parametros)) ? $parametros['--id_punto_cobranza'] : null;
 
-        $this->modelo->sincronizar_comprobantes_huerfanos($id);
+        $this->modelo->sincronizar_comprobantes_huerfanos($id_punto_cobranza);
     }
 }

php/extension_toba/pirapire_modelo.php

@@ -64,9 +64,9 @@ class pirapire_modelo extends toba_aplicacion_modelo_base
         $dao_sq_nucleo->sincronizar_condiciones_venta($id);
     }
 
-    public function sincronizar_comprobantes_huerfanos($id){
+    public function sincronizar_comprobantes_huerfanos($id_punto_cobranza){
         $dao_sq_nucleo = new dao_sq_nucleo();
 
-        $dao_sq_nucleo->sincronizar_comprobantes_huerfanos($id);
+        $dao_sq_nucleo->sincronizar_comprobantes_huerfanos($id_punto_cobranza);
     }
 }