Skip to content

refactor devolucion factura#1903

Open
elguitarraverde wants to merge 1 commit intoNeoRazorX:masterfrom
elguitarraverde:refactor-devolucion-factura
Open

refactor devolucion factura#1903
elguitarraverde wants to merge 1 commit intoNeoRazorX:masterfrom
elguitarraverde:refactor-devolucion-factura

Conversation

@elguitarraverde
Copy link
Contributor

@elguitarraverde elguitarraverde commented Mar 6, 2026

Descripción

  • Avanzamos implementando funciones mediante API, pero en algunos casos estamos duplicando código.
  • En este caso he refactorizado la creación de una factura rectificativa, creando una sola clase y usándola tanto en el controlador principial, como en el controlador de la API.
  • Como la tendencia en php es el tipado estricto he creado una clase DAO(RefundInvoiceParams) para evitar pasar arrays como parámetro difícilmente "tipables".
  • De este modo también podemos testear mejor la creación de una devolución de factura.

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant