Skip to content

WIP: Bug Mostrar campo Proyectos en EditAsiento#49

Draft
EstebanSM85 wants to merge 1 commit into
masterfrom
Añadir-campo-poryecto-a-editAsiento

Hidden character warning

The head ref may contain hidden characters: "A\u00f1adir-campo-poryecto-a-editAsiento"
Draft

WIP: Bug Mostrar campo Proyectos en EditAsiento#49
EstebanSM85 wants to merge 1 commit into
masterfrom
Añadir-campo-poryecto-a-editAsiento

Conversation

@EstebanSM85

@EstebanSM85 EstebanSM85 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Arregla el bug que no aparece el campo para vincular proyectos en asientos

  • Se ha añadido la clase EditAsiento para gestionar la edición de asientos.
  • Se ha creado la clase Asiento para manejar la lógica de inserción y actualización de datos.
  • Se ha añadido la vista AccountingEntry.html.twig para la interfaz de usuario de entrada contable.
  • Se ha modificado Init.php para cargar las nuevas extensiones EditAsiento y Asiento.
Captura de pantalla 2026-06-08 141932

Arregla el bug que no aparece el campo para vincular proyectos en asientos
- Se ha añadido la clase EditAsiento para gestionar la edición de asientos.
- Se ha creado la clase Asiento para manejar la lógica de inserción y actualización de datos.
- Se ha añadido la vista AccountingEntry.html.twig para la interfaz de usuario de entrada contable.
- Se ha modificado Init.php para cargar las nuevas extensiones EditAsiento y Asiento.
@NeoRazorX
NeoRazorX marked this pull request as draft July 17, 2026 19:19
@NeoRazorX NeoRazorX changed the title Bug Mostrar campo Proyectos en EditAsiento WIP: Bug Mostrar campo Proyectos en EditAsiento Jul 17, 2026
@NeoRazorX

Copy link
Copy Markdown
Member

Marcado como WIP / draft.

El enfoque actual funciona (se ve el campo y persiste idproyecto), pero depende de copiar literalmente ~190 líneas del template del core (Core/View/Tab/AccountingEntry.html.twig) más manipulación del DOM en restructureHeader(). Eso es frágil y una carga de mantenimiento: cualquier cambio del core en el formulario de asientos no llegaría al plugin y podría romper la copia.

La solución correcta es añadir el punto de extensión necesario en FacturaScripts core (renderizar el campo proyecto o exponer un hook en la cabecera del asiento). Una vez esté en el core, el override del plugin se reduce a unas pocas líneas.

Se deja en espera hasta hacer ese cambio en el core. Cuando esté, se retoma este PR simplificándolo.

Notas menores para cuando se retome:

  • Quitar el use ProjectControllerSalesPurchases; en Extension/Controller/EditAsiento.php: la nueva UI usa un <select> estático y nunca dispara la acción autocomplete-project, por lo que el trait queda sin uso.
  • getProyectos() hace Proyecto::all([], ..., 0, 0) (carga todos los proyectos, sin filtrar por empresa); valorar autocomplete o filtro por idempresa.
  • <option> vacío duplicado y acceso directo a $_POST['data'] en vez de $this->request.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants