Ajout outil de réservation de salles (nouvelle MDE)

[martinni]

Ceci est une pull request pour ajouter le travail fait par Nina Martin (martinni), Anthony Legiret (leganto) et Melissa Benali (mel-BR) dans le cadre de la TX suivie par Harry Claisse et Lucas Brondy, pour permettre aux étudiants de réserver les salles de la nouvelle MDE via le portail des assos.

Nous avons travaillé sur les apps frontend et bde. cote frontend -> ajout d'un calendrier de réservation sur le modèle du calendrier des évènements, formulaire d'ajout/edition/supression de reservation cote backend -> ajout d'une section permettant aux responsables BDE de gérer les réservations

[MiLk]

De maniere generale, le style des identations n'est pas consistant. Veillez a suivre les conventions du projet. https://github.com/simde-utc/portail/wiki/Convention-et-r%C3%A8gles

[martinni]

Nous avons committé les modifications demandées. Tenez nous au courant si c'est ok pour vous.

[brcorentin]

Bon ça à l'air ok. Il y a juste un point qui me chiffonne c'est l'indentation elle est assez anarchique essayer d'aligner les instructions dans vos blocs et de rajouter une indentation à chaque création de bloc car cela améliore grandement la lisibilité.

De mon côté je vais tester tout ça après-demain et si tout fonctionne et que vous avez modifié les indentations, je mergerais.

Je sais que je suis un peu chiant mais un code mal indenté est assez dur à lire et si un jour on a besoin de modifier le code il faut que l'on puisse le lire et le comprendre rapidement.

A part ça beau boulot tout de même :-)

PS: Je sens que @jofroid meurt aussi d'envie de tester le code ^^

[jofroid]

Allé, demain je teste le code moi aussi, les pieds dans la piscine :-) Le 8 août 2014 23:50, "Corentin" notifications@github.com a écrit :

Bon ça à l'air ok. Il y a juste un point qui me chiffonne c'est l'indentation elle est assez anarchique essayer d'aligner les instructions dans vos blocs et de rajouter une indentation à chaque création de bloc car cela améliore grandement la lisibilité.

De mon côté je vais tester tout ça après-demain et si tout fonctionne et que vous avez modifié les indentations, je mergerais.

Je sais que je suis un peu chiant mais un code mal indenté est assez dur à lire et si un jour on a besoin de modifier le code il faut que l'on puisse le lire et le comprendre rapidement.

A part ça beau boulot tout de même :-)

PS: Je sens que @jofroid https://github.com/jofroid meurt aussi d'envie de tester le code ^^

— Reply to this email directly or view it on GitHub https://github.com/simde-utc/portail/pull/167#issuecomment-51661867.

[martinni]

Je vais me pencher sur les indentations cet aprem, j'ai des tendances un peu bordéliques donc je m'étais pas rendue compte, mais je comprend :)

Je vous tiens au courant

Le 8 août 2014 23:52, jofroid notifications@github.com a écrit :

Allé, demain je teste le code moi aussi, les pieds dans la piscine :-) Le 8 août 2014 23:50, "Corentin" notifications@github.com a écrit :

Bon ça à l'air ok. Il y a juste un point qui me chiffonne c'est l'indentation elle est assez anarchique essayer d'aligner les instructions dans vos blocs et de rajouter une indentation à chaque création de bloc car cela améliore grandement la lisibilité.

De mon côté je vais tester tout ça après-demain et si tout fonctionne et que vous avez modifié les indentations, je mergerais.

Je sais que je suis un peu chiant mais un code mal indenté est assez dur à lire et si un jour on a besoin de modifier le code il faut que l'on puisse le lire et le comprendre rapidement.

A part ça beau boulot tout de même :-)

PS: Je sens que @jofroid https://github.com/jofroid meurt aussi d'envie de tester le code ^^

— Reply to this email directly or view it on GitHub https://github.com/simde-utc/portail/pull/167#issuecomment-51661867.

— Reply to this email directly or view it on GitHub https://github.com/simde-utc/portail/pull/167#issuecomment-51662025.

Nina MARTIN +33652182717

Etudiante Ingénieure Génie Informatique - Université de Technologie de Compiegne Software Engineering Student - University of Technology of Compiègne

[martinni]

J'ai fait un peu cleanup sur l'indentation des principaux fichiers PHP, c'est pushé

Le 9 août 2014 13:29, Nina Martin nina.utc@gmail.com a écrit :

Je vais me pencher sur les indentations cet aprem, j'ai des tendances un peu bordéliques donc je m'étais pas rendue compte, mais je comprend :)

Je vous tiens au courant

Le 8 août 2014 23:52, jofroid notifications@github.com a écrit :

Allé, demain je teste le code moi aussi, les pieds dans la piscine :-)

Le 8 août 2014 23:50, "Corentin" notifications@github.com a écrit :

Bon ça à l'air ok. Il y a juste un point qui me chiffonne c'est l'indentation elle est assez anarchique essayer d'aligner les instructions dans vos blocs et de rajouter une indentation à chaque création de bloc car cela améliore grandement la lisibilité.

De mon côté je vais tester tout ça après-demain et si tout fonctionne et que vous avez modifié les indentations, je mergerais.

Je sais que je suis un peu chiant mais un code mal indenté est assez dur à lire et si un jour on a besoin de modifier le code il faut que l'on puisse le lire et le comprendre rapidement.

A part ça beau boulot tout de même :-)

PS: Je sens que @jofroid https://github.com/jofroid meurt aussi d'envie de tester le code ^^

— Reply to this email directly or view it on GitHub https://github.com/simde-utc/portail/pull/167#issuecomment-51661867.

— Reply to this email directly or view it on GitHub https://github.com/simde-utc/portail/pull/167#issuecomment-51662025.

Nina MARTIN +33652182717

Etudiante Ingénieure Génie Informatique - Université de Technologie de Compiegne Software Engineering Student - University of Technology of Compiègne

Nina MARTIN +33652182717

Etudiante Ingénieure Génie Informatique - Université de Technologie de Compiegne Software Engineering Student - University of Technology of Compiègne

[brcorentin]

Merci c'est beaucoup mieux je suis actuellement en train de tester.

[brcorentin]

Bon j'ai un bug de migration vu qu'il n'y a pas de fichier migration j'ai essaye de la faire moi même: $ php symfony doctrine:generate-migrations-diff $ git pull https://github.com/martinni/portail.git $ php symfony doctrine:generate-migrations-diff $ php symfony doctrine:build --all-classes --and-migrate Dites moi si je fais une erreur dans l'exécution des commandes.

ce qui est encore plus bizarre c'est que "php symfony doctrine:generate-migrations-diff" fait une erreur de segmentation. Ca m'énerve je m'arrete pour ce soir. Je réinstallerais tout le portail demain pour m'assurer qu'il n'y ai pas de probleme.

@jofroid tu a réussit à tout faire fonctionner toi?

[brcorentin]

J'ai parfois une erreur de segmentation lors de la migration .... Mais ce coup-ci ça a marché ....

Ca derange quelqu'un si je fais une PR pour envoyer les fichier de migrations après avoir testé le module. ( @MiLk @apuyou )

@jofroid : Pour installer le module tu pull la branche de la PR et tu fais $ php symfony doctrine:generate-migrations-diff $ php symfony doctrine:build --all-classes --and-migrate Dis moi si ça marche chez toi.

[MiLk]

Va pour une autre PR pour les migrations.

[jofroid]

Après la première instruction, j'ai un message erreur : "Could not generate migration classes from difference". Donc non ça ne marche pas pour le moment pour moi.

Geoffroy Dietsch

GI02 - Université de Technologie de Compiègne

2014-08-12 17:24 GMT+02:00 Emilien Kenler notifications@github.com:

Va pour une autre PR pour les migrations.

— Reply to this email directly or view it on GitHub https://github.com/simde-utc/portail/pull/167#issuecomment-51929934.

[brcorentin]

@jofroid ok je vous pusherez les fichiers de migrations quand j'aurais merge ça.

@martinni Peux tu m'expliquer vite fait le fonctionnement des requetes sur le modules? Voila ce que j'en ai compris. -1 membre du bde attribue une salle comme disponible. -Un membre du bureau d'une asso réserve la salle -Ici il y a une étape que je ne comprends pas je bloque ici. -Le bde valide la demande. -Le calendrier est mis à jour.

Peux tu apporter quelques précisions sur le système?

[martinni]

Je me suis penchée sur le pb de la migration. En fait je pense que ce n'est pas nécessaire de faire cette commande. En effet, doctrine:generate-migrations-diff se charge de verifier que les fichiers de classes dans le dossier lib/model/doctrine contiennent bien tous les element du fichier config/doctrine/schema.yml.

Or c'est déja le cas puisque toutes les classes du modele correspondant aux nouvelles tables (Reservation et Salle) sont déja présentes dans le code que vous avez pullé (on les avait générées nous meme). Du coup il n'y a pas de différence a générer, d'ou le : "Could not generate migration classes from difference"

En fait la seule chose a faire est de reconstruire la bdd a partir du schéma, et c'est la commande doctrine:build --all qui s'en occupe. Et ensuite insérer les données du dump bien sur (notre dump s'apelle _dump1404840262.sql)

Enfin en tout cas c'est ce que je comprend pour la migration.

Pour le fonctionnement du module, je reprend tes étapes une par une :

-1 membre du bde attribue une salle comme disponible. Oui, plus précisément il peut faire toutes les opérations CRUD sur les salles et les attribue a un pole

-Un membre du bureau d'une asso réserve la salle Oui, c'est la que les poles sont importants car un utilisateur ne voit que les salles des poles auxquels il appartient (on appartient a un pole quand on fait partie d'une asso du dit pole, ex : si je suis a UTCoupe et aucune autre asso je ne vais voir que les salles du PTE). Exception pour les salles catégorisées dans le pole BDE qui sont réservables par tout le monde y compris des personnes dans aucune asso. (Je précise tout ca pour que vous ne soyez pas surpris lors de vos tests)

L'utilisateur doit sélectionner une salle puis cliquer sur un horaire libre sur le calendrier et un formulaire de réservation s'affiche.

-Ici il y a une étape que je ne comprends pas je bloque ici. Lorsque la demande est envoyée, la réservation est enregistrée dans la tables des réservations, on peut alors etre face a 2 cas de figures :

• soit la réservation nécessite une validation (cas fixés avec le BDE, par exemple une réservation sur toute une journée ou plus de 2 semaines a l'avance), dans ce cas le champ estvalide de la table Reservation est initialisé a faux et c'est une validation ultérieure du BDE qui mettra le champ a vrai
• soit la réservation ne nécessite pas de validation et le champ est estvalide est initialisé a vrai

-Le bde valide la demande. Oui, si cela est nécessaire. Il peut aussi choisir de modifier ou supprimer toute réservation existante, déja valide ou non. Une telle action entrainera l'envoi d'un mail au créateur de la réservation avec un message d'explication entré par le membre du BDE.

-Le calendrier est mis à jour. Oui. En fait toute réservation, validée ou non, apparait sur le calendrier des que la demande a été faite. Par contre, les réservations pas encore validées apparaissent en transparent.

J'ai mis le rapport de la TX en piece jointe. Ca sera surement plus clair avec les screenshots et tout. Mais bien sur si vous avez d'autres questions n'hésitez pas.

Le 12 août 2014 20:03, Corentin notifications@github.com a écrit :

@jofroid https://github.com/jofroid ok je vous pusherez les fichiers de migrations quand j'aurais merge ça.

@martinni https://github.com/martinni Peux tu m'expliquer vite fait le fonctionnement des requetes sur le modules? Voila ce que j'en ai compris. -1 membre du bde attribue une salle comme disponible. -Un membre du bureau d'une asso réserve la salle -Ici il y a une étape que je ne comprends pas je bloque ici. -Le bde valide la demande. -Le calendrier est mis à jour.

Peux tu apporter quelques précisions sur le système?

— Reply to this email directly or view it on GitHub https://github.com/simde-utc/portail/pull/167#issuecomment-51952495.

Nina MARTIN +33652182717

Etudiante Ingénieure Génie Informatique - Université de Technologie de Compiegne Software Engineering Student - University of Technology of Compiègne

[brcorentin]

Ok très bien cela fonctionne.

@martinni Juste une question: Comment fait-on pour crée une salle pour laquelle il faut que le bde valide la reservation?

Pour la migration: Le but de la migration est de faire évoluer la base de données sans l'altérer. Il est en effet possible de créer la base de données comme tu le dis mais le problème est qu'il sera impossible de mettre le module en prod. En effet, pour la mise en production il est nécessaire de ne pas altérer les données déjà existante. Pour cela il faut effectuer une migration.

Ne t'inquiète pas pour ça j'ai réussi à générer les fichiers de migrations et je les mettrais sur le portail donc il n'y a aucun problème.

Quand tu auras répondu à ma question je mergerais la PR. Encore une fois beau travail.

PS: On a pas reçu le rapport de la TX peut tu l'envoyé à l'adresse email du SiMDE?

[martinni]

Ah oui effectivement je n'avais pas pensé aux données de prod. Bon si vous avez pu vous en sortir quand même c'est cool !

Pour répondre à ta question, en fait la nécessité de valider ou non ne dépend pas de la salle mais de la réservation. Le BDE doit faire une validation dans 2 cas :

• si la salle est réservée plus de 2 semaines à l'avance
• si l'utilisateur a réservé la salle pour une journée entière (option cochable dans le formulaire)

Le 13 août 2014 19:23, Corentin notifications@github.com a écrit :

Ok très bien cela fonctionne.

@martinni https://github.com/martinni Juste une question: Comment fait-on pour crée une salle pour laquelle il faut que le bde valide la reservation?

Pour la migration: Le but de la migration est de faire évoluer la base de données sans l'altérer. Il est en effet possible de créer la base de données comme tu le dis mais le problème est qu'il sera impossible de mettre le module en prod. En effet, pour la mise en production il est nécessaire de ne pas altérer les données déjà existante. Pour cela il faut effectuer une migration.

Ne t'inquiète pas pour ça j'ai réussi à générer les fichiers de migrations et je les mettrais sur le portail donc il n'y a aucun problème.

Quand tu auras répondu à ma question je mergerais la PR. Encore une fois beau travail.

— Reply to this email directly or view it on GitHub https://github.com/simde-utc/portail/pull/167#issuecomment-52080831.

Nina MARTIN +33652182717

Etudiante Ingénieure Génie Informatique - Université de Technologie de Compiegne Software Engineering Student - University of Technology of Compiègne

[brcorentin]

Ok très bien je merge