Remove the meeting_user when removing a user from a meeting (#3206)

* Require groups in meeting users, fix most tests

* Make user.update delete meeting_users

* Make changes for motions

* Make changes for chat messages

* Make changes for personal note

* Update docs

* Write and test migration

* motion_supporter test code

* Add test for issue 3133

Author: luisa-beerboom

data/example-data.json

@@ -1,5 +1,5 @@
 {
-    "_migration_index": 74,
+    "_migration_index": 75,
     "gender":{
         "1":{
             "id": 1,

data/initial-data.json

@@ -1,5 +1,5 @@
 {
-    "_migration_index": 74,
+    "_migration_index": 75,
     "gender":{
         "1":{
             "id": 1,

docs/actions/chat_message.create.md

@@ -14,4 +14,5 @@ Creates a new `chat_message` for the `chat_group` given by the key `chat_group_i
 
 ## Permission
 
-Every user, who is in one of the write groups of a chat group, or has the permission `chat.can_manage` can create a `chat_message`.
+Every user, who is in one of the write groups of a chat group, or has the permission `chat.can_manage` can create a `chat_message`.
+User also needs to be part of the meeting.

docs/actions/motion.create.md

@@ -26,7 +26,7 @@
 
 // Optional special fields, see notes below
     workflow_id: Id;
-    submitter_ids: Id[];
+    submitter_meeting_user_ids: Id[];
 
 // Non-model fields for customizing the agenda item creation, optional
     agenda_create: boolean;
@@ -58,7 +58,7 @@ This is the logic for other fields depending on the motion type:
 There are some fields that need special attention:
 - `workflow_id`: If it is given, the motion's state is set to the workflow's first state. The workflow must be from the same meeting. If the field is not given, one of the three default (`meeting/motions_default_workflow_id` or `meeting/motions_default_amendment_workflow_id`) workflows is used depending on the type of the motion to create.
 - `additional_submitter` a text field where text-based submitter information may be entered. Cannot be set unless `meeting/motions_create_enable_additional_submitter_text` is `true`. Requires permissions `Motion.CAN_CREATE` and `Motion.CAN_MANAGE_METADATA`.
-- `submitter_ids`: These are **user ids** and not ids of the `motion_submitter` model. If nothing is given (`[]`) and the field `additional_submitter` isn't filled, the request user's id is used. For each id in the list a `motion_submitter` model is created. The weight must be set to the order of the given list. Requires permissions `Motion.CAN_CREATE`, `Motion.CAN_MANAGE_METADATA` and `User.CAN_SEE`.
+- `submitter_meeting_user_ids`: These are ids of the meeting users that should get a `motion_submitter` model. Can be left empty. The weight of the new submitters is set to the order of the given list. Requires permissions `Motion.CAN_CREATE`, `Motion.CAN_MANAGE_METADATA` and `User.CAN_SEE` (the latter two only if not setting oneself).
 - `agenda_*`: See [Agenda](https://github.com/OpenSlides/OpenSlides/wiki/Agenda#additional-fields-during-creation-of-agenda-content-objects).
 
 Other things to do when creating motions:

docs/actions/user.update.md

@@ -85,6 +85,7 @@ Updates a user.
 Note: `is_present_in_meeting_ids` is not available in update, since there is no possibility to partially update this field. This can be done via [user.set_present](user.set_present.md).
 
 If the user is removed from all groups of the meeting, all his unstarted speakers in that meeting will be deleted.
+His meeting_user for that meeting will also be deleted.
 
 If the user was the last member of the meetings admin group and he happens to be removed from the latter through this action, as long as the meeting is not a template, there will be an error.
 

meta

@@ -4,7 +4,7 @@
 from ....models.models import ChatMessage
 from ....permissions.permission_helper import has_perm
 from ....permissions.permissions import Permissions
-from ....shared.exceptions import PermissionDenied
+from ....shared.exceptions import ActionException, PermissionDenied
 from ....shared.patterns import fqid_from_collection_and_id
 from ...mixins.create_action_with_inferred_meeting import (
     CreateActionWithInferredMeeting,
@@ -28,9 +28,13 @@ class ChatMessageCreate(MeetingUserHelperMixin, CreateActionWithInferredMeeting)
 
     def update_instance(self, instance: dict[str, Any]) -> dict[str, Any]:
         instance = super().update_instance(instance)
-        instance["meeting_user_id"] = self.create_or_get_meeting_user(
-            instance["meeting_id"], self.user_id
-        )
+        instance["meeting_user_id"] = (
+            self.get_meeting_user(instance["meeting_id"], self.user_id, ["id"]) or {}
+        ).get("id")
+        if not instance.get("meeting_user_id"):
+            raise ActionException(
+                "Cannot create chat message: You are not a participant of the meeting."
+            )
         instance["created"] = round(time())
         return instance
 

openslides_backend/action/actions/chat_message/create.py

@@ -0,0 +1,25 @@
+from openslides_backend.shared.patterns import fqid_from_collection_and_id
+from openslides_backend.shared.typing import HistoryInformation
+
+from ....models.models import MeetingUser
+from ...generics.delete import DeleteAction
+from ...util.default_schema import DefaultSchema
+
+
+class MeetingUserBaseDelete(DeleteAction):
+    """
+    Base action to delete a meeting user.
+    """
+
+    model = MeetingUser()
+    schema = DefaultSchema(MeetingUser()).get_delete_schema()
+
+    def get_history_information(self) -> HistoryInformation | None:
+        users = self.get_instances_with_fields(["user_id", "meeting_id"])
+        return {
+            fqid_from_collection_and_id("user", user["user_id"]): [
+                "Participant removed from meeting {}",
+                fqid_from_collection_and_id("meeting", user["meeting_id"]),
+            ]
+            for user in users
+        }

openslides_backend/action/actions/meeting_user/base_delete.py

@@ -1,37 +1,21 @@
 from typing import Any
 
 from openslides_backend.shared.patterns import fqid_from_collection_and_id
-from openslides_backend.shared.typing import HistoryInformation
 
-from ....models.models import MeetingUser
-from ...generics.delete import DeleteAction
 from ...util.action_type import ActionType
-from ...util.default_schema import DefaultSchema
 from ...util.register import register_action
 from ..user.conditional_speaker_cascade_mixin import (
     ConditionalSpeakerCascadeMixinHelper,
 )
+from .base_delete import MeetingUserBaseDelete
 
 
 @register_action("meeting_user.delete", action_type=ActionType.BACKEND_INTERNAL)
-class MeetingUserDelete(ConditionalSpeakerCascadeMixinHelper, DeleteAction):
+class MeetingUserDelete(ConditionalSpeakerCascadeMixinHelper, MeetingUserBaseDelete):
     """
     Action to delete a meeting user.
     """
 
-    model = MeetingUser()
-    schema = DefaultSchema(MeetingUser()).get_delete_schema()
-
-    def get_history_information(self) -> HistoryInformation | None:
-        users = self.get_instances_with_fields(["user_id", "meeting_id"])
-        return {
-            fqid_from_collection_and_id("user", user["user_id"]): [
-                "Participant removed from meeting {}",
-                fqid_from_collection_and_id("meeting", user["meeting_id"]),
-            ]
-            for user in users
-        }
-
     def update_instance(self, instance: dict[str, Any]) -> dict[str, Any]:
         meeting_user = self.datastore.get(
             fqid_from_collection_and_id("meeting_user", instance["id"]),

openslides_backend/action/actions/meeting_user/delete.py

@@ -255,7 +255,7 @@ def handle_group_updates(
             group_information: list[str] = []
             if added and removed:
                 group_information.append("Groups changed")
-            else:
+            elif added or removed:
                 if added:
                     group_information.append("Participant added to")
                 else:
@@ -272,7 +272,8 @@ def handle_group_updates(
                 group_information.append(
                     fqid_from_collection_and_id("meeting", meeting_id)
                 )
-            instance_information.append(tuple(group_information))
+            if group_information:
+                instance_information.append(tuple(group_information))
 
     def handle_delegations(
         self,