Leaders can only delete events that have not started

+ update descriptions

Author: gdaviet

collectives/models/event/role.py

@@ -6,6 +6,7 @@
 from collectives.models.event.event_type import EventType
 from collectives.models.user import User
 from collectives.models.registration import RegistrationLevels
+from collectives.utils.time import current_time
 
 
 class EventRoleMixin:
@@ -84,12 +85,17 @@ def has_edit_rights(self, user):
     def has_delete_rights(self, user: User) -> bool:
         """Check if a user can delete this event.
 
-        Equivalent to :func:`has_edit_rights`
+        For events in the future, equivalent to :func:`has_edit_rights`
+        For past events, needs supervisor or moderator rights
 
         :param user: User which will be tested.
         :return: True if user can delete the event.
         """
-        return self.has_edit_rights(user)
+
+        if self.start > current_time():
+            return self.has_edit_rights(user)
+
+        return user.is_moderator() or self.is_supervisor(user)
 
     def can_remove_leader(self, user, leader):
         """

collectives/models/role.py

@@ -71,7 +71,7 @@ def display_names(cls) -> Dict["RoleIds", str]:
 
     @classmethod
     def descriptions(cls) -> Dict["RoleIds", str]:
-        """Display names for all rolesrole
+        """Description for all roles
 
         :return: dictionnary role -> role name
         """
@@ -89,10 +89,11 @@ def descriptions(cls) -> Dict["RoleIds", str]:
             "nécéssitant pas d'autorisation d'encadrement (soirées ...), mais ne peut "
             "pas créer de collectives.",
             cls.EventLeader: "Un encadrant peut créer et encadrer tout type d'événement lié à "
-            "l'activité, dont des collectives.",
-            cls.ActivitySupervisor: "Un responsable d'activité peut proposer un évènement dans son "
-            "activité, gérer les encadrants et co encadrants de son "
-            "activité, et supprimer des évènements.",
+            "l'activité, dont des collectives. Il peut supprimer ses événements à condition"
+            "qu'aucun paiement n'ait été effectué et que l'événement n'ait pas débuté.",
+            cls.ActivitySupervisor: "Un responsable d'activité peut proposer un événement dans son "
+            "activité, gérer les rôles (encadrement, ...), badges (bénévolat, ...) et documents "
+            "liés à son activité, et a pouvoir de modération sur tout événement de l'activité.",
             cls.Trainee: "Un encadrant en formation peut être noté comme co-encadrant d'un "
             "événement auquel il est inscrit. Il peut aussi créer certains "
             "types d'événements ne nécéssitant pas d'autorisation d'encadrement "