shlist

commit 9d11ff537d45288de5dcd4269fa27ee2918bde6e
parent da76b2c388cb21781c6b75ad4074be5e4ec9f5c5
Author: Kyle Milz <kyle@0x30.net>
Date:   Sun, 24 Jan 2016 18:46:57 -0700

sl: trim down list_update message handler

- don't send back anything in response to a successful list_update message
- remove extra debugging code that was snuck in the last commit

Diffstat:
M
server/sl
 | 
12
++----------

1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/server/sl b/server/sl
@@ -275,25 +275,17 @@ sub msg_list_update {
 		return make_error("client tried to update a list it was not in");
 	}
 
-	# Do the update
+	# Update list row, note that some values here can be optional
 	$db->{update_list}->execute($list->{name}, $list->{date}, time, $list->{num});
 	$log->print("num  = $list->{num}\n");
 	$log->print("name = '$list->{name}'\n") if (exists $list->{name});
 	$log->print("date = $list->{date}\n") if (exists $list->{date});
 
-	$db->{list_select}->execute($list->{num});
-	my ($num, $name, $date) = $db->{list_select}->fetchrow_array();
-
 	# XXX: send push notifications to all my mutual friends to update their
 	# 'other lists' section, also tell all list members to update their
 	# 'lists' section
 
-	my $resp_list = {
-		num => $list->{num},
-		name => $list->{name},
-		date => $list->{date},
-	};
-	return make_ok( { list => $resp_list } );
+	return make_ok();
 }
 
 sub msg_list_item_add {