Skip to content

Tarea 3586 buscar referencia proveedor#1982

Open
EstebanSM85 wants to merge 3 commits into
NeoRazorX:masterfrom
EstebanSM85:Tarea-3586-buscar-referencia-proveedor
Open

Tarea 3586 buscar referencia proveedor#1982
EstebanSM85 wants to merge 3 commits into
NeoRazorX:masterfrom
EstebanSM85:Tarea-3586-buscar-referencia-proveedor

Conversation

@EstebanSM85

Copy link
Copy Markdown

https://facturascripts.com/roadmap/3586

  • Modificado PurchaseDocument.php para incluir la búsqueda de productos por referencia del proveedor si no se encuentra por referencia o código de barras.
  • Actualizado PurchasesController.php para permitir la búsqueda de referencias de proveedor en la lista de productos.

Estos cambios mejoran la funcionalidad de búsqueda de productos, facilitando la localización de artículos a partir de referencias de proveedor.

…digo postal y provincia

- Modificado ServicioAT.php para mejorar la lógica de guardado de datos de dirección.
- Añadido archivo EditServicioAT.xml para la configuración de la vista de edición.
- Creado ListServicioAT.php con la función para crear vistas de lista.
- Añadido archivo ListServicioAT.xml para la configuración de la vista de lista.
- Añadido archivo serviciosat.xml para definir la estructura de la tabla de datos.
…chasesController

https://facturascripts.com/roadmap/3586
- Modificado PurchaseDocument.php para incluir la búsqueda de productos por referencia del proveedor si no se encuentra por referencia o código de barras.
- Actualizado PurchasesController.php para permitir la búsqueda de referencias de proveedor en la lista de productos.

Estos cambios mejoran la funcionalidad de búsqueda de productos, facilitando la localización de artículos a partir de referencias de proveedor.
…chasesController

https://facturascripts.com/roadmap/3586
- Modificado PurchaseDocument.php para incluir la búsqueda de productos por referencia del proveedor si no se encuentra por referencia o código de barras.
- Actualizado PurchasesController.php para permitir la búsqueda de referencias de proveedor en la lista de productos.

Estos cambios mejoran la funcionalidad de búsqueda de productos, facilitando la localización de artículos a partir de referencias de proveedor.
@NeoRazorX

Copy link
Copy Markdown
Owner

Gracias por el PR. La funcionalidad está bien, pero hay que limitar el alcance del cambio: no se debería tocar Core/Model/Base/PurchaseDocument.php para esta tarea. Esa es una clase base del modelo de documentos de compra y la búsqueda por referencia de proveedor no pertenece ahí.

Por favor, deja los cambios circunscritos a Core/Lib/AjaxForms/ (en PurchasesController), que es donde corresponde la lógica del autocompletado/búsqueda. Revierte la modificación de getNewProductLine() en PurchaseDocument.php.

Un par de cosas a tener en cuenta al rehacerlo en el controlador:

  • Filtrar por proveedor: el autocompletado busca actualmente coincidencias de refproveedor de todos los proveedores. En un documento de un proveedor concreto deberían acotarse a codproveedor, para no sugerir referencias de otros proveedores.
  • Evitar el N+1: el bucle hace loadWhere + getProducto() + description() por cada ProductoProveedor en cada pulsación de tecla. Mejor resolverlo con una sola consulta con JOIN entre productosprov, variantes y productos, filtrando ya por bloqueado/secompra.

Cuando lo tengas circunscrito a Core/Lib/AjaxForms/ lo revisamos de nuevo.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants