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/AdminPageFramework_Property_admin_page.php

Doc Comments +5 added lines

@@ -281,6 +281,8 @@ 
      * @remark      Used by the setInPageTabsVisibility() method.
      * @since       2.0.0
      * @since       2.1.5   The $oCaller parameter was added.
+     * @param AdminPageFramework_Router $oCaller
+     * @param string $sClassName
      */ 
     public function __construct( $oCaller, $sCallerPath, $sClassName, $aisOptionKey, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
 
@@ -419,6 +421,7 @@ 
          * 
          * This is to support transient form data which disappears with the set timeout.
          * 
+         * @param string $sOptionType
          * @return      array       The retrieved options array.
          */ 
         private function _getOptionsByType( $sOptionType ) {
@@ -486,6 +489,7 @@ 
         /**
          * 
          * @since       3.5.9
+         * @param string $sOptionType
          * @return      boolean     True if saved; otherwise, false.
          */
         private function _updateOptionsByType( $aOptions, $sOptionType ) {
@@ -581,6 +585,7 @@ 
      * @since 2.1.5 Made it public and moved from the AdminPageFramework_Page class since this method is used by the AdminPageFramework_HeadTab class as well.
      * @internal
      * @remark Used in the __call() method in the main class.
+     * @param string $sPageSlug
      * @return string The default in-page tab slug if found; otherwise, an empty string.
      */         
     public function getDefaultInPageTab( $sPageSlug ) {

Spacing +15 added lines, -15 removed lines

@@ -218,7 +218,7 @@ 
      */
     public $aDisallowedQueryKeys = array( 
         'settings-updated', 
-        'confirmation',     // 3.3.0+
+        'confirmation', // 3.3.0+
         'field_errors'      // 3.4.1+
     );
         
@@ -264,7 +264,7 @@ 
      * Stores the cache lifetime of the transient used for the form options when the user passes an integer to the option key parameter.
      * @since       3.5.9
      */
-    public $iOptionTransientDuration  = 0; 
+    public $iOptionTransientDuration = 0; 
      
     /**
      * Constructs the instance of AdminPageFramework_Property_admin_page class object.
@@ -273,10 +273,10 @@ 
      * @since       2.0.0
      * @since       2.1.5   The $oCaller parameter was added.
      */ 
-    public function __construct( $oCaller, $sCallerPath, $sClassName, $aisOptionKey, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $oCaller, $sCallerPath, $sClassName, $aisOptionKey, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         // 3.7.0+ This must be set before the parent constructor. As the form arguments array uses this value.
-        $this->_sFormRegistrationHook = 'load_after_' . $sClassName;
+        $this->_sFormRegistrationHook = 'load_after_'.$sClassName;
 
         parent::__construct( 
             $oCaller, 
@@ -352,7 +352,7 @@ 
                 }
                 // The user want to save options in a transient.
                 if ( in_array( $_sType, array( 'integer' ) ) ) {
-                    return 'apf_' . $sClassName . '_' . get_current_user_id();
+                    return 'apf_'.$sClassName.'_'.get_current_user_id();
                 }
                 
                 // Unknown type - maybe the user is trying to do something advanced.
@@ -379,7 +379,7 @@ 
          */
         protected function _isAdminPage() {
             
-            if ( ! is_admin() ) {
+            if ( !is_admin() ) {
                 return false;
             }
             return isset( $_GET[ 'page' ] );
@@ -419,7 +419,7 @@ 
                 case 'transient':
                     return $this->getAsArray(
                         $this->getTransient( 
-                            $this->sOptionKey,  // transient key
+                            $this->sOptionKey, // transient key
                             array() // default
                         )
                     );
@@ -433,7 +433,7 @@ 
      * @since       3.1.1       Made it return a value.
      * @return      boolean     True if saved; otherwise, false.
      */
-    public function updateOption( $aOptions=null ) {
+    public function updateOption( $aOptions = null ) {
 
         if ( $this->_bDisableSavingOptions ) {
             return false;
@@ -461,7 +461,7 @@ 
                     );
                 case 'transient':
                     return $this->setTransient( 
-                        $this->sOptionKey,  // transient key
+                        $this->sOptionKey, // transient key
                         $aOptions,
                         $this->iOptionTransientDuration
                     );
@@ -476,7 +476,7 @@ 
      * @since       2.1.0       Set the default value to the parameter and if the parameter value is empty, it applies the current $_GET[ 'page' ] value.
      * @return      boolean     Returns true if it is of framework's added page; otherwise, false.
      */
-    public function isPageAdded( $sPageSlug='' ) {    
+    public function isPageAdded( $sPageSlug = '' ) {    
         
         $sPageSlug = trim( $sPageSlug );
         $sPageSlug = $sPageSlug 
@@ -495,7 +495,7 @@ 
      */
     public function getCurrentPageSlug() {
         return $this->getElement( 
-            $_GET,  // subject array
+            $_GET, // subject array
             'page', // key
             ''      // default
         );            
@@ -513,7 +513,7 @@ 
      * @return      string      The found tab slug. An empty string if not found.
      * @remark      Do not return `null` when not found as some framework methods check the returned value with `isset()` and if null is given, `isset()` yields `false` while it does `true` for an empty string ('').
      */    
-    public function getCurrentTabSlug( $sCurrentPageSlug='' ) {
+    public function getCurrentTabSlug( $sCurrentPageSlug = '' ) {
         
         // It is possible that the tab slug is not set if it is the default tab.
         $_sTabSlug = $this->getElement( $_GET, 'tab' );
@@ -533,7 +533,7 @@ 
          * 
          * @deprecated  3.5.3
          */
-        public function getCurrentTab( $sCurrentPageSlug='' ) {
+        public function getCurrentTab( $sCurrentPageSlug = '' ) {
             return $this->getCurrentTabSlug( $sCurrentPageSlug );            
         }
     
@@ -548,11 +548,11 @@ 
      */         
     public function getDefaultInPageTab( $sPageSlug ) {
     
-        if ( ! $sPageSlug ) { 
+        if ( !$sPageSlug ) { 
             return ''; 
         }
         return $this->getElement( 
-            $this->aDefaultInPageTabs,  // subject array
+            $this->aDefaultInPageTabs, // subject array
             $sPageSlug, // key
             ''    // default
         );

Braces +4 added lines, -4 removed lines

@@ -322,7 +322,7 @@ 
          * @since       3.5.9
          * @return      void
          */
-        private function _setOptionsProperties( $aisOptionKey, $sClassName ) {            
+        private function _setOptionsProperties( $aisOptionKey, $sClassName ) {
                         
             $_aArguments = is_array( $aisOptionKey ) 
                 ? $aisOptionKey
@@ -513,7 +513,7 @@ 
      * @since       2.1.0       Set the default value to the parameter and if the parameter value is empty, it applies the current $_GET[ 'page' ] value.
      * @return      boolean      Returns true if it is of framework's added page; otherwise, false.
      */
-    public function isPageAdded( $sPageSlug='' ) {    
+    public function isPageAdded( $sPageSlug='' ) {
         
         $sPageSlug = trim( $sPageSlug );
         $sPageSlug = $sPageSlug 
@@ -554,7 +554,7 @@ 
         
         // It is possible that the tab slug is not set if it is the default tab.
         $_sTabSlug = $this->getElement( $_GET, 'tab' );
-        if ( $_sTabSlug ) { 
+        if ( $_sTabSlug ) {
             return $_sTabSlug;
         }
         $sCurrentPageSlug = $sCurrentPageSlug
@@ -585,7 +585,7 @@ 
      */         
     public function getDefaultInPageTab( $sPageSlug ) {
     
-        if ( ! $sPageSlug ) { 
+        if ( ! $sPageSlug ) {
             return ''; 
         }
         return $this->getElement( 

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/admin_page/AdminPageFramework_Router.php

Doc Comments +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
      * 
      * @since       3.7.10
      * @internal
-     * @return      null|object
+     * @return      AdminPageFramework_Link_admin_page
      */
     protected function _getLinkObject() {
         return new AdminPageFramework_Link_admin_page( $this->oProp, $this->oMsg );
@@ -69,7 +69,7 @@ 
      * 
      * @since       3.7.10
      * @internal
-     * @return      null|object
+     * @return      AdminPageFramework_PageLoadInfo_admin_page
      */    
     protected function _getPageLoadObject() {
         return new AdminPageFramework_PageLoadInfo_admin_page( $this->oProp, $this->oMsg );

Spacing +14 added lines, -14 removed lines

@@ -27,7 +27,7 @@ 
      * 
      * @since       3.3.0
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sOptionKey = null, $sCallerPath = null, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         $this->oProp = isset( $this->oProp ) 
             ? $this->oProp // for the AdminPageFramework_NetworkAdmin class
@@ -45,7 +45,7 @@ 
         if ( $this->oProp->bIsAdminAjax ) {
             return;
         }     
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
         
@@ -89,7 +89,7 @@ 
      * @since       3.3.1       Moved from `AdminPageFramework_Base`.
      * @internal
      */
-    public function __call( $sMethodName, $aArgs=null ) {     
+    public function __call( $sMethodName, $aArgs = null ) {     
 
         $_sPageSlug             = $this->oProp->getCurrentPageSlug();
         $_sTabSlug              = $this->oProp->getCurrentTabSlug( $_sPageSlug );
@@ -100,7 +100,7 @@ 
             'load_pre_',
         );        
         
-        switch( $this->_getCallbackName( $sMethodName, $_sPageSlug, $_aKnownMethodPrefixes ) ) {
+        switch ( $this->_getCallbackName( $sMethodName, $_sPageSlug, $_aKnownMethodPrefixes ) ) {
                 
             // A callback of the call_page_{page slug} action hook
             // @deprecated  3.7.10
@@ -109,11 +109,11 @@ 
              
             // add_settings_section() callback 
             case 'section_pre_':
-                return $this->_renderSectionDescription( $sMethodName );    // defined in AdminPageFramework_Setting
+                return $this->_renderSectionDescription( $sMethodName ); // defined in AdminPageFramework_Setting
                 
             // add_settings_field() callback - 
             case 'field_pre_':
-                return $this->_renderSettingField( $_mFirstArg, $_sPageSlug );  // defined in AdminPageFramework_Setting
+                return $this->_renderSettingField( $_mFirstArg, $_sPageSlug ); // defined in AdminPageFramework_Setting
             
             // load-{page} callback            
             case 'load_pre_':
@@ -130,7 +130,7 @@ 
          * @since       3.5.3
          * @return      string      The found callback method name or the prefix of a known callback method name. An empty string if not found.
          */
-        private function _getCallbackName( $sMethodName, $sPageSlug, array $aKnownMethodPrefixes=array() ) {
+        private function _getCallbackName( $sMethodName, $sPageSlug, array $aKnownMethodPrefixes = array() ) {
             
             // @deprecated      3.7.10
             // if ( in_array( 
@@ -143,7 +143,7 @@ 
                 // return $sMethodName;
             // }
             
-            foreach( $aKnownMethodPrefixes as $_sMethodPrefix ) {
+            foreach ( $aKnownMethodPrefixes as $_sMethodPrefix ) {
                 if ( $this->oUtil->hasPrefix( $_sMethodPrefix, $sMethodName ) ) {
                     return $_sMethodPrefix;
                 }
@@ -167,7 +167,7 @@ 
          */ 
         protected function _doPageLoadCall( $sMethodName, $sPageSlug, $sTabSlug, $oScreen ) {
             
-            if ( ! $this->_isPageLoadCall( $sMethodName, $sPageSlug, $oScreen->id ) ) {
+            if ( !$this->_isPageLoadCall( $sMethodName, $sPageSlug, $oScreen->id ) ) {
                 return;
             }
 
@@ -189,7 +189,7 @@ 
             
             $this->oUtil->addAndDoActions( 
                 $this, // the caller object
-                array( "load_{$sPageSlug}_" . $this->oProp->getCurrentTabSlug( $sPageSlug ) ),
+                array( "load_{$sPageSlug}_".$this->oProp->getCurrentTabSlug( $sPageSlug ) ),
                 $this // the admin page object - this lets third-party scripts use the framework methods.
             );         
             
@@ -217,7 +217,7 @@ 
                 if ( substr( $sMethodName, strlen( 'load_pre_' ) ) !== $sPageSlug ) {
                     return false;
                 }
-                if ( ! isset( $this->oProp->aPageHooks[ $sPageSlug ] ) ) {
+                if ( !isset( $this->oProp->aPageHooks[ $sPageSlug ] ) ) {
                     return false;
                 }
                 if ( $sScreenID !== $this->oProp->aPageHooks[ $sPageSlug ] ) {
@@ -244,7 +244,7 @@ 
         }
         
         // Nothing to do in the network admin area.
-        return ! is_network_admin();
+        return !is_network_admin();
         
     }
     
@@ -259,11 +259,11 @@ 
     public function _isInThePage() {
         
         // If the setUp method is not loaded yet,
-        if ( ! did_action( 'set_up_' . $this->oProp->sClassName ) ) {
+        if ( !did_action( 'set_up_'.$this->oProp->sClassName ) ) {
             return true;
         }    
 
-        if ( ! isset( $_GET[ 'page' ] ) ) { 
+        if ( !isset( $_GET[ 'page' ] ) ) { 
             return false; 
         }        
         

Braces +2 added lines, -2 removed lines

@@ -89,7 +89,7 @@ 
      * @since       3.3.1       Moved from `AdminPageFramework_Base`.
      * @internal
      */
-    public function __call( $sMethodName, $aArgs=null ) {     
+    public function __call( $sMethodName, $aArgs=null ) {
 
         $_sPageSlug             = $this->oProp->getCurrentPageSlug();
         $_sTabSlug              = $this->oProp->getCurrentTabSlug( $_sPageSlug );
@@ -263,7 +263,7 @@ 
             return true;
         }    
 
-        if ( ! isset( $_GET[ 'page' ] ) ) { 
+        if ( ! isset( $_GET[ 'page' ] ) ) {
             return false; 
         }        
         

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 ) {
                 

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

Indentation +7 added lines, -7 removed lines

@@ -1,11 +1,11 @@ 
 <?php
 /**
- * Admin Page Framework
- * 
- * http://en.michaeluno.jp/admin-page-framework/
- * Copyright (c) 2013-2016 Michael Uno; Licensed MIT
- * 
- */
+     * Admin Page Framework
+     * 
+     * http://en.michaeluno.jp/admin-page-framework/
+     * Copyright (c) 2013-2016 Michael Uno; Licensed MIT
+     * 
+     */
 
 /**
  * Sends email as a part of form validation.
@@ -168,7 +168,7 @@ 
                 return '';
                 
             }
-       /**
+        /**
          * Sets the mail content type to HTML.
          * @since       3.3.0
          * @since       3.4.2       Moved from the validation class.

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/factory/taxonomy_field/AdminPageFramework_TaxonomyField_Router.php

Doc Comments +1 added lines

@@ -23,6 +23,7 @@
      * Sets up hooks.
      * 
      * @since       3.5.0
+     * @param AdminPageFramework_Property_taxonomy_field $oProp
      */
     public function __construct( $oProp ) {
                 

Spacing +4 added lines, -4 removed lines

@@ -28,7 +28,7 @@ 
                 
         parent::__construct( $oProp );
 
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
         
@@ -39,7 +39,7 @@ 
         );
         
         add_action( 
-            'set_up_' . $this->oProp->sClassName,
+            'set_up_'.$this->oProp->sClassName,
             array( $this, '_replyToSetUpHooks' )
         );
         
@@ -63,7 +63,7 @@ 
             return false; 
         }
         
-        if ( isset( $_GET['taxonomy'] ) && ! in_array( $_GET['taxonomy'], $this->oProp->aTaxonomySlugs ) ) {
+        if ( isset( $_GET[ 'taxonomy' ] ) && !in_array( $_GET[ 'taxonomy' ], $this->oProp->aTaxonomySlugs ) ) {
             return false;
         }        
         
@@ -80,7 +80,7 @@ 
      */
     public function _replyToSetUpHooks( $oFactory ) {
         
-        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {     
+        foreach ( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {     
             
             // Validation callbacks need to be set regardless of whether the current page is edit-tags.php or not.
             add_action( "created_{$_sTaxonomySlug}", array( $this, '_replyToValidateOptions' ), 10, 2 );

Braces +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
             return true;
         }    
         
-        if ( 'edit-tags.php' != $this->oProp->sPageNow ) { 
+        if ( 'edit-tags.php' != $this->oProp->sPageNow ) {
             return false; 
         }
         
@@ -80,7 +80,7 @@ 
      */
     public function _replyToSetUpHooks( $oFactory ) {
         
-        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {     
+        foreach( $this->oProp->aTaxonomySlugs as $_sTaxonomySlug ) {
             
             // Validation callbacks need to be set regardless of whether the current page is edit-tags.php or not.
             add_action( "created_{$_sTaxonomySlug}", array( $this, '_replyToValidateOptions' ), 10, 2 );

development/factory/user_meta/AdminPageFramework_UserMeta_Router.php

Doc Comments +1 added lines

@@ -22,6 +22,7 @@
      * Sets up hooks.
      * 
      * @since       3.5.0
+     * @param AdminPageFramework_Property_user_meta $oProp
      */
     public function __construct( $oProp ) {
         

Spacing +8 added lines, -8 removed lines

@@ -27,7 +27,7 @@ 
         
         parent::__construct( $oProp );
         
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
         
@@ -37,7 +37,7 @@ 
         );
         
         // 3.7.10+
-        add_action( 'set_up_' . $this->oProp->sClassName, array( $this, '_replyToSetUpHooks' ) );
+        add_action( 'set_up_'.$this->oProp->sClassName, array( $this, '_replyToSetUpHooks' ) );
                         
     }
     
@@ -50,7 +50,7 @@ 
      */
     public function _isInThePage() {
                
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return false;
         }
 
@@ -71,14 +71,14 @@ 
     public function _replyToSetUpHooks( $oFactory ) {
         
         // Hooks to display fields.
-        add_action( 'show_user_profile', array( $this, '_replyToPrintFields' ) );   // profile.php
-        add_action( 'edit_user_profile', array( $this, '_replyToPrintFields' ) );   // profile.php
-        add_action( 'user_new_form', array( $this, '_replyToPrintFields' ) );   // user-new.php
+        add_action( 'show_user_profile', array( $this, '_replyToPrintFields' ) ); // profile.php
+        add_action( 'edit_user_profile', array( $this, '_replyToPrintFields' ) ); // profile.php
+        add_action( 'user_new_form', array( $this, '_replyToPrintFields' ) ); // user-new.php
         
         // Hooks to save field values.
         add_action( 'personal_options_update', array( $this, '_replyToSaveFieldValues' ) ); // profile.php
-        add_action( 'edit_user_profile_update', array( $this, '_replyToSaveFieldValues' ) );    // profile.php
-        add_action('user_register', array( $this, '_replyToSaveFieldValues' ) );    // user-new.php
+        add_action( 'edit_user_profile_update', array( $this, '_replyToSaveFieldValues' ) ); // profile.php
+        add_action( 'user_register', array( $this, '_replyToSaveFieldValues' ) ); // user-new.php
                        
     }