diff --git a/ci4/app/Controllers/Presupuestos/Presupuestocliente.php b/ci4/app/Controllers/Presupuestos/Presupuestocliente.php index 75e2c176..cb6faeae 100755 --- a/ci4/app/Controllers/Presupuestos/Presupuestocliente.php +++ b/ci4/app/Controllers/Presupuestos/Presupuestocliente.php @@ -137,13 +137,12 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController // Se obtiene el cliente ID a partir del usuario de la sesion $model_user = model('App\Models\Usuarios\UserModel'); $user = $model_user->find(auth()->user()->id); - if(auth()->user()->inGroup('cliente-admin') || auth()->user()->inGroup('cliente-editor')){ + if (auth()->user()->inGroup('cliente-admin') || auth()->user()->inGroup('cliente-editor')) { $clienteId = $user->cliente_id; - } - else{ + } else { $clienteId = $presupuestoEntity->cliente_id; } - + $this->viewData['formAction'] = 'edit'; $this->viewData['paisList'] = $this->getPaisListItems(); @@ -361,8 +360,8 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController ); $return_data = $this->calcular_presupuesto($datos_presupuesto, 0, false); //TRUE FOR DEBUG - if(array_key_exists('errors', $return_data)){ - if($return_data['errors']->status == 1){ + if (array_key_exists('errors', $return_data)) { + if ($return_data['errors']->status == 1) { $return_data = [ 'errors' => (object) ([ 'status' => 1 @@ -370,13 +369,12 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController ]; return $return_data; } - } - + if (array_key_exists('exception', $return_data)) { return $this->failServerError( $return_data['exception'] . ' - ' . - $return_data['file'] . ' - ' . $return_data['line'] + $return_data['file'] . ' - ' . $return_data['line'] ); } @@ -412,7 +410,6 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController ]), ]; return $return_data; - } else { $coste_envio += $coste_direccion->coste; } @@ -449,8 +446,6 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController $return_data['precio_u'][$i] = round(floatval($return_data['precio_u'][$i]) + $coste_envio / $tirada[$i], 4); } - - } return $this->respond($return_data); @@ -719,7 +714,7 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController 'isColor' => $isColor, 'isHq' => $isHq, 'paginasCuadernillo' => $paginasCuadernillo, - + 'interior' => $interior, 'cubierta' => $cubierta, 'sobrecubierta' => $sobrecubierta, @@ -787,8 +782,6 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController $resultado_presupuesto['info']['totales'][$i]['coste_envio'] = $coste_envio - $margen_envio; $resultado_presupuesto['info']['totales'][$i]['margen_envio'] = $margen_envio; } - - } $model_presupuesto = new PresupuestoModel(); @@ -823,7 +816,7 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController $borrar_antes = false; if ($id != 0) { - $borrar_antes = true; + $borrar_antes = true; } if ($datos_presupuesto['sobrecubierta']) { @@ -834,13 +827,13 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController } $acabado_id = $this->obtenerTarifasAcabado($datos_presupuesto['cubierta']['acabadosCubierta']); if (count($acabado_id) > 0) { - if(array_key_exists('plastificado', $acabado_id)){ + if (array_key_exists('plastificado', $acabado_id)) { $datos_presupuesto['cubierta']['acabadosCubierta']['plastificado'] = $acabado_id['plastificado']; } - if(array_key_exists('barniz', $acabado_id)){ + if (array_key_exists('barniz', $acabado_id)) { $datos_presupuesto['cubierta']['acabadosCubierta']['barniz'] = $acabado_id['barniz']; } - if(array_key_exists('estampado', $acabado_id)){ + if (array_key_exists('estampado', $acabado_id)) { $datos_presupuesto['cubierta']['acabadosCubierta']['estampado'] = $acabado_id['estampado']; } } else { @@ -898,8 +891,11 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController } } - foreach ($reqData['direcciones'] as $direccion) { - $this->guardarLineaEnvio($id, $direccion, $peso_libro); + if (array_key_exists('direcciones', $reqData)) { + + foreach ($reqData['direcciones'] as $direccion) { + $this->guardarLineaEnvio($id, $direccion, $peso_libro); + } } if ($confirmar) { @@ -971,15 +967,15 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController $data['cubierta']['cabezada'] = $presupuesto->cabezada; $modelAcabado = model("App\Models\Tarifas\Acabados\TarifaAcabadoModel"); $data['cubierta']['plastificado'] = $modelAcabado->getCodeFromId($presupuesto->acabado_cubierta_id); - if($data['cubierta']['plastificado'] == ''){ + if ($data['cubierta']['plastificado'] == '') { $data['cubierta']['plastificado'] = 'NONE'; } $data['cubierta']['barniz'] = $modelAcabado->getCodeFromId($presupuesto->barniz_cubierta_id); - if($data['cubierta']['barniz'] == ''){ + if ($data['cubierta']['barniz'] == '') { $data['cubierta']['barniz'] = 'NONE'; } $data['cubierta']['estampado'] = $modelAcabado->getCodeFromId($presupuesto->estampado_cubierta_id); - if($data['cubierta']['estampado'] == ''){ + if ($data['cubierta']['estampado'] == '') { $data['cubierta']['estampado'] = 'NONE'; } $data['cubierta']['retractilado'] = $presupuesto->retractilado ? 1 : 0; @@ -1003,16 +999,13 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController for ($i = 0; $i < count($tiradas_alternativas); $i++) { $tirada = $tiradas_alternativas[$i]; $data['datosGenerales']['tirada' . ($i + 2)] = $tirada->tirada; - } - + return $this->respond([ 'status' => 1, 'data' => $data ]); } - - } else { return $this->failUnauthorized('Invalid request', 403); } @@ -1404,7 +1397,7 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController if ($interior == -1) { $errorModel = new ErrorPresupuesto(); $errorModel->insertError( - $datos_entrada['id']==0?null:$datos_entrada['id'], + $datos_entrada['id'] == 0 ? null : $datos_entrada['id'], auth()->user()->id, ' No se puede obtener el interior', @@ -1949,7 +1942,6 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController } return $return_data; - } catch (Exception $e) { return [ 'exception' => $e->getMessage(), @@ -2142,8 +2134,7 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController $color = 'negro'; $model = model('App\Models\Presupuestos\PresupuestoLineaModel'); - $data = $model->where('presupuesto_id', $presupuestoId)->findAll(); - ; + $data = $model->where('presupuesto_id', $presupuestoId)->findAll();; foreach ($data as $linea) { if (strpos($linea->tipo, "hq") !== false) { // $linea->tipo contains the substring "hq" @@ -2443,5 +2434,4 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController return [$paginasNegro, $paginasColor]; } - }