From 894c4b6cb944833a8cea4532e1115b97cfc4b1cf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jaime=20Jim=C3=A9nez=20Ortega?= Date: Sun, 12 Nov 2023 17:27:54 +0100 Subject: [PATCH] solucionados bugs (pe, cuando es cubierta o sobrecubierta no se necesita color=1) --- .../Presupuestos/Cosidotapablanda.php | 4 +- ci4/app/Language/en/Presupuestos.php | 1 + ci4/app/Language/es/Presupuestos.php | 3 +- .../Configuracion/PapelGenericoModel.php | 11 +-- .../Configuracion/PapelImpresionModel.php | 13 ++-- .../_datosPresupuestoClienteItems.php | 3 +- .../_lineasPresupuestoItems.php | 50 ++++++++----- .../cosidotapablanda/comparador.js | 17 ++++- .../viewCosidotapablandaForm.php | 70 ++++++++----------- 9 files changed, 98 insertions(+), 74 deletions(-) diff --git a/ci4/app/Controllers/Presupuestos/Cosidotapablanda.php b/ci4/app/Controllers/Presupuestos/Cosidotapablanda.php index baf1c877..fa2e1b64 100644 --- a/ci4/app/Controllers/Presupuestos/Cosidotapablanda.php +++ b/ci4/app/Controllers/Presupuestos/Cosidotapablanda.php @@ -527,14 +527,14 @@ class Cosidotapablanda extends \App\Controllers\GoBaseResourceController if( $uso == 'cubierta' ){ $opciones_papel = array( 'cubierta' => 1, - 'color' => 1, + //'color' => 1, 'rotativa' => 0, ); } else if ( $uso == 'sobrecubierta' ){ $opciones_papel = array( 'sobrecubierta' => 1, - 'color' => 1, + //'color' => 1, 'rotativa' => 0, ); } diff --git a/ci4/app/Language/en/Presupuestos.php b/ci4/app/Language/en/Presupuestos.php index 5805cb1c..a928c11a 100644 --- a/ci4/app/Language/en/Presupuestos.php +++ b/ci4/app/Language/en/Presupuestos.php @@ -146,6 +146,7 @@ return [ 'tiradas' => 'The field print must be greater than zero', 'dimension' => 'The book dimension must be greater than 60mm', 'lineaDuplicada' => 'That type of line already exists in the budget.', + 'errorRotColor' => 'Papers and weights must be the same in color and BW', ], /* '4x0' => '4x0', diff --git a/ci4/app/Language/es/Presupuestos.php b/ci4/app/Language/es/Presupuestos.php index 1bb07df4..8339ceb2 100644 --- a/ci4/app/Language/es/Presupuestos.php +++ b/ci4/app/Language/es/Presupuestos.php @@ -174,7 +174,8 @@ return [ 'paginas' => 'El campo páginas tiene que ser mayor que cero', 'tiradas' => 'El campo tiradas tiene que ser mayor que cero', 'dimension' => 'La dimensión del libro tiene que ser mayor que 60mm', - 'lineaDuplicada' => 'Ya existe ese tipo de linea en el presupuesto' + 'lineaDuplicada' => 'Ya existe ese tipo de linea en el presupuesto', + 'errorRotColor' => 'Papeles y gramajes deben ser iguales en color y BN', ], diff --git a/ci4/app/Models/Configuracion/PapelGenericoModel.php b/ci4/app/Models/Configuracion/PapelGenericoModel.php index ce45bbfa..4d412398 100644 --- a/ci4/app/Models/Configuracion/PapelGenericoModel.php +++ b/ci4/app/Models/Configuracion/PapelGenericoModel.php @@ -226,10 +226,6 @@ class PapelGenericoModel extends \App\Models\GoBaseModel $uso_tarifa = 'interior'; - if($tipo == 'negro' || $tipo == 'negrohq') - $builder->where("t2.bn", 1); - else if ($tipo == 'color' || $tipo == 'colorhq') - $builder->where("t2.color", 1); if ($uso == 'cubierta'){ $uso_tarifa = 'cubierta'; $builder->where("t2.cubierta", 1); @@ -238,6 +234,13 @@ class PapelGenericoModel extends \App\Models\GoBaseModel $uso_tarifa = 'sobrecubierta'; $builder->where("t2.sobrecubierta", 1); } + else{ + if($tipo == 'negro' || $tipo == 'negrohq') + $builder->where("t2.bn", 1); + else if ($tipo == 'color' || $tipo == 'colorhq') + $builder->where("t2.color", 1); + } + if($uso=='rotativa') $builder->where("t2.rotativa", 1); else diff --git a/ci4/app/Models/Configuracion/PapelImpresionModel.php b/ci4/app/Models/Configuracion/PapelImpresionModel.php index f9474e97..7e0cacc9 100644 --- a/ci4/app/Models/Configuracion/PapelImpresionModel.php +++ b/ci4/app/Models/Configuracion/PapelImpresionModel.php @@ -283,20 +283,21 @@ class PapelImpresionModel extends \App\Models\GoBaseModel $builder->where("t2.id", $papel_generico); $builder->where("t1.gramaje", $gramaje); - if($tipo == 'negro' || $tipo == 'negrohq') - $builder->where("t1.bn", 1); - else if ($tipo == 'color' || $tipo == 'colorhq') - $builder->where("t1.color", 1); if ($uso == 'cubierta') $builder->where("t1.cubierta", 1); else if ($uso == 'sobrecubierta') $builder->where("t1.sobrecubierta", 1); + else{ + if($tipo == 'negro' || $tipo == 'negrohq') + $builder->where("t1.bn", 1); + else if ($tipo == 'color' || $tipo == 'colorhq') + $builder->where("t1.color", 1); + } if($uso=='rotativa') $builder->where("t1.rotativa", 1); else $builder->where("t1.rotativa", 0); - - + return $builder->orderBy("t1.id", "asc")->get()->getResultObject(); } } diff --git a/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/_datosPresupuestoClienteItems.php b/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/_datosPresupuestoClienteItems.php index a380239f..d66cd0a2 100644 --- a/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/_datosPresupuestoClienteItems.php +++ b/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/_datosPresupuestoClienteItems.php @@ -345,7 +345,8 @@ - : + + : 0.00 diff --git a/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/_lineasPresupuestoItems.php b/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/_lineasPresupuestoItems.php index 01476f71..06493685 100644 --- a/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/_lineasPresupuestoItems.php +++ b/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/_lineasPresupuestoItems.php @@ -997,52 +997,66 @@ $('.insertarLinea').on("click", function (e) { $('#addLineasPresupuesto').on("change", function (e) { switch($('#addLineasPresupuesto').val()){ + // Se chequean todas las lineas de negro y todas las de color + // (solo una de cada en interior) case 'lp_bn': - if($('#lp_bn').css('display') == 'none'){ - $('.lp_bn').css('display', 'flex') + if($('#lp_bn').css('display') != 'none' || + $('#lp_bnhq').css('display') != 'none' || + $('#lp_rot_bn').css('display') != 'none'){ + popErrorAlert('', 'divAlarmasLineasPresupuesto'); } else{ - popErrorAlert('', 'divAlarmasLineasPresupuesto'); + $('.lp_bn').css('display', 'flex') } break case 'lp_bnhq': - if($('#lp_bnhq').css('display') == 'none'){ - $('.lp_bnhq').css('display', 'flex') + if($('#lp_bn').css('display') != 'none' || + $('#lp_bnhq').css('display') != 'none' || + $('#lp_rot_bn').css('display') != 'none'){ + popErrorAlert('', 'divAlarmasLineasPresupuesto'); } else{ - popErrorAlert('', 'divAlarmasLineasPresupuesto'); + $('.lp_bnhq').css('display', 'flex') } break case 'lp_color': - if($('#lp_color').css('display') == 'none'){ - $('.lp_color').css('display', 'flex') + if($('#lp_color').css('display') != 'none' || + $('#lp_colorhq').css('display') != 'none' || + $('#lp_rot_color').css('display') != 'none'){ + popErrorAlert('', 'divAlarmasLineasPresupuesto'); } else{ - popErrorAlert('', 'divAlarmasLineasPresupuesto'); + $('.lp_color').css('display', 'flex') } break case 'lp_colorhq': - if($('#lp_colorhq').css('display') == 'none'){ - $('.lp_colorhq').css('display', 'flex') + if($('#lp_color').css('display') != 'none' || + $('#lp_colorhq').css('display') != 'none' || + $('#lp_rot_color').css('display') != 'none'){ + popErrorAlert('', 'divAlarmasLineasPresupuesto'); } else{ - popErrorAlert('', 'divAlarmasLineasPresupuesto'); + $('.lp_colorhq').css('display', 'flex') } break case 'lp_rot_bn': - if($('#lp_rot_bn').css('display') == 'none'){ - $('.lp_rot_bn').css('display', 'flex') + if($('#lp_bn').css('display') != 'none' || + $('#lp_bnhq').css('display') != 'none' || + $('#lp_rot_bn').css('display') != 'none'){ + popErrorAlert('', 'divAlarmasLineasPresupuesto'); } else{ - popErrorAlert('', 'divAlarmasLineasPresupuesto'); + $('.lp_rot_bn').css('display', 'flex') } break case 'lp_rot_color': - if($('#lp_rot_color').css('display') == 'none'){ - $('.lp_rot_color').css('display', 'flex') + if($('#lp_color').css('display') != 'none' || + $('#lp_colorhq').css('display') != 'none' || + $('#lp_rot_color').css('display') != 'none'){ + popErrorAlert('', 'divAlarmasLineasPresupuesto'); } else{ - popErrorAlert('', 'divAlarmasLineasPresupuesto'); + $('.lp_rot_color').css('display', 'flex') } break case 'lp_cubierta': diff --git a/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/comparador.js b/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/comparador.js index 433e0d47..70093224 100644 --- a/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/comparador.js +++ b/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/comparador.js @@ -357,7 +357,7 @@ function clearNegro() { function clearIntLineas(is_color){ - $("#tableCompIntPlana").DataTable().rows( '.selected' ).deselect(); + //$("#tableCompIntPlana").DataTable().rows( '.selected' ).deselect(); var tabla = $('#tableCompIntPlana').DataTable(); tabla .rows( function ( idx, data, node ) { @@ -365,6 +365,7 @@ function clearIntLineas(is_color){ } ) .remove() .draw(); + //$('#tableCompIntPlana').DataTable().clear().draw(); $('#insertarPlanaBtn').css('display', 'none'); } @@ -372,12 +373,15 @@ function clearIntLineas(is_color){ function clearCubierta(){ $('#tableCompCubierta').DataTable().clear().draw(); + $('#insertarCubiertaBtn').css('display', 'none'); } function clearIntRot(){ + $('#errorComRot').html(''); $('#tableCompIntRotativa').DataTable().clear().draw(); + $('#insertarRotativaBtn').css('display', 'none'); } @@ -568,3 +572,14 @@ function fillCubierta(data, is_color, is_hq){ .draw() }) } + +function clearCompTable(clearNegro=true,clearColor=false,clearCubierta=false){ + if(clearNegro){ + + } + + $('#insertarPlanaBtn').addClass('d-none') + $('#insertarRotativaBtn').addClass('d-none') + $('#total_comp_plana').html('0.00') + $('#total_comp_rot').html('0.00') +} diff --git a/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/viewCosidotapablandaForm.php b/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/viewCosidotapablandaForm.php index 5644cb26..5e9f843b 100644 --- a/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/viewCosidotapablandaForm.php +++ b/ci4/app/Views/themes/backend/vuexy/form/presupuestos/cosidotapablanda/viewCosidotapablandaForm.php @@ -564,33 +564,6 @@ }); - function checkComparadorInt(is_color, is_hq) { - - elementos = getIDsComparador(is_color, is_hq) - - if ($(''+ elementos.papel).select2('data').length > 0 && - $(''+ elementos.gramaje).select2('data').length > 0 && - checkDatosPedidoForComp()) { - - getLineasIntPlana(is_color, is_hq).then((result) =>{ - - // Para rotativa, si es color el papel y el gramaje tiene que ser igual - if(!is_color || ($('#compPapelNegro').select2('data')[0].id == $('#compPapelColor').select2('data')[0].id && - $('#compGramajeNegro').select2('data')[0].text.trim() == $('#compGramajeColor').select2('data')[0].text.trim())) - { - getLineasIntRot(is_hq); - } - }); - } - else{ - clearIntLineas(is_color); - clearIntRot(); - $('#title_int_rot').html(''); - $('#title_int_plana').html(''); - - } - } - $('.comp_negro_items').on('change', function (e) { checkComparadorInt(false,false); @@ -818,23 +791,38 @@ function checkComparadorInt(is_color, is_hq) { - elementos = getIDsComparador(is_color, is_hq) + clearIntLineas(is_color); + clearIntRot(); + $('#title_int_rot').html(''); + $('#title_int_plana').html(''); + elementos = getIDsComparador(is_color, is_hq) + if ($(''+ elementos.papel).select2('data').length > 0 && $(''+ elementos.gramaje).select2('data').length > 0 && + parseInt($(''+ elementos.paginas).val()) > 0 && checkDatosPedidoForComp()) { getLineasIntPlana(is_color, is_hq).then((result) =>{ - getLineasIntRot(is_hq); + // Para rotativa, si es color el papel y el gramaje tiene que ser igual + if(!is_color) + { + getLineasIntRot(is_hq); + } + else + { + if(($('#compPapelNegro').select2('data')[0].id == $('#compPapelColor').select2('data')[0].id && + $('#compGramajeNegro').select2('data')[0].text.trim() == $('#compGramajeColor').select2('data')[0].text.trim())) + { + getLineasIntRot(is_hq); + } + else + { + $('#errorComRot').html(''); + } + } }); } - else{ - clearIntLineas(is_color); - clearIntRot(); - $('#title_int_rot').html(''); - $('#title_int_plana').html(''); - - } } @@ -854,7 +842,7 @@ $('#compGramajeNegro').empty().trigger("change"); $('#compGramajeNegro').prop('disabled', false); $('#compGramajeNegro').select2({ - allowClear: true, + allowClear: false, minimumResultsForSearch: Infinity, ajax: { url: '', @@ -896,7 +884,7 @@ $('#compGramajeNegrohq').empty().trigger("change"); $('#compGramajeNegrohq').prop('disabled', false); $('#compGramajeNegrohq').select2({ - allowClear: true, + allowClear: false, minimumResultsForSearch: Infinity, ajax: { url: '', @@ -940,7 +928,7 @@ $('#compGramajeColor').prop('disabled', false); $('#compPapelColor').find('option[value="0"]').remove(); $('#compGramajeColor').select2({ - allowClear: true, + allowClear: false, minimumResultsForSearch: Infinity, ajax: { url: '', @@ -984,7 +972,7 @@ $('#compGramajeColorhq').prop('disabled', false); $('#compPapelColorhq').find('option[value="0"]').remove(); $('#compGramajeColorhq').select2({ - allowClear: true, + allowClear: false, minimumResultsForSearch: Infinity, ajax: { url: '', @@ -1024,7 +1012,7 @@ $('#compGramajeCubierta').prop('disabled', false); $('#compPapelCubierta').find('option[value="0"]').remove(); $('#compGramajeCubierta').select2({ - allowClear: true, + allowClear: false, minimumResultsForSearch: Infinity, ajax: { url: '',