DGNum/gestioCOF

Fork 1

Kerl/clean code amend #439

Merged

mpepin merged 3 commits from Kerl/clean_code_amend into Aufinal/clean_code

2016-12-21 14:13:21 +01:00

mpepin commented

2016-12-21 02:23:27 +01:00

(Migrated from git.eleves.ens.fr)

Ajoute quelques commentaires

Reformate le texte en 80 colonnes

Améliore l'algo d'inscription à la revente en inscrivant les gens à la revente ayant le moins d'inscrits

Ajoute quelques commentaires Reformate le texte en 80 colonnes Améliore l'algo d'inscription à la revente en inscrivant les gens à la revente ayant le moins d'inscrits

mpepin commented

2016-12-21 02:28:13 +01:00

(Migrated from git.eleves.ens.fr)

mentioned in merge request !114

mpepin commented

2016-12-21 02:34:02 +01:00

(Migrated from git.eleves.ens.fr)

Dans l'ancienne version du code, on essaie d'inscrire les gens à une revente disponible (le else) même si des places au shotgun existaient. À
l'époque il était théoriquement impossible d'avoir les deux en même temps mais désormais c'est possible, il m'a donc semblé légitime de séparer complètement les deux cas.

Je le signale pour que cette différence n'échappe pas à la review, confirme moi que c'est correct :)

Dans l'ancienne version du code, on essaie d'inscrire les gens à une revente disponible (le `else`) même si des places au shotgun existaient. À l'époque il était théoriquement impossible d'avoir les deux en même temps mais désormais c'est possible, il m'a donc semblé légitime de séparer complètement les deux cas. Je le signale pour que cette différence n'échappe pas à la review, confirme moi que c'est correct :)

lstephan commented

2016-12-21 03:06:39 +01:00

(Migrated from git.eleves.ens.fr)

Heureusement, pour que ça se produise il faudrait une invraisemblable conjonction d'offre et de demande, donc ce n'est jamais arrivé ; mais c'était une erreur =)

Nope, dans l'ancienne version du code il était tout aussi possible d'avoir des places au shotgun + une revente en cours ; et apparemment j'inscrivais le participant a une revente même s'il y avait des places au shotgun. Heureusement, pour que ça se produise il faudrait une invraisemblable conjonction d'offre et de demande, donc ce n'est jamais arrivé ; mais c'était une erreur =)

lstephan commented

2016-12-21 03:08:15 +01:00

(Migrated from git.eleves.ens.fr)

Ca manque d'un message pour le prévenir quand même qu'il a été inscrit ; mais je vais faire ça sur la branche principale.

mpepin commented

2016-12-21 03:10:22 +01:00

(Migrated from git.eleves.ens.fr)

T'es sûr ? Il y a déjà le message de succès en vert en haut de l'écran

lstephan commented

2016-12-21 03:37:00 +01:00

(Migrated from git.eleves.ens.fr)

Il te dit uniquement que tu t'es inscrit pour recevoir les mails des prochaines reventes, pas que tu t'es fait inscrire à une revente en cours, si ?

mpepin commented

2016-12-21 03:38:00 +01:00

(Migrated from git.eleves.ens.fr)

Hum, right

lstephan commented

2016-12-21 04:07:32 +01:00

(Migrated from git.eleves.ens.fr)

Il va falloir rebase, mais je suis OK pour merge.

mpepin commented

2016-12-21 11:31:29 +01:00

(Migrated from git.eleves.ens.fr)

Added 11 commits:

28df0355...2f70dbf9 - 10 commits from branch Aufinal/clean_code
847345bb - Merge branch 'Aufinal/clean_code' into Kerl/clean_code_amend

Added 11 commits: * 28df0355...2f70dbf9 - 10 commits from branch `Aufinal/clean_code` * 847345bb - Merge branch 'Aufinal/clean_code' into Kerl/clean_code_amend

lstephan commented

2016-12-21 14:13:21 +01:00

(Migrated from git.eleves.ens.fr)

Status changed to merged

lstephan commented

2016-12-21 14:13:21 +01:00

(Migrated from git.eleves.ens.fr)

mentioned in commit acfcddce07

mentioned in commit acfcddce078466244a84c7b4771af6581c0a7c6d