Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accès aux jours fériés via l'API #39

Merged
merged 7 commits into from
May 21, 2018
Merged

Accès aux jours fériés via l'API #39

merged 7 commits into from
May 21, 2018

Conversation

prytoegrian
Copy link
Member

@prytoegrian prytoegrian commented May 15, 2018

Nouveau jour, nouvelle table portée sur l'API. Rien de difficile, il nous suffit de copier un package et de changer les informations qui s'y trouvent, le reste suit naturellement. Cependant, comme je m'y attendais, nous découvrons chaque fois des nouvelles choses qui nous font préciser l'ébauche de design que nous avions commencé à mettre en place. En effet, jusque-là un package métier était constitué du contrôleur, de l'entité, de la DAO et du repo. Oui mais voilà, une entité se singularise par son identité, là où un jour férié n'a pas en soit d'identité ; ce serait plutôt un value object.
Puisque ce n'était pas le sujet de la PR, je suis allé au plus simple, j'ai créé une entité JourFerieEntite. Je suppose qu'il va être temps d'utiliser un framework d'injection de dépendance pour éviter de faire des choses génériques (ce que nous avons fait jusque-là), sans pour autant s'ennuyer à traîner des tables de configuration et d'association. J'ai utilisé dans mon précédent boulot https://github.com/PHP-DI/PHP-DI ; je l'aime bien, il fait le job et je n'ai pas encore rencontré de défaut.
Les dernières PR remontent aussi un besoin de retravailler de design, je propose donc qu'on se garde cette idée dans un coin de notre tête pour le moment où nous nous y attellerons.

Test

Comme d'hab, il n'y a pas de difficulté. Les jours fériés sont censés être lisible par les HautResponsables, pas les autres. Il n'y a pas de moyen de voir un élément unique.

@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 15, 2018
@libertempo libertempo deleted a comment May 21, 2018
@libertempo libertempo deleted a comment May 21, 2018
@libertempo libertempo deleted a comment May 21, 2018
@libertempo libertempo deleted a comment May 21, 2018
@prytoegrian prytoegrian merged commit 4b85f36 into master May 21, 2018
@prytoegrian prytoegrian deleted the pry/ferie branch May 21, 2018 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant