From 6eeab5fc1ee9680b559807c642fa5f800f8eb5d6 Mon Sep 17 00:00:00 2001
From: Maximilian Krambach <maximilian.krambach@intevation.de>
Date: Tue, 28 May 2019 18:25:41 +0200
Subject: [PATCH] Multi move motions in category - if there is a
 multiselection, a 'move to' button will be active - TODO: better
 position/layout  for that button and dialogue

---
 .../sorting-list/sorting-list.component.ts    | 18 ++++++++--
 .../category-sort.component.html              |  7 +++-
 .../category-sort/category-sort.component.ts  | 35 +++++++++++++++++--
 3 files changed, 54 insertions(+), 6 deletions(-)

diff --git a/client/src/app/shared/components/sorting-list/sorting-list.component.ts b/client/src/app/shared/components/sorting-list/sorting-list.component.ts
index 8011cb1ae..5da1cb2cc 100644
--- a/client/src/app/shared/components/sorting-list/sorting-list.component.ts
+++ b/client/src/app/shared/components/sorting-list/sorting-list.component.ts
@@ -162,9 +162,15 @@ export class SortingListComponent implements OnInit, OnDestroy {
     /**
      * drop event
      * @param event the event
+     * @param dropBehind (optional) toggle explicit 'insert behind'(true) or
+     * 'insert before' (false) behavior instead of relying on a
+     * 'natural drop logic'
      */
-    public drop(event: CdkDragDrop<Selectable[]>): void {
-        if (this.multiSelectedIndex.length < 2) {
+    public drop(
+        event: CdkDragDrop<Selectable[]> | { currentIndex: number; previousIndex: number },
+        dropBehind?: boolean
+    ): void {
+        if (!this.multiSelectedIndex.length) {
             moveItemInArray(this.array, event.previousIndex, event.currentIndex);
         } else {
             const before: Selectable[] = [];
@@ -177,7 +183,13 @@ export class SortingListComponent implements OnInit, OnDestroy {
                     } else if (i > event.currentIndex) {
                         behind.push(this.array[i]);
                     } else {
-                        event.currentIndex < 1 ? behind.push(this.array[i]) : before.push(this.array[i]);
+                        if (dropBehind === false) {
+                            behind.push(this.array[i]);
+                        } else if (dropBehind === true) {
+                            before.push(this.array[i]);
+                        } else {
+                            event.currentIndex < 1 ? behind.push(this.array[i]) : before.push(this.array[i]);
+                        }
                     }
                 } else {
                     insertions.push(this.array[i]);
diff --git a/client/src/app/site/motions/modules/category/components/category-sort/category-sort.component.html b/client/src/app/site/motions/modules/category/components/category-sort/category-sort.component.html
index 7b8df5fc7..e2bf477a0 100644
--- a/client/src/app/site/motions/modules/category/components/category-sort/category-sort.component.html
+++ b/client/src/app/site/motions/modules/category/components/category-sort/category-sort.component.html
@@ -24,7 +24,12 @@
     >
         <span translate>Number motions</span>
     </button>
-
+    <div *ngIf="isMultiSelect">
+        <span> {{ sortSelector.multiSelectedIndex.length }}&nbsp;</span>
+        <span translate>selected</span>
+        <button mat-button (click)="moveToPosition()"><span translate>move ...</span>
+        </button>
+    </div>
     <os-sorting-list
         (sortEvent)="onListUpdate($event)"
         [input]="motionObservable"
diff --git a/client/src/app/site/motions/modules/category/components/category-sort/category-sort.component.ts b/client/src/app/site/motions/modules/category/components/category-sort/category-sort.component.ts
index 15e64422c..2b24c4a8e 100644
--- a/client/src/app/site/motions/modules/category/components/category-sort/category-sort.component.ts
+++ b/client/src/app/site/motions/modules/category/components/category-sort/category-sort.component.ts
@@ -6,13 +6,14 @@ import { TranslateService } from '@ngx-translate/core';
 
 import { BaseViewComponent } from 'app/site/base/base-view';
 import { CategoryRepositoryService } from 'app/core/repositories/motions/category-repository.service';
+import { CanComponentDeactivate } from 'app/shared/utils/watch-sorting-tree.guard';
+import { ChoiceService } from 'app/core/ui-services/choice.service';
 import { MatSnackBar } from '@angular/material';
 import { MotionRepositoryService } from 'app/core/repositories/motions/motion-repository.service';
 import { PromptService } from 'app/core/ui-services/prompt.service';
 import { SortingListComponent } from 'app/shared/components/sorting-list/sorting-list.component';
 import { ViewCategory } from 'app/site/motions/models/view-category';
 import { ViewMotion } from 'app/site/motions/models/view-motion';
-import { CanComponentDeactivate } from 'app/shared/utils/watch-sorting-tree.guard';
 
 /**
  * View for rearranging and renumbering the motions of a category. The {@link onNumberMotions}
@@ -56,6 +57,10 @@ export class CategorySortComponent extends BaseViewComponent implements OnInit,
      */
     private motionsBackup: ViewMotion[] = [];
 
+    public get isMultiSelect(): boolean {
+        return this.sortSelector.multiSelectedIndex.length > 0;
+    }
+
     /**
      * @returns an observable for the {@link motionsSubject}
      */
@@ -89,6 +94,7 @@ export class CategorySortComponent extends BaseViewComponent implements OnInit,
      * @param repo
      * @param route
      * @param motionRepo
+     * @param choiceService
      */
     public constructor(
         title: Title,
@@ -97,7 +103,8 @@ export class CategorySortComponent extends BaseViewComponent implements OnInit,
         private promptService: PromptService,
         private repo: CategoryRepositoryService,
         private route: ActivatedRoute,
-        private motionRepo: MotionRepositoryService
+        private motionRepo: MotionRepositoryService,
+        private choiceService: ChoiceService
     ) {
         super(title, translate, matSnackBar);
     }
@@ -231,4 +238,28 @@ export class CategorySortComponent extends BaseViewComponent implements OnInit,
         }
         return true;
     }
+
+    public async moveToPosition(): Promise<void> {
+        if (this.sortSelector.multiSelectedIndex.length) {
+        }
+        const content = this.translate.instant('Move the selected items behind');
+        const choices = this.sortSelector.array
+            .map((item, index) => {
+                return { id: index, label: item.getTitle() };
+            })
+            .filter(f => !this.sortSelector.multiSelectedIndex.includes(f.id));
+        const actions = [this.translate.instant('Insert before'), this.translate.instant('Insert behind')];
+        const selectedChoice = await this.choiceService.open(content, choices, false, actions);
+        if (selectedChoice) {
+            const newIndex = selectedChoice.items as number;
+
+            this.sortSelector.drop(
+                {
+                    currentIndex: newIndex,
+                    previousIndex: null
+                },
+                selectedChoice.action === actions[1] // true if 'insert behind'
+            );
+        }
+    }
 }