fix updating only done state or only item descriptiom

[lsl.git] / lib / App / LazyShoppingList / API / v1.pm

diff --git a/lib/App/LazyShoppingList/API/v1.pm b/lib/App/LazyShoppingList/API/v1.pm
index d09bc647182cb1265de68d08b1a3904bc08f6552..185c0947ba85960a13db52b2de72e60ede8a7d21 100644 (file)
--- a/lib/App/LazyShoppingList/API/v1.pm
+++ b/lib/App/LazyShoppingList/API/v1.pm
@@ -166,7 +166,8 @@ put '/list/:list_id/:item_id' => sub {
     my $req = request_data;
 
     my $descr = $req->{description};
-    my $done  = JSON->boolean( $req->{done} // 0 );
+    my $done =
+        exists $req->{done} ? JSON->boolean( $req->{done} // 0 ) : undef;
     my $version = $req->{version};
 
     length($version) and $version =~ /^\d{1,18}$/
@@ -213,7 +214,7 @@ put '/list/:list_id/:item_id' => sub {
     return \%r;
 };
 
-# modify shipping list
+# modify shopping list
 put '/list/:list_id' => sub {
     my $list_id = route_parameters->get('list_id');
     length($list_id) and $list_id =~ /^\d{1,18}$/
@@ -274,4 +275,36 @@ del '/list/:list_id' => sub {
     return \%r;
 };
 
+# delete shopping list item
+del '/list/:list_id/:item_id' => sub {
+    my $list_id = route_parameters->get('list_id');
+    length($list_id) and $list_id =~ /^\d{1,18}$/
+        or return invalid_input('bad list ID');
+
+    my $item_id = route_parameters->get('item_id');
+    length($item_id) and $item_id =~ /^\d{1,18}$/
+        or return invalid_input('bad item ID');
+
+    my $dbh = get_database;
+
+    my %r = (
+        lists_version => get_lists_version($dbh),
+    );
+
+    my $list = $dbh->resultset('ShoppingList')->find($list_id)
+        or return exception 404, 'No such list';
+
+    my $item = $dbh->resultset('ShoppingListItem')
+        ->find( { shopping_list => $list->id, id => $item_id } );
+
+    if ($item) {
+        $item->delete;
+        $list->update({version => $list->version + 1});
+    }
+
+    $r{list_version} = $list->version;
+
+    return \%r;
+};
+
 true;