Fix motion create form

OpenSlides · Oct 22, 2024 · 9232265 · 9232265
1 parent 6a52d30
commit 9232265
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/...ons/pages/motion-detail/pages/motion-form/components/motion-form/motion-form.component.ts b/...ons/pages/motion-detail/pages/motion-form/components/motion-form/motion-form.component.ts
@@ -100,6 +100,10 @@ export class MotionFormComponent extends BaseMeetingComponent implements OnInit
         return this.perms.isAllowed(`change_metadata`, this.motion);
     }
 
+    public get canManageAgenda(): boolean {
+        return this.perms.canManageAgenda();
+    }
+
     public get isParagraphBasedAmendment(): boolean {
         return this.isExisting && this.motion.isParagraphBasedAmendment();
     }
@@ -208,6 +212,11 @@ export class MotionFormComponent extends BaseMeetingComponent implements OnInit
         return async () => {
             const update = event || this.temporaryMotion;
             if (this.newMotion) {
+                for (const key in update) {
+                    if (update[key] === null || update[key].length === 0) {
+                        delete update[key];
+                    }
+                }
                 await this.createMotion(update);
             } else {
                 await this.updateMotion(update, this.motion);
@@ -582,7 +591,9 @@ export class MotionFormComponent extends BaseMeetingComponent implements OnInit
                 number: [
                     ``,
                     isUniqueAmong<string>(this._motionNumbersSubject, (a, b) => a === b, [``, null, undefined])
-                ],
+                ]
+            }),
+            ...(this.canManageAgenda && {
                 agenda_create: [``],
                 agenda_type: [``]
             })

diff --git a/...ings/pages/motions/services/common/motion-permission.service/motion-permission.service.ts b/...ings/pages/motions/services/common/motion-permission.service/motion-permission.service.ts
@@ -55,6 +55,10 @@ export class MotionPermissionService {
         );
     }
 
+    public canManageAgenda(): boolean {
+        return this.operator.hasPerms(Permission.agendaItemCanManage);
+    }
+
     public canDoActionWhileDelegationEnabled(isAdditionalDelegationSettingEnabled: boolean): boolean {
         return !(
             !this.operator.isAnonymous &&