michaeluno / admin-page-framework

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_Hook.php

Doc Comments +1 added lines, -9 removed lines

@@ -89,11 +89,6 @@ 
      * @access      public
      * @since       2.0.0
      * @remark      Accepts variadic parameters.
-     * @param       object      $oCallerObject      the object that holds the callback method that matches the action hook name.
-     * @param       array       $aActionHooks       a numerically index array consisting of action hook names that serve as the callback method names. 
-     * @param       mixed       $vArgs1             the argument to pass to the hook callback functions.
-     * @param       mixed       $vArgs2             another argument to pass to the hook callback functions.
-     * @param       mixed       $_and_more          add as many arguments as necessary to the next parameters.
      * @return      void
      */ 
     static public function addAndDoActions( /* $oCallerObject, $aActionHooks, $vArgs1=null, $vArgs2=null, $_and_more=null */ ) {
@@ -192,10 +187,6 @@ 
      * `
      * 
      * @since       2.0.0
-     * @param       object      $oCallerObject
-     * @param       string      $sFilter            The filter hook name.
-     * @param       mixed       $vData              The filtering data
-     * @param       mixed       $vArgs              The arguments.
      */
     static public function addAndApplyFilter( /* $oCallerObject, $sFilter, $vData, $vArgs... */ ) { 
         
@@ -233,6 +224,7 @@ 
      * 
      * @since       2.0.0
      * @access      public
+     * @param string $sPrefix
      * @return      array       Returns an array consisting of the filters.
      */ 
     static public function getFilterArrayByPrefix( $sPrefix, $sClassName, $sPageSlug, $sTabSlug, $bReverse=false ) {

Spacing +17 added lines, -17 removed lines

@@ -26,7 +26,7 @@ 
      * @since       3.4.2
      * @todo        Create a similar function for filters.
      */
-    static public function registerAction( $sActionHook, $oCallable, $iPriority=10 ) {
+    static public function registerAction( $sActionHook, $oCallable, $iPriority = 10 ) {
       
         if ( did_action( $sActionHook ) ) {
             return call_user_func_array( $oCallable, array() );
@@ -63,13 +63,13 @@ 
      * @param       mixed       $_and_more      add as many arguments as necessary to the next parameters.
      * @return      void        does not return a value.
      */     
-    static public function doActions( $aActionHooks, $vArgs1=null, $vArgs2=null, $_and_more=null ) {
+    static public function doActions( $aActionHooks, $vArgs1 = null, $vArgs2 = null, $_and_more = null ) {
         
         $aArgs          = func_get_args();     
         $aActionHooks   = $aArgs[ 0 ];
-        foreach( ( array ) $aActionHooks as $sActionHook  ) {
+        foreach ( ( array ) $aActionHooks as $sActionHook ) {
             $aArgs[ 0 ] = $sActionHook;
-            call_user_func_array( 'do_action' , $aArgs );
+            call_user_func_array( 'do_action', $aArgs );
         }
 
     }
@@ -101,12 +101,12 @@ 
         $aArgs          = func_get_args();    
         $oCallerObject  = $aArgs[ 0 ];
         $aActionHooks   = $aArgs[ 1 ];
-        foreach( ( array ) $aActionHooks as $sActionHook ) {
-            if ( ! $sActionHook ) { 
+        foreach ( ( array ) $aActionHooks as $sActionHook ) {
+            if ( !$sActionHook ) { 
                 continue; 
             }
             $aArgs[ 1 ] = $sActionHook;    
-            call_user_func_array( array( get_class(), 'addAndDoAction' ) , $aArgs );     
+            call_user_func_array( array( get_class(), 'addAndDoAction' ), $aArgs );     
         }
         
     }
@@ -125,7 +125,7 @@ 
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sActionHook    = $_aArgs[ 1 ];
-        if ( ! $_sActionHook ) { 
+        if ( !$_sActionHook ) { 
             return; 
         }
         
@@ -137,7 +137,7 @@ 
 
         // Remove the first element, the caller object.
         array_shift( $_aArgs ); 
-        call_user_func_array( 'do_action' , $_aArgs );
+        call_user_func_array( 'do_action', $_aArgs );
         
     }
     
@@ -165,12 +165,12 @@ 
         $_aFilters      = $_aArgs[ 1 ];
         $_vInput        = $_aArgs[ 2 ];
 
-        foreach( ( array ) $_aFilters as $_sFilter ) {
-            if ( ! $_sFilter ) { 
+        foreach ( ( array ) $_aFilters as $_sFilter ) {
+            if ( !$_sFilter ) { 
                 continue; 
             }
             $_aArgs[ 1 ] = $_sFilter;
-            $_aArgs[ 2 ] = $_vInput;    // assigns the updated value as it is filtered in previous iterations
+            $_aArgs[ 2 ] = $_vInput; // assigns the updated value as it is filtered in previous iterations
             $_vInput = call_user_func_array( 
                 array( get_class(), 'addAndApplyFilter' ), 
                 $_aArgs 
@@ -204,7 +204,7 @@ 
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sFilter        = $_aArgs[ 1 ];
-        if ( ! $_sFilter ) { 
+        if ( !$_sFilter ) { 
             return $_aArgs[ 2 ]; 
         }
         
@@ -235,17 +235,17 @@ 
      * @access      public
      * @return      array       Returns an array consisting of the filters.
      */ 
-    static public function getFilterArrayByPrefix( $sPrefix, $sClassName, $sPageSlug, $sTabSlug, $bReverse=false ) {
+    static public function getFilterArrayByPrefix( $sPrefix, $sClassName, $sPageSlug, $sTabSlug, $bReverse = false ) {
                 
         $_aFilters = array();
         if ( $sTabSlug && $sPageSlug ) {
-            $_aFilters[] = "{$sPrefix}{$sPageSlug}_{$sTabSlug}";
+            $_aFilters[ ] = "{$sPrefix}{$sPageSlug}_{$sTabSlug}";
         }
         if ( $sPageSlug ) {
-            $_aFilters[] = "{$sPrefix}{$sPageSlug}";     
+            $_aFilters[ ] = "{$sPrefix}{$sPageSlug}";     
         }
         if ( $sClassName ) {
-            $_aFilters[] = "{$sPrefix}{$sClassName}";
+            $_aFilters[ ] = "{$sPrefix}{$sClassName}";
         }
         return $bReverse 
             ? array_reverse( $_aFilters ) 

Braces +6 added lines, -6 removed lines

@@ -102,7 +102,7 @@ 
         $oCallerObject  = $aArgs[ 0 ];
         $aActionHooks   = $aArgs[ 1 ];
         foreach( ( array ) $aActionHooks as $sActionHook ) {
-            if ( ! $sActionHook ) { 
+            if ( ! $sActionHook ) {
                 continue; 
             }
             $aArgs[ 1 ] = $sActionHook;    
@@ -125,7 +125,7 @@ 
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sActionHook    = $_aArgs[ 1 ];
-        if ( ! $_sActionHook ) { 
+        if ( ! $_sActionHook ) {
             return; 
         }
         
@@ -158,7 +158,7 @@ 
      * 
      * @since       2.0.0
      */    
-    static public function addAndApplyFilters( /* $oCallerObject, $aFilters, $vInput, $vArgs... */ ) { 
+    static public function addAndApplyFilters( /* $oCallerObject, $aFilters, $vInput, $vArgs... */ ) {
             
         $_aArgs         = func_get_args();    
         // $oCallerObject  = $_aArgs[ 0 ];
@@ -166,7 +166,7 @@ 
         $_vInput        = $_aArgs[ 2 ];
 
         foreach( ( array ) $_aFilters as $_sFilter ) {
-            if ( ! $_sFilter ) { 
+            if ( ! $_sFilter ) {
                 continue; 
             }
             $_aArgs[ 1 ] = $_sFilter;
@@ -197,14 +197,14 @@ 
      * @param       mixed       $vData              The filtering data
      * @param       mixed       $vArgs              The arguments.
      */
-    static public function addAndApplyFilter( /* $oCallerObject, $sFilter, $vData, $vArgs... */ ) { 
+    static public function addAndApplyFilter( /* $oCallerObject, $sFilter, $vData, $vArgs... */ ) {
         
         // Parameters
         $_iArgs          = func_num_args();
         $_aArgs          = func_get_args();
         $_oCallerObject  = $_aArgs[ 0 ];
         $_sFilter        = $_aArgs[ 1 ];
-        if ( ! $_sFilter ) { 
+        if ( ! $_sFilter ) {
             return $_aArgs[ 2 ]; 
         }
         

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_Page.php

Doc Comments +1 added lines

@@ -284,6 +284,7 @@
              * Checkes whether the passed base url name is of the admin index page.
              * @since       3.5.3
              * return       boolean      Whether the passed base url name is of the admin index page.
+             * @param string $sPageNow
              */
             static private function _isInAdminIndex( $sPageNow ) {
                 return in_array(

Spacing +11 added lines, -11 removed lines

@@ -57,7 +57,7 @@ 
                 'getPostTypeByTypeNow',
                 'getPostTypeByScreenObject',
                 'getPostTypeByREQUEST',
-                'getPostTypeByPostObject',  // 3.6.0+ Moved to the last as it is not reliable.
+                'getPostTypeByPostObject', // 3.6.0+ Moved to the last as it is not reliable.
             );
             foreach ( $_aMethodsToTry as $_sMethodName ) {
                 $_sPostType = call_user_func( array( __CLASS__, $_sMethodName ) );
@@ -121,9 +121,9 @@ 
      * @param       array|string        The post type slug(s) to check. If this is empty, the method just checks the current page is a taxonomy page.
      * @return      boolean
      */    
-    static public function isCustomTaxonomyPage( $asPostTypes=array() ) {
+    static public function isCustomTaxonomyPage( $asPostTypes = array() ) {
         
-        if ( ! in_array( self::getPageNow(), array( 'tags.php', 'edit-tags.php', ) ) ) {
+        if ( !in_array( self::getPageNow(), array( 'tags.php', 'edit-tags.php',) ) ) {
             return false;
         }
         return self::isCurrentPostTypeIn( $asPostTypes );
@@ -138,10 +138,10 @@ 
      * Otherwise, it will check if the page belongs to the given post type(s).
      * @return      boolean
      */
-    static public function isPostDefinitionPage( $asPostTypes=array() ) {
+    static public function isPostDefinitionPage( $asPostTypes = array() ) {
         
         // If it's not the post definition page, 
-        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) { 
+        if ( !in_array( self::getPageNow(), array( 'post.php', 'post-new.php',) ) ) { 
             return false;
         }
         return self::isCurrentPostTypeIn( $asPostTypes );
@@ -174,7 +174,7 @@ 
      * 
      * @since 3.0.0
      */
-    static public function isPostListingPage( $asPostTypes=array() ) {
+    static public function isPostListingPage( $asPostTypes = array() ) {
                 
         if ( 'edit.php' != self::getPageNow() ) { 
             return false;
@@ -182,7 +182,7 @@ 
         
         $_aPostTypes = self::getAsArray( $asPostTypes );    
         
-        if ( ! isset( $_GET[ 'post_type' ] )  ) { 
+        if ( !isset( $_GET[ 'post_type' ] ) ) { 
             return in_array( 'post', $_aPostTypes );
         }
 
@@ -220,7 +220,7 @@ 
             0 => '_getPageNow_FrontEnd',
             1 => '_getPageNow_BackEnd',
         );
-        $_sMethodName  = $_aMethodNames[ ( integer ) is_admin() ];
+        $_sMethodName = $_aMethodNames[ ( integer ) is_admin() ];
         self::$_sPageNow = self::$_sMethodName();            
         return self::$_sPageNow;          
         
@@ -315,7 +315,7 @@ 
         
         if ( isset( $GLBOALS[ 'page_hook' ] ) ) {
             return is_network_admin() 
-                ? $GLBOALS[ 'page_hook' ] . '-network'
+                ? $GLBOALS[ 'page_hook' ].'-network'
                 : $GLBOALS[ 'page_hook' ];
         }
         
@@ -329,11 +329,11 @@ 
      * @since       3.7.0
      * @return      boolean
      */
-    static public function doesMetaBoxExist( $sContext='' ) {
+    static public function doesMetaBoxExist( $sContext = '' ) {
         
         $_aDimensions = array( 'wp_meta_boxes', $GLOBALS[ 'page_hook' ] );
         if ( $sContext ) {
-            $_aDimensions[] = $sContext;
+            $_aDimensions[ ] = $sContext;
         }
         $_aSideMetaBoxes = self::getElementAsArray( 
             $GLOBALS, 

Braces +7 added lines, -9 removed lines

@@ -29,7 +29,7 @@ 
      */
     static public function getCurrentPostType() {
                          
-        if ( isset( self::$_sCurrentPostType ) ) { 
+        if ( isset( self::$_sCurrentPostType ) ) {
             return self::$_sCurrentPostType; 
         }
         self::$_sCurrentPostType = self::_getCurrentPostType();
@@ -141,7 +141,7 @@ 
     static public function isPostDefinitionPage( $asPostTypes=array() ) {
         
         // If it's not the post definition page, 
-        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) { 
+        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) {
             return false;
         }
         return self::isCurrentPostTypeIn( $asPostTypes );
@@ -160,7 +160,7 @@ 
         $_aPostTypes = self::getAsArray( $asPostTypes );        
         
         // If the parameter is empty, 
-        if ( empty( $_aPostTypes ) ) { 
+        if ( empty( $_aPostTypes ) ) {
             return true;
         }
 
@@ -176,13 +176,13 @@ 
      */
     static public function isPostListingPage( $asPostTypes=array() ) {
                 
-        if ( 'edit.php' != self::getPageNow() ) { 
+        if ( 'edit.php' != self::getPageNow() ) {
             return false;
         }
         
         $_aPostTypes = self::getAsArray( $asPostTypes );    
         
-        if ( ! isset( $_GET[ 'post_type' ] )  ) { 
+        if ( ! isset( $_GET[ 'post_type' ] )  ) {
             return in_array( 'post', $_aPostTypes );
         }
 
@@ -273,11 +273,9 @@ 
                 
                 if ( is_network_admin() ) {
                     $_sNeedle = '#/wp-admin/network/?(.*?)$#i';
-                }
-                else if ( is_user_admin() ) {
+                } else if ( is_user_admin() ) {
                     $_sNeedle = '#/wp-admin/user/?(.*?)$#i';
-                }
-                else {
+                } else {
                     $_sNeedle = '#/wp-admin/?(.*?)$#i';
                 }                
                 preg_match( $_sNeedle, $_SERVER[ 'PHP_SELF' ], $_aMatches );

development/factory/admin_page/_model/delegate/AdminPageFramework_Model_Menu__RegisterMenu.php

Doc Comments +4 added lines, -1 removed lines

@@ -27,6 +27,7 @@ 
      * Sets up hooks
      * 
      * @internal
+     * @param AdminPageFramework_Model_Menu $oFactory
      */
     public function __construct( $oFactory, $sActionHook='admin_menu' ) {
         
@@ -228,6 +229,7 @@ 
              * @since       3.3.0
              * @since       3.1.1       Moved from `AdminPageFramework_Menu`.
              * @since       3.7.4       Added the `$nOrder` parameter.
+             * @param string $sMenuSlug
              * @return      string      The page hook of the added page.
              * @uses        add_submenu_page
              */
@@ -340,7 +342,7 @@ 
                 /**
                  * @since       3.7.4
                  * @return      void
-                 * @param       numeric     $$nOrder            A user set order (menu position, index).
+                 * @param       numeric     $nOrder            A user set order (menu position, index).
                  * @param       array       $aSubMenuItem       The sub menu item array set in the global `$submenu` array.
                  */
                 private function _setSubMenuPageByIndex( $nOrder, $aSubMenuItem, $sMenuSlug ) {
@@ -464,6 +466,7 @@ 
              * @since       3.1.1       Moved from `AdminPageFramework_Menu`.
              * @since       3.5.3       Added the `$bShowInMenu` parameter.
              * @since       3.7.4       Added the `$nOrder` parameter.
+             * @param string $sMenuSlug
              * @return      void
              */
             private function _addLinkSubmenuItem( $sMenuSlug, $sTitle, $sCapability, $sHref, $bShowInMenu, $nOrder ) {

Spacing +33 added lines, -33 removed lines

@@ -28,7 +28,7 @@ 
      * 
      * @internal
      */
-    public function __construct( $oFactory, $sActionHook='admin_menu' ) {
+    public function __construct( $oFactory, $sActionHook = 'admin_menu' ) {
         
         $this->oFactory = $oFactory;
         
@@ -71,8 +71,8 @@ 
         
         // Apply filters to let other scripts add sub menu pages.
         $this->oFactory->oProp->aPages = $this->addAndApplyFilter( 
-            $this->oFactory,    // caller object
-            "pages_{$this->oFactory->oProp->sClassName}",   // filter
+            $this->oFactory, // caller object
+            "pages_{$this->oFactory->oProp->sClassName}", // filter
             $this->oFactory->oProp->aPages  // arguments
         );
         
@@ -120,7 +120,7 @@ 
 
             foreach ( $this->oFactory->oProp->aPages as $_aPage ) {
                 
-                if ( ! isset( $_aPage[ 'page_slug' ] ) ) { 
+                if ( !isset( $_aPage[ 'page_slug' ] ) ) { 
                     continue; 
                 }
                 $this->oFactory->oProp->sDefaultPageSlug = $_aPage[ 'page_slug' ];
@@ -140,12 +140,12 @@ 
         private function _registerRootMenuPage() {
 
             $this->oFactory->oProp->aRootMenu[ '_page_hook' ] = add_menu_page(  
-                $this->oFactory->oProp->sClassName,                 // Page title - will be invisible anyway
-                $this->oFactory->oProp->aRootMenu[ 'sTitle' ],      // Menu title - should be the root page title.
-                $this->oFactory->oProp->sCapability,                // Capability - access right
-                $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ],   // Menu ID 
-                '',                                       // Callback function for the page content output - the root page will be removed so no need to register a function.
-                $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ],  // icon path
+                $this->oFactory->oProp->sClassName, // Page title - will be invisible anyway
+                $this->oFactory->oProp->aRootMenu[ 'sTitle' ], // Menu title - should be the root page title.
+                $this->oFactory->oProp->sCapability, // Capability - access right
+                $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ], // Menu ID 
+                '', // Callback function for the page content output - the root page will be removed so no need to register a function.
+                $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ], // icon path
                 $this->getElement( 
                     $this->oFactory->oProp->aRootMenu, 
                     'iPosition', 
@@ -168,7 +168,7 @@ 
          */ 
         private function _registerSubMenuItem( array $aArgs ) {
 
-            if ( ! current_user_can( $aArgs[ 'capability' ] ) ) {
+            if ( !current_user_can( $aArgs[ 'capability' ] ) ) {
                 return '';
             }
                                     
@@ -233,16 +233,16 @@ 
              */
             private function _addPageSubmenuItem( $sRootPageSlug, $sMenuSlug, $sPageSlug, $sPageTitle, $sMenuTitle, $sCapability, $bShowInMenu, $nOrder ) {
                 
-                if ( ! $sPageSlug ) {
+                if ( !$sPageSlug ) {
                     return '';
                 }
                 
                 $_sPageHook = add_submenu_page( 
-                    $sRootPageSlug,         // the root (parent) page slug
-                    $sPageTitle,            // page title
-                    $sMenuTitle,            // menu title
-                    $sCapability,           // capability
-                    $sPageSlug,             // menu slug
+                    $sRootPageSlug, // the root (parent) page slug
+                    $sPageTitle, // page title
+                    $sMenuTitle, // menu title
+                    $sCapability, // capability
+                    $sPageSlug, // menu slug
                     /**
                      * In admin.php ( line 149 of WordPress v3.6.1 ), do_action($page_hook) ( where $page_hook is $_sPageHook )
                      * will be executed and it triggers the __call() magic method with the method name of "md5 class hash + _page_ + this page slug".
@@ -267,14 +267,14 @@ 
                 $_aRemovedMenuItem = $this->_removePageSubmenuItem( $_nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle );
                 
                 // If the visibility option is `false`, remove the one just added from the sub-menu global array
-                if ( ! $bShowInMenu && ! $this->_isCurrentPage( $sPageSlug ) ) {
+                if ( !$bShowInMenu && !$this->_isCurrentPage( $sPageSlug ) ) {
                     return $_sPageHook;
                 }                
                 
                 // Set the order index in the element of the `submenu` global array.
                 $this->_setSubMenuPageByIndex( 
-                    $nOrder,                // user-set order
-                    $_aRemovedMenuItem,     // will be reassign with a new index
+                    $nOrder, // user-set order
+                    $_aRemovedMenuItem, // will be reassign with a new index
                     $sMenuSlug
                 );
 
@@ -313,7 +313,7 @@ 
                      */
                     add_action( 
                         'current_screen', 
-                        array( $this->oFactory, "load_pre_" . $sPageSlug ), 
+                        array( $this->oFactory, "load_pre_".$sPageSlug ), 
                         20 
                     );
                     
@@ -322,15 +322,15 @@ 
                      * Set a low priority because the user may add in-page tabs in their callback method of this action hook.
                      * @since       3.6.3
                      */
-                    add_action( "load_" . $sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 );
+                    add_action( "load_".$sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 );
                     
                     // 3.6.3+
-                    add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) );
-                    add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) );  // 3.7.10+
+                    add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) );
+                    add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) ); // 3.7.10+
                           
                     $this->oFactory->oProp->aPageHooks[ $sPageSlug ] = $this->getAOrB(
                         is_network_admin(),
-                        $sPageHook . '-network',
+                        $sPageHook.'-network',
                         $sPageHook
                     );                          
 
@@ -347,7 +347,7 @@ 
 
                     $_nNewIndex = $this->getUnusedNumericIndex( 
                         $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ), // subject array to parser
-                        $nOrder,    // a desired menu position
+                        $nOrder, // a desired menu position
                         5           // offset 
                     );
 
@@ -361,9 +361,9 @@ 
                  */
                 private function _getSubMenuPageIndex( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug ) {
                     
-                    foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
+                    foreach ( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
                       
-                        if ( ! isset( $_aSubMenu[ 3 ] ) ) { 
+                        if ( !isset( $_aSubMenu[ 3 ] ) ) { 
                             continue; 
                         }
                                                
@@ -395,7 +395,7 @@ 
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`. Chagned the return type.
                  * @return      array       removed menu item.
                  */
-                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){
+                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) {
  
                     $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex( 
                         $nSubMenuPageIndex,
@@ -468,7 +468,7 @@ 
              */
             private function _addLinkSubmenuItem( $sMenuSlug, $sTitle, $sCapability, $sHref, $bShowInMenu, $nOrder ) {
                 
-                if ( ! $bShowInMenu ) {
+                if ( !$bShowInMenu ) {
                     return;
                 }
     
@@ -483,9 +483,9 @@ 
                     5   // offset
                 );
                 $_aSubMenuItems[ $_nIndex ] = array(
-                    $sTitle,        // 0
-                    $sCapability,   // 1
-                    $sHref,         // 2   
+                    $sTitle, // 0
+                    $sCapability, // 1
+                    $sHref, // 2   
                 );
                 $GLOBALS[ 'submenu' ][ $sMenuSlug ] = $_aSubMenuItems;                
                 

Braces +4 added lines, -4 removed lines

@@ -120,7 +120,7 @@ 
 
             foreach ( $this->oFactory->oProp->aPages as $_aPage ) {
                 
-                if ( ! isset( $_aPage[ 'page_slug' ] ) ) { 
+                if ( ! isset( $_aPage[ 'page_slug' ] ) ) {
                     continue; 
                 }
                 $this->oFactory->oProp->sDefaultPageSlug = $_aPage[ 'page_slug' ];
@@ -363,7 +363,7 @@ 
                     
                     foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
                       
-                        if ( ! isset( $_aSubMenu[ 3 ] ) ) { 
+                        if ( ! isset( $_aSubMenu[ 3 ] ) ) {
                             continue; 
                         }
                                                
@@ -378,7 +378,7 @@ 
                             $sPageTitle,
                             $sPageSlug,
                         );
-                        if ( $_aA !== $_aB ) { 
+                        if ( $_aA !== $_aB ) {
                             continue;
                         }
                         return $_iIndex;
@@ -395,7 +395,7 @@ 
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`. Chagned the return type.
                  * @return      array       removed menu item.
                  */
-                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){
+                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) {
  
                     $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex( 
                         $nSubMenuPageIndex,

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.
      */

Spacing +11 added lines, -11 removed lines

@@ -91,18 +91,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;
             } 
             
@@ -146,7 +146,7 @@ 
     
         $_aPointers = $this->_getValidPointers( $this->_getPointers() );
              
-        if ( empty( $_aPointers ) || ! is_array( $_aPointers ) ) {
+        if ( empty( $_aPointers ) || !is_array( $_aPointers ) ) {
             return;
         }
         
@@ -165,11 +165,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();
             
@@ -182,7 +182,7 @@ 
         private function _getValidPointers( $_aPointers ) {
         
             // Get dismissed pointers
-            $_aDismissed      = explode( 
+            $_aDismissed = explode( 
                 ',', 
                 ( string ) get_user_meta( 
                     get_current_user_id(), 
@@ -209,7 +209,7 @@ 
                 $_aPointer[ 'pointer_id' ] = $_iPointerID;
                 
                 // Add the pointer to $_aValidPointers array
-                $_aValidPointers[] =  $_aPointer;
+                $_aValidPointers[ ] = $_aPointer;
                 
             }            
             return $_aValidPointers;
@@ -279,9 +279,9 @@ 
          * @since       DEVVER
          * @return      string     
          */
-        public function _getInlineScript( $aPointers=array() ) {
+        public function _getInlineScript( $aPointers = array() ) {
 
-            $_aJSArray      = json_encode( $aPointers );
+            $_aJSArray = json_encode( $aPointers );
 
             /**
              * Checks checkboxes in siblings.

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;
                 }

Indentation +8 added lines, -8 removed lines

@@ -1,12 +1,12 @@ 
 <?php
 /**
- * Displays notification in the administration area.
- *    
- * @package      Admin Page Framework
- * @copyright    Copyright (c) 2015, <Michael Uno>
- * @author       Michael Uno
- * @authorurl    http://michaeluno.jp
- */
+     * Displays notification in the administration area.
+     *    
+     * @package      Admin Page Framework
+     * @copyright    Copyright (c) 2015, <Michael Uno>
+     * @author       Michael Uno
+     * @authorurl    http://michaeluno.jp
+     */
 
 /**
  * Displays pointer tool boxes in the admin area.
@@ -188,7 +188,7 @@ 
                     get_current_user_id(), 
                     'dismissed_wp_pointers', 
                     true 
-               )
+                )
             );
             $_aValidPointers = array();
          

development/utility/requirement/AdminPageFramework_Requirement.php

Doc Comments +3 added lines

@@ -134,6 +134,7 @@ 
          * Returns a php warning if present.
          * @since       3.5.3
          * @internal
+         * @param string $sType
          * @return      string      The warning.
          */
         private function _getWarningByType( $sType ) {
@@ -226,6 +227,7 @@ 
              * if it returns non true (false), it stores the subject warning and returns the array holding the warnings.
              * 
              * @since       3.4.6
+             * @param string $sFuncName
              * @return      array           The warning array.
              */
             private function _getWarningsByFunctionName( $sFuncName, $aSubjects ) {
@@ -285,6 +287,7 @@ 
     /**
      * Deactivates the plugin.
      * @since       3.4.6
+     * @param string $sPluginFilePath
      */
     public function deactivatePlugin( $sPluginFilePath, $sMessage='', $bIsOnActivation=false ) {
         

Indentation +1 added lines, -1 removed lines

@@ -278,7 +278,7 @@
                     ?  "<strong>" . $this->_sScriptName . "</strong>:&nbsp;" 
                     : '';            
                 return $_sScripTitle
-                   . implode( '<br />', $_aWarnings );
+                    . implode( '<br />', $_aWarnings );
                 
             }
         

Spacing +24 added lines, -24 removed lines

@@ -76,12 +76,12 @@ 
      * 
      * @since       3.4.6
      */ 
-    public function __construct( array $aRequirements=array(), $sScriptName='' ) {
+    public function __construct( array $aRequirements = array(), $sScriptName = '' ) {
         
         // Avoid undefined index warnings.
         $aRequirements          = $aRequirements + $this->_aDefaultRequirements;    
         $aRequirements          = array_filter( $aRequirements, 'is_array' );
-        foreach( array( 'php', 'mysql', 'wordpress' ) as $_iIndex => $_sName ) {
+        foreach ( array( 'php', 'mysql', 'wordpress' ) as $_iIndex => $_sName ) {
             if ( isset( $aRequirements[ $_sName ] ) ) {
                 $aRequirements[ $_sName ] = $aRequirements[ $_sName ] + $this->_aDefaultRequirements[ $_sName ];
             }
@@ -105,9 +105,9 @@ 
         $_aWarnings = array();
         
         // PHP, WordPress, MySQL
-        $_aWarnings[] = $this->_getWarningByType( 'php' );
-        $_aWarnings[] = $this->_getWarningByType( 'wordpress' );
-        $_aWarnings[] = $this->_getWarningByType( 'mysql' );
+        $_aWarnings[ ] = $this->_getWarningByType( 'php' );
+        $_aWarnings[ ] = $this->_getWarningByType( 'wordpress' );
+        $_aWarnings[ ] = $this->_getWarningByType( 'mysql' );
         
         // Ensure necessary array elements.
         $this->_aRequirements = $this->_aRequirements + array(
@@ -120,10 +120,10 @@ 
         // Check the rest.
         $_aWarnings = array_merge(
             $_aWarnings,
-            $this->_checkFunctions( $this->_aRequirements['functions'] ),
-            $this->_checkClasses( $this->_aRequirements['classes'] ),
-            $this->_checkConstants( $this->_aRequirements['constants'] ),
-            $this->_checkFiles( $this->_aRequirements['files'] )
+            $this->_checkFunctions( $this->_aRequirements[ 'functions' ] ),
+            $this->_checkClasses( $this->_aRequirements[ 'classes' ] ),
+            $this->_checkConstants( $this->_aRequirements[ 'constants' ] ),
+            $this->_checkFiles( $this->_aRequirements[ 'files' ] )
         );
         
         $this->aWarnings = array_filter( $_aWarnings ); // drop empty elements.
@@ -137,7 +137,7 @@ 
          * @return      string      The warning.
          */
         private function _getWarningByType( $sType ) {
-            if ( ! isset( $this->_aRequirements[ $sType ][ 'version' ] ) ) {
+            if ( !isset( $this->_aRequirements[ $sType ][ 'version' ] ) ) {
                 return '';
             }
             if ( $this->_checkPHPVersion( $this->_aRequirements[ $sType ][ 'version' ] ) ) {
@@ -164,7 +164,7 @@ 
          * @since       3.4.6
          */
         private function _checkWordPressVersion( $sWordPressVersion ) {
-            return version_compare( $GLOBALS['wp_version'], $sWordPressVersion, ">=" );
+            return version_compare( $GLOBALS[ 'wp_version' ], $sWordPressVersion, ">=" );
         }
         
         /**
@@ -230,9 +230,9 @@ 
              */
             private function _getWarningsByFunctionName( $sFuncName, $aSubjects ) {
                 $_aWarnings = array();
-                foreach( $aSubjects as $_sSubject => $_sWarning ) {
-                    if ( ! call_user_func_array( $sFuncName, array( $_sSubject ) ) ) {
-                        $_aWarnings[] = sprintf( $_sWarning, $_sSubject );
+                foreach ( $aSubjects as $_sSubject => $_sWarning ) {
+                    if ( !call_user_func_array( $sFuncName, array( $_sSubject ) ) ) {
+                        $_aWarnings[ ] = sprintf( $_sWarning, $_sSubject );
                     }
                 }
                 return $_aWarnings;                
@@ -252,7 +252,7 @@ 
          */    
         public function _replyToPrintAdminNotices() {
             
-            $_aWarnings     = array_unique( $this->aWarnings );
+            $_aWarnings = array_unique( $this->aWarnings );
             if ( empty( $_aWarnings ) ) {
                 return;
             }
@@ -270,12 +270,12 @@ 
              */
             private function _getWarnings() {
 
-                $_aWarnings     = array_unique( $this->aWarnings );            
+                $_aWarnings = array_unique( $this->aWarnings );            
                 if ( empty( $_aWarnings ) ) {
                     return '';
                 }        
-                $_sScripTitle   = $this->_sScriptName 
-                    ?  "<strong>" . $this->_sScriptName . "</strong>:&nbsp;" 
+                $_sScripTitle = $this->_sScriptName 
+                    ? "<strong>".$this->_sScriptName."</strong>:&nbsp;" 
                     : '';            
                 return $_sScripTitle
                    . implode( '<br />', $_aWarnings );
@@ -286,12 +286,12 @@ 
      * Deactivates the plugin.
      * @since       3.4.6
      */
-    public function deactivatePlugin( $sPluginFilePath, $sMessage='', $bIsOnActivation=false ) {
+    public function deactivatePlugin( $sPluginFilePath, $sMessage = '', $bIsOnActivation = false ) {
         
         add_action( 'admin_notices', array( $this, '_replyToPrintAdminNotices' ) );
-        $this->aWarnings[] = '<strong>' . $sMessage . '</strong>';
-        if ( ! function_exists( 'deactivate_plugins' ) ) {
-            if ( ! @include( ABSPATH . '/wp-admin/includes/plugin.php' ) ) {
+        $this->aWarnings[ ] = '<strong>'.$sMessage.'</strong>';
+        if ( !function_exists( 'deactivate_plugins' ) ) {
+            if ( !@include( ABSPATH.'/wp-admin/includes/plugin.php' ) ) {
                 return;
             }
         }
@@ -301,8 +301,8 @@ 
         // Before that, we can display messages to the user.
         if ( $bIsOnActivation ) {
             
-            $_sPluginListingPage = add_query_arg( array(), $GLOBALS['pagenow'] );
-            wp_die( $this->_getWarnings() . "<p><a href='$_sPluginListingPage'>Go back</a>.</p>" );
+            $_sPluginListingPage = add_query_arg( array(), $GLOBALS[ 'pagenow' ] );
+            wp_die( $this->_getWarnings()."<p><a href='$_sPluginListingPage'>Go back</a>.</p>" );
             
         }
         

Braces +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
      * @since       3.4.6
      * @return      integer         The number of warnings.
      */
-    public function check() {      
+    public function check() {
         
         $_aWarnings = array();
         

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(