Tarea 3783 bug asientos filtro canales vacio#1985
Merged
NeoRazorX merged 6 commits intoJun 29, 2026
Merged
Conversation
…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.
…acías https://facturascripts.com/roadmap/3783 - Se agrega la traducción "Sin canal" en es_ES.json. - Se modifica SelectFilter.php para manejar el valor '__null__' en los filtros. - Se mejora la generación de opciones en el filtro para mostrar un texto adecuado cuando no hay canal disponible.
- Se ha creado un nuevo archivo .user.ini con la configuración de memory_limit establecida en 1024M. - Se ha eliminado la referencia a .user.ini del archivo .gitignore.
- Se ha removido la lógica que manejaba el valor '__null__' en el método getDataBaseWhere. - Se ha simplificado la generación de opciones en el método correspondiente, eliminando el manejo de opciones vacías.
- Modifica ListAsiento.php para incluir un addfilterwherefilter que permite seleccionar canales. - Se añade la opción "sin canal" para manejar casos donde el canal es nulo. - Mejora la gestión de opciones de filtro para una mejor experiencia de usuario.
- Modifica la forma en que se añaden botones a las vistas utilizando el método tab()->addButton(). - Reemplaza instancias de DataBaseWhere por la nueva clase Where para la gestión de condiciones en los filtros. - Mejora la legibilidad del código al utilizar métodos más claros para las condiciones de los filtros. Estos cambios optimizan la estructura del código y facilitan futuras modificaciones.
Owner
|
Revisado y mergeado ✅ Esta es la solución completa al filtro de canal en el listado de asientos: reemplaza el `addFilterSelect` por un `addFilterSelectWhere` que lista cada canal y añade además la opción Sin canal (`Where::isNull(\x27canal\x27)`), usando `codeModel->all(..., false)` para incluir los nulos — que es justo lo que el select anterior ocultaba. Nota: el PR incluye refactor de alcance extra (migración `DataBaseWhere`→`Core\Where`, `addButton`→`tab()->addButton`, año de copyright). Es razonable, lo dejamos pasar. Queda como duplicado del #1979, que cierro al estar cubierto por este. Gracias 🙌 |
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://facturascripts.com/roadmap/3783