YOURLS / YOURLS

includes/functions-plugins.php

Spacing +161 added lines, -161 removed lines

@@ -15,7 +15,7 @@ 
  * @since 1.5
  */
 
-if ( !isset( $yourls_filters ) ) {
+if (!isset($yourls_filters)) {
     $yourls_filters = [];
 }
 /*
@@ -23,7 +23,7 @@ 
  * $yourls_filters['hook']['array of priorities']['serialized function names']['array of ['array (functions, accepted_args, filter or action)]']
  */
 
-if ( !isset( $yourls_actions ) ) {
+if (!isset($yourls_actions)) {
     $yourls_actions = [];
 }
 /*
@@ -48,12 +48,12 @@ 
  * @param string   $type
  * @global array   $yourls_filters Storage for all of the filters
  */
-function yourls_add_filter( $hook, $function_name, $priority = 10, $accepted_args = NULL, $type = 'filter' ) {
+function yourls_add_filter($hook, $function_name, $priority = 10, $accepted_args = NULL, $type = 'filter') {
     global $yourls_filters;
     // At this point, we cannot check if the function exists, as it may well be defined later (which is OK)
-    $id = yourls_filter_unique_id( $hook, $function_name, $priority );
+    $id = yourls_filter_unique_id($hook, $function_name, $priority);
 
-    $yourls_filters[ $hook ][ $priority ][ $id ] = [
+    $yourls_filters[$hook][$priority][$id] = [
         'function'      => $function_name,
         'accepted_args' => $accepted_args,
         'type'          => $type,
@@ -73,8 +73,8 @@ 
  * @param int $priority optional. Used to specify the order in which the functions associated with a particular action are executed (default: 10). Lower numbers correspond with earlier execution, and functions with the same priority are executed in the order in which they were added to the action.
  * @param int $accepted_args optional. The number of arguments the function accept (default 1).
  */
-function yourls_add_action( $hook, $function_name, $priority = 10, $accepted_args = 1 ) {
-	yourls_add_filter( $hook, $function_name, $priority, $accepted_args, 'action' );
+function yourls_add_action($hook, $function_name, $priority = 10, $accepted_args = 1) {
+	yourls_add_filter($hook, $function_name, $priority, $accepted_args, 'action');
 }
 
 /**
@@ -88,28 +88,28 @@ 
  * @param int|bool $priority used in counting how many hooks were applied.  If === false and $function is an object reference, we return the unique id only if it already has one, false otherwise.
  * @return string unique ID for usage as array key
  */
-function yourls_filter_unique_id( $hook, $function, $priority ) {
+function yourls_filter_unique_id($hook, $function, $priority) {
     // If function then just skip all of the tests and not overwrite the following.
-    if ( is_string( $function ) ) {
+    if (is_string($function)) {
         return $function;
     }
 
-    if ( is_object( $function ) ) {
+    if (is_object($function)) {
         // Closures are currently implemented as objects
-        $function = [ $function, '' ];
+        $function = [$function, ''];
     }
     else {
-        $function = (array)$function;
+        $function = (array) $function;
     }
 
     // Object Class Calling
-    if ( is_object( $function[ 0 ] ) ) {
-        return spl_object_hash( $function[ 0 ] ).$function[ 1 ];
+    if (is_object($function[0])) {
+        return spl_object_hash($function[0]).$function[1];
     }
 
     // Static Calling
-    if ( is_string( $function[ 0 ] ) ) {
-        return $function[ 0 ].'::'.$function[ 1 ];
+    if (is_string($function[0])) {
+        return $function[0].'::'.$function[1];
     }
     // TODO: missing final return;
 }
@@ -134,39 +134,39 @@ 
  * @param mixed $value the value of the element before filtering
  * @return mixed
  */
-function yourls_apply_filter( $hook, $value = '' ) {
+function yourls_apply_filter($hook, $value = '') {
     global $yourls_filters;
-    if ( !isset( $yourls_filters[ $hook ] ) ) {
+    if (!isset($yourls_filters[$hook])) {
         return $value;
     }
 
     $args = func_get_args();
 
     // Sort filters by priority
-    ksort( $yourls_filters[ $hook ] );
+    ksort($yourls_filters[$hook]);
 
     // Loops through each filter
-    reset( $yourls_filters[ $hook ] );
+    reset($yourls_filters[$hook]);
     do {
-        foreach ( (array)current( $yourls_filters[ $hook ] ) as $the_ ) {
-            if ( !is_null( $the_[ 'function' ] ) ) {
-                $args[ 1 ] = $value;
-                $count = $the_[ 'accepted_args' ];
-                if ( is_null( $count ) ) {
-                    $_value = call_user_func_array( $the_[ 'function' ], array_slice( $args, 1 ) );
+        foreach ((array) current($yourls_filters[$hook]) as $the_) {
+            if (!is_null($the_['function'])) {
+                $args[1] = $value;
+                $count = $the_['accepted_args'];
+                if (is_null($count)) {
+                    $_value = call_user_func_array($the_['function'], array_slice($args, 1));
                 }
                 else {
-                    $_value = call_user_func_array( $the_[ 'function' ], array_slice( $args, 1, (int)$count ) );
+                    $_value = call_user_func_array($the_['function'], array_slice($args, 1, (int) $count));
                 }
             }
-            if ( $the_[ 'type' ] == 'filter' ) {
+            if ($the_['type'] == 'filter') {
                 $value = $_value;
             }
         }
-    } while ( next( $yourls_filters[ $hook ] ) !== false );
+    } while (next($yourls_filters[$hook]) !== false);
 
     // TODO: Missing final return - Why not just return all the time? It's up receiving code to use or not.
-    if ( $the_[ 'type' ] == 'filter' ) {
+    if ($the_['type'] == 'filter') {
         return $value;
     }
 }
@@ -177,33 +177,33 @@ 
  * @param string $hook the name of the YOURLS action
  * @param mixed $arg action arguments
  */
-function yourls_do_action( $hook, $arg = '' ) {
+function yourls_do_action($hook, $arg = '') {
     global $yourls_actions;
 
     // Keep track of actions that are "done"
-    if ( !isset( $yourls_actions ) ) {
+    if (!isset($yourls_actions)) {
         $yourls_actions = [];
     }
-    if ( !isset( $yourls_actions[ $hook ] ) ) {
-        $yourls_actions[ $hook ] = 1;
+    if (!isset($yourls_actions[$hook])) {
+        $yourls_actions[$hook] = 1;
     }
     else {
-        ++$yourls_actions[ $hook ];
+        ++$yourls_actions[$hook];
     }
 
     $args = [];
-    if ( is_array( $arg ) && 1 == count( $arg ) && isset( $arg[ 0 ] ) && is_object( $arg[ 0 ] ) ) { // array(&$this)
-        $args[] =& $arg[ 0 ];
+    if (is_array($arg) && 1 == count($arg) && isset($arg[0]) && is_object($arg[0])) { // array(&$this)
+        $args[] = & $arg[0];
     }
     else {
         $args[] = $arg;
     }
 
-    for ( $a = 2 ; $a < func_num_args() ; $a++ ) {
-        $args[] = func_get_arg( $a );
+    for ($a = 2; $a < func_num_args(); $a++) {
+        $args[] = func_get_arg($a);
     }
 
-    yourls_apply_filter( $hook, $args );
+    yourls_apply_filter($hook, $args);
 }
 
 /**
@@ -212,9 +212,9 @@ 
  * @param string $hook Name of the action hook.
  * @return int The number of times action hook <tt>$hook</tt> is fired
  */
-function yourls_did_action( $hook ) {
+function yourls_did_action($hook) {
     global $yourls_actions;
-    return empty( $yourls_actions[ $hook ] ) ? 0 : $yourls_actions[ $hook ];
+    return empty($yourls_actions[$hook]) ? 0 : $yourls_actions[$hook];
 }
 
 /**
@@ -233,17 +233,17 @@ 
  * @param int $priority optional. The priority of the function (default: 10).
  * @return bool Whether the function was registered as a filter before it was removed.
  */
-function yourls_remove_filter( $hook, $function_to_remove, $priority = 10 ) {
+function yourls_remove_filter($hook, $function_to_remove, $priority = 10) {
     global $yourls_filters;
 
-    $function_to_remove = yourls_filter_unique_id( $hook, $function_to_remove, $priority );
+    $function_to_remove = yourls_filter_unique_id($hook, $function_to_remove, $priority);
 
-    $remove = isset( $yourls_filters[ $hook ][ $priority ][ $function_to_remove ] );
+    $remove = isset($yourls_filters[$hook][$priority][$function_to_remove]);
 
-    if ( $remove === true ) {
-        unset ( $yourls_filters[ $hook ][ $priority ][ $function_to_remove ] );
-        if ( empty( $yourls_filters[ $hook ][ $priority ] ) ) {
-            unset( $yourls_filters[ $hook ] );
+    if ($remove === true) {
+        unset ($yourls_filters[$hook][$priority][$function_to_remove]);
+        if (empty($yourls_filters[$hook][$priority])) {
+            unset($yourls_filters[$hook]);
         }
     }
     return $remove;
@@ -258,8 +258,8 @@ 
  * @param int      $priority           optional. The priority of the function (default: 10).
  * @return bool                        Whether the function was registered as an action before it was removed.
  */
-function yourls_remove_action( $hook, $function_to_remove, $priority = 10 ) {
-    return yourls_remove_filter( $hook, $function_to_remove, $priority );
+function yourls_remove_action($hook, $function_to_remove, $priority = 10) {
+    return yourls_remove_filter($hook, $function_to_remove, $priority);
 }
 
 /**
@@ -271,8 +271,8 @@ 
  * @param int|false $priority optional. The priority of the functions to remove
  * @return bool true when it's finished
  */
-function yourls_remove_all_actions( $hook, $priority = false ) {
-    return yourls_remove_all_filters( $hook, $priority );
+function yourls_remove_all_actions($hook, $priority = false) {
+    return yourls_remove_all_filters($hook, $priority);
 }
 
 /**
@@ -283,15 +283,15 @@ 
  * @param int|false $priority optional. The priority of the functions to remove
  * @return bool true when it's finished
  */
-function yourls_remove_all_filters( $hook, $priority = false ) {
+function yourls_remove_all_filters($hook, $priority = false) {
     global $yourls_filters;
 
-    if ( isset( $yourls_filters[ $hook ] ) ) {
-        if ( $priority === false ) {
-            unset( $yourls_filters[ $hook ] );
+    if (isset($yourls_filters[$hook])) {
+        if ($priority === false) {
+            unset($yourls_filters[$hook]);
         }
-        elseif ( isset( $yourls_filters[ $hook ][ $priority ] ) ) {
-            unset( $yourls_filters[ $hook ][ $priority ] );
+        elseif (isset($yourls_filters[$hook][$priority])) {
+            unset($yourls_filters[$hook][$priority]);
         }
     }
 
@@ -306,20 +306,20 @@ 
  * @return int|bool Optionally returns the priority on that hook for the specified function.
  * @global array         $yourls_filters    storage for all of the filters
  */
-function yourls_has_filter( $hook, $function_to_check = false ) {
+function yourls_has_filter($hook, $function_to_check = false) {
     global $yourls_filters;
 
-    $has = !empty( $yourls_filters[ $hook ] );
-    if ( false === $function_to_check || false === $has ) {
+    $has = !empty($yourls_filters[$hook]);
+    if (false === $function_to_check || false === $has) {
         return $has;
     }
 
-    if ( !$idx = yourls_filter_unique_id( $hook, $function_to_check, false ) ) {
+    if (!$idx = yourls_filter_unique_id($hook, $function_to_check, false)) {
         return false;
     }
 
-    foreach ( array_keys( $yourls_filters[ $hook ] ) as $priority ) {
-        if ( isset( $yourls_filters[ $hook ][ $priority ][ $idx ] ) ) {
+    foreach (array_keys($yourls_filters[$hook]) as $priority) {
+        if (isset($yourls_filters[$hook][$priority][$idx])) {
             return $priority;
         }
     }
@@ -331,8 +331,8 @@ 
  * @param callable|false $function_to_check
  * @return bool|int
  */
-function yourls_has_action( $hook, $function_to_check = false ) {
-    return yourls_has_filter( $hook, $function_to_check );
+function yourls_has_action($hook, $function_to_check = false) {
+    return yourls_has_filter($hook, $function_to_check);
 }
 
 /**
@@ -341,7 +341,7 @@ 
  * @return int Number of activated plugins
  */
 function yourls_has_active_plugins() {
-    return count( yourls_get_db()->get_plugins() );
+    return count(yourls_get_db()->get_plugins());
 }
 
 /**
@@ -350,16 +350,16 @@ 
  * @return array Array of [/plugindir/plugin.php]=>array('Name'=>'Ozh', 'Title'=>'Hello', )
  */
 function yourls_get_plugins() {
-    $plugins = (array)glob( YOURLS_PLUGINDIR.'/*/plugin.php' );
+    $plugins = (array) glob(YOURLS_PLUGINDIR.'/*/plugin.php');
 
-    if ( is_array( $plugins ) ) {
-        foreach ( $plugins as $key => $plugin ) {
-            $plugins[ yourls_plugin_basename( $plugin ) ] = yourls_get_plugin_data( $plugin );
-            unset( $plugins[ $key ] );
+    if (is_array($plugins)) {
+        foreach ($plugins as $key => $plugin) {
+            $plugins[yourls_plugin_basename($plugin)] = yourls_get_plugin_data($plugin);
+            unset($plugins[$key]);
         }
     }
 
-    return empty( $plugins ) ? [] : $plugins;
+    return empty($plugins) ? [] : $plugins;
 }
 
 /**
@@ -368,9 +368,9 @@ 
  * @param string $plugin Physical path to plugin file
  * @return bool
  */
-function yourls_is_active_plugin( $plugin ) {
+function yourls_is_active_plugin($plugin) {
     return yourls_has_active_plugins() > 0 ?
-        in_array( yourls_plugin_basename( $plugin ), yourls_get_db()->get_plugins() )
+        in_array(yourls_plugin_basename($plugin), yourls_get_db()->get_plugins())
         : false;
 }
 
@@ -401,27 +401,27 @@ 
  * @param string $file Physical path to plugin file
  * @return array Array of 'Field'=>'Value' from plugin comment header lines of the form "Field: Value"
  */
-function yourls_get_plugin_data( $file ) {
-    $fp = fopen( $file, 'r' ); // assuming $file is readable, since yourls_load_plugins() filters this
-    $data = fread( $fp, 8192 ); // get first 8kb
-    fclose( $fp );
+function yourls_get_plugin_data($file) {
+    $fp = fopen($file, 'r'); // assuming $file is readable, since yourls_load_plugins() filters this
+    $data = fread($fp, 8192); // get first 8kb
+    fclose($fp);
 
     // Capture all the header within first comment block
-    if ( !preg_match( '!.*?/\*(.*?)\*/!ms', $data, $matches ) )
+    if (!preg_match('!.*?/\*(.*?)\*/!ms', $data, $matches))
         return [];
 
     // Capture each line with "Something: some text"
-    unset( $data );
-    $lines = preg_split( "[\n|\r]", $matches[ 1 ] );
-    unset( $matches );
+    unset($data);
+    $lines = preg_split("[\n|\r]", $matches[1]);
+    unset($matches);
 
     $plugin_data = [];
-    foreach ( $lines as $line ) {
-        if ( !preg_match( '!(\s*)?\*?(\s*)?(.*?):\s+(.*)!', $line, $matches ) ) {
+    foreach ($lines as $line) {
+        if (!preg_match('!(\s*)?\*?(\s*)?(.*?):\s+(.*)!', $line, $matches)) {
             continue;
         }
 
-        $plugin_data[ trim($matches[3]) ] = yourls_esc_html(trim($matches[4]));
+        $plugin_data[trim($matches[3])] = yourls_esc_html(trim($matches[4]));
     }
 
     return $plugin_data;
@@ -440,15 +440,15 @@ 
  */
 function yourls_load_plugins() {
     // Don't load plugins when installing or updating
-    if ( yourls_is_installing() OR yourls_is_upgrading() OR !yourls_is_installed() ) {
+    if (yourls_is_installing() OR yourls_is_upgrading() OR !yourls_is_installed()) {
         return [
             'loaded' => false,
             'info'   => 'install/upgrade'
         ];
     }
 
-    $active_plugins = (array)yourls_get_option( 'active_plugins' );
-    if ( empty( $active_plugins ) ) {
+    $active_plugins = (array) yourls_get_option('active_plugins');
+    if (empty($active_plugins)) {
         return [
             'loaded' => false,
             'info'   => 'no active plugin'
@@ -456,25 +456,25 @@ 
     }
 
     $plugins = [];
-    foreach ( $active_plugins as $key => $plugin ) {
-        if ( yourls_validate_plugin_file( YOURLS_PLUGINDIR.'/'.$plugin ) ) {
-            include_once( YOURLS_PLUGINDIR.'/'.$plugin );
+    foreach ($active_plugins as $key => $plugin) {
+        if (yourls_validate_plugin_file(YOURLS_PLUGINDIR.'/'.$plugin)) {
+            include_once(YOURLS_PLUGINDIR.'/'.$plugin);
             $plugins[] = $plugin;
-            unset( $active_plugins[ $key ] );
+            unset($active_plugins[$key]);
         }
     }
 
     // Replace active plugin list with list of plugins we just activated
-    yourls_get_db()->set_plugins( $plugins );
-    $info = count( $plugins ).' activated';
+    yourls_get_db()->set_plugins($plugins);
+    $info = count($plugins).' activated';
 
     // $active_plugins should be empty now, if not, a plugin could not be find: remove it
-    $missing_count = count( $active_plugins );
-    if ( $missing_count > 0 ) {
-        yourls_update_option( 'active_plugins', $plugins );
-        $message = yourls_n( 'Could not find and deactivate plugin :', 'Could not find and deactivate plugins :', $missing_count );
-        $missing = '<strong>'.implode( '</strong>, <strong>', $active_plugins ).'</strong>';
-        yourls_add_notice( $message.' '.$missing );
+    $missing_count = count($active_plugins);
+    if ($missing_count > 0) {
+        yourls_update_option('active_plugins', $plugins);
+        $message = yourls_n('Could not find and deactivate plugin :', 'Could not find and deactivate plugins :', $missing_count);
+        $missing = '<strong>'.implode('</strong>, <strong>', $active_plugins).'</strong>';
+        yourls_add_notice($message.' '.$missing);
         $info .= ', '.$missing_count.' removed';
     }
 
@@ -490,11 +490,11 @@ 
  * @param string $file Full pathname to a file
  * @return bool
  */
-function yourls_validate_plugin_file( $file ) {
-    return false === strpos( $file, '..' )
-           && false === strpos( $file, './' )
-           && 'plugin.php' === substr( $file, -10 )
-           && is_readable( $file );
+function yourls_validate_plugin_file($file) {
+    return false === strpos($file, '..')
+           && false === strpos($file, './')
+           && 'plugin.php' === substr($file, -10)
+           && is_readable($file);
 }
 
 /**
@@ -503,37 +503,37 @@ 
  * @param string $plugin Plugin filename (full or relative to plugins directory)
  * @return string|true  string if error or true if success
  */
-function yourls_activate_plugin( $plugin ) {
+function yourls_activate_plugin($plugin) {
     // validate file
-    $plugin = yourls_plugin_basename( $plugin );
-    $plugindir = yourls_sanitize_filename( YOURLS_PLUGINDIR );
-    if ( !yourls_validate_plugin_file( $plugindir.'/'.$plugin ) ) {
-        return yourls__( 'Not a valid plugin file' );
+    $plugin = yourls_plugin_basename($plugin);
+    $plugindir = yourls_sanitize_filename(YOURLS_PLUGINDIR);
+    if (!yourls_validate_plugin_file($plugindir.'/'.$plugin)) {
+        return yourls__('Not a valid plugin file');
     }
 
     // check not activated already
     $ydb = yourls_get_db();
-    if ( yourls_is_active_plugin( $plugin ) ) {
-        return yourls__( 'Plugin already activated' );
+    if (yourls_is_active_plugin($plugin)) {
+        return yourls__('Plugin already activated');
     }
 
     // attempt activation. TODO: uber cool fail proof sandbox like in WP.
     ob_start();
-    include_once( $plugindir.'/'.$plugin );
-    if ( ob_get_length() > 0 ) {
+    include_once($plugindir.'/'.$plugin);
+    if (ob_get_length() > 0) {
         // there was some output: error
         // @codeCoverageIgnoreStart
         $output = ob_get_clean();
-        return yourls_s( 'Plugin generated unexpected output. Error was: <br/><pre>%s</pre>', $output );
+        return yourls_s('Plugin generated unexpected output. Error was: <br/><pre>%s</pre>', $output);
         // @codeCoverageIgnoreEnd
     }
     ob_end_clean();
 
     // so far, so good: update active plugin list
-    $ydb->add_plugin( $plugin );
-    yourls_update_option( 'active_plugins', $ydb->get_plugins() );
-    yourls_do_action( 'activated_plugin', $plugin );
-    yourls_do_action( 'activated_'.$plugin );
+    $ydb->add_plugin($plugin);
+    yourls_update_option('active_plugins', $ydb->get_plugins());
+    yourls_do_action('activated_plugin', $plugin);
+    yourls_do_action('activated_'.$plugin);
 
     return true;
 }
@@ -544,26 +544,26 @@ 
  * @param string $plugin Plugin filename (full relative to plugins directory)
  * @return string|true  string if error or true if success
  */
-function yourls_deactivate_plugin( $plugin ) {
-    $plugin = yourls_plugin_basename( $plugin );
+function yourls_deactivate_plugin($plugin) {
+    $plugin = yourls_plugin_basename($plugin);
 
     // Check plugin is active
-    if ( !yourls_is_active_plugin( $plugin ) ) {
-        return yourls__( 'Plugin not active' );
+    if (!yourls_is_active_plugin($plugin)) {
+        return yourls__('Plugin not active');
     }
 
     // Deactivate the plugin
     global $ydb;
     $plugins = $ydb->get_plugins();
-    $key = array_search( $plugin, $plugins );
-    if ( $key !== false ) {
-        array_splice( $plugins, $key, 1 );
+    $key = array_search($plugin, $plugins);
+    if ($key !== false) {
+        array_splice($plugins, $key, 1);
     }
 
-    $ydb->set_plugins( $plugins );
-    yourls_update_option( 'active_plugins', $plugins );
-    yourls_do_action( 'deactivated_plugin', $plugin );
-    yourls_do_action( 'deactivated_'.$plugin );
+    $ydb->set_plugins($plugins);
+    yourls_update_option('active_plugins', $plugins);
+    yourls_do_action('deactivated_plugin', $plugin);
+    yourls_do_action('deactivated_'.$plugin);
 
     return true;
 }
@@ -573,8 +573,8 @@ 
  * @param string $file
  * @return string
  */
-function yourls_plugin_basename( $file ) {
-	return trim( str_replace( yourls_sanitize_filename( YOURLS_PLUGINDIR ), '', yourls_sanitize_filename( $file ) ), '/' );
+function yourls_plugin_basename($file) {
+	return trim(str_replace(yourls_sanitize_filename(YOURLS_PLUGINDIR), '', yourls_sanitize_filename($file)), '/');
 }
 
 /**
@@ -582,12 +582,12 @@ 
  * @param string $file
  * @return string
  */
-function yourls_plugin_url( $file ) {
-    $url = YOURLS_PLUGINURL.'/'.yourls_plugin_basename( $file );
-    if ( yourls_is_ssl() or yourls_needs_ssl() ) {
-        $url = str_replace( 'http://', 'https://', $url );
+function yourls_plugin_url($file) {
+    $url = YOURLS_PLUGINURL.'/'.yourls_plugin_basename($file);
+    if (yourls_is_ssl() or yourls_needs_ssl()) {
+        $url = str_replace('http://', 'https://', $url);
     }
-    return (string)yourls_apply_filter( 'plugin_url', $url, $file );
+    return (string) yourls_apply_filter('plugin_url', $url, $file);
 }
 
 /**
@@ -597,10 +597,10 @@ 
  */
 function yourls_list_plugin_admin_pages() {
     $plugin_links = [];
-    foreach ( yourls_get_db()->get_plugin_pages() as $plugin => $page ) {
-        $plugin_links[ $plugin ] = [
-            'url'    => yourls_admin_url( 'plugins.php?page='.$page[ 'slug' ] ),
-            'anchor' => $page[ 'title' ],
+    foreach (yourls_get_db()->get_plugin_pages() as $plugin => $page) {
+        $plugin_links[$plugin] = [
+            'url'    => yourls_admin_url('plugins.php?page='.$page['slug']),
+            'anchor' => $page['title'],
         ];
     }
     return $plugin_links;
@@ -612,8 +612,8 @@ 
  * @param string   $title
  * @param callable $function
  */
-function yourls_register_plugin_page( $slug, $title, $function ) {
-    yourls_get_db()->add_plugin_page( $slug, $title, $function );
+function yourls_register_plugin_page($slug, $title, $function) {
+    yourls_get_db()->add_plugin_page($slug, $title, $function);
 }
 
 /**
@@ -621,20 +621,20 @@ 
  *
  * @param string $plugin_page
  */
-function yourls_plugin_admin_page( $plugin_page ) {
+function yourls_plugin_admin_page($plugin_page) {
     // Check the plugin page is actually registered
     $pages = yourls_get_db()->get_plugin_pages();
-    if ( !isset( $pages[ $plugin_page ] ) ) {
-        yourls_die( yourls__( 'This page does not exist. Maybe a plugin you thought was activated is inactive?' ), yourls__( 'Invalid link' ) );
+    if (!isset($pages[$plugin_page])) {
+        yourls_die(yourls__('This page does not exist. Maybe a plugin you thought was activated is inactive?'), yourls__('Invalid link'));
     }
 
     // Draw the page itself
-    yourls_do_action( 'load-'.$plugin_page );
-    yourls_html_head( 'plugin_page_'.$plugin_page, $pages[ $plugin_page ][ 'title' ] );
+    yourls_do_action('load-'.$plugin_page);
+    yourls_html_head('plugin_page_'.$plugin_page, $pages[$plugin_page]['title']);
     yourls_html_logo();
     yourls_html_menu();
 
-    call_user_func( $pages[ $plugin_page ][ 'function' ] );
+    call_user_func($pages[$plugin_page]['function']);
 
     yourls_html_footer();
 }
@@ -649,22 +649,22 @@ 
  * @param array $plugin_b
  * @return int 0, 1 or -1, see uasort()
  */
-function yourls_plugins_sort_callback( $plugin_a, $plugin_b ) {
-    $orderby = yourls_apply_filter( 'plugins_sort_callback', 'Plugin Name' );
-    $order = yourls_apply_filter( 'plugins_sort_callback', 'ASC' );
+function yourls_plugins_sort_callback($plugin_a, $plugin_b) {
+    $orderby = yourls_apply_filter('plugins_sort_callback', 'Plugin Name');
+    $order = yourls_apply_filter('plugins_sort_callback', 'ASC');
 
-    $a = isset( $plugin_a[ $orderby ] ) ? $plugin_a[ $orderby ] : '';
-    $b = isset( $plugin_b[ $orderby ] ) ? $plugin_b[ $orderby ] : '';
+    $a = isset($plugin_a[$orderby]) ? $plugin_a[$orderby] : '';
+    $b = isset($plugin_b[$orderby]) ? $plugin_b[$orderby] : '';
 
-    if ( $a == $b ) {
+    if ($a == $b) {
         return 0;
     }
 
-    if ( 'DESC' == $order ) {
-        return ( $a < $b ) ? 1 : -1;
+    if ('DESC' == $order) {
+        return ($a < $b) ? 1 : -1;
     }
     else {
-        return ( $a < $b ) ? -1 : 1;
+        return ($a < $b) ? -1 : 1;
     }
 }
 
@@ -685,7 +685,7 @@ 
  * @return void
  */
 function yourls_shutdown() {
-    yourls_do_action( 'shutdown' );
+    yourls_do_action('shutdown');
 }
 
 /**