diff --git a/controller/admin_agentes.php b/controller/admin_agentes.php index 4f93be8..66dc022 100644 --- a/controller/admin_agentes.php +++ b/controller/admin_agentes.php @@ -90,8 +90,9 @@ private function nuevo_agente() if ($age0->save()) { $this->new_message("Empleado " . $age0->codagente . " guardado correctamente."); header('location: ' . $age0->url()); - } else + } else { $this->new_error_msg("¡Imposible guardar el empleado!"); + } } private function eliminar_agente() @@ -104,10 +105,12 @@ private function eliminar_agente() $this->new_error_msg('No tienes permiso para eliminar en esta página.'); } else if ($age0->delete()) { $this->new_message("Empleado " . $age0->codagente . " eliminado correctamente."); - } else + } else { $this->new_error_msg("¡Imposible eliminar el empleado!"); - } else + } + } else { $this->new_error_msg("¡Empleado no encontrado!"); + } } public function paginas() diff --git a/controller/admin_empresa.php b/controller/admin_empresa.php index 83d43b4..a3f16f5 100644 --- a/controller/admin_empresa.php +++ b/controller/admin_empresa.php @@ -124,7 +124,7 @@ protected function private_core() $fsvar->simple_save('install_step', $step); } - if ($step == 3 AND $this->empresa->contintegrada) { + if ($step == 3 && $this->empresa->contintegrada) { $this->new_message('Recuerda que tienes que importar los datos del ejercicio.'); } diff --git a/controller/articulo_subcuentas.php b/controller/articulo_subcuentas.php index 15062a1..1b0d04e 100644 --- a/controller/articulo_subcuentas.php +++ b/controller/articulo_subcuentas.php @@ -57,7 +57,7 @@ protected function private_core() $this->articulo->codsubcuentairpfcom = $_POST['codsubcuentairpfcom']; $aprops = array('codsubcuentaventa' => $_POST['codsubcuentaventa']); - if ($this->articulo->save() AND $ap->array_save($this->articulo->referencia, $aprops)) { + if ($this->articulo->save() && $ap->array_save($this->articulo->referencia, $aprops)) { $this->new_message('Datos guardados correctamente.'); } else { $this->new_error_msg('Error al guardar las subcuentas.'); diff --git a/controller/articulo_trazabilidad.php b/controller/articulo_trazabilidad.php index d2f7496..5c467fa 100644 --- a/controller/articulo_trazabilidad.php +++ b/controller/articulo_trazabilidad.php @@ -46,7 +46,7 @@ protected function private_core() if ($this->articulo) { if (isset($_POST['numserie'])) { - if ($_POST['numserie'] != '' OR $_POST['lote'] != '') { + if ($_POST['numserie'] != '' || $_POST['lote'] != '') { $this->edit_traza(); } else { $this->new_error_msg('Debes escribir un número de serie o un lote o ambos,' diff --git a/controller/base_wizard.php b/controller/base_wizard.php index 73c8da6..910437a 100644 --- a/controller/base_wizard.php +++ b/controller/base_wizard.php @@ -78,7 +78,7 @@ private function private_core2() $fsvar = new fs_var(); $this->step = $fsvar->simple_get('install_step'); - if ($this->step < 2 OR isset($_GET['restart'])) { + if ($this->step < 2 || isset($_GET['restart'])) { $this->step = 2; } @@ -123,7 +123,7 @@ private function private_core2() /// avanzamos el asistente $this->step = 3; - if ($this->empresa->codpais == 'ESP' OR $this->empresa->codpais == 'ES') { + if ($this->empresa->codpais == 'ESP' || $this->empresa->codpais == 'ES') { /// si es España nos podemos ahorrar un paso $this->empresa->coddivisa = 'EUR'; $this->empresa->save(); @@ -230,7 +230,7 @@ private function check_menu() /// leemos todos los controladores del plugin foreach (scandir(__DIR__) as $f) { - if ($f != '.' AND $f != '..' AND is_string($f) AND strlen($f) > 4 AND ! is_dir($f) AND $f != __CLASS__ . '.php') { + if ($f != '.' && $f != '..' && is_string($f) && strlen($f) > 4 && ! is_dir($f) && $f != __CLASS__ . '.php') { /// obtenemos el nombre $page_name = substr($f, 0, -4); diff --git a/controller/compras_actualiza_arts.php b/controller/compras_actualiza_arts.php index 90e6e25..12410af 100644 --- a/controller/compras_actualiza_arts.php +++ b/controller/compras_actualiza_arts.php @@ -42,7 +42,7 @@ protected function private_core() $this->documento = FALSE; $this->lineas = array(); $this->tipodoc = 'Documento'; - if (isset($_REQUEST['doc']) AND isset($_REQUEST['id'])) { + if (isset($_REQUEST['doc']) && isset($_REQUEST['id'])) { if ($_REQUEST['doc'] == 'pedido') { $pedido0 = new pedido_proveedor(); $this->documento = $pedido0->get($_REQUEST['id']); @@ -119,7 +119,7 @@ protected function private_core() } /// ¿usamos la referencia de proveedor como equivalencia? - if ($_POST['refproveedor_' . $value->idlinea] != '' AND $_POST['refproveedor_' . $value->idlinea] != $articulo->referencia) { + if ($_POST['refproveedor_' . $value->idlinea] != '' && $_POST['refproveedor_' . $value->idlinea] != $articulo->referencia) { if (is_null($articulo->equivalencia)) { $articulo->equivalencia = $_POST['refproveedor_' . $value->idlinea]; } @@ -179,7 +179,7 @@ public function url() { $url = parent::url(); - if (isset($_REQUEST['doc']) AND isset($_REQUEST['id'])) { + if (isset($_REQUEST['doc']) && isset($_REQUEST['id'])) { $url .= '&doc=' . $_REQUEST['doc'] . '&id=' . $_REQUEST['id']; } diff --git a/controller/compras_agrupar_albaranes.php b/controller/compras_agrupar_albaranes.php index f7469c2..ecb303e 100644 --- a/controller/compras_agrupar_albaranes.php +++ b/controller/compras_agrupar_albaranes.php @@ -311,7 +311,7 @@ public function pendientes() /// Comprobamos si el proveedor ya está en la lista. $encontrado = FALSE; foreach ($pendientes as $i => $pe) { - if ($alb->codproveedor == $pe['codproveedor'] AND $alb->codserie == $pe['codserie'] AND $alb->coddivisa == $pe['coddivisa']) { + if ($alb->codproveedor == $pe['codproveedor'] && $alb->codserie == $pe['codserie'] && $alb->coddivisa == $pe['coddivisa']) { $encontrado = TRUE; $pendientes[$i]['num'] ++; diff --git a/controller/compras_albaran.php b/controller/compras_albaran.php index 325ad56..9c5afe6 100644 --- a/controller/compras_albaran.php +++ b/controller/compras_albaran.php @@ -96,7 +96,7 @@ protected function private_core() /// comprobamos el albarán $this->albaran->full_test(); - if (isset($_POST['facturar']) AND isset($_POST['petid']) AND $this->albaran->ptefactura) { + if (isset($_POST['facturar']) && isset($_POST['petid']) && $this->albaran->ptefactura) { if ($this->duplicated_petition($_POST['petid'])) { $this->new_error_msg('Petición duplicada. Evita hacer doble clic sobre los botones.'); } else @@ -243,7 +243,7 @@ private function modificar() $lineas[$k]->iva = 0; $lineas[$k]->recargo = 0; $lineas[$k]->irpf = floatval($_POST['irpf_' . $num]); - if (!$serie->siniva AND $regimeniva != 'Exento') { + if (!$serie->siniva && $regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $num]); if ($imp0) { $lineas[$k]->codimpuesto = $imp0->codimpuesto; @@ -278,12 +278,12 @@ private function modificar() } /// añadimos la línea - if (!$encontrada AND intval($_POST['idlinea_' . $num]) == -1 AND isset($_POST['referencia_' . $num])) { + if (!$encontrada && intval($_POST['idlinea_' . $num]) == -1 && isset($_POST['referencia_' . $num])) { $linea = new linea_albaran_proveedor(); $linea->idalbaran = $this->albaran->idalbaran; $linea->descripcion = $_POST['desc_' . $num]; - if (!$serie->siniva AND $regimeniva != 'Exento') { + if (!$serie->siniva && $regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $num]); if ($imp0) { $linea->codimpuesto = $imp0->codimpuesto; diff --git a/controller/compras_albaranes.php b/controller/compras_albaranes.php index 29c7fae..0ad6882 100644 --- a/controller/compras_albaranes.php +++ b/controller/compras_albaranes.php @@ -108,7 +108,7 @@ protected function private_core() if (isset($_POST['delete'])) { $this->delete_albaran(); } else { - if (!isset($_GET['mostrar']) AND ( $this->query != '' OR isset($_REQUEST['codagente']) OR isset($_REQUEST['codproveedor']) OR isset($_REQUEST['codserie']))) { + if (!isset($_GET['mostrar']) && ( $this->query != '' || isset($_REQUEST['codagente']) || isset($_REQUEST['codproveedor']) || isset($_REQUEST['codserie']))) { /** * si obtenermos un codagente, un codproveedor o un codserie pasamos direcatemente * a la pestaña de búsqueda, a menos que tengamos un mostrar, que @@ -117,11 +117,9 @@ protected function private_core() $this->mostrar = 'buscar'; } - if (isset($_REQUEST['codproveedor'])) { - if ($_REQUEST['codproveedor'] != '') { - $pro0 = new proveedor(); - $this->proveedor = $pro0->get($_REQUEST['codproveedor']); - } + if (isset($_REQUEST['codproveedor']) && $_REQUEST['codproveedor'] != '') { + $pro0 = new proveedor(); + $this->proveedor = $pro0->get($_REQUEST['codproveedor']); } if (isset($_REQUEST['codagente'])) { @@ -193,9 +191,9 @@ public function url($busqueda = FALSE) . "&hasta=" . $this->hasta; return $url; - } else { - return parent::url(); } + + return parent::url(); } public function paginas() @@ -247,7 +245,7 @@ private function delete_albaran() * no tocamos, porque sigue estando ese otro albarán. * (las facturas pueden agrupar albaranes). */ - if ($linea->referencia AND ( is_null($linea->idalbaran) OR $linea->idalbaran == $alb1->idalbaran)) { + if ($linea->referencia && ( is_null($linea->idalbaran) || $linea->idalbaran == $alb1->idalbaran)) { $art0 = $articulo->get($linea->referencia); if ($art0) { $art0->sum_stock($alb1->codalmacen, 0 - $linea->cantidad, TRUE, $linea->codcombinacion); @@ -270,10 +268,12 @@ private function delete_albaran() if ($alb1->delete()) { $this->clean_last_changes(); - } else + } else { $this->new_error_msg("¡Imposible eliminar el " . FS_ALBARAN . "!"); - } else + } + } else { $this->new_error_msg("¡" . FS_ALBARAN . " no encontrado!"); + } } private function share_extension() diff --git a/controller/compras_factura.php b/controller/compras_factura.php index 2f79386..58bf0c3 100644 --- a/controller/compras_factura.php +++ b/controller/compras_factura.php @@ -62,7 +62,7 @@ protected function private_core() */ $this->mostrar_boton_pagada = TRUE; foreach ($this->extensions as $ext) { - if ($ext->type == 'config' AND $ext->text == 'no_button_pagada') { + if ($ext->type == 'config' && $ext->text == 'no_button_pagada') { $this->mostrar_boton_pagada = FALSE; break; } @@ -88,7 +88,7 @@ protected function private_core() $proveedor = new proveedor(); $this->proveedor = $proveedor->get($this->factura->codproveedor); - if (isset($_GET['gen_asiento']) AND isset($_GET['petid'])) { + if (isset($_GET['gen_asiento']) && isset($_GET['petid'])) { if ($this->duplicated_petition($_GET['petid'])) { $this->new_error_msg('Petición duplicada. Evita hacer doble clic sobre los botones.'); } else { @@ -182,7 +182,7 @@ private function pagar($pagada = TRUE) if (is_null($this->factura->idasiento)) { $this->factura->pagada = $pagada; $this->factura->save(); - } else if (!$pagada AND $this->factura->pagada) { + } else if (!$pagada && $this->factura->pagada) { /// marcar como impagada $this->factura->pagada = FALSE; @@ -200,14 +200,14 @@ private function pagar($pagada = TRUE) } else { $this->new_error_msg('Error al modificar la factura.'); } - } else if ($pagada AND ! $this->factura->pagada) { + } else if ($pagada && ! $this->factura->pagada) { /// marcar como pagada $asiento = $this->factura->get_asiento(); if ($asiento) { /// nos aseguramos que el proveedor tenga subcuenta en el ejercicio actual $subpro = FALSE; $eje = $this->ejercicio->get_by_fecha($_POST['fpagada']); - if ($eje AND $this->proveedor) { + if ($eje && $this->proveedor) { $subpro = $this->proveedor->get_subcuenta($eje->codejercicio); } diff --git a/controller/compras_facturas.php b/controller/compras_facturas.php index 23c55c1..470c9bb 100644 --- a/controller/compras_facturas.php +++ b/controller/compras_facturas.php @@ -111,7 +111,7 @@ protected function private_core() if (isset($_GET['delete'])) { $this->delete_factura(); } else { - if (!isset($_GET['mostrar']) AND ( $this->query != '' OR isset($_REQUEST['codagente']) OR isset($_REQUEST['codproveedor']) OR isset($_REQUEST['codserie']))) { + if (!isset($_GET['mostrar']) && ( $this->query != '' || isset($_REQUEST['codagente']) || isset($_REQUEST['codproveedor']) || isset($_REQUEST['codserie']))) { /** * si obtenermos un codagente, un codproveedor o un codserie pasamos direcatemente * a la pestaña de búsqueda, a menos que tengamos un mostrar, que @@ -120,11 +120,9 @@ protected function private_core() $this->mostrar = 'buscar'; } - if (isset($_REQUEST['codproveedor'])) { - if ($_REQUEST['codproveedor'] != '') { - $pro0 = new proveedor(); - $this->proveedor = $pro0->get($_REQUEST['codproveedor']); - } + if (isset($_REQUEST['codproveedor']) && $_REQUEST['codproveedor'] != '') { + $pro0 = new proveedor(); + $this->proveedor = $pro0->get($_REQUEST['codproveedor']); } if (isset($_REQUEST['codagente'])) { @@ -194,9 +192,9 @@ public function url($busqueda = FALSE) . "&hasta=" . $this->hasta; return $url; - } else { - return parent::url(); } + + return parent::url(); } public function paginas() @@ -392,10 +390,12 @@ private function delete_factura() } $this->clean_last_changes(); - } else + } else { $this->new_error_msg("¡Imposible eliminar la factura!"); - } else + } + } else { $this->new_error_msg("Factura no encontrada."); + } } public function orden() diff --git a/controller/compras_imprimir.php b/controller/compras_imprimir.php index b3d73d5..c17e27d 100644 --- a/controller/compras_imprimir.php +++ b/controller/compras_imprimir.php @@ -41,17 +41,34 @@ public function __construct($name = __CLASS__, $title = 'imprimir', $folder = 'c parent::__construct($name, $title, $folder, FALSE, FALSE); } - protected function private_core() + protected function init() { $this->articulo_proveedor = new articulo_proveedor(); $this->documento = FALSE; $this->impuesto = new impuesto(); $this->proveedor = FALSE; $this->cargar_config(); + $this->share_extensions(); + } - if (isset($_REQUEST['albaran']) AND isset($_REQUEST['id'])) { - $this->articulo_traza = new articulo_traza(); + protected function cargar_config() + { + /// obtenemos los datos de configuración de impresión + $this->impresion = array( + 'print_ref' => '1', + 'print_dto' => '1', + 'print_alb' => '0', + 'print_formapago' => '1' + ); + $fsvar = new fs_var(); + $this->impresion = $fsvar->array_get($this->impresion, FALSE); + } + protected function private_core() + { + $this->init(); + + if (isset($_REQUEST['albaran']) && isset($_REQUEST['id'])) { $alb = new albaran_proveedor(); $this->documento = $alb->get($_REQUEST['id']); if ($this->documento) { @@ -62,11 +79,9 @@ protected function private_core() if (isset($_POST['email'])) { $this->enviar_email_proveedor(); } else { - $this->generar_pdf_albaran(); + $this->generar_pdf_albaran_proveedor(); } - } else if (isset($_REQUEST['factura']) AND isset($_REQUEST['id'])) { - $this->articulo_traza = new articulo_traza(); - + } else if (isset($_REQUEST['factura']) && isset($_REQUEST['id'])) { $fac = new factura_proveedor(); $this->documento = $fac->get($_REQUEST['id']); if ($this->documento) { @@ -74,23 +89,8 @@ protected function private_core() $this->proveedor = $proveedor->get($this->documento->codproveedor); } - $this->generar_pdf_factura(); + $this->generar_pdf_factura_proveedor(); } - - $this->share_extensions(); - } - - protected function cargar_config() - { - /// obtenemos los datos de configuración de impresión - $this->impresion = array( - 'print_ref' => '1', - 'print_dto' => '1', - 'print_alb' => '0', - 'print_formapago' => '1' - ); - $fsvar = new fs_var(); - $this->impresion = $fsvar->array_get($this->impresion, FALSE); } protected function share_extensions() @@ -138,7 +138,7 @@ protected function generar_pdf_lineas(&$pdf_doc, &$lineas, &$linea_actual, &$lpp while ($linea_a < count($lineas)) { $lppag2 = $lppag; foreach ($lineas as $i => $lin) { - if ($i >= $linea_a AND $i < $linea_a + $lppag2) { + if ($i >= $linea_a && $i < $linea_a + $lppag2) { $linea_size = 1; $len = mb_strlen($lin->referencia . ' ' . $lin->descripcion); while ($len > 85) { @@ -210,7 +210,7 @@ protected function generar_pdf_lineas(&$pdf_doc, &$lineas, &$linea_actual, &$lpp } /// restamos líneas al documento en función del tamaño de la descripción - if ($i >= $linea_actual AND $i < $linea_actual + $lppag) { + if ($i >= $linea_actual && $i < $linea_actual + $lppag) { $linea_size = 1; $len = mb_strlen($lin->referencia . ' ' . $lin->descripcion); while ($len > 85) { @@ -258,7 +258,7 @@ protected function generar_pdf_lineas(&$pdf_doc, &$lineas, &$linea_actual, &$lpp $table_header['importe'] = 'Importe'; $pdf_doc->add_table_header($table_header); - for ($i = $linea_actual; (($linea_actual < ($lppag + $i)) AND ( $linea_actual < count($lineas)));) { + for ($i = $linea_actual; (($linea_actual < ($lppag + $i)) && ( $linea_actual < count($lineas)));) { $descripcion = fs_fix_html($lineas[$linea_actual]->descripcion); if (!is_null($lineas[$linea_actual]->referencia) && $this->impresion['print_ref']) { $descripcion = '' . $this->get_referencia_proveedor($lineas[$linea_actual]->referencia) @@ -318,8 +318,6 @@ protected function generar_pdf_lineas(&$pdf_doc, &$lineas, &$linea_actual, &$lpp if ($linea_actual == count($lineas) && $this->documento->observaciones != '') { $pdf_doc->pdf->ezText("\n" . fs_fix_html($this->documento->observaciones), 9); } - - $pdf_doc->set_y(80); } protected function get_referencia_proveedor($ref) @@ -339,6 +337,10 @@ protected function get_referencia_proveedor($ref) */ protected function generar_trazabilidad($linea, $tabla1 = 'linea_albaran_proveedor', $columna1 = 'idlalbcompra', $tabla2 = 'linea_factura_proveedor', $columna2 = 'idlfaccompra') { + if (!isset($this->articulo_traza)) { + $this->articulo_traza = new articulo_traza(); + } + $lineast = array(); if (get_class_name($linea) == $tabla1) { $lineast = $this->articulo_traza->all_from_linea($columna1, $linea->idlinea); @@ -355,7 +357,7 @@ protected function generar_trazabilidad($linea, $tabla1 = 'linea_albaran_proveed $salto = ''; } - if ($lt->lote AND $lt->lote != $lote) { + if ($lt->lote && $lt->lote != $lote) { $txt .= $salto . 'Lote: ' . $lt->lote; $lote = $lt->lote; } @@ -377,6 +379,8 @@ protected function generar_pdf_datos_proveedor(&$pdf_doc) } else { $tipo_doc = 'Factura'; } + } else if (get_class_name($this->documento) == 'pedido_proveedor') { + $tipo_doc = ucfirst(FS_PEDIDO); } $tipoidfiscal = FS_CIFNIF; @@ -490,7 +494,7 @@ public function generar_pdf_pedido_proveedor($archivo = FALSE) } - public function generar_pdf_albaran($archivo = FALSE) + public function generar_pdf_albaran_proveedor($archivo = FALSE) { if (!$archivo) { /// desactivamos la plantilla HTML @@ -521,6 +525,8 @@ public function generar_pdf_albaran($archivo = FALSE) $pdf_doc->generar_pdf_cabecera($this->empresa, $lppag); $this->generar_pdf_datos_proveedor($pdf_doc); $this->generar_pdf_lineas($pdf_doc, $lineas, $linea_actual, $lppag); + + $pdf_doc->set_y(80); $this->generar_pdf_totales($pdf_doc, $lineas_iva, $pagina); $pagina++; } @@ -539,7 +545,7 @@ public function generar_pdf_albaran($archivo = FALSE) } } - public function generar_pdf_factura($archivo = FALSE) + public function generar_pdf_factura_proveedor($archivo = FALSE) { if (!$archivo) { /// desactivamos la plantilla HTML @@ -570,6 +576,8 @@ public function generar_pdf_factura($archivo = FALSE) $pdf_doc->generar_pdf_cabecera($this->empresa, $lppag); $this->generar_pdf_datos_proveedor($pdf_doc); $this->generar_pdf_lineas($pdf_doc, $lineas, $linea_actual, $lppag); + + $pdf_doc->set_y(80); $this->generar_pdf_totales($pdf_doc, $lineas_iva, $pagina); $pagina++; } @@ -588,7 +596,7 @@ public function generar_pdf_factura($archivo = FALSE) } } - private function enviar_email_proveedor($doc = 'albaran') + protected function enviar_email_proveedor($doc = 'albaran') { if ($this->empresa->can_send_mail()) { if ($this->proveedor && $_POST['email'] != $this->proveedor->email && isset($_POST['guardar'])) { @@ -598,7 +606,7 @@ private function enviar_email_proveedor($doc = 'albaran') $filename = $doc . '_' . $this->documento->codigo . '.pdf'; if ($doc == 'albaran') { - $this->generar_pdf_albaran($filename); + $this->generar_pdf_albaran_proveedor($filename); } else { $this->generar_pdf_pedido_proveedor($filename); } @@ -640,13 +648,9 @@ private function enviar_email_proveedor($doc = 'albaran') $mail->addAttachment($_FILES['adjunto']['tmp_name'], $_FILES['adjunto']['name']); } - if ($this->empresa->mail_connect($mail)) { - if ($mail->send()) { - $this->new_message('Mensaje enviado correctamente.'); - $this->empresa->save_mail($mail); - } else { - $this->new_error_msg("Error al enviar el email: " . $mail->ErrorInfo); - } + if ($this->empresa->mail_connect($mail) && $mail->send()) { + $this->new_message('Mensaje enviado correctamente.'); + $this->empresa->save_mail($mail); } else { $this->new_error_msg("Error al enviar el email: " . $mail->ErrorInfo); } diff --git a/controller/compras_proveedor.php b/controller/compras_proveedor.php index ed63324..208c3b4 100644 --- a/controller/compras_proveedor.php +++ b/controller/compras_proveedor.php @@ -82,8 +82,9 @@ public function url() return parent::url(); } else if ($this->proveedor) { return $this->proveedor->url(); - } else - return $this->ppage->url(); + } + + return $this->ppage->url(); } private function modificar() @@ -113,8 +114,9 @@ private function modificar() if ($this->proveedor->save()) { $this->new_message('Datos del proveedor modificados correctamente.'); - } else + } else { $this->new_error_msg('¡Imposible modificar los datos del proveedor!'); + } } private function edit_cuenta_banco() @@ -143,10 +145,12 @@ private function delete_cuenta_banco() if ($cuenta) { if ($cuenta->delete()) { $this->new_message('Cuenta bancaria eliminada correctamente.'); - } else + } else { $this->new_error_msg('Imposible eliminar la cuenta bancaria.'); - } else + } + } else { $this->new_error_msg('Cuenta bancaria no encontrada.'); + } } private function edit_direccion() @@ -167,8 +171,9 @@ private function edit_direccion() $direccion->provincia = $_POST['provincia']; if ($direccion->save()) { $this->new_message("Dirección guardada correctamente."); - } else + } else { $this->new_error_msg("¡Imposible guardar la dirección!"); + } } private function delete_direccion() @@ -178,10 +183,12 @@ private function delete_direccion() if ($dir0) { if ($dir0->delete()) { $this->new_message('Dirección eliminada correctamente.'); - } else + } else { $this->new_error_msg('Imposible eliminar la dirección.'); - } else + } + } else { $this->new_error_msg('Dirección no encontrada.'); + } } /** @@ -211,8 +218,9 @@ private function convertir() if ($cliente->save()) { $this->proveedor->codcliente = $cliente->codcliente; $cliente_ok = $this->proveedor->save(); - } else + } else { $cliente_ok = FALSE; + } if ($cliente_ok) { /* cuentas de banco */ @@ -240,8 +248,9 @@ private function convertir() } $this->new_message('Cliente creado correctamente.'); - } else + } else { $this->new_error_msg("¡Imposible crear el cliente!"); + } } public function tiene_facturas() @@ -257,7 +266,7 @@ public function tiene_facturas() } } - if (!$tiene AND $this->db->table_exists('albaranesprov')) { + if (!$tiene && $this->db->table_exists('albaranesprov')) { $sql = "SELECT * FROM albaranesprov WHERE codproveedor = " . $this->proveedor->var2str($this->proveedor->codproveedor); $data = $this->db->select_limit($sql, 5, 0); diff --git a/controller/compras_trazabilidad.php b/controller/compras_trazabilidad.php index 965e42c..99a098d 100644 --- a/controller/compras_trazabilidad.php +++ b/controller/compras_trazabilidad.php @@ -40,7 +40,7 @@ public function __construct() protected function private_core() { $this->documento = FALSE; - if (isset($_GET['doc']) AND isset($_GET['id'])) { + if (isset($_GET['doc']) && isset($_GET['id'])) { if ($_GET['doc'] == 'albaran') { $alb = new albaran_proveedor(); $this->documento = $alb->get($_GET['id']); @@ -214,7 +214,7 @@ private function modificar() $this->lineas[$i]->lote = $_POST['lote_' . $i]; } - if (is_null($this->lineas[$i]->numserie) AND is_null($this->lineas[$i]->lote)) { + if (is_null($this->lineas[$i]->numserie) && is_null($this->lineas[$i]->lote)) { if ($ok) { $this->new_error_msg('En la linea ' . ($i + 1) . ' debes escribir un número de serie' . ' o un lote o ambos, pero algo debes escribir.'); diff --git a/controller/contabilidad_asiento.php b/controller/contabilidad_asiento.php index 2823f98..87f3c51 100644 --- a/controller/contabilidad_asiento.php +++ b/controller/contabilidad_asiento.php @@ -51,7 +51,7 @@ protected function private_core() $this->asiento = $asiento->get($_GET['id']); } - if (isset($_POST['fecha']) AND isset($_POST['query'])) { + if (isset($_POST['fecha']) && isset($_POST['query'])) { $this->new_search(); } else if ($this->asiento) { $this->page->title = 'Asiento: ' . $this->asiento->numero; @@ -60,7 +60,7 @@ protected function private_core() $this->bloquear(); } else if (isset($_GET['desbloquear'])) { $this->desbloquear(); - } else if (isset($_POST['fecha']) AND $this->asiento->editable) { + } else if (isset($_POST['fecha']) && $this->asiento->editable) { $this->modificar(); } @@ -79,8 +79,9 @@ public function url() return parent::url(); } else if ($this->asiento) { return $this->asiento->url(); - } else - return $this->ppage->url(); + } + + return $this->ppage->url(); } private function new_search() @@ -101,8 +102,9 @@ private function bloquear() $this->asiento->editable = FALSE; if ($this->asiento->save()) { $this->new_message('Asiento bloqueado correctamente.'); - } else + } else { $this->new_error_msg('Imposible bloquear el asiento.'); + } } private function desbloquear() @@ -114,16 +116,16 @@ private function desbloquear() $regiva0 = new regularizacion_iva(); $excluir = array($ejercicio->idasientoapertura, $ejercicio->idasientocierre, $ejercicio->idasientopyg); - if ($regiva0->get_fecha_inside($this->asiento->fecha) AND ! in_array($this->asiento->idasiento, $excluir)) { + if ($regiva0->get_fecha_inside($this->asiento->fecha) && ! in_array($this->asiento->idasiento, $excluir)) { $this->asiento->editable = FALSE; $this->new_error_msg('El asiento está dentro de una regularización de ' . FS_IVA . '. No se puede modificar.'); } else if ($this->asiento->save()) { $this->new_message('Asiento desbloqueado correctamente.'); - } else + } else { $this->new_error_msg('Imposible desbloquear el asiento.'); - } - else { + } + } else { $this->new_error_msg('Imposible desbloquear el asiento: el ejercicio ' . $ejercicio->nombre . ' está cerrado.'); } @@ -139,7 +141,7 @@ private function modificar() if ($ejercicio->abierto()) { $regiva0 = new regularizacion_iva(); $excluir = array($ejercicio->idasientoapertura, $ejercicio->idasientocierre, $ejercicio->idasientopyg); - if ($regiva0->get_fecha_inside($this->asiento->fecha) AND ! in_array($this->asiento->idasiento, $excluir)) { + if ($regiva0->get_fecha_inside($this->asiento->fecha) && ! in_array($this->asiento->idasiento, $excluir)) { $this->new_error_msg('El asiento está dentro de una regularización de ' . FS_IVA . '. No se puede modificar.'); } else { @@ -152,7 +154,7 @@ private function modificar() } } - if ($bloquear AND $this->asiento->editable) { + if ($bloquear && $this->asiento->editable) { $this->asiento->editable = FALSE; $this->asiento->save(); } @@ -184,7 +186,7 @@ private function modificar2(&$eje0) /// obtenemos la divisa de las partidas $div0 = $this->divisa->get($_POST['divisa']); - if (!$eje0 OR ! $div0) { + if (!$eje0 || ! $div0) { $this->new_error_msg('Imposible modificar el asiento.'); } else if ($this->asiento->save()) { $continuar = TRUE; @@ -194,20 +196,16 @@ private function modificar2(&$eje0) foreach ($this->asiento->get_partidas() as $pa) { $encontrada = FALSE; for ($i = 1; $i <= $numlineas; $i++) { - if (isset($_POST['idpartida_' . $i])) { - if (intval($_POST['idpartida_' . $i]) == $pa->idpartida) { - $encontrada = TRUE; - break; - } - } - } - if (!$encontrada) { - if (!$pa->delete()) { - $this->new_error_msg('Imposible eliminar la línea debe=' . $pa->debe . ' haber=' . $pa->haber); - $continuar = FALSE; + if (isset($_POST['idpartida_' . $i]) && intval($_POST['idpartida_' . $i]) == $pa->idpartida) { + $encontrada = TRUE; break; } } + if (!$encontrada && !$pa->delete()) { + $this->new_error_msg('Imposible eliminar la línea debe=' . $pa->debe . ' haber=' . $pa->haber); + $continuar = FALSE; + break; + } } /// añadimos y modificamos @@ -252,19 +250,17 @@ private function modificar2(&$eje0) $partida->cifnif = NULL; $partida->iva = 0; $partida->baseimponible = 0; - if (isset($_POST['codcontrapartida_' . $i])) { - if ($_POST['codcontrapartida_' . $i] != '') { - $subc1 = $this->subcuenta->get_by_codigo($_POST['codcontrapartida_' . $i], $eje0->codejercicio); - if ($subc1) { - $partida->idcontrapartida = $subc1->idsubcuenta; - $partida->codcontrapartida = $subc1->codsubcuenta; - $partida->cifnif = $_POST['cifnif_' . $i]; - $partida->iva = floatval($_POST['iva_' . $i]); - $partida->baseimponible = floatval($_POST['baseimp_' . $i]); - } else { - $this->new_error_msg('Subcuenta ' . $_POST['codcontrapartida_' . $i] . ' no encontrada.'); - $continuar = FALSE; - } + if (isset($_POST['codcontrapartida_' . $i]) && $_POST['codcontrapartida_' . $i] != '') { + $subc1 = $this->subcuenta->get_by_codigo($_POST['codcontrapartida_' . $i], $eje0->codejercicio); + if ($subc1) { + $partida->idcontrapartida = $subc1->idsubcuenta; + $partida->codcontrapartida = $subc1->codsubcuenta; + $partida->cifnif = $_POST['cifnif_' . $i]; + $partida->iva = floatval($_POST['iva_' . $i]); + $partida->baseimponible = floatval($_POST['baseimp_' . $i]); + } else { + $this->new_error_msg('Subcuenta ' . $_POST['codcontrapartida_' . $i] . ' no encontrada.'); + $continuar = FALSE; } } @@ -276,8 +272,9 @@ private function modificar2(&$eje0) $this->new_error_msg('Subcuenta ' . $_POST['codsubcuenta_' . $i] . ' de la línea ' . $i . ' no encontrada.'); $continuar = FALSE; } - } else + } else { break; + } } } @@ -290,8 +287,9 @@ private function modificar2(&$eje0) $this->new_message('Asiento modificado correctamente.'); $this->new_change('Asiento ' . $this->asiento->numero, $this->asiento->url()); } - } else + } else { $this->new_error_msg('Imposible modificar el asiento.'); + } } private function get_lineas_asiento() diff --git a/controller/contabilidad_asientos.php b/controller/contabilidad_asientos.php index 9df7f2f..79e54ae 100644 --- a/controller/contabilidad_asientos.php +++ b/controller/contabilidad_asientos.php @@ -73,7 +73,7 @@ private function ini_filters() $this->desde = ''; $this->hasta = ''; $this->orden = 'fecha DESC, numero DESC'; - if (isset($_REQUEST['desde']) OR isset($_REQUEST['hasta']) OR isset($_REQUEST['orden'])) { + if (isset($_REQUEST['desde']) || isset($_REQUEST['hasta']) || isset($_REQUEST['orden'])) { $this->desde = $_REQUEST['desde']; $this->hasta = $_REQUEST['hasta']; $this->orden = $_REQUEST['orden']; @@ -139,9 +139,9 @@ public function url($busqueda = FALSE) . "&desde=" . $this->desde . "&hasta=" . $this->hasta . "&orden=" . $this->orden; - } else { - return parent::url(); } + + return parent::url(); } private function eliminar_asiento() @@ -153,9 +153,11 @@ private function eliminar_asiento() } else if ($asiento->delete()) { $this->new_message("Asiento eliminado correctamente. (ID: " . $asiento->idasiento . ", " . $asiento->fecha . ")", TRUE); $this->clean_last_changes(); - } else + } else { $this->new_error_msg("¡Imposible eliminar el asiento!"); - } else + } + } else { $this->new_error_msg("¡Asiento no encontrado!"); + } } } diff --git a/controller/contabilidad_cuenta.php b/controller/contabilidad_cuenta.php index 2914412..7700996 100644 --- a/controller/contabilidad_cuenta.php +++ b/controller/contabilidad_cuenta.php @@ -81,7 +81,7 @@ private function cargar_cuenta() { $cuenta = new cuenta(); $this->cuenta = $cuenta->get($_GET['id']); - if ($this->cuenta AND isset($_POST['descripcion'])) { + if ($this->cuenta && isset($_POST['descripcion'])) { $this->cuenta->descripcion = $_POST['descripcion']; if ($_POST['idcuentaesp'] == '---') { $this->cuenta->idcuentaesp = NULL; diff --git a/controller/contabilidad_cuentas.php b/controller/contabilidad_cuentas.php index 802477f..c97c4c9 100644 --- a/controller/contabilidad_cuentas.php +++ b/controller/contabilidad_cuentas.php @@ -72,9 +72,9 @@ protected function private_core() public function anterior_url() { $url = ''; - if ($this->query != '' AND $this->offset > 0) { + if ($this->query != '' && $this->offset > 0) { $url = $this->url() . "&query=" . $this->query . "&offset=" . ($this->offset - FS_ITEM_LIMIT); - } else if ($this->query == '' AND $this->offset > 0) { + } else if ($this->query == '' && $this->offset > 0) { $url = $this->url() . "&offset=" . ($this->offset - FS_ITEM_LIMIT); } return $url; @@ -83,9 +83,9 @@ public function anterior_url() public function siguiente_url() { $url = ''; - if ($this->query != '' AND count($this->resultados) == FS_ITEM_LIMIT) { + if ($this->query != '' && count($this->resultados) == FS_ITEM_LIMIT) { $url = $this->url() . "&query=" . $this->query . "&offset=" . ($this->offset + FS_ITEM_LIMIT); - } else if ($this->query == '' AND count($this->resultados) == FS_ITEM_LIMIT) { + } else if ($this->query == '' && count($this->resultados) == FS_ITEM_LIMIT) { $url = $this->url() . "&offset=" . ($this->offset + FS_ITEM_LIMIT); } return $url; diff --git a/controller/contabilidad_ejercicio.php b/controller/contabilidad_ejercicio.php index 73f3dab..9f8b494 100644 --- a/controller/contabilidad_ejercicio.php +++ b/controller/contabilidad_ejercicio.php @@ -42,9 +42,9 @@ protected function private_core() /// cargamos las putas secuencias para que se actualicen. /// Abanq/Eneboo, yo te maldigooooo!!!!!!!!!!!!!!!!!!!!!! - $sec0 = new secuencia_ejercicio(); - $sec1 = new secuencia_contabilidad(); - $sec2 = new secuencia(); + new secuencia_ejercicio(); + new secuencia_contabilidad(); + new secuencia(); $this->ejercicio = FALSE; if (isset($_POST['codejercicio'])) { @@ -73,7 +73,7 @@ protected function private_core() } else { $this->page->title = $this->ejercicio->codejercicio . ' (' . $this->ejercicio->nombre . ')'; - if (isset($_GET['cerrar']) AND isset($_GET['petid'])) { + if (isset($_GET['cerrar']) && isset($_GET['petid'])) { if ($this->duplicated_petition($_GET['petid'])) { $this->new_error_msg('Petición duplicada. Evita hacer doble clic sobre los botones.'); } else @@ -290,7 +290,7 @@ private function importar_xml() unlink('tmp/' . FS_TMP_NAME . 'ejercicio.xml'); } - if ($_POST['fuente'] == 'archivo' AND isset($_POST['archivo'])) { + if ($_POST['fuente'] == 'archivo' && isset($_POST['archivo'])) { if (copy($_FILES['farchivo']['tmp_name'], 'tmp/' . FS_TMP_NAME . 'ejercicio.xml')) { $import_step = 1; $this->url_recarga = $this->url() . '&importar=' . (1 + $import_step); @@ -318,7 +318,7 @@ private function importar_xml() } } - if (file_exists('tmp/' . FS_TMP_NAME . 'ejercicio.xml') AND $import_step > 0) { + if (file_exists('tmp/' . FS_TMP_NAME . 'ejercicio.xml') && $import_step > 0) { $offset = 0; if (isset($_GET['offset'])) { $offset = intval($_GET['offset']); @@ -334,7 +334,7 @@ private function importar_xml() $xml = simplexml_load_file('tmp/' . FS_TMP_NAME . 'ejercicio.xml'); if ($xml) { - if ($xml->balance AND $import_step == 1) { + if ($xml->balance && $import_step == 1) { foreach ($xml->balance as $b) { $balance = new balance(); if (!$balance->get($b->codbalance)) { @@ -363,7 +363,7 @@ private function importar_xml() foreach ($xml->balance_cuenta as $bc) { $encontrado = FALSE; foreach ($all_bcs as $bc2) { - if ($bc2->codbalance == $bc->codbalance AND $bc2->codcuenta == $bc->codcuenta) { + if ($bc2->codbalance == $bc->codbalance && $bc2->codcuenta == $bc->codcuenta) { $encontrado = TRUE; break; } @@ -387,7 +387,7 @@ private function importar_xml() foreach ($xml->balance_cuenta_a as $bc) { $encontrado = FALSE; foreach ($all_bcas as $bc2) { - if ($bc2->codbalance == $bc->codbalance AND $bc2->codcuenta == $bc->codcuenta) { + if ($bc2->codbalance == $bc->codbalance && $bc2->codcuenta == $bc->codcuenta) { $encontrado = TRUE; break; } @@ -466,7 +466,7 @@ private function importar_xml() } } - if ($xml->cuenta AND $import_step == 3) { + if ($xml->cuenta && $import_step == 3) { $epigrafe = new epigrafe(); foreach ($xml->cuenta as $c) { $cuenta = new cuenta(); @@ -488,7 +488,7 @@ private function importar_xml() } } - if ($xml->subcuenta AND $import_step == 4) { + if ($xml->subcuenta && $import_step == 4) { $cuenta = new cuenta(); foreach ($xml->subcuenta as $sc) { $subcuenta = new subcuenta(); @@ -534,7 +534,7 @@ private function importar_xml() } } - if ($error OR count($this->get_errors()) > 0) { + if ($error || count($this->get_errors()) > 0) { $this->new_error_msg('Proceso detenido.'); $this->url_recarga = FALSE; break; @@ -562,7 +562,7 @@ private function importar_xml() } } - if ($error OR count($this->get_errors()) > 0) { + if ($error || count($this->get_errors()) > 0) { $this->new_error_msg('Proceso detenido.'); $this->url_recarga = FALSE; break; diff --git a/controller/contabilidad_epigrafes.php b/controller/contabilidad_epigrafes.php index 6bdb710..1b6f616 100644 --- a/controller/contabilidad_epigrafes.php +++ b/controller/contabilidad_epigrafes.php @@ -117,7 +117,7 @@ private function ver_grupo(&$grupo0) { $this->epigrafe = FALSE; $this->grupo = $grupo0->get($_GET['grupo']); - if ($this->grupo AND isset($_POST['descripcion'])) { + if ($this->grupo && isset($_POST['descripcion'])) { $this->grupo->descripcion = $_POST['descripcion']; if ($this->grupo->save()) { $this->new_message('Grupo modificado correctamente.'); @@ -174,7 +174,7 @@ private function ver_epigrafe(&$epi0) { $this->grupo = FALSE; $this->epigrafe = $epi0->get($_GET['epi']); - if ($this->ejercicio AND isset($_POST['descripcion'])) { + if ($this->ejercicio && isset($_POST['descripcion'])) { $this->epigrafe->descripcion = $_POST['descripcion']; if ($this->epigrafe->save()) { diff --git a/controller/contabilidad_formas_pago.php b/controller/contabilidad_formas_pago.php index 1c76d79..82b1944 100644 --- a/controller/contabilidad_formas_pago.php +++ b/controller/contabilidad_formas_pago.php @@ -42,7 +42,7 @@ protected function private_core() */ $this->button_plazos = FALSE; foreach ($this->extensions as $ext) { - if ($ext->type == 'config' AND $ext->text == 'button_plazos') { + if ($ext->type == 'config' && $ext->text == 'button_plazos') { $this->button_plazos = $ext->from; break; } @@ -90,12 +90,12 @@ private function editar_forma_pago() } } - if ($fp0->codpago == '' OR $fp0->descripcion == '') { + if ($fp0->codpago == '' || $fp0->descripcion == '') { $this->new_error_msg('Debes poner algún nombre a la forma de pago.'); } else if ($fp0->save()) { $this->new_message('Forma de pago ' . $fp0->codpago . ' guardada correctamente.'); - if ($nueva AND $this->button_plazos AND $fp0->genrecibos == 'Emitidos') { + if ($nueva && $this->button_plazos && $fp0->genrecibos == 'Emitidos') { header('Location: index.php?page=' . $this->button_plazos . '&cod=' . $fp0->codpago . '&nueva=TRUE'); } } else { diff --git a/controller/contabilidad_nuevo_asiento.php b/controller/contabilidad_nuevo_asiento.php index 8eeff3c..35fafce 100644 --- a/controller/contabilidad_nuevo_asiento.php +++ b/controller/contabilidad_nuevo_asiento.php @@ -49,9 +49,9 @@ protected function private_core() $this->resultados = array(); $this->subcuenta = new subcuenta(); - if (isset($_POST['fecha']) AND isset($_POST['query'])) { + if (isset($_POST['fecha']) && isset($_POST['query'])) { $this->new_search(); - } else if (isset($_POST['fecha']) AND isset($_POST['concepto']) AND isset($_POST['divisa'])) { + } else if (isset($_POST['fecha']) && isset($_POST['concepto']) && isset($_POST['divisa'])) { if ($_POST['autonomo'] != '0') { if (floatval($_POST['autonomo']) > 0) { $this->nuevo_asiento_autonomo(); @@ -76,8 +76,6 @@ protected function private_core() private function get_ejercicio($fecha) { - $ejercicio = FALSE; - $ejercicio = $this->ejercicio->get_by_fecha($fecha); if ($ejercicio) { $regiva0 = new regularizacion_iva(); @@ -126,7 +124,7 @@ private function nuevo_asiento() $numlineas = intval($_POST['numlineas']); for ($i = 1; $i <= $numlineas; $i++) { if (isset($_POST['codsubcuenta_' . $i])) { - if ($_POST['codsubcuenta_' . $i] != '' AND $continuar) { + if ($_POST['codsubcuenta_' . $i] != '' && $continuar) { $sub0 = $this->subcuenta->get_by_codigo($_POST['codsubcuenta_' . $i], $eje0->codejercicio); if ($sub0) { $partida = new partida(); diff --git a/controller/contabilidad_series.php b/controller/contabilidad_series.php index 14375c1..2cdae13 100644 --- a/controller/contabilidad_series.php +++ b/controller/contabilidad_series.php @@ -77,7 +77,7 @@ private function editar_serie() $serie->irpf = floatval($_POST['irpf']); if ($this->num_personalizada) { - if ($_POST['codejercicio'] != $serie->codejercicio OR $_POST['numfactura'] != $serie->numfactura) { + if ($_POST['codejercicio'] != $serie->codejercicio || $_POST['numfactura'] != $serie->numfactura) { if ($this->user->admin) { if ($this->hay_facturas_venta($serie->codserie)) { $this->new_error_msg('Ya hay facturas con esta serie, no puedes cambiar la numeración inicial.'); diff --git a/controller/contabilidad_subcuenta.php b/controller/contabilidad_subcuenta.php index 0cd81d6..c544c09 100644 --- a/controller/contabilidad_subcuenta.php +++ b/controller/contabilidad_subcuenta.php @@ -89,8 +89,9 @@ public function url() return parent::url(); } else if ($this->subcuenta) { return $this->subcuenta->url(); - } else - return $this->ppage->url(); + } + + return $this->ppage->url(); } public function paginas() @@ -109,8 +110,9 @@ private function modificar() foreach ($this->resultados as $pa) { if (isset($_POST['punteada'])) { $valor = in_array($pa->idpartida, $_POST['punteada']); - } else + } else { $valor = FALSE; + } if ($pa->punteada != $valor) { $pa->punteada = $valor; diff --git a/controller/dashboard.php b/controller/dashboard.php index 30bcce8..a46ec21 100644 --- a/controller/dashboard.php +++ b/controller/dashboard.php @@ -211,7 +211,7 @@ private function calcula_periodo(&$stats, $pendiente = FALSE) $this->calcular_facturas($stats, $desde_anterior, $hasta_anterior, TRUE, 'facturasprov'); } - if ($this->db->table_exists('co_partidas') AND $this->empresa->codpais == 'ESP') { + if ($this->db->table_exists('co_partidas') && $this->empresa->codpais == 'ESP') { /// calculamos el saldo de todos aquellos asientos que afecten a caja y no se correspondan con facturas $sql = "select sum(debe-haber) as total from co_partidas where codsubcuenta LIKE '57%' and idasiento" . " in (select idasiento from co_asientos where tipodocumento IS NULL" @@ -398,23 +398,24 @@ private function calcular_facturas(&$stats, $desde, $hasta, $anterior = FALSE, $ private function calcular_diff($nuevo, $anterior) { - if ($nuevo == 0 OR $anterior == 0) { - if ($nuevo == 0 AND $anterior > 0) { + if ($nuevo == 0 || $anterior == 0) { + if ($nuevo == 0 && $anterior > 0) { return -100; - } else if ($nuevo > 0 AND $anterior == 0) { + } else if ($nuevo > 0 && $anterior == 0) { return 100; - } else - return 0; + } + + return 0; } else if ($nuevo != $anterior) { if ($anterior < 0) { return 0 - (($nuevo * 100 / $anterior) - 100); - } else { - return ($nuevo * 100 / $anterior) - 100; } - } else { - return 0; + + return ($nuevo * 100 / $anterior) - 100; } + + return 0; } private function elegir_consejos() diff --git a/controller/editar_balances.php b/controller/editar_balances.php index f7ffde6..986b342 100644 --- a/controller/editar_balances.php +++ b/controller/editar_balances.php @@ -55,7 +55,7 @@ protected function private_core() $bc0 = new balance_cuenta(); $bca0 = new balance_cuenta_a(); - if (isset($_POST['nueva_cuenta']) OR isset($_POST['nueva_cuenta_a'])) { + if (isset($_POST['nueva_cuenta']) || isset($_POST['nueva_cuenta_a'])) { if ($_POST['nueva_cuenta']) { $this->nuevo_balance_cuenta($bc0); } else if ($_POST['nueva_cuenta_a']) { @@ -65,7 +65,7 @@ protected function private_core() $this->eliminar_balance_cuenta($bc0); } else if (isset($_GET['rm_cuenta_a'])) { $this->eliminar_balance_cuenta_abreviado($bca0); - } else if (isset($_POST['descripcion']) AND ! isset($_POST['ncodbalance'])) { + } else if (isset($_POST['descripcion']) && ! isset($_POST['ncodbalance'])) { $this->editar_balance(); } diff --git a/controller/informe_albaranes.php b/controller/informe_albaranes.php index f5e249b..8996fbe 100644 --- a/controller/informe_albaranes.php +++ b/controller/informe_albaranes.php @@ -77,8 +77,8 @@ protected function private_core() parent::private_core(); /// declaramos los objetos sólo para asegurarnos de que existen las tablas - $albaran_cli = new albaran_cliente(); - $albaran_pro = new albaran_proveedor(); + new albaran_cliente(); + new albaran_proveedor(); $this->agente = new agente(); $this->almacen = new almacen(); @@ -802,7 +802,7 @@ protected function generar_pdf($tipo = 'compra') ) ); - for ($i = 0; $i < $lppag AND $linea_actual < $total_lineas; $i++) { + for ($i = 0; $i < $lppag && $linea_actual < $total_lineas; $i++) { $linea = array( 'serie' => $documentos[$linea_actual]->codserie, 'doc' => $documentos[$linea_actual]->codigo, diff --git a/controller/informe_articulos.php b/controller/informe_articulos.php index 1d1b827..1a4a056 100644 --- a/controller/informe_articulos.php +++ b/controller/informe_articulos.php @@ -78,15 +78,15 @@ protected function private_core() $this->stats = $this->stats(); /// forzamos la comprobación de las tablas de lineas de facturas - $linea_fac_cli = new linea_factura_cliente(); - $linea_fac_pro = new linea_factura_proveedor(); + new linea_factura_cliente(); + new linea_factura_proveedor(); $this->top_ventas = $this->top_articulo_faccli(); $this->sin_vender = $this->sin_vender(); $this->top_compras = $this->top_articulo_facpro(); } else if ($this->pestanya == 'stock') { /// forzamos la comprobación de la tabla stock - $stock = new stock(); + new stock(); $this->tipo_stock = 'todo'; if (isset($_GET['tipo'])) { @@ -97,7 +97,7 @@ protected function private_core() if ($this->tipo_stock == 'reg') { /// forzamos la comprobación de la tabla stocks - $reg = new regularizacion_stock(); + new regularizacion_stock(); $this->resultados = $this->regularizaciones_stock($this->offset); } else if (isset($_GET['download'])) { @@ -300,7 +300,7 @@ private function top_articulo_faccli() { /// buscamos el resultado en caché $toplist = $this->cache->get_array('faccli_top_articulos'); - if (!$toplist OR isset($_POST['desde'])) { + if (!$toplist || isset($_POST['desde'])) { $toplist = array(); $articulo = new articulo(); $sql = "SELECT l.referencia, SUM(l.cantidad) as unidades, SUM(l.pvptotal/f.tasaconv) as total" @@ -362,7 +362,7 @@ private function sin_vender() private function top_articulo_facpro() { $toplist = $this->cache->get('facpro_top_articulos'); - if (!$toplist OR isset($_POST['desde'])) { + if (!$toplist || isset($_POST['desde'])) { $articulo = new articulo(); $sql = "SELECT l.referencia, SUM(l.cantidad) as compras FROM lineasfacturasprov l, facturasprov f" . " WHERE l.idfactura = f.idfactura AND l.referencia IS NOT NULL" diff --git a/controller/informe_contabilidad.php b/controller/informe_contabilidad.php index a728e27..8997b3a 100644 --- a/controller/informe_contabilidad.php +++ b/controller/informe_contabilidad.php @@ -45,14 +45,15 @@ protected function private_core() $this->buscar_subcuenta(); } else if (isset($_GET['diario'])) { $this->libro_diario_csv($_GET['diario']); - } else if (isset($_GET['balance']) AND isset($_GET['eje'])) { + } else if (isset($_GET['balance']) && isset($_GET['eje'])) { $this->template = FALSE; $iba = new inventarios_balances($this->db); if ($_GET['balance'] == 'pyg') { $iba->generar_pyg($_GET['eje']); - } else + } else { $iba->generar_sit($_GET['eje']); + } } else if (isset($_POST['informe'])) { if ($_POST['informe'] == 'sumasysaldos') { $this->balance_sumas_y_saldos(); @@ -164,7 +165,7 @@ private function libro_diario_csv($codeje, $desde = FALSE, $hasta = FALSE) . " WHERE a.codejercicio = " . $this->empresa->var2str($codeje) . " AND p.idasiento = a.idasiento"; - if ($desde AND $hasta) { + if ($desde && $hasta) { $sql .= " AND a.fecha >= " . $this->empresa->var2str($desde); $sql .= " AND a.fecha <= " . $this->empresa->var2str($hasta); } @@ -267,13 +268,13 @@ private function balance_sumas_y_saldos() { $eje = $this->ejercicio->get($_POST['codejercicio']); if ($eje) { - if (strtotime($_POST['desde']) < strtotime($eje->fechainicio) OR strtotime($_POST['hasta']) > strtotime($eje->fechafin)) { + if (strtotime($_POST['desde']) < strtotime($eje->fechainicio) || strtotime($_POST['hasta']) > strtotime($eje->fechafin)) { $this->new_error_msg('La fecha está fuera del rango del ejercicio.'); } else { $this->template = FALSE; $excluir = FALSE; - if (isset($eje->idasientocierre) AND isset($eje->idasientopyg)) { + if (isset($eje->idasientocierre) && isset($eje->idasientopyg)) { $excluir = array($eje->idasientocierre, $eje->idasientopyg); } @@ -402,7 +403,7 @@ public function sumas_y_saldos(&$pdf_doc, &$eje, $tipo = 3, $titulo, $fechaini, } /// añadimos la cuenta - if ($debe != 0 OR $haber != 0) { + if ($debe != 0 || $haber != 0) { $cuenta = $cuenta0->get_by_codigo($i . $j . $k, $eje->codejercicio); if ($cuenta) { $lineas[] = array( @@ -542,7 +543,7 @@ private function balance_situacion() { $eje = $this->ejercicio->get($_POST['codejercicio']); if ($eje) { - if (strtotime($_POST['desde']) < strtotime($eje->fechainicio) OR strtotime($_POST['hasta']) > strtotime($eje->fechafin)) { + if (strtotime($_POST['desde']) < strtotime($eje->fechainicio) || strtotime($_POST['hasta']) > strtotime($eje->fechafin)) { $this->new_error_msg('La fecha está fuera del rango del ejercicio.'); } else { $this->template = FALSE; @@ -598,14 +599,13 @@ private function situacion(&$pdf_doc, &$eje) $desc1 = ''; $desc2 = ''; $desc3 = ''; - $desc4 = ''; foreach ($nivel1 as $nv1) { foreach ($nivel2 as $nv2) { foreach ($nivel3 as $nv3) { foreach ($nivel4 as $nv4) { foreach ($balances as $bal) { - if ($bal->naturaleza == $nv0 AND $bal->nivel1 == $nv1 AND $bal->nivel2 == $nv2 AND $bal->nivel3 == $nv3 AND $bal->nivel4 == $nv4) { - if ($bal->descripcion1 != $desc1 AND $bal->descripcion1 != '') { + if ($bal->naturaleza == $nv0 && $bal->nivel1 == $nv1 && $bal->nivel2 == $nv2 && $bal->nivel3 == $nv3 && $bal->nivel4 == $nv4) { + if ($bal->descripcion1 != $desc1 && $bal->descripcion1 != '') { $pdf_doc->add_table_row( array( 'descripcion' => "\n" . $bal->descripcion1 . '', @@ -616,7 +616,7 @@ private function situacion(&$pdf_doc, &$eje) $desc1 = $bal->descripcion1; } - if ($bal->descripcion2 != $desc2 AND $bal->descripcion2 != '') { + if ($bal->descripcion2 != $desc2 && $bal->descripcion2 != '') { $pdf_doc->add_table_row( array( 'descripcion' => ' ' . $bal->descripcion2 . '', @@ -627,7 +627,7 @@ private function situacion(&$pdf_doc, &$eje) $desc2 = $bal->descripcion2; } - if ($bal->descripcion3 != $desc3 AND $bal->descripcion3 != '') { + if ($bal->descripcion3 != $desc3 && $bal->descripcion3 != '') { $pdf_doc->add_table_row( array( 'descripcion' => ' ' . $bal->descripcion3, @@ -679,7 +679,7 @@ private function balance_perdidasyg() { $eje = $this->ejercicio->get($_POST['codejercicio']); if ($eje) { - if (strtotime($_POST['desde']) < strtotime($eje->fechainicio) OR strtotime($_POST['hasta']) > strtotime($eje->fechafin)) { + if (strtotime($_POST['desde']) < strtotime($eje->fechainicio) || strtotime($_POST['hasta']) > strtotime($eje->fechafin)) { $this->new_error_msg('La fecha está fuera del rango del ejercicio.'); } else { $this->template = FALSE; @@ -744,7 +744,7 @@ private function perdidas_y_ganancias(&$pdf_doc, &$eje) $encontrado = FALSE; foreach ($balances as $bal) { - if ($bal->naturaleza == 'PG' AND strstr($bal->codbalance, 'PG-A-' . $num . '-')) { + if ($bal->naturaleza == 'PG' && strstr($bal->codbalance, 'PG-A-' . $num . '-')) { $saldo1 = $this->get_saldo_balance('PG-A-' . $num . '-', $eje); /// añadimos la fila diff --git a/controller/informe_errores.php b/controller/informe_errores.php index 0ad90a8..8f14319 100644 --- a/controller/informe_errores.php +++ b/controller/informe_errores.php @@ -124,7 +124,7 @@ private function continuar_test() while (!feof($file)) { $linea = explode(';', trim(fgets($file))); if (count($linea) == 7) { - if ($numlinea > $this->informe['show_page'] * FS_ITEM_LIMIT AND $numlinea <= (1 + $this->informe['show_page']) * FS_ITEM_LIMIT) { + if ($numlinea > $this->informe['show_page'] * FS_ITEM_LIMIT && $numlinea <= (1 + $this->informe['show_page']) * FS_ITEM_LIMIT) { $this->errores[] = array( 'error' => $linea[0], 'model' => $linea[1], @@ -412,7 +412,7 @@ public function all_pages() /// ahora descartamos foreach ($allp as $j => $value) { - if (($value['num'] > 1 AND $j < $show_p - 3 AND $value['num'] % 10) OR ( $j > $show_p + 3 AND $j < $i - 1 AND $value['num'] % 10)) { + if (($value['num'] > 1 && $j < $show_p - 3 && $value['num'] % 10) || ( $j > $show_p + 3 && $j < $i - 1 && $value['num'] % 10)) { unset($allp[$j]); } } diff --git a/controller/informe_facturas.php b/controller/informe_facturas.php index 4431e9e..e53d5de 100644 --- a/controller/informe_facturas.php +++ b/controller/informe_facturas.php @@ -37,8 +37,8 @@ public function __construct() protected function private_core() { /// declaramos los objetos sólo para asegurarnos de que existen las tablas - $factura_cli = new factura_cliente(); - $factura_pro = new factura_proveedor(); + new factura_cliente(); + new factura_proveedor(); $this->nombre_docs = 'Facturas'; $this->pais = new pais(); @@ -829,7 +829,7 @@ protected function generar_pdf($tipo = 'compra') ) ); - for ($i = 0; $i < $lppag AND $linea_actual < $total_lineas; $i++) { + for ($i = 0; $i < $lppag && $linea_actual < $total_lineas; $i++) { $linea = array( 'serie' => $documentos[$linea_actual]->codserie, 'doc' => $documentos[$linea_actual]->codigo, diff --git a/controller/nueva_compra.php b/controller/nueva_compra.php index 3aa52f1..9421b17 100644 --- a/controller/nueva_compra.php +++ b/controller/nueva_compra.php @@ -78,30 +78,28 @@ protected function private_core() } else if (isset($_POST['proveedor'])) { $this->proveedor_s = $this->proveedor->get($_POST['proveedor']); - if (isset($_POST['nuevo_proveedor'])) { - if ($_POST['nuevo_proveedor'] != '') { - $this->proveedor_s = FALSE; - if ($_POST['nuevo_cifnif'] != '') { - $this->proveedor_s = $this->proveedor->get_by_cifnif($_POST['nuevo_cifnif']); - if ($this->proveedor_s) { - $this->new_advice('Ya existe un proveedor con ese ' . FS_CIFNIF . '. Se ha seleccionado.'); - } + if (isset($_POST['nuevo_proveedor']) && $_POST['nuevo_proveedor'] != '') { + $this->proveedor_s = FALSE; + if ($_POST['nuevo_cifnif'] != '') { + $this->proveedor_s = $this->proveedor->get_by_cifnif($_POST['nuevo_cifnif']); + if ($this->proveedor_s) { + $this->new_advice('Ya existe un proveedor con ese ' . FS_CIFNIF . '. Se ha seleccionado.'); } + } - if (!$this->proveedor_s) { - $this->proveedor_s = new proveedor(); - $this->proveedor_s->codproveedor = $this->proveedor_s->get_new_codigo(); - $this->proveedor_s->nombre = $this->proveedor_s->razonsocial = $_POST['nuevo_proveedor']; - $this->proveedor_s->tipoidfiscal = $_POST['nuevo_tipoidfiscal']; - $this->proveedor_s->cifnif = $_POST['nuevo_cifnif']; - $this->proveedor_s->acreedor = isset($_POST['acreedor']); - $this->proveedor_s->personafisica = isset($_POST['personafisica']); - $this->proveedor_s->save(); - - if ($this->empresa->contintegrada) { - /// forzamos crear la subcuenta - $this->proveedor_s->get_subcuenta($this->empresa->codejercicio); - } + if (!$this->proveedor_s) { + $this->proveedor_s = new proveedor(); + $this->proveedor_s->codproveedor = $this->proveedor_s->get_new_codigo(); + $this->proveedor_s->nombre = $this->proveedor_s->razonsocial = $_POST['nuevo_proveedor']; + $this->proveedor_s->tipoidfiscal = $_POST['nuevo_tipoidfiscal']; + $this->proveedor_s->cifnif = $_POST['nuevo_cifnif']; + $this->proveedor_s->acreedor = isset($_POST['acreedor']); + $this->proveedor_s->personafisica = isset($_POST['personafisica']); + $this->proveedor_s->save(); + + if ($this->empresa->contintegrada) { + /// forzamos crear la subcuenta + $this->proveedor_s->get_subcuenta($this->empresa->codejercicio); } } } @@ -139,7 +137,7 @@ public function tipos_a_guardar() { $tipos = array(); - if ($this->user->have_access_to('compras_pedido') AND class_exists('pedido_proveedor')) { + if ($this->user->have_access_to('compras_pedido') && class_exists('pedido_proveedor')) { $tipos[] = array('tipo' => 'pedido', 'nombre' => ucfirst(FS_PEDIDO) . ' de compra'); } @@ -203,7 +201,7 @@ private function new_articulo() $art0->codfabricante = $_REQUEST['codfabricante']; } - if ($_POST['refproveedor'] != '' AND $_POST['refproveedor'] != $_POST['referencia']) { + if ($_POST['refproveedor'] != '' && $_POST['refproveedor'] != $_POST['referencia']) { $art0->equivalencia = $_POST['refproveedor']; } @@ -266,7 +264,7 @@ private function new_search() /// añadimos el stock del almacén y el general $this->results[$i]->stockalm = $this->results[$i]->stockfis; - if ($this->multi_almacen AND isset($_REQUEST['codalmacen'])) { + if ($this->multi_almacen && isset($_REQUEST['codalmacen'])) { $this->results[$i]->stockalm = $stock->total_from_articulo($this->results[$i]->referencia, $_REQUEST['codalmacen']); } @@ -280,7 +278,7 @@ private function new_search() /// ejecutamos las funciones de las extensiones foreach ($this->extensions as $ext) { - if ($ext->type == 'function' AND $ext->params == 'new_search') { + if ($ext->type == 'function' && $ext->params == 'new_search') { $name = $ext->text; $name($this->db, $this->results); } @@ -409,7 +407,7 @@ private function nuevo_pedido_proveedor() $linea->idpedido = $pedido->idpedido; $linea->descripcion = $_POST['desc_' . $i]; - if (!$serie->siniva AND $proveedor->regimeniva != 'Exento') { + if (!$serie->siniva && $proveedor->regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $i]); if ($imp0) { $linea->codimpuesto = $imp0->codimpuesto; @@ -438,7 +436,7 @@ private function nuevo_pedido_proveedor() if ($linea->save()) { if ($articulo && isset($_POST['costemedio'])) { - if ($articulo->costemedio == 0 AND $linea->cantidad > 0) { + if ($articulo->costemedio == 0 && $linea->cantidad > 0) { $articulo->costemedio = $linea->pvptotal / $linea->cantidad; $articulo->save(); } @@ -580,7 +578,7 @@ private function nuevo_albaran_proveedor() $linea->idalbaran = $albaran->idalbaran; $linea->descripcion = $_POST['desc_' . $i]; - if (!$serie->siniva AND $proveedor->regimeniva != 'Exento') { + if (!$serie->siniva && $proveedor->regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $i]); if ($imp0) { $linea->codimpuesto = $imp0->codimpuesto; @@ -772,7 +770,7 @@ private function nueva_factura_proveedor() $linea->idfactura = $factura->idfactura; $linea->descripcion = $_POST['desc_' . $i]; - if (!$serie->siniva AND $proveedor->regimeniva != 'Exento') { + if (!$serie->siniva && $proveedor->regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $i]); if ($imp0) { $linea->codimpuesto = $imp0->codimpuesto; @@ -939,7 +937,7 @@ private function search_from_proveedor() $sql .= $separador . " stockfis > 0"; } - if (isset($_REQUEST['solo_proveedor']) AND isset($_REQUEST['codproveedor'])) { + if (isset($_REQUEST['solo_proveedor']) && isset($_REQUEST['codproveedor'])) { $sql .= $separador . " referencia IN (SELECT referencia FROM articulosprov WHERE codproveedor = " . $this->articulo_prov->var2str($_REQUEST['codproveedor']) . ")"; } diff --git a/controller/nueva_venta.php b/controller/nueva_venta.php index 455b3e3..1f615c0 100644 --- a/controller/nueva_venta.php +++ b/controller/nueva_venta.php @@ -114,78 +114,76 @@ protected function private_core() /** * Nuevo cliente */ - if (isset($_POST['nuevo_cliente'])) { - if ($_POST['nuevo_cliente'] != '') { - $this->cliente_s = FALSE; - if ($_POST['nuevo_cifnif'] != '') { - $this->cliente_s = $this->cliente->get_by_cifnif($_POST['nuevo_cifnif']); - if ($this->cliente_s) { - $this->new_advice('Ya existe un cliente con ese ' . FS_CIFNIF . '. Se ha seleccionado.'); - } + if (isset($_POST['nuevo_cliente']) && $_POST['nuevo_cliente'] != '') { + $this->cliente_s = FALSE; + if ($_POST['nuevo_cifnif'] != '') { + $this->cliente_s = $this->cliente->get_by_cifnif($_POST['nuevo_cifnif']); + if ($this->cliente_s) { + $this->new_advice('Ya existe un cliente con ese ' . FS_CIFNIF . '. Se ha seleccionado.'); } + } - if (!$this->cliente_s) { - $this->cliente_s = new cliente(); - $this->cliente_s->codcliente = $this->cliente_s->get_new_codigo(); - $this->cliente_s->nombre = $this->cliente_s->razonsocial = $_POST['nuevo_cliente']; - $this->cliente_s->tipoidfiscal = $_POST['nuevo_tipoidfiscal']; - $this->cliente_s->cifnif = $_POST['nuevo_cifnif']; - $this->cliente_s->personafisica = isset($_POST['personafisica']); + if (!$this->cliente_s) { + $this->cliente_s = new cliente(); + $this->cliente_s->codcliente = $this->cliente_s->get_new_codigo(); + $this->cliente_s->nombre = $this->cliente_s->razonsocial = $_POST['nuevo_cliente']; + $this->cliente_s->tipoidfiscal = $_POST['nuevo_tipoidfiscal']; + $this->cliente_s->cifnif = $_POST['nuevo_cifnif']; + $this->cliente_s->personafisica = isset($_POST['personafisica']); - if (isset($_POST['nuevo_email'])) { - $this->cliente_s->email = $_POST['nuevo_email']; - } + if (isset($_POST['nuevo_email'])) { + $this->cliente_s->email = $_POST['nuevo_email']; + } - if (isset($_POST['codgrupo']) && $_POST['codgrupo'] != '') { - $this->cliente_s->codgrupo = $_POST['codgrupo']; - } + if (isset($_POST['codgrupo']) && $_POST['codgrupo'] != '') { + $this->cliente_s->codgrupo = $_POST['codgrupo']; + } - if (isset($_POST['nuevo_telefono1'])) { - $this->cliente_s->telefono1 = $_POST['nuevo_telefono1']; - } + if (isset($_POST['nuevo_telefono1'])) { + $this->cliente_s->telefono1 = $_POST['nuevo_telefono1']; + } - if (isset($_POST['nuevo_telefono2'])) { - $this->cliente_s->telefono2 = $_POST['nuevo_telefono2']; - } + if (isset($_POST['nuevo_telefono2'])) { + $this->cliente_s->telefono2 = $_POST['nuevo_telefono2']; + } - if ($this->cliente_s->save()) { - if ($this->empresa->contintegrada) { - /// forzamos crear la subcuenta - $this->cliente_s->get_subcuenta($this->empresa->codejercicio); - } + if ($this->cliente_s->save()) { + if ($this->empresa->contintegrada) { + /// forzamos crear la subcuenta + $this->cliente_s->get_subcuenta($this->empresa->codejercicio); + } - $dircliente = new direccion_cliente(); - $dircliente->codcliente = $this->cliente_s->codcliente; - $dircliente->codpais = $this->empresa->codpais; - $dircliente->provincia = $this->empresa->provincia; - $dircliente->ciudad = $this->empresa->ciudad; + $dircliente = new direccion_cliente(); + $dircliente->codcliente = $this->cliente_s->codcliente; + $dircliente->codpais = $this->empresa->codpais; + $dircliente->provincia = $this->empresa->provincia; + $dircliente->ciudad = $this->empresa->ciudad; - if (isset($_POST['nuevo_pais'])) { - $dircliente->codpais = $_POST['nuevo_pais']; - } + if (isset($_POST['nuevo_pais'])) { + $dircliente->codpais = $_POST['nuevo_pais']; + } - if (isset($_POST['nuevo_provincia'])) { - $dircliente->provincia = $_POST['nuevo_provincia']; - } + if (isset($_POST['nuevo_provincia'])) { + $dircliente->provincia = $_POST['nuevo_provincia']; + } - if (isset($_POST['nuevo_ciudad'])) { - $dircliente->ciudad = $_POST['nuevo_ciudad']; - } + if (isset($_POST['nuevo_ciudad'])) { + $dircliente->ciudad = $_POST['nuevo_ciudad']; + } - if (isset($_POST['nuevo_codpostal'])) { - $dircliente->codpostal = $_POST['nuevo_codpostal']; - } + if (isset($_POST['nuevo_codpostal'])) { + $dircliente->codpostal = $_POST['nuevo_codpostal']; + } - if (isset($_POST['nuevo_direccion'])) { - $dircliente->direccion = $_POST['nuevo_direccion']; - } + if (isset($_POST['nuevo_direccion'])) { + $dircliente->direccion = $_POST['nuevo_direccion']; + } - if ($dircliente->save()) { - $this->new_message('Cliente agregado correctamente.'); - } - } else { - $this->new_error_msg("¡Imposible guardar la dirección del cliente!"); + if ($dircliente->save()) { + $this->new_message('Cliente agregado correctamente.'); } + } else { + $this->new_error_msg("¡Imposible guardar la dirección del cliente!"); } } } @@ -265,11 +263,11 @@ public function tipos_a_guardar() { $tipos = array(); - if ($this->user->have_access_to('ventas_presupuesto') AND class_exists('presupuesto_cliente')) { + if ($this->user->have_access_to('ventas_presupuesto') && class_exists('presupuesto_cliente')) { $tipos[] = array('tipo' => 'presupuesto', 'nombre' => ucfirst(FS_PRESUPUESTO) . ' para cliente'); } - if ($this->user->have_access_to('ventas_pedido') AND class_exists('pedido_cliente')) { + if ($this->user->have_access_to('ventas_pedido') && class_exists('pedido_cliente')) { $tipos[] = array('tipo' => 'pedido', 'nombre' => ucfirst(FS_PEDIDO) . ' de cliente'); } @@ -367,14 +365,14 @@ private function new_search() /// añadimos el stock del almacén y el general $this->results[$i]->stockalm = $this->results[$i]->stockfis; - if ($this->multi_almacen AND isset($_REQUEST['codalmacen'])) { + if ($this->multi_almacen && isset($_REQUEST['codalmacen'])) { $this->results[$i]->stockalm = $stock->total_from_articulo($this->results[$i]->referencia, $_REQUEST['codalmacen']); } } /// ejecutamos las funciones de las extensiones foreach ($this->extensions as $ext) { - if ($ext->type == 'function' AND $ext->params == 'new_search') { + if ($ext->type == 'function' && $ext->params == 'new_search') { $name = $ext->text; $name($this->db, $this->results); } @@ -584,7 +582,7 @@ private function nuevo_albaran_cliente() $linea->idalbaran = $albaran->idalbaran; $linea->descripcion = $_POST['desc_' . $i]; - if (!$serie->siniva AND $cliente->regimeniva != 'Exento') { + if (!$serie->siniva && $cliente->regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $i]); if ($imp0) { $linea->codimpuesto = $imp0->codimpuesto; @@ -616,13 +614,13 @@ private function nuevo_albaran_cliente() } if ($linea->save()) { - if ($articulo AND isset($_POST['stock'])) { + if ($articulo && isset($_POST['stock'])) { $stockfis = $articulo->stockfis; if ($this->multi_almacen) { $stockfis = $stock0->total_from_articulo($articulo->referencia, $albaran->codalmacen); } - if (!$articulo->controlstock AND $linea->cantidad > $stockfis) { + if (!$articulo->controlstock && $linea->cantidad > $stockfis) { $this->new_error_msg("No hay suficiente stock del artículo " . $linea->referencia . '.'); $linea->delete(); $continuar = FALSE; @@ -811,7 +809,7 @@ private function nueva_factura_cliente() $linea->idfactura = $factura->idfactura; $linea->descripcion = $_POST['desc_' . $i]; - if (!$serie->siniva AND $cliente->regimeniva != 'Exento') { + if (!$serie->siniva && $cliente->regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $i]); if ($imp0) { $linea->codimpuesto = $imp0->codimpuesto; @@ -843,13 +841,13 @@ private function nueva_factura_cliente() } if ($linea->save()) { - if ($articulo AND isset($_POST['stock'])) { + if ($articulo && isset($_POST['stock'])) { $stockfis = $articulo->stockfis; if ($this->multi_almacen) { $stockfis = $stock0->total_from_articulo($articulo->referencia, $factura->codalmacen); } - if (!$articulo->controlstock AND $linea->cantidad > $stockfis) { + if (!$articulo->controlstock && $linea->cantidad > $stockfis) { $this->new_error_msg("No hay suficiente stock del artículo " . $linea->referencia . '.'); $linea->delete(); $continuar = FALSE; @@ -1055,7 +1053,7 @@ private function nuevo_presupuesto_cliente() $linea->idpresupuesto = $presupuesto->idpresupuesto; $linea->descripcion = $_POST['desc_' . $i]; - if (!$serie->siniva AND $cliente->regimeniva != 'Exento') { + if (!$serie->siniva && $cliente->regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $i]); if ($imp0) { $linea->codimpuesto = $imp0->codimpuesto; @@ -1236,7 +1234,7 @@ private function nuevo_pedido_cliente() $linea->idpedido = $pedido->idpedido; $linea->descripcion = $_POST['desc_' . $i]; - if (!$serie->siniva AND $cliente->regimeniva != 'Exento') { + if (!$serie->siniva && $cliente->regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $i]); if ($imp0) { $linea->codimpuesto = $imp0->codimpuesto; diff --git a/controller/subcuenta_asociada.php b/controller/subcuenta_asociada.php index b0e43cd..f0b151f 100644 --- a/controller/subcuenta_asociada.php +++ b/controller/subcuenta_asociada.php @@ -51,7 +51,7 @@ protected function private_core() $this->codejercicio = $this->default_items->codejercicio(); } - if (isset($_POST['ejercicio']) AND isset($_POST['query'])) { + if (isset($_POST['ejercicio']) && isset($_POST['query'])) { $this->new_search(); } else if (isset($_REQUEST['cli'])) { $this->tipo = 'cli'; diff --git a/controller/tpv_recambios.php b/controller/tpv_recambios.php index 5838836..13318ed 100644 --- a/controller/tpv_recambios.php +++ b/controller/tpv_recambios.php @@ -107,18 +107,20 @@ protected function private_core() $this->caja->dinero_fin = floatval($_POST['d_inicial']); if ($this->caja->save()) { $this->new_message("Caja iniciada con " . $this->show_precio($this->caja->dinero_inicial)); - } else + } else { $this->new_error_msg("¡Imposible guardar los datos de caja!"); - } else + } + } else { $this->new_error_msg('El terminal ya no está disponible.'); - } - else if (isset($_GET['terminal'])) { + } + } else if (isset($_GET['terminal'])) { $this->terminal = $terminal0->get($_GET['terminal']); if ($this->terminal) { $this->terminal->abrir_cajon(); $this->terminal->save(); - } else + } else { $this->new_error_msg('Terminal no encontrado.'); + } } } @@ -223,7 +225,7 @@ private function new_search() /// ejecutamos las funciones de las extensiones foreach ($this->extensions as $ext) { - if ($ext->type == 'function' AND $ext->params == 'new_search') { + if ($ext->type == 'function' && $ext->params == 'new_search') { $name = $ext->text; $name($this->db, $this->results); } @@ -246,24 +248,22 @@ private function new_search_postprocess() $this->results[$i]->cantidad = 1; $this->results[$i]->stockalm = $value->stockfis; - if ($this->multi_almacen AND isset($_REQUEST['codalmacen'])) { + if ($this->multi_almacen && isset($_REQUEST['codalmacen'])) { $this->results[$i]->stockalm = $stock->total_from_articulo($this->results[$i]->referencia, $_REQUEST['codalmacen']); } } if (isset($_REQUEST['codcliente'])) { $cliente = $this->cliente->get($_REQUEST['codcliente']); - if ($cliente) { - if ($cliente->codgrupo) { - $grupo0 = new grupo_clientes(); - $tarifa0 = new tarifa(); - - $grupo = $grupo0->get($cliente->codgrupo); - if ($grupo) { - $tarifa = $tarifa0->get($grupo->codtarifa); - if ($tarifa) { - $tarifa->set_precios($this->results); - } + if ($cliente && $cliente->codgrupo) { + $grupo0 = new grupo_clientes(); + $tarifa0 = new tarifa(); + + $grupo = $grupo0->get($cliente->codgrupo); + if ($grupo) { + $tarifa = $tarifa0->get($grupo->codtarifa); + if ($tarifa) { + $tarifa->set_precios($this->results); } } } @@ -436,7 +436,7 @@ private function nueva_factura_cliente() $linea->referencia = $articulo->referencia; $linea->descripcion = $_POST['desc_' . $i]; - if (!$serie->siniva OR $this->cliente_s->regimeniva != 'Exento') { + if (!$serie->siniva || $this->cliente_s->regimeniva != 'Exento') { $linea->codimpuesto = $articulo->codimpuesto; $linea->iva = floatval($_POST['iva_' . $i]); $linea->recargo = floatval($_POST['recargo_' . $i]); @@ -517,27 +517,28 @@ private function nueva_factura_cliente() } else { $this->new_error_msg("¡Imposible actualizar la caja!"); } - } else + } else { $this->new_error_msg("¡Imposible actualizar la factura!"); - } - else if ($factura->delete()) { + } + } else if ($factura->delete()) { $this->new_message("Factura eliminada correctamente."); - } else + } else { $this->new_error_msg("¡Imposible eliminar la factura!"); - } else + } + } else { $this->new_error_msg("¡Imposible guardar la factura!"); + } } } private function abrir_caja() { - if ($this->user->admin) { - if ($this->terminal) { - $this->terminal->abrir_cajon(); - $this->terminal->save(); - } - } else + if ($this->user->admin && $this->terminal) { + $this->terminal->abrir_cajon(); + $this->terminal->save(); + } else { $this->new_error_msg('Sólo un administrador puede abrir la caja.'); + } } private function cerrar_caja() @@ -567,8 +568,9 @@ private function cerrar_caja() /// recargamos la página header('location: ' . $this->url()); } - } else + } else { $this->new_error_msg("¡Imposible cerrar la caja!"); + } } private function reimprimir_ticket() @@ -577,17 +579,19 @@ private function reimprimir_ticket() $fac0 = FALSE; if ($_GET['reticket'] == '') { - foreach ($factura->all() as $fac) { + foreach ($factura->all(0, FS_ITEM_LIMIT, 'idfactura DESC') as $fac) { $fac0 = $fac; break; } - } else + } else { $fac0 = $factura->get_by_codigo($_GET['reticket']); + } if ($fac0) { $this->imprimir_ticket($fac0, 1, FALSE); - } else + } else { $this->new_error_msg("Ticket no encontrado."); + } } /** diff --git a/controller/ventas_agrupar_albaranes.php b/controller/ventas_agrupar_albaranes.php index 36634a5..ccfab2a 100644 --- a/controller/ventas_agrupar_albaranes.php +++ b/controller/ventas_agrupar_albaranes.php @@ -193,7 +193,7 @@ private function generar_factura($albaranes) * Si la dirección es de facturación y ha sido modificada posteriormente * al albarán, la usamos para la factura, porque está más actualizada. */ - if ($dir->domfacturacion AND strtotime($dir->fecha) > strtotime($albaranes[0]->fecha)) { + if ($dir->domfacturacion && strtotime($dir->fecha) > strtotime($albaranes[0]->fecha)) { $factura->apartado = $dir->apartado; $factura->cifnif = $cliente->cifnif; $factura->ciudad = $dir->ciudad; @@ -368,7 +368,7 @@ public function pendientes() /// Comprobamos si el cliente ya está en la lista. $encontrado = FALSE; foreach ($pendientes as $i => $pe) { - if ($alb->codcliente == $pe['codcliente'] AND $alb->codserie == $pe['codserie'] AND $alb->coddivisa == $pe['coddivisa']) { + if ($alb->codcliente == $pe['codcliente'] && $alb->codserie == $pe['codserie'] && $alb->coddivisa == $pe['coddivisa']) { $encontrado = TRUE; $pendientes[$i]['num'] ++; diff --git a/controller/ventas_albaran.php b/controller/ventas_albaran.php index e09928a..e30938b 100644 --- a/controller/ventas_albaran.php +++ b/controller/ventas_albaran.php @@ -103,10 +103,10 @@ protected function private_core() /// comprobamos el albarán $this->albaran->full_test(); - if (isset($_REQUEST['facturar']) AND isset($_REQUEST['petid'])) { + if (isset($_REQUEST['facturar']) && isset($_REQUEST['petid'])) { if ($this->duplicated_petition($_REQUEST['petid'])) { $this->new_error_msg('Petición duplicada. Evita hacer doble clic sobre los botones.'); - } else if (!$this->albaran->ptefactura OR ! is_null($this->albaran->idfactura)) { + } else if (!$this->albaran->ptefactura || ! is_null($this->albaran->idfactura)) { $this->new_error_msg('Parece que este ' . FS_ALBARAN . ' ya está facturado.'); } else $this->generar_factura(); @@ -294,7 +294,7 @@ private function modificar() $lineas[$k]->iva = 0; $lineas[$k]->recargo = 0; $lineas[$k]->irpf = floatval($_POST['irpf_' . $num]); - if (!$serie->siniva AND $regimeniva != 'Exento') { + if (!$serie->siniva && $regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $num]); if ($imp0) { $lineas[$k]->codimpuesto = $imp0->codimpuesto; @@ -329,12 +329,12 @@ private function modificar() } /// añadimos la línea - if (!$encontrada AND intval($_POST['idlinea_' . $num]) == -1 AND isset($_POST['referencia_' . $num])) { + if (!$encontrada && intval($_POST['idlinea_' . $num]) == -1 && isset($_POST['referencia_' . $num])) { $linea = new linea_albaran_cliente(); $linea->idalbaran = $this->albaran->idalbaran; $linea->descripcion = $_POST['desc_' . $num]; - if (!$serie->siniva AND $regimeniva != 'Exento') { + if (!$serie->siniva && $regimeniva != 'Exento') { $imp0 = $this->impuesto->get_by_iva($_POST['iva_' . $num]); if ($imp0) { $linea->codimpuesto = $imp0->codimpuesto; diff --git a/controller/ventas_albaranes.php b/controller/ventas_albaranes.php index beff781..41c620e 100644 --- a/controller/ventas_albaranes.php +++ b/controller/ventas_albaranes.php @@ -116,7 +116,7 @@ protected function private_core() if (isset($_POST['delete'])) { $this->delete_albaran(); } else { - if (!isset($_GET['mostrar']) AND ( $this->query != '' OR isset($_REQUEST['codagente']) OR isset($_REQUEST['codcliente']) OR isset($_REQUEST['codserie']))) { + if (!isset($_GET['mostrar']) && ( $this->query != '' || isset($_REQUEST['codagente']) || isset($_REQUEST['codcliente']) || isset($_REQUEST['codserie']))) { /** * si obtenermos un codagente, un codcliente o un codserie pasamos direcatemente * a la pestaña de búsqueda, a menos que tengamos un mostrar, que @@ -125,11 +125,9 @@ protected function private_core() $this->mostrar = 'buscar'; } - if (isset($_REQUEST['codcliente'])) { - if ($_REQUEST['codcliente'] != '') { - $cli0 = new cliente(); - $this->cliente = $cli0->get($_REQUEST['codcliente']); - } + if (isset($_REQUEST['codcliente']) && $_REQUEST['codcliente'] != '') { + $cli0 = new cliente(); + $this->cliente = $cli0->get($_REQUEST['codcliente']); } if (isset($_REQUEST['codagente'])) { @@ -212,9 +210,9 @@ public function url($busqueda = FALSE) . "&hasta=" . $this->hasta; return $url; - } else { - return parent::url(); } + + return parent::url(); } public function paginas() @@ -266,7 +264,7 @@ private function delete_albaran() * no tocamos, porque sigue estando ese otro albarán. * (las facturas pueden agrupar albaranes). */ - if ($linea->referencia AND ( is_null($linea->idalbaran) OR $linea->idalbaran == $alb1->idalbaran)) { + if ($linea->referencia && ( is_null($linea->idalbaran) || $linea->idalbaran == $alb1->idalbaran)) { $art0 = $articulo->get($linea->referencia); if ($art0) { $art0->sum_stock($alb1->codalmacen, $linea->cantidad, FALSE, $linea->codcombinacion); @@ -289,10 +287,12 @@ private function delete_albaran() if ($alb1->delete()) { $this->clean_last_changes(); - } else + } else { $this->new_error_msg("¡Imposible eliminar el " . FS_ALBARAN . "!"); - } else + } + } else { $this->new_error_msg("¡" . FS_ALBARAN . " no encontrado!"); + } } private function share_extension() diff --git a/controller/ventas_articulo.php b/controller/ventas_articulo.php index 8f497d2..82e203f 100644 --- a/controller/ventas_articulo.php +++ b/controller/ventas_articulo.php @@ -128,8 +128,9 @@ public function url() { if ($this->articulo) { return $this->articulo->url(); - } else - return $this->page->url(); + } + + return $this->page->url(); } private function check_extensions() @@ -140,7 +141,7 @@ private function check_extensions() */ $this->mostrar_boton_publicar = TRUE; foreach ($this->extensions as $ext) { - if ($ext->type == 'config' AND $ext->text == 'no_button_publicar') { + if ($ext->type == 'config' && $ext->text == 'no_button_publicar') { $this->mostrar_boton_publicar = FALSE; break; } @@ -164,7 +165,7 @@ private function check_extensions() */ $this->mostrar_tab_precios = TRUE; foreach ($this->extensions as $ext) { - if ($ext->type == 'config' AND $ext->text == 'no_tab_precios') { + if ($ext->type == 'config' && $ext->text == 'no_tab_precios') { $this->mostrar_tab_precios = FALSE; break; } @@ -176,7 +177,7 @@ private function check_extensions() */ $this->mostrar_tab_stock = TRUE; foreach ($this->extensions as $ext) { - if ($ext->type == 'config' AND $ext->text == 'no_tab_stock') { + if ($ext->type == 'config' && $ext->text == 'no_tab_stock') { $this->mostrar_tab_stock = FALSE; break; } @@ -296,8 +297,9 @@ private function edit_imagen() $this->articulo->set_imagen(file_get_contents($_FILES['fimagen']['tmp_name']), $png); if ($this->articulo->save()) { $this->new_message("Imagen del articulo modificada correctamente"); - } else + } else { $this->new_error_msg("¡Error al guardar la imagen del articulo!"); + } } } @@ -306,8 +308,9 @@ private function eliminar_imagen() $this->articulo->set_imagen(NULL); if ($this->articulo->save()) { $this->new_message("Imagen del articulo eliminada correctamente"); - } else + } else { $this->new_error_msg("¡Error al eliminar la imagen del articulo!"); + } } private function modificar_articulo() @@ -396,8 +399,9 @@ private function modificar_articulo() $this->db->exec("UPDATE lineasfabricados SET referencia = " . $this->empresa->var2str($_POST['nreferencia']) . " WHERE referencia = " . $this->empresa->var2str($_POST['referencia']) . ";"); } - } else + } else { $this->new_error_msg("¡Error al guardar el articulo!"); + } } private function nueva_combinacion() diff --git a/controller/ventas_articulos.php b/controller/ventas_articulos.php index 61df596..d68305d 100644 --- a/controller/ventas_articulos.php +++ b/controller/ventas_articulos.php @@ -67,7 +67,7 @@ protected function private_core() */ $this->mostrar_tab_tarifas = TRUE; foreach ($this->extensions as $ext) { - if ($ext->type == 'config' AND $ext->text == 'no_tab_tarifas') { + if ($ext->type == 'config' && $ext->text == 'no_tab_tarifas') { $this->mostrar_tab_tarifas = FALSE; break; } @@ -77,7 +77,7 @@ protected function private_core() $this->edit_tarifa(); } else if (isset($_GET['delete_tarifa'])) { $this->delete_tarifa(); - } else if (isset($_POST['referencia']) AND isset($_POST['codfamilia']) AND isset($_POST['codimpuesto'])) { + } else if (isset($_POST['referencia']) && isset($_POST['codfamilia']) && isset($_POST['codimpuesto'])) { $this->new_articulo($articulo); } else if (isset($_GET['delete'])) { $this->delete_articulo($articulo); @@ -175,8 +175,9 @@ private function edit_tarifa() $tar0->maxpvp = isset($_POST['maxpvp']); if ($tar0->save()) { $this->new_message("Tarifa guardada correctamente."); - } else + } else { $this->new_error_msg("¡Imposible guardar la tarifa!"); + } } private function delete_tarifa() @@ -187,10 +188,12 @@ private function delete_tarifa() $this->new_error_msg('No tienes permiso para eliminar en esta página.'); } else if ($tar0->delete()) { $this->new_message("Tarifa " . $tar0->codtarifa . " eliminada correctamente.", TRUE); - } else + } else { $this->new_error_msg("¡Imposible eliminar la tarifa!"); - } else + } + } else { $this->new_error_msg("¡La tarifa no existe!"); + } } private function new_articulo(&$articulo) @@ -230,8 +233,9 @@ private function new_articulo(&$articulo) if ($articulo->save()) { header('location: ' . $articulo->url()); - } else + } else { $this->new_error_msg("¡Error al crear el articulo!"); + } } } @@ -243,10 +247,10 @@ private function delete_articulo(&$articulo) $this->new_error_msg('No tienes permiso para eliminar en esta página.'); } else if ($art->delete()) { $this->new_message("Articulo " . $art->referencia . " eliminado correctamente.", TRUE); - } else + } else { $this->new_error_msg("¡Error al eliminarl el articulo!"); - } - else { + } + } else { $this->new_error_msg("Articulo no encontrado."); } } diff --git a/controller/ventas_cliente.php b/controller/ventas_cliente.php index fb6bb70..cad39a1 100644 --- a/controller/ventas_cliente.php +++ b/controller/ventas_cliente.php @@ -86,8 +86,9 @@ public function url() return parent::url(); } else if ($this->cliente) { return $this->cliente->url(); - } else - return $this->ppage->url(); + } + + return $this->ppage->url(); } private function modificar() @@ -155,8 +156,9 @@ private function edit_cuenta_banco() if ($cuentab->save()) { $this->new_message('Cuenta bancaria guardada correctamente.'); - } else + } else { $this->new_error_msg('Imposible guardar la cuenta bancaria.'); + } } private function delete_cuenta_banco() @@ -165,10 +167,12 @@ private function delete_cuenta_banco() if ($cuenta) { if ($cuenta->delete()) { $this->new_message('Cuenta bancaria eliminada correctamente.'); - } else + } else { $this->new_error_msg('Imposible eliminar la cuenta bancaria.'); - } else + } + } else { $this->new_error_msg('Cuenta bancaria no encontrada.'); + } } private function edit_direccion() @@ -189,8 +193,9 @@ private function edit_direccion() $dir->provincia = $_POST['provincia']; if ($dir->save()) { $this->new_message("Dirección guardada correctamente."); - } else + } else { $this->new_message("¡Imposible guardar la dirección!"); + } } private function delete_direccion() @@ -200,10 +205,12 @@ private function delete_direccion() if ($dir0) { if ($dir0->delete()) { $this->new_message('Dirección eliminada correctamente.'); - } else + } else { $this->new_error_msg('Imposible eliminar la dirección.'); - } else + } + } else { $this->new_error_msg('Dirección no encontrada.'); + } } private function convertir() @@ -226,12 +233,12 @@ private function convertir() $proveedor->codserie = $this->cliente->codserie; $proveedor->codcliente = $this->cliente->codcliente; - $proveedor_ok = TRUE; if ($proveedor->save()) { $this->cliente->codproveedor = $proveedor->codproveedor; $proveedor_ok = $this->cliente->save(); - } else + } else { $proveedor_ok = FALSE; + } if ($proveedor_ok) { /* cuentas de banco */ @@ -259,8 +266,9 @@ private function convertir() } $this->new_message('Proveedor creado correctamente.'); - } else + } else { $this->new_error_msg("¡Imposible crear el proveedor!"); + } } public function tiene_facturas() @@ -276,7 +284,7 @@ public function tiene_facturas() } } - if (!$tiene AND $this->db->table_exists('albaranescli')) { + if (!$tiene && $this->db->table_exists('albaranescli')) { $sql = "SELECT * FROM albaranescli WHERE codcliente = " . $this->cliente->var2str($this->cliente->codcliente); $data = $this->db->select_limit($sql, 5, 0); diff --git a/controller/ventas_cliente_articulos.php b/controller/ventas_cliente_articulos.php index 02e36cc..e448498 100644 --- a/controller/ventas_cliente_articulos.php +++ b/controller/ventas_cliente_articulos.php @@ -46,7 +46,7 @@ protected function private_core() $this->resultados = array(); /// recibimos el código del cliente - if (isset($_REQUEST['cod']) AND ! empty($_REQUEST['cod'])) { + if (isset($_REQUEST['cod']) && ! empty($_REQUEST['cod'])) { $cli0 = new cliente(); $this->cliente = $cli0->get($_REQUEST['cod']); } diff --git a/controller/ventas_clientes.php b/controller/ventas_clientes.php index f90c130..fb267dd 100644 --- a/controller/ventas_clientes.php +++ b/controller/ventas_clientes.php @@ -221,7 +221,7 @@ private function buscar() $and = ' AND '; } - if ($this->ciudad != '' OR $this->provincia != '' OR $this->codpais != '') { + if ($this->ciudad != '' || $this->provincia != '' || $this->codpais != '') { $sql .= $and . " codcliente IN (SELECT codcliente FROM dirclientes WHERE "; $and2 = ''; diff --git a/controller/ventas_factura.php b/controller/ventas_factura.php index 368fe67..7a1d139 100644 --- a/controller/ventas_factura.php +++ b/controller/ventas_factura.php @@ -68,7 +68,7 @@ protected function private_core() */ $this->mostrar_boton_pagada = TRUE; foreach ($this->extensions as $ext) { - if ($ext->type == 'config' AND $ext->text == 'no_button_pagada') { + if ($ext->type == 'config' && $ext->text == 'no_button_pagada') { $this->mostrar_boton_pagada = FALSE; break; } @@ -99,7 +99,7 @@ protected function private_core() $cliente = new cliente(); $this->cliente = $cliente->get($this->factura->codcliente); - if (isset($_GET['gen_asiento']) AND isset($_GET['petid'])) { + if (isset($_GET['gen_asiento']) && isset($_GET['petid'])) { if ($this->duplicated_petition($_GET['petid'])) { $this->new_error_msg('Petición duplicada. Evita hacer doble clic sobre los botones.'); } else { @@ -136,8 +136,9 @@ public function url() return parent::url(); } else if ($this->factura) { return $this->factura->url(); - } else - return $this->ppage->url(); + } + + return $this->ppage->url(); } private function modificar() @@ -194,8 +195,9 @@ private function modificar() } $this->new_message("Factura modificada correctamente."); $this->new_change('Factura Cliente ' . $this->factura->codigo, $this->factura->url()); - } else + } else { $this->new_error_msg("¡Imposible modificar la factura!"); + } } private function actualizar_direccion() @@ -215,8 +217,9 @@ private function actualizar_direccion() if ($this->factura->save()) { $this->new_message('Dirección actualizada correctamente.'); - } else + } else { $this->new_error_msg('Imposible actualizar la dirección de la factura.'); + } break; } @@ -255,7 +258,7 @@ private function pagar($pagada = TRUE) if (is_null($this->factura->idasiento)) { $this->factura->pagada = $pagada; $this->factura->save(); - } else if (!$pagada AND $this->factura->pagada) { + } else if (!$pagada && $this->factura->pagada) { /// marcar como impagada $this->factura->pagada = FALSE; @@ -273,14 +276,14 @@ private function pagar($pagada = TRUE) } else { $this->new_error_msg('Error al modificar la factura.'); } - } else if ($pagada AND ! $this->factura->pagada) { + } else if ($pagada && ! $this->factura->pagada) { /// marcar como pagada $asiento = $this->factura->get_asiento(); if ($asiento) { /// nos aseguramos que el cliente tenga subcuenta en el ejercicio actual $subcli = FALSE; $eje = $this->ejercicio->get_by_fecha($_POST['fpagada']); - if ($eje AND $this->cliente) { + if ($eje && $this->cliente) { $subcli = $this->cliente->get_subcuenta($eje->codejercicio); } diff --git a/controller/ventas_facturas.php b/controller/ventas_facturas.php index c10f1a1..76fc3f8 100644 --- a/controller/ventas_facturas.php +++ b/controller/ventas_facturas.php @@ -124,7 +124,7 @@ protected function private_core() if (isset($_GET['delete'])) { $this->delete_factura(); } else { - if (!isset($_GET['mostrar']) AND ( $this->query != '' OR isset($_REQUEST['codagente']) OR isset($_REQUEST['codcliente']) OR isset($_REQUEST['codserie']))) { + if (!isset($_GET['mostrar']) && ( $this->query != '' || isset($_REQUEST['codagente']) || isset($_REQUEST['codcliente']) || isset($_REQUEST['codserie']))) { /** * si obtenermos un codagente, un codcliente o un codserie pasamos direcatemente * a la pestaña de búsqueda, a menos que tengamos un mostrar, que @@ -133,11 +133,9 @@ protected function private_core() $this->mostrar = 'buscar'; } - if (isset($_REQUEST['codcliente'])) { - if ($_REQUEST['codcliente'] != '') { - $cli0 = new cliente(); - $this->cliente = $cli0->get($_REQUEST['codcliente']); - } + if (isset($_REQUEST['codcliente']) && $_REQUEST['codcliente'] != '') { + $cli0 = new cliente(); + $this->cliente = $cli0->get($_REQUEST['codcliente']); } if (isset($_REQUEST['codagente'])) { @@ -195,8 +193,9 @@ protected function private_core() } } else if ($this->mostrar == 'buscar') { $this->buscar($order2); - } else + } else { $this->resultados = $this->factura->all($this->offset, FS_ITEM_LIMIT, $this->order . $order2); + } } } @@ -221,9 +220,9 @@ public function url($busqueda = FALSE) . "&hasta=" . $this->hasta; return $url; - } else { - return parent::url(); } + + return parent::url(); } public function paginas() @@ -441,10 +440,12 @@ private function delete_factura() } $this->clean_last_changes(); - } else + } else { $this->new_error_msg("¡Imposible eliminar la factura!"); - } else + } + } else { $this->new_error_msg("Factura no encontrada."); + } } public function orden() diff --git a/controller/ventas_grupo.php b/controller/ventas_grupo.php index fc20fc4..aa54b7c 100644 --- a/controller/ventas_grupo.php +++ b/controller/ventas_grupo.php @@ -162,47 +162,40 @@ private function buscar_clientes() $query = mb_strtolower($this->grupo->no_html($this->query), 'UTF8'); $sql = " FROM clientes WHERE (debaja = false OR debaja IS NULL) AND " . "(codgrupo IS NULL OR codgrupo != " . $this->grupo->var2str($this->grupo->codgrupo) . ')'; - $and = ' AND '; if (is_numeric($query)) { - $sql .= $and . "(nombre LIKE '%" . $query . "%'" + $sql .= " AND (nombre LIKE '%" . $query . "%'" . " OR razonsocial LIKE '%" . $query . "%'" . " OR codcliente LIKE '%" . $query . "%'" . " OR cifnif LIKE '%" . $query . "%'" . " OR telefono1 LIKE '" . $query . "%'" . " OR telefono2 LIKE '" . $query . "%'" . " OR observaciones LIKE '%" . $query . "%')"; - $and = ' AND '; } else if ($query != '') { $buscar = str_replace(' ', '%', $query); - $sql .= $and . "(lower(nombre) LIKE '%" . $buscar . "%'" + $sql .= " AND (lower(nombre) LIKE '%" . $buscar . "%'" . " OR lower(razonsocial) LIKE '%" . $buscar . "%'" . " OR lower(cifnif) LIKE '%" . $buscar . "%'" . " OR lower(observaciones) LIKE '%" . $buscar . "%'" . " OR lower(email) LIKE '%" . $buscar . "%')"; - $and = ' AND '; } if ($this->ciudad != '' OR $this->provincia != '' OR $this->codpais != '') { - $sql .= $and . " codcliente IN (SELECT codcliente FROM dirclientes WHERE "; - $and2 = ''; + $sql .= " AND codcliente IN (SELECT codcliente FROM dirclientes WHERE 1=1 "; if ($this->ciudad != '') { - $sql .= $and2 . "lower(ciudad) = " . $this->grupo->var2str(mb_strtolower($this->ciudad, 'UTF8')); - $and2 = ' AND '; + $sql .= " AND lower(ciudad) = " . $this->grupo->var2str(mb_strtolower($this->ciudad, 'UTF8')); } if ($this->provincia != '') { - $sql .= $and2 . "lower(provincia) = " . $this->grupo->var2str(mb_strtolower($this->provincia, 'UTF8')); - $and2 = ' AND '; + $sql .= " AND lower(provincia) = " . $this->grupo->var2str(mb_strtolower($this->provincia, 'UTF8')); } if ($this->codpais != '') { - $sql .= $and2 . "codpais = " . $this->grupo->var2str($this->codpais); + $sql .= " AND codpais = " . $this->grupo->var2str($this->codpais); } $sql .= ")"; - $and = ' AND '; } $data = $this->db->select_limit("SELECT *" . $sql . " ORDER BY " . $this->orden, 100, 0); diff --git a/controller/ventas_imprimir.php b/controller/ventas_imprimir.php index fdf0052..17e7ed7 100644 --- a/controller/ventas_imprimir.php +++ b/controller/ventas_imprimir.php @@ -33,16 +33,17 @@ public function __construct($name = __CLASS__, $title = 'imprimir', $folder = 'v parent::__construct($name, $title, $folder); } - protected function private_core() + protected function init() { + parent::init(); $this->cliente = FALSE; - $this->documento = FALSE; - $this->impuesto = new impuesto(); - $this->cargar_config(); + } - if (isset($_REQUEST['albaran']) AND isset($_REQUEST['id'])) { - $this->articulo_traza = new articulo_traza(); + protected function private_core() + { + $this->init(); + if (isset($_REQUEST['albaran']) && isset($_REQUEST['id'])) { $alb = new albaran_cliente(); $this->documento = $alb->get($_REQUEST['id']); if ($this->documento) { @@ -55,9 +56,7 @@ protected function private_core() } else { $this->generar_pdf_albaran(); } - } else if (isset($_REQUEST['factura']) AND isset($_REQUEST['id'])) { - $this->articulo_traza = new articulo_traza(); - + } else if (isset($_REQUEST['factura']) && isset($_REQUEST['id'])) { $fac = new factura_cliente(); $this->documento = $fac->get($_REQUEST['id']); if ($this->documento) { @@ -71,8 +70,6 @@ protected function private_core() $this->generar_pdf_factura($_REQUEST['tipo']); } } - - $this->share_extensions(); } protected function share_extensions() @@ -144,7 +141,7 @@ protected function generar_pdf_lineas_venta(&$pdf_doc, &$lineas, &$linea_actual, while ($linea_a < count($lineas)) { $lppag2 = $lppag; foreach ($lineas as $i => $lin) { - if ($i >= $linea_a AND $i < $linea_a + $lppag2) { + if ($i >= $linea_a && $i < $linea_a + $lppag2) { $linea_size = 1; $len = mb_strlen($lin->referencia . ' ' . $lin->descripcion); while ($len > 85) { @@ -216,7 +213,7 @@ protected function generar_pdf_lineas_venta(&$pdf_doc, &$lineas, &$linea_actual, } /// restamos líneas al documento en función del tamaño de la descripción - if ($i >= $linea_actual AND $i < $linea_actual + $lppag) { + if ($i >= $linea_actual && $i < $linea_actual + $lppag) { $linea_size = 1; $len = mb_strlen($lin->referencia . ' ' . $lin->descripcion); while ($len > 85) { @@ -266,19 +263,19 @@ protected function generar_pdf_lineas_venta(&$pdf_doc, &$lineas, &$linea_actual, unset($table_header['alb']); } - if ($this->impresion['print_dto'] AND ! isset($_GET['noval'])) { + if ($this->impresion['print_dto'] && !isset($_GET['noval'])) { $table_header['dto'] = 'Dto.'; } - if ($multi_iva AND ! isset($_GET['noval'])) { + if ($multi_iva && !isset($_GET['noval'])) { $table_header['iva'] = '' . FS_IVA . ''; } - if ($multi_re AND ! isset($_GET['noval'])) { + if ($multi_re && !isset($_GET['noval'])) { $table_header['re'] = 'R.E.'; } - if ($multi_irpf AND ! isset($_GET['noval'])) { + if ($multi_irpf && !isset($_GET['noval'])) { $table_header['irpf'] = '' . FS_IRPF . ''; } @@ -290,7 +287,7 @@ protected function generar_pdf_lineas_venta(&$pdf_doc, &$lineas, &$linea_actual, $pdf_doc->add_table_header($table_header); - for ($i = $linea_actual; (($linea_actual < ($lppag + $i)) AND ( $linea_actual < count($lineas)));) { + for ($i = $linea_actual; (($linea_actual < ($lppag + $i)) && ( $linea_actual < count($lineas)));) { $descripcion = fs_fix_html($lineas[$linea_actual]->descripcion); if (!is_null($lineas[$linea_actual]->referencia) && $this->impresion['print_ref']) { $descripcion = '' . $lineas[$linea_actual]->referencia . ' ' . $descripcion; @@ -336,7 +333,7 @@ protected function generar_pdf_lineas_venta(&$pdf_doc, &$lineas, &$linea_actual, $fila['importe'] = ''; } - if (get_class_name($lineas[$linea_actual]) == 'linea_factura_cliente' AND $this->impresion['print_alb']) { + if (get_class_name($lineas[$linea_actual]) == 'linea_factura_cliente' && $this->impresion['print_alb']) { $fila['alb'] = $lineas[$linea_actual]->albaran_numero(); } @@ -641,45 +638,7 @@ public function generar_pdf_factura($tipo = 'simple', $archivo = FALSE) */ if ($tipo == 'carta') { $pdf_doc->generar_pdf_cabecera($this->empresa, $lppag); - - $direccion = $this->documento->nombrecliente . "\n" . $this->documento->direccion; - if ($this->documento->apartado) { - $direccion .= "\n " . ucfirst(FS_APARTADO) . ": " . $this->documento->apartado; - } - - if ($this->documento->codpostal) { - $direccion .= "\n CP: " . $this->documento->codpostal . ' - '; - } else { - $direccion .= "\n"; - } - $direccion .= $this->documento->ciudad . "\n(" . $this->documento->provincia . ")"; - if ($this->documento->codpais != $this->empresa->codpais) { - $pais0 = new pais(); - $pais = $pais0->get($this->documento->codpais); - if ($pais) { - $direccion .= ' ' . $pais->nombre; - } - } - $pdf_doc->new_table(); - $pdf_doc->add_table_row( - array( - 'campos' => "" . ucfirst(FS_FACTURA) . ":\nFecha:\n" . $this->cliente->tipoidfiscal . ":", - 'factura' => $this->documento->codigo . "\n" . $this->documento->fecha . "\n" . $this->documento->cifnif, - 'cliente' => fs_fix_html($direccion) - ) - ); - $pdf_doc->save_table( - array( - 'cols' => array( - 'campos' => array('justification' => 'right', 'width' => 100), - 'factura' => array('justification' => 'left'), - 'cliente' => array('justification' => 'right') - ), - 'showLines' => 0, - 'width' => 520 - ) - ); - $pdf_doc->pdf->ezText("\n\n\n", 14); + $this->generar_pdf_carta($pdf_doc); } else { /// esta es la cabecera de la página para el modelo 'simple' $pdf_doc->generar_pdf_cabecera($this->empresa, $lppag); $this->generar_pdf_datos_cliente($pdf_doc, $lppag); @@ -687,51 +646,8 @@ public function generar_pdf_factura($tipo = 'simple', $archivo = FALSE) $this->generar_pdf_lineas_venta($pdf_doc, $lineas, $linea_actual, $lppag, $this->documento); - if ($linea_actual == count($lineas)) { - if (!$this->documento->pagada AND $this->impresion['print_formapago']) { - $fp0 = new forma_pago(); - $forma_pago = $fp0->get($this->documento->codpago); - if ($forma_pago) { - $texto_pago = "\nForma de pago: " . $forma_pago->descripcion; - - if (!$forma_pago->imprimir) { - /// nada - } else if ($forma_pago->domiciliado) { - $cbc0 = new cuenta_banco_cliente(); - $encontrada = FALSE; - foreach ($cbc0->all_from_cliente($this->documento->codcliente) as $cbc) { - $texto_pago .= "\nDomiciliado en: "; - if ($cbc->iban) { - $texto_pago .= $cbc->iban(TRUE); - } - - if ($cbc->swift) { - $texto_pago .= "\nSWIFT/BIC: " . $cbc->swift; - } - $encontrada = TRUE; - break; - } - if (!$encontrada) { - $texto_pago .= "\nEl cliente no tiene cuenta bancaria asignada."; - } - } else if ($forma_pago->codcuenta) { - $cb0 = new cuenta_banco(); - $cuenta_banco = $cb0->get($forma_pago->codcuenta); - if ($cuenta_banco) { - if ($cuenta_banco->iban) { - $texto_pago .= "\nIBAN: " . $cuenta_banco->iban(TRUE); - } - - if ($cuenta_banco->swift) { - $texto_pago .= "\nSWIFT o BIC: " . $cuenta_banco->swift; - } - } - } - - $texto_pago .= "\nVencimiento: " . $this->documento->vencimiento; - $pdf_doc->pdf->ezText($texto_pago, 9); - } - } + if ($linea_actual == count($lineas) && !$this->documento->pagada && $this->impresion['print_formapago']) { + $this->generar_pdf_forma_pago($pdf_doc); } $pdf_doc->set_y(80); @@ -759,6 +675,97 @@ public function generar_pdf_factura($tipo = 'simple', $archivo = FALSE) } } + protected function generar_pdf_carta(&$pdf_doc) + { + $direccion = $this->documento->nombrecliente . "\n" . $this->documento->direccion; + if ($this->documento->apartado) { + $direccion .= "\n " . ucfirst(FS_APARTADO) . ": " . $this->documento->apartado; + } + + if ($this->documento->codpostal) { + $direccion .= "\n CP: " . $this->documento->codpostal . ' - '; + } else { + $direccion .= "\n"; + } + $direccion .= $this->documento->ciudad . "\n(" . $this->documento->provincia . ")"; + if ($this->documento->codpais != $this->empresa->codpais) { + $pais0 = new pais(); + $pais = $pais0->get($this->documento->codpais); + if ($pais) { + $direccion .= ' ' . $pais->nombre; + } + } + $pdf_doc->new_table(); + $pdf_doc->add_table_row( + array( + 'campos' => "" . ucfirst(FS_FACTURA) . ":\nFecha:\n" . $this->cliente->tipoidfiscal . ":", + 'factura' => $this->documento->codigo . "\n" . $this->documento->fecha . "\n" . $this->documento->cifnif, + 'cliente' => fs_fix_html($direccion) + ) + ); + $pdf_doc->save_table( + array( + 'cols' => array( + 'campos' => array('justification' => 'right', 'width' => 100), + 'factura' => array('justification' => 'left'), + 'cliente' => array('justification' => 'right') + ), + 'showLines' => 0, + 'width' => 520 + ) + ); + $pdf_doc->pdf->ezText("\n\n\n", 14); + } + + protected function generar_pdf_forma_pago(&$pdf_doc) + { + $fp0 = new forma_pago(); + $forma_pago = $fp0->get($this->documento->codpago); + if ($forma_pago) { + $texto_pago = "\nForma de pago: " . $forma_pago->descripcion; + + if (!$forma_pago->imprimir) { + /// nada + } else if ($forma_pago->domiciliado) { + $cbc0 = new cuenta_banco_cliente(); + $encontrada = FALSE; + foreach ($cbc0->all_from_cliente($this->documento->codcliente) as $cbc) { + $texto_pago .= "\nDomiciliado en: "; + if ($cbc->iban) { + $texto_pago .= $cbc->iban(TRUE); + } + + if ($cbc->swift) { + $texto_pago .= "\nSWIFT/BIC: " . $cbc->swift; + } + $encontrada = TRUE; + break; + } + if (!$encontrada) { + $texto_pago .= "\nEl cliente no tiene cuenta bancaria asignada."; + } + } else if ($forma_pago->codcuenta) { + $cb0 = new cuenta_banco(); + $cuenta_banco = $cb0->get($forma_pago->codcuenta); + if ($cuenta_banco) { + if ($cuenta_banco->iban) { + $texto_pago .= "\nIBAN: " . $cuenta_banco->iban(TRUE); + } + + if ($cuenta_banco->swift) { + $texto_pago .= "\nSWIFT o BIC: " . $cuenta_banco->swift; + } + } + } + + if (isset($this->documento->vencimiento)) { + $texto_pago .= "\nVencimiento: " . $this->documento->vencimiento; + } + + $pdf_doc->pdf->ezText($texto_pago, 9); + } + } + public function enviar_email($doc, $tipo = 'simple') { if ($this->empresa->can_send_mail()) { @@ -777,7 +784,7 @@ public function enviar_email($doc, $tipo = 'simple') } $razonsocial = $this->documento->nombrecliente; - if ($this->cliente && $_POST['email'] != $this->cliente->email AND isset($_POST['guardar'])) { + if ($this->cliente && $_POST['email'] != $this->cliente->email && isset($_POST['guardar'])) { $this->cliente->email = $_POST['email']; $this->cliente->save(); } @@ -822,18 +829,14 @@ public function enviar_email($doc, $tipo = 'simple') $mail->addAttachment($_FILES['adjunto']['tmp_name'], $_FILES['adjunto']['name']); } - if ($this->empresa->mail_connect($mail)) { - if ($mail->send()) { - $this->new_message('Mensaje enviado correctamente.'); + if ($this->empresa->mail_connect($mail) && $mail->send()) { + $this->new_message('Mensaje enviado correctamente.'); - /// nos guardamos la fecha de envío - $this->documento->femail = $this->today(); - $this->documento->save(); + /// nos guardamos la fecha de envío + $this->documento->femail = $this->today(); + $this->documento->save(); - $this->empresa->save_mail($mail); - } else { - $this->new_error_msg("Error al enviar el email: " . $mail->ErrorInfo); - } + $this->empresa->save_mail($mail); } else { $this->new_error_msg("Error al enviar el email: " . $mail->ErrorInfo); } diff --git a/controller/ventas_maquetar.php b/controller/ventas_maquetar.php index 9bfdcb3..67d834d 100644 --- a/controller/ventas_maquetar.php +++ b/controller/ventas_maquetar.php @@ -146,15 +146,12 @@ public function url() switch (get_class_name($this->documento)) { case 'albaran_cliente': return 'index.php?page=' . __CLASS__ . '&albaran=TRUE&id=' . $this->documento->idalbaran; - break; case 'factura_cliente': return 'index.php?page=' . __CLASS__ . '&factura=TRUE&id=' . $this->documento->idfactura; - break; default: return parent::url(); - break; } } } diff --git a/controller/ventas_trazabilidad.php b/controller/ventas_trazabilidad.php index baaf10c..f522b88 100644 --- a/controller/ventas_trazabilidad.php +++ b/controller/ventas_trazabilidad.php @@ -40,7 +40,7 @@ public function __construct() protected function private_core() { $this->documento = FALSE; - if (isset($_GET['doc']) AND isset($_GET['id'])) { + if (isset($_GET['doc']) && isset($_GET['id'])) { if ($_GET['doc'] == 'albaran') { $alb = new albaran_cliente(); $this->documento = $alb->get($_GET['id']); @@ -120,7 +120,6 @@ public function url() private function asignar() { - $art0 = new articulo(); $at0 = new articulo_traza(); $ok = TRUE; diff --git a/extras/fbase_controller.php b/extras/fbase_controller.php index 3b0cab3..1a36a59 100644 --- a/extras/fbase_controller.php +++ b/extras/fbase_controller.php @@ -136,7 +136,7 @@ protected function fbase_paginas($url, $total, $offset) * descartamos todo excepto la primera, la última, la de enmedio, * la actual, las 5 anteriores y las 5 siguientes */ - if (($j > 1 AND $j < $actual - 5 AND $j != $enmedio) OR ( $j > $actual + 5 AND $j < $i - 1 AND $j != $enmedio)) { + if (($j > 1 && $j < $actual - 5 && $j != $enmedio) || ( $j > $actual + 5 && $j < $i - 1 && $j != $enmedio)) { unset($paginas[$j]); } } diff --git a/extras/inventarios_balances.php b/extras/inventarios_balances.php index 76a2ae3..878d774 100644 --- a/extras/inventarios_balances.php +++ b/extras/inventarios_balances.php @@ -48,7 +48,7 @@ public function cron_job() $ejercicios = $ejercicio->all(); $random = mt_rand(0, count($ejercicios) - 1); foreach ($ejercicios as $num => $eje) { - if ($num < 2 OR $num == $random) { + if ($num < 2 || $num == $random) { $this->generar_libro($eje); } } @@ -75,7 +75,7 @@ private function generar_libro(&$eje) $pdf_doc->pdf->ezStartPageNumbers(580, 10, 10, 'left', '{PAGENUM} de {TOTALPAGENUM}'); $excluir = FALSE; - if (isset($eje->idasientocierre) AND isset($eje->idasientopyg)) { + if (isset($eje->idasientocierre) && isset($eje->idasientopyg)) { $excluir = array($eje->idasientocierre, $eje->idasientopyg); } @@ -234,7 +234,7 @@ public function sumas_y_saldos(&$pdf_doc, &$eje, $titulo, $fechaini, $fechafin, } /// añadimos la cuenta - if ($debe != 0 OR $haber != 0) { + if ($debe != 0 || $haber != 0) { $cuenta = $cuenta0->get_by_codigo($i . $j . $k, $eje->codejercicio); if ($cuenta) { $lineas[] = array( @@ -416,7 +416,7 @@ private function perdidas_y_ganancias(&$pdf_doc, &$eje, $np = TRUE) $encontrado = FALSE; foreach ($balances as $bal) { - if ($bal->naturaleza == 'PG' AND strstr($bal->codbalance, 'PG-A-' . $num . '-')) { + if ($bal->naturaleza == 'PG' && strstr($bal->codbalance, 'PG-A-' . $num . '-')) { $saldo1 = $this->get_saldo_balance('PG-A-' . $num . '-', $eje); $saldo0 = $this->get_saldo_balance('PG-A-' . $num . '-', $eje0); @@ -511,7 +511,7 @@ private function perdidas_y_ganancias(&$pdf_doc, &$eje, $np = TRUE) $encontrado = FALSE; foreach ($balances as $bal) { - if ($bal->naturaleza == 'PG' AND strstr($bal->codbalance, 'PG-A-' . $num . '-')) { + if ($bal->naturaleza == 'PG' && strstr($bal->codbalance, 'PG-A-' . $num . '-')) { $saldo1 = $this->get_saldo_balance('PG-A-' . $num . '-', $eje); /// añadimos la fila @@ -615,14 +615,13 @@ private function situacion(&$pdf_doc, &$eje, $np = TRUE) $desc1 = ''; $desc2 = ''; $desc3 = ''; - $desc4 = ''; foreach ($nivel1 as $nv1) { foreach ($nivel2 as $nv2) { foreach ($nivel3 as $nv3) { foreach ($nivel4 as $nv4) { foreach ($balances as $bal) { - if ($bal->naturaleza == $nv0 AND $bal->nivel1 == $nv1 AND $bal->nivel2 == $nv2 AND $bal->nivel3 == $nv3 AND $bal->nivel4 == $nv4) { - if ($bal->descripcion1 != $desc1 AND $bal->descripcion1 != '') { + if ($bal->naturaleza == $nv0 && $bal->nivel1 == $nv1 && $bal->nivel2 == $nv2 && $bal->nivel3 == $nv3 && $bal->nivel4 == $nv4) { + if ($bal->descripcion1 != $desc1 && $bal->descripcion1 != '') { $pdf_doc->add_table_row( array( 'descripcion' => "\n" . fs_fix_html($bal->descripcion1) . '', @@ -634,7 +633,7 @@ private function situacion(&$pdf_doc, &$eje, $np = TRUE) $desc1 = $bal->descripcion1; } - if ($bal->descripcion2 != $desc2 AND $bal->descripcion2 != '') { + if ($bal->descripcion2 != $desc2 && $bal->descripcion2 != '') { $pdf_doc->add_table_row( array( 'descripcion' => ' ' . fs_fix_html($bal->descripcion2) . '', @@ -646,7 +645,7 @@ private function situacion(&$pdf_doc, &$eje, $np = TRUE) $desc2 = $bal->descripcion2; } - if ($bal->descripcion3 != $desc3 AND $bal->descripcion3 != '') { + if ($bal->descripcion3 != $desc3 && $bal->descripcion3 != '') { $pdf_doc->add_table_row( array( 'descripcion' => ' ' . fs_fix_html($bal->descripcion3), @@ -723,14 +722,13 @@ private function situacion(&$pdf_doc, &$eje, $np = TRUE) $desc1 = ''; $desc2 = ''; $desc3 = ''; - $desc4 = ''; foreach ($nivel1 as $nv1) { foreach ($nivel2 as $nv2) { foreach ($nivel3 as $nv3) { foreach ($nivel4 as $nv4) { foreach ($balances as $bal) { - if ($bal->naturaleza == $nv0 AND $bal->nivel1 == $nv1 AND $bal->nivel2 == $nv2 AND $bal->nivel3 == $nv3 AND $bal->nivel4 == $nv4) { - if ($bal->descripcion1 != $desc1 AND $bal->descripcion1 != '') { + if ($bal->naturaleza == $nv0 && $bal->nivel1 == $nv1 && $bal->nivel2 == $nv2 && $bal->nivel3 == $nv3 && $bal->nivel4 == $nv4) { + if ($bal->descripcion1 != $desc1 && $bal->descripcion1 != '') { $pdf_doc->add_table_row( array( 'descripcion' => "\n" . fs_fix_html($bal->descripcion1) . '', @@ -741,7 +739,7 @@ private function situacion(&$pdf_doc, &$eje, $np = TRUE) $desc1 = $bal->descripcion1; } - if ($bal->descripcion2 != $desc2 AND $bal->descripcion2 != '') { + if ($bal->descripcion2 != $desc2 && $bal->descripcion2 != '') { $pdf_doc->add_table_row( array( 'descripcion' => ' ' . fs_fix_html($bal->descripcion2) . '', @@ -752,7 +750,7 @@ private function situacion(&$pdf_doc, &$eje, $np = TRUE) $desc2 = $bal->descripcion2; } - if ($bal->descripcion3 != $desc3 AND $bal->descripcion3 != '') { + if ($bal->descripcion3 != $desc3 && $bal->descripcion3 != '') { $pdf_doc->add_table_row( array( 'descripcion' => ' ' . fs_fix_html($bal->descripcion3), diff --git a/extras/libromayor.php b/extras/libromayor.php index 9edee9a..55c7e3c 100644 --- a/extras/libromayor.php +++ b/extras/libromayor.php @@ -42,7 +42,7 @@ public function cron_job() $ejercicios = $this->ejercicio->all(); $random = mt_rand(0, count($ejercicios) - 1); foreach ($ejercicios as $num => $eje) { - if ($num < 2 OR $num == $random) { + if ($num < 2 || $num == $random) { foreach ($this->subcuenta->all_from_ejercicio($eje->codejercicio, TRUE, 300) as $sc) { /** * Recalcular los saldos de las subcuentas es un proceso lento, @@ -129,7 +129,7 @@ public function libro_mayor(&$subc, $echos = FALSE) 'saldo' => 'Saldo' ) ); - for ($i = $linea_actual; (($linea_actual < ($lppag + $i)) AND ( $linea_actual < $lineasfact));) { + for ($i = $linea_actual; (($linea_actual < ($lppag + $i)) && ( $linea_actual < $lineasfact));) { $pdf_doc->add_table_row( array( 'asiento' => $partidas[$linea_actual]->numero, diff --git a/facturascripts.ini b/facturascripts.ini index 19fe3e6..359a794 100644 --- a/facturascripts.ini +++ b/facturascripts.ini @@ -1,4 +1,4 @@ -version = 127 +version = 128 version_url = 'https://raw.githubusercontent.com/NeoRazorX/facturacion_base/master/facturascripts.ini' update_url = 'https://github.com/NeoRazorX/facturacion_base/archive/master.zip' wizard = 'base_wizard' diff --git a/functions.php b/functions.php index f3cbe1a..622297d 100644 --- a/functions.php +++ b/functions.php @@ -57,7 +57,7 @@ function fs_documento_new_numero(&$db, $table_name, $codejercicio, $codserie, $n $sec = FALSE; } - if (!$sec OR $numero <= 1) { + if (!$sec || $numero <= 1) { $sql = "SELECT MAX(" . $db->sql_to_int('numero') . ") as num FROM " . $table_name; if (FS_NEW_CODIGO != 'NUM' && FS_NEW_CODIGO != '0-NUM') { $sql .= " WHERE codejercicio = " . $sec0->var2str($codejercicio) . " AND codserie = " . $sec0->var2str($codserie) . ";"; @@ -166,7 +166,7 @@ function fs_huecos_facturas_cliente(&$db, $table_name) * así evitamos agotar la memoria en caso de error grave. */ $pasos = 0; - while ($num < intval($d['numero']) AND $pasos < 100) { + while ($num < intval($d['numero']) && $pasos < 100) { $huecolist[] = array( 'codigo' => fs_documento_new_codigo(FS_FACTURA, $eje->codejercicio, $codserie, $num), 'fecha' => Date('d-m-Y', strtotime($d['fecha'])), @@ -205,8 +205,9 @@ function remote_printer() $terminal->tickets = ''; $terminal->save(); - } else + } else { echo 'ERROR: terminal no encontrado.'; + } } } } diff --git a/model/cierre_ejercicio.php b/model/cierre_ejercicio.php index 01c650a..dc40335 100644 --- a/model/cierre_ejercicio.php +++ b/model/cierre_ejercicio.php @@ -185,7 +185,7 @@ public function paso2() */ $diferencia = 0; foreach ($subcuenta->all_from_ejercicio($this->ejercicio->codejercicio) as $sc) { - if (in_array(substr($sc->codcuenta, 0, 1), array('6', '7')) AND $sc->tiene_saldo()) { + if (in_array(substr($sc->codcuenta, 0, 1), array('6', '7')) && $sc->tiene_saldo()) { $ppyg = new partida(); $ppyg->idasiento = $asiento_pyg->idasiento; $ppyg->concepto = $asiento_pyg->concepto; diff --git a/model/core/agencia_transporte.php b/model/core/agencia_transporte.php index 1ee41c9..7746125 100644 --- a/model/core/agencia_transporte.php +++ b/model/core/agencia_transporte.php @@ -47,15 +47,15 @@ class agencia_transporte extends \fs_model */ public $activo; - public function __construct($p = FALSE) + public function __construct($data = FALSE) { parent::__construct('agenciastrans'); - if ($p) { - $this->codtrans = $p['codtrans']; - $this->nombre = $p['nombre']; - $this->telefono = $p['telefono']; - $this->web = $p['web']; - $this->activo = $this->str2bool($p['activo']); + if ($data) { + $this->codtrans = $data['codtrans']; + $this->nombre = $data['nombre']; + $this->telefono = $data['telefono']; + $this->web = $data['web']; + $this->activo = $this->str2bool($data['activo']); } else { $this->codtrans = NULL; $this->nombre = NULL; diff --git a/model/core/albaran_cliente.php b/model/core/albaran_cliente.php index a2b4500..ea6c8cc 100644 --- a/model/core/albaran_cliente.php +++ b/model/core/albaran_cliente.php @@ -214,77 +214,77 @@ class albaran_cliente extends \fs_model */ public $numdocs; - public function __construct($a = FALSE) + public function __construct($data = FALSE) { parent::__construct('albaranescli'); - if ($a) { - $this->idalbaran = $this->intval($a['idalbaran']); + if ($data) { + $this->idalbaran = $this->intval($data['idalbaran']); - $this->idfactura = $this->intval($a['idfactura']); + $this->idfactura = $this->intval($data['idfactura']); if ($this->idfactura == 0) { $this->idfactura = NULL; } - $this->ptefactura = $this->str2bool($a['ptefactura']); + $this->ptefactura = $this->str2bool($data['ptefactura']); if ($this->idfactura) { /// si ya hay una factura enlazada, no puede estar pendiente de factura $this->ptefactura = FALSE; } - $this->codigo = $a['codigo']; - $this->codagente = $a['codagente']; - $this->codserie = $a['codserie']; - $this->codejercicio = $a['codejercicio']; - $this->codcliente = $a['codcliente']; - $this->codpago = $a['codpago']; - $this->coddivisa = $a['coddivisa']; - $this->codalmacen = $a['codalmacen']; - $this->codpais = $a['codpais']; - $this->coddir = $a['coddir']; - $this->codpostal = $a['codpostal']; - $this->numero = $a['numero']; - $this->numero2 = $a['numero2']; - $this->nombrecliente = $a['nombrecliente']; - $this->cifnif = $a['cifnif']; - $this->direccion = $a['direccion']; - $this->ciudad = $a['ciudad']; - $this->provincia = $a['provincia']; - $this->apartado = $a['apartado']; - $this->fecha = Date('d-m-Y', strtotime($a['fecha'])); + $this->codigo = $data['codigo']; + $this->codagente = $data['codagente']; + $this->codserie = $data['codserie']; + $this->codejercicio = $data['codejercicio']; + $this->codcliente = $data['codcliente']; + $this->codpago = $data['codpago']; + $this->coddivisa = $data['coddivisa']; + $this->codalmacen = $data['codalmacen']; + $this->codpais = $data['codpais']; + $this->coddir = $data['coddir']; + $this->codpostal = $data['codpostal']; + $this->numero = $data['numero']; + $this->numero2 = $data['numero2']; + $this->nombrecliente = $data['nombrecliente']; + $this->cifnif = $data['cifnif']; + $this->direccion = $data['direccion']; + $this->ciudad = $data['ciudad']; + $this->provincia = $data['provincia']; + $this->apartado = $data['apartado']; + $this->fecha = Date('d-m-Y', strtotime($data['fecha'])); $this->hora = '00:00:00'; - if (!is_null($a['hora'])) { - $this->hora = date('H:i:s', strtotime($a['hora'])); + if (!is_null($data['hora'])) { + $this->hora = date('H:i:s', strtotime($data['hora'])); } - $this->neto = floatval($a['neto']); - $this->total = floatval($a['total']); - $this->totaliva = floatval($a['totaliva']); - $this->totaleuros = floatval($a['totaleuros']); - $this->irpf = floatval($a['irpf']); - $this->totalirpf = floatval($a['totalirpf']); - $this->porcomision = floatval($a['porcomision']); - $this->tasaconv = floatval($a['tasaconv']); - $this->totalrecargo = floatval($a['totalrecargo']); - $this->observaciones = $this->no_html($a['observaciones']); + $this->neto = floatval($data['neto']); + $this->total = floatval($data['total']); + $this->totaliva = floatval($data['totaliva']); + $this->totaleuros = floatval($data['totaleuros']); + $this->irpf = floatval($data['irpf']); + $this->totalirpf = floatval($data['totalirpf']); + $this->porcomision = floatval($data['porcomision']); + $this->tasaconv = floatval($data['tasaconv']); + $this->totalrecargo = floatval($data['totalrecargo']); + $this->observaciones = $this->no_html($data['observaciones']); $this->femail = NULL; - if (!is_null($a['femail'])) { - $this->femail = Date('d-m-Y', strtotime($a['femail'])); + if (!is_null($data['femail'])) { + $this->femail = Date('d-m-Y', strtotime($data['femail'])); } - $this->envio_codtrans = $a['codtrans']; - $this->envio_codigo = $a['codigoenv']; - $this->envio_nombre = $a['nombreenv']; - $this->envio_apellidos = $a['apellidosenv']; - $this->envio_apartado = $a['apartadoenv']; - $this->envio_direccion = $a['direccionenv']; - $this->envio_codpostal = $a['codpostalenv']; - $this->envio_ciudad = $a['ciudadenv']; - $this->envio_provincia = $a['provinciaenv']; - $this->envio_codpais = $a['codpaisenv']; - - $this->numdocs = intval($a['numdocs']); + $this->envio_codtrans = $data['codtrans']; + $this->envio_codigo = $data['codigoenv']; + $this->envio_nombre = $data['nombreenv']; + $this->envio_apellidos = $data['apellidosenv']; + $this->envio_apartado = $data['apartadoenv']; + $this->envio_direccion = $data['direccionenv']; + $this->envio_codpostal = $data['codpostalenv']; + $this->envio_ciudad = $data['ciudadenv']; + $this->envio_provincia = $data['provinciaenv']; + $this->envio_codpais = $data['codpaisenv']; + + $this->numdocs = intval($data['numdocs']); } else { $this->idalbaran = NULL; $this->idfactura = NULL; @@ -570,7 +570,7 @@ public function full_test($duplicados = TRUE) } } - if ($status AND $duplicados) { + if ($status && $duplicados) { /// comprobamos si es un duplicado $data = $this->db->select("SELECT * FROM " . $this->table_name . " WHERE fecha = " . $this->var2str($this->fecha) . " AND codcliente = " . $this->var2str($this->codcliente) diff --git a/model/core/albaran_proveedor.php b/model/core/albaran_proveedor.php index 3afb646..832f513 100644 --- a/model/core/albaran_proveedor.php +++ b/model/core/albaran_proveedor.php @@ -170,52 +170,52 @@ class albaran_proveedor extends \fs_model */ public $numdocs; - public function __construct($a = FALSE) + public function __construct($data = FALSE) { parent::__construct('albaranesprov'); - if ($a) { - $this->idalbaran = $this->intval($a['idalbaran']); + if ($data) { + $this->idalbaran = $this->intval($data['idalbaran']); - $this->idfactura = $this->intval($a['idfactura']); + $this->idfactura = $this->intval($data['idfactura']); if ($this->idfactura == 0) { $this->idfactura = NULL; } - $this->ptefactura = $this->str2bool($a['ptefactura']); + $this->ptefactura = $this->str2bool($data['ptefactura']); if ($this->idfactura) { /// si ya hay una factura enlazada, no puede estar pendiente de factura $this->ptefactura = FALSE; } - $this->codigo = $a['codigo']; - $this->numero = $a['numero']; - $this->numproveedor = $a['numproveedor']; - $this->codejercicio = $a['codejercicio']; - $this->codserie = $a['codserie']; - $this->coddivisa = $a['coddivisa']; - $this->codpago = $a['codpago']; - $this->codagente = $a['codagente']; - $this->codalmacen = $a['codalmacen']; - $this->fecha = Date('d-m-Y', strtotime($a['fecha'])); + $this->codigo = $data['codigo']; + $this->numero = $data['numero']; + $this->numproveedor = $data['numproveedor']; + $this->codejercicio = $data['codejercicio']; + $this->codserie = $data['codserie']; + $this->coddivisa = $data['coddivisa']; + $this->codpago = $data['codpago']; + $this->codagente = $data['codagente']; + $this->codalmacen = $data['codalmacen']; + $this->fecha = Date('d-m-Y', strtotime($data['fecha'])); $this->hora = '00:00:00'; - if (!is_null($a['hora'])) { - $this->hora = date('H:i:s', strtotime($a['hora'])); + if (!is_null($data['hora'])) { + $this->hora = date('H:i:s', strtotime($data['hora'])); } - $this->codproveedor = $a['codproveedor']; - $this->nombre = $a['nombre']; - $this->cifnif = $a['cifnif']; - $this->neto = floatval($a['neto']); - $this->total = floatval($a['total']); - $this->totaliva = floatval($a['totaliva']); - $this->totaleuros = floatval($a['totaleuros']); - $this->irpf = floatval($a['irpf']); - $this->totalirpf = floatval($a['totalirpf']); - $this->tasaconv = floatval($a['tasaconv']); - $this->totalrecargo = floatval($a['totalrecargo']); - $this->observaciones = $this->no_html($a['observaciones']); - $this->numdocs = intval($a['numdocs']); + $this->codproveedor = $data['codproveedor']; + $this->nombre = $data['nombre']; + $this->cifnif = $data['cifnif']; + $this->neto = floatval($data['neto']); + $this->total = floatval($data['total']); + $this->totaliva = floatval($data['totaliva']); + $this->totaleuros = floatval($data['totaleuros']); + $this->irpf = floatval($data['irpf']); + $this->totalirpf = floatval($data['totalirpf']); + $this->tasaconv = floatval($data['tasaconv']); + $this->totalrecargo = floatval($data['totalrecargo']); + $this->observaciones = $this->no_html($data['observaciones']); + $this->numdocs = intval($data['numdocs']); } else { $this->idalbaran = NULL; $this->idfactura = NULL; @@ -450,7 +450,7 @@ public function full_test($duplicados = TRUE) } } - if ($status AND $duplicados) { + if ($status && $duplicados) { /// comprobamos si es un duplicado $data = $this->db->select("SELECT * FROM " . $this->table_name . " WHERE fecha = " . $this->var2str($this->fecha) . " AND codproveedor = " . $this->var2str($this->codproveedor) diff --git a/model/core/articulo.php b/model/core/articulo.php index 0a63e0e..2ab5385 100644 --- a/model/core/articulo.php +++ b/model/core/articulo.php @@ -186,7 +186,7 @@ class articulo extends \fs_model private static $cleaned_cache; private static $column_list; - public function __construct($a = FALSE) + public function __construct($data = FALSE) { parent::__construct('articulos'); @@ -198,38 +198,38 @@ public function __construct($a = FALSE) 'partnumber,codsubcuentacom,codsubcuentairpfcom,trazabilidad'; } - if ($a) { - $this->referencia = $a['referencia']; - $this->tipo = $a['tipo']; - $this->codfamilia = $a['codfamilia']; - $this->codfabricante = $a['codfabricante']; - $this->descripcion = $this->no_html($a['descripcion']); - $this->pvp = floatval($a['pvp']); - $this->factualizado = Date('d-m-Y', strtotime($a['factualizado'])); - $this->costemedio = floatval($a['costemedio']); - $this->preciocoste = floatval($a['preciocoste']); - $this->codimpuesto = $a['codimpuesto']; - $this->stockfis = floatval($a['stockfis']); - $this->stockmin = floatval($a['stockmin']); - $this->stockmax = floatval($a['stockmax']); - - $this->controlstock = $this->str2bool($a['controlstock']); - $this->nostock = $this->str2bool($a['nostock']); + if ($data) { + $this->referencia = $data['referencia']; + $this->tipo = $data['tipo']; + $this->codfamilia = $data['codfamilia']; + $this->codfabricante = $data['codfabricante']; + $this->descripcion = $this->no_html($data['descripcion']); + $this->pvp = floatval($data['pvp']); + $this->factualizado = Date('d-m-Y', strtotime($data['factualizado'])); + $this->costemedio = floatval($data['costemedio']); + $this->preciocoste = floatval($data['preciocoste']); + $this->codimpuesto = $data['codimpuesto']; + $this->stockfis = floatval($data['stockfis']); + $this->stockmin = floatval($data['stockmin']); + $this->stockmax = floatval($data['stockmax']); + + $this->controlstock = $this->str2bool($data['controlstock']); + $this->nostock = $this->str2bool($data['nostock']); if ($this->nostock) { $this->controlstock = TRUE; } - $this->bloqueado = $this->str2bool($a['bloqueado']); - $this->secompra = $this->str2bool($a['secompra']); - $this->sevende = $this->str2bool($a['sevende']); - $this->publico = $this->str2bool($a['publico']); - $this->equivalencia = $a['equivalencia']; - $this->partnumber = $a['partnumber']; - $this->codbarras = $a['codbarras']; - $this->observaciones = $this->no_html($a['observaciones']); - $this->codsubcuentacom = $a['codsubcuentacom']; - $this->codsubcuentairpfcom = $a['codsubcuentairpfcom']; - $this->trazabilidad = $this->str2bool($a['trazabilidad']); + $this->bloqueado = $this->str2bool($data['bloqueado']); + $this->secompra = $this->str2bool($data['secompra']); + $this->sevende = $this->str2bool($data['sevende']); + $this->publico = $this->str2bool($data['publico']); + $this->equivalencia = $data['equivalencia']; + $this->partnumber = $data['partnumber']; + $this->codbarras = $data['codbarras']; + $this->observaciones = $this->no_html($data['observaciones']); + $this->codsubcuentacom = $data['codsubcuentacom']; + $this->codsubcuentairpfcom = $data['codsubcuentairpfcom']; + $this->trazabilidad = $this->str2bool($data['trazabilidad']); $this->imagen = NULL; $this->exists = TRUE; } else { @@ -304,7 +304,7 @@ public function pvp_iva() */ public function preciocoste() { - if ($this->secompra AND FS_COST_IS_AVERAGE) { + if ($this->secompra && FS_COST_IS_AVERAGE) { return $this->costemedio; } @@ -540,14 +540,14 @@ public function get_costemedio() * Ahora comprobamos la fecha del primer elemento de una y otra lista * para ver cual usamos. */ - if ($lineasfac AND $lineasalb) { + if ($lineasfac && $lineasalb) { if (strtotime($lineasalb[0]->show_fecha()) > strtotime($lineasfac[0]->show_fecha())) { /** * la fecha del último albarán es posterior a la de la última factura. * Usamos los albaranes para el cálculo. */ foreach ($lineasalb as $linea) { - if ($stock < $this->stockfis OR $this->stockfis <= 0) { + if ($stock < $this->stockfis || $this->stockfis <= 0) { $coste += $linea->pvptotal; $stock += $linea->cantidad; } else { @@ -560,7 +560,7 @@ public function get_costemedio() if (!empty($lineasfac)) { /// usamos las facturas para el cálculo. foreach ($lineasfac as $linea) { - if ($stock < $this->stockfis OR $this->stockfis <= 0) { + if ($stock < $this->stockfis || $this->stockfis <= 0) { $coste += $linea->pvptotal; $stock += $linea->cantidad; } else { @@ -572,7 +572,7 @@ public function get_costemedio() if (!empty($lineasalb)) { /// usamos los albaranes para el cálculo. foreach ($lineasalb as $linea) { - if ($stock < $this->stockfis OR $this->stockfis <= 0) { + if ($stock < $this->stockfis || $this->stockfis <= 0) { $coste += $linea->pvptotal; $stock += $linea->cantidad; } else { @@ -660,9 +660,9 @@ public function set_pvp_iva($p) public function set_referencia($ref) { $ref = trim($ref); - if (is_null($ref) OR strlen($ref) < 1 OR strlen($ref) > 18) { + if (is_null($ref) || strlen($ref) < 1 || strlen($ref) > 18) { $this->new_error_msg("¡Referencia de artículo no válida! Debe tener entre 1 y 18 caracteres."); - } else if ($ref != $this->referencia AND ! is_null($this->referencia)) { + } else if ($ref != $this->referencia && ! is_null($this->referencia)) { $sql = "UPDATE " . $this->table_name . " SET referencia = " . $this->var2str($ref) . " WHERE referencia = " . $this->var2str($this->referencia) . ";"; if ($this->db->exec($sql)) { @@ -837,7 +837,7 @@ public function sum_stock($codalmacen, $cantidad = 1, $recalcula_coste = FALSE, } /// ¿Alguna combinación? - if ($codcombinacion AND $result) { + if ($codcombinacion && $result) { $com0 = new \articulo_combinacion(); foreach ($com0->all_from_codigo($codcombinacion) as $combi) { $combi->stockfis += $cantidad; @@ -897,9 +897,9 @@ public function test() $this->publico = FALSE; } - if (is_null($this->referencia) OR strlen($this->referencia) < 1 OR strlen($this->referencia) > 18) { + if (is_null($this->referencia) || strlen($this->referencia) < 1 || strlen($this->referencia) > 18) { $this->new_error_msg("Referencia de artículo no válida: " . $this->referencia . ". Debe tener entre 1 y 18 caracteres."); - } else if (isset($this->equivalencia) AND strlen($this->equivalencia) > 25) { + } else if (isset($this->equivalencia) && strlen($this->equivalencia) > 25) { $this->new_error_msg("Código de equivalencia del artículos no válido: " . $this->equivalencia . ". Debe tener entre 1 y 25 caracteres."); } else { @@ -947,7 +947,7 @@ public function save() ", trazabilidad = " . $this->var2str($this->trazabilidad) . " WHERE referencia = " . $this->var2str($this->referencia) . ";"; - if ($this->nostock AND $this->stockfis != 0) { + if ($this->nostock && $this->stockfis != 0) { $this->stockfis = 0.0; $sql .= "DELETE FROM stocks WHERE referencia = " . $this->var2str($this->referencia) . ";"; $sql .= "UPDATE " . $this->table_name . " SET stockfis = " . $this->var2str($this->stockfis) . @@ -1096,7 +1096,7 @@ public function search($query = '', $offset = 0, $codfamilia = '', $con_stock = $artilist = array(); $query = $this->no_html(mb_strtolower($query, 'UTF8')); - if ($query != '' AND $offset == 0 AND $codfamilia == '' AND $codfabricante == '' AND ! $con_stock AND ! $bloqueados) { + if ($query != '' && $offset == 0 && $codfamilia == '' && $codfabricante == '' && ! $con_stock && ! $bloqueados) { /// intentamos obtener los datos de memcache if ($this->new_search_tag($query)) { $artilist = $this->cache->get_array('articulos_search_' . $query); diff --git a/model/core/articulo_combinacion.php b/model/core/articulo_combinacion.php index cabca1e..3672a19 100644 --- a/model/core/articulo_combinacion.php +++ b/model/core/articulo_combinacion.php @@ -104,21 +104,21 @@ class articulo_combinacion extends \fs_model */ public $stockfis; - public function __construct($c = FALSE) + public function __construct($data = FALSE) { parent::__construct('articulo_combinaciones'); - if ($c) { - $this->id = $this->intval($c['id']); - $this->codigo = $c['codigo']; - $this->codigo2 = $c['codigo2']; - $this->referencia = $c['referencia']; - $this->idvalor = $this->intval($c['idvalor']); - $this->nombreatributo = $c['nombreatributo']; - $this->valor = $c['valor']; - $this->refcombinacion = $c['refcombinacion']; - $this->codbarras = $c['codbarras']; - $this->impactoprecio = floatval($c['impactoprecio']); - $this->stockfis = floatval($c['stockfis']); + if ($data) { + $this->id = $this->intval($data['id']); + $this->codigo = $data['codigo']; + $this->codigo2 = $data['codigo2']; + $this->referencia = $data['referencia']; + $this->idvalor = $this->intval($data['idvalor']); + $this->nombreatributo = $data['nombreatributo']; + $this->valor = $data['valor']; + $this->refcombinacion = $data['refcombinacion']; + $this->codbarras = $data['codbarras']; + $this->impactoprecio = floatval($data['impactoprecio']); + $this->stockfis = floatval($data['stockfis']); } else { $this->id = NULL; $this->codigo = NULL; diff --git a/model/core/articulo_propiedad.php b/model/core/articulo_propiedad.php index f6baaa6..6b26f15 100644 --- a/model/core/articulo_propiedad.php +++ b/model/core/articulo_propiedad.php @@ -31,13 +31,13 @@ class articulo_propiedad extends \fs_model public $referencia; public $text; - public function __construct($a = FALSE) + public function __construct($data = FALSE) { parent::__construct('articulo_propiedades'); - if ($a) { - $this->name = $a['name']; - $this->referencia = $a['referencia']; - $this->text = $a['text']; + if ($data) { + $this->name = $data['name']; + $this->referencia = $data['referencia']; + $this->text = $data['text']; } else { $this->name = NULL; $this->referencia = NULL; @@ -56,7 +56,7 @@ protected function install() */ public function exists() { - if (is_null($this->name) OR is_null($this->referencia)) { + if (is_null($this->name) || is_null($this->referencia)) { return FALSE; } diff --git a/model/core/articulo_proveedor.php b/model/core/articulo_proveedor.php index ac8bd95..a993074 100644 --- a/model/core/articulo_proveedor.php +++ b/model/core/articulo_proveedor.php @@ -91,7 +91,7 @@ class articulo_proveedor extends \fs_model private static $impuestos; private static $nombres; - public function __construct($a = FALSE) + public function __construct($data = FALSE) { parent::__construct('articulosprov'); @@ -103,25 +103,25 @@ public function __construct($a = FALSE) self::$nombres = array(); } - if ($a) { - $this->id = $this->intval($a['id']); - $this->referencia = $a['referencia']; - $this->codproveedor = $a['codproveedor']; - $this->refproveedor = $a['refproveedor']; - $this->descripcion = $a['descripcion']; + if ($data) { + $this->id = $this->intval($data['id']); + $this->referencia = $data['referencia']; + $this->codproveedor = $data['codproveedor']; + $this->refproveedor = $data['refproveedor']; + $this->descripcion = $data['descripcion']; /// En algunos módulos de eneboo se usa coste como precio - if (is_null($a['precio']) AND isset($a['coste'])) { - $this->precio = floatval($a['coste']); + if (is_null($data['precio']) && isset($data['coste'])) { + $this->precio = floatval($data['coste']); } else - $this->precio = floatval($a['precio']); - - $this->dto = floatval($a['dto']); - $this->codimpuesto = $a['codimpuesto']; - $this->stock = floatval($a['stock']); - $this->nostock = $this->str2bool($a['nostock']); - $this->codbarras = $a['codbarras']; - $this->partnumber = $a['partnumber']; + $this->precio = floatval($data['precio']); + + $this->dto = floatval($data['dto']); + $this->codimpuesto = $data['codimpuesto']; + $this->stock = floatval($data['stock']); + $this->nostock = $this->str2bool($data['nostock']); + $this->codbarras = $data['codbarras']; + $this->partnumber = $data['partnumber']; } else { $this->id = NULL; $this->referencia = NULL; @@ -283,7 +283,7 @@ public function test() $this->stock = 0.0; } - if (is_null($this->refproveedor) OR strlen($this->refproveedor) < 1 OR strlen($this->refproveedor) > 25) { + if (is_null($this->refproveedor) || strlen($this->refproveedor) < 1 || strlen($this->refproveedor) > 25) { $this->new_error_msg('La referencia de proveedor debe contener entre 1 y 25 caracteres.'); return FALSE; } diff --git a/model/core/articulo_traza.php b/model/core/articulo_traza.php index 606a4ea..892d0c8 100644 --- a/model/core/articulo_traza.php +++ b/model/core/articulo_traza.php @@ -80,27 +80,27 @@ class articulo_traza extends \fs_model public $fecha_entrada; public $fecha_salida; - public function __construct($n = FALSE) + public function __construct($data = FALSE) { parent::__construct('articulo_trazas'); - if ($n) { - $this->id = $this->intval($n['id']); - $this->referencia = $n['referencia']; - $this->numserie = $n['numserie']; - $this->lote = $n['lote']; - $this->idlalbventa = $this->intval($n['idlalbventa']); - $this->idlfacventa = $this->intval($n['idlfacventa']); - $this->idlalbcompra = $this->intval($n['idlalbcompra']); - $this->idlfaccompra = $this->intval($n['idlfaccompra']); + if ($data) { + $this->id = $this->intval($data['id']); + $this->referencia = $data['referencia']; + $this->numserie = $data['numserie']; + $this->lote = $data['lote']; + $this->idlalbventa = $this->intval($data['idlalbventa']); + $this->idlfacventa = $this->intval($data['idlfacventa']); + $this->idlalbcompra = $this->intval($data['idlalbcompra']); + $this->idlfaccompra = $this->intval($data['idlfaccompra']); $this->fecha_entrada = NULL; - if (isset($n['fecha_entrada'])) { - $this->fecha_entrada = date('d-m-Y', strtotime($n['fecha_entrada'])); + if (isset($data['fecha_entrada'])) { + $this->fecha_entrada = date('d-m-Y', strtotime($data['fecha_entrada'])); } $this->fecha_salida = NULL; - if (isset($n['fecha_salida']) AND ( $this->idlalbventa OR $this->idlfacventa)) { - $this->fecha_salida = date('d-m-Y', strtotime($n['fecha_salida'])); + if (isset($data['fecha_salida']) && ( $this->idlalbventa || $this->idlfacventa)) { + $this->fecha_salida = date('d-m-Y', strtotime($data['fecha_salida'])); } } else { $this->id = NULL; diff --git a/model/core/asiento.php b/model/core/asiento.php index 30aeca5..e0da0e0 100644 --- a/model/core/asiento.php +++ b/model/core/asiento.php @@ -48,26 +48,26 @@ class asiento extends \fs_model public $importe; private $coddivisa; - public function __construct($a = FALSE) + public function __construct($data = FALSE) { parent::__construct('co_asientos'); - if ($a) { - $this->idasiento = $this->intval($a['idasiento']); - $this->numero = $this->intval($a['numero']); - $this->idconcepto = $a['idconcepto']; + if ($data) { + $this->idasiento = $this->intval($data['idasiento']); + $this->numero = $this->intval($data['numero']); + $this->idconcepto = $data['idconcepto']; $this->concepto = '---'; - if ($a['concepto'] != '') { - $this->concepto = $a['concepto']; + if ($data['concepto'] != '') { + $this->concepto = $data['concepto']; } - $this->fecha = Date('d-m-Y', strtotime($a['fecha'])); - $this->codejercicio = $a['codejercicio']; - $this->codplanasiento = $a['codplanasiento']; - $this->editable = $this->str2bool($a['editable']); - $this->documento = $a['documento']; - $this->tipodocumento = $a['tipodocumento']; - $this->importe = floatval($a['importe']); + $this->fecha = Date('d-m-Y', strtotime($data['fecha'])); + $this->codejercicio = $data['codejercicio']; + $this->codplanasiento = $data['codplanasiento']; + $this->editable = $this->str2bool($data['editable']); + $this->documento = $data['documento']; + $this->tipodocumento = $data['tipodocumento']; + $this->importe = floatval($data['importe']); } else { $this->idasiento = NULL; $this->numero = NULL; @@ -161,9 +161,9 @@ public function coddivisa() public function get($id) { if (isset($id)) { - $asiento = $this->db->select("SELECT * FROM " . $this->table_name . " WHERE idasiento = " . $this->var2str($id) . ";"); - if ($asiento) { - return new \asiento($asiento[0]); + $data = $this->db->select("SELECT * FROM " . $this->table_name . " WHERE idasiento = " . $this->var2str($id) . ";"); + if ($data) { + return new \asiento($data[0]); } } @@ -263,13 +263,13 @@ public function full_test($duplicados = TRUE) $ejercicio = new \ejercicio(); $eje0 = $ejercicio->get($this->codejercicio); if ($eje0) { - if (strtotime($this->fecha) < strtotime($eje0->fechainicio) OR strtotime($this->fecha) > strtotime($eje0->fechafin)) { + if (strtotime($this->fecha) < strtotime($eje0->fechainicio) || strtotime($this->fecha) > strtotime($eje0->fechafin)) { $this->new_error_msg("La fecha de este asiento está fuera del rango del ejercicio."); $status = FALSE; } } - if ($status AND $duplicados) { + if ($status && $duplicados) { /// comprobamos si es un duplicado $asientos = $this->db->select("SELECT * FROM " . $this->table_name . " WHERE fecha = " . $this->var2str($this->fecha) . " AND concepto = " . $this->var2str($this->concepto) . " AND importe = " . $this->var2str($this->importe) . " @@ -310,7 +310,6 @@ public function fix() $debe += $p->debe; $haber += $p->haber; } - $total = $debe - $haber; $this->importe = max(array(abs($debe), abs($haber))); /// corregimos descuadres de menos de 0.01 @@ -551,7 +550,7 @@ public function renumerar() . " ORDER BY codejercicio ASC, fecha ASC, idasiento ASC"; $asientos = $this->db->select_limit($consulta, 1000, $posicion); - while ($asientos AND $continuar) { + while ($asientos && $continuar) { foreach ($asientos as $col) { if ($col['numero'] != $numero) { $sql .= "UPDATE " . $this->table_name . " SET numero = " . $this->var2str($numero) diff --git a/model/core/asiento_factura.php b/model/core/asiento_factura.php index 26fc604..48781c2 100644 --- a/model/core/asiento_factura.php +++ b/model/core/asiento_factura.php @@ -112,7 +112,6 @@ public function generar_asiento_compra(&$factura) $ok = FALSE; $this->asiento = FALSE; $proveedor0 = new \proveedor(); - $subcuenta_prov = FALSE; /// obtenemos las tasas de conversión, para las ocasiones en que la factura está en otra divisa $tasaconv = 1; @@ -178,9 +177,9 @@ public function generar_asiento_compra(&$factura) $subcuenta_iva = $this->subcuenta->get_cuentaesp('IVAREP', $asiento->codejercicio); } - if ($li->totaliva == 0 AND $li->totalrecargo == 0) { + if ($li->totaliva == 0 && $li->totalrecargo == 0) { /// no hacemos nada si no hay IVA ni RE - } else if ($subcuenta_iva AND $asiento_correcto) { + } else if ($subcuenta_iva && $asiento_correcto) { $partida1 = new \partida(); $partida1->idasiento = $asiento->idasiento; $partida1->concepto = $asiento->concepto; @@ -242,7 +241,7 @@ public function generar_asiento_compra(&$factura) } $subcuenta_compras = $this->subcuenta->get_cuentaesp('COMPRA', $asiento->codejercicio); - if ($subcuenta_compras AND $asiento_correcto) { + if ($subcuenta_compras && $asiento_correcto) { $partida2 = new \partida(); $partida2->idasiento = $asiento->idasiento; $partida2->concepto = $asiento->concepto; @@ -263,7 +262,7 @@ public function generar_asiento_compra(&$factura) } /// ¿IRPF? - if ($factura->totalirpf != 0 AND $asiento_correcto) { + if ($factura->totalirpf != 0 && $asiento_correcto) { $subcuenta_irpf = $this->subcuenta->get_cuentaesp('IRPFPR', $asiento->codejercicio); if ($subcuenta_irpf) { $partida3 = new \partida(); @@ -290,7 +289,7 @@ public function generar_asiento_compra(&$factura) if ($asiento_correcto) { $this->comprobar_subc_art_compra($factura, $asiento, $partida2, $subcuenta_compras, $asiento_correcto, $tasaconv, $tasaconv2); - if (isset($partida3) AND $asiento_correcto) { + if (isset($partida3) && $asiento_correcto) { $this->comprobar_subc_art_compra_irpf($factura, $asiento, $partida3, $subcuenta_irpf, $asiento_correcto, $tasaconv, $tasaconv2); } } @@ -323,7 +322,7 @@ public function generar_asiento_compra(&$factura) if ($asiento_correcto) { /// si es una factura negativa o rectificativa, invertimos los importes - if ($factura->idfacturarect OR $factura->total < 0) { + if ($factura->idfacturarect || $factura->total < 0) { $this->invertir_asiento($asiento); } @@ -414,7 +413,7 @@ protected function comprobar_subc_art_compra(&$factura, &$asiento, &$partida2, & } } - if ($partidaA->idsubcuenta AND $partidaA->codsubcuenta) { + if ($partidaA->idsubcuenta && $partidaA->codsubcuenta) { $partidaA->debe = round($partidaA->debe, FS_NF0); $restar += $partidaA->debe; if ($partidaA->save()) { @@ -427,7 +426,7 @@ protected function comprobar_subc_art_compra(&$factura, &$asiento, &$partida2, & } } else { $asiento_correcto = FALSE; - $this->new_error_msg("¡Imposible generar la partida para la subcuenta del artículo " . $lin->referencia . "!"); + $this->new_error_msg("¡Imposible generar la partida para la subcuenta " . $partidaA->codsubcuenta . "!"); } } } @@ -465,7 +464,7 @@ protected function comprobar_subc_art_compra_irpf(&$factura, &$asiento, &$partid $articulo = $art0->get($lin->referencia); } - if ($articulo AND $lin->irpf) { + if ($articulo && $lin->irpf) { if ($articulo->codsubcuentairpfcom) { $subcart = $this->subcuenta->get_by_codigo($articulo->codsubcuentairpfcom, $factura->codejercicio); } @@ -501,7 +500,7 @@ protected function comprobar_subc_art_compra_irpf(&$factura, &$asiento, &$partid } } - if ($partidaA->idsubcuenta AND $partidaA->codsubcuenta) { + if ($partidaA->idsubcuenta && $partidaA->codsubcuenta) { $partidaA->debe = round($partidaA->debe, FS_NF0); $restar += $partidaA->debe; if ($partidaA->save()) { @@ -514,7 +513,7 @@ protected function comprobar_subc_art_compra_irpf(&$factura, &$asiento, &$partid } } else { $asiento_correcto = FALSE; - $this->new_error_msg("¡Imposible generar la partida para la subcuenta del artículo " . $lin->referencia . "!"); + $this->new_error_msg("¡Imposible generar la partida para la subcuenta " . $partidaA->codsubcuenta . "!"); } } } @@ -530,7 +529,6 @@ public function generar_asiento_venta(&$factura) $ok = FALSE; $this->asiento = FALSE; $cliente0 = new \cliente(); - $subcuenta_cli = FALSE; /// obtenemos las tasas de conversión, para las ocasiones en que la factura está en otra divisa $tasaconv = 1; @@ -591,9 +589,9 @@ public function generar_asiento_venta(&$factura) $subcuenta_iva = $this->subcuenta->get_cuentaesp('IVAREP', $asiento->codejercicio); } - if ($li->totaliva == 0 AND $li->totalrecargo == 0) { + if ($li->totaliva == 0 && $li->totalrecargo == 0) { /// no hacemos nada si no hay IVA ni RE - } else if ($subcuenta_iva AND $asiento_correcto) { + } else if ($subcuenta_iva && $asiento_correcto) { $partida1 = new \partida(); $partida1->idasiento = $asiento->idasiento; $partida1->concepto = $asiento->concepto; @@ -655,7 +653,7 @@ public function generar_asiento_venta(&$factura) } $subcuenta_ventas = $this->subcuenta->get_cuentaesp('VENTAS', $asiento->codejercicio); - if ($subcuenta_ventas AND $asiento_correcto) { + if ($subcuenta_ventas && $asiento_correcto) { $partida2 = new \partida(); $partida2->idasiento = $asiento->idasiento; $partida2->concepto = $asiento->concepto; @@ -676,7 +674,7 @@ public function generar_asiento_venta(&$factura) } /// ¿IRPF? - if ($factura->totalirpf != 0 AND $asiento_correcto) { + if ($factura->totalirpf != 0 && $asiento_correcto) { $subcuenta_irpf = $this->subcuenta->get_cuentaesp('IRPF', $asiento->codejercicio); if (!$subcuenta_irpf) { @@ -758,7 +756,7 @@ public function generar_asiento_venta(&$factura) } } - if ($partidaA->idsubcuenta AND $partidaA->codsubcuenta) { + if ($partidaA->idsubcuenta && $partidaA->codsubcuenta) { $partidaA->haber = round($partidaA->haber, FS_NF0); $restar += $partidaA->haber; if ($partidaA->save()) { @@ -771,8 +769,7 @@ public function generar_asiento_venta(&$factura) } } else { $asiento_correcto = FALSE; - $this->new_error_msg("¡Imposible generar la partida para la subcuenta del artículo " - . $lin->referencia . "!"); + $this->new_error_msg("¡Imposible generar la partida para la subcuenta ". $partidaA->codsubcuenta . "!"); } } } @@ -806,7 +803,7 @@ public function generar_asiento_venta(&$factura) if ($asiento_correcto) { /// si es una factura negativa o rectificativa, invertimos los importes - if ($factura->idfacturarect OR $factura->total < 0) { + if ($factura->idfacturarect || $factura->total < 0) { $this->invertir_asiento($asiento); } @@ -916,10 +913,10 @@ public function generar_asiento_pago(&$asiento, $codpago = FALSE, $fecha = FALSE break; } - if ($nasientop->floatcmp(abs($par->debe), $importe, FS_NF0) OR $nasientop->floatcmp(abs($par->debe), $importe2, FS_NF0)) { + if ($nasientop->floatcmp(abs($par->debe), $importe, FS_NF0) || $nasientop->floatcmp(abs($par->debe), $importe2, FS_NF0)) { $seleccionar = $i; break; - } else if ($nasientop->floatcmp(abs($par->haber), $importe, FS_NF0) OR $nasientop->floatcmp(abs($par->haber), $importe2, FS_NF0)) { + } else if ($nasientop->floatcmp(abs($par->haber), $importe, FS_NF0) || $nasientop->floatcmp(abs($par->haber), $importe2, FS_NF0)) { $seleccionar = $i; break; } diff --git a/model/core/atributo.php b/model/core/atributo.php index 80c3102..9ddf37e 100644 --- a/model/core/atributo.php +++ b/model/core/atributo.php @@ -33,12 +33,12 @@ class atributo extends \fs_model public $codatributo; public $nombre; - public function __construct($a = FALSE) + public function __construct($data = FALSE) { parent::__construct('atributos'); - if ($a) { - $this->codatributo = $a['codatributo']; - $this->nombre = $a['nombre']; + if ($data) { + $this->codatributo = $data['codatributo']; + $this->nombre = $data['nombre']; } else { $this->codatributo = NULL; $this->nombre = NULL; diff --git a/model/core/atributo_valor.php b/model/core/atributo_valor.php index c08227a..0b3b225 100644 --- a/model/core/atributo_valor.php +++ b/model/core/atributo_valor.php @@ -39,13 +39,13 @@ class atributo_valor extends \fs_model public $codatributo; public $valor; - public function __construct($a = FALSE) + public function __construct($data = FALSE) { parent::__construct('atributos_valores'); - if ($a) { - $this->id = $this->intval($a['id']); - $this->codatributo = $a['codatributo']; - $this->valor = $a['valor']; + if ($data) { + $this->id = $this->intval($data['id']); + $this->codatributo = $data['codatributo']; + $this->valor = $data['valor']; } else { $this->id = NULL; $this->codatributo = NULL; diff --git a/model/core/caja.php b/model/core/caja.php index f94fc51..6668348 100644 --- a/model/core/caja.php +++ b/model/core/caja.php @@ -78,7 +78,7 @@ class caja extends \fs_model */ private static $agentes; - public function __construct($c = FALSE) + public function __construct($data = FALSE) { parent::__construct('cajas'); @@ -86,24 +86,24 @@ public function __construct($c = FALSE) self::$agentes = array(); } - if ($c) { - $this->id = $this->intval($c['id']); - $this->fs_id = $this->intval($c['fs_id']); - $this->fecha_inicial = Date('d-m-Y H:i:s', strtotime($c['f_inicio'])); - $this->dinero_inicial = floatval($c['d_inicio']); + if ($data) { + $this->id = $this->intval($data['id']); + $this->fs_id = $this->intval($data['fs_id']); + $this->fecha_inicial = Date('d-m-Y H:i:s', strtotime($data['f_inicio'])); + $this->dinero_inicial = floatval($data['d_inicio']); - if (is_null($c['f_fin'])) { + if (is_null($data['f_fin'])) { $this->fecha_fin = NULL; } else - $this->fecha_fin = Date('d-m-Y H:i:s', strtotime($c['f_fin'])); + $this->fecha_fin = Date('d-m-Y H:i:s', strtotime($data['f_fin'])); - $this->dinero_fin = floatval($c['d_fin']); - $this->codagente = $c['codagente']; - $this->tickets = intval($c['tickets']); + $this->dinero_fin = floatval($data['d_fin']); + $this->codagente = $data['codagente']; + $this->tickets = intval($data['tickets']); $this->ip = NULL; - if (isset($c['ip'])) { - $this->ip = $c['ip']; + if (isset($data['ip'])) { + $this->ip = $data['ip']; } foreach (self::$agentes as $ag) { diff --git a/model/core/cliente.php b/model/core/cliente.php index 8ce44f3..5991577 100644 --- a/model/core/cliente.php +++ b/model/core/cliente.php @@ -145,45 +145,45 @@ class cliente extends \fs_model public $codproveedor; private static $regimenes_iva; - public function __construct($c = FALSE) + public function __construct($data = FALSE) { parent::__construct('clientes'); - if ($c) { - $this->codcliente = $c['codcliente']; - $this->nombre = $c['nombre']; + if ($data) { + $this->codcliente = $data['codcliente']; + $this->nombre = $data['nombre']; - if (is_null($c['razonsocial'])) { - $this->razonsocial = $c['nombrecomercial']; + if (is_null($data['razonsocial'])) { + $this->razonsocial = $data['nombrecomercial']; } else { - $this->razonsocial = $c['razonsocial']; + $this->razonsocial = $data['razonsocial']; } - $this->tipoidfiscal = $c['tipoidfiscal']; - $this->cifnif = $c['cifnif']; - $this->telefono1 = $c['telefono1']; - $this->telefono2 = $c['telefono2']; - $this->fax = $c['fax']; - $this->email = $c['email']; - $this->web = $c['web']; - $this->codserie = $c['codserie']; - $this->coddivisa = $c['coddivisa']; - $this->codpago = $c['codpago']; - $this->codagente = $c['codagente']; - $this->codgrupo = $c['codgrupo']; - $this->debaja = $this->str2bool($c['debaja']); + $this->tipoidfiscal = $data['tipoidfiscal']; + $this->cifnif = $data['cifnif']; + $this->telefono1 = $data['telefono1']; + $this->telefono2 = $data['telefono2']; + $this->fax = $data['fax']; + $this->email = $data['email']; + $this->web = $data['web']; + $this->codserie = $data['codserie']; + $this->coddivisa = $data['coddivisa']; + $this->codpago = $data['codpago']; + $this->codagente = $data['codagente']; + $this->codgrupo = $data['codgrupo']; + $this->debaja = $this->str2bool($data['debaja']); $this->fechabaja = NULL; - if ($c['fechabaja']) { - $this->fechabaja = date('d-m-Y', strtotime($c['fechabaja'])); + if ($data['fechabaja']) { + $this->fechabaja = date('d-m-Y', strtotime($data['fechabaja'])); } - $this->fechaalta = date('d-m-Y', strtotime($c['fechaalta'])); - $this->observaciones = $this->no_html($c['observaciones']); - $this->regimeniva = $c['regimeniva']; - $this->recargo = $this->str2bool($c['recargo']); - $this->personafisica = $this->str2bool($c['personafisica']); - $this->diaspago = $c['diaspago']; - $this->codproveedor = $c['codproveedor']; + $this->fechaalta = date('d-m-Y', strtotime($data['fechaalta'])); + $this->observaciones = $this->no_html($data['observaciones']); + $this->regimeniva = $data['regimeniva']; + $this->recargo = $this->str2bool($data['recargo']); + $this->personafisica = $this->str2bool($data['personafisica']); + $this->diaspago = $data['diaspago']; + $this->codproveedor = $data['codproveedor']; } else { $this->codcliente = NULL; $this->nombre = ''; @@ -320,7 +320,7 @@ public function get($cod) */ public function get_by_cifnif($cifnif, $razon = FALSE) { - if ($cifnif == '' AND $razon) { + if ($cifnif == '' && $razon) { $razon = $this->no_html(mb_strtolower($razon, 'UTF8')); $sql = "SELECT * FROM " . $this->table_name . " WHERE cifnif = '' AND lower(razonsocial) = " . $this->var2str($razon) . ";"; } else { @@ -403,7 +403,6 @@ public function get_subcuenta($codejercicio) if (!$subcuenta) { /// intentamos crear la subcuenta y asociarla - $continuar = TRUE; $cuenta = new \cuenta(); $ccli = $cuenta->get_cuentaesp('CLIENT', $codejercicio); @@ -497,7 +496,7 @@ public function test() /// validamos los dias de pago $array_dias = array(); foreach (str_getcsv($this->diaspago) as $d) { - if (intval($d) >= 1 AND intval($d) <= 31) { + if (intval($d) >= 1 && intval($d) <= 31) { $array_dias[] = intval($d); } } @@ -508,12 +507,13 @@ public function test() if (!preg_match("/^[A-Z0-9]{1,6}$/i", $this->codcliente)) { $this->new_error_msg("Código de cliente no válido: " . $this->codcliente); - } else if (strlen($this->nombre) < 1 OR strlen($this->nombre) > 100) { + } else if (strlen($this->nombre) < 1 || strlen($this->nombre) > 100) { $this->new_error_msg("Nombre de cliente no válido: " . $this->nombre); - } else if (strlen($this->razonsocial) < 1 OR strlen($this->razonsocial) > 100) { + } else if (strlen($this->razonsocial) < 1 || strlen($this->razonsocial) > 100) { $this->new_error_msg("Razón social del cliente no válida: " . $this->razonsocial); - } else + } else { $status = TRUE; + } return $status; } diff --git a/model/core/cliente_propiedad.php b/model/core/cliente_propiedad.php index c8cb7f8..8228109 100644 --- a/model/core/cliente_propiedad.php +++ b/model/core/cliente_propiedad.php @@ -30,13 +30,13 @@ class cliente_propiedad extends \fs_model public $codcliente; public $text; - public function __construct($a = FALSE) + public function __construct($data = FALSE) { parent::__construct('cliente_propiedades'); - if ($a) { - $this->name = $a['name']; - $this->codcliente = $a['codcliente']; - $this->text = $a['text']; + if ($data) { + $this->name = $data['name']; + $this->codcliente = $data['codcliente']; + $this->text = $data['text']; } else { $this->name = NULL; $this->codcliente = NULL; @@ -51,7 +51,7 @@ protected function install() public function exists() { - if (is_null($this->name) OR is_null($this->codcliente)) { + if (is_null($this->name) || is_null($this->codcliente)) { return FALSE; } diff --git a/model/core/concepto_partida.php b/model/core/concepto_partida.php index 7729ebe..9984af7 100644 --- a/model/core/concepto_partida.php +++ b/model/core/concepto_partida.php @@ -33,12 +33,12 @@ class concepto_partida extends \fs_model public $idconceptopar; public $concepto; - public function __construct($c = FALSE) + public function __construct($data = FALSE) { parent::__construct('co_conceptospar'); - if ($c) { - $this->idconceptopar = $c['idconceptopar']; - $this->concepto = $c['concepto']; + if ($data) { + $this->idconceptopar = $data['idconceptopar']; + $this->concepto = $data['concepto']; } else { $this->idconceptopar = NULL; $this->concepto = NULL; diff --git a/model/core/cuenta.php b/model/core/cuenta.php index 3bef647..2b4614f 100644 --- a/model/core/cuenta.php +++ b/model/core/cuenta.php @@ -50,17 +50,17 @@ class cuenta extends \fs_model public $descripcion; public $idcuentaesp; - public function __construct($c = FALSE) + public function __construct($data = FALSE) { parent::__construct('co_cuentas'); - if ($c) { - $this->idcuenta = $this->intval($c['idcuenta']); - $this->codcuenta = $c['codcuenta']; - $this->codejercicio = $c['codejercicio']; - $this->idepigrafe = $this->intval($c['idepigrafe']); - $this->codepigrafe = $c['codepigrafe']; - $this->descripcion = $c['descripcion']; - $this->idcuentaesp = $c['idcuentaesp']; + if ($data) { + $this->idcuenta = $this->intval($data['idcuenta']); + $this->codcuenta = $data['codcuenta']; + $this->codejercicio = $data['codejercicio']; + $this->idepigrafe = $this->intval($data['idepigrafe']); + $this->codepigrafe = $data['codepigrafe']; + $this->descripcion = $data['descripcion']; + $this->idcuentaesp = $data['idcuentaesp']; } else { $this->idcuenta = NULL; $this->codcuenta = NULL; @@ -75,7 +75,7 @@ public function __construct($c = FALSE) protected function install() { /// forzamos la creación de la tabla epigrafes - $epi = new \epigrafe(); + new \epigrafe(); return ''; } @@ -156,7 +156,7 @@ public function test() { $this->descripcion = $this->no_html($this->descripcion); - if (strlen($this->codcuenta) > 0 AND strlen($this->descripcion) > 0) { + if (strlen($this->codcuenta) > 0 && strlen($this->descripcion) > 0) { return TRUE; } diff --git a/model/core/cuenta_banco_cliente.php b/model/core/cuenta_banco_cliente.php index b568055..8e9790f 100644 --- a/model/core/cuenta_banco_cliente.php +++ b/model/core/cuenta_banco_cliente.php @@ -53,20 +53,20 @@ class cuenta_banco_cliente extends \fs_model */ public $fmandato; - public function __construct($c = FALSE) + public function __construct($data = FALSE) { parent::__construct('cuentasbcocli'); - if ($c) { - $this->codcliente = $c['codcliente']; - $this->codcuenta = $c['codcuenta']; - $this->descripcion = $c['descripcion']; - $this->iban = $c['iban']; - $this->swift = $c['swift']; - $this->principal = $this->str2bool($c['principal']); + if ($data) { + $this->codcliente = $data['codcliente']; + $this->codcuenta = $data['codcuenta']; + $this->descripcion = $data['descripcion']; + $this->iban = $data['iban']; + $this->swift = $data['swift']; + $this->principal = $this->str2bool($data['principal']); $this->fmandato = NULL; - if ($c['fmandato']) { - $this->fmandato = date('d-m-Y', strtotime($c['fmandato'])); + if ($data['fmandato']) { + $this->fmandato = date('d-m-Y', strtotime($data['fmandato'])); } } else { $this->codcliente = NULL; diff --git a/model/core/cuenta_banco_proveedor.php b/model/core/cuenta_banco_proveedor.php index 1715e28..b10c586 100644 --- a/model/core/cuenta_banco_proveedor.php +++ b/model/core/cuenta_banco_proveedor.php @@ -42,16 +42,16 @@ class cuenta_banco_proveedor extends \fs_model public $swift; public $principal; - public function __construct($c = FALSE) + public function __construct($data = FALSE) { parent::__construct('cuentasbcopro'); - if ($c) { - $this->codcuenta = $c['codcuenta']; - $this->codproveedor = $c['codproveedor']; - $this->descripcion = $c['descripcion']; - $this->iban = $c['iban']; - $this->swift = $c['swift']; - $this->principal = $this->str2bool($c['principal']); + if ($data) { + $this->codcuenta = $data['codcuenta']; + $this->codproveedor = $data['codproveedor']; + $this->descripcion = $data['descripcion']; + $this->iban = $data['iban']; + $this->swift = $data['swift']; + $this->principal = $this->str2bool($data['principal']); } else { $this->codcuenta = NULL; $this->codproveedor = NULL; diff --git a/model/core/cuenta_especial.php b/model/core/cuenta_especial.php index 3e497f4..a15cc56 100644 --- a/model/core/cuenta_especial.php +++ b/model/core/cuenta_especial.php @@ -34,12 +34,12 @@ class cuenta_especial extends \fs_model public $idcuentaesp; public $descripcion; - public function __construct($c = FALSE) + public function __construct($data = FALSE) { parent::__construct('co_cuentasesp'); - if ($c) { - $this->idcuentaesp = $c['idcuentaesp']; - $this->descripcion = $c['descripcion']; + if ($data) { + $this->idcuentaesp = $data['idcuentaesp']; + $this->descripcion = $data['descripcion']; } else { $this->idcuentaesp = NULL; $this->descripcion = NULL; @@ -82,9 +82,9 @@ protected function install() public function get($id) { - $cuentae = $this->db->select("SELECT * FROM " . $this->table_name . " WHERE idcuentaesp = " . $this->var2str($id) . ";"); - if ($cuentae) { - return new \cuenta_especial($cuentae[0]); + $data = $this->db->select("SELECT * FROM " . $this->table_name . " WHERE idcuentaesp = " . $this->var2str($id) . ";"); + if ($data) { + return new \cuenta_especial($data[0]); } return FALSE; diff --git a/model/core/direccion_cliente.php b/model/core/direccion_cliente.php index dd5d363..05df088 100644 --- a/model/core/direccion_cliente.php +++ b/model/core/direccion_cliente.php @@ -63,22 +63,22 @@ class direccion_cliente extends \fs_model */ public $fecha; - public function __construct($d = FALSE) + public function __construct($data = FALSE) { parent::__construct('dirclientes'); - if ($d) { - $this->id = $this->intval($d['id']); - $this->codcliente = $d['codcliente']; - $this->codpais = $d['codpais']; - $this->apartado = $d['apartado']; - $this->provincia = $d['provincia']; - $this->ciudad = $d['ciudad']; - $this->codpostal = $d['codpostal']; - $this->direccion = $d['direccion']; - $this->domenvio = $this->str2bool($d['domenvio']); - $this->domfacturacion = $this->str2bool($d['domfacturacion']); - $this->descripcion = $d['descripcion']; - $this->fecha = date('d-m-Y', strtotime($d['fecha'])); + if ($data) { + $this->id = $this->intval($data['id']); + $this->codcliente = $data['codcliente']; + $this->codpais = $data['codpais']; + $this->apartado = $data['apartado']; + $this->provincia = $data['provincia']; + $this->ciudad = $data['ciudad']; + $this->codpostal = $data['codpostal']; + $this->direccion = $data['direccion']; + $this->domenvio = $this->str2bool($data['domenvio']); + $this->domfacturacion = $this->str2bool($data['domfacturacion']); + $this->descripcion = $data['descripcion']; + $this->fecha = date('d-m-Y', strtotime($data['fecha'])); } else { $this->id = NULL; $this->codcliente = NULL; diff --git a/model/core/direccion_proveedor.php b/model/core/direccion_proveedor.php index c3f568c..7c1c3dc 100644 --- a/model/core/direccion_proveedor.php +++ b/model/core/direccion_proveedor.php @@ -57,21 +57,21 @@ class direccion_proveedor extends \fs_model */ public $fecha; - public function __construct($d = FALSE) + public function __construct($data = FALSE) { parent::__construct('dirproveedores'); - if ($d) { - $this->id = $this->intval($d['id']); - $this->codproveedor = $d['codproveedor']; - $this->codpais = $d['codpais']; - $this->apartado = $d['apartado']; - $this->provincia = $d['provincia']; - $this->ciudad = $d['ciudad']; - $this->codpostal = $d['codpostal']; - $this->direccion = $d['direccion']; - $this->direccionppal = $this->str2bool($d['direccionppal']); - $this->descripcion = $d['descripcion']; - $this->fecha = date('d-m-Y', strtotime($d['fecha'])); + if ($data) { + $this->id = $this->intval($data['id']); + $this->codproveedor = $data['codproveedor']; + $this->codpais = $data['codpais']; + $this->apartado = $data['apartado']; + $this->provincia = $data['provincia']; + $this->ciudad = $data['ciudad']; + $this->codpostal = $data['codpostal']; + $this->direccion = $data['direccion']; + $this->direccionppal = $this->str2bool($data['direccionppal']); + $this->descripcion = $data['descripcion']; + $this->fecha = date('d-m-Y', strtotime($data['fecha'])); } else { $this->id = NULL; $this->codproveedor = NULL; diff --git a/model/core/epigrafe.php b/model/core/epigrafe.php index 34b0d21..4f99293 100644 --- a/model/core/epigrafe.php +++ b/model/core/epigrafe.php @@ -173,7 +173,7 @@ public function test() { $this->descripcion = $this->no_html($this->descripcion); - if (strlen($this->codepigrafe) > 0 AND strlen($this->descripcion) > 0) { + if (strlen($this->codepigrafe) > 0 && strlen($this->descripcion) > 0) { return TRUE; } diff --git a/model/core/fabricante.php b/model/core/fabricante.php index f262284..c31acea 100644 --- a/model/core/fabricante.php +++ b/model/core/fabricante.php @@ -33,12 +33,12 @@ class fabricante extends \fs_model public $codfabricante; public $nombre; - public function __construct($f = FALSE) + public function __construct($data = FALSE) { parent::__construct('fabricantes'); - if ($f) { - $this->codfabricante = $f['codfabricante']; - $this->nombre = $f['nombre']; + if ($data) { + $this->codfabricante = $data['codfabricante']; + $this->nombre = $data['nombre']; } else { $this->codfabricante = NULL; $this->nombre = ''; @@ -101,9 +101,9 @@ public function test() $this->codfabricante = $this->no_html($this->codfabricante); $this->nombre = $this->no_html($this->nombre); - if (strlen($this->codfabricante) < 1 OR strlen($this->codfabricante) > 8) { + if (strlen($this->codfabricante) < 1 || strlen($this->codfabricante) > 8) { $this->new_error_msg("Código de fabricante no válido. Deben ser entre 1 y 8 caracteres."); - } else if (strlen($this->nombre) < 1 OR strlen($this->nombre) > 100) { + } else if (strlen($this->nombre) < 1 || strlen($this->nombre) > 100) { $this->new_error_msg("Descripción de fabricante no válida."); } else { $status = TRUE; diff --git a/model/core/factura_cliente.php b/model/core/factura_cliente.php index a7bb5b0..8adbd2a 100644 --- a/model/core/factura_cliente.php +++ b/model/core/factura_cliente.php @@ -233,77 +233,77 @@ class factura_cliente extends \fs_model */ public $numdocs; - public function __construct($f = FALSE) + public function __construct($data = FALSE) { parent::__construct('facturascli'); - if ($f) { - $this->idfactura = $this->intval($f['idfactura']); - $this->idasiento = $this->intval($f['idasiento']); - $this->idasientop = $this->intval($f['idasientop']); - $this->idfacturarect = $this->intval($f['idfacturarect']); - $this->codigo = $f['codigo']; - $this->numero = $f['numero']; - $this->numero2 = $f['numero2']; - $this->codigorect = $f['codigorect']; - $this->codejercicio = $f['codejercicio']; - $this->codserie = $f['codserie']; - $this->codalmacen = $f['codalmacen']; - $this->codpago = $f['codpago']; - $this->coddivisa = $f['coddivisa']; - $this->fecha = Date('d-m-Y', strtotime($f['fecha'])); + if ($data) { + $this->idfactura = $this->intval($data['idfactura']); + $this->idasiento = $this->intval($data['idasiento']); + $this->idasientop = $this->intval($data['idasientop']); + $this->idfacturarect = $this->intval($data['idfacturarect']); + $this->codigo = $data['codigo']; + $this->numero = $data['numero']; + $this->numero2 = $data['numero2']; + $this->codigorect = $data['codigorect']; + $this->codejercicio = $data['codejercicio']; + $this->codserie = $data['codserie']; + $this->codalmacen = $data['codalmacen']; + $this->codpago = $data['codpago']; + $this->coddivisa = $data['coddivisa']; + $this->fecha = Date('d-m-Y', strtotime($data['fecha'])); $this->hora = '00:00:00'; - if (!is_null($f['hora'])) { - $this->hora = date('H:i:s', strtotime($f['hora'])); + if (!is_null($data['hora'])) { + $this->hora = date('H:i:s', strtotime($data['hora'])); } - $this->codcliente = $f['codcliente']; - $this->nombrecliente = $f['nombrecliente']; - $this->cifnif = $f['cifnif']; - $this->direccion = $f['direccion']; - $this->ciudad = $f['ciudad']; - $this->provincia = $f['provincia']; - $this->apartado = $f['apartado']; - $this->coddir = $f['coddir']; - $this->codpostal = $f['codpostal']; - $this->codpais = $f['codpais']; - $this->codagente = $f['codagente']; - $this->neto = floatval($f['neto']); - $this->totaliva = floatval($f['totaliva']); - $this->total = floatval($f['total']); - $this->totaleuros = floatval($f['totaleuros']); - $this->irpf = floatval($f['irpf']); - $this->totalirpf = floatval($f['totalirpf']); - $this->porcomision = floatval($f['porcomision']); - $this->tasaconv = floatval($f['tasaconv']); - $this->totalrecargo = floatval($f['totalrecargo']); - $this->observaciones = $this->no_html($f['observaciones']); - $this->pagada = $this->str2bool($f['pagada']); - $this->anulada = $this->str2bool($f['anulada']); - - $this->vencimiento = Date('d-m-Y', strtotime($f['fecha'] . ' +1 day')); - if (!is_null($f['vencimiento'])) { - $this->vencimiento = Date('d-m-Y', strtotime($f['vencimiento'])); + $this->codcliente = $data['codcliente']; + $this->nombrecliente = $data['nombrecliente']; + $this->cifnif = $data['cifnif']; + $this->direccion = $data['direccion']; + $this->ciudad = $data['ciudad']; + $this->provincia = $data['provincia']; + $this->apartado = $data['apartado']; + $this->coddir = $data['coddir']; + $this->codpostal = $data['codpostal']; + $this->codpais = $data['codpais']; + $this->codagente = $data['codagente']; + $this->neto = floatval($data['neto']); + $this->totaliva = floatval($data['totaliva']); + $this->total = floatval($data['total']); + $this->totaleuros = floatval($data['totaleuros']); + $this->irpf = floatval($data['irpf']); + $this->totalirpf = floatval($data['totalirpf']); + $this->porcomision = floatval($data['porcomision']); + $this->tasaconv = floatval($data['tasaconv']); + $this->totalrecargo = floatval($data['totalrecargo']); + $this->observaciones = $this->no_html($data['observaciones']); + $this->pagada = $this->str2bool($data['pagada']); + $this->anulada = $this->str2bool($data['anulada']); + + $this->vencimiento = Date('d-m-Y', strtotime($data['fecha'] . ' +1 day')); + if (!is_null($data['vencimiento'])) { + $this->vencimiento = Date('d-m-Y', strtotime($data['vencimiento'])); } $this->femail = NULL; - if (!is_null($f['femail'])) { - $this->femail = Date('d-m-Y', strtotime($f['femail'])); + if (!is_null($data['femail'])) { + $this->femail = Date('d-m-Y', strtotime($data['femail'])); } - $this->envio_codtrans = $f['codtrans']; - $this->envio_codigo = $f['codigoenv']; - $this->envio_nombre = $f['nombreenv']; - $this->envio_apellidos = $f['apellidosenv']; - $this->envio_apartado = $f['apartadoenv']; - $this->envio_direccion = $f['direccionenv']; - $this->envio_codpostal = $f['codpostalenv']; - $this->envio_ciudad = $f['ciudadenv']; - $this->envio_provincia = $f['provinciaenv']; - $this->envio_codpais = $f['codpaisenv']; - - $this->idimprenta = $this->intval($f['idimprenta']); - $this->numdocs = intval($f['numdocs']); + $this->envio_codtrans = $data['codtrans']; + $this->envio_codigo = $data['codigoenv']; + $this->envio_nombre = $data['nombreenv']; + $this->envio_apellidos = $data['apellidosenv']; + $this->envio_apartado = $data['apartadoenv']; + $this->envio_direccion = $data['direccionenv']; + $this->envio_codpostal = $data['codpostalenv']; + $this->envio_ciudad = $data['ciudadenv']; + $this->envio_provincia = $data['provinciaenv']; + $this->envio_codpais = $data['codpaisenv']; + + $this->idimprenta = $this->intval($data['idimprenta']); + $this->numdocs = intval($data['numdocs']); } else { $this->idfactura = NULL; $this->idasiento = NULL; @@ -428,7 +428,7 @@ public function set_fecha_hora($fecha, $hora) $data = $this->db->select($sql); if ($data) { - if (strtotime($data[0]['hora']) > strtotime($hora) OR $cambio) { + if (strtotime($data[0]['hora']) > strtotime($hora) || $cambio) { $hora = date('H:i:s', strtotime($data[0]['hora'])); $cambio = TRUE; } @@ -554,7 +554,7 @@ public function get_lineas_iva() $i = 0; $encontrada = FALSE; while ($i < count($lineasi)) { - if ($l->iva == $lineasi[$i]->iva AND $l->recargo == $lineasi[$i]->recargo) { + if ($l->iva == $lineasi[$i]->iva && $l->recargo == $lineasi[$i]->recargo) { $encontrada = TRUE; $lineasi[$i]->neto += $l->pvptotal; $lineasi[$i]->totaliva += ($l->pvptotal * $l->iva) / 100.0; @@ -839,7 +839,7 @@ public function full_test($duplicados = TRUE) $ejercicio = new \ejercicio(); $eje0 = $ejercicio->get($this->codejercicio); if ($eje0) { - if (strtotime($this->fecha) < strtotime($eje0->fechainicio) OR strtotime($this->fecha) > strtotime($eje0->fechafin)) { + if (strtotime($this->fecha) < strtotime($eje0->fechainicio) || strtotime($this->fecha) > strtotime($eje0->fechafin)) { $status = FALSE; $this->new_error_msg("La fecha de esta factura está fuera del rango del" . " ejercicio."); @@ -916,10 +916,10 @@ public function full_test($duplicados = TRUE) if (isset($this->idasiento)) { $asiento = $this->get_asiento(); if ($asiento) { - if ($asiento->tipodocumento != 'Factura de cliente' OR $asiento->documento != $this->codigo) { + if ($asiento->tipodocumento != 'Factura de cliente' || $asiento->documento != $this->codigo) { $this->new_error_msg("Esta factura apunta a un asiento incorrecto."); $status = FALSE; - } else if ($this->coddivisa == $this->default_items->coddivisa() AND ( abs($asiento->importe) - abs($this->total + $this->totalirpf) >= .02)) { + } else if ($this->coddivisa == $this->default_items->coddivisa() && ( abs($asiento->importe) - abs($this->total + $this->totalirpf) >= .02)) { $this->new_error_msg("El importe del asiento es distinto al de la factura."); $status = FALSE; } else { @@ -939,7 +939,7 @@ public function full_test($duplicados = TRUE) } } - if ($status AND $duplicados) { + if ($status && $duplicados) { /// comprobamos si es un duplicado $data = $this->db->select("SELECT * FROM " . $this->table_name . " WHERE fecha = " . $this->var2str($this->fecha) . " AND codcliente = " . $this->var2str($this->codcliente) diff --git a/model/core/factura_proveedor.php b/model/core/factura_proveedor.php index 1bae3a5..71a8cef 100644 --- a/model/core/factura_proveedor.php +++ b/model/core/factura_proveedor.php @@ -187,46 +187,46 @@ class factura_proveedor extends \fs_model */ public $numdocs; - public function __construct($f = FALSE) + public function __construct($data = FALSE) { parent::__construct('facturasprov'); - if ($f) { - $this->anulada = $this->str2bool($f['anulada']); - $this->cifnif = $f['cifnif']; - $this->codagente = $f['codagente']; - $this->codalmacen = $f['codalmacen']; - $this->coddivisa = $f['coddivisa']; - $this->codejercicio = $f['codejercicio']; - $this->codigo = $f['codigo']; - $this->codigorect = $f['codigorect']; - $this->codpago = $f['codpago']; - $this->codproveedor = $f['codproveedor']; - $this->codserie = $f['codserie']; - $this->fecha = Date('d-m-Y', strtotime($f['fecha'])); + if ($data) { + $this->anulada = $this->str2bool($data['anulada']); + $this->cifnif = $data['cifnif']; + $this->codagente = $data['codagente']; + $this->codalmacen = $data['codalmacen']; + $this->coddivisa = $data['coddivisa']; + $this->codejercicio = $data['codejercicio']; + $this->codigo = $data['codigo']; + $this->codigorect = $data['codigorect']; + $this->codpago = $data['codpago']; + $this->codproveedor = $data['codproveedor']; + $this->codserie = $data['codserie']; + $this->fecha = Date('d-m-Y', strtotime($data['fecha'])); $this->hora = '00:00:00'; - if (!is_null($f['hora'])) { - $this->hora = date('H:i:s', strtotime($f['hora'])); + if (!is_null($data['hora'])) { + $this->hora = date('H:i:s', strtotime($data['hora'])); } - $this->idasiento = $this->intval($f['idasiento']); - $this->idasientop = $this->intval($f['idasientop']); - $this->idfactura = $this->intval($f['idfactura']); - $this->idfacturarect = $this->intval($f['idfacturarect']); - $this->irpf = floatval($f['irpf']); - $this->neto = floatval($f['neto']); - $this->nombre = $f['nombre']; - $this->numero = $f['numero']; - $this->numproveedor = $f['numproveedor']; - $this->observaciones = $this->no_html($f['observaciones']); - $this->pagada = $this->str2bool($f['pagada']); - $this->tasaconv = floatval($f['tasaconv']); - $this->total = floatval($f['total']); - $this->totaleuros = floatval($f['totaleuros']); - $this->totalirpf = floatval($f['totalirpf']); - $this->totaliva = floatval($f['totaliva']); - $this->totalrecargo = floatval($f['totalrecargo']); - $this->numdocs = intval($f['numdocs']); + $this->idasiento = $this->intval($data['idasiento']); + $this->idasientop = $this->intval($data['idasientop']); + $this->idfactura = $this->intval($data['idfactura']); + $this->idfacturarect = $this->intval($data['idfacturarect']); + $this->irpf = floatval($data['irpf']); + $this->neto = floatval($data['neto']); + $this->nombre = $data['nombre']; + $this->numero = $data['numero']; + $this->numproveedor = $data['numproveedor']; + $this->observaciones = $this->no_html($data['observaciones']); + $this->pagada = $this->str2bool($data['pagada']); + $this->tasaconv = floatval($data['tasaconv']); + $this->total = floatval($data['total']); + $this->totaleuros = floatval($data['totaleuros']); + $this->totalirpf = floatval($data['totalirpf']); + $this->totaliva = floatval($data['totaliva']); + $this->totalrecargo = floatval($data['totalrecargo']); + $this->numdocs = intval($data['numdocs']); } else { $this->anulada = FALSE; $this->cifnif = ''; @@ -415,7 +415,7 @@ public function get_lineas_iva() $i = 0; $encontrada = FALSE; while ($i < count($lineasi)) { - if ($l->iva == $lineasi[$i]->iva AND $l->recargo == $lineasi[$i]->recargo) { + if ($l->iva == $lineasi[$i]->iva && $l->recargo == $lineasi[$i]->recargo) { $encontrada = TRUE; $lineasi[$i]->neto += $l->pvptotal; $lineasi[$i]->totaliva += ($l->pvptotal * $l->iva) / 100; @@ -674,7 +674,7 @@ public function full_test($duplicados = TRUE) $ejercicio = new \ejercicio(); $eje0 = $ejercicio->get($this->codejercicio); if ($eje0) { - if (strtotime($this->fecha) < strtotime($eje0->fechainicio) OR strtotime($this->fecha) > strtotime($eje0->fechafin)) { + if (strtotime($this->fecha) < strtotime($eje0->fechainicio) || strtotime($this->fecha) > strtotime($eje0->fechafin)) { $status = FALSE; $this->new_error_msg("La fecha de esta factura está fuera del rango del" . " ejercicio."); @@ -731,10 +731,10 @@ public function full_test($duplicados = TRUE) if (isset($this->idasiento)) { $asiento = $this->get_asiento(); if ($asiento) { - if ($asiento->tipodocumento != 'Factura de proveedor' OR $asiento->documento != $this->codigo) { + if ($asiento->tipodocumento != 'Factura de proveedor' || $asiento->documento != $this->codigo) { $this->new_error_msg("Esta factura apunta a un asiento incorrecto."); $status = FALSE; - } else if ($this->coddivisa == $this->default_items->coddivisa() AND ( abs($asiento->importe) - abs($this->total + $this->totalirpf) >= .02)) { + } else if ($this->coddivisa == $this->default_items->coddivisa() && ( abs($asiento->importe) - abs($this->total + $this->totalirpf) >= .02)) { $this->new_error_msg("El importe del asiento es distinto al de la factura."); $status = FALSE; } else { @@ -754,7 +754,7 @@ public function full_test($duplicados = TRUE) } } - if ($status AND $duplicados) { + if ($status && $duplicados) { /// comprobamos si es un duplicado $facturas = $this->db->select("SELECT * FROM " . $this->table_name . " WHERE fecha = " . $this->var2str($this->fecha) . " AND codproveedor = " . $this->var2str($this->codproveedor) diff --git a/model/core/familia.php b/model/core/familia.php index e32ee27..bc827c7 100644 --- a/model/core/familia.php +++ b/model/core/familia.php @@ -40,21 +40,21 @@ class familia extends \fs_model public $madre; public $nivel; - public function __construct($f = FALSE) + public function __construct($data = FALSE) { parent::__construct('familias'); - if ($f) { - $this->codfamilia = $f['codfamilia']; - $this->descripcion = $f['descripcion']; + if ($data) { + $this->codfamilia = $data['codfamilia']; + $this->descripcion = $data['descripcion']; $this->madre = NULL; - if (isset($f['madre'])) { - $this->madre = $f['madre']; + if (isset($data['madre'])) { + $this->madre = $data['madre']; } $this->nivel = ''; - if (isset($f['nivel'])) { - $this->nivel = $f['nivel']; + if (isset($data['nivel'])) { + $this->nivel = $data['nivel']; } } else { $this->codfamilia = NULL; @@ -133,9 +133,9 @@ public function test() $this->codfamilia = $this->no_html($this->codfamilia); $this->descripcion = $this->no_html($this->descripcion); - if (strlen($this->codfamilia) < 1 OR strlen($this->codfamilia) > 8) { + if (strlen($this->codfamilia) < 1 || strlen($this->codfamilia) > 8) { $this->new_error_msg("Código de familia no válido. Deben ser entre 1 y 8 caracteres."); - } else if (strlen($this->descripcion) < 1 OR strlen($this->descripcion) > 100) { + } else if (strlen($this->descripcion) < 1 || strlen($this->descripcion) > 100) { $this->new_error_msg("Descripción de familia no válida."); } else { $status = TRUE; @@ -222,7 +222,7 @@ public function all() /** * Completa los datos de la lista de familias con el nivel - * @param string $familias + * @param array $familias * @param string $madre * @param string $nivel * @return \familia[] diff --git a/model/core/grupo_clientes.php b/model/core/grupo_clientes.php index d3eaa45..c8c206f 100644 --- a/model/core/grupo_clientes.php +++ b/model/core/grupo_clientes.php @@ -44,13 +44,13 @@ class grupo_clientes extends \fs_model */ public $codtarifa; - public function __construct($g = FALSE) + public function __construct($data = FALSE) { parent::__construct('gruposclientes'); - if ($g) { - $this->codgrupo = $g['codgrupo']; - $this->nombre = $g['nombre']; - $this->codtarifa = $g['codtarifa']; + if ($data) { + $this->codgrupo = $data['codgrupo']; + $this->nombre = $data['nombre']; + $this->codtarifa = $data['codtarifa']; } else { $this->codgrupo = NULL; $this->nombre = NULL; diff --git a/model/core/grupo_epigrafes.php b/model/core/grupo_epigrafes.php index 0f5b604..60867e6 100644 --- a/model/core/grupo_epigrafes.php +++ b/model/core/grupo_epigrafes.php @@ -107,7 +107,7 @@ public function test() { $this->descripcion = $this->no_html($this->descripcion); - if (strlen($this->codejercicio) > 0 AND strlen($this->codgrupo) > 0 AND strlen($this->descripcion) > 0) { + if (strlen($this->codejercicio) > 0 && strlen($this->codgrupo) > 0 && strlen($this->descripcion) > 0) { return TRUE; } diff --git a/model/core/impuesto.php b/model/core/impuesto.php index 7fba34c..9b9655a 100644 --- a/model/core/impuesto.php +++ b/model/core/impuesto.php @@ -48,16 +48,16 @@ class impuesto extends \fs_model public $iva; public $recargo; - public function __construct($i = FALSE) + public function __construct($data = FALSE) { parent::__construct('impuestos'); - if ($i) { - $this->codimpuesto = $i['codimpuesto']; - $this->codsubcuentarep = $i['codsubcuentarep']; - $this->codsubcuentasop = $i['codsubcuentasop']; - $this->descripcion = $i['descripcion']; - $this->iva = floatval($i['iva']); - $this->recargo = floatval($i['recargo']); + if ($data) { + $this->codimpuesto = $data['codimpuesto']; + $this->codsubcuentarep = $data['codsubcuentarep']; + $this->codsubcuentasop = $data['codsubcuentasop']; + $this->descripcion = $data['descripcion']; + $this->iva = floatval($data['iva']); + $this->recargo = floatval($data['recargo']); } else { $this->codimpuesto = NULL; $this->codsubcuentarep = NULL; @@ -136,9 +136,9 @@ public function test() $this->codimpuesto = trim($this->codimpuesto); $this->descripcion = $this->no_html($this->descripcion); - if (strlen($this->codimpuesto) < 1 OR strlen($this->codimpuesto) > 10) { + if (strlen($this->codimpuesto) < 1 || strlen($this->codimpuesto) > 10) { $this->new_error_msg("Código del impuesto no válido. Debe tener entre 1 y 10 caracteres."); - } else if (strlen($this->descripcion) < 1 OR strlen($this->descripcion) > 50) { + } else if (strlen($this->descripcion) < 1 || strlen($this->descripcion) > 50) { $this->new_error_msg("Descripción del impuesto no válida."); } else { $status = TRUE; diff --git a/model/core/linea_albaran_cliente.php b/model/core/linea_albaran_cliente.php index 8d4b6ba..b3c3a13 100644 --- a/model/core/linea_albaran_cliente.php +++ b/model/core/linea_albaran_cliente.php @@ -133,7 +133,7 @@ class linea_albaran_cliente extends \fs_model private $fecha; private static $albaranes; - public function __construct($l = FALSE) + public function __construct($data = FALSE) { parent::__construct('lineasalbaranescli'); @@ -141,26 +141,26 @@ public function __construct($l = FALSE) self::$albaranes = array(); } - if ($l) { - $this->idlinea = $this->intval($l['idlinea']); - $this->idlineapedido = $this->intval($l['idlineapedido']); - $this->idalbaran = $this->intval($l['idalbaran']); - $this->idpedido = $this->intval($l['idpedido']); - $this->referencia = $l['referencia']; - $this->codcombinacion = $l['codcombinacion']; - $this->descripcion = $l['descripcion']; - $this->cantidad = floatval($l['cantidad']); - $this->dtopor = floatval($l['dtopor']); - $this->codimpuesto = $l['codimpuesto']; - $this->iva = floatval($l['iva']); - $this->pvptotal = floatval($l['pvptotal']); - $this->pvpsindto = floatval($l['pvpsindto']); - $this->pvpunitario = floatval($l['pvpunitario']); - $this->irpf = floatval($l['irpf']); - $this->recargo = floatval($l['recargo']); - $this->orden = intval($l['orden']); - $this->mostrar_cantidad = $this->str2bool($l['mostrar_cantidad']); - $this->mostrar_precio = $this->str2bool($l['mostrar_precio']); + if ($data) { + $this->idlinea = $this->intval($data['idlinea']); + $this->idlineapedido = $this->intval($data['idlineapedido']); + $this->idalbaran = $this->intval($data['idalbaran']); + $this->idpedido = $this->intval($data['idpedido']); + $this->referencia = $data['referencia']; + $this->codcombinacion = $data['codcombinacion']; + $this->descripcion = $data['descripcion']; + $this->cantidad = floatval($data['cantidad']); + $this->dtopor = floatval($data['dtopor']); + $this->codimpuesto = $data['codimpuesto']; + $this->iva = floatval($data['iva']); + $this->pvptotal = floatval($data['pvptotal']); + $this->pvpsindto = floatval($data['pvpsindto']); + $this->pvpunitario = floatval($data['pvpunitario']); + $this->irpf = floatval($data['irpf']); + $this->recargo = floatval($data['recargo']); + $this->orden = intval($data['orden']); + $this->mostrar_cantidad = $this->str2bool($data['mostrar_cantidad']); + $this->mostrar_precio = $this->str2bool($data['mostrar_precio']); } else { $this->idlinea = NULL; $this->idlineapedido = NULL; @@ -278,7 +278,7 @@ public function url() public function articulo_url() { - if (is_null($this->referencia) OR $this->referencia == '') { + if (is_null($this->referencia) || $this->referencia == '') { return "index.php?page=ventas_articulos"; } diff --git a/model/core/linea_albaran_proveedor.php b/model/core/linea_albaran_proveedor.php index 2213d53..4a86efc 100644 --- a/model/core/linea_albaran_proveedor.php +++ b/model/core/linea_albaran_proveedor.php @@ -115,7 +115,7 @@ class linea_albaran_proveedor extends \fs_model private $fecha; private static $albaranes; - public function __construct($l = FALSE) + public function __construct($data = FALSE) { parent::__construct('lineasalbaranesprov'); @@ -123,23 +123,23 @@ public function __construct($l = FALSE) self::$albaranes = array(); } - if ($l) { - $this->idlinea = $this->intval($l['idlinea']); - $this->idlineapedido = $this->intval($l['idlineapedido']); - $this->idalbaran = $this->intval($l['idalbaran']); - $this->idpedido = $this->intval($l['idpedido']); - $this->referencia = $l['referencia']; - $this->codcombinacion = $l['codcombinacion']; - $this->descripcion = $l['descripcion']; - $this->cantidad = floatval($l['cantidad']); - $this->dtopor = floatval($l['dtopor']); - $this->codimpuesto = $l['codimpuesto']; - $this->iva = floatval($l['iva']); - $this->pvptotal = floatval($l['pvptotal']); - $this->pvpsindto = floatval($l['pvpsindto']); - $this->pvpunitario = floatval($l['pvpunitario']); - $this->irpf = floatval($l['irpf']); - $this->recargo = floatval($l['recargo']); + if ($data) { + $this->idlinea = $this->intval($data['idlinea']); + $this->idlineapedido = $this->intval($data['idlineapedido']); + $this->idalbaran = $this->intval($data['idalbaran']); + $this->idpedido = $this->intval($data['idpedido']); + $this->referencia = $data['referencia']; + $this->codcombinacion = $data['codcombinacion']; + $this->descripcion = $data['descripcion']; + $this->cantidad = floatval($data['cantidad']); + $this->dtopor = floatval($data['dtopor']); + $this->codimpuesto = $data['codimpuesto']; + $this->iva = floatval($data['iva']); + $this->pvptotal = floatval($data['pvptotal']); + $this->pvpsindto = floatval($data['pvpsindto']); + $this->pvpunitario = floatval($data['pvpunitario']); + $this->irpf = floatval($data['irpf']); + $this->recargo = floatval($data['recargo']); } else { $this->idlinea = NULL; $this->idlineapedido = NULL; @@ -253,7 +253,7 @@ public function url() public function articulo_url() { - if (is_null($this->referencia) OR $this->referencia == '') { + if (is_null($this->referencia) || $this->referencia == '') { return "index.php?page=ventas_articulos"; } diff --git a/model/core/linea_factura_cliente.php b/model/core/linea_factura_cliente.php index 59a0e10..b0b5245 100644 --- a/model/core/linea_factura_cliente.php +++ b/model/core/linea_factura_cliente.php @@ -137,7 +137,7 @@ class linea_factura_cliente extends \fs_model private static $facturas; private static $albaranes; - public function __construct($l = FALSE) + public function __construct($data = FALSE) { parent::__construct('lineasfacturascli'); @@ -146,26 +146,26 @@ public function __construct($l = FALSE) self::$albaranes = array(); } - if ($l) { - $this->idlinea = $this->intval($l['idlinea']); - $this->idlineaalbaran = $this->intval($l['idlineaalbaran']); - $this->idfactura = $this->intval($l['idfactura']); - $this->idalbaran = $this->intval($l['idalbaran']); - $this->referencia = $l['referencia']; - $this->codcombinacion = $l['codcombinacion']; - $this->descripcion = $l['descripcion']; - $this->cantidad = floatval($l['cantidad']); - $this->pvpunitario = floatval($l['pvpunitario']); - $this->pvpsindto = floatval($l['pvpsindto']); - $this->dtopor = floatval($l['dtopor']); - $this->pvptotal = floatval($l['pvptotal']); - $this->codimpuesto = $l['codimpuesto']; - $this->iva = floatval($l['iva']); - $this->recargo = floatval($l['recargo']); - $this->irpf = floatval($l['irpf']); - $this->orden = intval($l['orden']); - $this->mostrar_cantidad = $this->str2bool($l['mostrar_cantidad']); - $this->mostrar_precio = $this->str2bool($l['mostrar_precio']); + if ($data) { + $this->idlinea = $this->intval($data['idlinea']); + $this->idlineaalbaran = $this->intval($data['idlineaalbaran']); + $this->idfactura = $this->intval($data['idfactura']); + $this->idalbaran = $this->intval($data['idalbaran']); + $this->referencia = $data['referencia']; + $this->codcombinacion = $data['codcombinacion']; + $this->descripcion = $data['descripcion']; + $this->cantidad = floatval($data['cantidad']); + $this->pvpunitario = floatval($data['pvpunitario']); + $this->pvpsindto = floatval($data['pvpsindto']); + $this->dtopor = floatval($data['dtopor']); + $this->pvptotal = floatval($data['pvptotal']); + $this->codimpuesto = $data['codimpuesto']; + $this->iva = floatval($data['iva']); + $this->recargo = floatval($data['recargo']); + $this->irpf = floatval($data['irpf']); + $this->orden = intval($data['orden']); + $this->mostrar_cantidad = $this->str2bool($data['mostrar_cantidad']); + $this->mostrar_precio = $this->str2bool($data['mostrar_precio']); } else { $this->idlinea = NULL; $this->idlineaalbaran = NULL; @@ -226,7 +226,7 @@ private function fill() foreach (self::$albaranes as $a) { if ($a->idalbaran == $this->idalbaran) { $this->albaran_codigo = $a->codigo; - if (is_null($a->numero2) OR $a->numero2 == '') { + if (is_null($a->numero2) || $a->numero2 == '') { $this->albaran_numero = $a->numero; } else { $this->albaran_numero = $a->numero2; @@ -241,7 +241,7 @@ private function fill() $alb = $alb->get($this->idalbaran); if ($alb) { $this->albaran_codigo = $alb->codigo; - if (is_null($alb->numero2) OR $alb->numero2 == '') { + if (is_null($alb->numero2) || $alb->numero2 == '') { $this->albaran_numero = $alb->numero; } else { $this->albaran_numero = $alb->numero2; @@ -338,7 +338,7 @@ public function albaran_fecha() public function articulo_url() { - if (is_null($this->referencia) OR $this->referencia == '') { + if (is_null($this->referencia) || $this->referencia == '') { return "index.php?page=ventas_articulos"; } diff --git a/model/core/linea_factura_proveedor.php b/model/core/linea_factura_proveedor.php index a36d938..056082f 100644 --- a/model/core/linea_factura_proveedor.php +++ b/model/core/linea_factura_proveedor.php @@ -118,7 +118,7 @@ class linea_factura_proveedor extends \fs_model private static $facturas; private static $albaranes; - public function __construct($l = FALSE) + public function __construct($data = FALSE) { parent::__construct('lineasfacturasprov'); @@ -127,23 +127,23 @@ public function __construct($l = FALSE) self::$albaranes = array(); } - if ($l) { - $this->idlinea = $this->intval($l['idlinea']); - $this->idlineaalbaran = $this->intval($l['idlineaalbaran']); - $this->idfactura = $this->intval($l['idfactura']); - $this->idalbaran = $this->intval($l['idalbaran']); - $this->referencia = $l['referencia']; - $this->codcombinacion = $l['codcombinacion']; - $this->descripcion = $l['descripcion']; - $this->cantidad = floatval($l['cantidad']); - $this->pvpunitario = floatval($l['pvpunitario']); - $this->pvpsindto = floatval($l['pvpsindto']); - $this->dtopor = floatval($l['dtopor']); - $this->pvptotal = floatval($l['pvptotal']); - $this->codimpuesto = $l['codimpuesto']; - $this->iva = floatval($l['iva']); - $this->recargo = floatval($l['recargo']); - $this->irpf = floatval($l['irpf']); + if ($data) { + $this->idlinea = $this->intval($data['idlinea']); + $this->idlineaalbaran = $this->intval($data['idlineaalbaran']); + $this->idfactura = $this->intval($data['idfactura']); + $this->idalbaran = $this->intval($data['idalbaran']); + $this->referencia = $data['referencia']; + $this->codcombinacion = $data['codcombinacion']; + $this->descripcion = $data['descripcion']; + $this->cantidad = floatval($data['cantidad']); + $this->pvpunitario = floatval($data['pvpunitario']); + $this->pvpsindto = floatval($data['pvpsindto']); + $this->dtopor = floatval($data['dtopor']); + $this->pvptotal = floatval($data['pvptotal']); + $this->codimpuesto = $data['codimpuesto']; + $this->iva = floatval($data['iva']); + $this->recargo = floatval($data['recargo']); + $this->irpf = floatval($data['irpf']); } else { $this->idlinea = NULL; $this->idlineaalbaran = NULL; @@ -198,7 +198,7 @@ private function fill() foreach (self::$albaranes as $a) { if ($a->idalbaran == $this->idalbaran) { $this->albaran_codigo = $a->codigo; - if (is_null($a->numproveedor) OR $a->numproveedor == '') { + if (is_null($a->numproveedor) || $a->numproveedor == '') { $this->albaran_numero = $a->numero; } else { $this->albaran_numero = $a->numproveedor; @@ -212,7 +212,7 @@ private function fill() $alb = $alb->get($this->idalbaran); if ($alb) { $this->albaran_codigo = $alb->codigo; - if (is_null($alb->numproveedor) OR $alb->numproveedor == '') { + if (is_null($alb->numproveedor) || $alb->numproveedor == '') { $this->albaran_numero = $alb->numero; } else { $this->albaran_numero = $alb->numproveedor; @@ -299,7 +299,7 @@ public function albaran_numero() public function articulo_url() { - if (is_null($this->referencia) OR $this->referencia == '') { + if (is_null($this->referencia) || $this->referencia == '') { return "index.php?page=ventas_articulos"; } diff --git a/model/core/linea_iva_factura_cliente.php b/model/core/linea_iva_factura_cliente.php index 592bc6f..844ba7d 100644 --- a/model/core/linea_iva_factura_cliente.php +++ b/model/core/linea_iva_factura_cliente.php @@ -81,19 +81,19 @@ class linea_iva_factura_cliente extends \fs_model */ public $neto; - public function __construct($l = FALSE) + public function __construct($data = FALSE) { parent::__construct('lineasivafactcli'); - if ($l) { - $this->idlinea = $this->intval($l['idlinea']); - $this->idfactura = $this->intval($l['idfactura']); - $this->neto = floatval($l['neto']); - $this->codimpuesto = $l['codimpuesto']; - $this->iva = floatval($l['iva']); - $this->totaliva = floatval($l['totaliva']); - $this->recargo = floatval($l['recargo']); - $this->totalrecargo = floatval($l['totalrecargo']); - $this->totallinea = floatval($l['totallinea']); + if ($data) { + $this->idlinea = $this->intval($data['idlinea']); + $this->idfactura = $this->intval($data['idfactura']); + $this->neto = floatval($data['neto']); + $this->codimpuesto = $data['codimpuesto']; + $this->iva = floatval($data['iva']); + $this->totaliva = floatval($data['totaliva']); + $this->recargo = floatval($data['recargo']); + $this->totalrecargo = floatval($data['totalrecargo']); + $this->totallinea = floatval($data['totallinea']); } else { $this->idlinea = NULL; $this->idfactura = NULL; diff --git a/model/core/linea_iva_factura_proveedor.php b/model/core/linea_iva_factura_proveedor.php index b393707..3c67269 100644 --- a/model/core/linea_iva_factura_proveedor.php +++ b/model/core/linea_iva_factura_proveedor.php @@ -81,19 +81,19 @@ class linea_iva_factura_proveedor extends \fs_model */ public $neto; - public function __construct($l = FALSE) + public function __construct($data = FALSE) { parent::__construct('lineasivafactprov'); - if ($l) { - $this->idlinea = $this->intval($l['idlinea']); - $this->idfactura = $this->intval($l['idfactura']); - $this->neto = floatval($l['neto']); - $this->codimpuesto = $l['codimpuesto']; - $this->iva = floatval($l['iva']); - $this->totaliva = floatval($l['totaliva']); - $this->recargo = floatval($l['recargo']); - $this->totalrecargo = floatval($l['totalrecargo']); - $this->totallinea = floatval($l['totallinea']); + if ($data) { + $this->idlinea = $this->intval($data['idlinea']); + $this->idfactura = $this->intval($data['idfactura']); + $this->neto = floatval($data['neto']); + $this->codimpuesto = $data['codimpuesto']; + $this->iva = floatval($data['iva']); + $this->totaliva = floatval($data['totaliva']); + $this->recargo = floatval($data['recargo']); + $this->totalrecargo = floatval($data['totalrecargo']); + $this->totallinea = floatval($data['totallinea']); } else { $this->idlinea = NULL; $this->idfactura = NULL; diff --git a/model/core/linea_transferencia_stock.php b/model/core/linea_transferencia_stock.php index ccb4d67..03bfff2 100644 --- a/model/core/linea_transferencia_stock.php +++ b/model/core/linea_transferencia_stock.php @@ -22,24 +22,24 @@ class linea_transferencia_stock extends \fs_model private $fecha; private $hora; - public function __construct($d = FALSE) + public function __construct($data = FALSE) { parent::__construct('lineastransstock'); - if ($d) { - $this->idlinea = $this->intval($d['idlinea']); - $this->idtrans = $this->intval($d['idtrans']); - $this->referencia = $d['referencia']; - $this->cantidad = floatval($d['cantidad']); - $this->descripcion = $d['descripcion']; + if ($data) { + $this->idlinea = $this->intval($data['idlinea']); + $this->idtrans = $this->intval($data['idtrans']); + $this->referencia = $data['referencia']; + $this->cantidad = floatval($data['cantidad']); + $this->descripcion = $data['descripcion']; $this->fecha = NULL; - if (isset($d['fecha'])) { - $this->fecha = date('d-m-Y', strtotime($d['fecha'])); + if (isset($data['fecha'])) { + $this->fecha = date('d-m-Y', strtotime($data['fecha'])); } $this->hora = NULL; - if (isset($d['hora'])) { - $this->hora = $d['hora']; + if (isset($data['hora'])) { + $this->hora = $data['hora']; } } else { /// valores predeterminados diff --git a/model/core/partida.php b/model/core/partida.php index 4b50ecb..9c5fabc 100644 --- a/model/core/partida.php +++ b/model/core/partida.php @@ -75,38 +75,38 @@ class partida extends \fs_model public $sum_debe; public $sum_haber; - public function __construct($p = FALSE) + public function __construct($data = FALSE) { parent::__construct('co_partidas'); - if ($p) { - $this->idpartida = $this->intval($p['idpartida']); - $this->idasiento = $this->intval($p['idasiento']); - $this->idsubcuenta = $this->intval($p['idsubcuenta']); - $this->codsubcuenta = $p['codsubcuenta']; - $this->idconcepto = $this->intval($p['idconcepto']); - $this->concepto = $p['concepto']; - $this->idcontrapartida = $p['idcontrapartida']; - $this->codcontrapartida = $p['codcontrapartida']; - $this->punteada = $this->str2bool($p['punteada']); - - $this->tasaconv = floatval($p['tasaconv']); - $this->coddivisa = $p['coddivisa']; + if ($data) { + $this->idpartida = $this->intval($data['idpartida']); + $this->idasiento = $this->intval($data['idasiento']); + $this->idsubcuenta = $this->intval($data['idsubcuenta']); + $this->codsubcuenta = $data['codsubcuenta']; + $this->idconcepto = $this->intval($data['idconcepto']); + $this->concepto = $data['concepto']; + $this->idcontrapartida = $data['idcontrapartida']; + $this->codcontrapartida = $data['codcontrapartida']; + $this->punteada = $this->str2bool($data['punteada']); + + $this->tasaconv = floatval($data['tasaconv']); + $this->coddivisa = $data['coddivisa']; if (is_null($this->coddivisa)) { $this->coddivisa = $this->default_items->coddivisa(); } - $this->haberme = floatval($p['haberme']); - $this->debeme = floatval($p['debeme']); - $this->recargo = floatval($p['recargo']); - $this->iva = floatval($p['iva']); - $this->baseimponible = floatval($p['baseimponible']); - $this->factura = $this->intval($p['factura']); - $this->codserie = $p['codserie']; - $this->tipodocumento = $p['tipodocumento']; - $this->documento = $p['documento']; - $this->cifnif = $p['cifnif']; - $this->debe = floatval($p['debe']); - $this->haber = floatval($p['haber']); + $this->haberme = floatval($data['haberme']); + $this->debeme = floatval($data['debeme']); + $this->recargo = floatval($data['recargo']); + $this->iva = floatval($data['iva']); + $this->baseimponible = floatval($data['baseimponible']); + $this->factura = $this->intval($data['factura']); + $this->codserie = $data['codserie']; + $this->tipodocumento = $data['tipodocumento']; + $this->documento = $data['documento']; + $this->cifnif = $data['cifnif']; + $this->debe = floatval($data['debe']); + $this->haber = floatval($data['haber']); } else { $this->idpartida = NULL; $this->idasiento = NULL; @@ -323,7 +323,7 @@ public function all_from_subcuenta($id, $offset = 0) $saldo += floatval($po['debe']) - floatval($po['haber']); $sum_debe += floatval($po['debe']); $sum_haber += floatval($po['haber']); - if ($i >= $offset AND $i < ($offset + FS_ITEM_LIMIT)) { + if ($i >= $offset && $i < ($offset + FS_ITEM_LIMIT)) { $aux = $partida->get($po['idpartida']); if ($aux) { $aux->numero = intval($po['numero']); diff --git a/model/core/proveedor.php b/model/core/proveedor.php index 9c3530f..b7b2542 100644 --- a/model/core/proveedor.php +++ b/model/core/proveedor.php @@ -122,37 +122,37 @@ class proveedor extends \fs_model public $codcliente; private static $regimenes_iva; - public function __construct($p = FALSE) + public function __construct($data = FALSE) { parent::__construct('proveedores'); - if ($p) { - $this->codproveedor = $p['codproveedor']; - $this->nombre = $p['nombre']; + if ($data) { + $this->codproveedor = $data['codproveedor']; + $this->nombre = $data['nombre']; - if (is_null($p['razonsocial'])) { - $this->razonsocial = $p['nombrecomercial']; + if (is_null($data['razonsocial'])) { + $this->razonsocial = $data['nombrecomercial']; } else { - $this->razonsocial = $p['razonsocial']; + $this->razonsocial = $data['razonsocial']; } - $this->tipoidfiscal = $p['tipoidfiscal']; - $this->cifnif = $p['cifnif']; - $this->telefono1 = $p['telefono1']; - $this->telefono2 = $p['telefono2']; - $this->fax = $p['fax']; - $this->email = $p['email']; - $this->web = $p['web']; - $this->codserie = $p['codserie']; - $this->coddivisa = $p['coddivisa']; - $this->codpago = $p['codpago']; - $this->observaciones = $this->no_html($p['observaciones']); - $this->regimeniva = $p['regimeniva']; - $this->acreedor = $this->str2bool($p['acreedor']); - $this->personafisica = $this->str2bool($p['personafisica']); - - $this->debaja = $this->str2bool($p['debaja']); + $this->tipoidfiscal = $data['tipoidfiscal']; + $this->cifnif = $data['cifnif']; + $this->telefono1 = $data['telefono1']; + $this->telefono2 = $data['telefono2']; + $this->fax = $data['fax']; + $this->email = $data['email']; + $this->web = $data['web']; + $this->codserie = $data['codserie']; + $this->coddivisa = $data['coddivisa']; + $this->codpago = $data['codpago']; + $this->observaciones = $this->no_html($data['observaciones']); + $this->regimeniva = $data['regimeniva']; + $this->acreedor = $this->str2bool($data['acreedor']); + $this->personafisica = $this->str2bool($data['personafisica']); + + $this->debaja = $this->str2bool($data['debaja']); $this->fechabaja = date('d-m-Y'); - $this->codcliente = $p['codcliente']; + $this->codcliente = $data['codcliente']; } else { $this->codproveedor = NULL; $this->nombre = ''; @@ -280,7 +280,7 @@ public function get($cod) */ public function get_by_cifnif($cifnif, $razon = FALSE) { - if ($cifnif == '' AND $razon) { + if ($cifnif == '' && $razon) { $razon = mb_strtolower($this->no_html($razon), 'UTF8'); $sql = "SELECT * FROM " . $this->table_name . " WHERE cifnif = ''" . " AND lower(razonsocial) = " . $this->var2str($razon) . ";"; @@ -461,9 +461,9 @@ public function test() if (!preg_match("/^[A-Z0-9]{1,6}$/i", $this->codproveedor)) { $this->new_error_msg("Código de proveedor no válido."); - } else if (strlen($this->nombre) < 1 OR strlen($this->nombre) > 100) { + } else if (strlen($this->nombre) < 1 || strlen($this->nombre) > 100) { $this->new_error_msg("Nombre de proveedor no válido."); - } else if (strlen($this->razonsocial) < 1 OR strlen($this->razonsocial) > 100) { + } else if (strlen($this->razonsocial) < 1 || strlen($this->razonsocial) > 100) { $this->new_error_msg("Razón social del proveedor no válida."); } else { $status = TRUE; diff --git a/model/core/recalcular_stock.php b/model/core/recalcular_stock.php index f8d0c41..0780375 100644 --- a/model/core/recalcular_stock.php +++ b/model/core/recalcular_stock.php @@ -37,10 +37,10 @@ public function __construct() $this->regularizacion_stock = new \regularizacion_stock(); /// forzamos la comprobación de las tablas de albaranes - $albc = new albaran_cliente(); - $lin1 = new linea_albaran_cliente(); - $albp = new albaran_proveedor(); - $lin2 = new linea_albaran_proveedor(); + new albaran_cliente(); + new linea_albaran_cliente(); + new albaran_proveedor(); + new linea_albaran_proveedor(); } public function get_movimientos($ref, $codalmacen = '', $desde = '', $hasta = '', $codagente = '') diff --git a/model/core/regularizacion_iva.php b/model/core/regularizacion_iva.php index ae6e15b..6767903 100644 --- a/model/core/regularizacion_iva.php +++ b/model/core/regularizacion_iva.php @@ -43,17 +43,17 @@ class regularizacion_iva extends \fs_model public $fechainicio; public $periodo; - public function __construct($r = FALSE) + public function __construct($data = FALSE) { parent::__construct('co_regiva'); - if ($r) { - $this->idregiva = $this->intval($r['idregiva']); - $this->idasiento = $this->intval($r['idasiento']); - $this->codejercicio = $r['codejercicio']; - $this->fechaasiento = Date('d-m-Y', strtotime($r['fechaasiento'])); - $this->fechafin = Date('d-m-Y', strtotime($r['fechafin'])); - $this->fechainicio = Date('d-m-Y', strtotime($r['fechainicio'])); - $this->periodo = $r['periodo']; + if ($data) { + $this->idregiva = $this->intval($data['idregiva']); + $this->idasiento = $this->intval($data['idasiento']); + $this->codejercicio = $data['codejercicio']; + $this->fechaasiento = Date('d-m-Y', strtotime($data['fechaasiento'])); + $this->fechafin = Date('d-m-Y', strtotime($data['fechafin'])); + $this->fechainicio = Date('d-m-Y', strtotime($data['fechainicio'])); + $this->periodo = $data['periodo']; } else { $this->idregiva = NULL; $this->idasiento = NULL; diff --git a/model/core/regularizacion_stock.php b/model/core/regularizacion_stock.php index 36d0e3e..89ef02c 100644 --- a/model/core/regularizacion_stock.php +++ b/model/core/regularizacion_stock.php @@ -55,24 +55,24 @@ class regularizacion_stock extends \fs_model */ public $nick; - public function __construct($r = FALSE) + public function __construct($data = FALSE) { parent::__construct('lineasregstocks'); - if ($r) { - $this->id = $this->intval($r['id']); - $this->idstock = $this->intval($r['idstock']); - $this->cantidadini = floatval($r['cantidadini']); - $this->cantidadfin = floatval($r['cantidadfin']); - $this->codalmacendest = $r['codalmacendest']; - $this->fecha = date('d-m-Y', strtotime($r['fecha'])); + if ($data) { + $this->id = $this->intval($data['id']); + $this->idstock = $this->intval($data['idstock']); + $this->cantidadini = floatval($data['cantidadini']); + $this->cantidadfin = floatval($data['cantidadfin']); + $this->codalmacendest = $data['codalmacendest']; + $this->fecha = date('d-m-Y', strtotime($data['fecha'])); $this->hora = '00:00:00'; - if (!is_null($r['hora'])) { - $this->hora = date('H:i:s', strtotime($r['hora'])); + if (!is_null($data['hora'])) { + $this->hora = date('H:i:s', strtotime($data['hora'])); } - $this->motivo = $r['motivo']; - $this->nick = $r['nick']; + $this->motivo = $data['motivo']; + $this->nick = $data['nick']; } else { $this->id = NULL; $this->idstock = NULL; diff --git a/model/core/secuencia.php b/model/core/secuencia.php index d6a1a4d..b7aefc7 100644 --- a/model/core/secuencia.php +++ b/model/core/secuencia.php @@ -61,7 +61,9 @@ public function __construct($data = FALSE) protected function install() { - $sece = new \secuencia_ejercicio(); + /// necesitamos comprobar este modelo para que cree la tabla, para la clave ajena + new \secuencia_ejercicio(); + return ''; } diff --git a/model/core/stock.php b/model/core/stock.php index fa4556e..4bb13e9 100644 --- a/model/core/stock.php +++ b/model/core/stock.php @@ -43,22 +43,22 @@ class stock extends \fs_model public $cantidadultreg; public $ubicacion; - public function __construct($s = FALSE) + public function __construct($data = FALSE) { parent::__construct('stocks'); - if ($s) { - $this->idstock = $this->intval($s['idstock']); - $this->codalmacen = $s['codalmacen']; - $this->referencia = $s['referencia']; - $this->nombre = $s['nombre']; - $this->cantidad = floatval($s['cantidad']); - $this->reservada = floatval($s['reservada']); - $this->disponible = floatval($s['disponible']); - $this->pterecibir = floatval($s['pterecibir']); - $this->stockmin = floatval($s['stockmin']); - $this->stockmax = floatval($s['stockmax']); - $this->cantidadultreg = floatval($s['cantidadultreg']); - $this->ubicacion = $s['ubicacion']; + if ($data) { + $this->idstock = $this->intval($data['idstock']); + $this->codalmacen = $data['codalmacen']; + $this->referencia = $data['referencia']; + $this->nombre = $data['nombre']; + $this->cantidad = floatval($data['cantidad']); + $this->reservada = floatval($data['reservada']); + $this->disponible = floatval($data['disponible']); + $this->pterecibir = floatval($data['pterecibir']); + $this->stockmin = floatval($data['stockmin']); + $this->stockmax = floatval($data['stockmax']); + $this->cantidadultreg = floatval($data['cantidadultreg']); + $this->ubicacion = $data['ubicacion']; } else { $this->idstock = NULL; $this->codalmacen = NULL; @@ -103,7 +103,7 @@ public function set_cantidad($c = 0) { $this->cantidad = floatval($c); - if ($this->cantidad < 0 AND ! FS_STOCK_NEGATIVO) { + if ($this->cantidad < 0 && ! FS_STOCK_NEGATIVO) { $this->cantidad = 0; } @@ -115,7 +115,7 @@ public function sum_cantidad($c = 0) /// convertimos a flot por si acaso nos ha llegado un string $this->cantidad += floatval($c); - if ($this->cantidad < 0 AND ! FS_STOCK_NEGATIVO) { + if ($this->cantidad < 0 && ! FS_STOCK_NEGATIVO) { $this->cantidad = 0; } diff --git a/model/core/subcuenta.php b/model/core/subcuenta.php index a941746..63684b9 100644 --- a/model/core/subcuenta.php +++ b/model/core/subcuenta.php @@ -54,23 +54,23 @@ class subcuenta extends \fs_model public $recargo; public $iva; - public function __construct($s = FALSE) + public function __construct($data = FALSE) { parent::__construct('co_subcuentas'); - if ($s) { - $this->idsubcuenta = $this->intval($s['idsubcuenta']); - $this->codsubcuenta = $s['codsubcuenta']; - $this->idcuenta = $this->intval($s['idcuenta']); - $this->codcuenta = $s['codcuenta']; - $this->codejercicio = $s['codejercicio']; - $this->coddivisa = $s['coddivisa']; - $this->codimpuesto = $s['codimpuesto']; - $this->descripcion = $s['descripcion']; - $this->debe = floatval($s['debe']); - $this->haber = floatval($s['haber']); - $this->saldo = floatval($s['saldo']); - $this->recargo = floatval($s['recargo']); - $this->iva = floatval($s['iva']); + if ($data) { + $this->idsubcuenta = $this->intval($data['idsubcuenta']); + $this->codsubcuenta = $data['codsubcuenta']; + $this->idcuenta = $this->intval($data['idcuenta']); + $this->codcuenta = $data['codcuenta']; + $this->codejercicio = $data['codejercicio']; + $this->coddivisa = $data['coddivisa']; + $this->codimpuesto = $data['codimpuesto']; + $this->descripcion = $data['descripcion']; + $this->debe = floatval($data['debe']); + $this->haber = floatval($data['haber']); + $this->saldo = floatval($data['saldo']); + $this->recargo = floatval($data['recargo']); + $this->iva = floatval($data['iva']); } else { $this->idsubcuenta = NULL; $this->codsubcuenta = NULL; @@ -116,7 +116,8 @@ protected function install() } /// forzamos la creación de la tabla de cuentas - $cuenta = new \cuenta(); + new \cuenta(); + return ''; } @@ -307,7 +308,7 @@ public function test() $this->clean_cache(); } - if (strlen($this->codsubcuenta) > 0 AND strlen($this->descripcion) > 0) { + if (strlen($this->codsubcuenta) > 0 && strlen($this->descripcion) > 0) { return TRUE; } @@ -441,7 +442,7 @@ public function all_from_ejercicio($codejercicio, $random = FALSE, $limit = FALS { $sublist = array(); - if ($random AND $limit) { + if ($random && $limit) { if (strtolower(FS_DB_TYPE) == 'mysql') { $sql = "SELECT * FROM " . $this->table_name . " WHERE codejercicio = " . $this->var2str($codejercicio) . " ORDER BY RAND()"; diff --git a/model/core/subcuenta_cliente.php b/model/core/subcuenta_cliente.php index 676d2f5..2adfd2a 100644 --- a/model/core/subcuenta_cliente.php +++ b/model/core/subcuenta_cliente.php @@ -46,15 +46,15 @@ class subcuenta_cliente extends \fs_model public $codsubcuenta; public $codejercicio; - public function __construct($s = FALSE) + public function __construct($data = FALSE) { parent::__construct('co_subcuentascli'); - if ($s) { - $this->id = $this->intval($s['id']); - $this->idsubcuenta = $this->intval($s['idsubcuenta']); - $this->codcliente = $s['codcliente']; - $this->codsubcuenta = $s['codsubcuenta']; - $this->codejercicio = $s['codejercicio']; + if ($data) { + $this->id = $this->intval($data['id']); + $this->idsubcuenta = $this->intval($data['idsubcuenta']); + $this->codcliente = $data['codcliente']; + $this->codsubcuenta = $data['codsubcuenta']; + $this->codejercicio = $data['codejercicio']; } else { $this->id = NULL; $this->idsubcuenta = NULL; diff --git a/model/core/subcuenta_proveedor.php b/model/core/subcuenta_proveedor.php index 5a62f62..aca382f 100644 --- a/model/core/subcuenta_proveedor.php +++ b/model/core/subcuenta_proveedor.php @@ -46,15 +46,15 @@ class subcuenta_proveedor extends \fs_model public $codsubcuenta; public $codejercicio; - public function __construct($s = FALSE) + public function __construct($data = FALSE) { parent::__construct('co_subcuentasprov'); - if ($s) { - $this->id = $this->intval($s['id']); - $this->idsubcuenta = $this->intval($s['idsubcuenta']); - $this->codproveedor = $s['codproveedor']; - $this->codsubcuenta = $s['codsubcuenta']; - $this->codejercicio = $s['codejercicio']; + if ($data) { + $this->id = $this->intval($data['id']); + $this->idsubcuenta = $this->intval($data['idsubcuenta']); + $this->codproveedor = $data['codproveedor']; + $this->codsubcuenta = $data['codsubcuenta']; + $this->codejercicio = $data['codejercicio']; } else { $this->id = NULL; $this->idsubcuenta = NULL; diff --git a/model/core/tarifa.php b/model/core/tarifa.php index 7c099ca..05f3624 100644 --- a/model/core/tarifa.php +++ b/model/core/tarifa.php @@ -68,20 +68,20 @@ class tarifa extends \fs_model */ public $maxpvp; - public function __construct($t = FALSE) + public function __construct($data = FALSE) { parent::__construct('tarifas'); - if ($t) { - $this->codtarifa = $t['codtarifa']; - $this->nombre = $t['nombre']; - $this->incporcentual = floatval($t['incporcentual']); - $this->inclineal = floatval($t['inclineal']); - $this->mincoste = $this->str2bool($t['mincoste']); - $this->maxpvp = $this->str2bool($t['maxpvp']); + if ($data) { + $this->codtarifa = $data['codtarifa']; + $this->nombre = $data['nombre']; + $this->incporcentual = floatval($data['incporcentual']); + $this->inclineal = floatval($data['inclineal']); + $this->mincoste = $this->str2bool($data['mincoste']); + $this->maxpvp = $this->str2bool($data['maxpvp']); $this->aplicar_a = 'pvp'; - if (!is_null($t['aplicar_a'])) { - $this->aplicar_a = $t['aplicar_a']; + if (!is_null($data['aplicar_a'])) { + $this->aplicar_a = $data['aplicar_a']; } } else { $this->codtarifa = NULL; @@ -146,7 +146,6 @@ public function set_y($inc) */ public function diff() { - $texto = ''; $x = $this->x(); $y = $this->y(); @@ -192,7 +191,7 @@ public function set_precios(&$articulos) $pvp = $articulos[$i]->pvp; if ($this->aplicar_a == 'pvp') { - if ($this->y() == 0 AND $this->x() >= 0) { + if ($this->y() == 0 && $this->x() >= 0) { /// si y == 0 y x >= 0, usamos x como descuento $articulos[$i]->dtopor = $this->x(); } else { @@ -258,9 +257,9 @@ public function test() $this->codtarifa = trim($this->codtarifa); $this->nombre = $this->no_html($this->nombre); - if (strlen($this->codtarifa) < 1 OR strlen($this->codtarifa) > 6) { + if (strlen($this->codtarifa) < 1 || strlen($this->codtarifa) > 6) { $this->new_error_msg("Código de tarifa no válido. Debe tener entre 1 y 6 caracteres."); - } else if (strlen($this->nombre) < 1 OR strlen($this->nombre) > 50) { + } else if (strlen($this->nombre) < 1 || strlen($this->nombre) > 50) { $this->new_error_msg("Nombre de tarifa no válido. Debe tener entre 1 y 50 caracteres."); } else { $status = TRUE; diff --git a/model/core/terminal_caja.php b/model/core/terminal_caja.php index 270f0b1..5e6b013 100644 --- a/model/core/terminal_caja.php +++ b/model/core/terminal_caja.php @@ -87,37 +87,37 @@ class terminal_caja extends \fs_model */ public $sin_comandos; - public function __construct($t = FALSE) + public function __construct($data = FALSE) { parent::__construct('cajas_terminales'); - if ($t) { - $this->id = $this->intval($t['id']); - $this->codalmacen = $t['codalmacen']; - $this->codserie = $t['codserie']; - $this->codcliente = $t['codcliente']; - $this->tickets = $t['tickets']; + if ($data) { + $this->id = $this->intval($data['id']); + $this->codalmacen = $data['codalmacen']; + $this->codserie = $data['codserie']; + $this->codcliente = $data['codcliente']; + $this->tickets = $data['tickets']; $this->anchopapel = 40; - if (isset($t['anchopapel'])) { - $this->anchopapel = intval($t['anchopapel']); + if (isset($data['anchopapel'])) { + $this->anchopapel = intval($data['anchopapel']); } $this->comandocorte = '27.105'; - if (isset($t['comandocorte'])) { - $this->comandocorte = $t['comandocorte']; + if (isset($data['comandocorte'])) { + $this->comandocorte = $data['comandocorte']; } $this->comandoapertura = '27.112.48'; - if (isset($t['comandoapertura'])) { - $this->comandoapertura = $t['comandoapertura']; + if (isset($data['comandoapertura'])) { + $this->comandoapertura = $data['comandoapertura']; } $this->num_tickets = 1; - if (isset($t['num_tickets'])) { - $this->num_tickets = intval($t['num_tickets']); + if (isset($data['num_tickets'])) { + $this->num_tickets = intval($data['num_tickets']); } - $this->sin_comandos = $this->str2bool($t['sin_comandos']); + $this->sin_comandos = $this->str2bool($data['sin_comandos']); } else { $this->id = NULL; $this->codalmacen = NULL; diff --git a/model/core/transferencia_stock.php b/model/core/transferencia_stock.php index 43847b9..34a5ed7 100644 --- a/model/core/transferencia_stock.php +++ b/model/core/transferencia_stock.php @@ -21,16 +21,16 @@ class transferencia_stock extends \fs_model public $hora; public $usuario; - public function __construct($d = FALSE) + public function __construct($data = FALSE) { parent::__construct('transstock'); - if ($d) { - $this->idtrans = $this->intval($d['idtrans']); - $this->codalmadestino = $d['codalmadestino']; - $this->codalmaorigen = $d['codalmaorigen']; - $this->fecha = date("d-m-Y", strtotime($d['fecha'])); - $this->hora = date('H:i:s', strtotime($d['hora'])); - $this->usuario = $d['usuario']; + if ($data) { + $this->idtrans = $this->intval($data['idtrans']); + $this->codalmadestino = $data['codalmadestino']; + $this->codalmaorigen = $data['codalmaorigen']; + $this->fecha = date("d-m-Y", strtotime($data['fecha'])); + $this->hora = date('H:i:s', strtotime($data['hora'])); + $this->usuario = $data['usuario']; } else { /// valores predeterminados $this->idtrans = NULL; diff --git a/view/admin_almacenes.html b/view/admin_almacenes.html index 7c35378..57340ae 100644 --- a/view/admin_almacenes.html +++ b/view/admin_almacenes.html @@ -74,7 +74,7 @@

Código: {$value->codalmacen}

País @@ -112,21 +112,21 @@

Nuevo...

- {if="$value->idlalbcompra OR $value->idlfaccompra"} + {if="$value->idlalbcompra || $value->idlfaccompra"} {/if} - {if="$value->idlalbcompra OR $value->idlfaccompra"} + {if="$value->idlalbcompra || $value->idlfaccompra"}
{$value->fecha_entrada}
{else} {/if} - {if="$value->idlalbventa OR $value->idlfacventa"} + {if="$value->idlalbventa || $value->idlfacventa"} diff --git a/view/block/nueva_compra.html b/view/block/nueva_compra.html index 0611c02..a9c4133 100644 --- a/view/block/nueva_compra.html +++ b/view/block/nueva_compra.html @@ -71,7 +71,7 @@

{loop="$fsc->serie->all()"} {if="$value->codserie==$fsc->proveedor_s->codserie"} - {elseif="$value->is_default() AND is_null($fsc->proveedor_s->codserie)"} + {elseif="$value->is_default() && is_null($fsc->proveedor_s->codserie)"} {else} diff --git a/view/block/nueva_venta.html b/view/block/nueva_venta.html index 0126fb9..073b7d0 100644 --- a/view/block/nueva_venta.html +++ b/view/block/nueva_venta.html @@ -80,7 +80,7 @@

{loop="$fsc->serie->all()"} {if="$value->codserie==$fsc->cliente_s->codserie"} - {elseif="$value->is_default() AND is_null($fsc->cliente_s->codserie)"} + {elseif="$value->is_default() && is_null($fsc->cliente_s->codserie)"} {else} diff --git a/view/block/ventas_articulo_precios.html b/view/block/ventas_articulo_precios.html index 01ec0b9..b25d153 100644 --- a/view/block/ventas_articulo_precios.html +++ b/view/block/ventas_articulo_precios.html @@ -46,7 +46,7 @@
Precio de Coste: - {if="$fsc->articulo->secompra AND FS_COST_IS_AVERAGE"} + {if="$fsc->articulo->secompra && FS_COST_IS_AVERAGE"}

El precio de compra se calcula automáticamente. Si quieres editarlo puedes diff --git a/view/compras_albaran.html b/view/compras_albaran.html index 7b911f3..6c9faf6 100644 --- a/view/compras_albaran.html +++ b/view/compras_albaran.html @@ -373,7 +373,7 @@ - {if="$fsc->button_plazos AND $value->genrecibos=='Emitidos'"} + {if="$fsc->button_plazos && $value->genrecibos=='Emitidos'"}   Plazos de pago diff --git a/view/feedback.html b/view/feedback.html index aaeb23b..baa1d6d 100644 --- a/view/feedback.html +++ b/view/feedback.html @@ -1,4 +1,4 @@ -

+ -{if="$fsc->empresa AND !FS_DEMO AND mt_rand(0,49)==0"} +{if="$fsc->empresa && !FS_DEMO && mt_rand(0,49)==0"}
- {if="mt_rand(0,2)>0 AND $fsc->user->logged_on"} + {if="mt_rand(0,2)>0 && $fsc->user->logged_on"} {else} - + + {/if}
{/if} diff --git a/view/ventas_albaran.html b/view/ventas_albaran.html index a10aced..752c419 100644 --- a/view/ventas_albaran.html +++ b/view/ventas_albaran.html @@ -389,7 +389,7 @@ -
+

  {$value->id}   |   diff --git a/view/ventas_factura.html b/view/ventas_factura.html index ae235d5..52fe132 100644 --- a/view/ventas_factura.html +++ b/view/ventas_factura.html @@ -253,7 +253,7 @@
Vencimiento: - {if="$fsc->factura->pagada OR !$fsc->mostrar_boton_pagada"} + {if="$fsc->factura->pagada || !$fsc->mostrar_boton_pagada"} {else} diff --git a/view/ventas_facturas.html b/view/ventas_facturas.html index 16b7496..bb108f6 100644 --- a/view/ventas_facturas.html +++ b/view/ventas_facturas.html @@ -356,7 +356,7 @@ {loop="$fsc->resultados"} - vencida() OR $value->anulada"} danger{elseif="$value->pagada"} success{elseif="$value->total<=0"} warning{/if}" href="{$value->url()}"> + vencida() || $value->anulada"} danger{elseif="$value->pagada"} success{elseif="$value->total<=0"} warning{/if}" href="{$value->url()}"> {if="$value->pagada"} @@ -466,7 +466,7 @@
{else} {loop="$fsc->extensions"} - {if="$value->type=='tab' AND $fsc->mostrar=='ext_'.$value->name"} + {if="$value->type=='tab' && $fsc->mostrar=='ext_'.$value->name"} {/if} {/loop} diff --git a/view/ventas_trazabilidad.html b/view/ventas_trazabilidad.html index 63bc9a8..f7936e7 100644 --- a/view/ventas_trazabilidad.html +++ b/view/ventas_trazabilidad.html @@ -52,7 +52,7 @@

{if="$value2->lote"}{$value2->lote}{else}-{/if} - {if="$value2->idlalbcompra OR $value2->idlfaccompra"} + {if="$value2->idlalbcompra || $value2->idlfaccompra"} @@ -102,7 +102,7 @@

{/if} - {if="$fsc->lineas OR $fsc->disponibles"} + {if="$fsc->lineas || $fsc->disponibles"}