FIX: Pie cuotas valores uf
This commit is contained in:
@ -24,7 +24,8 @@ class Pago extends Model
|
||||
|
||||
public function valor(string $moneda = Pago::UF): float
|
||||
{
|
||||
return $this->valor / (($moneda === Pago::UF) ? ($this->uf > 0 ? $this->uf : 1) : 1);
|
||||
$uf = $this->uf ?? ($this->uf > 0.0 ? $this->uf : 1);
|
||||
return $this->valor / (($moneda === Pago::UF) ? $uf : 1);
|
||||
}
|
||||
|
||||
public function jsonSerialize(): mixed
|
||||
|
@ -25,7 +25,7 @@ class Pie extends Model
|
||||
}
|
||||
if ($pagadas) {
|
||||
return array_filter($this->cuotasArray, function(Cuota $cuota) {
|
||||
return $cuota->pago->currentEstado->tipoEstadoPago->descripcion !== 'no pagado';
|
||||
return in_array($cuota->pago->currentEstado->tipoEstadoPago->descripcion, ['depositado', 'abonado']);
|
||||
});
|
||||
}
|
||||
return array_filter($this->cuotasArray, function(Cuota $cuota) {
|
||||
|
@ -136,14 +136,23 @@ class Pago
|
||||
{
|
||||
$pago->estados = $this->estadoPagoRepository->fetchByPago($pago->id);
|
||||
$pago->currentEstado = $this->estadoPagoRepository->fetchCurrentByPago($pago->id);
|
||||
$pago->uf = $this->getUF($pago);
|
||||
return $pago;
|
||||
}
|
||||
protected function getUF(Model\Venta\Pago $pago): ?float
|
||||
{
|
||||
if (($pago->uf === null or $pago->uf === 0.0)
|
||||
and $pago->currentEstado->tipoEstadoPago->descripcion === 'abonado'
|
||||
and $pago->currentEstado->fecha <= new DateTimeImmutable()) {
|
||||
$pago->uf = $this->moneyService->getUF($pago->currentEstado->fecha);
|
||||
if ($pago->uf !== 0.0) {
|
||||
$this->pagoRepository->edit($pago, ['uf' => $pago->uf]);
|
||||
$uf = $this->moneyService->getUF($pago->currentEstado->fecha);
|
||||
if ($uf !== 0.0) {
|
||||
$this->pagoRepository->edit($pago, ['uf' => $uf]);
|
||||
return $uf;
|
||||
}
|
||||
} elseif ($pago->uf === 0.0) {
|
||||
$this->pagoRepository->edit($pago, ['uf' => null]);
|
||||
return null;
|
||||
}
|
||||
return $pago;
|
||||
return $pago->uf;
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user