michaeluno / admin-page-framework

development/factory/page_meta_box/AdminPageFramework_PageMetaBox_Controller.php

Spacing +4 added lines, -4 removed lines

@@ -21,7 +21,7 @@ 
      *
      * @since 3.0.0
      */
-    public function enqueueStyles( $aSRCs, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueStyles( $aSRCs, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueStyles' ) ) {
             return $this->oResource->_enqueueStyles( $aSRCs, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -37,7 +37,7 @@ 
      * @param array (optional) The argument array for more advanced parameters.
      * @return string The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      */
-    public function enqueueStyle( $sSRC, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueStyle( $sSRC, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueStyle' ) ) {
             return $this->oResource->_enqueueStyle( $sSRC, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -47,7 +47,7 @@ 
      *
      * @since 2.1.5
      */
-    public function enqueueScripts( $aSRCs, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueScripts( $aSRCs, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueScripts' ) ) {
             return $this->oResource->_enqueueScripts( $aSRCs, $sPageSlug, $sTabSlug, $aCustomArgs );
         }
@@ -63,7 +63,7 @@ 
      * @param             array (optional) The argument array for more advanced parameters.
      * @return string The script handle ID. If the passed url is not a valid url string, an empty string will be returned.
      */
-    public function enqueueScript( $sSRC, $sPageSlug='', $sTabSlug='', $aCustomArgs=array() ) {
+    public function enqueueScript( $sSRC, $sPageSlug = '', $sTabSlug = '', $aCustomArgs = array() ) {
         if ( method_exists( $this->oResource, '_enqueueScript' ) ) {
             return $this->oResource->_enqueueScript( $sSRC, $sPageSlug, $sTabSlug, $aCustomArgs );
         }

development/factory/admin_page/_controller/AdminPageFramework_Link_admin_page.php

Spacing +28 added lines, -28 removed lines

@@ -36,7 +36,7 @@ 
      */
     public function _replyToSetFooterInfo() {
 
-        if ( ! $this->oProp->isPageAdded() ) { 
+        if ( !$this->oProp->isPageAdded() ) { 
             return; 
         }
         parent::_replyToSetFooterInfo();
@@ -56,8 +56,8 @@ 
      */
     public function _addLinkToPluginDescription( $asLinks ) {
         
-        if ( ! is_array( $asLinks ) ) {
-            $this->oProp->aPluginDescriptionLinks[] = $asLinks;
+        if ( !is_array( $asLinks ) ) {
+            $this->oProp->aPluginDescriptionLinks[ ] = $asLinks;
         } else {
             $this->oProp->aPluginDescriptionLinks = array_merge( 
                 $this->oProp->aPluginDescriptionLinks, 
@@ -84,8 +84,8 @@ 
      */    
     public function _addLinkToPluginTitle( $asLinks ) {
         
-        if ( ! is_array( $asLinks ) ) {
-            $this->oProp->aPluginTitleLinks[] = $asLinks;
+        if ( !is_array( $asLinks ) ) {
+            $this->oProp->aPluginTitleLinks[ ] = $asLinks;
         } else {
             $this->oProp->aPluginTitleLinks = array_merge( 
                 $this->oProp->aPluginTitleLinks, 
@@ -120,7 +120,7 @@ 
             }
             $_sPluginBaseName = plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] );
             add_filter( 
-                $this->_sFilterSuffix_PluginActionLinks . $_sPluginBaseName, 
+                $this->_sFilterSuffix_PluginActionLinks.$_sPluginBaseName, 
                 array( $this, '_replyToAddPluginActionLinks' ) 
             );            
         }
@@ -133,14 +133,14 @@ 
      * @return      string
      * @internal
      */ 
-    public function _replyToAddInfoInFooterLeft( $sLinkHTML='' ) {
+    public function _replyToAddInfoInFooterLeft( $sLinkHTML = '' ) {
 
-        if ( ! $this->_isPageAdded() ) {
+        if ( !$this->_isPageAdded() ) {
             return $sLinkHTML; // $sLinkHTML is given by the hook.
         }
-        $sLinkHTML  = empty( $this->oProp->aScriptInfo['sName'] )
+        $sLinkHTML = empty( $this->oProp->aScriptInfo[ 'sName' ] )
             ? $sLinkHTML
-            : $this->oProp->aFooterInfo['sLeft'];     
+            : $this->oProp->aFooterInfo[ 'sLeft' ];     
             
         $_sPageSlug = $this->oProp->getCurrentPageSlug();
         $_sTabSlug  = $this->oProp->getCurrentTabSlug();        
@@ -151,11 +151,11 @@ 
             array(                 
                 $this->getAOrB( 
                     $_sTabSlug,
-                    'footer_left_' . $_sPageSlug . '_' . $_sTabSlug,
+                    'footer_left_'.$_sPageSlug.'_'.$_sTabSlug,
                     null    // will be ignored
                 ),
-                'footer_left_' . $_sPageSlug,
-                'footer_left_' . $this->oProp->sClassName
+                'footer_left_'.$_sPageSlug,
+                'footer_left_'.$this->oProp->sClassName
             ),
             $sLinkHTML
         );        
@@ -169,9 +169,9 @@ 
      * @return      string
      * @internal
      */
-    public function _replyToAddInfoInFooterRight( $sLinkHTML='' ) {
+    public function _replyToAddInfoInFooterRight( $sLinkHTML = '' ) {
 
-        if ( ! $this->_isPageAdded() ) {
+        if ( !$this->_isPageAdded() ) {
             return $sLinkHTML; // $sLinkTHML is given by the hook.
         }
             
@@ -184,13 +184,13 @@ 
             array(                 
                 $this->getAOrB( 
                     $_sTabSlug,
-                    'footer_right_' . $_sPageSlug . '_' . $_sTabSlug,
+                    'footer_right_'.$_sPageSlug.'_'.$_sTabSlug,
                     null    // will be ignored
                 ),
-                'footer_right_' . $_sPageSlug,
-                'footer_right_' . $this->oProp->sClassName
+                'footer_right_'.$_sPageSlug,
+                'footer_right_'.$this->oProp->sClassName
             ),
-            $this->oProp->aFooterInfo['sRight']
+            $this->oProp->aFooterInfo[ 'sRight' ]
         );             
             
     }
@@ -203,7 +203,7 @@ 
          */
         private function _isPageAdded() {
             
-            if ( ! isset( $_GET[ 'page' ] ) ) {
+            if ( !isset( $_GET[ 'page' ] ) ) {
                 return false;
             }            
             return ( bool ) $this->oProp->isPageAdded( $_GET[ 'page' ] );
@@ -228,7 +228,7 @@ 
             : $this->oProp->sLabelPluginSettingsLink;        
 
         // If the user disables the settings link, the label property is empty. If so, do not add it.
-        if ( ! $this->oProp->sLabelPluginSettingsLink ) {
+        if ( !$this->oProp->sLabelPluginSettingsLink ) {
             return $aLinks;
         }
 
@@ -246,13 +246,13 @@ 
         array_unshift(    
             $aLinks,
             // '<a href="' . esc_url( $_sLinkURL ) . '">' 
-            '<a ' . $this->getAttributes(
+            '<a '.$this->getAttributes(
                 array(
                     'href'      => esc_url( $_sLinkURL ),
                     // 3.5.7+ Added for acceptance testing
                     'class'     => 'apf-plugin-title-action-link apf-post-type',
                 )
-            ) . '>' 
+            ).'>' 
                 . $this->oProp->sLabelPluginSettingsLink 
             . '</a>'
         ); 
@@ -275,16 +275,16 @@ 
         
         // Backward compatibility sanitisation.
         $_aAddingLinks = array();
-        foreach( array_filter( $this->oProp->aPluginDescriptionLinks ) as $_sLLinkHTML ) {
+        foreach ( array_filter( $this->oProp->aPluginDescriptionLinks ) as $_sLLinkHTML ) {
             
-            if ( ! $_sLLinkHTML ) {
+            if ( !$_sLLinkHTML ) {
                 continue;
             }
             if ( is_array( $_sLLinkHTML ) ) {  // should not be an array
                 $_aAddingLinks = array_merge( $_sLLinkHTML, $_aAddingLinks );
                 continue;
             } 
-            $_aAddingLinks[] = ( string ) $_sLLinkHTML;
+            $_aAddingLinks[ ] = ( string ) $_sLLinkHTML;
             
         }
         return array_merge( $aLinks, $_aAddingLinks );
@@ -301,13 +301,13 @@ 
     public function _replyToAddPluginActionLinks( $aLinks ) {
 
         $_aAddingLinks = array();
-        foreach( array_filter( $this->oProp->aPluginTitleLinks ) as $_sLinkHTML ) {
+        foreach ( array_filter( $this->oProp->aPluginTitleLinks ) as $_sLinkHTML ) {
 
             if ( is_array( $_sLinkHTML ) ) {
                 $_aAddingLinks = array_merge( $_sLinkHTML, $_aAddingLinks );
                 continue;
             } 
-            $_aAddingLinks[] = ( string ) $_sLinkHTML;
+            $_aAddingLinks[ ] = ( string ) $_sLinkHTML;
             
         }
         return array_merge( $aLinks, $_aAddingLinks );

development/factory/_common/utility/AdminPageFramework_ErrorReporting.php

Spacing +9 added lines, -9 removed lines

@@ -37,7 +37,7 @@ 
 
     private $_iLevel;
 
-    public function __construct( $iLevel=null ) {
+    public function __construct( $iLevel = null ) {
         $this->_iLevel = null !== $iLevel
             ? $iLevel
             : error_reporting();
@@ -56,11 +56,11 @@ 
         private function _getIncluded() {
 
             $_aIncluded = array();
-            foreach( $this->_aLevels as $_iLevel => $iLevelText ) {
+            foreach ( $this->_aLevels as $_iLevel => $iLevelText ) {
 
                 // This is where we check if a level was used or not
                 if ( $this->_iLevel & $_iLevel ) {
-                    $_aIncluded[] = $_iLevel;
+                    $_aIncluded[ ] = $_iLevel;
                 }
 
             }
@@ -74,16 +74,16 @@ 
             $_aValues       = array();
 
             if ( count( $aIncluded ) > $_iAll / 2 ) {
-                $_aValues[] = 'E_ALL';
-                foreach( $this->_aLevels as $_iLevel => $iLevelText ) {
-                    if ( ! in_array( $_iLevel, $aIncluded ) ) {
-                        $_aValues[] = $iLevelText;
+                $_aValues[ ] = 'E_ALL';
+                foreach ( $this->_aLevels as $_iLevel => $iLevelText ) {
+                    if ( !in_array( $_iLevel, $aIncluded ) ) {
+                        $_aValues[ ] = $iLevelText;
                     }
                 }
                 return implode( ' & ~', $_aValues );
             }
-            foreach( $aIncluded as $_iLevel ) {
-                $_aValues[] = $this->_aLevels[ $_iLevel ];
+            foreach ( $aIncluded as $_iLevel ) {
+                $_aValues[ ] = $this->_aLevels[ $_iLevel ];
             }
             return implode( ' | ', $_aValues );
 

admin-page-framework-loader.php

Spacing +22 added lines, -22 removed lines

@@ -16,9 +16,9 @@ 
  */
 class AdminPageFrameworkLoader_Registry_Base {
 
-    const VERSION        = '3.8.29b01';    // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
+    const VERSION        = '3.8.29b01'; // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
     const NAME           = 'Admin Page Framework - Loader'; // the name is not 'Admin Page Framework' because warning messages gets confusing.
-    const SHORTNAME      = 'Admin Page Framework';  // used for a menu title etc.
+    const SHORTNAME      = 'Admin Page Framework'; // used for a menu title etc.
     const DESCRIPTION    = 'Loads Admin Page Framework which facilitates WordPress plugin and theme development.';
     const URI            = 'http://admin-page-framework.michaeluno.jp/';
     const AUTHOR         = 'miunosoft (Michael Uno)';
@@ -55,14 +55,14 @@ 
      * @remark      This is also accessed from `uninstall.php` so do not remove.
      * @remark      Do not exceed 8 characters as a transient name allows 45 characters or less ( 40 for site transients ) so that md5 (32 characters) can be added.
      */
-    const TRANSIENT_PREFIX         = 'APFL_';
+    const TRANSIENT_PREFIX = 'APFL_';
 
     /**
      * The hook slug used for the prefix of action and filter hook names.
      *
      * @remark      The ending underscore is not necessary.
      */
-    const HOOK_SLUG                = 'admin_page_framework_loader';
+    const HOOK_SLUG = 'admin_page_framework_loader';
 
     /**
      * The text domain slug and its path.
@@ -116,7 +116,7 @@ 
      */
     static public $aAdminPages = array(
         // key => 'page slug'
-        'about'     => 'apfl_about',        // the welcome page
+        'about'     => 'apfl_about', // the welcome page
         'addon'     => 'apfl_addons',
         'tool'      => 'apfl_tools',
         'help'      => 'apfl_contact',
@@ -154,12 +154,12 @@ 
      * @since       3.5.0
      * @return      string
      */
-    public static function getPluginURL( $sRelativePath='' ) {
+    public static function getPluginURL( $sRelativePath = '' ) {
         if ( isset( self::$_sPluginURLCache ) ) {
-            return self::$_sPluginURLCache . $sRelativePath;
+            return self::$_sPluginURLCache.$sRelativePath;
         }
         self::$_sPluginURLCache = trailingslashit( plugins_url( '', self::$sFilePath ) );
-        return self::$_sPluginURLCache . $sRelativePath;
+        return self::$_sPluginURLCache.$sRelativePath;
     }
         /**
          * @since       3.7.9
@@ -189,13 +189,13 @@ 
      * @since       3.5.0
      * @return      void
      */
-    static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
-        if ( ! is_admin() ) {
+    static public function setAdminNotice( $sMessage, $sClassAttribute = 'error' ) {
+        if ( !is_admin() ) {
             return;
         }
-        self::$_aAdminNotices[] = array(
+        self::$_aAdminNotices[ ] = array(
             'message'           => $sMessage,
-            'class_attribute'   => trim( $sClassAttribute ) . ' notice is-dismissible',
+            'class_attribute'   => trim( $sClassAttribute ).' notice is-dismissible',
         );
         add_action( 'admin_notices', array( __CLASS__, '_replyToSetAdminNotice' ) );
     }
@@ -205,12 +205,12 @@ 
          * @return      void
          */
         static public function _replyToSetAdminNotice() {
-            foreach( self::$_aAdminNotices as $_aAdminNotice ) {
-                echo "<div class='" . esc_attr( $_aAdminNotice['class_attribute'] ) . " notice is-dismissible'>"
+            foreach ( self::$_aAdminNotices as $_aAdminNotice ) {
+                echo "<div class='".esc_attr( $_aAdminNotice[ 'class_attribute' ] )." notice is-dismissible'>"
                         ."<p>"
                             . sprintf(
-                                '<strong>%1$s</strong>: ' . $_aAdminNotice['message'],
-                                self::NAME . ' ' . self::VERSION
+                                '<strong>%1$s</strong>: '.$_aAdminNotice[ 'message' ],
+                                self::NAME.' '.self::VERSION
                             )
                         . "</p>"
                     . "</div>";
@@ -222,7 +222,7 @@ 
 AdminPageFrameworkLoader_Registry::setUp( __FILE__ );
 
 // Initial checks. - Do no load if accessed directly, not exiting because the 'uninstall.php' and inclusion list generator will load this file.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( !defined( 'ABSPATH' ) ) {
     return;
 }
 if ( defined( 'DOING_UNINSTALL' ) && DOING_UNINSTALL ) {
@@ -234,8 +234,8 @@ 
 
     $_bFrameworkLoaded = class_exists( 'AdminPageFramework_Registry', false );
     if (
-        ! $_bFrameworkLoaded
-        || ! defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
+        !$_bFrameworkLoaded
+        || !defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
         || version_compare( AdminPageFramework_Registry::VERSION, AdminPageFrameworkLoader_Registry::VERSION, '<' )
     ) {
         AdminPageFrameworkLoader_Registry::setAdminNotice(
@@ -253,18 +253,18 @@ 
 add_action( 'admin_init', 'AdminPageFrameworkLoader_Warning' );
 
 // Include the library file - the development version will be available if you cloned the GitHub repository.
-$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/development/admin-page-framework.php';
+$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/development/admin-page-framework.php';
 $_bDebugMode              = defined( 'WP_DEBUG' ) && WP_DEBUG;
 $_bLoadDevelopmentVersion = $_bDebugMode && file_exists( $_sDevelopmentVersionPath );
 include(
     $_bLoadDevelopmentVersion
         ? $_sDevelopmentVersionPath
-        : AdminPageFrameworkLoader_Registry::$sDirPath . '/library/apf/admin-page-framework.php'
+        : AdminPageFrameworkLoader_Registry::$sDirPath.'/library/apf/admin-page-framework.php'
 );
 
 // Include the framework loader plugin components.
 include( AdminPageFramework_Registry::$aClassFiles[ 'AdminPageFramework_PluginBootstrap' ] );
-include( AdminPageFrameworkLoader_Registry::$sDirPath . '/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
+include( AdminPageFrameworkLoader_Registry::$sDirPath.'/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
 new AdminPageFrameworkLoader_Bootstrap(
     AdminPageFrameworkLoader_Registry::$sFilePath,
     AdminPageFrameworkLoader_Registry::HOOK_SLUG    // hook prefix

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

Spacing +18 added lines, -18 removed lines

@@ -33,7 +33,7 @@ 
      * @return      integer
      * @internal
      */        
-    static public function sortArrayByKey( $a, $b, $sKey='order' ) {
+    static public function sortArrayByKey( $a, $b, $sKey = 'order' ) {
         return isset( $a[ $sKey ], $b[ $sKey ] )
             ? $a[ $sKey ] - $b[ $sKey ]
             : 1;
@@ -76,7 +76,7 @@ 
     static public function unsetDimensionalArrayElement( &$mSubject, array $aKeys ) {
         
         $_sKey = array_shift( $aKeys );
-        if ( ! empty( $aKeys ) ) {
+        if ( !empty( $aKeys ) ) {
             if ( isset( $mSubject[ $_sKey ] ) && is_array( $mSubject[ $_sKey ] ) ) {
                 self::unsetDimensionalArrayElement( $mSubject[ $_sKey ], $aKeys );
             }
@@ -97,8 +97,8 @@ 
     static public function setMultiDimensionalArray( &$mSubject, array $aKeys, $mValue ) {
 
         $_sKey = array_shift( $aKeys );
-        if ( ! empty( $aKeys ) ) {
-            if( ! isset( $mSubject[ $_sKey ] ) || ! is_array( $mSubject[ $_sKey ] ) ) {
+        if ( !empty( $aKeys ) ) {
+            if ( !isset( $mSubject[ $_sKey ] ) || !is_array( $mSubject[ $_sKey ] ) ) {
                 $mSubject[ $_sKey ] = array();
             }
             self::setMultiDimensionalArray( $mSubject[ $_sKey ], $aKeys, $mValue );
@@ -152,10 +152,10 @@ 
 
         $_aNumeric      = self::getIntegerKeyElements( $aSubject );
         $_aAssociative  = self::invertCastArrayContents( $aSubject, $_aNumeric );
-        foreach( $_aNumeric as &$_aElem ) {
+        foreach ( $_aNumeric as &$_aElem ) {
             $_aElem = self::uniteArrays( $_aElem, $_aAssociative );
         }
-        if ( ! empty( $_aAssociative ) ) {
+        if ( !empty( $_aAssociative ) ) {
             array_unshift( $_aNumeric, $_aAssociative ); // insert the main section to the beginning of the array.
         }
         return $_aNumeric;
@@ -178,10 +178,10 @@ 
     public static function castArrayContents( array $aModel, array $aSubject ) {
         
         $_aCast = array();
-        foreach( $aModel as $_isKey => $_v ) {
+        foreach ( $aModel as $_isKey => $_v ) {
             $_aCast[ $_isKey ] = self::getElement(
-                $aSubject,  // subject array
-                $_isKey,    // key
+                $aSubject, // subject array
+                $_isKey, // key
                 null        // default
             );                 
         }
@@ -202,7 +202,7 @@ 
     public static function invertCastArrayContents( array $aModel, array $aSubject ) {
         
         $_aInvert = array();
-        foreach( $aModel as $_isKey => $_v ) {
+        foreach ( $aModel as $_isKey => $_v ) {
             if ( array_key_exists( $_isKey, $aSubject ) ) { 
                 continue; 
             }
@@ -228,7 +228,7 @@ 
     public static function uniteArrays( /* $aPrecedence, $aArray1, $aArray2, ... */ ) {
                 
         $_aArray = array();
-        foreach( array_reverse( func_get_args() ) as $_aArg ) {
+        foreach ( array_reverse( func_get_args() ) as $_aArg ) {
             $_aArray = self::uniteArraysRecursive( 
                 self::getAsArray( $_aArg ), 
                 $_aArray 
@@ -259,7 +259,7 @@ 
             $aPrecedence = array(); 
         }
                 
-        if ( ! is_array( $aDefault ) || ! is_array( $aPrecedence ) ) { 
+        if ( !is_array( $aDefault ) || !is_array( $aPrecedence ) ) { 
             return $aPrecedence; 
         }
         
@@ -268,10 +268,10 @@ 
             return $aPrecedence;
         }
         
-        foreach( $aDefault as $sKey => $v ) {
+        foreach ( $aDefault as $sKey => $v ) {
             
             // If the precedence does not have the key, assign the default's value.
-            if ( ! array_key_exists( $sKey, $aPrecedence ) || is_null( $aPrecedence[ $sKey ] ) ) {
+            if ( !array_key_exists( $sKey, $aPrecedence ) || is_null( $aPrecedence[ $sKey ] ) ) {
                 $aPrecedence[ $sKey ] = $v;
             } else {
                 
@@ -302,8 +302,8 @@ 
      *  - NULL
      * @return      array       The modified array.
      */
-    static public function dropElementsByType( array $aArray, $aTypes=array( 'array' ) ) {
-        foreach( $aArray as $isKey => $vValue ) {
+    static public function dropElementsByType( array $aArray, $aTypes = array( 'array' ) ) {
+        foreach ( $aArray as $isKey => $vValue ) {
             if ( in_array( gettype( $vValue ), $aTypes, true ) ) {
                 unset( $aArray[ $isKey ] );
             }
@@ -319,7 +319,7 @@ 
      */
     static public function dropElementByValue( array $aArray, $vValue ) {
          
-        foreach( self::getAsArray( $vValue, true ) as $_vValue ) {
+        foreach ( self::getAsArray( $vValue, true ) as $_vValue ) {
             $_sKey = array_search( $_vValue, $aArray, true );
             if ( $_sKey === false ) {
                 continue;
@@ -341,7 +341,7 @@ 
      */
     static public function dropElementsByKey( array $aArray, $asKeys ) {
         
-        foreach( self::getAsArray( $asKeys, true ) as $_isKey ) {
+        foreach ( self::getAsArray( $asKeys, true ) as $_isKey ) {
             unset( $aArray[ $_isKey ] );
         }
         return $aArray;