Tarea4740#94
Closed
abdedarghal111 wants to merge 12 commits into
Closed
Conversation
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
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.
Se ha modificado fsmaker para que contenga funciones útiles integradas para realizar operaciones con ficheros de manera segura. Esas funciones o "wrappers" informan por salida si algo ha salido mal y además devuelven false si no ha funcionado para parar el proceso de ejecución.
Estas funciones son las que se han implementado en "Utils" siendo
readFile(string $path): string|false,writeFile(string $path, string $content): bool,parseIniFile(string $path): array|false.Todos los demás cambios son la refactorización de los ficheros para que adapten esas modificaciones y corten la ejecución si falla alguno de los procesos.
Estos cambios no deberían de cambiar de ninguna manera fsmaker para el usuario promedio debido a que solo aparecerían errores si no hay permisos para abrir el fichero o no existe o similares.
https://facturascripts.com/roadmap/4740
Tras realizar pruebas también se han encontrado otras fallas en otros puntos y se han solucionado. (Problemas como que detecta que no tiene permisos pero seguía la ejecución porque no tenía condicionales de abortar).
Esas fallas pues han ocasionado el tener que modificar algunas funciones como por ejemplo
FileGeneratorpara que devuelva un booleano en vez de devolver null para así poder parar el flujo.