michaeluno / admin-page-framework

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

Spacing +2 added lines, -2 removed lines

@@ -38,11 +38,11 @@
      * Retrieves meta data by given keys and type (user or post).
      * @return      array
      */
-    static public function getMetaDataByKeys( $iObjectID, $aKeys, $sMetaType='post' ) {
+    static public function getMetaDataByKeys( $iObjectID, $aKeys, $sMetaType = 'post' ) {
                
         $_aSavedMeta = array();
                     
-        if ( ! $iObjectID ) {
+        if ( !$iObjectID ) {
             return $_aSavedMeta;
         }
         

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

Spacing +7 added lines, -7 removed lines

@@ -28,7 +28,7 @@ 
         if ( is_string( $_sCustomMenuSlug ) ) {
             return $_sCustomMenuSlug;
         }
-        return 'edit.php?post_type=' . $sPostTypeSlug;
+        return 'edit.php?post_type='.$sPostTypeSlug;
     }             
  
     /**
@@ -61,8 +61,8 @@ 
     static public function getWPAdminDirPath() {
          
         $_sWPAdminPath = str_replace( 
-            get_bloginfo( 'url' ) . '/', // search
-            ABSPATH,    // replace
+            get_bloginfo( 'url' ).'/', // search
+            ABSPATH, // replace
             get_admin_url() // subject - works even in the network admin
         );
         return rtrim( $_sWPAdminPath, '/' );
@@ -78,7 +78,7 @@ 
      * @since       3.7.0
      * @return      void
      */
-    static public function goToLocalURL( $sURL, $oCallbackOnError=null ) {
+    static public function goToLocalURL( $sURL, $oCallbackOnError = null ) {
         self::redirectByType( $sURL, 1, $oCallbackOnError );
     }
     
@@ -89,7 +89,7 @@ 
      * @since       3.7.0      Added the second callback parameter.
      * @return      void
      */
-    static public function goToURL( $sURL, $oCallbackOnError=null ) {
+    static public function goToURL( $sURL, $oCallbackOnError = null ) {
         self::redirectByType( $sURL, 0, $oCallbackOnError );
     }
     
@@ -99,7 +99,7 @@ 
      * @param       integer     $iType              0: external site, 1: local site (within the same domain).
      * @param       callable    $oCallbackOnError
      */
-    static public function redirectByType( $sURL, $iType=0, $oCallbackOnError=null ) {
+    static public function redirectByType( $sURL, $iType = 0, $oCallbackOnError = null ) {
      
         $_iRedirectError = self::getRedirectPreError( $sURL, $iType );
         if ( $_iRedirectError && is_callable( $oCallbackOnError ) ) {
@@ -130,7 +130,7 @@ 
     static public function getRedirectPreError( $sURL, $iType ) {
         
         // check only externnal urls as local ones can be a relative url and always fails the below check.
-        if ( ! $iType && filter_var( $sURL, FILTER_VALIDATE_URL) === false ) {
+        if ( !$iType && filter_var( $sURL, FILTER_VALIDATE_URL ) === false ) {
             return 1;
         }
         // If HTTP headers are already sent, redirect cannot be done.

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

Spacing +9 added lines, -9 removed lines

@@ -63,7 +63,7 @@ 
      * @since   3.1.3
      * @since   3.1.5   Added the $vDefault parameter.
      */    
-    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;  
@@ -103,7 +103,7 @@ 
      * @since       3.1.3
      * @return      boolean     True if set; otherwise, false.
      */
-    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;  
@@ -141,7 +141,7 @@ 
          * The method will replace last ending 33 characters if the given string in the first parameter exceeds the limit. So this number must be greater than 33.
          * @return      string
          */
-        static public function _getCompatibleTransientKey( $sSubject, $iAllowedCharacterLength=45 ) {
+        static public function _getCompatibleTransientKey( $sSubject, $iAllowedCharacterLength = 45 ) {
             
             // Check if the given string eceees the length limit.
             if ( strlen( $sSubject ) <= $iAllowedCharacterLength ) {
@@ -155,7 +155,7 @@ 
                 0, // start position
                 $_iPrefixLengthToKeep - 1 // how many characters to extract
             );
-            return $_sPrefixToKeep . '_' . md5( $sSubject );
+            return $_sPrefixToKeep.'_'.md5( $sSubject );
          
         }    
     
@@ -169,7 +169,7 @@ 
      * @param   mixed           $vDefault     the default value that will be returned if nothing is stored.
      * @param   array           $aAdditionalOptions     an additional options array to merge with the options array. 
      */
-    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 );
     }
     /**
@@ -184,7 +184,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' );        
     }  
         /**
@@ -193,16 +193,16 @@ 
          * @since       3.5.3
          * @return      mixed
          */
-        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/_common/utility/wp_utility/AdminPageFramework_WPUtility_File.php

Spacing +19 added lines, -19 removed lines

@@ -28,7 +28,7 @@ 
      * @since   3.6.2       Supported a text content to be passed to the first parameter.
      * @access  public
      */ 
-    static public function getScriptData( $sPathOrContent, $sType='plugin', $aDefaultHeaderKeys=array() ) {
+    static public function getScriptData( $sPathOrContent, $sType = 'plugin', $aDefaultHeaderKeys = array() ) {
         
         $_aHeaderKeys = $aDefaultHeaderKeys + array(
             // storing array key =>    the comment entry header label
@@ -66,19 +66,19 @@ 
 
         switch ( trim( $sType ) ) {
             case 'theme':    
-                $aData['sName'] = $aData['sThemeName'];
-                $aData['sURI'] = $aData['sThemeURI'];
+                $aData[ 'sName' ] = $aData[ 'sThemeName' ];
+                $aData[ 'sURI' ] = $aData[ 'sThemeURI' ];
                 break;
             case 'library':    
-                $aData['sName'] = $aData['sLibraryName'];
-                $aData['sURI'] = $aData['sLibraryURI'];
+                $aData[ 'sName' ] = $aData[ 'sLibraryName' ];
+                $aData[ 'sURI' ] = $aData[ 'sLibraryURI' ];
                 break;
             case 'script':    
-                $aData['sName'] = $aData['sScriptName'];
+                $aData[ 'sName' ] = $aData[ 'sScriptName' ];
                 break;     
             case 'plugin':    
-                $aData['sName'] = $aData['sPluginName'];
-                $aData['sURI'] = $aData['sPluginURI'];
+                $aData[ 'sName' ] = $aData[ 'sPluginName' ];
+                $aData[ 'sURI' ] = $aData[ 'sPluginURI' ];
                 break;
             default:    
                 break;     
@@ -93,22 +93,22 @@ 
      * @since       3.6.2
      * @return      array       The script data
      */
-    static public function getScriptDataFromContents( $sContent, $sType='plugin', $aDefaultHeaderKeys=array() ) {
+    static public function getScriptDataFromContents( $sContent, $sType = 'plugin', $aDefaultHeaderKeys = array() ) {
         
         // Make sure we catch CR-only line endings.
         $sContent = str_replace( "\r", "\n", $sContent );
         
-        $_aHeaders      = $aDefaultHeaderKeys;
+        $_aHeaders = $aDefaultHeaderKeys;
         if ( $sType ) {
             $_aExtraHeaders = apply_filters( "extra_{$sType}_headers", array() );
-            if ( ! empty( $_aExtraHeaders ) ) {
+            if ( !empty( $_aExtraHeaders ) ) {
                 $_aExtraHeaders = array_combine( $_aExtraHeaders, $_aExtraHeaders ); // keys equal values
                 $_aHeaders      = array_merge( $_aExtraHeaders, ( array ) $aDefaultHeaderKeys );
             }
         } 
 
         foreach ( $_aHeaders as $_sHeaderKey => $_sRegex ) {
-            $_bFound = preg_match( '/^[ \t\/*#@]*' . preg_quote( $_sRegex, '/' ) . ':(.*)$/mi', $sContent, $_aMatch );
+            $_bFound = preg_match( '/^[ \t\/*#@]*'.preg_quote( $_sRegex, '/' ).':(.*)$/mi', $sContent, $_aMatch );
             $_aHeaders[ $_sHeaderKey ] = $_bFound && $_aMatch[ 1 ]
                 ? _cleanup_header_comment( $_aMatch[ 1 ] )
                 : '';        
@@ -125,14 +125,14 @@ 
      * @since       3.4.2
      * @see         download_url() in file.php in core.
      */
-    static public function download( $sURL, $iTimeOut=300 ) {
+    static public function download( $sURL, $iTimeOut = 300 ) {
         
         if ( false === filter_var( $sURL, FILTER_VALIDATE_URL ) ) {
             return false;
         }
 
         $_sTmpFileName = self::setTempPath( self::getBaseNameOfURL( $sURL ) );
-        if ( ! $_sTmpFileName ) {
+        if ( !$_sTmpFileName ) {
             return false;
         }
 
@@ -150,7 +150,7 @@ 
             return false;
         }
 
-        if ( 200 != wp_remote_retrieve_response_code( $_aoResponse ) ){
+        if ( 200 != wp_remote_retrieve_response_code( $_aoResponse ) ) {
             unlink( $_sTmpFileName );
             return false;
         }
@@ -175,16 +175,16 @@ 
      * @since       3.4.2
      * @return      string      The set file path.
      */
-    static public function setTempPath( $sFilePath='' ) {
+    static public function setTempPath( $sFilePath = '' ) {
         
         $_sDir = get_temp_dir();
         
         $sFilePath = basename( $sFilePath );
         if ( empty( $sFilePath ) ) {            
-            $sFilePath = time() . '.tmp';
+            $sFilePath = time().'.tmp';
         }
 
-        $sFilePath = $_sDir . wp_unique_filename( $_sDir, $sFilePath );
+        $sFilePath = $_sDir.wp_unique_filename( $_sDir, $sFilePath );
         touch( $sFilePath );
         return $sFilePath;
         
@@ -197,7 +197,7 @@ 
      */
     static public function getBaseNameOfURL( $sURL ) {
         
-        $_sPath         = parse_url( $sURL, PHP_URL_PATH) ; 
+        $_sPath         = parse_url( $sURL, PHP_URL_PATH ); 
         $_sFileBaseName = basename( $_sPath );
         return $_sFileBaseName;
         

Braces +2 added lines, -2 removed lines

@@ -150,7 +150,7 @@ 
             return false;
         }
 
-        if ( 200 != wp_remote_retrieve_response_code( $_aoResponse ) ){
+        if ( 200 != wp_remote_retrieve_response_code( $_aoResponse ) ) {
             unlink( $_sTmpFileName );
             return false;
         }
@@ -180,7 +180,7 @@ 
         $_sDir = get_temp_dir();
         
         $sFilePath = basename( $sFilePath );
-        if ( empty( $sFilePath ) ) {            
+        if ( empty( $sFilePath ) ) {
             $sFilePath = time() . '.tmp';
         }
 

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

Spacing +4 added lines, -4 removed lines

@@ -42,7 +42,7 @@ 
                 : @mysql_get_server_info(),
         );
         
-        foreach( ( array ) $wpdb->get_results( "SHOW VARIABLES", ARRAY_A ) as $_iIndex => $_aItem ) {
+        foreach ( ( array ) $wpdb->get_results( "SHOW VARIABLES", ARRAY_A ) as $_iIndex => $_aItem ) {
             
             $_aItem     = array_values( $_aItem );
             $_sKey      = array_shift( $_aItem );
@@ -63,8 +63,8 @@ 
     static public function getMySQLErrorLogPath() {
         
         $_aMySQLInfo = self::getMySQLInfo();
-        return isset( $_aMySQLInfo['log_error'] )
-            ? $_aMySQLInfo['log_error']
+        return isset( $_aMySQLInfo[ 'log_error' ] )
+            ? $_aMySQLInfo[ 'log_error' ]
             : '';
         
     }
@@ -73,7 +73,7 @@ 
      * Returns a PHP error log.
      * @since       3.4.6
      */
-    static public function getMySQLErrorLog( $iLines=1 ) {
+    static public function getMySQLErrorLog( $iLines = 1 ) {
         
         $_sLog = self::getFileTailContents( self::getMySQLErrorLogPath(), $iLines );
         

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

Spacing +6 added lines, -6 removed lines

@@ -38,9 +38,9 @@ 
      
         $_sQuoteCharactor   = "'";
         $_aOutput           = array();
-        foreach( $aAttributes as $_sAttribute => $_mProperty ) {
+        foreach ( $aAttributes as $_sAttribute => $_mProperty ) {
             if ( is_scalar( $_mProperty ) ) {
-                $_aOutput[] = "{$_sAttribute}={$_sQuoteCharactor}" . esc_attr( $_mProperty ) . "{$_sQuoteCharactor}";
+                $_aOutput[ ] = "{$_sAttribute}={$_sQuoteCharactor}".esc_attr( $_mProperty )."{$_sQuoteCharactor}";
             }            
         }     
         return implode( ' ', $_aOutput );
@@ -79,11 +79,11 @@ 
      * @since       3.6.0
      * @return      string
      */
-    static public function getHTMLTag( $sTagName, array $aAttributes, $sValue=null ) {
+    static public function getHTMLTag( $sTagName, array $aAttributes, $sValue = null ) {
         $_sTag = tag_escape( $sTagName );
         return null === $sValue
-            ? "<" . $_sTag . " " . self::getAttributes( $aAttributes ) . " />"
-            : "<" . $_sTag . " " . self::getAttributes( $aAttributes ) . ">"
+            ? "<".$_sTag." ".self::getAttributes( $aAttributes )." />"
+            : "<".$_sTag." ".self::getAttributes( $aAttributes ).">"
                     . $sValue
                 . "</{$_sTag}>";        
     }    
@@ -93,7 +93,7 @@ 
          * @return      string
          * @deprecated  3.6.0       Use `getHTMLTag()` instead.
          */
-        static public function generateHTMLTag( $sTagName, array $aAttributes, $sValue=null ) {
+        static public function generateHTMLTag( $sTagName, array $aAttributes, $sValue = null ) {
             return self::getHTMLTag( $sTagName, $aAttributes, $sValue );
         }
     

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

Spacing +11 added lines, -11 removed lines

@@ -25,13 +25,13 @@ 
      */
     static public function getCurrentAdminURL() {
         
-        $sRequestURI    = $GLOBALS['is_IIS'] ? $_SERVER['PATH_INFO'] : $_SERVER["REQUEST_URI"];
-        $sPageURL       = 'on' == @$_SERVER["HTTPS"] ? "https://" : "http://";
+        $sRequestURI    = $GLOBALS[ 'is_IIS' ] ? $_SERVER[ 'PATH_INFO' ] : $_SERVER[ "REQUEST_URI" ];
+        $sPageURL       = 'on' == @$_SERVER[ "HTTPS" ] ? "https://" : "http://";
         
-        if ( "80" != $_SERVER["SERVER_PORT"] ) {
-            $sPageURL .= $_SERVER["SERVER_NAME"] . ":" . $_SERVER["SERVER_PORT"] . $sRequestURI;
+        if ( "80" != $_SERVER[ "SERVER_PORT" ] ) {
+            $sPageURL .= $_SERVER[ "SERVER_NAME" ].":".$_SERVER[ "SERVER_PORT" ].$sRequestURI;
         } else {
-            $sPageURL .= $_SERVER["SERVER_NAME"] . $sRequestURI;
+            $sPageURL .= $_SERVER[ "SERVER_NAME" ].$sRequestURI;
         }
         return $sPageURL;
         
@@ -48,7 +48,7 @@ 
      * @param string $sSubjectURL ( optional ) The subject url to modify
      * @return string The modified url.
      */
-    static public function getQueryAdminURL( $aAddingQueries=array(), $aRemovingQueryKeys=array(), $sSubjectURL='' ) {
+    static public function getQueryAdminURL( $aAddingQueries = array(), $aRemovingQueryKeys = array(), $sSubjectURL = '' ) {
         
         $_sAdminURL = is_network_admin()
             ? network_admin_url( AdminPageFramework_WPUtility_Page::getPageNow() )
@@ -98,7 +98,7 @@ 
         $_oWPStyles      = new WP_Styles(); // It doesn't matter whether the file is a style or not. Just use the built-in WordPress class to calculate the SRC URL.
         $_sRelativePath  = AdminPageFramework_Utility::getRelativePath( ABSPATH, $sFilePath );     
         $_sRelativePath  = preg_replace( "/^\.[\/\\\]/", '', $_sRelativePath, 1 ); // removes the heading ./ or .\ 
-        $_sHref          = trailingslashit( $_oWPStyles->base_url ) . $_sRelativePath;
+        $_sHref          = trailingslashit( $_oWPStyles->base_url ).$_sRelativePath;
         unset( $_oWPStyles ); // for PHP 5.2.x or below
         return $_sHref;
         
@@ -115,9 +115,9 @@ 
      * @since       3.7.9       Changed not to escape characters.
      * @return      string|null
      */    
-    static public function getResolvedSRC( $sSRC, $bReturnNullIfNotExist=false ) {    
+    static public function getResolvedSRC( $sSRC, $bReturnNullIfNotExist = false ) {    
 
-        if ( ! self::isResourcePath( $sSRC ) ) {
+        if ( !self::isResourcePath( $sSRC ) ) {
             return $bReturnNullIfNotExist 
                 ? null 
                 : $sSRC;
@@ -130,7 +130,7 @@ 
 
         // If the file exists, it means it is an absolute path. If so, calculate the URL from the path.
         if ( file_exists( realpath( $sSRC ) ) ) {
-            return self::getSRCFromPath( $sSRC );   // url escaping is done in the method
+            return self::getSRCFromPath( $sSRC ); // url escaping is done in the method
         }
         
         if ( $bReturnNullIfNotExist ) {
@@ -144,7 +144,7 @@ 
         /**
          * @deprecated      3.6.0       Use `getResolvedSRC()` instead.
          */
-        static public function resolveSRC( $sSRC, $bReturnNullIfNotExist=false ) {
+        static public function resolveSRC( $sSRC, $bReturnNullIfNotExist = false ) {
             return self::getResolvedSRC( $sSRC, $bReturnNullIfNotExist );
         }    
     

Braces +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
      * @since       3.6.0       Changed the name from `resolveSRC()`.
      * @return      string|null
      */    
-    static public function getResolvedSRC( $sSRC, $bReturnNullIfNotExist=false ) {    
+    static public function getResolvedSRC( $sSRC, $bReturnNullIfNotExist=false ) {
 
         if ( ! self::isResourcePath( $sSRC ) ) {
             return $bReturnNullIfNotExist 

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

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
      * @since       3.5.3
      * @return      string      The site language.
      */
-    static public function getSiteLanguage( $sDefault='en_US' ) {
+    static public function getSiteLanguage( $sDefault = 'en_US' ) {
         return defined( 'WPLANG' ) && WPLANG ? WPLANG : $sDefault;
     }
         

development/factory/_common/utility/AdminPageFramework_RegisterClasses.php

Braces +10 added lines, -9 removed lines

@@ -140,7 +140,8 @@ 
             $_aExcludeDirNames = ( array ) $aSearchOptions['exclude_dir_names'];
             $_bIsRecursive = $aSearchOptions[ 'is_recursive' ];
             
-            if ( defined( 'GLOB_BRACE' ) ) { // in some OSes this flag constant is not available.
+            if ( defined( 'GLOB_BRACE' ) ) {
+// in some OSes this flag constant is not available.
                 $_aFilePaths = $_bIsRecursive
                     ? $this->doRecursiveGlob( $sClassDirPath . '*.' . $this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE, $_aExcludeDirPaths, $_aExcludeDirNames )
                     : ( array ) glob( $sClassDirPath . '*.' . $this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE );
@@ -204,26 +205,26 @@ 
         /**
          * Responds to the PHP auto-loader and includes the passed class based on the previously stored path associated with the class name in the constructor.
          */
-        public function _replyToAutoLoad_include( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+        public function _replyToAutoLoad_include( $sCalledUnknownClassName ) {
+            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) {
                 return; 
             }
             include( $this->_aClasses[ $sCalledUnknownClassName ] );
         }
-        public function _replyToAutoLoad_include_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+        public function _replyToAutoLoad_include_once( $sCalledUnknownClassName ) {
+            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) {
                 return; 
             }
             include_once( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
-        public function _replyToAutoLoad_require( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+        public function _replyToAutoLoad_require( $sCalledUnknownClassName ) {
+            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) {
                 return; 
             }
             require( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
-        public function _replyToAutoLoad_require_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+        public function _replyToAutoLoad_require_once( $sCalledUnknownClassName ) {
+            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) {
                 return; 
             }
             require_once( $this->_aClasses[ $sCalledUnknownClassName ] );

Doc Comments +2 added lines, -1 removed lines

@@ -75,7 +75,7 @@ 
      * Sets up properties and performs registering classes.
      *
      * 
-     * @param    array|string       $asScanDirPath       the target directory path to scan
+     * @param    array|string       $asScanDirPaths       the target directory path to scan
      * @param    array              $aOptions            The recursive settings.
      * <ul>
      *      <li><code>is_recursive</code> - (boolean) determines whether the scan should be performed recursively.</li>
@@ -217,6 +217,7 @@ 
         /**
          * The recursive version of the glob() function.
          * @internal
+         * @param string $sPathPatten
          */
         protected function doRecursiveGlob( $sPathPatten, $nFlags=0, array $aExcludeDirs=array(), array $aExcludeDirNames=array() ) {
 

Spacing +25 added lines, -25 removed lines

@@ -66,7 +66,7 @@ 
         'is_recursive'          => true,
         'exclude_dir_paths'     => array(),
         'exclude_dir_names'     => array( 'asset', 'assets', 'css', 'js', 'image', 'images', 'license', 'document', 'documents' ),
-        'allowed_extensions'    => array( 'php', ), // 'inc'
+        'allowed_extensions'    => array( 'php',), // 'inc'
         'include_function'      => 'include',
         'exclude_class_names'   => array( /* 'SomeClass', 'SomeOtherClass' */ ),
     );
@@ -105,10 +105,10 @@ 
      * 
      * @remark The directory paths set for the 'exclude_dir_paths' option should use the system directory separator.
      */
-    public function __construct( $asScanDirPaths, array $aOptions=array(), array $aClasses=array() ) {
+    public function __construct( $asScanDirPaths, array $aOptions = array(), array $aClasses = array() ) {
         
         $_aOptions = $aOptions + self::$_aStructure_Options;
-        $this->_aClasses   = $aClasses + $this->_getClassArray( $asScanDirPaths, $_aOptions );
+        $this->_aClasses = $aClasses + $this->_getClassArray( $asScanDirPaths, $_aOptions );
         $this->_registerClasses( $_aOptions[ 'include_function' ] );
         
     }
@@ -134,7 +134,7 @@ 
             return array();
         }
         $_aFilePaths = array();
-        foreach( ( array ) $asScanDirPaths as $_sClassDirPath ) {
+        foreach ( ( array ) $asScanDirPaths as $_sClassDirPath ) {
             if ( realpath( $_sClassDirPath ) ) {
                 $_aFilePaths = array_merge( $this->getFilePaths( $_sClassDirPath, $aSearchOptions ), $_aFilePaths );
             }
@@ -142,11 +142,11 @@ 
         
         // Store classes in an array.
         $_aClasses = array();
-        foreach( $_aFilePaths as $_sFilePath ) {
+        foreach ( $_aFilePaths as $_sFilePath ) {
             
             // Class name without a file extension.
             $_sClassNameWOExt = pathinfo( $_sFilePath, PATHINFO_FILENAME );
-            if ( in_array( $_sClassNameWOExt, $aSearchOptions['exclude_class_names'] ) ) {
+            if ( in_array( $_sClassNameWOExt, $aSearchOptions[ 'exclude_class_names' ] ) ) {
                 continue;
             }
             $_aClasses[ $_sClassNameWOExt ] = $_sFilePath; 
@@ -176,26 +176,26 @@ 
          */
         protected function getFilePaths( $sClassDirPath, array $aSearchOptions ) {
             
-            $sClassDirPath = rtrim( $sClassDirPath, '\\/' ) . DIRECTORY_SEPARATOR; // ensures the trailing (back/)slash exists. 
-            $_aAllowedExtensions = $aSearchOptions['allowed_extensions'];
-            $_aExcludeDirPaths = ( array ) $aSearchOptions['exclude_dir_paths'];
-            $_aExcludeDirNames = ( array ) $aSearchOptions['exclude_dir_names'];
+            $sClassDirPath = rtrim( $sClassDirPath, '\\/' ).DIRECTORY_SEPARATOR; // ensures the trailing (back/)slash exists. 
+            $_aAllowedExtensions = $aSearchOptions[ 'allowed_extensions' ];
+            $_aExcludeDirPaths = ( array ) $aSearchOptions[ 'exclude_dir_paths' ];
+            $_aExcludeDirNames = ( array ) $aSearchOptions[ 'exclude_dir_names' ];
             $_bIsRecursive = $aSearchOptions[ 'is_recursive' ];
             
             if ( defined( 'GLOB_BRACE' ) ) { // in some OSes this flag constant is not available.
                 $_aFilePaths = $_bIsRecursive
-                    ? $this->doRecursiveGlob( $sClassDirPath . '*.' . $this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE, $_aExcludeDirPaths, $_aExcludeDirNames )
-                    : ( array ) glob( $sClassDirPath . '*.' . $this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE );
+                    ? $this->doRecursiveGlob( $sClassDirPath.'*.'.$this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE, $_aExcludeDirPaths, $_aExcludeDirNames )
+                    : ( array ) glob( $sClassDirPath.'*.'.$this->_getGlobPatternExtensionPart( $_aAllowedExtensions ), GLOB_BRACE );
                 return array_filter( $_aFilePaths ); // drop non-value elements.    
             } 
                 
             // For the Solaris operation system.
             $_aFilePaths = array();
-            foreach( $_aAllowedExtensions as $__sAllowedExtension ) {
+            foreach ( $_aAllowedExtensions as $__sAllowedExtension ) {
                                 
                 $__aFilePaths = $_bIsRecursive
-                    ? $this->doRecursiveGlob( $sClassDirPath . '*.' . $__sAllowedExtension, 0, $_aExcludeDirPaths, $_aExcludeDirNames )
-                    : ( array ) glob( $sClassDirPath . '*.' . $__sAllowedExtension );
+                    ? $this->doRecursiveGlob( $sClassDirPath.'*.'.$__sAllowedExtension, 0, $_aExcludeDirPaths, $_aExcludeDirNames )
+                    : ( array ) glob( $sClassDirPath.'*.'.$__sAllowedExtension );
 
                 $_aFilePaths = array_merge( $__aFilePaths, $_aFilePaths );
                 
@@ -208,28 +208,28 @@ 
          * Constructs the file pattern of the file extension part used for the glob() function with the given file extensions.
          * @internal
          */
-        protected function _getGlobPatternExtensionPart( array $aExtensions=array( 'php', 'inc' ) ) {
+        protected function _getGlobPatternExtensionPart( array $aExtensions = array( 'php', 'inc' ) ) {
             return empty( $aExtensions ) 
                 ? '*'
-                : '{' . implode( ',', $aExtensions ) . '}';
+                : '{'.implode( ',', $aExtensions ).'}';
         }
         
         /**
          * The recursive version of the glob() function.
          * @internal
          */
-        protected function doRecursiveGlob( $sPathPatten, $nFlags=0, array $aExcludeDirs=array(), array $aExcludeDirNames=array() ) {
+        protected function doRecursiveGlob( $sPathPatten, $nFlags = 0, array $aExcludeDirs = array(), array $aExcludeDirNames = array() ) {
 
             $_aFiles = glob( $sPathPatten, $nFlags );    
             $_aFiles = is_array( $_aFiles ) ? $_aFiles : array(); // glob() can return false.
-            $_aDirs = glob( dirname( $sPathPatten ) . DIRECTORY_SEPARATOR . '*', GLOB_ONLYDIR|GLOB_NOSORT );
+            $_aDirs = glob( dirname( $sPathPatten ).DIRECTORY_SEPARATOR.'*', GLOB_ONLYDIR | GLOB_NOSORT );
             $_aDirs = is_array( $_aDirs ) ? $_aDirs : array();
             foreach ( $_aDirs as $_sDirPath ) {
 
                 if ( in_array( $_sDirPath, $aExcludeDirs ) ) { continue; }
                 if ( in_array( pathinfo( $_sDirPath, PATHINFO_DIRNAME ), $aExcludeDirNames ) ) { continue; }
                 
-                $_aFiles = array_merge( $_aFiles, $this->doRecursiveGlob( $_sDirPath . DIRECTORY_SEPARATOR . basename( $sPathPatten ), $nFlags, $aExcludeDirs ) );
+                $_aFiles = array_merge( $_aFiles, $this->doRecursiveGlob( $_sDirPath.DIRECTORY_SEPARATOR.basename( $sPathPatten ), $nFlags, $aExcludeDirs ) );
                 
             }
             return $_aFiles;
@@ -244,7 +244,7 @@ 
      * @return      void
      */
     protected function _registerClasses( $sIncludeFunction ) {
-        spl_autoload_register( array( $this, '_replyToAutoLoad_' . $sIncludeFunction ) );
+        spl_autoload_register( array( $this, '_replyToAutoLoad_'.$sIncludeFunction ) );
     }    
         /**#@+
          * Responds to the PHP auto-loader and includes the passed class based on the previously stored path associated with the class name in the constructor.
@@ -252,25 +252,25 @@ 
          * @return      void
          */        
         public function _replyToAutoLoad_include( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
                 return; 
             }
             include( $this->_aClasses[ $sCalledUnknownClassName ] );
         }
         public function _replyToAutoLoad_include_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
                 return; 
             }
             include_once( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
         public function _replyToAutoLoad_require( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
                 return; 
             }
             require( $this->_aClasses[ $sCalledUnknownClassName ] );
         }        
         public function _replyToAutoLoad_require_once( $sCalledUnknownClassName ) {            
-            if ( ! isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
+            if ( !isset( $this->_aClasses[ $sCalledUnknownClassName ] ) ) { 
                 return; 
             }
             require_once( $this->_aClasses[ $sCalledUnknownClassName ] );