Skip to content

Modelo 303: corregir redondeo y mapear todas las operaciones (tarea 4542) #24

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
daniel89fg merged 3 commits into from
Jun 16, 2026
Merged

Modelo 303: corregir redondeo y mapear todas las operaciones (tarea 4542) #24

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
365b1e1
feat: 4542 mapear casillas del modelo 303 según el tipo de operación
Jun 11, 2026
fa34240
fix: 4542 corregir desfase de céntimos unificando consultas y cuota
Jun 11, 2026
9dfd2c8
feat: actualizar estructura de tablas en el modelo 303 para mejorar l…
Jun 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
204 changes: 112 additions & 92 deletions Controller/EditRegularizacionImpuesto.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,44 @@ protected function checkInvoicesWithoutAccounting($model): void
}
}

/**
* Builds the common filter used by the summary, purchases/sales and accounting-entry
* queries, so the three of them always work over the SAME set of accounting entries.
*
* @param int $group grupo de cuentas especiales (TAX_ALL, TAX_INPUT, TAX_OUTPUT)
* @return array
*/
private function commonTaxWhere(int $group): array
{
$model = $this->getModel();
$excludedOperations = implode(',', [Asiento::OPERATION_OPENING, Asiento::OPERATION_CLOSING]);

// ids de los asientos de TODAS las regularizaciones (para excluirlos)
$regIds = [];
foreach (RegularizacionImpuesto::all() as $reg) {
if ($reg->idasiento) {
$regIds[] = $reg->idasiento;
}
}

$subAccountTools = new SubAccountTools();
$where = [
Where::eq('asientos.idempresa', $model->idempresa),
Where::eq('asientos.codejercicio', $model->codejercicio),
Where::gte('asientos.fecha', $model->fechainicio),
Where::lte('asientos.fecha', $model->fechafin),
Where::eq('COALESCE(series.siniva, false)', false),
Where::notIn("COALESCE(asientos.operacion, '')", $excludedOperations),
$subAccountTools->whereForSpecialAccounts('COALESCE(subcuentas.codcuentaesp, cuentas.codcuentaesp)', $group),
];

if (false === empty($regIds)) {
$where[] = Where::notIn('asientos.idasiento', implode(',', $regIds));
}

return $where;
}

/**
* Create accounting entry action procedure.
*
Expand Down Expand Up @@ -199,6 +237,22 @@ protected function createViewsTaxSummary(string $viewName = 'Modelo303'): void
$this->modelo303 = new Modelo303();
}

/**
* Setup actions for view.
*
* @param string $viewName
* @param bool $clickable
* @return void
*/
private function disableButtons(string $viewName, bool $clickable = false): void
{
$this->setSettings($viewName, 'btnDelete', false)
->setSettings('btnNew', false)
->setSettings('checkBoxes', false)
->setSettings('clickable', $clickable)
->setSettings('btnPrint', true);
}

/**
* Run the actions that alter data before reading it.
*
Expand Down Expand Up @@ -270,78 +324,6 @@ protected function exportAction(): void
$this->exportManager->show($this->response);
}

/**
* Load data view procedure
*
* @param string $viewName
* @param BaseView $view
* @throws Exception
*/
protected function loadData($viewName, $view): void
{
switch ($viewName) {
case 'EditRegularizacionImpuesto':
parent::loadData($viewName, $view);
$this->settingsMainView();
break;

case 'ListPartidaImpuestoResumen':
$mainModel = $this->getModel();
$excludedOperations = implode(',', [Asiento::OPERATION_OPENING, Asiento::OPERATION_CLOSING]);
$where = [
Where::sub([
Where::like('partidas.codsubcuenta', '477%'),
Where::orLike('partidas.codsubcuenta', '472%'),
]),
Where::eq('asientos.idempresa', $mainModel->idempresa),
Where::gte('asientos.fecha', $mainModel->fechainicio),
Where::lte('asientos.fecha', $mainModel->fechafin),
Where::eq('COALESCE(series.siniva, false)', false),
Where::notIn("COALESCE(asientos.operacion, '')", $excludedOperations),
];

if (false === empty($mainModel->idasiento)) {
$where[] = Where::notEq('partidas.idasiento', $mainModel->idasiento);
}
$view->loadData(false, $where, [
'COALESCE(subcuentas.codcuentaesp, cuentas.codcuentaesp)' => 'ASC',
'partidas.codsubcuenta' => 'ASC',
]);

$this->modelo303->loadFromResumen($view->cursor); // Load data into Modelo303 View
$this->checkInvoicesWithoutAccounting($mainModel);
break;

case 'ListPartida':
$this->getListPartida($view);
break;

case 'ListPartidaImpuesto-1':
$this->getListPartidaImpuesto($view, SubAccountTools::SPECIAL_GROUP_TAX_INPUT);
break;

case 'ListPartidaImpuesto-2':
$this->getListPartidaImpuesto($view, SubAccountTools::SPECIAL_GROUP_TAX_OUTPUT);
break;
}
}

/**
* Setup actions for view.
*
* @param string $viewName
* @param bool $clickable
* @return void
*/
private function disableButtons(string $viewName, bool $clickable = false): void
{
$this->setSettings($viewName, 'btnDelete', false)
->setSettings('btnNew', false)
->setSettings('checkBoxes', false)
->setSettings('clickable', $clickable)
->setSettings('btnPrint', true);
}

/**
* Load data for accounting entry.
*
Expand Down Expand Up @@ -383,27 +365,65 @@ private function getListPartidaImpuesto(BaseView $view, int $group): void
*/
private function getPartidaImpuestoWhere(int $group): array
{
// obtenemos todos los ids de los asientos de las regularizaciones
$ids = [];
foreach (RegularizacionImpuesto::all() as $reg) {
if ($reg->idasiento) {
$ids[] = $reg->idasiento;
}
}
return $this->commonTaxWhere($group);
}

$subAccountTools = new SubAccountTools();
return [
Where::notIn('asientos.idasiento', implode(',', $ids)),
Where::eq('asientos.codejercicio', $this->getModel()->codejercicio),
Where::gte('asientos.fecha', $this->getModel()->fechainicio),
Where::lte('asientos.fecha', $this->getModel()->fechafin),
Where::eq('COALESCE(series.siniva, 0)', 0),
Where::sub([
Where::notEq('partidas.baseimponible', 0),
Where::orGt('COALESCE(partidas.iva, 0)', 0),
]),
$subAccountTools->whereForSpecialAccounts('COALESCE(subcuentas.codcuentaesp, cuentas.codcuentaesp)', $group)
];
/**
* Load data view procedure
*
* @param string $viewName
* @param BaseView $view
* @throws Exception
*/
protected function loadData($viewName, $view): void
{
switch ($viewName) {
case 'EditRegularizacionImpuesto':
parent::loadData($viewName, $view);
$this->settingsMainView();
break;

case 'ListPartidaImpuestoResumen':
$mainModel = $this->getModel();

// mismo criterio que las pestañas Compras/Ventas y que el asiento de regularización
$where = $this->commonTaxWhere(SubAccountTools::SPECIAL_GROUP_TAX_ALL);
$view->loadData(false, $where, [
'COALESCE(subcuentas.codcuentaesp, cuentas.codcuentaesp)' => 'ASC',
'partidas.codsubcuenta' => 'ASC',
]);

// cargamos el resumen de IVA (casillas del régimen general)
$this->modelo303->loadFromResumen($view->cursor);

// cargamos las bases exentas/informativas de ventas (casillas 59, 60 y 122)
$this->modelo303->loadFromSalesInvoices(
(int)$mainModel->idempresa,
(string)$mainModel->codejercicio,
(string)$mainModel->fechainicio,
(string)$mainModel->fechafin
);

// mostramos los avisos de importes que no encajan en ninguna casilla
foreach ($this->modelo303->getAvisos() as $aviso) {
Tools::log()->warning($aviso);
}

$this->checkInvoicesWithoutAccounting($mainModel);
break;

case 'ListPartida':
$this->getListPartida($view);
break;

case 'ListPartidaImpuesto-1':
$this->getListPartidaImpuesto($view, SubAccountTools::SPECIAL_GROUP_TAX_INPUT);
break;

case 'ListPartidaImpuesto-2':
$this->getListPartidaImpuesto($view, SubAccountTools::SPECIAL_GROUP_TAX_OUTPUT);
break;
}
}

/**
Expand Down
67 changes: 41 additions & 26 deletions Lib/Accounting/VatRegularizationToAccounting.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,18 +123,59 @@ protected function addAccountingTaxLines(Asiento $accEntry, RegularizacionImpues
return true;
}

/**
* Comprueba si hay facturas sin asiento contable
*
* @param $reg
*
* @return bool
*/
private function checkInvoicesWithoutAccEntry($reg): bool
{
$where = [
Where::eq('codejercicio', $reg->codejercicio),
Where::eq('idempresa', $reg->idempresa),
Where::gte('fecha', $reg->fechainicio),
Where::lte('fecha', $reg->fechafin),
Where::isNull('idasiento'),
];

$facturasSinAsiento = FacturaCliente::all($where, [], 0, 1);
if (false === empty($facturasSinAsiento)) {
return false;
}

$facturasSinAsiento = FacturaProveedor::all($where, [], 0, 1);
return empty($facturasSinAsiento);
}

protected function getSubtotals(RegularizacionImpuesto $reg): array
{
$accTools = new SubAccountTools();
$field = 'COALESCE(subcuentas.codcuentaesp, cuentas.codcuentaesp)';
$excludedOperations = implode(',', [Asiento::OPERATION_OPENING, Asiento::OPERATION_CLOSING]);

// mismo criterio que las pestañas Resumen/Compras/Ventas para que el asiento cuadre con ellas
$regIds = [];
foreach (RegularizacionImpuesto::all() as $other) {
if ($other->idasiento) {
$regIds[] = $other->idasiento;
}
}

$where = [
Where::eq('asientos.idempresa', $reg->idempresa),
Where::eq('asientos.codejercicio', $reg->codejercicio),
Where::gte('asientos.fecha', $reg->fechainicio),
Where::lte('asientos.fecha', $reg->fechafin),
Where::eq('COALESCE(series.siniva, false)', false),
Where::notIn("COALESCE(asientos.operacion, '')", $excludedOperations),
$accTools->whereForSpecialAccounts($field, SubAccountTools::SPECIAL_GROUP_TAX_ALL)
];

if (false === empty($regIds)) {
$where[] = Where::notIn('asientos.idasiento', implode(',', $regIds));
}
$orderBy = [
$field => 'ASC',
'partidas.iva' => 'ASC',
Expand All @@ -159,30 +200,4 @@ protected function getSubtotals(RegularizacionImpuesto $reg): array

return $subtotals;
}

/**
* Comprueba si hay facturas sin asiento contable
*
* @param $reg
*
* @return bool
*/
private function checkInvoicesWithoutAccEntry($reg): bool
{
$where = [
Where::eq('codejercicio', $reg->codejercicio),
Where::eq('idempresa', $reg->idempresa),
Where::gte('fecha', $reg->fechainicio),
Where::lte('fecha', $reg->fechafin),
Where::isNull('idasiento'),
];

$facturasSinAsiento = FacturaCliente::all($where, [], 0, 1);
if (false === empty($facturasSinAsiento)) {
return false;
}

$facturasSinAsiento = FacturaProveedor::all($where, [], 0, 1);
return empty($facturasSinAsiento);
}
}
Loading