From 1a0e017f8091ef2f1e65097593420c327122d99f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tobias=20Ho=CC=88=C3=9Fl?= <tobias@hoessl.eu>
Date: Wed, 16 Sep 2020 19:51:07 +0200
Subject: [PATCH] Fix changed amendment in slides

---
 .../motions/motion-repository.service.ts      | 16 +++++++
 .../motion-detail-diff.component.ts           | 14 +-----
 .../motion/motion-slide.component.html        | 47 ++++++++++++++++++-
 .../motions/motion/motion-slide.component.ts  | 38 +++++++++++++++
 4 files changed, 101 insertions(+), 14 deletions(-)

diff --git a/client/src/app/core/repositories/motions/motion-repository.service.ts b/client/src/app/core/repositories/motions/motion-repository.service.ts
index a80da6d5c..209e8d189 100644
--- a/client/src/app/core/repositories/motions/motion-repository.service.ts
+++ b/client/src/app/core/repositories/motions/motion-repository.service.ts
@@ -1121,4 +1121,20 @@ export class MotionRepositoryService extends BaseIsAgendaItemAndListOfSpeakersCo
             }
         });
     }
+
+    public changeHasCollissions(change: ViewUnifiedChange, changes: ViewUnifiedChange[]): boolean {
+        return (
+            changes.filter((otherChange: ViewUnifiedChange) => {
+                return (
+                    otherChange.getChangeId() !== change.getChangeId() &&
+                    ((otherChange.getLineFrom() >= change.getLineFrom() &&
+                        otherChange.getLineFrom() < change.getLineTo()) ||
+                        (otherChange.getLineTo() > change.getLineFrom() &&
+                            otherChange.getLineTo() <= change.getLineTo()) ||
+                        (otherChange.getLineFrom() < change.getLineFrom() &&
+                            otherChange.getLineTo() > change.getLineTo()))
+                );
+            }).length > 0
+        );
+    }
 }
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 d0b084323..cfac1619b 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
@@ -155,19 +155,7 @@ export class MotionDetailDiffComponent extends BaseViewComponent implements Afte
      * @param {ViewUnifiedChange[]} changes
      */
     public hasCollissions(change: ViewUnifiedChange, changes: ViewUnifiedChange[]): boolean {
-        return (
-            changes.filter((otherChange: ViewUnifiedChange) => {
-                return (
-                    otherChange.getChangeId() !== change.getChangeId() &&
-                    ((otherChange.getLineFrom() >= change.getLineFrom() &&
-                        otherChange.getLineFrom() < change.getLineTo()) ||
-                        (otherChange.getLineTo() > change.getLineFrom() &&
-                            otherChange.getLineTo() <= change.getLineTo()) ||
-                        (otherChange.getLineFrom() < change.getLineFrom() &&
-                            otherChange.getLineTo() > change.getLineTo()))
-                );
-            }).length > 0
-        );
+        return this.motionRepo.changeHasCollissions(change, changes);
     }
 
     /**
diff --git a/client/src/app/slides/motions/motion/motion-slide.component.html b/client/src/app/slides/motions/motion/motion-slide.component.html
index da4608861..5529bc8d4 100644
--- a/client/src/app/slides/motions/motion/motion-slide.component.html
+++ b/client/src/app/slides/motions/motion/motion-slide.component.html
@@ -105,7 +105,7 @@
             ></div>
 
             <!-- Amendment text -->
-            <section class="text-holder" *ngIf="isParagraphBasedAmendment()">
+            <section class="text-holder" *ngIf="isParagraphBasedAmendment() && crMode !== 'diff'">
                 <div class="alert alert-info" *ngIf="getAmendedParagraphs().length === 0">
                     <span>{{ 'No changes at the text.' | translate }}</span>
                 </div>
@@ -127,6 +127,51 @@
                 </div>
             </section>
 
+            <!-- The actual diff view of an amendment. This logic is based on the os-motion-detail-diff component -->
+            <section class="text-holder" *ngIf="isParagraphBasedAmendment() && crMode === 'diff'">
+                <div *ngIf="getTitleChangingObject() as changedTitle">
+                    <div class="diff-box diff-box-{{ changedTitle.getChangeId() }} clearfix">
+                        <div class="status-row" *ngIf="changedTitle.isRejected()">
+                            <i class="grey">{{ 'Rejected' | translate }}</i>
+                        </div>
+
+                        <div
+                            class="motion-text motion-text-diff"
+                            [class.line-numbers-none]="isLineNumberingNone()"
+                            [class.line-numbers-inline]="isLineNumberingInline()"
+                            [class.line-numbers-outside]="isLineNumberingOutside()"
+                            [attr.data-change-id]="changedTitle.getChangeId()"
+                        >
+                            <div class="bold">{{ 'Changed title' | translate }}:</div>
+                            <div [innerHTML]="getFormattedTitleDiff() | trust: 'html'"></div>
+                        </div>
+                    </div>
+                </div>
+                <div *ngFor="let change of getAllTextChangingObjects(); let i = index">
+                    <div
+                        class="diff-box diff-box-{{ change.getChangeId() }} clearfix"
+                        [class.collides]="hasCollissions(change, getAllTextChangingObjects())"
+                    >
+                        <div class="status-row" *ngIf="isChangeRecommendation(change) && change.isRejected()">
+                            <i class="grey">{{ 'Rejected' | translate }}</i>
+                        </div>
+
+                        <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"
+                            [class.line-numbers-none]="isLineNumberingNone()"
+                            [class.line-numbers-inline]="isLineNumberingInline()"
+                            [class.line-numbers-outside]="isLineNumberingOutside()"
+                            [attr.data-change-id]="change.getChangeId()"
+                            [innerHTML]="getAmendmentDiff(change) | trust: 'html'"
+                        ></div>
+                    </div>
+                </div>
+            </section>
+
             <!-- Reason -->
             <div *ngIf="data.data.reason">
                 <h3>{{ 'Reason' | translate }}</h3>
diff --git a/client/src/app/slides/motions/motion/motion-slide.component.ts b/client/src/app/slides/motions/motion/motion-slide.component.ts
index 103522400..813eeb8d8 100644
--- a/client/src/app/slides/motions/motion/motion-slide.component.ts
+++ b/client/src/app/slides/motions/motion/motion-slide.component.ts
@@ -446,4 +446,42 @@ export class MotionSlideComponent extends BaseMotionSlideComponent<MotionSlideDa
         const diffHtml = this.diff.diff(this.data.data.base_statute.text, this.data.data.text);
         return this.lineNumbering.insertLineBreaksWithoutNumbers(diffHtml, this.lineLength, true);
     }
+
+    /**
+     * Returns true if this change is colliding with another change
+     * @param {ViewUnifiedChange} change
+     * @param {ViewUnifiedChange[]} changes
+     */
+    public hasCollissions(change: ViewUnifiedChange, changes: ViewUnifiedChange[]): boolean {
+        return this.motionRepo.changeHasCollissions(change, changes);
+    }
+
+    /**
+     * Returns true if the change is an Amendment
+     *
+     * @param {ViewUnifiedChange} change
+     */
+    public isAmendment(change: ViewUnifiedChange): boolean {
+        return change.getChangeType() === ViewUnifiedChangeType.TYPE_AMENDMENT;
+    }
+
+    /**
+     * Returns true if the change is a Change Recommendation
+     *
+     * @param {ViewUnifiedChange} change
+     */
+    public isChangeRecommendation(change: ViewUnifiedChange): boolean {
+        return change.getChangeType() === ViewUnifiedChangeType.TYPE_CHANGE_RECOMMENDATION;
+    }
+
+    /**
+     * Returns the diff string from the motion to the change
+     * @param {ViewUnifiedChange} change
+     */
+    public getAmendmentDiff(change: ViewUnifiedChange): string {
+        const motion = this.data.data;
+        const baseHtml = this.lineNumbering.insertLineNumbers(motion.base_motion?.text, this.lineLength);
+
+        return this.diff.getChangeDiff(baseHtml, change, this.lineLength, this.highlightedLine);
+    }
 }