summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorZach Copley <zach@status.net>2009-12-03 17:06:58 -0800
committerZach Copley <zach@status.net>2009-12-03 17:06:58 -0800
commitaef4cc0a59276938f0f0aec4d67374f578f2117a (patch)
tree46b429f939a680926fc01fe2defed6e037985ec9 /lib
parentc89b10ffe4adb1df724b6a7c5c31b42c7dd3376b (diff)
Make it impossible to block (and thus unsubscribe from your
self-subscription) via the API. Additionally, make it impossible to block yourself or unsubscribe from yourself, period. I also made User use the subs.php helper function for unsubscribing during a block. Hopefully, these changes will get rid of the problem of people accidentally deleting their self-subscriptions once and for all (knock on wood).
Diffstat (limited to 'lib')
-rw-r--r--lib/subs.php6
1 files changed, 6 insertions, 0 deletions
diff --git a/lib/subs.php b/lib/subs.php
index 2fc3160de..4b6b03967 100644
--- a/lib/subs.php
+++ b/lib/subs.php
@@ -127,6 +127,12 @@ function subs_unsubscribe_to($user, $other)
if (!$user->isSubscribed($other))
return _('Not subscribed!');
+ // Don't allow deleting self subs
+
+ if ($user->id == $other->id) {
+ return _('Couldn\'t delete self-subscription.');
+ }
+
$sub = DB_DataObject::factory('subscription');
$sub->subscriber = $user->id;