From 3f634065caedd067de38742e63d1735c85d814aa Mon Sep 17 00:00:00 2001
From: kolaente <konrad@kola-entertainments.de>
Date: Thu, 21 Mar 2019 19:13:22 +0100
Subject: [PATCH] Started adding methods to be able to correctly modify labels
 on a list

---
 lib/api/label_task_bulk.dart  | 17 +++++++++++++++++
 lib/global.dart               |  3 +++
 lib/models/label.dart         |  2 +-
 lib/pages/list/task_edit.dart |  7 +++++--
 lib/service/services.dart     |  4 ++++
 5 files changed, 30 insertions(+), 3 deletions(-)
 create mode 100644 lib/api/label_task_bulk.dart

diff --git a/lib/api/label_task_bulk.dart b/lib/api/label_task_bulk.dart
new file mode 100644
index 0000000..77bf4ff
--- /dev/null
+++ b/lib/api/label_task_bulk.dart
@@ -0,0 +1,17 @@
+import 'package:vikunja_app/api/client.dart';
+import 'package:vikunja_app/api/service.dart';
+import 'package:vikunja_app/models/label.dart';
+import 'package:vikunja_app/models/task.dart';
+import 'package:vikunja_app/service/services.dart';
+
+class LabelTaskBulkAPIService extends APIService
+  implements LabelTaskBulkService {
+  LabelTaskBulkAPIService(Client client) : super(client);
+
+  @override
+  Future<Label> update(Task task, List<Label> labels) {
+    return client
+      .post('/tasks/${task.id}/labels/bulk', body: labels.map((l) => l.toJSON()).toList())
+      .then((labels) { print(labels.toString());});
+  }
+}
\ No newline at end of file
diff --git a/lib/global.dart b/lib/global.dart
index aa10900..2de8da9 100644
--- a/lib/global.dart
+++ b/lib/global.dart
@@ -2,6 +2,7 @@ import 'package:flutter/material.dart';
 import 'package:flutter_secure_storage/flutter_secure_storage.dart';
 import 'package:vikunja_app/api/client.dart';
 import 'package:vikunja_app/api/label_task.dart';
+import 'package:vikunja_app/api/label_task_bulk.dart';
 import 'package:vikunja_app/api/labels.dart';
 import 'package:vikunja_app/api/list_implementation.dart';
 import 'package:vikunja_app/api/namespace_implementation.dart';
@@ -52,6 +53,8 @@ class VikunjaGlobalState extends State<VikunjaGlobal> {
 
   LabelTaskService get labelTaskService => new LabelTaskAPIService(client);
 
+  LabelTaskBulkAPIService get labelTaskBulkService => new LabelTaskBulkAPIService(client);
+
   @override
   void initState() {
     super.initState();
diff --git a/lib/models/label.dart b/lib/models/label.dart
index 5e81894..f2b9f65 100644
--- a/lib/models/label.dart
+++ b/lib/models/label.dart
@@ -32,7 +32,7 @@ class Label {
         'id': id,
         'title': title,
         'description': description,
-        'hex_color': color.value.toRadixString(16).padLeft(8, '0').substring(2),
+        'hex_color': color?.value == null ? null : color.value.toRadixString(16).padLeft(8, '0').substring(2),
         'created_by': createdBy?.toJSON(),
         'updated': updated?.millisecondsSinceEpoch,
         'created': created?.millisecondsSinceEpoch,
diff --git a/lib/pages/list/task_edit.dart b/lib/pages/list/task_edit.dart
index 483f093..cf16c71 100644
--- a/lib/pages/list/task_edit.dart
+++ b/lib/pages/list/task_edit.dart
@@ -298,9 +298,11 @@ class _TaskEditPageState extends State<TaskEditPage> {
       endDate: _endDate,
       priority: _priority,
       repeatAfter: _repeatAfter,
-      labels: _labels,
     );
 
+    // update the labels
+    VikunjaGlobal.of(context).labelTaskBulkService.update(updatedTask, _labels);
+
     VikunjaGlobal.of(context).taskService.update(updatedTask).then((_) {
       setState(() => _loading = false);
       Scaffold.of(context).showSnackBar(SnackBar(
@@ -326,6 +328,8 @@ class _TaskEditPageState extends State<TaskEditPage> {
         .labelService
         .getAll(query: query)
         .then((labels) {
+          // Only show those labels which aren't already added to the task
+          labels.removeWhere((labelToRemove) => _labels.contains(labelToRemove));
           _suggestedLabels = labels;
           return labels.map((label) => label.title).toList();
     });
@@ -338,7 +342,6 @@ class _TaskEditPageState extends State<TaskEditPage> {
         _labels.add(label);
       }
     });
-    // TODO: really add the label to the list
   }
 
   // FIXME: Move the following two functions to an extra class or type.
diff --git a/lib/service/services.dart b/lib/service/services.dart
index 51e91d5..a3c48cc 100644
--- a/lib/service/services.dart
+++ b/lib/service/services.dart
@@ -48,4 +48,8 @@ abstract class LabelTaskService {
   Future<List<Label>> getAll(LabelTask lt, {String query});
   Future<Label> create(LabelTask lt);
   Future<Label> delete(LabelTask lt);
+}
+
+abstract class LabelTaskBulkService {
+  Future<Label> update(Task task, List<Label> labels);
 }
\ No newline at end of file