From 82fe7eea4b5884b5feec85b75ff7a74bf7532764 Mon Sep 17 00:00:00 2001
From: Sean Engelhardt <sean.f.t.engelhardt@gmail.com>
Date: Wed, 25 Sep 2019 15:51:15 +0200
Subject: [PATCH] Adds amendment filter types to motion list

Adds the possibility to filter if a motion has amendments or is an amendment
---
 .../app/site/motions/models/view-motion.ts    | 33 +++++++++++++++++++
 .../services/motion-filter-list.service.ts    | 16 ++++++++-
 2 files changed, 48 insertions(+), 1 deletion(-)

diff --git a/client/src/app/site/motions/models/view-motion.ts b/client/src/app/site/motions/models/view-motion.ts
index 665da9e9a..f5a75fa5f 100644
--- a/client/src/app/site/motions/models/view-motion.ts
+++ b/client/src/app/site/motions/models/view-motion.ts
@@ -40,6 +40,11 @@ export enum ChangeRecoMode {
     ModifiedFinal = 'modified_final_version'
 }
 
+export enum AmendmentType {
+    Amendment = 1,
+    Parent
+}
+
 export const verboseChangeRecoMode = {
     original: 'Original version',
     changed: 'Changed version',
@@ -340,6 +345,34 @@ export class ViewMotion extends BaseViewModelWithAgendaItemAndListOfSpeakers<Mot
         this._diffLines = value;
     }
 
+    /**
+     * Determine if a motion has a parent at all
+     */
+    public get hasParent(): boolean {
+        return !!this.parent_id;
+    }
+
+    /**
+     * Determine if a motion has amenments
+     */
+    public get hasAmendments(): boolean {
+        return !!this.amendments && !!this.amendments.length;
+    }
+
+    /**
+     * Determine if the motion has parents, is a parent of neither
+     */
+    public get amendmentType(): number {
+        if (this.hasAmendments) {
+            return AmendmentType.Parent;
+        } else if (this.hasParent) {
+            return AmendmentType.Amendment;
+        } else {
+            // not any amendment
+            return 0;
+        }
+    }
+
     /**
      * Get the number of the first diff line, in case a motion is an amendment
      */
diff --git a/client/src/app/site/motions/services/motion-filter-list.service.ts b/client/src/app/site/motions/services/motion-filter-list.service.ts
index 895b5f48a..e91a17974 100644
--- a/client/src/app/site/motions/services/motion-filter-list.service.ts
+++ b/client/src/app/site/motions/services/motion-filter-list.service.ts
@@ -17,7 +17,7 @@ import {
     OsFilterOptions
 } from 'app/core/ui-services/base-filter-list.service';
 import { ConfigService } from 'app/core/ui-services/config.service';
-import { ViewMotion } from '../models/view-motion';
+import { AmendmentType, ViewMotion } from '../models/view-motion';
 
 /**
  * Filter description to easier parse dynamically occurring workflows
@@ -97,6 +97,15 @@ export class MotionFilterListService extends BaseFilterListService<ViewMotion> {
         ]
     };
 
+    public AmendmentFilterOption: OsFilter = {
+        property: 'amendmentType',
+        label: 'Amendment',
+        options: [
+            { condition: AmendmentType.Amendment, label: this.translate.instant('Is amendment') },
+            { condition: AmendmentType.Parent, label: this.translate.instant('Has amendment') }
+        ]
+    };
+
     public personalNoteFilterOptions = [
         {
             property: 'star',
@@ -226,9 +235,14 @@ export class MotionFilterListService extends BaseFilterListService<ViewMotion> {
             filterDefinitions.push(this.hasSpeakerOptions);
         }
 
+        if (this.showAmendmentsInMainTable) {
+            filterDefinitions.push(this.AmendmentFilterOption);
+        }
+
         if (!this.operator.isAnonymous) {
             filterDefinitions = filterDefinitions.concat(this.personalNoteFilterOptions);
         }
+
         return filterDefinitions;
     }