michaeluno / admin-page-framework

development/factory/_common/utility/base_utility/AdminPageFramework_Utility_URL.php

Doc Comments +1 added lines

@@ -22,6 +22,7 @@
      *
      * @since       2.0.0
      * @since       3.5.7       Moved from `AdminPageFramework_Utility`.
+     * @param string $sQueryKey
      * @return      string|null
      */
     static public function getQueryValueInURLByKey( $sURL, $sQueryKey ) {

Spacing +5 added lines, -5 removed lines

@@ -52,7 +52,7 @@ 
             0 => '',
             1 => 's',
         );
-        $_sProtocol         = substr( $_sServerProtocol, 0, strpos( $_sServerProtocol, '/' ) )
+        $_sProtocol = substr( $_sServerProtocol, 0, strpos( $_sServerProtocol, '/' ) )
             . $_aProtocolSuffix[ ( int ) $_bSSL ];
 
         // Port: e.g. :80
@@ -65,8 +65,8 @@ 
                 ? $_SERVER[ 'HTTP_HOST' ]
                 : $_SERVER[ 'SERVER_NAME' ]
             );
-        $_sHost             = preg_replace( '/:.+/', '', $_sHost ); // remove the port part in case it is added.
-        return $_sProtocol . '://' . $_sHost . $_sPort . $_SERVER[ 'REQUEST_URI' ];
+        $_sHost = preg_replace( '/:.+/', '', $_sHost ); // remove the port part in case it is added.
+        return $_sProtocol.'://'.$_sHost.$_sPort.$_SERVER[ 'REQUEST_URI' ];
 
     }
         /**
@@ -80,10 +80,10 @@ 
                 ? ( string ) $_SERVER[ 'SERVER_PORT' ]
                 : '';
             $_aPort     = array(
-                0 => ':' . $_sPort,
+                0 => ':'.$_sPort,
                 1 => '',
             );
-            $_bPortSet  = ( ! $bSSL && '80' === $_sPort ) || ( $bSSL && '443' === $_sPort );
+            $_bPortSet = ( !$bSSL && '80' === $_sPort ) || ( $bSSL && '443' === $_sPort );
             return $_aPort[ ( int ) $_bPortSet ];
         }
 

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

Doc Comments +1 added lines, -1 removed lines

@@ -332,7 +332,7 @@
          * Retrieves the saved option value from the options table
          * with the given option key, field ID, and section ID by giving a function name.
          *
-         * @param $sOptionKey
+         * @param string $sOptionKey
          * @param null $asKey
          * @param null $vDefault
          * @param array $aAdditionalOptions

Spacing +26 added lines, -26 removed lines

@@ -36,7 +36,7 @@ 
      * @param   array|string    $asPrefixes
      * @retuen  void
      */
-    public static function cleanTransients( $asPrefixes=array( 'apf' ) ) {
+    public static function cleanTransients( $asPrefixes = array( 'apf' ) ) {
 
         $_aPrefixes = self::getAsArray( $asPrefixes );
         if ( self::isNetworkAdmin() ) {
@@ -46,11 +46,11 @@ 
         /**
          * @var wpdb
          */
-        $_oWPDB     = $GLOBALS[ 'wpdb' ];
-        foreach( $_aPrefixes as $_sPrefix ) {
+        $_oWPDB = $GLOBALS[ 'wpdb' ];
+        foreach ( $_aPrefixes as $_sPrefix ) {
             $_sSQLQuery = "DELETE FROM `{$_oWPDB->options}` "
                 . "WHERE `option_name` "
-                . "LIKE ( '_transient_%{$_sPrefix}%' )";    // this also matches _transient_timeout_{prefix}
+                . "LIKE ( '_transient_%{$_sPrefix}%' )"; // this also matches _transient_timeout_{prefix}
             $_oWPDB->query( $_sSQLQuery );
         }
 
@@ -66,8 +66,8 @@ 
         /**
          * @var wpdb
          */
-        $_oWPDB     = $GLOBALS[ 'wpdb' ];
-        foreach( $_aPrefixes as $_sPrefix ) {
+        $_oWPDB = $GLOBALS[ 'wpdb' ];
+        foreach ( $_aPrefixes as $_sPrefix ) {
             $_sSQLQuery = "DELETE FROM `{$_oWPDB->sitemeta}` "
                 . "WHERE "
                 // this matches _site_transient_timeout_{...} as well
@@ -82,7 +82,7 @@ 
      * @return array
      * @since  3.8.23
      */
-    static public function getTransientAsArray( $sTransientKey, $mDefault=null ) {
+    static public function getTransientAsArray( $sTransientKey, $mDefault = null ) {
         return self::getAsArray(
             self::getTransient( $sTransientKey, $mDefault )
         );
@@ -94,7 +94,7 @@ 
      * @return array
      * @since  3.8.23
      */
-    static public function getTransientWithoutCacheAsArray( $sTransientKey, $mDefault=null ) {
+    static public function getTransientWithoutCacheAsArray( $sTransientKey, $mDefault = null ) {
         return self::getAsArray(
             self::getTransientWithoutCache( $sTransientKey, $mDefault )
         );
@@ -111,9 +111,9 @@ 
      * @sicne   3.8.23
      * @return  mixed|false `false` on failing to retrieve the transient value.
      */
-    static public function getTransientWithoutCache( $sTransientKey, $mDefault=null ) {
+    static public function getTransientWithoutCache( $sTransientKey, $mDefault = null ) {
 
-        $sTransientKey  = self::_getCompatibleTransientKey( $sTransientKey );
+        $sTransientKey = self::_getCompatibleTransientKey( $sTransientKey );
         if ( self::isNetworkAdmin() ) {
             return self::getTransientWithoutCacheForNetwork( $sTransientKey, $mDefault );
         }
@@ -131,8 +131,8 @@ 
         $_mData = $_oWPDB->get_var(
             $_oWPDB->prepare(
                 $_sSQLQuery,
-                '_transient_' . $sTransientKey,
-                '_transient_timeout_' . $sTransientKey
+                '_transient_'.$sTransientKey,
+                '_transient_timeout_'.$sTransientKey
             )
         );
         return is_null( $_mData ) ? $mDefault : maybe_unserialize( $_mData );
@@ -145,7 +145,7 @@ 
      * @since   3.8.23
      * @return  mixed
      */
-    static public function getTransientWithoutCacheForNetwork($sTransientKey, $mDefault ) {
+    static public function getTransientWithoutCacheForNetwork( $sTransientKey, $mDefault ) {
 
         /**
          * @var wpdb $_oWPDB
@@ -161,8 +161,8 @@ 
         $_mData = $_oWPDB->get_var(
             $_oWPDB->prepare(
                 $_sSQLQuery,
-                '_site_transient_' . $sTransientKey,
-                '_site_transient_timeout_' . $sTransientKey,
+                '_site_transient_'.$sTransientKey,
+                '_site_transient_timeout_'.$sTransientKey,
                 get_current_network_id()
             )
         );
@@ -195,7 +195,7 @@ 
             0 => 'delete_transient',
             1 => 'delete_site_transient',
         );
-        $_vTransient     = $_aFunctionNames[ ( integer ) self::isNetworkAdmin() ]( $sTransientKey );
+        $_vTransient = $_aFunctionNames[ ( integer ) self::isNetworkAdmin() ]( $sTransientKey );
 
         // reset prior value of $_wp_using_ext_object_cache
         $_wp_using_ext_object_cache = $_bWpUsingExtObjectCacheTemp;
@@ -211,7 +211,7 @@ 
      * @param   mixed   $vDefault
      * @return  mixed
      */
-    static public function getTransient( $sTransientKey, $vDefault=null ) {
+    static public function getTransient( $sTransientKey, $vDefault = null ) {
 
         // Temporarily disable $_wp_using_ext_object_cache
         global $_wp_using_ext_object_cache;
@@ -223,7 +223,7 @@ 
             0 => 'get_transient',
             1 => 'get_site_transient',
         );
-        $_vTransient     = $_aFunctionNames[ ( integer ) self::isNetworkAdmin() ]( $sTransientKey );
+        $_vTransient = $_aFunctionNames[ ( integer ) self::isNetworkAdmin() ]( $sTransientKey );
 
         // Restore the prior value of `$_wp_using_ext_object_cache`.
         $_wp_using_ext_object_cache = $_bWpUsingExtObjectCacheTemp;
@@ -245,7 +245,7 @@ 
      * @param       mixed       $vValue
      * @param       integer     $iExpiration
      */
-    static public function setTransient( $sTransientKey, $vValue, $iExpiration=0 ) {
+    static public function setTransient( $sTransientKey, $vValue, $iExpiration = 0 ) {
 
         // Temporarily disable `$_wp_using_ext_object_cache`.
         global $_wp_using_ext_object_cache;
@@ -276,7 +276,7 @@ 
          * @return      string
          * @todo it is said as of WordPress 4.3, it will be 255 since the database table column type becomes VARCHAR(255).
          */
-        static public function _getCompatibleTransientKey( $sSubject, $iDeprecated=null ) {
+        static public function _getCompatibleTransientKey( $sSubject, $iDeprecated = null ) {
 
             $_iAllowedCharacterLength = isset( $iDeprecated )
                 ? $iDeprecated
@@ -294,7 +294,7 @@ 
                 0, // start position
                 $_iPrefixLengthToKeep - 1 // how many characters to extract
             );
-            return $_sPrefixToKeep . '_' . md5( $sSubject );
+            return $_sPrefixToKeep.'_'.md5( $sSubject );
 
         }
 
@@ -309,7 +309,7 @@ 
      * @param   array           $aAdditionalOptions     an additional options array to merge with the options array.
      * @return  mixed
      */
-    static public function getOption( $sOptionKey, $asKey=null, $vDefault=null, array $aAdditionalOptions=array() ) {
+    static public function getOption( $sOptionKey, $asKey = null, $vDefault = null, array $aAdditionalOptions = array() ) {
         return self::_getOptionByFunctionName( $sOptionKey, $asKey, $vDefault, $aAdditionalOptions );
     }
     /**
@@ -324,7 +324,7 @@ 
      * @remark  Used in the network admin area.
      * @return  mixed
      */
-    static public function getSiteOption( $sOptionKey, $asKey=null, $vDefault=null, array $aAdditionalOptions=array() ) {
+    static public function getSiteOption( $sOptionKey, $asKey = null, $vDefault = null, array $aAdditionalOptions = array() ) {
         return self::_getOptionByFunctionName( $sOptionKey, $asKey, $vDefault, $aAdditionalOptions, 'get_site_option' );
     }
 
@@ -340,16 +340,16 @@ 
          * @return      mixed
          * @since       3.5.3
          */
-        static private function _getOptionByFunctionName( $sOptionKey, $asKey=null, $vDefault=null, array $aAdditionalOptions=array(), $sFunctionName='get_option' ) {
+        static private function _getOptionByFunctionName( $sOptionKey, $asKey = null, $vDefault = null, array $aAdditionalOptions = array(), $sFunctionName = 'get_option' ) {
 
             // Entire options
-            if ( ! isset( $asKey ) ) {
+            if ( !isset( $asKey ) ) {
                 $_aOptions = $sFunctionName(
                     $sOptionKey,
                     isset( $vDefault )
                         ? $vDefault
                         : array()
-                );;
+                ); ;
                 return empty( $aAdditionalOptions )
                     ? $_aOptions
                     : self::uniteArrays(

development/factory/page_meta_box/AdminPageFramework_PageMetaBox.php

Spacing +4 added lines, -4 removed lines

@@ -76,21 +76,21 @@
      * @param       string          $sPriority      The priority, either `high`, `core`, `default` or `low`.
      * @param       string          $sCapability    The capability. See <a href="https://codex.wordpress.org/Roles_and_Capabilities" target="_blank">Roles and Capabilities</a>.
      */
-    public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs=array(), $sContext='normal', $sPriority='default', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sMetaBoxID, $sTitle, $asPageSlugs = array(), $sContext = 'normal', $sPriority = 'default', $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         if ( empty( $asPageSlugs ) ) {
             return;
         }
 
-        if ( ! $this->_isInstantiatable() ) {
+        if ( !$this->_isInstantiatable() ) {
             return;
         }
 
         // The property object needs to be done first before the parent constructor.
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
-        $this->oProp             = new $_sPropertyClassName(
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
+        $this->oProp = new $_sPropertyClassName(
             $this,
             get_class( $this ),
             $sCapability,

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
      * @param       string          The text domain. Default: `admin-page-framework`.
      * @return      void
      */
-    function __construct( $asTaxonomySlug, $sOptionKey='', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    function __construct( $asTaxonomySlug, $sOptionKey = '', $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         if ( empty( $asTaxonomySlug ) ) {
             return;
@@ -54,8 +54,8 @@ 
         // Properties
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
-        $this->oProp        = new $_sPropertyClassName(
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
+        $this->oProp = new $_sPropertyClassName(
             $this,
             get_class( $this ),
             $sCapability,

development/factory/user_meta/AdminPageFramework_UserMeta.php

Spacing +6 added lines, -6 removed lines

@@ -38,16 +38,16 @@
      * @since       3.7.4       Changed the default capability value to `read`.
      * @todo        Examine the appropriate default capability level.
      */
-    public function __construct( $sCapability='read', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sCapability = 'read', $sTextDomain = 'admin-page-framework' ) {
 
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
         $this->oProp = new $_sPropertyClassName(
-            $this,                  // the caller object
-            get_class( $this ),     // the caller class name
-            $sCapability,           // the capability level
-            $sTextDomain,           // the text domain
+            $this, // the caller object
+            get_class( $this ), // the caller class name
+            $sCapability, // the capability level
+            $sTextDomain, // the text domain
             $this->_sStructureType  // the structure type
         );
 

development/factory/admin_page/AdminPageFramework_Router.php

Spacing +21 added lines, -21 removed lines

@@ -26,11 +26,11 @@ 
      *
      * @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' ) {
 
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
 
         $this->oProp = new $_sPropertyClassName(
             $this,
@@ -47,12 +47,12 @@ 
 //        if ( $this->oProp->bIsAdminAjax ) {
 //            return;
 //        }
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
 
         add_action( 'wp_loaded', array( $this, '_replyToDetermineToLoad' ) );
-        add_action( 'set_up_' . $this->oProp->sClassName, array( $this, '_replyToLoadComponentsForAjax' ), 100 );
+        add_action( 'set_up_'.$this->oProp->sClassName, array( $this, '_replyToLoadComponentsForAjax' ), 100 );
 
     }
 
@@ -64,13 +64,13 @@ 
      * @callback        add_action      set_up_{extended class name}
      */
     public function _replyToLoadComponentsForAjax() {
-        if ( ! $this->oProp->bIsAdminAjax ) {
+        if ( !$this->oProp->bIsAdminAjax ) {
             return;
         }
 
         new AdminPageFramework_Model_Menu__RegisterMenu( $this, 'pseudo_admin_menu' );
         do_action( 'pseudo_admin_menu', '' );
-        do_action( 'pseudo_current_screen' );       // @deprecated 3.8.22 Kept for backward-compatibility
+        do_action( 'pseudo_current_screen' ); // @deprecated 3.8.22 Kept for backward-compatibility
 
         $_sPageSlug = $this->oProp->getCurrentPageSlug();
         if ( $this->oProp->isPageAdded( $_sPageSlug ) ) {
@@ -117,7 +117,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 );
@@ -128,15 +128,15 @@ 
             'load_pre_',
         );
 
-        switch( $this->_getCallbackName( $sMethodName, $_sPageSlug, $_aKnownMethodPrefixes ) ) {
+        switch ( $this->_getCallbackName( $sMethodName, $_sPageSlug, $_aKnownMethodPrefixes ) ) {
 
             // 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_':
@@ -154,9 +154,9 @@ 
          * @return      string      The found callback method name or the prefix of a known callback method name. An empty string if not found.
          * @internal
          */
-        private function _getCallbackName( $sMethodName, $sPageSlug, array $aKnownMethodPrefixes=array() ) {
+        private function _getCallbackName( $sMethodName, $sPageSlug, array $aKnownMethodPrefixes = array() ) {
 
-            foreach( $aKnownMethodPrefixes as $_sMethodPrefix ) {
+            foreach ( $aKnownMethodPrefixes as $_sMethodPrefix ) {
                 if ( $this->oUtil->hasPrefix( $_sMethodPrefix, $sMethodName ) ) {
                     return $_sMethodPrefix;
                 }
@@ -180,7 +180,7 @@ 
          */
         protected function _doPageLoadCall( $sMethodName, $sPageSlug, $sTabSlug, $oScreen ) {
 
-            if ( ! $this->_isPageLoadCall( $sMethodName, $sPageSlug, $oScreen ) ) {
+            if ( !$this->_isPageLoadCall( $sMethodName, $sPageSlug, $oScreen ) ) {
                 return;
             }
 
@@ -204,10 +204,10 @@ 
             // the tab slug will be empty unless an in-page tab is added.
             $sTabSlug = $this->oProp->getCurrentTabSlug( $sPageSlug );
             if ( strlen( $sTabSlug ) ) {
-                $this->_setShowDebugInfoProperty( $sPageSlug, $sTabSlug );  // 3.8.8+
+                $this->_setShowDebugInfoProperty( $sPageSlug, $sTabSlug ); // 3.8.8+
                 $this->oUtil->addAndDoActions(
                     $this, // the caller object
-                    array( "load_{$sPageSlug}_" . $sTabSlug ),
+                    array( "load_{$sPageSlug}_".$sTabSlug ),
                     $this // the admin page object - this lets third-party scripts use the framework methods.
                 );
                 add_filter( 'admin_title', array( $this, '_replyToSetAdminPageTitleForTab' ), 1, 2 );
@@ -233,10 +233,10 @@ 
              * @since       3.8.8
              * @return      void
              */
-            private function _setShowDebugInfoProperty( $sPageSlug, $sTabSlug='' ) {
+            private function _setShowDebugInfoProperty( $sPageSlug, $sTabSlug = '' ) {
 
                 // For the page,
-                if ( ! strlen( $sTabSlug ) ) {
+                if ( !strlen( $sTabSlug ) ) {
                     $this->oProp->bShowDebugInfo = $this->oUtil->getElement(
                         $this->oProp->aPages,
                         array( $sPageSlug, 'show_debug_info' ),
@@ -281,7 +281,7 @@ 
                 if ( substr( $sMethodName, strlen( 'load_pre_' ) ) !== $sPageSlug ) {
                     return false;
                 }
-                if ( ! isset( $this->oProp->aPageHooks[ $sPageSlug ] ) ) {
+                if ( !isset( $this->oProp->aPageHooks[ $sPageSlug ] ) ) {
                     return false;
                 }
                 $_sPageHook = is_object( $osScreenORPageHook )
@@ -307,7 +307,7 @@ 
         }
 
         // Nothing to do in the network admin area.
-        return ! is_network_admin();
+        return !is_network_admin();
 
     }
 
@@ -322,13 +322,13 @@ 
      */
     protected function _isInThePage() {
 
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return false;
         }
 
         // If the `setUp()` method is not loaded yet, nothing can be checked
         // as there is not page is added.
-        if ( ! did_action( 'set_up_' . $this->oProp->sClassName ) ) {
+        if ( !did_action( 'set_up_'.$this->oProp->sClassName ) ) {
             return true;
         }
 

development/factory/admin_page/AdminPageFramework_View_Page.php

Spacing +3 added lines, -3 removed lines

@@ -32,10 +32,10 @@ 
                 $this->oProp->aInPageTabs,
                 array( $this->oProp->getCurrentPageSlug(), $this->oProp->getCurrentTabSlug(), 'title' )
             );
-        if ( ! $_sTabTitle ) {
+        if ( !$_sTabTitle ) {
             return $sAdminTitle;
         }
-        return $_sTabTitle . ' ‹ ' . $sAdminTitle;
+        return $_sTabTitle.' ‹ '.$sAdminTitle;
     }
 
     /**
@@ -82,7 +82,7 @@ 
      * @return      void
      * @internal
      */
-    protected function _renderPage( $sPageSlug, $sTabSlug=null ) {
+    protected function _renderPage( $sPageSlug, $sTabSlug = null ) {
         $_oPageRenderer = new AdminPageFramework_View__PageRenderer( $this, $sPageSlug, $sTabSlug );
         $_oPageRenderer->render();
     }

development/factory/meta_box/AdminPageFramework_MetaBox.php

Spacing +5 added lines, -5 removed lines

@@ -53,9 +53,9 @@ 
      * @param       string            $sTextDomain              (optional) The text domain applied to the displayed text messages. Default: `admin-page-framework`.
      * @return      void
      */
-    public function __construct( $sMetaBoxID, $sTitle, $asPostTypeOrScreenID=array( 'post' ), $sContext='normal', $sPriority='default', $sCapability='edit_posts', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sMetaBoxID, $sTitle, $asPostTypeOrScreenID = array( 'post' ), $sContext = 'normal', $sPriority = 'default', $sCapability = 'edit_posts', $sTextDomain = 'admin-page-framework' ) {
 
-        if ( ! $this->_isInstantiatable() ) {
+        if ( !$this->_isInstantiatable() ) {
             return;
         }
         if ( empty( $asPostTypeOrScreenID ) ) {
@@ -65,15 +65,15 @@ 
         // A property object needs to be done first.
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
-        $this->oProp                = new $_sPropertyClassName(
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
+        $this->oProp = new $_sPropertyClassName(
             $this,
             get_class( $this ),
             $sCapability,
             $sTextDomain,
             $this->_sStructureType
         );
-        $this->oProp->aPostTypes    = is_string( $asPostTypeOrScreenID )
+        $this->oProp->aPostTypes = is_string( $asPostTypeOrScreenID )
             ? array( $asPostTypeOrScreenID )
             : $asPostTypeOrScreenID;
 

development/factory/post_type/AdminPageFramework_PostType.php

Spacing +7 added lines, -7 removed lines

@@ -103,7 +103,7 @@ 
     * @param        string      The text domain of the caller script.
     * @return       void
     */
-    public function __construct( $sPostType, $aArguments=array(), $sCallerPath=null, $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sPostType, $aArguments = array(), $sCallerPath = null, $sTextDomain = 'admin-page-framework' ) {
 
         if ( empty( $sPostType ) ) {
             return;
@@ -111,16 +111,16 @@ 
 
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
         $this->oProp = new $_sPropertyClassName(
             $this,
             $this->_getCallerScriptPath( $sCallerPath ),
-            get_class( $this ),     // class name
-            'publish_posts',        // capability
-            $sTextDomain,           // text domain
+            get_class( $this ), // class name
+            'publish_posts', // capability
+            $sTextDomain, // text domain
             $this->_sStructureType  // structure type
         );
-        $this->oProp->sPostType     = AdminPageFramework_WPUtility::sanitizeSlug( $sPostType );
+        $this->oProp->sPostType = AdminPageFramework_WPUtility::sanitizeSlug( $sPostType );
 
         // Post type argument array structure
         // @see http://codex.wordpress.org/Function_Reference/register_post_type#Arguments
@@ -143,7 +143,7 @@ 
                 return $sCallerPath;
             }
 
-            if ( ! is_admin() ) {
+            if ( !is_admin() ) {
                 return null;
             }
             $_sPageNow = AdminPageFramework_Utility::getElement( $GLOBALS, 'pagenow' );