Merge pull request 'Feature: dismissible tasks' (#1) from development into main

Reviewed-on: #1

lib/main.dart

@@ -4,7 +4,9 @@ import 'model/repositories/local_repository.dart';
 import 'pages/task_edit_page.dart';
 import 'pages/task_overview_page.dart';
 import 'service/controller_scope.dart';
-import 'service/task_controller.dart';
+import 'service/controllers/alarm_controller.dart';
+import 'service/controllers/location_controller.dart';
+import 'service/controllers/task_controller.dart';
 
 void main() async {
   final repository = LocalRepository();
@@ -13,9 +15,15 @@ void main() async {
 
   runApp(
     ControllerScope(
+      controller: LocationController(repository),
+      child: ControllerScope(
+        controller: AlarmController(repository),
+        child: ControllerScope(
           controller: TaskController(repository),
           child: const MainApp(),
         ),
+      ),
+    ),
   );
 }
 

lib/model/callback_models/create_task_request.dart

@@ -1,4 +1,3 @@
-import '../location.dart';
 import '../task.dart';
 
 class CreateTaskRequest {

lib/model/repositories/interfaces/location_repository.dart

@@ -0,0 +1,19 @@
+import '../../location.dart';
+
+abstract class LocationRepository {
+  // Create
+
+  Future<void> createLocation(Location location);
+
+  // Read
+
+  Future<List<Location>> loadLocations();
+
+  // Update
+
+  Future<void> updateLocation(Location location);
+
+  // Delete
+
+  Future<void> deleteLocation(Location location);
+}

lib/model/repositories/local_repository.dart

@@ -3,14 +3,18 @@ import 'dart:convert';
 import 'package:shared_preferences/shared_preferences.dart';
 
 import '../alarm.dart';
+import '../location.dart';
 import '../task.dart';
 import 'interfaces/alarm_repository.dart';
+import 'interfaces/location_repository.dart';
 import 'interfaces/task_repository.dart';
 
-class LocalRepository implements TaskRepository, AlarmRepository {
+class LocalRepository
+    implements TaskRepository, AlarmRepository, LocationRepository {
   static const String _tasksKey = 'tasks';
   static const String _taskOrderKey = 'taskOrder';
   static const String _alarmsKey = 'alarms';
+  static const String _locationsKey = 'locations';
 
   SharedPreferencesWithCache? _prefs;
 
@@ -18,7 +22,12 @@ class LocalRepository implements TaskRepository, AlarmRepository {
     if (_prefs == null) {
       await SharedPreferencesWithCache.create(
         cacheOptions: const SharedPreferencesWithCacheOptions(
-          allowList: <String>{_tasksKey, _taskOrderKey},
+          allowList: <String>{
+            _tasksKey,
+            _taskOrderKey,
+            _alarmsKey,
+            _locationsKey,
+          },
         ),
       ).then((value) => _prefs = value);
     }
@@ -29,6 +38,13 @@ class LocalRepository implements TaskRepository, AlarmRepository {
     _prefs!.setStringList(_tasksKey, jsonList);
   }
 
+  Future<void> _saveLocations(List<Location> locations) async {
+    final jsonList = locations
+        .map<String>((e) => jsonEncode(e.toJson()))
+        .toList();
+    _prefs!.setStringList(_locationsKey, jsonList);
+  }
+
   Future<void> _saveTaskOrder(List<String> taskOrder) async {
     final jsonList = taskOrder.map((e) => jsonEncode(e)).toList();
     return _prefs!.setStringList(_taskOrderKey, jsonList);
@@ -62,6 +78,13 @@ class LocalRepository implements TaskRepository, AlarmRepository {
     _saveAlarms(alarms);
   }
 
+  @override
+  Future<void> createLocation(Location location) async {
+    final locations = await loadLocations();
+    locations.add(location);
+    _saveLocations(locations);
+  }
+
   // Read
 
   @override
@@ -83,6 +106,15 @@ class LocalRepository implements TaskRepository, AlarmRepository {
     return jsonList.map<Alarm>((e) => Alarm.fromJson(jsonDecode(e))).toList();
   }
 
+  @override
+  Future<List<Location>> loadLocations() async {
+    final Iterable<String> jsonList =
+        _prefs!.getStringList(_locationsKey) ?? [];
+    return jsonList
+        .map<Location>((e) => Location.fromJson(jsonDecode(e)))
+        .toList();
+  }
+
   // Update
 
   @override
@@ -106,6 +138,14 @@ class LocalRepository implements TaskRepository, AlarmRepository {
     _saveAlarms(alarms);
   }
 
+  @override
+  Future<void> updateLocation(Location location) async {
+    final locations = await loadLocations();
+    locations.remove(location);
+    locations.add(location);
+    _saveLocations(locations);
+  }
+
   // Delete
 
   @override
@@ -128,4 +168,11 @@ class LocalRepository implements TaskRepository, AlarmRepository {
     alarms.remove(alarm);
     _saveAlarms(alarms);
   }
+
+  @override
+  Future<void> deleteLocation(Location location) async {
+    final locations = await loadLocations();
+    locations.remove(location);
+    _saveLocations(locations);
+  }
 }

lib/pages/task_edit_page.dart

@@ -3,7 +3,7 @@ import 'package:flutter/material.dart';
 import '../model/callback_models/create_task_request.dart';
 import '../model/extensions/controller_context.dart';
 import '../model/task.dart';
-import '../service/task_controller.dart';
+import '../service/controllers/task_controller.dart';
 import '../service/tools.dart';
 import '../widgets/time_selector.dart';
 

lib/pages/task_overview_page.dart

@@ -3,8 +3,9 @@ import 'package:flutter/material.dart';
 import '../model/callback_models/create_task_request.dart';
 import '../model/extensions/controller_context.dart';
 import '../model/task.dart';
-import '../service/task_controller.dart';
+import '../service/controllers/task_controller.dart';
 import '../service/tools.dart';
+import '../widgets/task_dismissible.dart';
 import 'task_edit_page.dart';
 
 class TaskOverviewPage extends StatefulWidget {
@@ -37,8 +38,11 @@ class _TaskOverviewPageState extends State<TaskOverviewPage> {
   Widget itemBuilder(BuildContext context, int index) {
     final task = tasks.elementAt(index);
 
-    return ListTile(
+    return TaskDismissible(
       key: Key(task.id),
+      onDismissedRight: () =>
+          context.controller<TaskController>().deleteTask(task),
+      child: ListTile(
         title: Text(task.title),
         subtitle: task.description.isNotEmpty ? Text(task.description) : null,
         trailing: Checkbox(
@@ -55,6 +59,7 @@ class _TaskOverviewPageState extends State<TaskOverviewPage> {
             );
           }
         },
+      ),
     );
   }
 

lib/service/controllers/alarm_controller.dart

@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
 
-import '../model/alarm.dart';
-import '../model/repositories/interfaces/alarm_repository.dart';
+import '../../model/alarm.dart';
+import '../../model/repositories/interfaces/alarm_repository.dart';
 
 class AlarmController extends ChangeNotifier {
   AlarmController(AlarmRepository repository) : _repository = repository {

lib/service/controllers/location_controller.dart

@@ -0,0 +1,33 @@
+import 'package:flutter/material.dart';
+
+import '../../model/location.dart';
+import '../../model/repositories/interfaces/location_repository.dart';
+
+class LocationController extends ChangeNotifier {
+  LocationController(LocationRepository repository) : _repository = repository {
+    _loadLocations();
+  }
+
+  final LocationRepository _repository;
+
+  final List<Location> _locations = [];
+
+  Future<void> addLocation(Location location) {
+    _locations.add(location);
+    notifyListeners();
+    return _repository.createLocation(location);
+  }
+
+  Future<void> deleteLocation(Location location) {
+    _locations.remove(location);
+    notifyListeners();
+    return _repository.deleteLocation(location);
+  }
+
+  Future<void> _loadLocations() {
+    _locations.clear();
+    return _repository.loadLocations().then(
+      (value) => _locations.addAll(value),
+    );
+  }
+}

lib/service/controllers/task_controller.dart

@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart' show ChangeNotifier;
 
-import '../model/repositories/interfaces/task_repository.dart';
-import '../model/task.dart';
+import '../../model/repositories/interfaces/task_repository.dart';
+import '../../model/task.dart';
 
 class TaskController extends ChangeNotifier {
   TaskController(TaskRepository repository) : _repository = repository {

lib/widgets/task_dismissible.dart

@@ -0,0 +1,41 @@
+import 'package:flutter/material.dart';
+
+class TaskDismissible extends StatelessWidget {
+  const TaskDismissible({
+    required super.key,
+    this.onDismissedRight,
+    required this.child,
+  });
+
+  final VoidCallback? onDismissedRight;
+  final Widget child;
+
+  @override
+  Widget build(BuildContext context) {
+    return Dismissible(
+      key: key!,
+      direction: DismissDirection.startToEnd,
+      background: Container(
+        color: Theme.of(context).colorScheme.error,
+        child: Align(
+          alignment: AlignmentGeometry.centerLeft,
+          child: Padding(
+            padding: const EdgeInsets.symmetric(horizontal: 12),
+            child: Icon(
+              Icons.delete,
+              color: Theme.of(context).colorScheme.onError,
+            ),
+          ),
+        ),
+      ),
+      onDismissed: onDismissed,
+      child: child,
+    );
+  }
+
+  void onDismissed(DismissDirection direction) {
+    if (direction == DismissDirection.startToEnd && onDismissedRight != null) {
+      onDismissedRight!();
+    }
+  }
+}