From 25839ea709b34b5f95139e01e7fb6f385c47eff8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tobias=20Ho=CC=88=C3=9Fl?= <tobias@hoessl.eu>
Date: Sat, 2 May 2020 17:23:52 +0200
Subject: [PATCH] Toggle to show all amendments in diff view

---
 .../models/view-motion-amended-paragraph.ts   |  2 +-
 .../motion-detail-diff.component.html         | 60 +++++++++++++++++--
 .../motion-detail-diff.component.ts           |  8 +++
 .../motion-detail.component.html              |  7 +++
 .../motion-detail/motion-detail.component.ts  | 11 +++-
 5 files changed, 81 insertions(+), 7 deletions(-)

diff --git a/client/src/app/site/motions/models/view-motion-amended-paragraph.ts b/client/src/app/site/motions/models/view-motion-amended-paragraph.ts
index 1c365752b..145c65f77 100644
--- a/client/src/app/site/motions/models/view-motion-amended-paragraph.ts
+++ b/client/src/app/site/motions/models/view-motion-amended-paragraph.ts
@@ -15,7 +15,7 @@ export class ViewMotionAmendedParagraph implements ViewUnifiedChange {
     }
 
     public constructor(
-        private amendment: ViewMotion,
+        public amendment: ViewMotion,
         private paragraphNo: number,
         private newText: string,
         private lineRange: LineRange
diff --git a/client/src/app/site/motions/modules/motion-detail/components/motion-detail-diff/motion-detail-diff.component.html b/client/src/app/site/motions/modules/motion-detail/components/motion-detail-diff/motion-detail-diff.component.html
index 5d8c464e6..cfde3008a 100644
--- a/client/src/app/site/motions/modules/motion-detail/components/motion-detail-diff/motion-detail-diff.component.html
+++ b/client/src/app/site/motions/modules/motion-detail/components/motion-detail-diff/motion-detail-diff.component.html
@@ -26,10 +26,10 @@
                     -->
                 </span>
                 <span class="status">
-                    <ng-container *ngIf="change.isRejected()">
+                    <ng-container *ngIf="change.isRejected() && !isAmendment(change)">
                         – <span>{{ 'Rejected' | translate }}</span></ng-container
                     >
-                    <ng-container *ngIf="change.isAccepted() && isAmendment(change)">
+                    <ng-container *ngIf="(change.isAccepted() || isAmendment(change)) && change.stateName">
                         – {{ change.stateName | translate }}</ng-container
                     >
                 </span>
@@ -104,6 +104,15 @@
                 >
                     <mat-icon>more_horiz</mat-icon>
                 </button>
+                <button
+                    mat-icon-button
+                    *ngIf="isAmendment(change)"
+                    type="button"
+                    [matMenuTriggerFor]="amendmentMenu"
+                    [matMenuTriggerData]="{ change: change }"
+                >
+                    <mat-icon>more_horiz</mat-icon>
+                </button>
 
                 <!--
                     <a ng-if="change.type == 'amendment'" ui-sref="motions.motion.detail({id: change.original.id})"
@@ -114,9 +123,17 @@
                     </a>
                 -->
             </div>
-            <div class="status-row" *ngIf="change.isRejected()">
+            <div class="status-row" *ngIf="isChangeRecommendation(change) && change.isRejected()">
                 <i class="grey">{{ 'Rejected' | translate }}</i>
             </div>
+            
+            <!-- If it's an amendment, only accepted ones will be passed to this component
+                 by default (== !showAllAmendments). In this case, don't show the prefix.
+                 However, if showAllAmendments === true, the prefix should be shown alwayes,
+                 even if it's accepted -->
+            <div class="status-row" *ngIf="isAmendment(change) && showAllAmendments">
+                <i class="grey">{{ change.amendment.identifier }} - {{ change.amendment.state.name | translate }}</i>
+            </div>
 
             <div
                 class="motion-text motion-text-diff"
@@ -143,12 +160,46 @@
     </div>
 </div>
 
+<mat-menu #amendmentMenu="matMenu">
+    <ng-template matMenuContent let-change="change">
+        <button
+            type="button"
+            mat-menu-item
+            [routerLink]="change.amendment.getDetailStateURL()" [state]="{ back: 'true' }"
+        >
+            <mat-icon>visibility_on</mat-icon>
+            <span>{{ 'Show amendment' | translate }}</span>
+        </button>
+
+        <mat-divider *ngIf="change.amendment.state.next_states.length > 0"></mat-divider>
+
+        <button *ngFor="let state of change.amendment.state.next_states" mat-menu-item (click)="setAmendmentState(change, state.id)">
+            {{ state.name | translate }} <span *ngIf="state.show_state_extension_field">&nbsp;...</span>
+        </button>
+        <mat-divider *ngIf="change.amendment.state.next_states.length > 0"></mat-divider>
+        <button
+            *ngFor="let state of change.amendment.state.previous_states"
+            mat-menu-item
+            (click)="setAmendmentState(change, state.id)"
+        >
+            <mat-icon>arrow_back</mat-icon>
+            {{ state.name | translate }}
+            <span *ngIf="state.show_state_extension_field">&nbsp;...</span>
+        </button>
+        <button mat-menu-item (click)="setAmendmentState(change, null)">
+            <mat-icon>replay</mat-icon>
+            {{ 'Reset state' | translate }}
+        </button>
+
+        <!-- perms.isAllowed('change_state', motion) -->
+    </ng-template>
+</mat-menu>
+
 <mat-menu #changeRecommendationMenu="matMenu">
     <ng-template matMenuContent let-change="change">
         <button
             type="button"
             mat-menu-item
-            [disabled]="hasCollissions(change, changes)"
             (click)="setAcceptanceValue(change, 'accepted')"
         >
             <mat-icon>thumb_up</mat-icon>
@@ -158,7 +209,6 @@
         <button
             type="button"
             mat-menu-item
-            [disabled]="hasCollissions(change, changes)"
             (click)="setAcceptanceValue(change, 'rejected')"
         >
             <mat-icon>thumb_down</mat-icon>
diff --git a/client/src/app/site/motions/modules/motion-detail/components/motion-detail-diff/motion-detail-diff.component.ts b/client/src/app/site/motions/modules/motion-detail/components/motion-detail-diff/motion-detail-diff.component.ts
index 2876c0ae8..9f02b3ae4 100644
--- a/client/src/app/site/motions/modules/motion-detail/components/motion-detail-diff/motion-detail-diff.component.ts
+++ b/client/src/app/site/motions/modules/motion-detail/components/motion-detail-diff/motion-detail-diff.component.ts
@@ -26,6 +26,7 @@ import {
     MotionTitleChangeRecommendationDialogComponent,
     MotionTitleChangeRecommendationDialogComponentData
 } from '../motion-title-change-recommendation-dialog/motion-title-change-recommendation-dialog.component';
+import { ViewMotionAmendedParagraph } from '../../../../models/view-motion-amended-paragraph';
 
 /**
  * This component displays the original motion text with the change blocks inside.
@@ -45,6 +46,7 @@ import {
  *       [scrollToChange]="change"
  *       [highlightedLine]="highlightedLine"
  *       [lineNumberingMode]="lnMode"
+ *       [showAllAmendments]="showAllAmendments"
  *       (createChangeRecommendation)="createChangeRecommendation($event)"
  * ></os-motion-detail-diff>
  * ```
@@ -70,6 +72,8 @@ export class MotionDetailDiffComponent extends BaseViewComponent implements Afte
     public highlightedLine: number;
     @Input()
     public lineNumberingMode: LineNumberingMode;
+    @Input()
+    public showAllAmendments: boolean;
 
     @Output()
     public createChangeRecommendation: EventEmitter<LineRange> = new EventEmitter<LineRange>();
@@ -364,6 +368,10 @@ export class MotionDetailDiffComponent extends BaseViewComponent implements Afte
         });
     }
 
+    public setAmendmentState(change: ViewUnifiedChange, state: number): void {
+        this.motionRepo.setState((change as ViewMotionAmendedParagraph).amendment, state).catch(this.raiseError);
+    }
+
     /**
      * Scrolls to the native element specified by [scrollToChange]
      */
diff --git a/client/src/app/site/motions/modules/motion-detail/components/motion-detail/motion-detail.component.html b/client/src/app/site/motions/modules/motion-detail/components/motion-detail/motion-detail.component.html
index 657a2ea0b..1c4213ef2 100644
--- a/client/src/app/site/motions/modules/motion-detail/components/motion-detail/motion-detail.component.html
+++ b/client/src/app/site/motions/modules/motion-detail/components/motion-detail/motion-detail.component.html
@@ -463,6 +463,11 @@
                 <span *ngIf="amendments.length === 1">{{ 'Amendment' | translate }}</span>
                 <span *ngIf="amendments.length > 1">{{ 'Amendments' | translate }}</span>
             </a>
+
+            <br>
+            <mat-checkbox [(ngModel)]="showAllAmendments" *ngIf="isRecoMode(ChangeRecoMode.Diff)">
+                {{ 'Temporarily show all in text' | translate }}
+            </mat-checkbox>
         </div>
 
         <!-- motion polls -->
@@ -681,6 +686,7 @@
                 [changes]="getChangesForDiffMode()"
                 [scrollToChange]="scrollToChange"
                 [highlightedLine]="highlightedLine"
+                [showAllAmendments]="showAllAmendments"
                 [lineNumberingMode]="lnMode"
                 (createChangeRecommendation)="createChangeRecommendation($event)"
             ></os-motion-detail-diff>
@@ -904,6 +910,7 @@
                 [changes]="getChangesForDiffMode()"
                 [scrollToChange]="scrollToChange"
                 [highlightedLine]="highlightedLine"
+                [showAllAmendments]="showAllAmendments"
                 [lineNumberingMode]="lnMode"
                 (createChangeRecommendation)="createChangeRecommendation($event)"
             ></os-motion-detail-diff>
diff --git a/client/src/app/site/motions/modules/motion-detail/components/motion-detail/motion-detail.component.ts b/client/src/app/site/motions/modules/motion-detail/components/motion-detail/motion-detail.component.ts
index e5f10e195..d845cea7f 100644
--- a/client/src/app/site/motions/modules/motion-detail/components/motion-detail/motion-detail.component.ts
+++ b/client/src/app/site/motions/modules/motion-detail/components/motion-detail/motion-detail.component.ts
@@ -327,6 +327,11 @@ export class MotionDetailComponent extends BaseViewComponent implements OnInit,
      */
     private amendmentTextMode: string;
 
+    /**
+     * Show all amendments in the text, not only the ones with the apropriate state
+     */
+    public showAllAmendments = false;
+
     /**
      * For using the enum constants from the template
      */
@@ -1005,7 +1010,11 @@ export class MotionDetailComponent extends BaseViewComponent implements OnInit,
 
     public getChangesForDiffMode(): ViewUnifiedChange[] {
         return this.allChangingObjects.filter(change => {
-            return change.showInDiffView();
+            if (this.showAllAmendments) {
+                return true;
+            } else {
+                return change.showInDiffView();
+            }
         });
     }