michaeluno / admin-page-framework

development/factory/post_type/_model/AdminPageFramework_PostType_Model__FlushRewriteRules.php

Doc Comments +1 added lines

@@ -26,6 +26,7 @@
      * Sets up hooks and properties.
      * 
      * @internal
+     * @param AdminPageFramework_PostType_Model $oFactory
      */
     public function __construct( $oFactory ) {
                 

Indentation +1 added lines, -1 removed lines

@@ -277,7 +277,7 @@
         );        
     }    
     
-       /**
+        /**
          * Extracts meta box form fields options array from the given options array of an admin page.
          * 
          * @since       3.5.6

Spacing +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
      */
     public function __construct( $oFactory ) {
                 
-        if ( ! $this->_shouldProceed( $oFactory ) ) {
+        if ( !$this->_shouldProceed( $oFactory ) ) {
             return;
         }        
         
@@ -54,10 +54,10 @@ 
          */
         private function _shouldProceed( $oFactory ) {
 
-            if ( ! $oFactory->oProp->bIsAdmin ) {
+            if ( !$oFactory->oProp->bIsAdmin ) {
                 return false;
             }        
-            if ( ! $oFactory->oProp->sCallerPath ) {
+            if ( !$oFactory->oProp->sCallerPath ) {
                 return false;
             }
             return 'plugin' === $oFactory->oProp->sScriptType;
@@ -87,7 +87,7 @@ 
         }
                 
         // If the execution flow in the plugin activation hook, schedule flushing rewrite rules.
-        if ( did_action( 'activate_' . plugin_basename( $this->oFactory->oProp->sCallerPath ) ) ) {
+        if ( did_action( 'activate_'.plugin_basename( $this->oFactory->oProp->sCallerPath ) ) ) {
             add_action( 'shutdown', array( $this, '_replyToFlushRewriteRules' ) );
         }
 

Braces +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
      * @callback    action      activate_{plugin base name}     
      * @since       3.7.6
      */
-    public function _replyToSetUpPostType() {            
+    public function _replyToSetUpPostType() {
         do_action( "set_up_{$this->oFactory->oProp->sClassName}", $this );
     }        
     

development/factory/post_type/_model/AdminPageFramework_PostType_Model__SubMenuOrder.php

Doc Comments +3 added lines

@@ -26,6 +26,7 @@ 
      * Sets up hooks and properties.
      * 
      * @internal
+     * @param AdminPageFramework_PostType_Model $oFactory
      */
     public function __construct( $oFactory ) {
           
@@ -82,6 +83,7 @@ 
     
         /**
          * @since       3.7.4
+         * @param string $sSubMenuSlug
          */
         private function _setSubMenuSlugForSorting( $sSubMenuSlug ) {
                 
@@ -94,6 +96,7 @@ 
         
         /**
          * @since       3.7.4
+         * @param string $sSubMenuSlug
          */
         private function _setSubMenuItemIndex( $sSubMenuSlug ) {                
                                         

Spacing +12 added lines, -12 removed lines

@@ -31,7 +31,7 @@ 
           
         $this->oFactory = $oFactory;
           
-        if ( ! $oFactory->oProp->bIsAdmin ) {
+        if ( !$oFactory->oProp->bIsAdmin ) {
             return;
         }
         
@@ -47,7 +47,7 @@ 
         
         add_action(
             'admin_menu',
-            array( $this, 'sortAdminSubMenu' ),  // defined in the framework utility class.
+            array( $this, 'sortAdminSubMenu' ), // defined in the framework utility class.
             9999
         );            
             
@@ -62,15 +62,15 @@ 
             
         // Check the post type `show_ui` and other related UI arguments.
         $_bsShowInMeenu = $this->getShowInMenuPostTypeArgument( $this->oFactory->oProp->aPostTypeArgs );
-        if ( ! $_bsShowInMeenu ) {
+        if ( !$_bsShowInMeenu ) {
             return;
         }
         
         // If the user sets a menu slug to the 'show_in_menu' argument, use that. 
         // It is used to set a custom post type sub-menu belong to another menu.
-        $_sSubMenuSlug  = is_string( $_bsShowInMeenu )
+        $_sSubMenuSlug = is_string( $_bsShowInMeenu )
             ? $_bsShowInMeenu
-            : 'edit.php?post_type=' . $this->oFactory->oProp->sPostType;
+            : 'edit.php?post_type='.$this->oFactory->oProp->sPostType;
         
         // Set the index to the framework specific global array for sorting.
         $this->_setSubMenuSlugForSorting( $_sSubMenuSlug );
@@ -131,11 +131,11 @@ 
 
                 // @remark      This is the partial link url set in the third element ( index of 2 ) in the third dimension of submenu global array element.
                 // This is not the submenu slug.
-                $_sLinkSlugManage = 'edit.php?post_type=' . $this->oFactory->oProp->sPostType;
+                $_sLinkSlugManage = 'edit.php?post_type='.$this->oFactory->oProp->sPostType;
                                 
                 $_aLinkSlugs = array(
                     $_sLinkSlugManage => $_nSubMenuOrderManage,
-                    'post-new.php?post_type=' . $this->oFactory->oProp->sPostType => $_nSubMenuOrderAddNew,
+                    'post-new.php?post_type='.$this->oFactory->oProp->sPostType => $_nSubMenuOrderAddNew,
                 );
 
                 // If the user does not set a custom value, unset it
@@ -144,8 +144,8 @@ 
                 }
                 
                 // If the user does not want to show the Add New sub menu, no need to change the order.
-                if ( ! $_bShowAddNew || 10 == $_nSubMenuOrderAddNew ) {
-                    unset( $_aLinkSlugs[ 'post-new.php?post_type=' . $this->oFactory->oProp->sPostType ] );
+                if ( !$_bShowAddNew || 10 == $_nSubMenuOrderAddNew ) {
+                    unset( $_aLinkSlugs[ 'post-new.php?post_type='.$this->oFactory->oProp->sPostType ] );
                 }                   
                 
                 return $_aLinkSlugs;
@@ -165,9 +165,9 @@ 
          */
         private function _setSubMenuIndexByLinksSlugs( $sSubMenuSlug, array $aLinkSlugs ) {
 
-            foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sSubMenuSlug ) ) as $_nIndex => $_aSubMenuItem ) {
+            foreach ( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sSubMenuSlug ) ) as $_nIndex => $_aSubMenuItem ) {
                 
-                foreach( $aLinkSlugs as $_sLinkSlug => $_nOrder ) {
+                foreach ( $aLinkSlugs as $_sLinkSlug => $_nOrder ) {
                     
                     $_bIsSet = $this->_setSubMenuIndexByLinksSlug( $sSubMenuSlug, $_nIndex, $_aSubMenuItem, $_sLinkSlug, $_nOrder ); 
                     
@@ -193,7 +193,7 @@ 
             private function _setSubMenuIndexByLinksSlug( $sSubMenuSlug, $nIndex, $aSubMenuItem, $sLinkSlug, $nOrder ) {
                 
                 // The third item is the link slug.
-                if ( ! isset( $aSubMenuItem[ 2 ] ) ) { 
+                if ( !isset( $aSubMenuItem[ 2 ] ) ) { 
                     return false; 
                 }
                 if ( $aSubMenuItem[ 2 ] !== $sLinkSlug ) {

Braces +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
  * @subpackage      PostType
  * @internal
  */
-class AdminPageFramework_PostType_Model__SubMenuOrder extends AdminPageFramework_FrameworkUtility {    
+class AdminPageFramework_PostType_Model__SubMenuOrder extends AdminPageFramework_FrameworkUtility {
     
     /**
      * Stores a post type factory object.
@@ -95,7 +95,7 @@ 
         /**
          * @since       3.7.4
          */
-        private function _setSubMenuItemIndex( $sSubMenuSlug ) {                
+        private function _setSubMenuItemIndex( $sSubMenuSlug ) {
                                         
             // Only if custom values are set, set them.                
             $this->_setSubMenuIndexByLinksSlugs( 
@@ -193,7 +193,7 @@ 
             private function _setSubMenuIndexByLinksSlug( $sSubMenuSlug, $nIndex, $aSubMenuItem, $sLinkSlug, $nOrder ) {
                 
                 // The third item is the link slug.
-                if ( ! isset( $aSubMenuItem[ 2 ] ) ) { 
+                if ( ! isset( $aSubMenuItem[ 2 ] ) ) {
                     return false; 
                 }
                 if ( $aSubMenuItem[ 2 ] !== $sLinkSlug ) {

development/utility/plugin_bootstrap/AdminPageFramework_PluginBootstrap.php

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
      * @param       string      $sPluginFilePath        The plugin file path.
      * @param       string      $sPluginHookPrefix      The plugin hook slug without underscore. This will be used to construct hook names.
      * @param       string      $sSetUpHook             The action hook name for the setUp callback. Default 'plugins_loaded'.
-     * @param       string      $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
+     * @param       integer      $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
      */
     public function __construct( $sPluginFilePath, $sPluginHookPrefix='', $sSetUpHook='plugins_loaded', $iPriority=10 ) {
         

Spacing +5 added lines, -5 removed lines

@@ -40,7 +40,7 @@ 
      * @param       string      $sSetUpHook             The action hook name for the setUp callback. Default 'plugins_loaded'.
      * @param       string      $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
      */
-    public function __construct( $sPluginFilePath, $sPluginHookPrefix='', $sSetUpHook='plugins_loaded', $iPriority=10 ) {
+    public function __construct( $sPluginFilePath, $sPluginHookPrefix = '', $sSetUpHook = 'plugins_loaded', $iPriority = 10 ) {
         
         // Check if it has been loaded.
         if ( $this->_hasLoaded() ) {
@@ -76,7 +76,7 @@ 
         register_deactivation_hook( $this->sFilePath, array( $this, 'replyToPluginDeactivation' ) );
                  
         // 8. Schedule to load plugin specific components.
-        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) )  {
+        if ( !$this->sSetUpHook || did_action( $this->sSetUpHook ) ) {
             $this->_replyToLoadPluginComponents();
         } else {
             add_action( $this->sSetUpHook, array( $this, '_replyToLoadPluginComponents' ), $this->iPriority );
@@ -116,14 +116,14 @@ 
         protected function _registerClasses() {
             
             // This class should be used in the framework bootstrap so disabling the auto-load option for performance.
-            if ( ! class_exists( 'AdminPageFramework_RegisterClasses', false ) ) {
+            if ( !class_exists( 'AdminPageFramework_RegisterClasses', false ) ) {
                 return;
             }              
             
             // Register classes
             new AdminPageFramework_RegisterClasses( 
-                $this->getScanningDirs(),   // scanning directory paths
-                array(),                    // autoloader options
+                $this->getScanningDirs(), // scanning directory paths
+                array(), // autoloader options
                 $this->getClasses()         // pre-generated class list
             );
                     

Braces +2 added lines, -2 removed lines

@@ -76,7 +76,7 @@ 
         register_deactivation_hook( $this->sFilePath, array( $this, 'replyToPluginDeactivation' ) );
                  
         // 8. Schedule to load plugin specific components.
-        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) )  {
+        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) ) {
             $this->_replyToLoadPluginComponents();
         } else {
             add_action( $this->sSetUpHook, array( $this, '_replyToLoadPluginComponents' ), $this->iPriority );
@@ -100,7 +100,7 @@ 
         protected function _hasLoaded() {
             
             static $_bLoaded = false;
-            if ( $_bLoaded ) { 
+            if ( $_bLoaded ) {
                 return true; 
             }
             $_bLoaded = true;            

development/utility/pointer_tool_tip/AdminPageFramework_PointerToolTip.php

Doc Comments +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
      * 
      * @since       DEVVER
      * @see         https://codex.wordpress.org/Plugin_API/Admin_Screen_Reference
-     * @param       array|strin     $asScreenIDs        Screen IDs or page slug.
+     * @param       string[]     $asScreenIDs        Screen IDs or page slug.
      * @param       string          $sPointerID         A unique pointer ID.
      * @Param       array           $aPointerData       The pointer data.
      */

Indentation +1 added lines, -1 removed lines

@@ -173,7 +173,7 @@
                     get_current_user_id(), 
                     'dismissed_wp_pointers', 
                     true 
-               )
+                )
             );
             $_aValidPointers = array(
                 'pointers'  => array(),

Braces +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
     public function __construct( $asScreenIDs, $sPointerID, array $aPointerData ) {
 
         // Bail if the WordPress version is less than 3.3,
-        if ( version_compare( $GLOBALS[ 'wp_version' ], '3.3', '<' ) ) {        
+        if ( version_compare( $GLOBALS[ 'wp_version' ], '3.3', '<' ) ) {
             return false;
         }       
     
@@ -93,7 +93,7 @@ 
          * 
          */
         private function _setHooks( $aScreenIDs ) {
-            foreach( $aScreenIDs as $_sScreenID ) {            
+            foreach( $aScreenIDs as $_sScreenID ) {
                 if ( ! $_sScreenID ) {
                     continue;
                 }

Spacing +11 added lines, -11 removed lines

@@ -112,18 +112,18 @@ 
          */
         private function _setHooks( $aScreenIDs ) {
             
-            foreach( $aScreenIDs as $_sScreenID ) {            
-                if ( ! $_sScreenID ) {
+            foreach ( $aScreenIDs as $_sScreenID ) {            
+                if ( !$_sScreenID ) {
                     continue;
                 }
                 add_filter( 
-                    get_class( $this ) . '-' . $_sScreenID, 
+                    get_class( $this ).'-'.$_sScreenID, 
                     array( $this, '_replyToSetPointer' )
                 );
                                 
             }       
             
-            if ( ! $this->_hasBeenCalled() ) {
+            if ( !$this->_hasBeenCalled() ) {
                 return;
             } 
             
@@ -170,7 +170,7 @@ 
     
         $_aPointers = $this->_getValidPointers( $this->_getPointers() );
              
-        if ( empty( $_aPointers ) || ! is_array( $_aPointers ) ) {
+        if ( empty( $_aPointers ) || !is_array( $_aPointers ) ) {
             return;
         }
         
@@ -190,11 +190,11 @@ 
             $_oScreen   = get_current_screen();
             $_sScreenID = $_oScreen->id;    
             if ( in_array( $_sScreenID, $this->aScreenIDs ) ) {
-                return apply_filters( get_class( $this ) . '-' . $_sScreenID, array() );
+                return apply_filters( get_class( $this ).'-'.$_sScreenID, array() );
             } 
             
             if ( isset( $_GET[ 'page' ] ) ) {
-                return apply_filters( get_class( $this ) . '-' . $_GET[ 'page' ], array() );
+                return apply_filters( get_class( $this ).'-'.$_GET[ 'page' ], array() );
             }
             return array();
             
@@ -208,7 +208,7 @@ 
         private function _getValidPointers( $_aPointers ) {
         
             // Get dismissed pointers
-            $_aDismissed      = explode( 
+            $_aDismissed = explode( 
                 ',', 
                 ( string ) get_user_meta( 
                     get_current_user_id(), 
@@ -236,7 +236,7 @@ 
                 $_aPointer[ 'pointer_id' ] = $_iPointerID;
                 
                 // Add the pointer to $_aValidPointers array
-                $_aValidPointers[] =  $_aPointer;
+                $_aValidPointers[ ] = $_aPointer;
                 
             }            
             return $_aValidPointers;
@@ -312,9 +312,9 @@ 
          * @return      string  
          * @internal
          */
-        public function _getInternalScript( $aPointers=array() ) {
+        public function _getInternalScript( $aPointers = array() ) {
 
-            $_aJSArray      = json_encode( $aPointers );
+            $_aJSArray = json_encode( $aPointers );
 
             /**
              * Checks check-boxes in siblings.

include/class/admin/_abstract/AdminPageFrameworkLoader_AdminPage_Page_Base.php

Doc Comments +1 added lines

@@ -25,6 +25,7 @@
 
     /**
      * Sets up hooks and properties.
+     * @param AdminPageFrameworkLoader_AdminPage $oFactory
      */
     public function __construct( $oFactory, array $aPageArguments ) {
         

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
     public function __construct( $oFactory, array $aPageArguments ) {
         
         $this->oFactory     = $oFactory;
-        $this->sPageSlug    = $aPageArguments['page_slug'];
+        $this->sPageSlug    = $aPageArguments[ 'page_slug' ];
         $this->_addPage( $aPageArguments );
         $this->construct( $oFactory );
         
@@ -58,8 +58,8 @@ 
      */
     public function replyToLoadResources( $oFactory ) {
 
-        $_sCSSPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/' . $this->sPageSlug . '.css';
-        if ( ! file_exists( $_sCSSPath ) ) {
+        $_sCSSPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/'.$this->sPageSlug.'.css';
+        if ( !file_exists( $_sCSSPath ) ) {
             return;
         }
         $this->oFactory->enqueueStyle( 

include/class/AdminPageFrameworkLoader_Bootstrap.php

Spacing +9 added lines, -9 removed lines

@@ -31,8 +31,8 @@ 
     public function getClasses() {
         
         // Include the include lists. The including file reassigns the list(array) to the $_aClassFiles variable.
-        $_aClassFiles   = array();
-        include( dirname( $this->sFilePath ) . '/include/loader-class-list.php' );
+        $_aClassFiles = array();
+        include( dirname( $this->sFilePath ).'/include/loader-class-list.php' );
         $this->_aClassFiles = $_aClassFiles;
         return $_aClassFiles;
                 
@@ -60,7 +60,7 @@ 
             if ( $_oRequirementCheck->check() ) {            
                 $_oRequirementCheck->deactivatePlugin( 
                     $this->sFilePath, 
-                    __( 'Deactivating the plugin', 'admin-page-framework-loader' ),  // additional message
+                    __( 'Deactivating the plugin', 'admin-page-framework-loader' ), // additional message
                     true    // is in the activation hook. This will exit the script.
                 );
             }        
@@ -75,7 +75,7 @@ 
     public function setLocalization() {
         
         // This plugin does not have messages to be displayed in the front end.
-        if ( ! $this->bIsAdmin ) { 
+        if ( !$this->bIsAdmin ) { 
             return; 
         }
        
@@ -83,13 +83,13 @@ 
         load_plugin_textdomain( 
             AdminPageFrameworkLoader_Registry::TEXT_DOMAIN, 
             false, 
-            $_sPluginBaseNameDirName . '/' . AdminPageFrameworkLoader_Registry::TEXT_DOMAIN_PATH
+            $_sPluginBaseNameDirName.'/'.AdminPageFrameworkLoader_Registry::TEXT_DOMAIN_PATH
         );
             
         load_plugin_textdomain( 
             'admin-page-framework', 
             false, 
-            $_sPluginBaseNameDirName . '/' . AdminPageFrameworkLoader_Registry::TEXT_DOMAIN_PATH
+            $_sPluginBaseNameDirName.'/'.AdminPageFrameworkLoader_Registry::TEXT_DOMAIN_PATH
         );        
         
     }        
@@ -112,14 +112,14 @@ 
 
             // Loader plugin admin pages.
             new AdminPageFrameworkLoader_AdminPage( 
-                AdminPageFrameworkLoader_Registry::$aOptionKeys[ 'main' ],    // the option key
+                AdminPageFrameworkLoader_Registry::$aOptionKeys[ 'main' ], // the option key
                 $this->sFilePath   // caller script path
             );
 
             // Network admin pages.
             if ( is_network_admin() ) {
                 new AdminPageFrameworkLoader_NetworkAdmin(
-                    AdminPageFrameworkLoader_Registry::$aOptionKeys[ 'main' ],    // the option key
+                    AdminPageFrameworkLoader_Registry::$aOptionKeys[ 'main' ], // the option key
                     $this->sFilePath   // caller script path            
                 );
             }
@@ -139,7 +139,7 @@ 
          */
         private function _shouldShowAdminPages() {
        
-            if ( ! $this->bIsAdmin ) {
+            if ( !$this->bIsAdmin ) {
                 return false;
             }
             if ( AdminPageFrameworkLoader_Utility::isSilentMode() ) {

Braces +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
                 AdminPageFrameworkLoader_Registry::NAME
             );
             
-            if ( $_oRequirementCheck->check() ) {            
+            if ( $_oRequirementCheck->check() ) {
                 $_oRequirementCheck->deactivatePlugin( 
                     $this->sFilePath, 
                     __( 'Deactivating the plugin', 'admin-page-framework-loader' ),  // additional message
@@ -76,7 +76,7 @@ 
     public function setLocalization() {
         
         // This plugin does not have messages to be displayed in the front end.
-        if ( ! $this->bIsAdmin ) { 
+        if ( ! $this->bIsAdmin ) {
             return; 
         }
         

include/class/admin/dashboard/AdminPageFrameworkLoader_AdminPageWelcome.php

Spacing +22 added lines, -22 removed lines

@@ -25,7 +25,7 @@ 
      */
     public function start() {
         
-        if ( ! is_admin() ) {
+        if ( !is_admin() ) {
             return;
         }
         
@@ -47,7 +47,7 @@ 
 
             // When newly installed, the 'welcomed' value is not set.
             $_oOption = AdminPageFrameworkLoader_Option::getInstance();
-            if ( ! $_oOption->get( 'welcomed' ) ) {                
+            if ( !$_oOption->get( 'welcomed' ) ) {                
                 $this->_setInitialOptions( $_oOption, AdminPageFrameworkLoader_Registry::VERSION );
                 $this->_goToWelcomePage(); // will exit
             }
@@ -73,7 +73,7 @@ 
         private function _goToWelcomePage() {    
         
             $_sWelcomePageURL = apply_filters(
-                AdminPageFrameworkLoader_Registry::HOOK_SLUG . '_filter_admin_welcome_redirect_url',
+                AdminPageFrameworkLoader_Registry::HOOK_SLUG.'_filter_admin_welcome_redirect_url',
                 add_query_arg( 
                     array( 'page' => AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ] ),
                     admin_url( 'index.php' )   // Dashboard
@@ -93,11 +93,11 @@ 
     public function setUp() {
   
         $_oOption = AdminPageFrameworkLoader_Option::getInstance();
-        if ( ! $_oOption->get( 'enable_admin_pages' ) ) {
+        if ( !$_oOption->get( 'enable_admin_pages' ) ) {
             return;
         }
         
-        $this->sPageSlug  = AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ];
+        $this->sPageSlug = AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ];
         
         // Root page
         $this->setRootMenuPage( 
@@ -108,12 +108,12 @@ 
         $this->addSubMenuItems( 
             array(
                 'title'         => AdminPageFrameworkLoader_Registry::SHORTNAME,
-                'page_slug'     => AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ],    // page slug
+                'page_slug'     => AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ], // page slug
                 'show_in_menu'  => false,
                 'style'         => array(
-                    AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/about.css', 
-                    AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/column.css', 
-                    AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/javascript/flip/jquery.m.flip.css',
+                    AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/about.css', 
+                    AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/column.css', 
+                    AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/javascript/flip/jquery.m.flip.css',
                     version_compare( $GLOBALS[ 'wp_version' ], '3.8', '<' )
                         ? ".about-wrap .introduction h2 {
                                 padding: 1em;
@@ -127,7 +127,7 @@ 
                         }",     
                 ),
                 'script'        => array(
-                    AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/javascript/flip/jquery.m.flip.js',
+                    AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/javascript/flip/jquery.m.flip.js',
                     "jQuery( document ).ready( function() {
                         jQuery( '.apf-badge-image' ).mflip();
                     } );",
@@ -138,8 +138,8 @@ 
         $this->setPluginSettingsLinkLabel( '' ); // pass an empty string to disable it.
         
         // Hook
-        add_action( "load_" . $this->oProp->sClassName, array( $this, 'replyToLoadClassPages' ) );
-        add_action( "load_" . AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ], array( $this, 'replyToLoadPage' ) );
+        add_action( "load_".$this->oProp->sClassName, array( $this, 'replyToLoadClassPages' ) );
+        add_action( "load_".AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ], array( $this, 'replyToLoadPage' ) );
 
     }   
     
@@ -165,14 +165,14 @@ 
 
         $_sPageSlug = AdminPageFrameworkLoader_Registry::$aAdminPages[ 'about' ];
         new AdminPageFrameworkLoader_AdminPageWelcome_Welcome( 
-            $this,              // factory object
-            $_sPageSlug,        // page slug
+            $this, // factory object
+            $_sPageSlug, // page slug
             array(
                 'tab_slug'      => 'welcome',
                 // 'title'         => __( "What's New", 'admin-page-framework-loader' ),   // '
                 'style'         => array(
-                    AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/admin.css',
-                    AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/code.css',
+                    AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/admin.css',
+                    AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/code.css',
                 ),
             )                
         );        
@@ -201,24 +201,24 @@ 
      */
     public function replyToFilterContentTop( $sContent ) {
 
-        $_sVersion      = '- ' . AdminPageFrameworkLoader_Registry::VERSION;
-        $_sPluginName   = AdminPageFrameworkLoader_Registry::SHORTNAME . ' ' . $_sVersion;
+        $_sVersion      = '- '.AdminPageFrameworkLoader_Registry::VERSION;
+        $_sPluginName   = AdminPageFrameworkLoader_Registry::SHORTNAME.' '.$_sVersion;
         
         $_sBadgeURL     = esc_url( AdminPageFrameworkLoader_Registry::getPluginURL( 'asset/image/icon-128x128.png' ) );
         
         $_aOutput   = array();
-        $_aOutput[] = "<h1>" 
+        $_aOutput[ ] = "<h1>" 
                 . sprintf( __( 'Welcome to %1$s', 'admin-page-framework-loader' ), $_sPluginName )
             . "</h1>";
-        $_aOutput[] = "<div class='about-text'>"
+        $_aOutput[ ] = "<div class='about-text'>"
                 . sprintf( __( 'Thank you for updating to the latest version! %1$s is ready to make your plugin or theme development faster, more organized and better!', 'admin-page-framework-loader' ), $_sPluginName )
             . "</div>";
-        $_aOutput[] = ''
+        $_aOutput[ ] = ''
                 . "<div class='apf-badge'>"
                     . "<div class='apf-badge-image m-flip'>"
                         . "<img src='{$_sBadgeURL}' />"
                     . "</div>"
-                    . "<span class='label'>" . sprintf( __( 'Version %1$s', 'admin-page-framework-loader' ), $_sVersion ) . "</span>"
+                    . "<span class='label'>".sprintf( __( 'Version %1$s', 'admin-page-framework-loader' ), $_sVersion )."</span>"
                 . "</div>";
            
         return implode( PHP_EOL, $_aOutput )

Braces +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
 
             // When newly installed, the 'welcomed' value is not set.
             $_oOption = AdminPageFrameworkLoader_Option::getInstance();
-            if ( ! $_oOption->get( 'welcomed' ) ) {                
+            if ( ! $_oOption->get( 'welcomed' ) ) {
                 $this->_setInitialOptions( $_oOption, AdminPageFrameworkLoader_Registry::VERSION );
                 $this->_goToWelcomePage(); // will exit
             }
@@ -70,7 +70,7 @@ 
                 
             }
             
-        private function _goToWelcomePage() {    
+        private function _goToWelcomePage() {
         
             $_sWelcomePageURL = apply_filters(
                 AdminPageFrameworkLoader_Registry::HOOK_SLUG . '_filter_admin_welcome_redirect_url',

include/class/admin/admin-page-framework/AdminPageFrameworkLoader_AdminPage.php

Spacing +30 added lines, -30 removed lines

@@ -18,12 +18,12 @@ 
      */
     public function start() {
   
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
       
         // Allows the user to switch the menu visibility.
-        if ( isset( $_GET['enable_apfl_admin_pages'] ) ) {
+        if ( isset( $_GET[ 'enable_apfl_admin_pages' ] ) ) {
             
             // Update the options and reload the page
             $_oOption = AdminPageFrameworkLoader_Option::getInstance( AdminPageFrameworkLoader_Registry::$aOptionKeys[ 'main' ] );
@@ -67,15 +67,15 @@ 
     public function setUp() {
 
         $_aOptions          = $this->oProp->aOptions;
-        $_bAdminPageEnabled = ! is_array( $_aOptions )  // for the first time of loading, the option is not set and it is not an array. 
-            || ( isset( $_aOptions['enable_admin_pages'] ) && $_aOptions['enable_admin_pages'] );    
+        $_bAdminPageEnabled = !is_array( $_aOptions )  // for the first time of loading, the option is not set and it is not an array. 
+            || ( isset( $_aOptions[ 'enable_admin_pages' ] ) && $_aOptions[ 'enable_admin_pages' ] );    
     
         // Set up pages
         if ( $_bAdminPageEnabled ) {
             
             $this->setRootMenuPage( 
-                AdminPageFrameworkLoader_Registry::SHORTNAME,     // menu slug
-                AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/image/wp-logo_16x16.png', // menu icon
+                AdminPageFrameworkLoader_Registry::SHORTNAME, // menu slug
+                AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/image/wp-logo_16x16.png', // menu icon
                 4  // menu position
             ); 
                         
@@ -86,19 +86,19 @@ 
                     'page_slug' => AdminPageFrameworkLoader_Registry::$aAdminPages[ 'tool' ],
                     'title'     => __( 'Tools', 'admin-page-framework-loader' ),
                     'style'     => array(
-                        AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/code.css',
-                        AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/admin.css'
+                        AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/code.css',
+                        AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/admin.css'
                     ),
                 )
             );
             new AdminPageFrameworkLoader_AdminPage_Addon(
                 $this,
                 array(
-                    'page_slug' => AdminPageFrameworkLoader_Registry::$aAdminPages[ 'addon' ],    // page slug
+                    'page_slug' => AdminPageFrameworkLoader_Registry::$aAdminPages[ 'addon' ], // page slug
                     'title'     => __( 'Add Ons', 'admin-page-framework-loader' ),
                     'style'     => array(
-                        AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/code.css',
-                        AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/admin.css'
+                        AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/code.css',
+                        AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/admin.css'
                     ),                    
                 )                
             );
@@ -109,13 +109,13 @@ 
                     'title'     => __( 'Help', 'admin-page-framework-loader' ),
                     'order'     => 1000, // to be the last menu item
                     'style'     => array(
-                        AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/code.css',
-                        AdminPageFrameworkLoader_Registry::$sDirPath . '/asset/css/admin.css'
+                        AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/code.css',
+                        AdminPageFrameworkLoader_Registry::$sDirPath.'/asset/css/admin.css'
                     ),                    
                 )                
             );
             
-            add_action( 'load_' . $this->oProp->sClassName, array( $this, 'replyToDoPageSettings' ) );
+            add_action( 'load_'.$this->oProp->sClassName, array( $this, 'replyToDoPageSettings' ) );
             
         }
               
@@ -128,7 +128,7 @@ 
          */
         private function _addActionLinks( $_bAdminPageEnabled, $_aOptions ) {
           
-            if ( 'plugins.php' !==  $this->oProp->sPageNow ) {
+            if ( 'plugins.php' !== $this->oProp->sPageNow ) {
                 return;
             }
           
@@ -152,24 +152,24 @@ 
              * @return      string
              */
             private function _getAdminURLTools( $_bAdminPageEnabled ) {
-                if ( ! $_bAdminPageEnabled ) {
+                if ( !$_bAdminPageEnabled ) {
                     return;
                 }
-                $_sLink    = esc_url(
+                $_sLink = esc_url(
                     add_query_arg( 
                         array( 
-                            'page' => AdminPageFrameworkLoader_Registry::$aAdminPages['tool'],
+                            'page' => AdminPageFrameworkLoader_Registry::$aAdminPages[ 'tool' ],
                         ),
                         admin_url( 'admin.php' )
                     )
                 );                
-                return "<a href='{$_sLink}'>" . __( 'Tools', 'admin-page-framework-loader' ) . "</a>";
+                return "<a href='{$_sLink}'>".__( 'Tools', 'admin-page-framework-loader' )."</a>";
             }
             /**
              * Returns the Enable /Disable Admin Pages link.
              */
             private function _getAdminPageSwitchLink( $bEnabled ) {
-                $_sLink    = esc_url( 
+                $_sLink = esc_url( 
                     add_query_arg( 
                         array( 
                             'enable_apfl_admin_pages' => $bEnabled ? 0 : 1,
@@ -177,19 +177,19 @@ 
                     )
                 );            
                 return $bEnabled
-                    ? "<a href='{$_sLink}'>" . __( 'Disable Admin Pages', 'admin-page-framework-loader' ) . "</a>"
-                    : "<a href='{$_sLink}'>" . __( 'Enable Admin Pages', 'admin-page-framework-loader' ) . "</a>";                     
+                    ? "<a href='{$_sLink}'>".__( 'Disable Admin Pages', 'admin-page-framework-loader' )."</a>"
+                    : "<a href='{$_sLink}'>".__( 'Enable Admin Pages', 'admin-page-framework-loader' )."</a>";                     
             }
             /**
              * Returns the switch link of the demo pages.
              */
-            private function _getDemoSwitcherLink( $_bAdminPageEnabled, $mOptions=array() ) {
+            private function _getDemoSwitcherLink( $_bAdminPageEnabled, $mOptions = array() ) {
                 
-                if ( ! $_bAdminPageEnabled ) {
+                if ( !$_bAdminPageEnabled ) {
                     return '';
                 }
-                $_bEnabled  = isset( $mOptions['enable_demo'] ) && $mOptions['enable_demo'];
-                $_sLink    = esc_url( 
+                $_bEnabled = isset( $mOptions[ 'enable_demo' ] ) && $mOptions[ 'enable_demo' ];
+                $_sLink = esc_url( 
                     add_query_arg( 
                         array( 
                             'enable_apfl_demo_pages' => $_bEnabled ? 0 : 1,
@@ -197,8 +197,8 @@ 
                     )
                 );        
                 return $_bEnabled
-                    ? "<a href='{$_sLink}'>" . __( 'Disable Demo', 'admin-page-framework-loader' ) . "</a>"
-                    : "<a href='{$_sLink}'><strong id='activate-demo-action-link' style='font-size: 1em;'>" . __( 'Enable Demo', 'admin-page-framework-loader' ) . "</strong></a>";
+                    ? "<a href='{$_sLink}'>".__( 'Disable Demo', 'admin-page-framework-loader' )."</a>"
+                    : "<a href='{$_sLink}'><strong id='activate-demo-action-link' style='font-size: 1em;'>".__( 'Enable Demo', 'admin-page-framework-loader' )."</strong></a>";
                 
             }      
 
@@ -214,14 +214,14 @@ 
     
         // Page meta boxes            
         new AdminPageFrameworkLoader_AdminPageMetaBox_Notification(
-            null,                                           // meta box id - passing null will make it auto generate
+            null, // meta box id - passing null will make it auto generate
             __( 'Notification', 'admin-page-framework-loader' ), // title
             array( // page slugs
                 AdminPageFrameworkLoader_Registry::$aAdminPages[ 'tool' ],
                 AdminPageFrameworkLoader_Registry::$aAdminPages[ 'addon' ],
                 AdminPageFrameworkLoader_Registry::$aAdminPages[ 'help' ],
             ),
-            'side',                                       // context
+            'side', // context
             'default'                                     // priority
         );        
     

include/class/admin/admin-page-framework/addon/AdminPageFrameworkLoader_AdminPage_Addon.php

Spacing +2 added lines, -2 removed lines

@@ -19,8 +19,8 @@
     public function replyToLoadPage( $oFactory ) {
         
         new AdminPageFrameworkLoader_AdminPage_Addon_Top( 
-            $oFactory,          // factory object
-            $this->sPageSlug,   // page slug
+            $oFactory, // factory object
+            $this->sPageSlug, // page slug
             array(              // tab definition
                 'tab_slug'      => 'top',
                 'title'         => __( 'Add Ons', 'admin-page-framework-loader' ),