Let developers remove themselves from apps.

reddit-archive · Aug 22, 2012 · 6ca678e · 6ca678e
1 parent 432a0c3
commit 6ca678e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/r2/r2/controllers/api.py b/r2/r2/controllers/api.py
@@ -2912,12 +2912,15 @@ def POST_adddeveloper(self, form, jquery, client, account):
     @validatedForm(VUser(),
                    VModhash(),
                    client=VOAuth2ClientDeveloper(),
-                   account=VExistingUnameNotSelf('name'))
+                   account=VExistingUname('name'))
     @api_doc(api_section.apps)
     def POST_removedeveloper(self, form, jquery, client, account):
         if client and account and not form.has_errors('name'):
             client.remove_developer(account)
-            jquery('li#app-dev-%s-%s' % (client._id, account._id)).fadeOut()
+            if account._id == c.user._id:
+                jquery('#developed-app-%s' % client._id).fadeOut()
+            else:
+                jquery('li#app-dev-%s-%s' % (client._id, account._id)).fadeOut()
 
     @noresponse(VUser(),
                 VModhash(),

diff --git a/r2/r2/templates/prefapps.html b/r2/r2/templates/prefapps.html
@@ -32,10 +32,9 @@
     ${dev.name}&#32;
     %if c.user == dev:
       <span class="gray">${_("(that's you!)")}</span>&#32;
-    %else:
-      ${ajax_ynbutton(_("remove"), "removedeveloper",
-                      hidden_data=dict(client_id=app._id, name=dev.name))}
     %endif
+    ${ajax_ynbutton(_("remove"), "removedeveloper",
+                    hidden_data=dict(client_id=app._id, name=dev.name))}
   </li>
 </%def>