]> git.mxchange.org Git - quix0rs-gnu-social.git/commitdiff
Cosmetic changes to common_redirect, clientError, serverError
authorMikael Nordfeldth <mmn@hethane.se>
Sun, 9 Mar 2014 23:19:35 +0000 (00:19 +0100)
committerMikael Nordfeldth <mmn@hethane.se>
Sun, 9 Mar 2014 23:25:57 +0000 (00:25 +0100)
Since these functions exit (or throw exception) after running, there
is no need to have a 'return' statement or similar afterwards.

132 files changed:
actions/addpeopletag.php
actions/allrss.php
actions/apiaccountupdatedeliverydevice.php
actions/apiaccountupdateprofile.php
actions/apiaccountverifycredentials.php
actions/apifriendshipsshow.php
actions/apimediaupload.php
actions/apioauthaccesstoken.php
actions/apistatusesshow.php
actions/approvegroup.php
actions/approvesub.php
actions/atompubshowsubscription.php
actions/atompubsubscriptionfeed.php
actions/attachment.php
actions/avatarsettings.php
actions/block.php
actions/blockedfromgroup.php
actions/cancelgroup.php
actions/cancelsubscription.php
actions/confirmaddress.php
actions/deleteapplication.php
actions/deletegroup.php
actions/deleteuser.php
actions/editapplication.php
actions/editgroup.php
actions/editpeopletag.php
actions/emailsettings.php
actions/favoritesrss.php
actions/foaf.php
actions/foafgroup.php
actions/getfile.php
actions/grantrole.php
actions/groupblock.php
actions/groupbyid.php
actions/grouplogo.php
actions/groupqueue.php
actions/grouprss.php
actions/groupunblock.php
actions/imsettings.php
actions/joingroup.php
actions/leavegroup.php
actions/makeadmin.php
actions/microsummary.php
actions/nudge.php
actions/oauthappssettings.php
actions/oauthconnectionssettings.php
actions/otp.php
actions/passwordsettings.php
actions/peopletag.php
actions/peopletagautocomplete.php
actions/peopletagged.php
actions/peopletagsbyuser.php
actions/peopletagsforuser.php
actions/peopletagsubscribers.php
actions/peopletagsubscriptions.php
actions/pluginenable.php
actions/profilecompletion.php
actions/profiletagbyid.php
actions/public.php
actions/recoverpassword.php
actions/redirecturl.php
actions/register.php
actions/removepeopletag.php
actions/repeat.php
actions/replies.php
actions/repliesrss.php
actions/revokerole.php
actions/rsd.php
actions/sandbox.php
actions/showapplication.php
actions/showfavorites.php
actions/showmessage.php
actions/shownotice.php
actions/showprofiletag.php
actions/silence.php
actions/siteadminpanel.php
actions/smssettings.php
actions/subedit.php
actions/subqueue.php
actions/subscribe.php
actions/subscribepeopletag.php
actions/tag.php
actions/tagprofile.php
actions/tagrss.php
actions/unblock.php
actions/unsandbox.php
actions/unsilence.php
actions/unsubscribe.php
actions/unsubscribepeopletag.php
actions/urlsettings.php
actions/usergroups.php
index.php
lib/adminpanelaction.php
lib/apiaction.php
lib/apilistusers.php
lib/galleryaction.php
lib/groupaction.php
lib/mailbox.php
lib/profileformaction.php
lib/settingsaction.php
plugins/AnonymousFave/actions/anondisfavor.php
plugins/AnonymousFave/actions/anonfavor.php
plugins/Bookmark/BookmarkPlugin.php
plugins/Bookmark/actions/apitimelinebookmarks.php
plugins/Bookmark/actions/bookmarks.php
plugins/Bookmark/actions/bookmarksrss.php
plugins/CasAuthentication/actions/caslogin.php
plugins/EmailRegistration/actions/emailregister.php
plugins/Event/actions/timelist.php
plugins/ExtendedProfile/actions/profiledetailsettings.php
plugins/FacebookBridge/FacebookBridgePlugin.php
plugins/FacebookBridge/actions/facebookfinishlogin.php
plugins/FacebookBridge/actions/facebooksettings.php
plugins/GNUsocialProfileExtensions/actions/profilefields.php
plugins/GroupPrivateMessage/actions/groupinbox.php
plugins/GroupPrivateMessage/actions/newgroupmessage.php
plugins/Irc/lib/ircmanager.php
plugins/Mapstraction/actions/map.php
plugins/Minify/actions/minify.php
plugins/OStatus/OStatusPlugin.php
plugins/OStatus/actions/groupsalmon.php
plugins/OStatus/actions/ostatustag.php
plugins/OpenID/OpenIDPlugin.php
plugins/OpenID/actions/finishopenidlogin.php
plugins/OpenID/actions/openidserver.php
plugins/OpenID/actions/openidtrust.php
plugins/SearchSub/actions/searchsub.php
plugins/SubMirror/actions/basemirror.php
plugins/TagSub/actions/tagsub.php
plugins/TwitterBridge/actions/twitterauthorization.php
plugins/TwitterBridge/actions/twittersettings.php
plugins/UserFlag/actions/adminprofileflag.php

index 1c0ce55a2b40f20c132c7faa0e74237d4e6cc651..b501ce0fd9679b245d36f300164f7b433e9cce26 100644 (file)
@@ -77,7 +77,6 @@ class AddpeopletagAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         // Only for logged-in users
@@ -87,7 +86,6 @@ class AddpeopletagAction extends Action
         if (empty($this->user)) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
 
         // Profile to subscribe to
@@ -99,7 +97,6 @@ class AddpeopletagAction extends Action
         if (empty($this->tagged)) {
             // TRANS: Client error displayed trying to perform an action related to a non-existing profile.
             $this->clientError(_('No such profile.'));
-            return false;
         }
 
         $id = $this->arg('peopletag_id');
@@ -108,7 +105,6 @@ class AddpeopletagAction extends Action
         if (empty($this->peopletag)) {
             // TRANS: Client error displayed trying to reference a non-existing list.
             $this->clientError(_('No such list.'));
-            return false;
         }
 
         return true;
@@ -144,7 +140,6 @@ class AddpeopletagAction extends Action
                                       'The remote server is probably not responding correctly. ' .
                                       'Please try retrying later.'), $this->profile->profileurl));
             }
-            return false;
         }
         if ($this->boolean('ajax')) {
             $this->startHTML('text/xml;charset=utf-8');
index 637352bf4b32f28da288f0a147dea703c48d80cd..fee52c79ab3c7301f307865c6504b12ec39d6d5e 100644 (file)
@@ -67,7 +67,6 @@ class AllrssAction extends Rss10Action
         if (!$this->user) {
             // TRANS: Client error when user not found for an rss related action.
             $this->clientError(_('No such user.'));
-            return false;
         } else {
             $this->notices = $this->getNotices($this->limit);
             return true;
index 425624707ae218995e18b81ad94139cfb872c7b5..a3cbb418b65d60c899ca865fbbe9a34efcbc69e1 100644 (file)
@@ -84,7 +84,6 @@ class ApiAccountUpdateDeliveryDeviceAction extends ApiAuthAction
                 404,
                 $this->format
             );
-            return;
         }
 
         // Note: Twitter no longer supports IM
@@ -93,7 +92,6 @@ class ApiAccountUpdateDeliveryDeviceAction extends ApiAuthAction
             // TRANS: Client error displayed when no valid device parameter is provided for a user's delivery device setting.
             $this->clientError(_( 'You must specify a parameter named ' .
                                   '\'device\' with a value of one of: sms, im, none.' ));
-            return;
         }
 
         if (empty($this->user)) {
@@ -124,7 +122,6 @@ class ApiAccountUpdateDeliveryDeviceAction extends ApiAuthAction
             common_log_db_error($this->user, 'UPDATE', __FILE__);
             // TRANS: Server error displayed when a user's delivery device cannot be updated.
             $this->serverError(_('Could not update user.'));
-            return;
         }
 
         $profile = $this->user->getProfile();
index 0e4a4fb376d42314dff12e11bfa3f089ac90c869..60fed1e3c58215a6fd1ee98640615b4984d128e4 100644 (file)
@@ -92,7 +92,6 @@ class ApiAccountUpdateProfileAction extends ApiAuthAction
         if (empty($profile)) {
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->clientError(_('User has no profile.'));
-            return;
         }
 
         $original = clone($profile);
@@ -128,7 +127,6 @@ class ApiAccountUpdateProfileAction extends ApiAuthAction
             common_log_db_error($profile, 'UPDATE', __FILE__);
             // TRANS: Server error displayed if a user profile could not be saved.
             $this->serverError(_('Could not save profile.'));
-            return;
         }
 
         $twitter_user = $this->twitterUserArray($profile, true);
index ccb3bd7a600258da358ffd5d6fb51a2f1c27fa7d..6cc61cf6da7f62bd4ea300ac910e4af10dd742d9 100644 (file)
@@ -64,7 +64,6 @@ class ApiAccountVerifyCredentialsAction extends ApiAuthAction
         if (!in_array($this->format, array('xml', 'json'))) {
             // TRANS: Client error displayed when coming across a non-supported API method.
             $this->clientError(_('API method not found.'), $code = 404);
-            return;
         }
 
         $twitter_user = $this->twitterUserArray($this->auth_user->getProfile(), true);
index 58e610ecdb529f8dec5d5ba10e6d2321cf2f4831..5810e148747452ba808ccc4319429b56fd0b3ca7 100644 (file)
@@ -120,7 +120,6 @@ class ApiFriendshipsShowAction extends ApiBareAuthAction
         if (!in_array($this->format, array('xml', 'json'))) {
             // TRANS: Client error displayed when coming across a non-supported API method.
             $this->clientError(_('API method not found.'), 404);
-            return;
         }
 
         if (empty($this->source)) {
@@ -129,7 +128,6 @@ class ApiFriendshipsShowAction extends ApiBareAuthAction
                 _('Could not determine source user.'),
                 404
              );
-            return;
         }
 
         if (empty($this->target)) {
@@ -138,7 +136,6 @@ class ApiFriendshipsShowAction extends ApiBareAuthAction
                 _('Could not find target user.'),
                 404
             );
-            return;
         }
 
         $result = $this->twitterRelationshipArray($this->source, $this->target);
index b4f8709fe23071d82b283c36b1f90f9ee602c395..6f8781fab5b57b86b8bfd0e30485ae18b414cece 100644 (file)
@@ -63,7 +63,6 @@ class ApiMediaUploadAction extends ApiAuthAction
                 _('This method requires a POST.'),
                 400, $this->format
             );
-            return;
         }
 
         // Workaround for PHP returning empty $_POST and $_FILES when POST
@@ -79,7 +78,6 @@ class ApiMediaUploadAction extends ApiAuthAction
                       'The server was unable to handle that much POST data (%s bytes) due to its current configuration.',
                       intval($_SERVER['CONTENT_LENGTH']));
             $this->clientError(sprintf($msg, $_SERVER['CONTENT_LENGTH']));
-            return;
         }
 
         $upload = null;
@@ -88,7 +86,6 @@ class ApiMediaUploadAction extends ApiAuthAction
             $upload = MediaFile::fromUpload('media', $this->auth_user->getProfile());
         } catch (Exception $e) {
             $this->clientError($e->getMessage(), $e->getCode());
-            return;
         }
 
         if (isset($upload)) {
@@ -96,7 +93,6 @@ class ApiMediaUploadAction extends ApiAuthAction
         } else {
             // TRANS: Client error displayed when uploading a media file has failed.
             $this->clientError(_('Upload failed.'));
-            return;
         }
     }
 
index 9379834d656eec79876f5ef3a933c1b8c45c3007..a1a70a9b9e2be36aa949c163b1a142cca48863a9 100644 (file)
@@ -80,7 +80,6 @@ class ApiOAuthAccessTokenAction extends ApiOAuthAction
             common_debug(var_export($req, true));
             $code = $e->getCode();
             $this->clientError($e->getMessage(), empty($code) ? 401 : $code, 'text');
-            return;
         }
 
         if (empty($atok)) {
index 67d04a505ce76981bffdd91ec6ebc93d208c8536..3a60b87637e571dd27a6c5653441b59afa380f57 100644 (file)
@@ -101,7 +101,6 @@ class ApiStatusesShowAction extends ApiPrivateAuthAction
         if (!in_array($this->format, array('xml', 'json', 'atom'))) {
             // TRANS: Client error displayed when coming across a non-supported API method.
             $this->clientError(_('API method not found.'), 404);
-            return;
         }
 
         switch ($_SERVER['REQUEST_METHOD']) {
@@ -114,7 +113,6 @@ class ApiStatusesShowAction extends ApiPrivateAuthAction
         default:
             // TRANS: Client error displayed calling an unsupported HTTP error in API status show.
             $this->clientError(_('HTTP method not supported.'), 405);
-            return;
         }
     }
 
@@ -223,7 +221,6 @@ class ApiStatusesShowAction extends ApiPrivateAuthAction
         if ($this->format != 'atom') {
             // TRANS: Client error displayed when trying to delete a notice not using the Atom format.
             $this->clientError(_('Can only delete using the Atom format.'));
-            return;
         }
 
         if (empty($this->auth_user) ||
@@ -231,7 +228,6 @@ class ApiStatusesShowAction extends ApiPrivateAuthAction
              !$this->auth_user->hasRight(Right::DELETEOTHERSNOTICE))) {
             // TRANS: Client error displayed when a user has no rights to delete notices of other users.
             $this->clientError(_('Cannot delete this notice.'), 403);
-            return;
         }
 
         if (Event::handle('StartDeleteOwnNotice', array($this->auth_user, $this->notice))) {
index db64e0596273c414b0815ebfbfc671dc707c076c..2c8b354d347539bba7a630b81a640ef09e6bdf1a 100644 (file)
@@ -57,7 +57,6 @@ class ApprovegroupAction extends Action
         if (!common_logged_in()) {
             // TRANS: Client error displayed when trying to leave a group while not logged in.
             $this->clientError(_('You must be logged in to leave a group.'));
-            return false;
         }
 
         $nickname_arg = $this->trimmed('nickname');
@@ -72,7 +71,6 @@ class ApprovegroupAction extends Action
             if ($nickname_arg != $nickname) {
                 $args = array('nickname' => $nickname);
                 common_redirect(common_local_url('leavegroup', $args), 301);
-                return false;
             }
 
             $local = Local_group::getKV('nickname', $nickname);
@@ -80,27 +78,23 @@ class ApprovegroupAction extends Action
             if (!$local) {
                 // TRANS: Client error displayed when trying to leave a non-local group.
                 $this->clientError(_('No such group.'), 404);
-                return false;
             }
 
             $this->group = User_group::getKV('id', $local->group_id);
         } else {
             // TRANS: Client error displayed when trying to leave a group without providing a group name or group ID.
             $this->clientError(_('No nickname or ID.'), 404);
-            return false;
         }
 
         if (!$this->group) {
             // TRANS: Client error displayed when trying to leave a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
         if (empty($cur)) {
             // TRANS: Client error displayed trying to approve group membership while not logged in.
             $this->clientError(_('Must be logged in.'), 403);
-            return false;
         }
         if ($this->arg('profile_id')) {
             if ($cur->isAdmin($this->group)) {
@@ -108,12 +102,10 @@ class ApprovegroupAction extends Action
             } else {
                 // TRANS: Client error displayed trying to approve group membership while not a group administrator.
                 $this->clientError(_('Only group admin can approve or cancel join requests.'), 403);
-                return false;
             }
         } else {
             // TRANS: Client error displayed trying to approve group membership without specifying a profile to approve.
             $this->clientError(_('Must specify a profile.'));
-            return false;
         }
 
         $this->request = Group_join_queue::pkeyGet(array('profile_id' => $this->profile->id,
@@ -186,9 +178,7 @@ class ApprovegroupAction extends Action
             $this->elementEnd('body');
             $this->endHTML();
         } else {
-            common_redirect(common_local_url('groupmembers', array('nickname' =>
-                                                                   $this->group->nickname)),
-                            303);
+            common_redirect(common_local_url('groupmembers', array('nickname' => $this->group->nickname)), 303);
         }
     }
 }
index b92ad66e7ae16d65fc13fe61082dc90f3973e7f4..c9fa804412a97b4abd0669c38be051b77cc81a64 100644 (file)
@@ -58,14 +58,12 @@ class ApprovesubAction extends Action
         if (empty($cur)) {
             // TRANS: Client error displayed trying to approve group membership while not logged in.
             $this->clientError(_('Must be logged in.'), 403);
-            return false;
         }
         if ($this->arg('profile_id')) {
             $this->profile = Profile::getKV('id', $this->arg('profile_id'));
         } else {
             // TRANS: Client error displayed trying to approve subscriptionswithout specifying a profile to approve.
             $this->clientError(_('Must specify a profile.'));
-            return false;
         }
 
         $this->request = Subscription_queue::pkeyGet(array('subscriber' => $this->profile->id,
index ec3949c4d839deb54d44493ef20cf34150ea39c7..04681f3445133a5a8a5f11470a95a5fbabb77c9d 100644 (file)
@@ -118,7 +118,6 @@ class AtompubshowsubscriptionAction extends ApiAuthAction
         default:
             // TRANS: Client error shown when using a non-supported HTTP method.
             $this->clientError(_('HTTP method not supported.'), 405);
-            return;
         }
 
         return;
index 1b9f4b98a573be7fe9b4545b4159eac01df3ef7f..5c7184e5c1da365bdf9bb1b24aa1403f65a0f000 100644 (file)
@@ -105,7 +105,6 @@ class AtompubsubscriptionfeedAction extends ApiAuthAction
         default:
             // TRANS: Client exception thrown when using an unsupported HTTP method.
             $this->clientError(_('HTTP method not supported.'), 405);
-            return;
         }
 
         return;
@@ -232,7 +231,6 @@ class AtompubsubscriptionfeedAction extends ApiAuthAction
             $dom->documentElement->localName != 'entry') {
             // TRANS: Client error displayed when not using an Atom entry.
             $this->clientError(_('Atom post must be an Atom entry.'));
-            return;
         }
 
         $activity = new Activity($dom->documentElement);
@@ -244,7 +242,6 @@ class AtompubsubscriptionfeedAction extends ApiAuthAction
             if ($activity->verb != ActivityVerb::FOLLOW) {
                 // TRANS: Client error displayed when not using the follow verb.
                 $this->clientError(_('Can only handle Follow activities.'));
-                return;
             }
 
             $person = $activity->objects[0];
@@ -252,7 +249,6 @@ class AtompubsubscriptionfeedAction extends ApiAuthAction
             if ($person->type != ActivityObject::PERSON) {
                 // TRANS: Client exception thrown when subscribing to an object that is not a person.
                 $this->clientError(_('Can only follow people.'));
-                return;
             }
 
             // XXX: OStatus discovery (maybe)
@@ -263,7 +259,6 @@ class AtompubsubscriptionfeedAction extends ApiAuthAction
                 // TRANS: Client exception thrown when subscribing to a non-existing profile.
                 // TRANS: %s is the unknown profile ID.
                 $this->clientError(sprintf(_('Unknown profile %s.'), $person->id));
-                return;
             }
 
             if (Subscription::exists($this->_profile, $profile)) {
@@ -273,7 +268,6 @@ class AtompubsubscriptionfeedAction extends ApiAuthAction
                 $this->clientError(sprintf(_('Already subscribed to %s.'),
                                            $person->id),
                                    409);
-                return;
             }
 
             if (Subscription::start($this->_profile, $profile)) {
index 321525a55f1fb40b2b4c43f51d1c1adfced0e02c..56ecceb860e00cd31e51816abbbd43fa66fb8afc 100644 (file)
@@ -71,7 +71,6 @@ class AttachmentAction extends Action
         if (empty($this->attachment)) {
             // TRANS: Client error displayed trying to get a non-existing attachment.
             $this->clientError(_('No such attachment.'), 404);
-            return false;
         }
         return true;
     }
index dd8093b765c57af9599772ecdf90c8764b24385f..d3d5b310026416a6875a80caf38b6f062e9de0e6 100644 (file)
@@ -106,7 +106,6 @@ class AvatarsettingsAction extends SettingsAction
             common_log_db_error($user, 'SELECT', __FILE__);
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'));
-            return;
         }
 
         $this->elementStart('form', array('enctype' => 'multipart/form-data',
@@ -195,7 +194,6 @@ class AvatarsettingsAction extends SettingsAction
             common_log_db_error($user, 'SELECT', __FILE__);
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'));
-            return;
         }
 
         $this->elementStart('form', array('method' => 'post',
@@ -362,7 +360,6 @@ class AvatarsettingsAction extends SettingsAction
         if (!$filedata) {
             // TRANS: Server error displayed if an avatar upload went wrong somehow server side.
             $this->serverError(_('Lost our file data.'));
-            return;
         }
 
         $file_d = ($filedata['width'] > $filedata['height'])
index f195fb5a88ffc8663373feccf94898c6811b4253..7458caa6ac73200ea50e2be58c3fd97c2f9ac839 100644 (file)
@@ -66,7 +66,6 @@ class BlockAction extends ProfileFormAction
         if ($cur->hasBlocked($this->profile)) {
             // TRANS: Client error displayed when blocking a user that has already been blocked.
             $this->clientError(_('You already blocked that user.'));
-            return false;
         }
 
         return true;
@@ -187,7 +186,6 @@ class BlockAction extends ProfileFormAction
         if (!$result) {
             // TRANS: Server error displayed when blocking a user fails.
             $this->serverError(_('Failed to save block information.'));
-            return;
         }
     }
 
index a00c12ef583c755e883775485ec13d95369a2102..a2e7c5767f64be9610681f638be8ef899140ff0a 100644 (file)
@@ -65,13 +65,11 @@ class BlockedfromgroupAction extends GroupAction
                 $args['page'] = $this->page;
             }
             common_redirect(common_local_url('blockedfromgroup', $args), 301);
-            return false;
         }
 
         if (!$nickname) {
             // TRANS: Client error displayed when requesting a list of blocked users for a group without providing a group nickname.
             $this->clientError(_('No nickname.'), 404);
-            return false;
         }
 
         $local = Local_group::getKV('nickname', $nickname);
@@ -79,7 +77,6 @@ class BlockedfromgroupAction extends GroupAction
         if (!$local) {
             // TRANS: Client error displayed when requesting a list of blocked users for a non-local group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $this->group = User_group::getKV('id', $local->group_id);
@@ -87,7 +84,6 @@ class BlockedfromgroupAction extends GroupAction
         if (!$this->group) {
             // TRANS: Client error displayed when requesting a list of blocked users for a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         return true;
index 4196eb3ffc5e01356f79d948bbd7adae191e350f..93f630e0609123a59757bc96fc880279b1bdcffe 100644 (file)
@@ -57,7 +57,6 @@ class CancelgroupAction extends Action
         if (!common_logged_in()) {
             // TRANS: Client error displayed when trying to leave a group while not logged in.
             $this->clientError(_('You must be logged in to leave a group.'));
-            return false;
         }
 
         $nickname_arg = $this->trimmed('nickname');
@@ -71,7 +70,6 @@ class CancelgroupAction extends Action
             if ($nickname_arg != $nickname) {
                 $args = array('nickname' => $nickname);
                 common_redirect(common_local_url('leavegroup', $args), 301);
-                return false;
             }
 
             $local = Local_group::getKV('nickname', $nickname);
@@ -79,27 +77,23 @@ class CancelgroupAction extends Action
             if (!$local) {
                 // TRANS: Client error displayed when trying to leave a non-local group.
                 $this->clientError(_('No such group.'), 404);
-                return false;
             }
 
             $this->group = User_group::getKV('id', $local->group_id);
         } else {
             // TRANS: Client error displayed when trying to leave a group without providing a group name or group ID.
             $this->clientError(_('No nickname or ID.'), 404);
-            return false;
         }
 
         if (!$this->group) {
             // TRANS: Client error displayed when trying to leave a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
         if (empty($cur)) {
             // TRANS: Client error displayed when trying to leave a group while not logged in.
             $this->clientError(_('Must be logged in.'), 403);
-            return false;
         }
         if ($this->arg('profile_id')) {
             if ($cur->isAdmin($this->group)) {
@@ -108,7 +102,6 @@ class CancelgroupAction extends Action
                 // TRANS: Client error displayed when trying to approve or cancel a group join request without
                 // TRANS: being a group administrator.
                 $this->clientError(_('Only group admin can approve or cancel join requests.'), 403);
-                return false;
             }
         } else {
             $this->profile = $cur->getProfile();
@@ -164,9 +157,7 @@ class CancelgroupAction extends Action
             $this->elementEnd('body');
             $this->endHTML();
         } else {
-            common_redirect(common_local_url('groupmembers', array('nickname' =>
-                                                                   $this->group->nickname)),
-                            303);
+            common_redirect(common_local_url('groupmembers', array('nickname' => $this->group->nickname)), 303);
         }
     }
 }
index 590fde9f2ef309c5192a8a073b23637aec7466af..3c3e84df3f66636a1d14cbeeece8aa1b1e3bb6f9 100644 (file)
@@ -55,7 +55,6 @@ class CancelsubscriptionAction extends Action
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return;
         }
 
         $user = common_current_user();
@@ -63,7 +62,6 @@ class CancelsubscriptionAction extends Action
         if ($_SERVER['REQUEST_METHOD'] != 'POST') {
             common_redirect(common_local_url('subscriptions',
                                              array('nickname' => $user->nickname)));
-            return;
         }
 
         /* Use a session token for CSRF protection. */
@@ -74,7 +72,6 @@ class CancelsubscriptionAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token. ' .
                                  'Try again, please.'));
-            return;
         }
 
         $other_id = $this->arg('unsubscribeto');
@@ -82,7 +79,6 @@ class CancelsubscriptionAction extends Action
         if (!$other_id) {
             // TRANS: Client error displayed when trying to leave a group without specifying an ID.
             $this->clientError(_('No profile ID in request.'));
-            return;
         }
 
         $other = Profile::getKV('id', $other_id);
@@ -90,7 +86,6 @@ class CancelsubscriptionAction extends Action
         if (!$other) {
             // TRANS: Client error displayed when trying to leave a non-existing group.
             $this->clientError(_('No profile with that ID.'));
-            return;
         }
 
         $this->request = Subscription_queue::pkeyGet(array('subscriber' => $user->id,
index 4aae0675a2f219edcbdb507be264cf5b8f36d3c7..05ddce75a92e0d7f45663c487aba377970e0fb2c 100644 (file)
@@ -66,25 +66,21 @@ class ConfirmaddressAction extends Action
         if (!common_logged_in()) {
             common_set_returnto($this->selfUrl());
             common_redirect(common_local_url('login'));
-            return;
         }
         $code = $this->trimmed('code');
         if (!$code) {
             // TRANS: Client error displayed when not providing a confirmation code in the contact address confirmation action.
             $this->clientError(_('No confirmation code.'));
-            return;
         }
         $confirm = Confirm_address::getKV('code', $code);
         if (!$confirm) {
             // TRANS: Client error displayed when providing a non-existing confirmation code in the contact address confirmation action.
             $this->clientError(_('Confirmation code not found.'));
-            return;
         }
         $cur = common_current_user();
         if ($cur->id != $confirm->user_id) {
             // TRANS: Client error displayed when not providing a confirmation code for another user in the contact address confirmation action.
             $this->clientError(_('That confirmation code is not for you!'));
-            return;
         }
         $type = $confirm->address_type;
         $transports = array();
@@ -92,7 +88,6 @@ class ConfirmaddressAction extends Action
         if (!in_array($type, array('email', 'sms')) && !in_array($type, array_keys($transports))) {
             // TRANS: Server error for an unknown address type, which can be 'email', 'sms', or the name of an IM network (such as 'xmpp' or 'aim')
             $this->serverError(sprintf(_('Unrecognized address type %s'), $type));
-            return;
         }
         $this->address = $confirm->address;
         $cur->query('BEGIN');
@@ -101,7 +96,6 @@ class ConfirmaddressAction extends Action
             if ($cur->$type == $confirm->address) {
                 // TRANS: Client error for an already confirmed email/jabber/sms address.
                 $this->clientError(_('That address has already been confirmed.'));
-                return;
             }
 
             $orig_user = clone($cur);
@@ -120,7 +114,6 @@ class ConfirmaddressAction extends Action
                 common_log_db_error($cur, 'UPDATE', __FILE__);
                 // TRANS: Server error displayed when confirming an e-mail address or IM address fails.
                 $this->serverError(_('Could not update user.'));
-                return;
             }
 
             if ($type == 'email') {
@@ -136,7 +129,6 @@ class ConfirmaddressAction extends Action
                 if($user_im_prefs->screenname == $confirm->address){
                     // TRANS: Client error for an already confirmed IM address.
                     $this->clientError(_('That address has already been confirmed.'));
-                    return;
                 }
                 $user_im_prefs->screenname = $confirm->address;
                 $result = $user_im_prefs->update();
@@ -145,7 +137,6 @@ class ConfirmaddressAction extends Action
                     common_log_db_error($user_im_prefs, 'UPDATE', __FILE__);
                     // TRANS: Server error displayed when updating IM preferences fails.
                     $this->serverError(_('Could not update user IM preferences.'));
-                    return;
                 }
             }else{
                 $user_im_prefs = new User_im_prefs();
@@ -158,7 +149,6 @@ class ConfirmaddressAction extends Action
                     common_log_db_error($user_im_prefs, 'INSERT', __FILE__);
                     // TRANS: Server error displayed when adding IM preferences fails.
                     $this->serverError(_('Could not insert user IM preferences.'));
-                    return;
                 }
             }
 
@@ -171,7 +161,6 @@ class ConfirmaddressAction extends Action
             // TRANS: Server error displayed when an address confirmation code deletion from the
             // TRANS: database fails in the contact address confirmation action.
             $this->serverError(_('Could not delete address confirmation.'));
-            return;
         }
 
         $cur->query('COMMIT');
index 763d58c762992a2efddc493508b71dc773844bd6..5d7441098c8f73f1068491d5509208935626420e 100644 (file)
@@ -60,7 +60,6 @@ class DeleteapplicationAction extends Action
         if (!common_logged_in()) {
             // TRANS: Client error displayed trying to delete an application while not logged in.
             $this->clientError(_('You must be logged in to delete an application.'));
-            return false;
         }
 
         $id        = (int)$this->arg('id');
@@ -69,7 +68,6 @@ class DeleteapplicationAction extends Action
         if (empty($this->app)) {
             // TRANS: Client error displayed trying to delete an application that does not exist.
             $this->clientError(_('Application not found.'));
-            return false;
         }
 
         $cur = common_current_user();
@@ -77,7 +75,6 @@ class DeleteapplicationAction extends Action
         if ($cur->id != $this->app->owner) {
             // TRANS: Client error displayed trying to delete an application the current user does not own.
             $this->clientError(_('You are not the owner of this application.'), 401);
-            return false;
         }
 
         return true;
@@ -101,7 +98,6 @@ class DeleteapplicationAction extends Action
             if (!$token || $token != common_session_token()) {
                 // TRANS: Client error displayed when the session token does not match or is not given.
                 $this->clientError(_('There was a problem with your session token.'));
-                return;
             }
 
             if ($this->arg('no')) {
index 5efb5e6a919feb23b70806055b46be86a2f98958..6d9d66d30e1f61ec86c8397373a63106b24414ab 100644 (file)
@@ -62,7 +62,6 @@ class DeletegroupAction extends RedirectingAction
         if (!common_logged_in()) {
             // TRANS: Client error when trying to delete group while not logged in.
             $this->clientError(_('You must be logged in to delete a group.'));
-            return false;
         }
 
         $nickname_arg = $this->trimmed('nickname');
@@ -77,7 +76,6 @@ class DeletegroupAction extends RedirectingAction
             if ($nickname_arg != $nickname) {
                 $args = array('nickname' => $nickname);
                 common_redirect(common_local_url('leavegroup', $args), 301);
-                return false;
             }
 
             $local = Local_group::getKV('nickname', $nickname);
@@ -85,27 +83,23 @@ class DeletegroupAction extends RedirectingAction
             if (!$local) {
                 // TRANS: Client error when trying to delete a non-local group.
                 $this->clientError(_('No such group.'), 404);
-                return false;
             }
 
             $this->group = User_group::getKV('id', $local->group_id);
         } else {
             // TRANS: Client error when trying to delete a group without providing a nickname or ID for the group.
             $this->clientError(_('No nickname or ID.'), 404);
-            return false;
         }
 
         if (!$this->group) {
             // TRANS: Client error when trying to delete a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
         if (!$cur->hasRight(Right::DELETEGROUP)) {
             // TRANS: Client error when trying to delete a group without having the rights to delete it.
             $this->clientError(_('You are not allowed to delete this group.'), 403);
-            return false;
         }
 
         return true;
@@ -166,8 +160,7 @@ class DeletegroupAction extends RedirectingAction
         } else {
             // @fixme if we could direct to the page on which this group
             // would have shown... that would be awesome
-            common_redirect(common_local_url('groups'),
-                            303);
+            common_redirect(common_local_url('groups'), 303);
         }
     }
 
index 9f13468dc498643dd72da3da279d206e65fb0717..48ad9272e04b8a10f30caef25edadda02fdb6385 100644 (file)
@@ -64,7 +64,6 @@ class DeleteuserAction extends ProfileFormAction
         if (!$cur->hasRight(Right::DELETEUSER)) {
             // TRANS: Client error displayed when trying to delete a user without having the right to delete users.
             $this->clientError(_('You cannot delete users.'));
-            return false;
         }
 
         $this->user = User::getKV('id', $this->profile->id);
@@ -72,7 +71,6 @@ class DeleteuserAction extends ProfileFormAction
         if (empty($this->user)) {
             // TRANS: Client error displayed when trying to delete a non-local user.
             $this->clientError(_('You can only delete local users.'));
-            return false;
         }
 
         return true;
index 00ed3971935ed520bc132c470e832620894e1d06..c7e5f9052c77fc81271687f72fd09edd34496c18 100644 (file)
@@ -64,7 +64,6 @@ class EditApplicationAction extends Action
         if (!common_logged_in()) {
             // TRANS: Client error displayed trying to edit an application while not logged in.
             $this->clientError(_('You must be logged in to edit an application.'));
-            return false;
         }
 
         $id = (int)$this->arg('id');
@@ -81,7 +80,6 @@ class EditApplicationAction extends Action
         if (!$this->app) {
             // TRANS: Client error displayed trying to edit an application that does not exist.
             $this->clientError(_('No such application.'));
-            return false;
         }
 
         return true;
@@ -130,7 +128,6 @@ class EditApplicationAction extends Action
         if (!$token || $token != common_session_token()) {
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token.'));
-            return;
         }
 
         $cur = common_current_user();
index befbd55992941714d6da921c9ce80dacb215137e..7b499d00ffdff60819788b6b0adbcae0f8ba0fe1 100644 (file)
@@ -66,7 +66,6 @@ class EditgroupAction extends GroupAction
         if (!common_logged_in()) {
             // TRANS: Client error displayed trying to edit a group while not logged in.
             $this->clientError(_('You must be logged in to create a group.'));
-            return false;
         }
 
         $nickname_arg = $this->trimmed('nickname');
@@ -77,13 +76,11 @@ class EditgroupAction extends GroupAction
         if ($nickname_arg != $nickname) {
             $args = array('nickname' => $nickname);
             common_redirect(common_local_url('editgroup', $args), 301);
-            return false;
         }
 
         if (!$nickname) {
             // TRANS: Client error displayed trying to edit a group while not proving a nickname for the group to edit.
             $this->clientError(_('No nickname.'), 404);
-            return false;
         }
 
         $groupid = $this->trimmed('groupid');
@@ -100,7 +97,6 @@ class EditgroupAction extends GroupAction
         if (!$this->group) {
             // TRANS: Client error displayed trying to edit a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
@@ -108,7 +104,6 @@ class EditgroupAction extends GroupAction
         if (!$cur->isAdmin($this->group)) {
             // TRANS: Client error displayed trying to edit a group while not being a group admin.
             $this->clientError(_('You must be an admin to edit the group.'), 403);
-            return false;
         }
 
         return true;
@@ -274,9 +269,7 @@ class EditgroupAction extends GroupAction
         }
 
         if ($this->group->nickname != $orig->nickname) {
-            common_redirect(common_local_url('editgroup',
-                                             array('nickname' => $nickname)),
-                            303);
+            common_redirect(common_local_url('editgroup', array('nickname' => $nickname)), 303);
         } else {
             // TRANS: Group edit form success message.
             $this->showForm(_('Options saved.'));
index 72dcc0df723cca9ff664797551ef0be0045ea487..c812255c700fecb8a7d12f7e0f88b2353f652c5a 100644 (file)
@@ -67,7 +67,6 @@ class EditpeopletagAction extends Action
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
 
         $id = $this->arg('id');
@@ -88,7 +87,6 @@ class EditpeopletagAction extends Action
         if ($tagger_arg != $tagger || $tag_arg != $tag) {
             $args = array('tagger' => $tagger, 'tag' => $tag);
             common_redirect(common_local_url('editpeopletag', $args), 301);
-            return false;
         }
 
         $user = null;
@@ -101,7 +99,6 @@ class EditpeopletagAction extends Action
             if (!$tagger) {
                 // TRANS: Error message displayed when trying to perform an action that requires a tagging user or ID.
                 $this->clientError(_('No tagger or ID.'), 404);
-                return false;
             }
 
             $user = User::getKV('nickname', $tagger);
@@ -111,20 +108,17 @@ class EditpeopletagAction extends Action
         if (!$this->peopletag) {
             // TRANS: Client error displayed when referring to a non-existing list.
             $this->clientError(_('No such list.'), 404);
-            return false;
         }
 
         if (!$user) {
             // This should not be happening
             // TRANS: Client error displayed when referring to non-local user.
             $this->clientError(_('Not a local user.'), 404);
-            return false;
         }
 
         if ($current->id != $user->id) {
             // TRANS: Client error displayed when reting to edit a tag that was not self-created.
             $this->clientError(_('You must be the creator of the tag to edit it.'), 404);
-            return false;
         }
 
         $this->tagger = $user->getProfile();
@@ -311,9 +305,7 @@ class EditpeopletagAction extends Action
             // This might take quite a bit of time.
             $this->peopletag->delete();
             // send home.
-            common_redirect(common_local_url('all',
-                                         array('nickname' => $this->tagger->nickname)),
-                            303);
+            common_redirect(common_local_url('all', array('nickname' => $this->tagger->nickname)), 303);
         }
 
         if ($tag != $orig->tag) {
index f12aecd57d4436aba3454226129c55b5807435c7..ad16d4da4b5827be18947ef0497df89bb72fb7bb 100644 (file)
@@ -351,7 +351,6 @@ class EmailsettingsAction extends SettingsAction
                 common_log_db_error($user, 'UPDATE', __FILE__);
                 // TRANS: Server error thrown on database error updating e-mail preferences.
                 $this->serverError(_('Could not update user.'));
-                return;
             }
 
             $user->query('COMMIT');
@@ -419,7 +418,6 @@ class EmailsettingsAction extends SettingsAction
                 common_log_db_error($confirm, 'INSERT', __FILE__);
                 // TRANS: Server error thrown on database error adding e-mail confirmation code.
                 $this->serverError(_('Could not insert confirmation code.'));
-                return;
             }
 
             mail_confirm_address($user, $confirm->code, $user->nickname, $email);
@@ -463,7 +461,6 @@ class EmailsettingsAction extends SettingsAction
             common_log_db_error($confirm, 'DELETE', __FILE__);
             // TRANS: Server error thrown on database error canceling e-mail address confirmation.
             $this->serverError(_('Could not delete email confirmation.'));
-            return;
         }
 
         // TRANS: Message given after successfully canceling e-mail address confirmation.
@@ -502,7 +499,6 @@ class EmailsettingsAction extends SettingsAction
             common_log_db_error($user, 'UPDATE', __FILE__);
             // TRANS: Server error thrown on database error removing a registered e-mail address.
             $this->serverError(_('Could not update user.'));
-            return;
         }
         $user->query('COMMIT');
 
index 8b5ea548bec3e1193fd9e736699921ab152d086c..de901b0b4643a3cfac992e52bb62bce7b3e46ad8 100644 (file)
@@ -70,7 +70,6 @@ class FavoritesrssAction extends Rss10Action
         if (!$this->user) {
             // TRANS: Client error displayed when trying to get the RSS feed with favorites of a user that does not exist.
             $this->clientError(_('No such user.'));
-            return false;
         } else {
             $this->notices = $this->getNotices($this->limit);
             return true;
index e0662aa2fa43cc53ecd526573941e657897db18d..83f7d97d5883cabad98d736fe753db24a94e9b6d 100644 (file)
@@ -40,7 +40,6 @@ class FoafAction extends Action
         if (empty($nickname_arg)) {
             // TRANS: Client error displayed when requesting Friends of a Friend feed without providing a user nickname.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->nickname = common_canonical_nickname($nickname_arg);
@@ -51,7 +50,6 @@ class FoafAction extends Action
             common_redirect(common_local_url('foaf',
                                              array('nickname' => $this->nickname)),
                             301);
-            return false;
         }
 
         $this->user = User::getKV('nickname', $this->nickname);
@@ -59,7 +57,6 @@ class FoafAction extends Action
         if (!$this->user) {
             // TRANS: Client error displayed when requesting Friends of a Friend feed for an object that is not a user.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->profile = $this->user->getProfile();
@@ -67,7 +64,6 @@ class FoafAction extends Action
         if (!$this->profile) {
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'), 500);
-            return false;
         }
 
         return true;
index 34332c7e2978ee88f805606d640587355ddc1f66..f9c61ac5dcda9f7a409917ec569fc82e59d84af9 100644 (file)
@@ -44,7 +44,6 @@ class FoafGroupAction extends Action
         if (empty($nickname_arg)) {
             // TRANS: Client error displayed when requesting Friends of a Friend feed without providing a group nickname.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $this->nickname = common_canonical_nickname($nickname_arg);
@@ -63,7 +62,6 @@ class FoafGroupAction extends Action
         if (!$local) {
             // TRANS: Client error displayed when requesting Friends of a Friend feed for a non-local group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $this->group = User_group::getKV('id', $local->group_id);
@@ -71,7 +69,6 @@ class FoafGroupAction extends Action
         if (!$this->group) {
             // TRANS: Client error displayed when requesting Friends of a Friend feed for a nickname that is not a group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         common_set_returnto($this->selfUrl());
index 99ff84bcefe7edd0aa7aebbb26dd2c6ed0dc2a2c..f0c98f74167702ef6bcdc4daeffb2f5e1d67f055 100644 (file)
@@ -71,12 +71,10 @@ class GetfileAction extends Action
         if (empty($path) or !file_exists($path)) {
             // TRANS: Client error displayed when requesting a non-existent file.
             $this->clientError(_('No such file.'), 404);
-            return false;
         }
         if (!is_readable($path)) {
             // TRANS: Client error displayed when requesting a file without having read access to it.
             $this->clientError(_('Cannot read file.'), 403);
-            return false;
         }
 
         $this->path = $path;
index 36369a86002f3c89a13ddd1706661d7c6d2839d8..35f0dcf961d02fc328466cc163f56fbec3407fd7 100644 (file)
@@ -59,12 +59,10 @@ class GrantRoleAction extends ProfileFormAction
         if (!Profile_role::isValid($this->role)) {
             // TRANS: Client error displayed when trying to assign an invalid role to a user.
             $this->clientError(_('Invalid role.'));
-            return false;
         }
         if (!Profile_role::isSettable($this->role)) {
             // TRANS: Client error displayed when trying to assign an reserved role to a user.
             $this->clientError(_('This role is reserved and cannot be set.'));
-            return false;
         }
 
         $cur = common_current_user();
@@ -74,7 +72,6 @@ class GrantRoleAction extends ProfileFormAction
         if (!$cur->hasRight(Right::GRANTROLE)) {
             // TRANS: Client error displayed when trying to assign a role to a user while not being allowed to set roles.
             $this->clientError(_('You cannot grant user roles on this site.'));
-            return false;
         }
 
         assert(!empty($this->profile)); // checked by parent
@@ -82,7 +79,6 @@ class GrantRoleAction extends ProfileFormAction
         if ($this->profile->hasRole($this->role)) {
             // TRANS: Client error displayed when trying to assign a role to a user that already has that role.
             $this->clientError(_('User already has this role.'));
-            return false;
         }
 
         return true;
index 1c6ccb9e59605ae00349f9a89f709fab2671b7a4..d65b62bdff7980128e5d634ddc687729448973d0 100644 (file)
@@ -205,7 +205,6 @@ class GroupblockAction extends RedirectingAction
         if (empty($block)) {
             // TRANS: Server error displayed when trying to block a user from a group fails because of an application error.
             $this->serverError(_("Database error blocking user from group."));
-            return false;
         }
 
         $this->returnToPrevious();
index 5baaab5e6da08f37a0ab5564a63e9863afa87282..b82a861e97319931b3f322cd1024e59e241969bf 100644 (file)
@@ -71,7 +71,6 @@ class GroupbyidAction extends Action
         if (!$id) {
             // TRANS: Client error displayed referring to a group's permalink without providing a group ID.
             $this->clientError(_('No ID.'));
-            return false;
         }
 
         common_debug("Got ID $id");
@@ -81,7 +80,6 @@ class GroupbyidAction extends Action
         if (!$this->group) {
             // TRANS: Client error displayed referring to a group's permalink for a non-existing group ID.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         return true;
index adffe63c388118999c9eda9eba874632a110d7e7..10fd33088d6a60fa3445b1339febd442e595d4be 100644 (file)
@@ -67,7 +67,6 @@ class GrouplogoAction extends GroupAction
         if (!common_logged_in()) {
             // TRANS: Client error displayed when trying to create a group while not logged in.
             $this->clientError(_('You must be logged in to create a group.'));
-            return false;
         }
 
         $nickname_arg = $this->trimmed('nickname');
@@ -78,13 +77,11 @@ class GrouplogoAction extends GroupAction
         if ($nickname_arg != $nickname) {
             $args = array('nickname' => $nickname);
             common_redirect(common_local_url('grouplogo', $args), 301);
-            return false;
         }
 
         if (!$nickname) {
             // TRANS: Client error displayed when trying to change group logo settings without providing a nickname.
             $this->clientError(_('No nickname.'), 404);
-            return false;
         }
 
         $groupid = $this->trimmed('groupid');
@@ -101,7 +98,6 @@ class GrouplogoAction extends GroupAction
         if (!$this->group) {
             // TRANS: Client error displayed when trying to update logo settings for a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
@@ -109,7 +105,6 @@ class GrouplogoAction extends GroupAction
         if (!$cur->isAdmin($this->group)) {
             // TRANS: Client error displayed when trying to change group logo settings while not being a group admin.
             $this->clientError(_('You must be an admin to edit the group.'), 403);
-            return false;
         }
 
         return true;
@@ -182,7 +177,6 @@ class GrouplogoAction extends GroupAction
             common_log_db_error($user, 'SELECT', __FILE__);
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'));
-            return;
         }
 
         $original = $this->group->original_logo;
@@ -389,7 +383,6 @@ class GrouplogoAction extends GroupAction
         if (!$filedata) {
             // TRANS: Server error displayed trying to crop an uploaded group logo that is no longer present.
             $this->serverError(_('Lost our file data.'));
-            return;
         }
 
         // If image is not being cropped assume pos & dimentions of original
index 7227b1109029dc75b169c9ef454147b4b815d0cb..c50eff36f83c5bc9279bacdc5d3095db15000ec0 100644 (file)
@@ -69,13 +69,11 @@ class GroupqueueAction extends GroupAction
                 $args['page'] = $this->page;
             }
             common_redirect(common_local_url('groupqueue', $args), 301);
-            return false;
         }
 
         if (!$nickname) {
             // TRANS: Client error displayed when trying to view group members without providing a group nickname.
             $this->clientError(_('No nickname.'), 404);
-            return false;
         }
 
         $local = Local_group::getKV('nickname', $nickname);
@@ -83,7 +81,6 @@ class GroupqueueAction extends GroupAction
         if (!$local) {
             // TRANS: Client error displayed when trying to view group members for a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $this->group = User_group::getKV('id', $local->group_id);
@@ -91,14 +88,12 @@ class GroupqueueAction extends GroupAction
         if (!$this->group) {
             // TRANS: Client error displayed when trying to view group members for an object that is not a group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
         if (!$cur || !$cur->isAdmin($this->group)) {
             // TRANS: Client error displayed when trying to approve group applicants without being a group administrator.
             $this->clientError(_('Only the group admin may approve users.'));
-            return false;
         }
         return true;
     }
index 4728f429f4488245a429c9891945528108c7f02e..87e34d73f80eccb6eb9020bd5ac005a606dea475 100644 (file)
@@ -81,13 +81,11 @@ class groupRssAction extends Rss10Action
         if ($nickname_arg != $nickname) {
             $args = array('nickname' => $nickname);
             common_redirect(common_local_url('showgroup', $args), 301);
-            return false;
         }
 
         if (!$nickname) {
             // TRANS: Client error displayed when requesting a group RSS feed without providing a group nickname.
             $this->clientError(_('No nickname.'), 404);
-            return false;
         }
 
         $local = Local_group::getKV('nickname', $nickname);
@@ -95,7 +93,6 @@ class groupRssAction extends Rss10Action
         if (!$local) {
             // TRANS: Client error displayed when requesting a group RSS feed for group that does not exist.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $this->group = User_group::getKV('id', $local->group_id);
@@ -103,7 +100,6 @@ class groupRssAction extends Rss10Action
         if (!$this->group) {
             // TRANS: Client error displayed when requesting a group RSS feed for an object that is not a group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $this->notices = $this->getNotices($this->limit);
index 6f26108155b647ebaed33f84bcdda85eee0fd578..a24177f64c0d94d8a50dd5b06abfa7066ac2ec09 100644 (file)
@@ -58,48 +58,40 @@ class GroupunblockAction extends Action
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
         $token = $this->trimmed('token');
         if (empty($token) || $token != common_session_token()) {
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token. Try again, please.'));
-            return;
         }
         $id = $this->trimmed('unblockto');
         if (empty($id)) {
             // TRANS: Client error displayed when trying to unblock a user from a group without providing a profile.
             $this->clientError(_('No profile specified.'));
-            return false;
         }
         $this->profile = Profile::getKV('id', $id);
         if (empty($this->profile)) {
             // TRANS: Client error displayed when trying to unblock a user from a group without providing an existing profile.
             $this->clientError(_('No profile with that ID.'));
-            return false;
         }
         $group_id = $this->trimmed('unblockgroup');
         if (empty($group_id)) {
             // TRANS: Client error displayed when trying to unblock a user from a group without providing a group.
             $this->clientError(_('No group specified.'));
-            return false;
         }
         $this->group = User_group::getKV('id', $group_id);
         if (empty($this->group)) {
             // TRANS: Client error displayed when trying to unblock a user from a non-existing group.
             $this->clientError(_('No such group.'));
-            return false;
         }
         $user = common_current_user();
         if (!$user->isAdmin($this->group)) {
             // TRANS: Client error displayed when trying to unblock a user from a group without being an administrator for the group.
             $this->clientError(_('Only an admin can unblock group members.'), 401);
-            return false;
         }
         if (!Group_block::isBlocked($this->group, $this->profile)) {
             // TRANS: Client error displayed when trying to unblock a non-blocked user from a group.
             $this->clientError(_('User is not blocked from group.'));
-            return false;
         }
         return true;
     }
@@ -131,7 +123,6 @@ class GroupunblockAction extends Action
         if (!$result) {
             // TRANS: Server error displayed when unblocking a user from a group fails because of an unknown error.
             $this->serverError(_('Error removing the block.'));
-            return;
         }
 
         foreach ($this->args as $k => $v) {
@@ -145,9 +136,7 @@ class GroupunblockAction extends Action
         if ($action) {
             common_redirect(common_local_url($action, $args), 303);
         } else {
-            common_redirect(common_local_url('blockedfromgroup',
-                                             array('nickname' => $this->group->nickname)),
-                            303);
+            common_redirect(common_local_url('blockedfromgroup', array('nickname' => $this->group->nickname)), 303);
         }
     }
 }
index 809dbd287980c333f62e63c5e25531370a36537c..92fff45a7db6d27f506be1f50520afb717b685af 100644 (file)
@@ -292,7 +292,6 @@ class ImsettingsAction extends SettingsAction
                     common_log_db_error($user, 'UPDATE', __FILE__);
                     // TRANS: Server error thrown on database error updating IM preferences.
                     $this->serverError(_('Could not update IM preferences.'));
-                    return;
                 }
             }while($user_im_prefs->fetch());
         }
@@ -364,7 +363,6 @@ class ImsettingsAction extends SettingsAction
             common_log_db_error($confirm, 'INSERT', __FILE__);
             // TRANS: Server error thrown on database error adding Instant Messaging confirmation code.
             $this->serverError(_('Could not insert confirmation code.'));
-            return;
         }
 
         Event::handle('SendImConfirmationCode', array($transport, $screenname, $confirm->code, $user));
@@ -407,7 +405,6 @@ class ImsettingsAction extends SettingsAction
             common_log_db_error($confirm, 'DELETE', __FILE__);
             // TRANS: Server error thrown on database error canceling IM address confirmation.
             $this->serverError(_('Could not delete confirmation.'));
-            return;
         }
 
         // TRANS: Message given after successfully canceling IM address confirmation.
@@ -445,7 +442,6 @@ class ImsettingsAction extends SettingsAction
             common_log_db_error($user, 'UPDATE', __FILE__);
             // TRANS: Server error thrown on database error removing a registered IM address.
             $this->serverError(_('Could not update user IM preferences.'));
-            return;
         }
 
         // XXX: unsubscribe to the old address
index af204698d85d9ae5ba1d7ac348fb64935340381e..c533ebdedb7d3dc2e7b8762a4910520d24553e89 100644 (file)
@@ -57,7 +57,6 @@ class JoingroupAction extends Action
         if (!common_logged_in()) {
             // TRANS: Client error displayed when trying to join a group while not logged in.
             $this->clientError(_('You must be logged in to join a group.'));
-            return false;
         }
 
         $nickname_arg = $this->trimmed('nickname');
@@ -72,7 +71,6 @@ class JoingroupAction extends Action
             if ($nickname_arg != $nickname) {
                 $args = array('nickname' => $nickname);
                 common_redirect(common_local_url('leavegroup', $args), 301);
-                return false;
             }
 
             $local = Local_group::getKV('nickname', $nickname);
@@ -80,20 +78,17 @@ class JoingroupAction extends Action
             if (!$local) {
                 // TRANS: Client error displayed when trying to join a non-local group.
                 $this->clientError(_('No such group.'), 404);
-                return false;
             }
 
             $this->group = User_group::getKV('id', $local->group_id);
         } else {
             // TRANS: Client error displayed when trying to join a group without providing a group name or group ID.
             $this->clientError(_('No nickname or ID.'), 404);
-            return false;
         }
 
         if (!$this->group) {
             // TRANS: Client error displayed when trying to join a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
@@ -101,13 +96,11 @@ class JoingroupAction extends Action
         if ($cur->isMember($this->group)) {
             // TRANS: Client error displayed when trying to join a group while already a member.
             $this->clientError(_('You are already a member of that group.'), 403);
-            return false;
         }
 
         if (Group_block::isBlocked($this->group, $cur->getProfile())) {
             // TRANS: Client error displayed when trying to join a group while being blocked form joining it.
             $this->clientError(_('You have been blocked from that group by the admin.'), 403);
-            return false;
         }
 
         return true;
@@ -165,9 +158,7 @@ class JoingroupAction extends Action
             $this->elementEnd('body');
             $this->endHTML();
         } else {
-            common_redirect(common_local_url('groupmembers', array('nickname' =>
-                                                                   $this->group->nickname)),
-                            303);
+            common_redirect(common_local_url('groupmembers', array('nickname' => $this->group->nickname)), 303);
         }
     }
 }
index d92dd37de56b693c27033d0a1fe9a131251bbb01..e0e2c278904249edc54b640909e4296f09b7f9de 100644 (file)
@@ -57,7 +57,6 @@ class LeavegroupAction extends Action
         if (!common_logged_in()) {
             // TRANS: Client error displayed when trying to leave a group while not logged in.
             $this->clientError(_('You must be logged in to leave a group.'));
-            return false;
         }
 
         $nickname_arg = $this->trimmed('nickname');
@@ -72,7 +71,6 @@ class LeavegroupAction extends Action
             if ($nickname_arg != $nickname) {
                 $args = array('nickname' => $nickname);
                 common_redirect(common_local_url('leavegroup', $args), 301);
-                return false;
             }
 
             $local = Local_group::getKV('nickname', $nickname);
@@ -80,20 +78,17 @@ class LeavegroupAction extends Action
             if (!$local) {
                 // TRANS: Client error displayed when trying to leave a non-local group.
                 $this->clientError(_('No such group.'), 404);
-                return false;
             }
 
             $this->group = User_group::getKV('id', $local->group_id);
         } else {
             // TRANS: Client error displayed when trying to leave a group without providing a group name or group ID.
             $this->clientError(_('No nickname or ID.'), 404);
-            return false;
         }
 
         if (!$this->group) {
             // TRANS: Client error displayed when trying to leave a non-existing group.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
@@ -101,7 +96,6 @@ class LeavegroupAction extends Action
         if (!$cur->isMember($this->group)) {
             // TRANS: Client error displayed when trying to join a group while already a member.
             $this->clientError(_('You are not a member of that group.'), 403);
-            return false;
         }
 
         return true;
@@ -147,9 +141,7 @@ class LeavegroupAction extends Action
             $this->elementEnd('body');
             $this->endHTML();
         } else {
-            common_redirect(common_local_url('groupmembers', array('nickname' =>
-                                                                   $this->group->nickname)),
-                            303);
+            common_redirect(common_local_url('groupmembers', array('nickname' => $this->group->nickname)), 303);
         }
     }
 }
index 5cc27032efa18002490c73b8c990bd1c1599ca50..05912e150cc4d493b537394c836edba78705aa1c 100644 (file)
@@ -60,44 +60,37 @@ class MakeadminAction extends RedirectingAction
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
         $token = $this->trimmed('token');
         if (empty($token) || $token != common_session_token()) {
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token. Try again, please.'));
-            return;
         }
         $id = $this->trimmed('profileid');
         if (empty($id)) {
             // TRANS: Client error displayed when not providing a profile ID on the Make Admin page.
             $this->clientError(_('No profile specified.'));
-            return false;
         }
         $this->profile = Profile::getKV('id', $id);
         if (empty($this->profile)) {
             // TRANS: Client error displayed when specifying an invalid profile ID on the Make Admin page.
             $this->clientError(_('No profile with that ID.'));
-            return false;
         }
         $group_id = $this->trimmed('groupid');
         if (empty($group_id)) {
             // TRANS: Client error displayed when not providing a group ID on the Make Admin page.
             $this->clientError(_('No group specified.'));
-            return false;
         }
         $this->group = User_group::getKV('id', $group_id);
         if (empty($this->group)) {
             // TRANS: Client error displayed when providing an invalid group ID on the Make Admin page.
             $this->clientError(_('No such group.'));
-            return false;
         }
         $user = common_current_user();
         if (!$user->isAdmin($this->group) &&
             !$user->hasRight(Right::MAKEGROUPADMIN)) {
             // TRANS: Client error displayed when trying to make another user admin on the Make Admin page while not an admin.
             $this->clientError(_('Only an admin can make another user an admin.'), 401);
-            return false;
         }
         if ($this->profile->isAdmin($this->group)) {
             // TRANS: Client error displayed when trying to make another user admin on the Make Admin page who already is admin.
@@ -106,7 +99,6 @@ class MakeadminAction extends RedirectingAction
                                        $this->profile->getBestName(),
                                        $this->group->getBestName()),
                                401);
-            return false;
         }
         return true;
     }
index 14c626ac513bd52b560910706a8a3a191c53c9f2..2742eb9a048f8977dcc0e674949a25605826b490 100644 (file)
@@ -61,7 +61,6 @@ class MicrosummaryAction extends Action
         if (!$user) {
             // TRANS: Client error displayed trying to make a micro summary without providing a valid user.
             $this->clientError(_('No such user.'), 404);
-            return;
         }
 
         $notice = $user->getCurrentNotice();
index 0ca3753dc7bc03427138f3163322f42bf93684f8..801d1f06810aa343dbfe78e2cb37e005db519cd8 100644 (file)
@@ -62,7 +62,6 @@ class NudgeAction extends Action
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return;
         }
 
         $user  = common_current_user();
@@ -71,7 +70,6 @@ class NudgeAction extends Action
         if ($_SERVER['REQUEST_METHOD'] != 'POST') {
             common_redirect(common_local_url('showstream',
                 array('nickname' => $other->nickname)));
-            return;
         }
 
         // CSRF protection
@@ -80,13 +78,11 @@ class NudgeAction extends Action
         if (!$token || $token != common_session_token()) {
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token. Try again, please.'));
-            return;
         }
 
         if (!$other->email || !$other->emailnotifynudge) {
             // TRANS: Client error displayed trying to nudge a user that cannot be nudged.
             $this->clientError(_('This user doesn\'t allow nudges or hasn\'t confirmed or set their email address yet.'));
-            return;
         }
 
         $this->notify($user, $other);
index d6477f6be964bcdebfa7a02019a9ea7f573d4759..29e6d5607333e9e3d17fd2ab21316df45c927339 100644 (file)
@@ -57,7 +57,6 @@ class OauthappssettingsAction extends SettingsAction
         if (!common_logged_in()) {
             // TRANS: Message displayed to an anonymous user trying to view OAuth application list.
             $this->clientError(_('You must be logged in to list your applications.'));
-            return false;
         }
 
         return true;
index eaa0fae8daa740ad1bb3eb284ab884934ca65610..9aa3ad434f452b7a3335d4e51e4b08911e17bdaf 100644 (file)
@@ -142,7 +142,6 @@ class OauthconnectionssettingsAction extends SettingsAction
         } else {
             // TRANS: Client error when submitting a form with unexpected information.
             $this->clientError(_('Unexpected form submission.'), 401);
-            return false;
         }
     }
 
@@ -163,7 +162,6 @@ class OauthconnectionssettingsAction extends SettingsAction
         if (empty($appUser)) {
             // TRANS: Client error when trying to revoke access for an application while not being a user of it.
             $this->clientError(_('You are not a user of that application.'), 401);
-            return false;
         }
 
         $app = Oauth_application::getKV('id', $appUser->application_id);
@@ -178,7 +176,6 @@ class OauthconnectionssettingsAction extends SettingsAction
             // TRANS: Client error when revoking access has failed for some reason.
             // TRANS: %s is the application ID revoking access failed for.
             $this->clientError(sprintf(_('Unable to revoke access for application: %s.'), $app->id));
-            return false;
         }
 
         $msg = 'API OAuth - user %s (id: %d) revoked access token %s for app id %d';
index f14bba9a2b3649620e7a3313be4a029a7b9e142a..c44f3673ae7524f2902158a67fa645dcfec10fb9 100644 (file)
@@ -60,7 +60,6 @@ class OtpAction extends Action
         if (common_is_real_login()) {
             // TRANS: Client error displayed trying to use "one time password login" when already logged in.
             $this->clientError(_('Already logged in.'));
-            return false;
         }
 
         $id = $this->trimmed('user_id');
@@ -68,7 +67,6 @@ class OtpAction extends Action
         if (empty($id)) {
             // TRANS: Client error displayed trying to use "one time password login" without specifying a user.
             $this->clientError(_('No user ID specified.'));
-            return false;
         }
 
         $this->user = User::getKV('id', $id);
@@ -76,7 +74,6 @@ class OtpAction extends Action
         if (empty($this->user)) {
             // TRANS: Client error displayed trying to use "one time password login" without using an existing user.
             $this->clientError(_('No such user.'));
-            return false;
         }
 
         $this->token = $this->trimmed('token');
@@ -84,7 +81,6 @@ class OtpAction extends Action
         if (empty($this->token)) {
             // TRANS: Client error displayed trying to use "one time password login" without specifying a login token.
             $this->clientError(_('No login token specified.'));
-            return false;
         }
 
         $this->lt = Login_token::getKV('user_id', $id);
@@ -92,13 +88,11 @@ class OtpAction extends Action
         if (empty($this->lt)) {
             // TRANS: Client error displayed trying to use "one time password login" without requesting a login token.
             $this->clientError(_('No login token requested.'));
-            return false;
         }
 
         if ($this->lt->token != $this->token) {
             // TRANS: Client error displayed trying to use "one time password login" while specifying an invalid login token.
             $this->clientError(_('Invalid login token specified.'));
-            return false;
         }
 
         if ($this->lt->modified > time() + Login_token::TIMEOUT) {
@@ -108,7 +102,6 @@ class OtpAction extends Action
             $this->lt = null;
             // TRANS: Client error displayed trying to use "one time password login" while specifying an expired login token.
             $this->clientError(_('Login token expired.'));
-            return false;
         }
 
         $this->rememberme = $this->boolean('rememberme');
@@ -125,7 +118,6 @@ class OtpAction extends Action
         if (!common_set_user($this->user)) {
             // TRANS: Server error displayed when a user object could not be created trying to login using "one time password login".
             $this->serverError(_('Error setting user. You are probably not authorized.'));
-            return;
         }
 
         // We're now logged in; disable the lt
index 3ac1a3f94619c3d5f9d819a2000259295153d3a1..db36b612a29becfa806733c885e57ff955504f48 100644 (file)
@@ -196,10 +196,9 @@ class PasswordsettingsAction extends SettingsAction
             }
 
             if (!$user->update($original)) {
-            // TRANS: Server error displayed on page where to change password when password change
-            // TRANS: could not be made because of a server error.
-            $this->serverError(_('Cannot save new password.'));
-                return;
+                // TRANS: Server error displayed on page where to change password when password change
+                // TRANS: could not be made because of a server error.
+                $this->serverError(_('Cannot save new password.'));
             }
             Event::handle('EndChangePassword', array($user));
         }
index dbdcf5fbc79490ee50aeb034c27cc6f52ca830fd..b0a6e2635deb72b94a2cb63b2de2ebe2e72418f9 100644 (file)
@@ -78,7 +78,6 @@ class PeopletagAction extends Action
                 $args['page'] = $this->page;
             }
             common_redirect(common_local_url('peopletag', $args), 301);
-            return false;
         }
         $this->tag = $tag;
 
index 5b6ae57a5393229126f5634e1994709021a1ada1..86d63545dcb0b4ed2aa4040e78aef8e8ebf9899e 100644 (file)
@@ -55,7 +55,6 @@ class PeopletagautocompleteAction extends Action
         if (empty($this->user)) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
 
         // CSRF protection
@@ -66,7 +65,6 @@ class PeopletagautocompleteAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         $profile = $this->user->getProfile();
index 92bfce6405101230b9ff3c06a09bc97b8c2252e1..cf9ec053ed404b64b243c8bfeeb8ccde3a0bb46d 100644 (file)
@@ -76,13 +76,11 @@ class PeopletaggedAction extends Action
                 $args['page'] = $this->page;
             }
             common_redirect(common_local_url('peopletagged', $args), 301);
-            return false;
         }
 
         if (!$tagger) {
             // TRANS: Client error displayed when a tagger is expected but not provided.
             $this->clientError(_('No tagger.'), 404);
-            return false;
         }
 
         $user = User::getKV('nickname', $tagger);
@@ -90,7 +88,6 @@ class PeopletaggedAction extends Action
         if (!$user) {
             // TRANS: Client error displayed when referring to non-existing user.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->tagger = $user->getProfile();
@@ -99,7 +96,6 @@ class PeopletaggedAction extends Action
         if (!$this->peopletag) {
             // TRANS: Client error displayed when referring to a non-existing list.
             $this->clientError(_('No such list.'), 404);
-            return false;
         }
 
         return true;
index 8a5a53eca4f2bb5720b08a322b14df8faa997aec..8b3a91917a5f0611b7fbebbc8b847edba393e67f 100644 (file)
@@ -92,7 +92,6 @@ class PeopletagsbyuserAction extends Action
                 $args['page'] = $this->arg['page'];
             }
             common_redirect(common_local_url('peopletagsbyuser', $args), 301);
-            return false;
         }
 
         $this->user = User::getKV('nickname', $nickname);
@@ -100,7 +99,6 @@ class PeopletagsbyuserAction extends Action
         if (!$this->user) {
             // TRANS: Client error displayed trying to perform an action related to a non-existing user.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->tagger = $this->user->getProfile();
@@ -108,7 +106,6 @@ class PeopletagsbyuserAction extends Action
         if (!$this->tagger) {
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'));
-            return false;
         }
 
         $this->page = ($this->arg('page')) ? ($this->arg('page')+0) : 1;
@@ -146,7 +143,6 @@ class PeopletagsbyuserAction extends Action
 
                if ($_SERVER['REQUEST_METHOD'] == 'POST') {
                    common_redirect(common_local_url('peopletagsbyuser', $this->getSelfUrlArgs()), 303);
-            return;
                }
 
         $this->showPage();
index 83122e41055ce0b22c7070d3fd35db7937222758..827b284d5c3acf567cf550f5afdae99ed06c721b 100644 (file)
@@ -74,7 +74,6 @@ class PeopletagsforuserAction extends Action
                 $args['page'] = $this->arg['page'];
             }
             common_redirect(common_local_url('peopletagsforuser', $args), 301);
-            return false;
         }
 
         $this->user = User::getKV('nickname', $nickname);
@@ -82,7 +81,6 @@ class PeopletagsforuserAction extends Action
         if (!$this->user) {
             // TRANS: Client error displayed trying to perform an action related to a non-existing user.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->tagged = $this->user->getProfile();
@@ -90,7 +88,6 @@ class PeopletagsforuserAction extends Action
         if (!$this->tagged) {
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'));
-            return false;
         }
 
         $this->page = ($this->arg('page')) ? ($this->arg('page')+0) : 1;
index 123b168596c4698185ac8f2ba8c871b78cf3aa53..589d49caa3be8469eddc6c3b7ba456039bf482c8 100644 (file)
@@ -76,13 +76,11 @@ class PeopletagsubscribersAction extends Action
                 $args['page'] = $this->page;
             }
             common_redirect(common_local_url('peopletagged', $args), 301);
-            return false;
         }
 
         if (!$tagger) {
             // TRANS: Client error displayed when a tagger is expected but not provided.
             $this->clientError(_('No tagger.'), 404);
-            return false;
         }
 
         $user = User::getKV('nickname', $tagger);
@@ -90,7 +88,6 @@ class PeopletagsubscribersAction extends Action
         if (!$user) {
             // TRANS: Client error displayed trying to perform an action related to a non-existing user.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->tagger = $user->getProfile();
@@ -99,7 +96,6 @@ class PeopletagsubscribersAction extends Action
         if (!$this->peopletag) {
             // TRANS: Client error displayed trying to reference a non-existing list.
             $this->clientError(_('No such list.'), 404);
-            return false;
         }
 
         return true;
index f6baed416febce7abb8b95b783546d04b9d218b8..64edd0929025b6c126955cfb8d22d4810b3c3d3f 100644 (file)
@@ -76,7 +76,6 @@ class PeopletagsubscriptionsAction extends Action
                 $args['page'] = $this->arg['page'];
             }
             common_redirect(common_local_url('peopletagsbyuser', $args), 301);
-            return false;
         }
 
         $user = User::getKV('nickname', $nickname);
@@ -84,7 +83,6 @@ class PeopletagsubscriptionsAction extends Action
         if (!$user) {
             // TRANS: Client error displayed trying to perform an action related to a non-existing user.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->profile = $user->getProfile();
@@ -92,7 +90,6 @@ class PeopletagsubscriptionsAction extends Action
         if (!$this->profile) {
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'));
-            return false;
         }
 
         $this->page = ($this->arg('page')) ? ($this->arg('page')+0) : 1;
index d48cef018368bd8c68f20967976419a10fcf9b11..707cbdecaab45870995a43d0d08c2354e1c9a9b1 100644 (file)
@@ -76,7 +76,6 @@ class PluginEnableAction extends Action
             // TRANS: Client error displayed when trying to use another method than POST.
             // TRANS: Do not translate POST.
             $this->clientError(_('This action only accepts POST requests.'));
-            return false;
         }
 
         // CSRF protection
@@ -87,7 +86,6 @@ class PluginEnableAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         // Only for logged-in users
@@ -97,13 +95,11 @@ class PluginEnableAction extends Action
         if (empty($this->user)) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
 
         if (!AdminPanelAction::canAdmin('plugins')) {
             // TRANS: Client error displayed when trying to enable or disable a plugin without access rights.
             $this->clientError(_('You cannot administer plugins.'));
-            return false;
         }
 
         $this->plugin = $this->arg('plugin');
@@ -111,7 +107,6 @@ class PluginEnableAction extends Action
         if (!array_key_exists($this->plugin, $defaultPlugins)) {
             // TRANS: Client error displayed when trying to enable or disable a non-existing plugin.
             $this->clientError(_('No such plugin.'));
-            return false;
         }
 
         return true;
index b859ccc9439035cda96866dbff3b13ca321d213c..aaf6026eb58115c0ccef5134f0cc5e6da8c2e3b9 100644 (file)
@@ -80,7 +80,6 @@ class ProfilecompletionAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         // Only for logged-in users
@@ -90,7 +89,6 @@ class ProfilecompletionAction extends Action
         if (empty($this->user)) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
 
         $id = $this->arg('peopletag_id');
@@ -99,7 +97,6 @@ class ProfilecompletionAction extends Action
         if (empty($this->peopletag)) {
             // TRANS: Client error displayed trying to reference a non-existing list.
             $this->clientError(_('No such list.'));
-            return false;
         }
 
         $field = $this->arg('field');
@@ -107,7 +104,6 @@ class ProfilecompletionAction extends Action
             // TRANS: Client error displayed when trying to add an unindentified field to profile.
             // TRANS: %s is a field name.
             $this->clientError(sprintf(_('Unidentified field %s.'), htmlspecialchars($field)), 404);
-            return false;
         }
         $this->field = $field;
 
index f20f1bde4fc62cbcbcd6349dbc9e48e4bd0499a1..ce7298a2f6b7038829c872533aff2026d2d700dd 100644 (file)
@@ -55,7 +55,6 @@ class ProfiletagbyidAction extends Action
         if (!$id) {
             // TRANS: Client error displayed trying to perform an action without providing an ID.
             $this->clientError(_('No ID.'));
-            return false;
         }
 
         common_debug("Peopletag id $id by user id $tagger_id");
@@ -65,7 +64,6 @@ class ProfiletagbyidAction extends Action
         if (!$this->peopletag) {
             // TRANS: Client error displayed trying to reference a non-existing list.
             $this->clientError(_('No such list.'), 404);
-            return false;
         }
 
         $user = User::getKV('id', $tagger_id);
index af2fcff842ed737e2b2c4c07b9be14fac8c8aae7..6ecdf2e50ab337bdcabd7364dbca35a09a2558d4 100644 (file)
@@ -102,7 +102,6 @@ class PublicAction extends Action
         if (!$this->notice) {
             // TRANS: Server error displayed when a public timeline cannot be retrieved.
             $this->serverError(_('Could not retrieve public timeline.'));
-            return;
         }
 
         if($this->page > 1 && $this->notice->N == 0){
index 1565ac9bf34e6999c840e0a7d9c1655cdf9fa45c..2bdb6774eadb2f18908e1e14309e31e44f422943 100644 (file)
@@ -35,7 +35,6 @@ class RecoverpasswordAction extends Action
         if (common_logged_in()) {
             // TRANS: Client error displayed trying to recover password while already logged in.
             $this->clientError(_('You are already logged in!'));
-            return;
         } else if ($_SERVER['REQUEST_METHOD'] == 'POST') {
             if ($this->arg('recover')) {
                 $this->recoverPassword();
@@ -62,12 +61,10 @@ class RecoverpasswordAction extends Action
         if (!$confirm) {
             // TRANS: Client error displayed when password recovery code is not correct.
             $this->clientError(_('No such recovery code.'));
-            return;
         }
         if ($confirm->address_type != 'recover') {
             // TRANS: Client error displayed when no proper password recovery code was submitted.
             $this->clientError(_('Not a recovery code.'));
-            return;
         }
 
         $user = User::getKV($confirm->user_id);
@@ -75,7 +72,6 @@ class RecoverpasswordAction extends Action
         if (!$user) {
             // TRANS: Server error displayed trying to recover password without providing a user.
             $this->serverError(_('Recovery code for unknown user.'));
-            return;
         }
 
         $touched = strtotime($confirm->modified);
@@ -89,7 +85,6 @@ class RecoverpasswordAction extends Action
             common_log_db_error($confirm, 'DELETE', __FILE__);
             // TRANS: Server error displayed removing a password recovery code from the database.
             $this->serverError(_('Error with confirmation code.'));
-            return;
         }
 
         // These should be reaped, but for now we just check mod time
@@ -102,7 +97,6 @@ class RecoverpasswordAction extends Action
             // TRANS: Client error displayed trying to recover password with too old a recovery code.
             $this->clientError(_('This confirmation code is too old. ' .
                                    'Please start again.'));
-            return;
         }
 
         // If we used an outstanding confirmation to send the email,
@@ -116,7 +110,6 @@ class RecoverpasswordAction extends Action
                 common_log_db_error($user, 'UPDATE', __FILE__);
                 // TRANS: Server error displayed when updating a user's e-mail address in the database fails while recovering a password.
                 $this->serverError(_('Could not update user with confirmed email address.'));
-                return;
             }
         }
 
@@ -310,7 +303,6 @@ class RecoverpasswordAction extends Action
         if (!$user) {
             // TRANS: Client error displayed when trying to reset as password without providing a user.
             $this->clientError(_('Unexpected password reset.'));
-            return;
         }
 
         $newpassword = $this->trimmed('newpassword');
@@ -337,7 +329,6 @@ class RecoverpasswordAction extends Action
             common_log_db_error($user, 'UPDATE', __FILE__);
             // TRANS: Reset password form validation error message.
             $this->serverError(_('Cannot save new password.'));
-            return;
         }
 
         $this->clearTempUser();
@@ -345,7 +336,6 @@ class RecoverpasswordAction extends Action
         if (!common_set_user($user->nickname)) {
             // TRANS: Server error displayed when something does wrong with the user object during password reset.
             $this->serverError(_('Error setting user.'));
-            return;
         }
 
         common_real_login(true);
index 5c559a8b7ea0bca9dadf1c51b8a381a28f8e7d2f..4befa4ab840c843c014c712f2b4d3e6e611593f1 100644 (file)
@@ -93,7 +93,6 @@ class RedirecturlAction extends Action
     function handle($argarray=null)
     {
         common_redirect($this->file->url, 307);
-        return;
     }
 
     /**
index 7c372892df8c27b57e8eacb3cf839885619b7527..cf2c049e73ed86580a9801d3a11fdce421f173f2 100644 (file)
@@ -79,7 +79,6 @@ class RegisterAction extends Action
         if (common_config('site', 'inviteonly') && empty($this->code)) {
             // TRANS: Client error displayed when trying to register to an invite-only site without an invitation.
             $this->clientError(_('Sorry, only invited people can register.'));
-            return false;
         }
 
         if (!empty($this->code)) {
@@ -87,7 +86,6 @@ class RegisterAction extends Action
             if (empty($this->invite)) {
             // TRANS: Client error displayed when trying to register to an invite-only site without a valid invitation.
                 $this->clientError(_('Sorry, invalid invitation code.'));
-                return false;
             }
             // Store this in case we need it
             common_ensure_session();
@@ -187,7 +185,6 @@ class RegisterAction extends Action
             if (common_config('site', 'inviteonly') && !($code && $invite)) {
                 // TRANS: Client error displayed when trying to register to an invite-only site without an invitation.
                 $this->clientError(_('Sorry, only invited people can register.'));
-                return;
             }
 
             // Input scrubbing
@@ -387,7 +384,6 @@ class RegisterAction extends Action
         if (common_config('site', 'inviteonly') && !($code && $invite)) {
             // TRANS: Client error displayed when trying to register to an invite-only site without an invitation.
             $this->clientError(_('Sorry, only invited people can register.'));
-            return;
         }
 
         $this->elementStart('form', array('method' => 'post',
index 7077573999b36fea515506d807b7afe7d3757dcf..8192e86b44fd4246c932136a92f6c01eee9f5022 100644 (file)
@@ -78,7 +78,6 @@ class RemovepeopletagAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         // Only for logged-in users
@@ -88,7 +87,6 @@ class RemovepeopletagAction extends Action
         if (empty($this->user)) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
 
         // Profile to subscribe to
@@ -100,7 +98,6 @@ class RemovepeopletagAction extends Action
         if (empty($this->tagged)) {
             // TRANS: Client error displayed when referring to a non-existing profile.
             $this->clientError(_('No such profile.'));
-            return false;
         }
 
         $id = $this->arg('peopletag_id');
@@ -109,7 +106,6 @@ class RemovepeopletagAction extends Action
         if (empty($this->peopletag)) {
             // TRANS: Client error displayed trying to reference a non-existing list.
             $this->clientError(_('No such list.'));
-            return false;
         }
 
         return true;
@@ -146,7 +142,6 @@ class RemovepeopletagAction extends Action
                                       'The remote server is probably not responding correctly, ' .
                                       'please try retrying later.'), $this->profile->profileurl));
             }
-            return false;
         }
         if ($this->boolean('ajax')) {
             $this->startHTML('text/xml;charset=utf-8');
index 9e527c526940a9600f79d2f0e7773d1142734e7b..c926d85095e4df75080b6baea13c958a7ed3e53d 100644 (file)
@@ -56,7 +56,6 @@ class RepeatAction extends Action
         if (empty($id)) {
             // TRANS: Client error displayed when trying to repeat a notice while not providing a notice ID.
             $this->clientError(_('No notice specified.'));
-            return false;
         }
 
         $this->notice = Notice::getKV('id', $id);
@@ -64,7 +63,6 @@ class RepeatAction extends Action
         if (!($this->notice instanceof Notice)) {
             // TRANS: Client error displayed when trying to repeat a non-existing notice.
             $this->clientError(_('No notice specified.'));
-            return false;
         }
 
         $token = $this->trimmed('token-'.$id);
@@ -72,7 +70,6 @@ class RepeatAction extends Action
         if (empty($token) || $token != common_session_token()) {
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token. Try again, please.'));
-            return false;
         }
 
         return true;
index 4c6fba9ece7206d0cc16c805e41b740a5481baef..d6d2cb4ba936938c38f65e8fcd48eaa3f424cf3d 100644 (file)
@@ -70,7 +70,6 @@ class RepliesAction extends Action
         if (!$this->user) {
             // TRANS: Client error displayed when trying to reply to a non-exsting user.
             $this->clientError(_('No such user.'));
-            return false;
         }
 
         $profile = $this->user->getProfile();
@@ -78,7 +77,6 @@ class RepliesAction extends Action
         if (!$profile) {
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'));
-            return false;
         }
 
         $this->page = ($this->arg('page')) ? ($this->arg('page')+0) : 1;
index 8a8f64e0c2b40dcbe8cf0355f2801f07414d7ecd..145b51aaea99dfc58cb252c7c1210897dd773a39 100644 (file)
@@ -36,7 +36,6 @@ class RepliesrssAction extends Rss10Action
         if (!$this->user) {
             // TRANS: Client error displayed when providing a non-existing nickname in a RSS 1.0 action.
             $this->clientError(_('No such user.'));
-            return false;
         } else {
             $this->notices = $this->getNotices($this->limit);
             return true;
index 1218c9e923312ab5ab902eb6ef3aa50f99664c99..b55399598eebd81c4e982de0b1d2d7ac71e4e0a9 100644 (file)
@@ -59,12 +59,10 @@ class RevokeRoleAction extends ProfileFormAction
         if (!Profile_role::isValid($this->role)) {
             // TRANS: Client error displayed when trying to revoke an invalid role.
             $this->clientError(_('Invalid role.'));
-            return false;
         }
         if (!Profile_role::isSettable($this->role)) {
             // TRANS: Client error displayed when trying to revoke a reserved role.
             $this->clientError(_('This role is reserved and cannot be set.'));
-            return false;
         }
 
         $cur = common_current_user();
@@ -74,7 +72,6 @@ class RevokeRoleAction extends ProfileFormAction
         if (!$cur->hasRight(Right::REVOKEROLE)) {
             // TRANS: Client error displayed when trying to revoke a role without having the right to do that.
             $this->clientError(_('You cannot revoke user roles on this site.'));
-            return false;
         }
 
         assert(!empty($this->profile)); // checked by parent
@@ -82,7 +79,6 @@ class RevokeRoleAction extends ProfileFormAction
         if (!$this->profile->hasRole($this->role)) {
             // TRANS: Client error displayed when trying to revoke a role that is not set.
             $this->clientError(_('User does not have this role.'));
-            return false;
         }
 
         return true;
index 921e7584c90139898e0878ac56b2ac6861108fd4..1ad3b815e8b542a77b7e74fbbde55f23173a60e7 100644 (file)
@@ -101,10 +101,7 @@ class RsdAction extends Action
             // Permanent redirect on non-canonical nickname
 
             if ($nickname_arg != $nickname) {
-                common_redirect(common_local_url('rsd',
-                                                 array('nickname' => $nickname)),
-                                301);
-                return false;
+                common_redirect(common_local_url('rsd', array('nickname' => $nickname)), 301);
             }
 
             $this->user = User::getKV('nickname', $nickname);
@@ -112,7 +109,6 @@ class RsdAction extends Action
             if (empty($this->user)) {
                 // TRANS: Client error.
                 $this->clientError(_('No such user.'), 404);
-                return false;
             }
         }
 
index ddbca40b2de6992dac0b86e65a300ddf6adf05f1..af2eab3842d268add1d5a8c0693464a81fb91871 100644 (file)
@@ -62,7 +62,6 @@ class SandboxAction extends ProfileFormAction
         if (!$cur->hasRight(Right::SANDBOXUSER)) {
             // TRANS: Client error displayed trying to sandbox users on a site where the feature is not enabled.
             $this->clientError(_('You cannot sandbox users on this site.'));
-            return false;
         }
 
         assert(!empty($this->profile)); // checked by parent
@@ -70,7 +69,6 @@ class SandboxAction extends ProfileFormAction
         if ($this->profile->isSandboxed()) {
             // TRANS: Client error displayed trying to sandbox an already sandboxed user.
             $this->clientError(_('User is already sandboxed.'));
-            return false;
         }
 
         return true;
index acee1862f4cc88730a8f6b014eedd4da4d6fb32a..b44dd5746a09e49f6bfbd174f45ae04af9302e09 100644 (file)
@@ -77,13 +77,11 @@ class ShowApplicationAction extends Action
         if (!common_logged_in()) {
             // TRANS: Client error displayed trying to display an OAuth application while not logged in.
             $this->clientError(_('You must be logged in to view an application.'));
-            return false;
         }
 
         if (empty($this->application)) {
             // TRANS: Client error displayed trying to display a non-existing OAuth application.
             $this->clientError(_('No such application.'), 404);
-            return false;
         }
 
         $cur = common_current_user();
@@ -91,7 +89,6 @@ class ShowApplicationAction extends Action
         if ($cur->id != $this->owner->id) {
             // TRANS: Client error displayed trying to display an OAuth application for which the logged in user is not the owner.
             $this->clientError(_('You are not the owner of this application.'), 401);
-            return false;
         }
 
         return true;
@@ -115,7 +112,6 @@ class ShowApplicationAction extends Action
             if (!$token || $token != common_session_token()) {
                 // TRANS: Client error displayed when the session token does not match or is not given.
                 $this->clientError(_('There was a problem with your session token.'));
-                return;
             }
 
             if ($this->arg('reset')) {
index a1a2dabb3877ef3a23359ee405a1bbc57c979453..0e028416da2cfa1314237403bf0b44bdcb576b4d 100644 (file)
@@ -104,7 +104,6 @@ class ShowfavoritesAction extends Action
         if (!$this->user) {
             // TRANS: Client error displayed when trying to display favourite notices for a non-existing user.
             $this->clientError(_('No such user.'));
-            return false;
         }
 
         $this->page = $this->trimmed('page');
@@ -132,7 +131,6 @@ class ShowfavoritesAction extends Action
         if (empty($this->notice)) {
             // TRANS: Server error displayed when favourite notices could not be retrieved from the database.
             $this->serverError(_('Could not retrieve favorite notices.'));
-            return;
         }
 
         if($this->page > 1 && $this->notice->N == 0){
index 24c7db1d50e74a4f0668f8625f09bc2f095fa818..86fbee8b73f95c303ac21f477da1a474d997b00c 100644 (file)
@@ -74,7 +74,6 @@ class ShowmessageAction extends Action
         if (!$this->message) {
             // TRANS: Client error displayed requesting a single message that does not exist.
             $this->clientError(_('No such message.'), 404);
-            return false;
         }
 
         $this->user = common_current_user();
index 1f92978062755b8d538fe685d29e349d58aa97f5..d86e98080bad9efc1c7653f8806a06415ded1258 100644 (file)
@@ -97,7 +97,6 @@ class ShownoticeAction extends Action
         if (empty($this->profile)) {
             // TRANS: Server error displayed trying to show a notice without a connected profile.
             $this->serverError(_('Notice has no profile.'), 500);
-            return false;
         }
 
         $this->user = User::getKV('id', $this->profile->id);
@@ -230,7 +229,6 @@ class ShownoticeAction extends Action
                 }
                 if ($target && $target != $this->selfUrl()) {
                     common_redirect($target, 301);
-                    return false;
                 }
             }
             $this->showPage();
index e7152c0046b22d48e6a4209824bf44e69e9e5e7c..4524e4aa8c23a8e71482bbf826b7977fe21e0336 100644 (file)
@@ -61,13 +61,11 @@ class ShowprofiletagAction extends Action
                 $args['page'] = $this->page;
             }
             common_redirect(common_local_url('showprofiletag', $args), 301);
-            return false;
         }
 
         if (!$tagger) {
             // TRANS: Client error displayed when a tagger is expected but not provided.
             $this->clientError(_('No tagger.'), 404);
-            return false;
         }
 
         $user = User::getKV('nickname', $tagger);
@@ -75,7 +73,6 @@ class ShowprofiletagAction extends Action
         if (!$user) {
             // TRANS: Client error displayed trying to perform an action related to a non-existing user.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->tagger = $user->getProfile();
@@ -88,7 +85,6 @@ class ShowprofiletagAction extends Action
         if (!$can_see) {
             // TRANS: Client error displayed trying to reference a non-existing list.
             $this->clientError(_('No such list.'), 404);
-            return false;
         }
 
         $this->page = ($this->arg('page')) ? ($this->arg('page')+0) : 1;
@@ -114,7 +110,6 @@ class ShowprofiletagAction extends Action
         if (!$this->peopletag) {
             // TRANS: Client error displayed trying to perform an action related to a non-existing user.
             $this->clientError(_('No such user.'));
-            return;
         }
 
         $this->showPage();
index c44aa3a6d788a1764cd33a9ac1d3730f0c93ee29..6a4f84deb9a15390d89548750597af81a7735630 100644 (file)
@@ -62,7 +62,6 @@ class SilenceAction extends ProfileFormAction
         if (!$cur->hasRight(Right::SILENCEUSER)) {
             // TRANS: Client error displayed trying to silence a user on a site where the feature is not enabled.
             $this->clientError(_('You cannot silence users on this site.'));
-            return false;
         }
 
         assert(!empty($this->profile)); // checked by parent
@@ -70,7 +69,6 @@ class SilenceAction extends ProfileFormAction
         if ($this->profile->isSilenced()) {
             // TRANS: Client error displayed trying to silence an already silenced user.
             $this->clientError(_('User is already silenced.'));
-            return false;
         }
 
         return true;
index 40c9a841b33d303785a56c426b7ef295baa5cc1a..b45e12518981d8b80e20476a65f5404ccabac3b5 100644 (file)
@@ -173,7 +173,6 @@ class SiteadminpanelAction extends AdminPanelAction
             !in_array($values['site']['timezone'], DateTimeZone::listIdentifiers())) {
             // TRANS: Client error displayed trying to save site settings without a timezone.
             $this->clientError(_('Timezone not selected.'));
-            return;
         }
 
         // Validate language
index a39eb2cf57bab84e63e810707117c18b4911da8e..d8e219db413c4c8dcbc60d01c44b68d84a83657b 100644 (file)
@@ -299,7 +299,6 @@ class SmssettingsAction extends SettingsAction
             common_log_db_error($user, 'UPDATE', __FILE__);
             // TRANS: Server error thrown on database error updating SMS preferences.
             $this->serverError(_('Could not update user.'));
-            return;
         }
 
         $user->query('COMMIT');
@@ -363,7 +362,6 @@ class SmssettingsAction extends SettingsAction
             common_log_db_error($confirm, 'INSERT', __FILE__);
             // TRANS: Server error thrown on database error adding SMS confirmation code.
             $this->serverError(_('Could not insert confirmation code.'));
-            return;
         }
 
         $carrier = Sms_carrier::getKV($carrier_id);
@@ -411,7 +409,6 @@ class SmssettingsAction extends SettingsAction
             common_log_db_error($confirm, 'DELETE', __FILE__);
             // TRANS: Server error thrown on database error canceling SMS phone number confirmation.
             $this->serverError(_('Could not delete SMS confirmation.'));
-            return;
         }
 
         // TRANS: Message given after successfully canceling SMS phone number confirmation.
@@ -452,7 +449,6 @@ class SmssettingsAction extends SettingsAction
             common_log_db_error($user, 'UPDATE', __FILE__);
             // TRANS: Server error thrown on database error removing a registered SMS phone number.
             $this->serverError(_('Could not update user.'));
-            return;
         }
         $user->query('COMMIT');
 
index 062c4167cd5c0748faf31bd0758986393edf753b..47fe19ea244f0c2cee9ed8cc77a9e4e08cfb0d3f 100644 (file)
@@ -31,7 +31,6 @@ class SubeditAction extends Action
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
 
         $token = $this->trimmed('token');
@@ -40,7 +39,6 @@ class SubeditAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token. '.
                                  'Try again, please.'));
-            return false;
         }
 
         $id = $this->trimmed('profile');
@@ -48,7 +46,6 @@ class SubeditAction extends Action
         if (!$id) {
             // TRANS: Client error displayed trying a change a subscription without providing a profile.
             $this->clientError(_('No profile specified.'));
-            return false;
         }
 
         $this->profile = Profile::getKV('id', $id);
@@ -56,7 +53,6 @@ class SubeditAction extends Action
         if (!$this->profile) {
             // TRANS: Client error displayed trying a change a subscription for a non-existant profile ID.
             $this->clientError(_('No profile with that ID.'));
-            return false;
         }
 
         return true;
@@ -74,7 +70,6 @@ class SubeditAction extends Action
             if (!$sub) {
                 // TRANS: Client error displayed trying a change a subscription for a non-subscribed profile.
                 $this->clientError(_('You are not subscribed to that profile.'));
-                return false;
             }
 
             $orig = clone($sub);
@@ -88,12 +83,9 @@ class SubeditAction extends Action
                 common_log_db_error($sub, 'UPDATE', __FILE__);
                 // TRANS: Server error displayed when updating a subscription fails with a database error.
                 $this->serverError(_('Could not save subscription.'));
-                return false;
             }
 
-            common_redirect(common_local_url('subscriptions',
-                                             array('nickname' => $cur->nickname)),
-                            303);
+            common_redirect(common_local_url('subscriptions', array('nickname' => $cur->nickname)), 303);
         }
     }
 }
index 38bc16e56236d28e1acdc471e07977ebab1961f2..b1c914ebda6c0bedf8afe75230e309175d82e0cd 100644 (file)
@@ -60,7 +60,6 @@ class SubqueueAction extends GalleryAction
         if (!$cur || $cur->id != $this->profile->id) {
             // TRANS: Client error displayed when trying to approve group applicants without being a group administrator.
             $this->clientError(_('You may only approve your own pending subscriptions.'));
-            return false;
         }
         return true;
     }
index 36f9fc724b53a1c552e02b7075a849a44591d640..4002c9fbb0164267250333f87255f782489616b1 100644 (file)
@@ -74,7 +74,6 @@ class SubscribeAction extends Action
             // TRANS: Client error displayed trying to perform any request method other than POST.
             // TRANS: Do not translate POST.
             $this->clientError(_('This action only accepts POST requests.'));
-            return false;
         }
 
         // CSRF protection
@@ -85,7 +84,6 @@ class SubscribeAction extends Action
             // TRANS: Client error displayed when the session token is not okay.
             $this->clientError(_('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         // Only for logged-in users
@@ -95,7 +93,6 @@ class SubscribeAction extends Action
         if (empty($this->user)) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
 
         // Profile to subscribe to
@@ -107,7 +104,6 @@ class SubscribeAction extends Action
         if (empty($this->other)) {
             // TRANS: Client error displayed trying to subscribe to a non-existing profile.
             $this->clientError(_('No such profile.'));
-            return false;
         }
 
         return true;
index 1bb14420ff66b8186ce824fc7c8dc297d0736778..b83ff70010e15c60a42a61de89d70b5a9a5d0d34 100644 (file)
@@ -57,14 +57,12 @@ class SubscribepeopletagAction extends Action
         if (!common_logged_in()) {
             // TRANS: Client error displayed when trying to perform an action while not logged in.
             $this->clientError(_('You must be logged in to unsubscribe from a list.'));
-            return false;
         }
         // Only allow POST requests
 
         if ($_SERVER['REQUEST_METHOD'] != 'POST') {
             // TRANS: Client error displayed when trying to use another method than POST.
             $this->clientError(_('This action only accepts POST requests.'));
-            return false;
         }
 
         // CSRF protection
@@ -75,7 +73,6 @@ class SubscribepeopletagAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         $tagger_arg = $this->trimmed('tagger');
@@ -87,13 +84,11 @@ class SubscribepeopletagAction extends Action
         } else {
             // TRANS: Client error displayed when trying to perform an action without providing an ID.
             $this->clientError(_('No ID given.'), 404);
-            return false;
         }
 
         if (!$this->peopletag || $this->peopletag->private) {
             // TRANS: Client error displayed trying to reference a non-existing list.
             $this->clientError(_('No such list.'), 404);
-            return false;
         }
 
         $this->tagger = Profile::getKV('id', $this->peopletag->tagger);
index e0cb9783d1e38b3a98adeb1a760f87e6b5a2c290..21cae9aba5127e43a83209fc52da13432f384d7b 100644 (file)
@@ -34,13 +34,10 @@ class TagAction extends Action
 
         if (!$this->tag) {
             common_redirect(common_local_url('publictagcloud'), 301);
-            return false;
         }
 
         if ($this->tag != $taginput) {
-            common_redirect(common_local_url('tag', array('tag' => $this->tag)),
-                            301);
-            return false;
+            common_redirect(common_local_url('tag', array('tag' => $this->tag)), 301);
         }
 
         $this->page = ($this->arg('page')) ? ($this->arg('page')+0) : 1;
index e9d855c4978e7101b7661773f10d804dcb573fc2..7538d7c8b6d62bdc0d9b5d2341f1eaf5e9bb32c5 100644 (file)
@@ -46,7 +46,6 @@ class TagprofileAction extends Action
             if (!$this->profile) {
                 // TRANS: Client error displayed when referring to non-existing profile ID.
                 $this->clientError(_('No profile with that ID.'));
-                return false;
             }
         }
 
@@ -55,7 +54,6 @@ class TagprofileAction extends Action
             // TRANS: Client error displayed when trying to tag a user that cannot be tagged.
             $this->clientError(_('You cannot tag this user.'));
         }
-        return true;
     }
 
     function handle($args)
index 091e383b3990b8ad4a24f1dd9d8c16856ce9c44a..468944d11f117a99c4819c693e99dc57bb367bec 100644 (file)
@@ -33,7 +33,6 @@ class TagrssAction extends Rss10Action
         if (!$this->tag) {
             // TRANS: Client error when requesting a tag feed for a non-existing tag.
             $this->clientError(_('No such tag.'));
-            return false;
         } else {
             $this->notices = $this->getNotices($this->limit);
             return true;
index 1f4d9606b85a7da875dc6ccf030feab2bc72c1ca..82d0d32c109149f3e732754c3bfffb61fd9e0121 100644 (file)
@@ -57,7 +57,6 @@ class UnblockAction extends ProfileFormAction
         if (!$cur->hasBlocked($this->profile)) {
             // TRANS: Client error displayed when trying to unblock a non-blocked user.
             $this->clientError(_("You haven't blocked that user."));
-            return false;
         }
 
         return true;
@@ -84,7 +83,6 @@ class UnblockAction extends ProfileFormAction
         if (!$result) {
             // TRANS: Server error displayed when removing a user block.
             $this->serverError(_('Error removing the block.'));
-            return;
         }
     }
 }
index 98bd2d4f54761f981898dc45f4bfa9ff73d6b57a..b87736fcffffbf54d7537cd1a307eded9c2b7c86 100644 (file)
@@ -62,7 +62,6 @@ class UnsandboxAction extends ProfileFormAction
         if (!$cur->hasRight(Right::SANDBOXUSER)) {
             // TRANS: Client error on page to unsandbox a user when the feature is not enabled.
             $this->clientError(_('You cannot sandbox users on this site.'));
-            return false;
         }
 
         assert(!empty($this->profile)); // checked by parent
@@ -70,7 +69,6 @@ class UnsandboxAction extends ProfileFormAction
         if (!$this->profile->isSandboxed()) {
             // TRANS: Client error on page to unsilence a user when the to be unsandboxed user has not been sandboxed.
             $this->clientError(_('User is not sandboxed.'));
-            return false;
         }
 
         return true;
index 723bf1bed5c92a1bb95c877d4c72012d1257838b..c01c141b1c193fd6337365b26497d9438bc71094 100644 (file)
@@ -62,7 +62,6 @@ class UnsilenceAction extends ProfileFormAction
         if (!$cur->hasRight(Right::SILENCEUSER)) {
             // TRANS: Client error on page to unsilence a user when the feature is not enabled.
             $this->clientError(_('You cannot silence users on this site.'));
-            return false;
         }
 
         assert(!empty($this->profile)); // checked by parent
@@ -70,7 +69,6 @@ class UnsilenceAction extends ProfileFormAction
         if (!$this->profile->isSilenced()) {
             // TRANS: Client error on page to unsilence a user when the to be unsilenced user has not been silenced.
             $this->clientError(_('User is not silenced.'));
-            return false;
         }
 
         return true;
index 1beb295aeca8aba768d17da87a14f6c1a0f85d85..4b85ba427ca1b18d611434865ffb0dfce1dcbff3 100644 (file)
@@ -50,13 +50,11 @@ class UnsubscribeAction extends Action
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return;
         }
 
         if ($_SERVER['REQUEST_METHOD'] != 'POST') {
             common_redirect(common_local_url('subscriptions',
                                              array('nickname' => $this->scoped->nickname)));
-            return;
         }
 
         /* Use a session token for CSRF protection. */
@@ -67,7 +65,6 @@ class UnsubscribeAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token. ' .
                                  'Try again, please.'));
-            return;
         }
 
         $other_id = $this->arg('unsubscribeto');
@@ -75,7 +72,6 @@ class UnsubscribeAction extends Action
         if (!$other_id) {
             // TRANS: Client error displayed when trying to unsubscribe without providing a profile ID.
             $this->clientError(_('No profile ID in request.'));
-            return;
         }
 
         $other = Profile::getKV('id', $other_id);
@@ -83,7 +79,6 @@ class UnsubscribeAction extends Action
         if (!($other instanceof Profile)) {
             // TRANS: Client error displayed when trying to unsubscribe while providing a non-existing profile ID.
             $this->clientError(_('No profile with that ID.'));
-            return;
         }
 
         try {
@@ -104,9 +99,7 @@ class UnsubscribeAction extends Action
             $this->elementEnd('body');
             $this->endHTML();
         } else {
-            common_redirect(common_local_url('subscriptions',
-                                             array('nickname' => $this->scoped->nickname)),
-                            303);
+            common_redirect(common_local_url('subscriptions', array('nickname' => $this->scoped->nickname)), 303);
         }
     }
 }
index c739c6760c55caee3e8102f9a40b520d96262e2c..b5757ad6fdb7d719b7ba78e46b5635fe635de6d4 100644 (file)
@@ -58,14 +58,12 @@ class UnsubscribepeopletagAction extends Action
         if (!common_logged_in()) {
             // TRANS: Client error displayed when trying to perform an action while not logged in.
             $this->clientError(_('You must be logged in to unsubscribe from a list.'));
-            return false;
         }
         // Only allow POST requests
 
         if ($_SERVER['REQUEST_METHOD'] != 'POST') {
             // TRANS: Client error displayed when trying to use another method than POST.
             $this->clientError(_('This action only accepts POST requests.'));
-            return false;
         }
 
         // CSRF protection
@@ -76,7 +74,6 @@ class UnsubscribepeopletagAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         $tagger_arg = $this->trimmed('tagger');
@@ -88,13 +85,11 @@ class UnsubscribepeopletagAction extends Action
         } else {
             // TRANS: Client error displayed when trying to perform an action without providing an ID.
             $this->clientError(_('No ID given.'), 404);
-            return false;
         }
 
         if (!$this->peopletag || $this->peopletag->private) {
             // TRANS: Client error displayed trying to reference a non-existing list.
             $this->clientError(_('No such list.'), 404);
-            return false;
         }
 
         $this->tagger = Profile::getKV('id', $this->peopletag->tagger);
@@ -137,11 +132,8 @@ class UnsubscribepeopletagAction extends Action
         } else {
             if (common_get_returnto()) {
                 common_redirect(common_get_returnto(), 303);
-                return true;
             }
-            common_redirect(common_local_url('peopletagsbyuser',
-                                array('nickname' => $this->tagger->nickname)),
-                            303);
+            common_redirect(common_local_url('peopletagsbyuser', array('nickname' => $this->tagger->nickname)), 303);
         }
     }
 }
index dd1218bc14f986ffbc8cd60acb5561969963db0e..69ad4c869066fb2434b117577560c4507b03ea37 100644 (file)
@@ -211,7 +211,6 @@ class UrlsettingsAction extends SettingsAction
             common_log_db_error($user, 'UPDATE', __FILE__);
             // TRANS: Server error displayed when "Other" settings in user profile could not be updated on the server.
             $this->serverError(_('Could not update user.'));
-            return;
         }
 
         $prefs = User_urlshortener_prefs::getPrefs($user);
index 45fed57d84714e13205c415aad65fc8f18e76a07..e2e2fff97f344ba7110ca687e835254d5be8ea56 100644 (file)
@@ -85,7 +85,6 @@ class UsergroupsAction extends ProfileAction
                 $args['page'] = $this->arg['page'];
             }
             common_redirect(common_local_url('usergroups', $args), 301);
-            return false;
         }
 
         $this->user = User::getKV('nickname', $nickname);
@@ -93,7 +92,6 @@ class UsergroupsAction extends ProfileAction
         if (!$this->user) {
             // TRANS: Client error displayed requesting groups for a non-existing user.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->profile = $this->user->getProfile();
@@ -101,7 +99,6 @@ class UsergroupsAction extends ProfileAction
         if (!$this->profile) {
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'));
-            return false;
         }
 
         $this->page = $this->trimmed('page', 1);
index 82be959455933141957f2eb12bc9f5a4a2ff987b..1d0aabb8bb375ea0592ba138e2b3a0ca72a9e5c7 100644 (file)
--- a/index.php
+++ b/index.php
@@ -286,7 +286,6 @@ function main()
     // If the request is HTTP and it should be HTTPS...
     if ($site_ssl != 'never' && !StatusNet::isHTTPS() && common_is_sensitive($args['action'])) {
         common_redirect(common_local_url($args['action'], $args));
-        return;
     }
 
     $args = array_merge($args, $_REQUEST);
@@ -297,7 +296,6 @@ function main()
 
     if (!$action || !preg_match('/^[a-zA-Z0-9_-]*$/', $action)) {
         common_redirect(common_local_url('public'));
-        return;
     }
 
     // If the site is private, and they're not on one of the "public"
@@ -324,7 +322,6 @@ function main()
         common_set_returnto(common_local_url($action, $rargs));
 
         common_redirect(common_local_url('login'));
-        return;
     }
 
     $action_class = ucfirst($action).'Action';
index 4868e9d492de116af2ba5b785f1185d7af17ca3c..2ac92cb5c6522a38f8b6e6d789e9b21c20a0b890 100644 (file)
@@ -69,7 +69,6 @@ class AdminPanelAction extends Action
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return false;
         }
 
         $user = common_current_user();
@@ -94,7 +93,6 @@ class AdminPanelAction extends Action
         if (!$user->hasRight(Right::CONFIGURESITE)) {
             // TRANS: Client error message thrown when a user tries to change admin settings but has no access rights.
             $this->clientError(_('You cannot make changes to this site.'));
-            return false;
         }
 
         // This panel must be enabled
@@ -106,7 +104,6 @@ class AdminPanelAction extends Action
         if (!self::canAdmin($name)) {
             // TRANS: Client error message throw when a certain panel's settings cannot be changed.
             $this->clientError(_('Changes to that panel are not allowed.'), 403);
-            return false;
         }
 
         return true;
@@ -220,7 +217,6 @@ class AdminPanelAction extends Action
     {
         // TRANS: Client error message.
         $this->clientError(_('showForm() not implemented.'));
-        return;
     }
 
     /**
@@ -248,7 +244,6 @@ class AdminPanelAction extends Action
     {
         // TRANS: Client error message
         $this->clientError(_('saveSettings() not implemented.'));
-        return;
     }
 
     function canAdmin($name)
index f8ffbc84a5b4bc449abe779c160bb2146aee1917..9f26b3b304135dec609a0794e39c778b45f3ccb3 100644 (file)
@@ -1324,7 +1324,6 @@ class ApiAction extends Action
         default:
             // TRANS: Client error on an API request with an unsupported data format.
             $this->clientError(_('Not a supported data format.'));
-            break;
         }
 
         return;
@@ -1351,7 +1350,6 @@ class ApiAction extends Action
         default:
             // TRANS: Client error on an API request with an unsupported data format.
             $this->clientError(_('Not a supported data format.'));
-            break;
         }
         return;
     }
@@ -1406,7 +1404,6 @@ class ApiAction extends Action
         default:
             // TRANS: Client error on an API request with an unsupported data format.
             $this->clientError(_('Not a supported data format.'));
-            return;
         }
         return;
     }
index 42a7bc5c99cc441a4f19a66b4380f70f49e5666c..13d1a8c580d44e92f18c347040d5ff330a1708a4 100644 (file)
@@ -63,7 +63,6 @@ class ApiListUsersAction extends ApiBareAuthAction
         if (empty($this->list)) {
             // TRANS: Client error displayed when referring to a non-existing list.
             $this->clientError(_('List not found.'), 404, $this->format);
-            return false;
         }
 
         if(!$this->create && !$this->delete) {
index 85e3f0de3ebb6e2e6b016ad79b0f9307d5dde90e..975933f6cceeb492c393f985d0d64594b1663b29 100644 (file)
@@ -51,7 +51,6 @@ class GalleryAction extends ProfileAction
                 $args['page'] = $this->arg['page'];
             }
             common_redirect(common_local_url($this->trimmed('action'), $args), 301);
-            return false;
         }
 
         $this->user = User::getKV('nickname', $nickname);
@@ -59,7 +58,6 @@ class GalleryAction extends ProfileAction
         if (!$this->user) {
             // TRANS: Client error displayed when trying to perform a gallery action with an unknown user.
             $this->clientError(_('No such user.'), 404);
-            return false;
         }
 
         $this->profile = $this->user->getProfile();
@@ -67,7 +65,6 @@ class GalleryAction extends ProfileAction
         if (!$this->profile) {
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_('User has no profile.'));
-            return false;
         }
 
         $this->page = ($this->arg('page')) ? ($this->arg('page')+0) : 1;
@@ -91,7 +88,6 @@ class GalleryAction extends ProfileAction
 
                if ($_SERVER['REQUEST_METHOD'] == 'POST') {
                    common_redirect($this->selfUrl(), 303);
-            return;
                }
 
         $this->showPage();
index 0e2867fba890170882b62c1a22a90489db525c88..a0141a83b94e80a573cd440f53e6bb6eba570373 100644 (file)
@@ -61,13 +61,11 @@ class GroupAction extends Action
                 $args['page'] = $this->page;
             }
             common_redirect(common_local_url('showgroup', $args), 301);
-            return false;
         }
 
         if (!$nickname) {
             // TRANS: Client error displayed if no nickname argument was given requesting a group page.
             $this->clientError(_('No nickname.'), 404);
-            return false;
         }
 
         $local = Local_group::getKV('nickname', $nickname);
@@ -80,12 +78,10 @@ class GroupAction extends Action
                     $args['page'] = $this->page;
                 }
                 common_redirect(common_local_url('groupbyid', $args), 301);
-                return false;
             } else {
                 common_log(LOG_NOTICE, "Couldn't find local group for nickname '$nickname'");
                 // TRANS: Client error displayed if no remote group with a given name was found requesting group page.
                 $this->clientError(_('No such group.'), 404);
-                return false;
             }
         }
 
@@ -94,7 +90,6 @@ class GroupAction extends Action
         if (!$this->group) {
             // TRANS: Client error displayed if no local group with a given name was found requesting group page.
             $this->clientError(_('No such group.'), 404);
-            return false;
         }
     }
 
index b1720fc82478dd033f534a2cdd6a115bff835d83..3c7281e9852121d2b948b39eac803d092058e313 100644 (file)
@@ -77,16 +77,13 @@ class MailboxAction extends Action
         if (!$this->user) {
             // TRANS: Client error displayed when trying to access a mailbox without providing a user.
             $this->clientError(_('No such user.'), 404);
-            return;
         }
 
         $cur = common_current_user();
 
         if (!$cur || $cur->id != $this->user->id) {
             // TRANS: Client error displayed when trying to access a mailbox that is not of the logged in user.
-            $this->clientError(_('Only the user can read their own mailboxes.'),
-                403);
-            return;
+            $this->clientError(_('Only the user can read their own mailboxes.'), 403);
         }
 
         $this->showPage();
index 4c738523a40b6846d5c05a0d534aa010e6da8336..bfc7a3c7dedf12d11388e346623afab435cd3df2 100644 (file)
@@ -77,7 +77,6 @@ class ProfileFormAction extends RedirectingAction
         if (!$id) {
             // TRANS: Client error displayed when trying to change user options without specifying a user to work on.
             $this->clientError(_('No profile specified.'));
-            return false;
         }
 
         $this->profile = Profile::getKV('id', $id);
@@ -85,7 +84,6 @@ class ProfileFormAction extends RedirectingAction
         if (!$this->profile) {
             // TRANS: Client error displayed when trying to change user options without specifying an existing user to work on.
             $this->clientError(_('No profile with that ID.'));
-            return false;
         }
 
         return true;
index 560534065f65c55a3149ce30d77496db9791c045..fd4885c830223072a9421435def6fb756b448613 100644 (file)
@@ -71,7 +71,6 @@ class SettingsAction extends Action
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_('Not logged in.'));
-            return;
         } else if (!common_is_real_login()) {
             // Cookie theft means that automatic logins can't
             // change important settings or see private info, and
index c54a39202c654084dbd01c242cf39fafc3739c70..15f1040b7c4a30fd94977335970eab91e540a620 100644 (file)
@@ -58,7 +58,6 @@ class AnonDisfavorAction extends RedirectingAction
                 // TRANS: Client error.
                 _m('Could not disfavor notice! Please make sure your browser has cookies enabled.')
             );
-            return;
         }
 
         $id     = $this->trimmed('notice');
@@ -79,7 +78,6 @@ class AnonDisfavorAction extends RedirectingAction
             common_log_db_error($fave, 'DELETE', __FILE__);
             // TRANS: Server error.
             $this->serverError(_m('Could not delete favorite.'));
-            return;
         }
 
         $profile->blowFavesCache();
index da8757082e670af92ff01b4f779e6cd4ed8e901a..2d35964c700cee95d1dec826b8dddcbb0388f162 100644 (file)
@@ -55,9 +55,7 @@ class AnonFavorAction extends RedirectingAction
 
         if (empty($profile) || $_SERVER['REQUEST_METHOD'] != 'POST') {
             // TRANS: Client error.
-            $this->clientError(_m('Could not favor notice! Please make sure your browser has cookies enabled.')
-            );
-            return;
+            $this->clientError(_m('Could not favor notice! Please make sure your browser has cookies enabled.'));
         }
 
         $id     = $this->trimmed('notice');
@@ -67,14 +65,12 @@ class AnonFavorAction extends RedirectingAction
         if ($profile->hasFave($notice)) {
             // TRANS: Client error.
             $this->clientError(_m('This notice is already a favorite!'));
-            return;
         }
         $fave = Fave::addNew($profile, $notice);
 
         if (!$fave) {
             // TRANS: Server error.
             $this->serverError(_m('Could not create favorite.'));
-            return;
         }
 
         $profile->blowFavesCache();
index 19ae8e9ae9a019fc57425d8a7637c232f6605cb1..c7e074193a4229f98196aca02dca678763e475ac 100644 (file)
@@ -294,7 +294,6 @@ class BookmarkPlugin extends MicroAppPlugin
         if (!$this->user) {
             // TRANS: Client error displayed when trying to display bookmarks for a non-existing user.
             $this->clientError(_('No such user.'));
-            return false;
         }
 
         $action->menuItem(common_local_url('bookmarks', array('nickname' => $this->user->nickname)),
index 4cc0dcde7f74d2bacd549e43553aba338f804557..01ef20a8c83c4a6aa20f0ae2043c8548ac9c70c3 100644 (file)
@@ -66,7 +66,6 @@ class ApiTimelineBookmarksAction extends ApiBareAuthAction
         if (empty($this->user)) {
             // TRANS: Client error displayed when requesting most recent favourite notices by a user for a non-existing user.
             $this->clientError(_('No such user.'), 404, $this->format);
-            return;
         }
 
         $this->notices = $this->getNotices();
@@ -172,7 +171,6 @@ class ApiTimelineBookmarksAction extends ApiBareAuthAction
         default:
             // TRANS: Client error displayed when coming across a non-supported API method.
             $this->clientError(_('API method not found.'), $code = 404);
-            break;
         }
     }
 
index fce1185cb5b0df983479950ef560f6dc85e95601..23ef0ec5fc1d5942256ae825b7a933e179b755e1 100644 (file)
@@ -80,7 +80,6 @@ class BookmarksAction extends Action
         if (!$this->user) {
             // TRANS: Client error displayed when trying to display bookmarks for a non-existing user.
             $this->clientError(_('No such user.'));
-            return false;
         }
 
         $this->page = ($this->arg('page')) ? ($this->arg('page')+0) : 1;
index c48ab9093417c03a92f1725ad48ab91c7130b35a..fc3331482f9538b752d6d20818f47ad079c433fe 100644 (file)
@@ -71,7 +71,6 @@ class BookmarksrssAction extends Rss10Action
         if (!$this->user) {
             // TRANS: Client error displayed when trying to get the RSS feed with bookmarks of a user that does not exist.
             $this->clientError(_('No such user.'));
-            return false;
         } else {
             $this->notices = $this->getNotices($this->limit);
             return true;
index 9d0fcb0507f2d0f695cae25a91c7099061eb73ad..09377c62ebb8b763a6584dcb19a293d95f1dcedb 100644 (file)
@@ -39,14 +39,12 @@ class CasloginAction extends Action
             if (!$user) {
                 // TRANS: Server error displayed when trying to log in with incorrect username or password.
                 $this->serverError(_m('Incorrect username or password.'));
-                return;
             }
 
             // success!
             if (!common_set_user($user)) {
                 // TRANS: Server error displayed when login fails in CAS authentication plugin.
                 $this->serverError(_m('Error setting user. You are probably not authorized.'));
-                return;
             }
 
             common_real_login(true);
index 39eb76d599432b9fc17a7c55906b2d482573e90f..aa7f4d6ab6fc145ca1f29c6eeb6d63fa80180d5f 100644 (file)
@@ -191,7 +191,6 @@ class EmailregisterAction extends Action
 
         if (!empty($cur)) {
             common_redirect(common_local_url('all', array('nickname' => $cur->nickname)));
-            return;
         }
 
         switch ($this->state) {
index cb8efb3c1db1985a813a77a16a3e2580cb917ac2..092fa6bda5922793551846452bc452a56a10e45c 100644 (file)
@@ -63,7 +63,6 @@ class TimelistAction extends Action {
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_m('Not logged in.'));
-            return;
         }
 
         if (!empty($this->start)) {
@@ -71,7 +70,6 @@ class TimelistAction extends Action {
         } else {
             // TRANS: Client error when submitting a form with unexpected information.
             $this->clientError(_m('Unexpected form submission.'));
-            return;
         }
 
         if ($this->boolean('ajax')) {
index 5bc2443481d1f9b565a7fb95e66d856b9485b680..8d8a33ec4493e03b84541b8223e875971442a8bd 100644 (file)
@@ -620,7 +620,6 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction
                 common_log_db_error($profile, 'UPDATE', __FILE__);
                 // TRANS: Server error thrown when user profile settings could not be saved.
                 $this->serverError(_m('Could not save profile.'));
-                return;
             }
 
             // Set the user tags
@@ -629,7 +628,6 @@ class ProfileDetailSettingsAction extends ProfileSettingsAction
             if (!$result) {
                 // TRANS: Server error thrown when user profile settings tags could not be saved.
                 $this->serverError(_m('Could not save tags.'));
-                return;
             }
 
             Event::handle('EndProfileSaveForm', array($this));
index 21b2cd1f2c9bf54fc44495f671e8abbff7631d43..b6c1f16f50c66271983ba9d20a198829b4649fb7 100644 (file)
@@ -414,7 +414,6 @@ ENDOFSCRIPT;
                 $action->logout();
 
                 common_redirect($logoutUrl, 303);
-                return false; // probably never get here, but hey
             }
 
             return true;
index ab83601ce1e976089bbbb26a114888add976f494..53815ad264b1cb5a8815c8193fbfc330c7852425 100644 (file)
@@ -48,7 +48,6 @@ class FacebookfinishloginAction extends Action
 
         if (empty($this->accessToken)) {
             $this->clientError(_m("Unable to authenticate you with Facebook."));
-            return false;
         }
 
         $graphUrl = 'https://graph.facebook.com/me?access_token=' . urlencode($this->accessToken);
@@ -117,8 +116,6 @@ class FacebookfinishloginAction extends Action
                 // TRANS: in the same StatusNet site.
                 _m('There is already a local account linked with that Facebook account.')
             );
-
-            return;
        }
 
        $cur = common_current_user();
@@ -133,8 +130,6 @@ class FacebookfinishloginAction extends Action
                 // TRANS: in the same StatusNet site.
                 _m('There is already a local account linked with that Facebook account.')
             );
-
-            return;
         }
     }
 
@@ -329,7 +324,6 @@ class FacebookfinishloginAction extends Action
         if (common_config('site', 'closed')) {
             // TRANS: Client error trying to register with registrations not allowed.
             $this->clientError(_m('Registration not allowed.'));
-            return;
         }
 
         $invite = null;
@@ -339,7 +333,6 @@ class FacebookfinishloginAction extends Action
             if (empty($code)) {
                 // TRANS: Client error trying to register with registrations 'invite only'.
                 $this->clientError(_m('Registration not allowed.'));
-                return;
             }
 
             $invite = Invitation::getKV($code);
@@ -347,7 +340,6 @@ class FacebookfinishloginAction extends Action
             if (empty($invite)) {
                 // TRANS: Client error trying to register with an invalid invitation code.
                 $this->clientError(_m('Not a valid invitation code.'));
-                return;
             }
         }
 
@@ -384,7 +376,6 @@ class FacebookfinishloginAction extends Action
         if (!$result) {
             // TRANS: Server error displayed when connecting to Facebook fails.
             $this->serverError(_m('Error connecting user to Facebook.'));
-            return;
         }
 
         // Add a Foreign_user record
@@ -518,7 +509,6 @@ class FacebookfinishloginAction extends Action
         if (empty($result)) {
             // TRANS: Server error displayed when connecting to Facebook fails.
             $this->serverError(_m('Error connecting user to Facebook.'));
-            return;
         }
     }
 
index 3f56421b4f270b332a2b4c8006225116f6dd2757..31e020a3ce7a42351856245ae2d13b0a82007946 100644 (file)
@@ -261,7 +261,6 @@ class FacebooksettingsAction extends SettingsAction {
             common_log_db_error($user, 'DELETE', __FILE__);
             // TRANS: Server error displayed when deleting the link to a Facebook account fails.
             $this->serverError(_m('Could not delete link to Facebook.'));
-            return;
         }
 
         // TRANS: Confirmation message. StatusNet account was unlinked from Facebook.
index 0076af1bd658d3150d9175c00f75019772eb0015..1feea867716baa25a26854b83fb9477f079017e8 100644 (file)
@@ -63,14 +63,12 @@ class ProfilefieldsAdminPanelAction extends AdminPanelAction
         $field->systemname = $this->trimmed('systemname');
         if (!gnusocial_field_systemname_validate($field->systemname)) {
             $this->clientError(_('Internal system name must be unique and consist of only alphanumeric characters!'));
-            return false;
         }
         if ($field->id) {
             if ($field->validate())
                 $field->update();
             else {
                 $this->clientError(_('There was an error with the field data.'));
-                return false;
             }
         }
         else {
index 84952d0b062cf116e73deb6fe815055de8ba13e0..f4656bfd8e4588fdf460a628750b0d5d7bedb37e 100644 (file)
@@ -73,7 +73,6 @@ class GroupinboxAction extends GroupAction
         if ($nickname != $nicknameArg) {
             $url = common_local_url('groupinbox', array('nickname' => $nickname));
             common_redirect($url);
-            return false;
         }
 
         $localGroup = Local_group::getKV('nickname', $nickname);
index 9b611edc860cb4c6a785b4b4704af9c6dd5c6876..be6a3cd86b6c875daf843af17f8fe59ed1680d1a 100644 (file)
@@ -81,7 +81,6 @@ class NewgroupmessageAction extends Action
         if ($nickname != $nicknameArg) {
             $url = common_local_url('newgroupmessage', array('nickname' => $nickname));
             common_redirect($url, 301);
-            return false;
         }
 
         $localGroup = Local_group::getKV('nickname', $nickname);
index 3c0a504eb06c80a384e5c39ae271c017c1a312f0..3dae68d133f26c05f476949feff240df2d24baf9 100644 (file)
@@ -245,7 +245,6 @@ class IrcManager extends ImManager {
                         common_log_db_error($confirm, 'DELETE', __FILE__);
                         // TRANS: Server error thrown on database error when deleting IRC nickname confirmation.
                         $this->serverError(_m('Could not delete confirmation.'));
-                        return;
                     }
                 }
             }
index 15f3b941d703be0de8a9216b62a84258c8a192b6..c03e6119d88993465f60ec2b72960afca97fb64c 100644 (file)
@@ -63,7 +63,6 @@ class MapAction extends Action
                 $args['page'] = $this->arg['page'];
             }
             common_redirect(common_local_url($this->trimmed('action'), $args), 301);
-            return false;
         }
 
         $this->user = User::getKV('nickname', $nickname);
@@ -71,7 +70,6 @@ class MapAction extends Action
         if (!$this->user) {
             // TRANS: Client error displayed when referring to a non-existing user.
             $this->clientError(_m('No such user.'), 404);
-            return false;
         }
 
         $this->profile = $this->user->getProfile();
@@ -79,7 +77,6 @@ class MapAction extends Action
         if (!$this->profile) {
             // TRANS: Error message displayed when referring to a user without a profile.
             $this->serverError(_m('User has no profile.'));
-            return false;
         }
 
         $page = $this->trimmed('page');
index b7bee7c8815e572b6a4fa301a7b608fe951383df..d7ec5d1392003cd0499584e169648e91424c836e 100644 (file)
@@ -48,12 +48,10 @@ class MinifyAction extends Action
             } else {
                 // TRANS: Client error displayed when not providing a valid path in parameter "f".
                 $this->clientError(_m('The parameter "f" is not a valid path.'),404);
-                return false;
             }
         }else{
             // TRANS: Client error displayed when not providing parameter "f".
             $this->clientError(_m('The parameter "f" is required but missing.'),500);
-            return false;
         }
     }
 
@@ -112,7 +110,6 @@ class MinifyAction extends Action
             default:
                 // TRANS: Client error displayed when trying to minify an unsupported file type.
                 $this->clientError(_m('File type not supported.'),500);
-                return false;
         }
         return $out;
     }
index 75d4fb74bc0f191aee973f5d643326ea8802482f..eeb3fffd284610b5e85f8c2975a61cf35a2b16d1 100644 (file)
@@ -281,7 +281,6 @@ class OStatusPlugin extends Plugin
 
                 // redirect to the new profile.
                 common_redirect(common_local_url('tagprofile', array('id' => $oprofile->profile_id)), 303);
-                return false;
 
             } catch (Exception $e) {
                 // TRANS: Error message in OStatus plugin. Do not translate the domain names example.com
index c8b2a73b681e42430c0e3958ae1b34ba8de67d32..e23312055fd3933ceca5ccff6b0643e779d14d59 100644 (file)
@@ -145,7 +145,6 @@ class GroupsalmonAction extends SalmonAction
         if (Group_block::isBlocked($this->group, $profile)) {
             // TRANS: Client error displayed when trying to join a group the user is blocked from by a group admin.
             $this->clientError(_m('You have been blocked from that group by the admin.'), 403);
-            return false;
         }
 
         try {
index 15392855cbd9ff461dddc093d29aa891cc34678a..fa999fe09a759b6b4d70d3fc5173e669d764bba9 100644 (file)
@@ -38,7 +38,6 @@ class OStatusTagAction extends OStatusInitAction
         if (common_logged_in()) {
             // TRANS: Client error displayed when trying to list a local object as if it is remote.
             $this->clientError(_m('You can use the local list functionality!'));
-            return false;
         }
 
         $this->nickname = $this->trimmed('nickname');
index b0bdc2885c1c41d5f9042655d31d070e4a5f2635..5adb56662619694c625bd9449fb94cb81c2146d9 100644 (file)
@@ -132,10 +132,8 @@ class OpenIDPlugin extends Plugin
                 $action = trim($args['action']);
                 if (in_array($action, array('login', 'register'))) {
                     common_redirect(common_local_url('openidlogin'));
-                    exit(0);
                 } else if ($action == 'passwordsettings') {
                     common_redirect(common_local_url('openidsettings'));
-                    exit(0);
                 } else if ($action == 'recoverpassword') {
                     // TRANS: Client exception thrown when an action is not available.
                     throw new ClientException(_m('Unavailable action.'));
@@ -444,7 +442,6 @@ class OpenIDPlugin extends Plugin
     {
         if (common_config('site', 'openid_only') || (!empty($user) && User_openid::hasOpenID($user->id))) {
             common_redirect(common_local_url('openidlogin'), 303);
-            return false;
         }
         return true;
     }
index 372c1e12b5d961eeda2ab07c8e31339b91b2fe40..3a99988fb59093fccf76e6b064ab35482abdae26 100644 (file)
@@ -322,7 +322,6 @@ class FinishopenidloginAction extends Action
         if (common_config('site', 'closed')) {
             // TRANS: OpenID plugin message. No new user registration is allowed on the site.
             $this->clientError(_m('Registration not allowed.'));
-            return;
         }
 
         $invite = null;
@@ -332,7 +331,6 @@ class FinishopenidloginAction extends Action
             if (empty($code)) {
                 // TRANS: OpenID plugin message. No new user registration is allowed on the site without an invitation code, and none was provided.
                 $this->clientError(_m('Registration not allowed.'));
-                return;
             }
 
             $invite = Invitation::getKV($code);
@@ -340,7 +338,6 @@ class FinishopenidloginAction extends Action
             if (empty($invite)) {
                 // TRANS: OpenID plugin message. No new user registration is allowed on the site without an invitation code, and the one provided was not valid.
                 $this->clientError(_m('Not a valid invitation code.'));
-                return;
             }
         }
 
@@ -356,7 +353,6 @@ class FinishopenidloginAction extends Action
         if (!$display || !$canonical) {
             // TRANS: OpenID plugin server error. A stored OpenID cannot be retrieved.
             $this->serverError(_m('Stored OpenID not found.'));
-            return;
         }
 
         // Possible race condition... let's be paranoid
@@ -366,7 +362,6 @@ class FinishopenidloginAction extends Action
         if ($other) {
             // TRANS: OpenID plugin server error.
             $this->serverError(_m('Creating new account for OpenID that already has a user.'));
-            return;
         }
 
         Event::handle('StartOpenIDCreateNewUser', array($canonical, &$sreg));
@@ -418,8 +413,7 @@ class FinishopenidloginAction extends Action
 
         Event::handle('EndRegistrationTry', array($this));
 
-        common_redirect(common_local_url('showstream', array('nickname' => $user->nickname)),
-                        303);
+        common_redirect(common_local_url('showstream', array('nickname' => $user->nickname)), 303);
     }
 
     function connectUser()
@@ -442,7 +436,6 @@ class FinishopenidloginAction extends Action
         if (!$display || !$canonical) {
             // TRANS: OpenID plugin server error. A stored OpenID cannot be found.
             $this->serverError(_m('Stored OpenID not found.'));
-            return;
         }
 
         $result = oid_link_user($user->id, $canonical, $display);
@@ -450,7 +443,6 @@ class FinishopenidloginAction extends Action
         if (!$result) {
             // TRANS: OpenID plugin server error. The user or user profile could not be saved.
             $this->serverError(_m('Error connecting user to OpenID.'));
-            return;
         }
 
         if (Event::handle('StartOpenIDUpdateUser', array($user, $canonical, &$sreg))) {
index 847eb48c65fbb773b7a9d585eea81951a1bf414a..b50a9129d72cb1da03e0c8fe75f8b7d914f7a648 100644 (file)
@@ -76,7 +76,6 @@ class OpenidserverAction extends Action
                     // were POSTed here.
                     common_set_returnto($_SERVER['REQUEST_URI']);
                     common_redirect(common_local_url('login'), 303);
-                    return;
                 }
             }else if(common_profile_url($user->nickname) == $request->identity || $request->idSelect()){
                 $user_openid_trustroot = User_openid_trustroot::pkeyGet(
@@ -101,7 +100,6 @@ class OpenidserverAction extends Action
                         // prompting user for form resubmission if we
                         // were POSTed here.
                         common_redirect(common_local_url('openidtrust'), 303);
-                        return;
                     }
                 }else{
                     //user has previously authorized this trust root
index 8f046429a011ea7662e49101453d27cd13c1c528..3f8a3a93b77afb73b0e6d3457684172cbda11b2f 100644 (file)
@@ -60,7 +60,6 @@ class OpenidtrustAction extends Action
             /* Go log in, and then come back. */
             common_set_returnto($_SERVER['REQUEST_URI']);
             common_redirect(common_local_url('login'));
-            return;
         }
         $this->trust_root = $_SESSION['openid_trust_root'];
         $this->allowUrl = $_SESSION['openid_allow_url'];
@@ -68,7 +67,6 @@ class OpenidtrustAction extends Action
         if(empty($this->trust_root) || empty($this->allowUrl) || empty($this->denyUrl)){
             // TRANS: Client error when visiting page directly.
             $this->clientError(_m('This page should only be reached during OpenID processing, not directly.'));
-            return;
         }
         return true;
     }
index dcd072e8f85922432b93ab6bc95c0750508e9653..cf9e5e5e949f3de1c4f2ee8ca499e6f026e4c5da 100644 (file)
@@ -76,7 +76,6 @@ class SearchsubAction extends Action
             // TRANS: Client error displayed trying to perform any request method other than POST.
             // TRANS: Do not translate POST.
             $this->clientError(_m('This action only accepts POST requests.'));
-            return false;
         }
 
         // CSRF protection
@@ -87,7 +86,6 @@ class SearchsubAction extends Action
             // TRANS: Client error displayed when the session token is not okay.
             $this->clientError(_m('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         // Only for logged-in users
@@ -97,7 +95,6 @@ class SearchsubAction extends Action
         if (empty($this->user)) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_m('Not logged in.'));
-            return false;
         }
 
         // Profile to subscribe to
@@ -107,7 +104,6 @@ class SearchsubAction extends Action
         if (empty($this->search)) {
             // TRANS: Client error displayed trying to subscribe to a non-existing profile.
             $this->clientError(_m('No such profile.'));
-            return false;
         }
 
         return true;
index b3dc1a0517bda905eba2ebce7c64d6a2bacb5ee9..8ed6c224e71a625ee05246a837fc2beb6c4c8001 100644 (file)
@@ -116,7 +116,6 @@ abstract class BaseMirrorAction extends Action
         if ($_SERVER['REQUEST_METHOD'] != 'POST') {
             // TRANS: Client error displayed when trying to use another method than POST.
             $this->clientError(_m('This action only accepts POST requests.'));
-            return false;
         }
 
         // CSRF protection
@@ -126,7 +125,6 @@ abstract class BaseMirrorAction extends Action
             // TRANS: Client error displayed when the session token does not match or is not given.
             $this->clientError(_m('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         // Only for logged-in users
@@ -136,7 +134,6 @@ abstract class BaseMirrorAction extends Action
         if (empty($this->user)) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_m('Not logged in.'));
-            return false;
         }
         return true;
     }
index dd8c76b53decd12755280bd6ae991c0a2cd50321..4550716cab8f39bb63c004dbbe15dff70e577f53 100644 (file)
@@ -76,7 +76,6 @@ class TagsubAction extends Action
             // TRANS: Client error displayed trying to perform any request method other than POST.
             // TRANS: Do not translate POST.
             $this->clientError(_m('This action only accepts POST requests.'));
-            return false;
         }
 
         // CSRF protection
@@ -87,7 +86,6 @@ class TagsubAction extends Action
             // TRANS: Client error displayed when the session token is not okay.
             $this->clientError(_m('There was a problem with your session token.'.
                                  ' Try again, please.'));
-            return false;
         }
 
         // Only for logged-in users
@@ -97,7 +95,6 @@ class TagsubAction extends Action
         if (empty($this->user)) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_m('Not logged in.'));
-            return false;
         }
 
         // Profile to subscribe to
@@ -107,7 +104,6 @@ class TagsubAction extends Action
         if (empty($this->tag)) {
             // TRANS: Client error displayed trying to subscribe to a non-existing profile.
             $this->clientError(_m('No such profile.'));
-            return false;
         }
 
         return true;
index 3c95d243c40c5cdc658cf3015e4fe98c5f45d5b7..aace7e4f09c67763003e1ebb26b0c2950960e0b8 100644 (file)
@@ -496,7 +496,6 @@ class TwitterauthorizationAction extends Action
         if (common_config('site', 'closed')) {
             // TRANS: Client error displayed when trying to create a new user while creating new users is not allowed.
             $this->clientError(_m('Registration not allowed.'));
-            return;
         }
 
         $invite = null;
@@ -506,7 +505,6 @@ class TwitterauthorizationAction extends Action
             if (empty($code)) {
                 // TRANS: Client error displayed when trying to create a new user while creating new users is not allowed.
                 $this->clientError(_m('Registration not allowed.'));
-                return;
             }
 
             $invite = Invitation::getKV($code);
@@ -514,7 +512,6 @@ class TwitterauthorizationAction extends Action
             if (empty($invite)) {
                 // TRANS: Client error displayed when trying to create a new user with an invalid invitation code.
                 $this->clientError(_m('Not a valid invitation code.'));
-                return;
             }
         }
 
@@ -543,7 +540,6 @@ class TwitterauthorizationAction extends Action
         if (empty($user)) {
             // TRANS: Server error displayed when creating a new user has failed.
             $this->serverError(_m('Error registering user.'));
-            return;
         }
 
         $result = $this->saveForeignLink($user->id,
@@ -555,7 +551,6 @@ class TwitterauthorizationAction extends Action
         if (!$result) {
             // TRANS: Server error displayed when connecting a user to a Twitter user has failed.
             $this->serverError(_m('Error connecting user to Twitter.'));
-            return;
         }
 
         common_set_user($user);
@@ -566,8 +561,7 @@ class TwitterauthorizationAction extends Action
 
         Event::handle('EndRegistrationTry', array($this));
 
-        common_redirect(common_local_url('showstream', array('nickname' => $user->nickname)),
-                        303);
+        common_redirect(common_local_url('showstream', array('nickname' => $user->nickname)), 303);
     }
 
     function connectNewUser()
@@ -598,7 +592,6 @@ class TwitterauthorizationAction extends Action
         if (!$result) {
             // TRANS: Server error displayed connecting a user to a Twitter user has failed.
             $this->serverError(_m('Error connecting user to Twitter.'));
-            return;
         }
 
         common_debug('TwitterBridge Plugin - ' .
@@ -619,7 +612,6 @@ class TwitterauthorizationAction extends Action
         if (empty($result)) {
             // TRANS: Server error displayed connecting a user to a Twitter user has failed.
             $this->serverError(_m('Error connecting user to Twitter.'));
-            return;
         }
 
         common_debug('TwitterBridge Plugin - ' .
index 88799e75a9817964b1c3eb6fe1161edb445745a6..cc3e34f345e15835c123ea21078443ffe37a1d29 100644 (file)
@@ -263,7 +263,6 @@ class TwittersettingsAction extends ProfileSettingsAction
         if (empty($flink)) {
             // TRANS: Client error displayed when trying to remove a connected Twitter account when there isn't one connected.
             $this->clientError(_m('No Twitter connection to remove.'));
-            return;
         }
 
         $result = $flink->safeDelete();
@@ -272,7 +271,6 @@ class TwittersettingsAction extends ProfileSettingsAction
             common_log_db_error($flink, 'DELETE', __FILE__);
             // TRANS: Server error displayed when trying to remove a connected Twitter account fails.
             $this->serverError(_m('Could not remove Twitter user.'));
-            return;
         }
 
         // TRANS: Success message displayed after disconnecting a Twitter account.
index b7538c6663368b5c99b06d7d07ea28f4f391fd7d..9b5b4088bf0f2555c1bf619750fda7a85ea296e6 100644 (file)
@@ -63,7 +63,6 @@ class AdminprofileflagAction extends Action
         if (!common_logged_in()) {
             // TRANS: Error message displayed when trying to perform an action that requires a logged in user.
             $this->clientError(_m('Not logged in.'));
-            return;
         }
 
         $user = common_current_user();
@@ -88,7 +87,6 @@ class AdminprofileflagAction extends Action
         if (!$user->hasRight(UserFlagPlugin::REVIEWFLAGS)) {
             // TRANS: Error message displayed when trying to review profile flags while not authorised.
             $this->clientError(_m('You cannot review profile flags.'));
-            return false;
         }
 
         $this->page = $this->trimmed('page');