actualizacion command-contadores

src/BackendBundle/Command/McafeeMigrarUsuariosCommand.php

@@ -37,6 +37,7 @@ class McafeeMigrarUsuariosCommand extends ContainerAwareCommand {
         $output->writeln("COMANDO INICIADO.");
         $log = Utiles::setLog('command::migrarUsuariosSuscripcion', 'ws/wsMigrarUsuariosMcAfee');
         $esTxPendiente = false;
+        $failedAlready = false;
         $ultimoUsuarioProcesado = null;
         $cantidadProcesados = 0;
         $cantidadPendientes = 0;
@@ -99,7 +100,7 @@ class McafeeMigrarUsuariosCommand extends ContainerAwareCommand {
                         <DATA>
                         <CUSTOMERCONTEXT ID="' . $customerContextId . '" REQUESTTYPE="UPDATE">
                          <ACCOUNT/>
-                         <ORDER PARTNERREF="' . $mcAfeeTx->getMtrReferencia().' REF="">
+                         <ORDER PARTNERREF="' . $mcAfeeTx->getMtrReferencia().'" REF="">
                          <ITEMS>
                          <ITEM SKU="' . self::SKU_DESTINY . '" QTY="1" ACTION="TU">
                          <SOURCESKU>' . $usuarioMigrar->getSku() . '</SOURCESKU></ITEM>
@@ -140,13 +141,14 @@ class McafeeMigrarUsuariosCommand extends ContainerAwareCommand {
                     $logError = Utiles::setLog('command::migrarUsuariosSuscripcion', 'error/wsMigrarUsuariosMcAfee');
                     $logError->err('$arrayRespuesta == NULL mara mcAfeeTx id->' . $mcAfeeTx->getMtrId(). 'PETICION:'. print_r($peticion, TRUE));
                     $cantidadPendientes++;
+                    $failedAlready  = true;
                 }
                 $log->debug('$arrayRespuesta McAfee->' . print_r($arrayRespuesta, true));
 
                 $mcAfeeTx->setMtrCodigoRetorno($arrayRespuesta['mcAfeeReturnCode']);
-
+                
                 $log->debug('Descripcion de la transaccion ->' . $arrayRespuesta['mcAfeeReturnDesc']);
-                if ($arrayRespuesta['mcAfeeReturnCode'] == '1000' || $arrayRespuesta['mcAfeeReturnCode'] == '5001'|| $arrayRespuesta['mcAfeeReturnCode'] == '5002') {
+                if (!$failedAlready || $arrayRespuesta['mcAfeeReturnCode'] == '1000' || $arrayRespuesta['mcAfeeReturnCode'] == '5001'|| $arrayRespuesta['mcAfeeReturnCode'] == '5002') {
                     //validacion datos Tx
                     if ($arrayRespuesta['mcAfeeCostumerId'] == $nuevoUsuario->getUmcCustomerId() && $arrayRespuesta['mcAfeeOrderPartnerRef'] == $mcAfeeTx->getMtrReferencia()) {
                         /**
@@ -158,7 +160,6 @@ class McafeeMigrarUsuariosCommand extends ContainerAwareCommand {
                         $mcAfeeTx->save();
                         $log->debug('Transaccion finalizada con exito!.  id Transaccion->' . $mcAfeeTx->getMtrId());
                         $cantidadProcesados++;
-
                     } else {
                         $mcAfeeTx->setMtrEstado(McafeeTransaccion::ESTADO_FRACASADA);
                         $mcAfeeTx->save();
@@ -172,6 +173,7 @@ class McafeeMigrarUsuariosCommand extends ContainerAwareCommand {
                         $logError->err('nro de referencia Respuesta->' . $arrayRespuesta['mcAfeeOrderPartnerRef'] . 'y nro referencia->' . $mcAfeeTx->getMtrReferencia());
                         $logError->err('la transaccion fracaso debido a inconsistencia en los datos.'.print_r($arrayRespuesta,true));
                         $cantidadPendientes++;
+                        $failedAlready = true;
                     }
                 } else {//codigo de retorno distinto de 1000 y 5001
                     $mcAfeeTx->setMtrEstado(McafeeTransaccion::ESTADO_FRACASADA);
@@ -183,6 +185,7 @@ class McafeeMigrarUsuariosCommand extends ContainerAwareCommand {
                     $logError = Utiles::setLog('command::migrarUsuariosSuscripcion', 'error/wsMigrarUsuariosMcAfee');
                     $logError->err('returnCode distinto de 1000 y 5001 que son los exitosos.'.print_r($arrayRespuesta,true));
                     $cantidadPendientes++;
+                    $failedAlready = true;
                 }
             } else {//$respuesta ==NULL                
                 $mcAfeeTx->setMtrEstado(McafeeTransaccion::ESTADO_FRACASADA);
@@ -193,6 +196,7 @@ class McafeeMigrarUsuariosCommand extends ContainerAwareCommand {
                 $logError = Utiles::setLog('command::migrarUsuariosSuscripcion', 'error/wsMigrarUsuariosMcAfee');
                 $logError->err('respuesta NULL para peticion:' . print_r($peticion, true));
                 $cantidadPendientes++;
+                $failedAlready = true;
             }
         } catch (\Exception $exc) {
 
@@ -206,6 +210,7 @@ class McafeeMigrarUsuariosCommand extends ContainerAwareCommand {
             $logError->err('Error al llamar WS. Exception:' . $exc->getMessage() . ' | peticion:' . print_r($peticion, true));
             //return FALSE;
             $cantidadPendientes++;
+            $failedAlready = true;
         }
         
         $ultimoUsuarioProcesado = $nuevoUsuario;