Skip to content

Commit a52b8bf

Browse files
committed
refactor: update activity deletion method to return a Future and simplify favorites handling
1 parent 1a4054c commit a52b8bf

File tree

2 files changed

+6
-16
lines changed

2 files changed

+6
-16
lines changed

lib/activity/data/datasource/activity_local_datasource.dart

+3-3
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,13 @@ abstract class ActivityLocalDataSource {
99

1010
class ActivityLocalDataSourceImpl implements ActivityLocalDataSource {
1111
@override
12-
Future<void> addActivity( activityCompanion) {
12+
Future<void> addActivity(activityCompanion) {
1313
return appDatabase.into(appDatabase.activity).insert(activityCompanion);
1414
}
1515

1616
@override
17-
void deleteActivity(int id) {
18-
return appDatabase.delete(appDatabase.activity).where((tbl) => tbl.id.equals(id));
17+
Future<int> deleteActivity(int id) {
18+
return (appDatabase.delete(appDatabase.activity)..where((tbl) => tbl.id.equals(id))).go();
1919
}
2020

2121
@override

lib/presentation/favorites.dart

+3-13
Original file line numberDiff line numberDiff line change
@@ -55,21 +55,11 @@ class _FavoritesState extends State<Favorites> {
5555
trailing: Row(
5656
mainAxisSize: MainAxisSize.min,
5757
children: [
58-
// IconButton(
59-
// onPressed: () {
60-
//
61-
// },
62-
// icon: Icon(Icons.done),
63-
// ),
6458
IconButton(
6559
icon: const Icon(Icons.cancel_outlined),
66-
onPressed: () {
67-
activitiesModel.removeActivity(activity.key);
68-
69-
setState(() {
70-
favorites.remove(activity);
71-
});
72-
// _updateFavorites(favorites);
60+
onPressed: () async {
61+
activityLocalDataSource.deleteActivity(activity.id!);
62+
_getFavorites();
7363
},
7464
),
7565
],

0 commit comments

Comments
 (0)