Skip to content

feat: añadir funcionalidad de comisiones generadas#33

Merged
daniel89fg merged 4 commits into
mainfrom
Tarea-4675-añadir-generacion-comisiones
May 22, 2026

Hidden character warning

The head ref may contain hidden characters: "Tarea-4675-a\u00f1adir-generacion-comisiones"
Merged

feat: añadir funcionalidad de comisiones generadas#33
daniel89fg merged 4 commits into
mainfrom
Tarea-4675-añadir-generacion-comisiones

Conversation

@EstebanSM85
Copy link
Copy Markdown
Contributor

https://facturascripts.com/roadmap/4675

  • Se ha añadido un nuevo archivo Comisiones.php que permite la creación de comisiones generadas utilizando Faker.
  • Se ha añadido un nuevo archivo Randomizer.php que incluye un controlador para cargar botones relacionados con las comisiones generadas.
  • Se ha actualizado el archivo es_ES.json para incluir la traducción de "Comisiones generadas: %quantity%".
  • Se ha modificado Init.php para cargar la extensión Randomizer si está disponible. todo tal y como lo hace el pluging poryectos

https://facturascripts.com/roadmap/4675
- Se ha añadido un nuevo archivo Comisiones.php que permite la creación de comisiones generadas utilizando Faker.
- Se ha añadido un nuevo archivo Randomizer.php que incluye un controlador para cargar botones relacionados con las comisiones generadas.
- Se ha actualizado el archivo es_ES.json para incluir la traducción de "Comisiones generadas: %quantity%".
- Se ha modificado Init.php para cargar la extensión Randomizer si está disponible.
todo tal y como lo hace el pluging poryectos
Se ha modificado el nombre del botón de 'plugins' a 'sales' en la función loadButtons() del archivo Randomizer.php.
Esto mejora la claridad y precisión en la interfaz de usuario.
- Se actualizó el año de copyright a 2026.
- Se cambió el autor de Carlos Garcia Gomez a Esteban Sánchez Martínez.
Se ha modificado el archivo Init.php para actualizar el año de copyright de 2025 a 2026.
@Yopli2k
Copy link
Copy Markdown
Collaborator

Yopli2k commented May 21, 2026

Veo que se usa fakerphp/faker pero el plugin no tiene el archivo de composer. Entiendo que lo debe tener el Randomizer, porque sino dará error la nueva clase Comisiones.php. Por otra parte normalmente los nombres de clase van en singular. Pero a ver que indica Carlos. Buen trabajo, gracias.

@EstebanSM85
Copy link
Copy Markdown
Contributor Author

Veo que se usa fakerphp/faker pero el plugin no tiene el archivo de composer. Entiendo que lo debe tener el Randomizer, porque sino dará error la nueva clase Comisiones.php. Por otra parte normalmente los nombres de clase van en singular. Pero a ver que indica Carlos. Buen trabajo, gracias.

Justo no lo necesita porque lo lleva Randomizer, Los modelos si son en singular, pero las clases de Randomizer no, si miras en Lib/Random/ veras que todas van en plural, incluso en plugin Proyectos (En el que me he basado para hacer este) también lleva la clase en plural.
Pero muchas gracias por el feedback, estoy aprendiendo y todo es muy bien recibido.

@Yopli2k
Copy link
Copy Markdown
Collaborator

Yopli2k commented May 21, 2026

Gracias a ti, no es un tema de modelo o clase particular, según el PSR-12 no debería ser así, pero entiendo lo que dices. Por eso me remití a que la última palabra la tenía Carlos.

@daniel89fg daniel89fg merged commit dadbe1c into main May 22, 2026
8 of 10 checks passed
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.

3 participants