[BO - Signalement] Correction de la possibilité d'éditer une visite si pas assigné #3106
Conversation
| @@ -29,8 +31,9 @@ public function __construct(private Security $security) | |||
|
|
|||
| protected function supports(string $attribute, $subject): bool | |||
| { | |||
| return \in_array($attribute, [self::EDIT, self::VIEW, self::DELETE, self::VALIDATE, self::CLOSE, self::ADD_VISITE, self::USAGER_EDIT, self::EDIT_NDE]) | |||
| && ($subject instanceof Signalement); | |||
| return (\in_array($attribute, [self::EDIT, self::VIEW, self::DELETE, self::VALIDATE, self::CLOSE, self::ADD_VISITE, self::USAGER_EDIT, self::EDIT_NDE]) | |||
There was a problem hiding this comment.
ça m'embête d'avoir fait un cas particulier pour l'intervention, mais je ne voyais pas trop comment faire autrement, sauf recréer un voter pour les intervention mais on aurait du séparer de ADD_VISITE, et je trouvais ça pire !
Et je crois qu'on a supprimé le voter des intervention y'a quelques temps, non ?
There was a problem hiding this comment.
perso j'aurais fait un voter séparé pour les interventions, ca me choque pas que la add_visite (qui dépend du signalement) soit séparé du edit_visite qui dépend de la visite.
Je trouve que ca devient vite compliqué à lire quand un Voter accepte différent type d'objets mais c'est que mon avis donc je bloque pas sur ça
There was a problem hiding this comment.
Je pense que je suis de l'avis de Numa.
| @@ -30,9 +30,6 @@ | |||
| </div> | |||
|
|
|||
| {% if signalement.interventions is empty %} | |||
| {% if is_granted('SIGN_ADD_VISITE', signalement) %} | |||
There was a problem hiding this comment.
j'ai supprimé ici parce que ça datait d'avant, quand le bouton d'ajout de visite était ajouté à cette hauteur.
| @@ -1,75 +1,72 @@ | |||
|
|
|||
| {% if signalement.interventions is not empty %} | |||
There was a problem hiding this comment.
la suppression de ce test devenu inutile (on teste la même chose juste avant de l'inclure) décale tout, c'est un peu compliqué niveau relecture
Pour lire clairement, on peut voir juste le premier commit, qui consiste simplent à changer l'attribut du voter
There was a problem hiding this comment.
Sinon en cachant les whitespaces ça se relit bien :)
emilschn
force-pushed
the
fix/3100-visites-editor
branch
from
dbb8a7e
to
1ced726
Compare
|
Ticket
#3100
Description
Les personnes qui ont le droit d'éditer (et par conséquent : confirmer, annuler, ajouter/supprimer des docs) une visite sont les suivantes : Super Admin, Admin du territoire du signalement, Utilisateur du partenaire qui est affecté à la visite.
Avant correction, tous les partenaires affectés au signalement et avec la compétence visite pouvaient éditer la visite.
Changements apportés
Tests