Improved targetting list for actions

mens-sana commented 8 years ago

J'ai amélioré la liste des cibles apparaissant pour une action. Par exemple, je trouvais ça dommage qu'un innocent puisse se cibler lui même avec son pouvoir, et que ça ne marche pas au moment de valider. Et puis je trouvais illogique que les mafieux puissent se voter entre eux pendant la nuit (puisqu'il n'y a pas de "loup blanc" et que tous les mafieux gagnent ensemble). Il faut le correctif de bug Openparty pour que ça marche.

En gros je passe à getPlayerSelectOptions un paramètre contenant les cibles valides. Ca contient 3 champs : alive, innocent, self.

Alive et innocent peuvent prendre 3 valeurs : yes, no, any. Yes ou no imposent de ne cibler que les vivants / que les morts ou que les innocents / que les mafieux. Any pour aucun critère sur ce champ. Self prend uniquement les valeurs yes ou no.

Bref pour résumer la PR :

Les habitants ne peuvent pas se voter eux mêmes le jour.
Les innocents qui ont un pouvoir ne peuvent pas se cibler eux mêmes (sauf le secouriste)
Pendant la phase mafia, les mafieux / parrains ne peuvent pas se voter entre eux, ni eux mêmes.

Remarque : pour l'instant les mafieux peuvent toujours éliminer le terroriste pendant la nuit. Je me suis longuement demandé s'ils avaient le droit et je préférais attendre ton avis :) La question se reposera si on ajoute d'autres rôles exotiques au camp mafia.

Remarque 2 : pour l'instant le champ alive est toujours à yes pour toutes les actions. Mais j'ai pensé que ce serait utile si on implémente un chaman qui doit cibler un mort pour lui parler (rôle non codé qu'on a joué hier).

Bien sûr si tu veux changer le paramétrage d'une action ce sera facile :) Les 2 commits en trop c'est pour les erreurs de forme :p

Lesterpig commented 8 years ago

Merci, je relis ça :)

Lesterpig commented 8 years ago

Bon travail, j'ai laissé quelques remarques. Tu peux faire un commit additionnel pour corriger tout ça, je ferai le nécessaire pour rendre l'historique Git propre :)

Merci !

mens-sana commented 8 years ago

Voilà ^^

Lesterpig commented 8 years ago

Je m'occupe de ça dans la matinée, j'enlèverai les commits relatifs au gambler au passage ;)

mens-sana commented 8 years ago

Désolé j'ai voulu créer une branch et j'ai push sur mon master à la place :p

Du coup j'ai revert, ma branch master contient uniquement les modif que tu m'as demandé..

Lesterpig commented 8 years ago

Ajouté dans master :) Merci !

$ git checkout mens-sana-master
$ git rebase -i master
# Pick the first, fixup others
$ git checkout master
$ git merge --no-ff mens-sana-master
$ git push

Lesterpig / openparty-mafia

Improved targetting list for actions #42