]> gitweb.fluxo.info Git - lorea/elgg.git/commitdiff
Fixes #4059 returning true when handling a page
authorcash <cash.costello@gmail.com>
Sat, 5 Nov 2011 03:29:54 +0000 (23:29 -0400)
committercash <cash.costello@gmail.com>
Sat, 5 Nov 2011 03:29:54 +0000 (23:29 -0400)
29 files changed:
engine/lib/admin.php
engine/lib/cron.php
engine/lib/elgglib.php
engine/lib/entities.php
engine/lib/pagehandler.php
engine/lib/river.php
engine/lib/tags.php
engine/lib/user_settings.php
engine/lib/users.php
mod/blog/start.php
mod/bookmarks/start.php
mod/categories/start.php
mod/dashboard/start.php
mod/developers/start.php
mod/externalpages/start.php
mod/file/start.php
mod/groups/start.php
mod/invitefriends/start.php
mod/members/start.php
mod/messageboard/start.php
mod/messages/start.php
mod/notifications/start.php
mod/pages/start.php
mod/profile/start.php
mod/reportedcontent/start.php
mod/search/start.php
mod/thewire/start.php
mod/twitter_api/start.php
mod/uservalidationbyemail/start.php

index c0a5c099597400edafb7834c10f0d2b85f63a1e2..3baf2ff617ff43c73fe5c5124a3287b24483aa3a 100644 (file)
@@ -427,7 +427,7 @@ function admin_pagesetup() {
  *
  * @param array $page Array of pages
  *
- * @return void
+ * @return bool
  * @access private
  */
 function admin_page_handler($page) {
@@ -478,6 +478,7 @@ function admin_page_handler($page) {
 
        $body = elgg_view_layout('admin', array('content' => $content, 'title' => $title));
        echo elgg_view_page($title, $body, 'admin');
+       return true;
 }
 
 /**
@@ -485,7 +486,7 @@ function admin_page_handler($page) {
  * admin_plugin_screenshot/<plugin_id>/<size>/<ss_name>.<ext>
  *
  * @param array $pages The pages array
- * @return void
+ * @return bool
  * @access private
  */
 function admin_plugin_screenshot_page_handler($pages) {
@@ -524,6 +525,7 @@ function admin_plugin_screenshot_page_handler($pages) {
                        echo file_get_contents($file);
                        break;
        }
+       return true;
 }
 
 /**
@@ -539,7 +541,7 @@ function admin_plugin_screenshot_page_handler($pages) {
  *     * LICENSE.txt
  *
  * @param type $page
- * @return void
+ * @return bool
  * @access private
  */
 function admin_markdown_page_handler($pages) {
@@ -595,6 +597,7 @@ function admin_markdown_page_handler($pages) {
        ));
        
        echo elgg_view_page($title, $body, 'admin');
+       return true;
 }
 
 /**
index 8c0b62a531869533d596f14178b56f4efba60a7b..8af640396f3c31f1ee050b6fe7f2400b766dee25 100644 (file)
@@ -25,7 +25,7 @@ function cron_init() {
  *
  * @param array $page Pages
  *
- * @return void
+ * @return bool
  * @access private
  */
 function cron_page_handler($page) {
@@ -63,6 +63,7 @@ function cron_page_handler($page) {
        $std_out = ob_get_clean();
 
        echo $std_out . $old_stdout;
+       return true;
 }
 
 /**
index 03774deb471689b5bbcb4ab88ff3b595345b34ce..10804fdcac7470d106557b7350b5975ea983b767 100644 (file)
@@ -1740,12 +1740,12 @@ function _elgg_shutdown_hook() {
  *
  * @param array $page The page array
  *
- * @return void
+ * @return bool
  * @elgg_pagehandler js
  * @access private
  */
 function elgg_js_page_handler($page) {
-       elgg_cacheable_view_page_handler($page, 'js');
+       return elgg_cacheable_view_page_handler($page, 'js');
 }
 
 /**
@@ -1755,7 +1755,7 @@ function elgg_js_page_handler($page) {
  *
  * @param array $page The page array
  *
- * @return void
+ * @return bool
  * @elgg_pagehandler ajax
  * @access private
  */
@@ -1776,7 +1776,9 @@ function elgg_ajax_page_handler($page) {
                }
 
                echo elgg_view($view, $vars);
+               return true;
        }
+       return false;
 }
 
 /**
@@ -1796,7 +1798,7 @@ function elgg_css_page_handler($page) {
                $page[0] = 'elgg';
        }
        
-       elgg_cacheable_view_page_handler($page, 'css');
+       return elgg_cacheable_view_page_handler($page, 'css');
 }
 
 /**
@@ -1807,7 +1809,7 @@ function elgg_css_page_handler($page) {
  * @param array  $page The page array
  * @param string $type The type: js or css
  *
- * @return void
+ * @return bool
  * @access private
  */
 function elgg_cacheable_view_page_handler($page, $type) {
@@ -1822,7 +1824,7 @@ function elgg_cacheable_view_page_handler($page, $type) {
                        break;
 
                default:
-                       return;
+                       return false;
                        break;
        }
 
@@ -1848,6 +1850,7 @@ function elgg_cacheable_view_page_handler($page, $type) {
                //header("Content-Length: " . strlen($return));
 
                echo $return;
+               return true;
        }
 }
 
index 98e48641a1a6dfa4e485de4e8327b2146a5595db..4b8b6694f9a817956638544b78e1e35e83c5ff77 100644 (file)
@@ -2071,7 +2071,7 @@ function is_registered_entity_type($type, $subtype = null) {
  *
  * @param array $page Page elements from pain page handler
  *
- * @return void
+ * @return bool
  * @elgg_page_handler view
  * @access private
  */
@@ -2080,7 +2080,9 @@ function entities_page_handler($page) {
                global $CONFIG;
                set_input('guid', $page[0]);
                include($CONFIG->path . "pages/entities/index.php");
+               return true;
        }
+       return false;
 }
 
 /**
index 91e568c1dc784ce3d0392e6871597278e9279f60..f331e2aae367dac07be1555c443332634cd7db93 100644 (file)
@@ -40,12 +40,13 @@ function page_handler($handler, $page) {
        $handler = $params['handler'];
        $page = $params['segments'];
 
+       $result = false;
        if (isset($CONFIG->pagehandler) && !empty($handler) && isset($CONFIG->pagehandler[$handler])) {
                $function = $CONFIG->pagehandler[$handler];
-               call_user_func($function, $page, $handler);
+               $result = call_user_func($function, $page, $handler);
        }
 
-       return headers_sent();
+       return $result || headers_sent();
 }
 
 /**
@@ -64,6 +65,7 @@ function page_handler($handler, $page) {
  * The context is set to the page handler identifier before the registered
  * page handler function is called. For the above example, the context is set to 'blog'.
  *
+ * Page handlers should return true to indicate that they handled the request.
  * Requests not handled are forwarded to the front page with a reason of 404.
  * Plugins can register for the 'forward', '404' plugin hook. @see forward()
  *
index b2fead824ce764f3349c1a553bdfa6975a9c08fc..466eca253113422afe13760766527d215d7e5049 100644 (file)
@@ -586,7 +586,7 @@ function update_river_access_by_object($object_guid, $access_id) {
  * Page handler for activiy
  *
  * @param array $page
- * @return void
+ * @return bool
  * @access private
  */
 function elgg_river_page_handler($page) {
@@ -607,6 +607,7 @@ function elgg_river_page_handler($page) {
        $entity_subtype = '';
 
        require_once("{$CONFIG->path}pages/river.php");
+       return true;
 }
 
 /**
index b2c9a672fc7ccdb24390b8f5df412da1da6689d8..a0887d0f3180db5a38a5b2ad95986d387731b53e 100644 (file)
@@ -321,7 +321,7 @@ function elgg_get_registered_tag_metadata_names() {
  *
  * @param array $page Page array
  *
- * @return void
+ * @return bool
  * @access private
  */
 function elgg_tagcloud_page_handler($page) {
@@ -337,6 +337,7 @@ function elgg_tagcloud_page_handler($page) {
        $body = elgg_view_layout('one_sidebar', array('content' => $content));
 
        echo elgg_view_page(elgg_echo('tags:site_cloud'), $body);
+       return true;
 }
 
 /**
index 12084662bf813f8c0034b8df316e4cf06c751996..1e2d6db101bd08d90b0d52c5c517082b91f3045a 100644 (file)
@@ -294,7 +294,7 @@ function usersettings_pagesetup() {
  *
  * @param array $page Pages array
  *
- * @return void
+ * @return bool
  * @access private
  */
 function usersettings_page_handler($page) {
@@ -330,6 +330,7 @@ function usersettings_page_handler($page) {
 
        if (isset($path)) {
                require $path;
+               return true;
        }
 }
 
index 3001a2ac6635724038952ab24572cd713d50ba1a..3aff2ca9d4e52337f5e0f5a4a60a32719beddbcd 100644 (file)
@@ -1048,7 +1048,7 @@ function collections_submenu_items() {
  *
  * @param array $page_elements Page elements
  *
- * @return void
+ * @return bool
  * @access private
  */
 function friends_page_handler($page_elements) {
@@ -1059,6 +1059,7 @@ function friends_page_handler($page_elements) {
                collections_submenu_items();
        }
        require_once(dirname(dirname(dirname(__FILE__))) . "/pages/friends/index.php");
+       return true;
 }
 
 /**
@@ -1066,7 +1067,7 @@ function friends_page_handler($page_elements) {
  *
  * @param array $page_elements Page elements
  *
- * @return void
+ * @return bool
  * @access private
  */
 function friends_of_page_handler($page_elements) {
@@ -1078,6 +1079,7 @@ function friends_of_page_handler($page_elements) {
                collections_submenu_items();
        }
        require_once(dirname(dirname(dirname(__FILE__))) . "/pages/friends/of.php");
+       return true;
 }
 
 /**
@@ -1085,7 +1087,7 @@ function friends_of_page_handler($page_elements) {
  *
  * @param array $page_elements Page elements
  *
- * @return void
+ * @return bool
  * @access private
  */
 function collections_page_handler($page_elements) {
@@ -1096,6 +1098,7 @@ function collections_page_handler($page_elements) {
                        set_page_owner(elgg_get_logged_in_user_guid());
                        collections_submenu_items();
                        require_once "{$base}pages/friends/collections/add.php";
+                       return true;
                } else {
                        $user = get_user_by_username($page_elements[0]);
                        if ($user) {
@@ -1104,9 +1107,11 @@ function collections_page_handler($page_elements) {
                                        collections_submenu_items();
                                }
                                require_once "{$base}pages/friends/collections/view.php";
+                               return true;
                        }
                }
        }
+       return false;
 }
 
 /**
@@ -1115,7 +1120,7 @@ function collections_page_handler($page_elements) {
  * @param array  $page_elements Page elements
  * @param string $handler The handler string
  *
- * @return void
+ * @return bool
  * @access private
  */
 function elgg_user_account_page_handler($page_elements, $handler) {
@@ -1131,7 +1136,10 @@ function elgg_user_account_page_handler($page_elements, $handler) {
                case 'register':
                        require_once("$base_dir/register.php");
                        break;
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
@@ -1140,17 +1148,18 @@ function elgg_user_account_page_handler($page_elements, $handler) {
  * This is a fallback for non-JS users who click on the
  * dropdown login link.
  *
- * @return void
+ * @return bool
  * @access private
  */
 function elgg_user_login_page_handler() {
        if (elgg_is_logged_in()) {
-               forward();
+               forward('');
        }
 
        $login_box = elgg_view('core/account/login_box');
        $content = elgg_view_layout('one_column', array('content' => $login_box));
        echo elgg_view_page(elgg_echo('login'), $content);
+       return true;
 }
 
 /**
@@ -1404,7 +1413,7 @@ function elgg_profile_fields_setup() {
  * /avatar/view/<username>/<size>/<icontime>
  *
  * @param array $page
- * @return void
+ * @return bool
  * @access private
  */
 function elgg_avatar_page_handler($page) {
@@ -1417,17 +1426,20 @@ function elgg_avatar_page_handler($page) {
 
        if ($page[0] == 'edit') {
                require_once("{$CONFIG->path}pages/avatar/edit.php");
+               return true;
        } else {
                set_input('size', $page[2]);
                require_once("{$CONFIG->path}pages/avatar/view.php");
+               return true;
        }
+       return false;
 }
 
 /**
  * Profile page handler
  *
  * @param array $page
- * @return void
+ * @return bool
  * @access private
  */
 function elgg_profile_page_handler($page) {
@@ -1438,7 +1450,9 @@ function elgg_profile_page_handler($page) {
 
        if ($page[1] == 'edit') {
                require_once("{$CONFIG->path}pages/profile/edit.php");
+               return true;
        }
+       return false;
 }
 
 /**
index b2da00bc769f7c6a00eba6dd7f43da35c7627d9a..052c636510808053efbb3cf8be680e3d08536bf7 100644 (file)
@@ -93,7 +93,7 @@ function blog_init() {
  * @todo no archives for all blogs or friends
  *
  * @param array $page
- * @return void
+ * @return bool
  */
 function blog_page_handler($page) {
 
@@ -141,15 +141,16 @@ function blog_page_handler($page) {
                case 'all':
                        $params = blog_get_page_content_list();
                        break;
+               default:
+                       return false;
        }
 
-       if (isset($params)) {
-               $params['sidebar'] .= elgg_view('blog/sidebar', array('page' => $page_type));
+       $params['sidebar'] .= elgg_view('blog/sidebar', array('page' => $page_type));
 
-               $body = elgg_view_layout('content', $params);
+       $body = elgg_view_layout('content', $params);
 
-               echo elgg_view_page($params['title'], $body);
-       }
+       echo elgg_view_page($params['title'], $body);
+       return true;
 }
 
 /**
index 6d9dc9b3d0aab141ebe781f5938da14766b9cf36..21a2d8940fb4204a641c1e9467e049596d726c06 100644 (file)
@@ -83,7 +83,7 @@ function bookmarks_init() {
  * Title is ignored
  *
  * @param array $page
- * @return void
+ * @return bool
  */
 function bookmarks_page_handler($page) {
        elgg_load_library('elgg:bookmarks');
@@ -146,9 +146,13 @@ function bookmarks_page_handler($page) {
                        set_input('container_guid', $page[1]);
                        include "$pages/bookmarklet.php";
                        break;
+
+               default:
+                       return false;
        }
 
        elgg_pop_context();
+       return true;
 }
 
 /**
index 3cec516f1b7eb2c28adabb4bc0b11f0c42a072a8..0aacf11e7a645e583568f5af498a96d47fbe0ffc 100644 (file)
@@ -28,10 +28,11 @@ function categories_init() {
 
 /**
  * Category page handler
- * @return void
+ * @return bool
  */
 function categories_page_handler() {
        include(dirname(__FILE__) . "/pages/categories/listing.php");
+       return true;
 }
 
 /**
index 0197ee64f15179013142a760a0b9b3aa054fe305..65007d1a8aea856419e1f550e8f52a6346d5946c 100644 (file)
@@ -31,7 +31,7 @@ function dashboard_init() {
 
 /**
  * Dashboard page handler
- * @return void
+ * @return bool
  */
 function dashboard_page_handler() {
        // Ensure that only logged-in users can see this page
@@ -55,6 +55,7 @@ function dashboard_page_handler() {
        $body = elgg_view_layout('one_column', array('content' => $widgets));
 
        echo elgg_view_page($title, $body);
+       return true;
 }
 
 
index c89e3d36f2944b64f8a5b53d92997a425ff53062..2e3df76638e5e4b8c8749083a6f83533c4de06c9 100644 (file)
@@ -157,7 +157,7 @@ function developers_log_events($name, $type) {
  * Serve the theme preview pages
  *
  * @param array $page
- * @return void
+ * @return bool
  */
 function developers_theme_preview_controller($page) {
        if (!isset($page[0])) {
@@ -192,4 +192,5 @@ function developers_theme_preview_controller($page) {
        ));
        
        echo elgg_view_page($title, $layout, 'theme_preview');
+       return true;
 }
index 271051ba93014ec9d53cb2d27fc6158814c8c225..13235af539fb1b00267f2209fd86fcdcfb3b11d7 100644 (file)
@@ -53,7 +53,7 @@ function expages_setup_footer_menu() {
  *
  * @param array  $page    URL segements
  * @param string $handler Handler identifier
- * @return void
+ * @return bool
  */
 function expages_page_handler($page, $handler) {
        if ($handler == 'expages') {
@@ -77,6 +77,7 @@ function expages_page_handler($page, $handler) {
 
        $body = elgg_view_layout("one_sidebar", array('content' => $content));
        echo elgg_view_page($title, $body);
+       return true;
 }
 
 /**
index bb20fc7ba8ab5ad5391caaabf15581811623149b..f8b7fa4236d27454156bbd4dc8d98aa83d7666dd 100644 (file)
@@ -100,7 +100,7 @@ function file_init() {
  * Title is ignored
  *
  * @param array $page
- * @return void
+ * @return bool
  */
 function file_page_handler($page) {
 
@@ -138,7 +138,10 @@ function file_page_handler($page) {
                case 'all':
                        include "$file_dir/world.php";
                        break;
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
index 710ccc0b25a3cd25eb6240614f4c292ddc94f311..5469da01341185b64fbf5ee814eb5e15dba0dacd 100644 (file)
@@ -192,7 +192,7 @@ function groups_setup_sidebar_menus() {
  *  Group members:        groups/members/<guid>
  *
  * @param array $page Array of url segments for routing
- * @return void
+ * @return bool
  */
 function groups_page_handler($page) {
 
@@ -239,7 +239,10 @@ function groups_page_handler($page) {
                case 'requests':
                        groups_handle_requests_page($page[1]);
                        break;
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
@@ -752,7 +755,7 @@ function discussion_init() {
  *  Edit discussion topic: discussion/edit/<guid>
  *
  * @param array $page Array of url segments for routing
- * @return void
+ * @return bool
  */
 function discussion_page_handler($page) {
 
@@ -776,7 +779,10 @@ function discussion_page_handler($page) {
                case 'view':
                        discussion_handle_view_page($page[1]);
                        break;
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
index 61cfed96c5b8f771a13da07d8a64da002aee5922..be853f2cec29f984666a9a63a4344b3604d8f9f7 100644 (file)
@@ -27,7 +27,7 @@ function invitefriends_init() {
  * Page handler function
  * 
  * @param array $page Page URL segments
- * @return void
+ * @return bool
  */
 function invitefriends_page_handler($page) {
        gatekeeper();
@@ -46,4 +46,5 @@ function invitefriends_page_handler($page) {
        $body = elgg_view_layout('one_sidebar', $params);
 
        echo elgg_view_page($title, $body);
+       return true;
 }
index 1d734da1786cd8c8cbce0ea3c5665fe2f04b686d..54787cd16d208a5b6253f3939d25f9f58cc571d5 100644 (file)
@@ -19,7 +19,7 @@ function members_init() {
  * Members page handler
  *
  * @param array $page url segments
- * @return void
+ * @return bool
  */
 function members_page_handler($page) {
        $base = elgg_get_plugins_path() . 'members/pages/members';
@@ -37,4 +37,5 @@ function members_page_handler($page) {
        } else {
                require_once "$base/index.php";
        }
+       return true;
 }
index 9db16cf3b9bcf7fa1490bbb2951b9e96ccd293db..0c65edc4ff86466297bcb03f32b0f01b6a5d1b2c 100644 (file)
@@ -42,7 +42,7 @@ function messageboard_init() {
  *  Group messageboard:                messageboard/group/<guid>/all (not implemented)
  *
  * @param array $page Array of page elements
- * @return void
+ * @return bool
  */
 function messageboard_page_handler($page) {
        $new_section_one = array('owner', 'add', 'group');
@@ -84,7 +84,11 @@ function messageboard_page_handler($page) {
                        set_input('page_owner_guid', $owner_guid);
                        include "$pages/owner.php";
                        break;
+
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
index 631a5f731bd87e35445e46b10e7a1cb568c9f9ca..e4ef4d43a53b5b8b01affce4e8d8d21851272dcc 100644 (file)
@@ -70,7 +70,7 @@ function messages_init() {
  * Messages page handler
  *
  * @param array $page Array of URL components for routing
- * @return void
+ * @return bool
  */
 function messages_page_handler($page) {
 
@@ -112,7 +112,10 @@ function messages_page_handler($page) {
                case 'add':
                        include("$base_dir/send.php");
                        break;
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
index 7ec82cfc937dd7c80c7c90b39f848fa146a3aaf1..761f17e400b8156a093e0858778e7d8dfba6dcca 100644 (file)
@@ -36,7 +36,7 @@ function notifications_plugin_init() {
  * Route page requests
  *
  * @param array $page Array of url parameters
- * @return void
+ * @return bool
  */
 function notifications_page_handler($page) {
 
@@ -54,7 +54,10 @@ function notifications_page_handler($page) {
                case 'personal':
                        require "$base/index.php";
                        break;
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
index fd1d7df4769f68e46bd0e336789a43a47ca28c31..b2f26c719b47e60f8012963dbed8cf02eeb5d7d7 100644 (file)
@@ -100,7 +100,7 @@ function pages_init() {
  * Title is ignored
  *
  * @param array $page
- * @return void
+ * @return bool
  */
 function pages_page_handler($page) {
 
@@ -152,7 +152,10 @@ function pages_page_handler($page) {
                case 'all':
                        include "$base_dir/world.php";
                        break;
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
index 1c82d55f52ada385b456afca5bd01d5cce128b05..0ee2d01526671235ce2d9456914a07696258805a 100644 (file)
@@ -48,7 +48,7 @@ function profile_init() {
  * Profile page handler
  *
  * @param array $page Array of URL segments passed by the page handling mechanism
- * @return void
+ * @return bool
  */
 function profile_page_handler($page) {
 
@@ -85,6 +85,7 @@ function profile_page_handler($page) {
 
        $body = elgg_view_layout('one_column', array('content' => $content));
        echo elgg_view_page($title, $body);
+       return true;
 }
 
 /**
index 0638feb52f87a66ad50d28dd6f11c0e8abed6c33..8b18a4d64932f780fa98c5ce0366434ccc4933aa 100644 (file)
@@ -60,7 +60,7 @@ function reportedcontent_init() {
  * Serves the add report page
  *
  * @param array $page Array of page routing elements
- * @return void
+ * @return bool
  */
 function reportedcontent_page_handler($page) {
        // only logged in users can report things
@@ -77,6 +77,7 @@ function reportedcontent_page_handler($page) {
        $body = elgg_view_layout('one_sidebar', $params);
 
        echo elgg_view_page(elgg_echo('reportedcontent:this'), $body);
+       return true;
 }
 
 /**
index a88c6ec031412edcfff5456082bb23e840ac1a7b..73a96dc0a778df9565485a7817bb9182eb2357ec 100644 (file)
@@ -56,7 +56,7 @@ function search_init() {
  * Page handler for search
  *
  * @param array $page Page elements from core page handler
- * @return void
+ * @return bool
  */
 function search_page_handler($page) {
 
@@ -72,6 +72,7 @@ function search_page_handler($page) {
        $base_dir = elgg_get_plugins_path() . 'search/pages/search';
 
        include_once("$base_dir/index.php");
+       return true;
 }
 
 /**
index f46d9ec757a0df7652fbb8fcb15ea857e5b2cbe2..2ed0f0b4da10f21048bfc4262bb291879f572652 100644 (file)
@@ -84,7 +84,7 @@ function thewire_init() {
  * thewire/tag/<tag>            View wire posts tagged with <tag>
  *
  * @param array $page From the page_handler function
- * @return void
+ * @return bool
  */
 function thewire_page_handler($page) {
 
@@ -134,7 +134,11 @@ function thewire_page_handler($page) {
                        }
                        include "$base_dir/previous.php";
                        break;
+
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
index fd2ff6493c0aeb6b85516a4836e1b483543eaabc..08bce54794d2230cfaa592af4afa6e8ef1551d11 100644 (file)
@@ -45,14 +45,14 @@ function twitter_api_init() {
  * Handles old pg/twitterservice/ handler
  *
  * @param array $page
- * @return void
+ * @return bool
  */
 function twitter_api_pagehandler_deprecated($page) {
        $url = elgg_get_site_url() . 'pg/twitter_api/authorize';
        $msg = elgg_echo('twitter_api:deprecated_callback_url', array($url));
        register_error($msg);
 
-       twitter_api_pagehandler($page);
+       return twitter_api_pagehandler($page);
 }
 
 
@@ -64,7 +64,7 @@ function twitter_api_pagehandler_deprecated($page) {
  */
 function twitter_api_pagehandler($page) {
        if (!isset($page[0])) {
-               return;
+               return false;
        }
 
        switch ($page[0]) {
@@ -92,7 +92,10 @@ function twitter_api_pagehandler($page) {
                        $pages = dirname(__FILE__) . '/pages/twitter_api';
                        include "$pages/interstitial.php";
                        break;
+               default:
+                       return false;
        }
+       return true;
 }
 
 /**
index a674511cfc64a019b4db88ca652a543b9fac7c8e..6904ce9e22e2d044d60b1decabe26c37243b04e7 100644 (file)
@@ -149,7 +149,7 @@ function uservalidationbyemail_check_auth_attempt($credentials) {
  * Checks sent passed validation code and user guids and validates the user.
  *
  * @param array $page
- * @return void
+ * @return bool
  */
 function uservalidationbyemail_page_handler($page) {