michaeluno / admin-page-framework

development/factory/page_meta_box/AdminPageFramework_PageMetaBox_Model.php

Braces +2 added lines, -2 removed lines

@@ -81,7 +81,7 @@ 
      */ 
     public function _replyToRegisterMetaBoxes( $sPageHook='' ) {
         foreach( $this->oProp->aPageSlugs as $_sKey => $_asPage ) {
-            if ( is_string( $_asPage ) )  {
+            if ( is_string( $_asPage ) ) {
                 $this->_registerMetaBox( $_asPage );
                 continue;
             }            
@@ -98,7 +98,7 @@ 
          */
         private function _registerMetaBoxes( $sPageSlug, $asPage ) {
             foreach( $this->oUtil->getAsArray( $asPage ) as $_sTabSlug ) {
-                if ( ! $this->oProp->isCurrentTab( $_sTabSlug ) ) { 
+                if ( ! $this->oProp->isCurrentTab( $_sTabSlug ) ) {
                     continue; 
                 }
                 $this->_registerMetaBox( $sPageSlug );

Indentation +1 added lines, -1 removed lines

@@ -256,7 +256,7 @@
 
     }    
     
-       /**
+        /**
          * Extracts meta box form fields options array from the given options array of an admin page.
          * 
          * @since       3.5.6

Spacing +21 added lines, -21 removed lines

@@ -45,11 +45,11 @@ 
     public function _replyToSetUpValidationHooks( $oScreen ) {
 
         // Validation hooks
-        foreach( $this->oProp->aPageSlugs as $_sIndexOrPageSlug => $_asTabArrayOrPageSlug ) {
+        foreach ( $this->oProp->aPageSlugs as $_sIndexOrPageSlug => $_asTabArrayOrPageSlug ) {
             
             if ( is_scalar( $_asTabArrayOrPageSlug ) ) {
                 $_sPageSlug = $_asTabArrayOrPageSlug;
-                add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 );  // 3.4.1+
+                add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 ); // 3.4.1+
                 add_filter( "validation_{$_sPageSlug}", array( $this, '_replyToValidateOptions' ), 10, 4 );
                 add_filter( "options_update_status_{$_sPageSlug}", array( $this, '_replyToModifyOptionsUpdateStatus' ) );
                 continue;
@@ -58,7 +58,7 @@ 
             // At this point, the array key is the page slug. It means the user specified the tab(s).
             $_sPageSlug = $_sIndexOrPageSlug;
             $_aTabs     = $_asTabArrayOrPageSlug;
-            foreach( $_aTabs as $_sTabSlug ) {
+            foreach ( $_aTabs as $_sTabSlug ) {
                 add_filter( "validation_{$_sPageSlug}_{$_sTabSlug}", array( $this, '_replyToValidateOptions' ), 10, 4 );
                 add_filter( "validation_saved_options_without_dynamic_elements_{$_sPageSlug}_{$_sTabSlug}", array( $this, '_replyToFilterPageOptionsWODynamicElements' ), 10, 2 ); // 3.4.1+
                 add_filter( "options_update_status_{$_sPageSlug}_{$_sTabSlug}", array( $this, '_replyToModifyOptionsUpdateStatus' ) );
@@ -78,9 +78,9 @@ 
      * @return      void
      * @callback    action      add_meta_boxes
      */ 
-    public function _replyToRegisterMetaBoxes( $sPageHook='' ) {
-        foreach( $this->oProp->aPageSlugs as $_sKey => $_asPage ) {
-            if ( is_string( $_asPage ) )  {
+    public function _replyToRegisterMetaBoxes( $sPageHook = '' ) {
+        foreach ( $this->oProp->aPageSlugs as $_sKey => $_asPage ) {
+            if ( is_string( $_asPage ) ) {
                 $this->_registerMetaBox( $_asPage );
                 continue;
             }            
@@ -96,8 +96,8 @@ 
          * @return      void
          */
         private function _registerMetaBoxes( $sPageSlug, $asPage ) {
-            foreach( $this->oUtil->getAsArray( $asPage ) as $_sTabSlug ) {
-                if ( ! $this->oProp->isCurrentTab( $_sTabSlug ) ) { 
+            foreach ( $this->oUtil->getAsArray( $asPage ) as $_sTabSlug ) {
+                if ( !$this->oProp->isCurrentTab( $_sTabSlug ) ) { 
                     continue; 
                 }
                 $this->_registerMetaBox( $sPageSlug );
@@ -114,12 +114,12 @@ 
          */
         private function _registerMetaBox( $sPageSlug ) {
             add_meta_box( 
-                $this->oProp->sMetaBoxID,                       // id
-                $this->oProp->sTitle,                           // title
+                $this->oProp->sMetaBoxID, // id
+                $this->oProp->sTitle, // title
                 array( $this, '_replyToPrintMetaBoxContents' ), // callback
                 $this->oProp->_getScreenIDOfPage( $sPageSlug ), // screen ID
-                $this->oProp->sContext,                         // context
-                $this->oProp->sPriority,                        // priority
+                $this->oProp->sContext, // context
+                $this->oProp->sPriority, // priority
                 null                                            // argument (deprecated)
             );
         }
@@ -167,17 +167,17 @@ 
         
         $_aNewMetaBoxInputs      = $this->oForm->getSubmittedData( $_POST );
         $_aOldMetaBoxInputs      = $this->oUtil->castArrayContents( 
-            $this->oForm->getDataStructureFromAddedFieldsets(),   // model
+            $this->oForm->getDataStructureFromAddedFieldsets(), // model
             $aOldPageOptions        // data source
         );
 
         // Apply filters - third party scripts will have access to the input.
         $_aNewMetaBoxInputsRaw   = $_aNewMetaBoxInputs; // copy one for validation errors.
         $_aNewMetaBoxInputs      = call_user_func_array( 
-            array( $this, 'validate' ),     // triggers __call()
+            array( $this, 'validate' ), // triggers __call()
             array( $_aNewMetaBoxInputs, $_aOldMetaBoxInputs, $this, $aSubmitInfo ) 
         ); // 3.5.3+
-        $_aNewMetaBoxInputs      = $this->oUtil->addAndApplyFilters( 
+        $_aNewMetaBoxInputs = $this->oUtil->addAndApplyFilters( 
             $this, 
             "validation_{$this->oProp->sClassName}", 
             $_aNewMetaBoxInputs, 
@@ -215,7 +215,7 @@ 
      */
     public function _replyToModifyOptionsUpdateStatus( $aStatus ) {
         
-        if ( ! $this->hasFieldError() ) {
+        if ( !$this->hasFieldError() ) {
             return $aStatus;
         }
         return array( 
@@ -245,12 +245,12 @@ 
 
         $_aPageOptions = $this->oUtil->addAndApplyFilter(
             $this, // the caller factory object
-            'options_' . $this->oProp->sClassName,
+            'options_'.$this->oProp->sClassName,
             $_aPageOptions
         );
 
         return $this->oUtil->castArrayContents( 
-            $this->oForm->getDataStructureFromAddedFieldsets(),   // model
+            $this->oForm->getDataStructureFromAddedFieldsets(), // model
             $_aPageOptions        // data source
         );
 
@@ -267,9 +267,9 @@ 
         private function _getPageMetaBoxOptionsFromPageOptions( array $aPageOptions, array $aFieldsets ) {
      
             $_aOptions = array();
-            foreach( $aFieldsets as $_sSectionID => $_aFieldsets ) {
-                if ( '_default' === $_sSectionID  ) {
-                    foreach( $_aFieldsets as $_aField ) {
+            foreach ( $aFieldsets as $_sSectionID => $_aFieldsets ) {
+                if ( '_default' === $_sSectionID ) {
+                    foreach ( $_aFieldsets as $_aField ) {
                         if ( array_key_exists( $_aField[ 'field_id' ], $aPageOptions ) ) {
                             $_aOptions[ $_aField[ 'field_id' ] ] = $aPageOptions[ $_aField[ 'field_id' ] ];
                         }

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,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; 
@@ -55,7 +55,7 @@ 
         // Properties 
         $_sProprtyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;        
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;        
         $this->oProp        = new $_sProprtyClassName( 
             $this, 
             get_class( $this ), 

Braces +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      */ 
     function __construct( $asTaxonomySlug, $sOptionKey='', $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
         
-        if ( empty( $asTaxonomySlug ) ) { 
+        if ( empty( $asTaxonomySlug ) ) {
             return; 
         }
         

development/factory/user_meta/AdminPageFramework_UserMeta.php

Spacing +6 added lines, -6 removed lines

@@ -39,16 +39,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' ) {
         
         $_sProprtyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;        
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;        
         $this->oProp = new $_sProprtyClassName( 
-            $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/_common/form/field_type/AdminPageFramework_FieldType_media.php

Spacing +19 added lines, -19 removed lines

@@ -21,7 +21,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'media', );
+    public $aFieldTypeSlugs = array( 'media',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -53,7 +53,7 @@ 
         return 
             $this->_getScript_MediaUploader(
                 "admin_page_framework"
-            ) . PHP_EOL
+            ).PHP_EOL
             . $this->_getScript_RegisterCallbacks();
     }    
     
@@ -148,7 +148,7 @@ 
             $_sInsertFromURL         = esc_js( $this->oMsg->get( 'insert_from_url' ) );
             
             // If the WordPress version is 3.4.x or below
-            if ( ! function_exists( 'wp_enqueue_media' ) ) {
+            if ( !function_exists( 'wp_enqueue_media' ) ) {
                 return <<<JAVASCRIPTS
                     /**
                      * Bind/rebinds the thickbox script the given selector element.
@@ -388,10 +388,10 @@ 
             // Do not include the escaping character (backslash) in the heredoc variable declaration 
             // because the minifier script will parse it and the <<<JAVASCRIPTS and JAVASCRIPTS; parts are converted to double quotes (")
             // which causes the PHP syntax error.
-            $_sButtonHTML       = '"' . $this->_getUploaderButtonHTML_Media( $sInputID, $aButtonAttributes, $bExternalSource ) . '"';
-            $_sRpeatable        = $this->getAOrB( ! empty( $abRepeatable ), 'true', 'false' );
+            $_sButtonHTML       = '"'.$this->_getUploaderButtonHTML_Media( $sInputID, $aButtonAttributes, $bExternalSource ).'"';
+            $_sRpeatable        = $this->getAOrB( !empty( $abRepeatable ), 'true', 'false' );
             $_sExternalSource   = $this->getAOrB( $bExternalSource, 'true', 'false' );
-            $_sScript                = <<<JAVASCRIPTS
+            $_sScript = <<<JAVASCRIPTS
 if ( jQuery( 'a#select_media_{$sInputID}' ).length == 0 ) {
     jQuery( 'input#{$sInputID}' ).after( $_sButtonHTML );
 }
@@ -404,7 +404,7 @@ 
                     . '/* <![CDATA[ */'
                     . $_sScript 
                     . '/* ]]> */'
-                . "</script>". PHP_EOL;
+                . "</script>".PHP_EOL;
 
         }
             /**
@@ -414,19 +414,19 @@ 
              */
             private function _getUploaderButtonHTML_Media( $sInputID, array $aButtonAttributes, $bExternalSource ) {
                     
-                $_bIsLabelSet = isset( $aButtonAttributes['data-label'] ) && $aButtonAttributes['data-label'];
+                $_bIsLabelSet = isset( $aButtonAttributes[ 'data-label' ] ) && $aButtonAttributes[ 'data-label' ];
                 $_aAttributes = $this->_getFormattedUploadButtonAttributes_Media( 
                     $sInputID, 
                     $aButtonAttributes, 
                     $_bIsLabelSet, 
                     $bExternalSource 
                 );
-                return "<a " . $this->getAttributes( $_aAttributes ) . ">"
+                return "<a ".$this->getAttributes( $_aAttributes ).">"
                         . $this->getAOrB( 
                             $_bIsLabelSet,
-                            $_aAttributes['data-label'],
+                            $_aAttributes[ 'data-label' ],
                             $this->getAOrB(
-                                strrpos( $_aAttributes['class'], 'dashicons' ),
+                                strrpos( $_aAttributes[ 'class' ], 'dashicons' ),
                                 '',
                                 $this->oMsg->get( 'select_file' )
                             )
@@ -441,26 +441,26 @@ 
                  */
                 private function _getFormattedUploadButtonAttributes_Media( $sInputID, array $aButtonAttributes, $_bIsLabelSet, $bExternalSource ) {
                    
-                    $_aAttributes           = array(
+                    $_aAttributes = array(
                             'id'        => "select_media_{$sInputID}",
                             'href'      => '#',            
-                            'data-uploader_type'            => ( string ) function_exists( 'wp_enqueue_media' ),    //  ? 1 : 0,
-                            'data-enable_external_source'   => ( string ) ( bool ) $bExternalSource,    //  ? 1 : 0, 
+                            'data-uploader_type'            => ( string ) function_exists( 'wp_enqueue_media' ), //  ? 1 : 0,
+                            'data-enable_external_source'   => ( string ) ( bool ) $bExternalSource, //  ? 1 : 0, 
                         ) 
                         + $aButtonAttributes
                         + array(
                             'title'     => $_bIsLabelSet 
-                                ? $aButtonAttributes['data-label'] 
+                                ? $aButtonAttributes[ 'data-label' ] 
                                 : $this->oMsg->get( 'select_file' ),
                             'data-label' => null,
                         );
-                    $_aAttributes['class']  = $this->getClassAttribute( 
+                    $_aAttributes[ 'class' ] = $this->getClassAttribute( 
                         'select_media button button-small ',
                         $this->getAOrB(
-                            trim( $aButtonAttributes['class'] ),
-                            $aButtonAttributes['class'],
+                            trim( $aButtonAttributes[ 'class' ] ),
+                            $aButtonAttributes[ 'class' ],
                             $this->getAOrB( 
-                                ! $_bIsLabelSet && version_compare( $GLOBALS['wp_version'], '3.8', '>=' ),
+                                !$_bIsLabelSet && version_compare( $GLOBALS[ 'wp_version' ], '3.8', '>=' ),
                                 'dashicons dashicons-portfolio',
                                 ''
                             )

admin-page-framework-loader.php

Braces +1 added lines, -1 removed lines

@@ -190,7 +190,7 @@
      * @return      void
      */ 
     static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
-        if ( ! is_admin() ) {  
+        if ( ! is_admin() ) {
             return; 
         }
         self::$_aAdminNotices[] = array(

Spacing +22 added lines, -22 removed lines

@@ -16,9 +16,9 @@ 
  */
 class AdminPageFrameworkLoader_Registry_Base {
 
-    const VERSION        = '3.8.18';    // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
+    const VERSION        = '3.8.18'; // <--- 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/taxonomy_field/AdminPageFramework_TaxonomyField_Router.php

Braces +1 added lines, -1 removed lines

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

Spacing +5 added lines, -5 removed lines

@@ -26,7 +26,7 @@ 
                 
         parent::__construct( $oProp );
 
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
         
@@ -37,7 +37,7 @@ 
         );
         
         add_action( 
-            'set_up_' . $this->oProp->sClassName,
+            'set_up_'.$this->oProp->sClassName,
             array( $this, '_replyToSetUpHooks' )
         );
         
@@ -58,11 +58,11 @@ 
             return true;
         }
         
-        if ( ! in_array( $this->oProp->sPageNow, array( 'edit-tags.php', 'term.php' ) ) ) {
+        if ( !in_array( $this->oProp->sPageNow, array( 'edit-tags.php', 'term.php' ) ) ) {
             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;
         }        
         
@@ -79,7 +79,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/post_type/AdminPageFramework_PostType.php

Spacing +7 added lines, -7 removed lines

@@ -104,7 +104,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; 
@@ -112,16 +112,16 @@ 
 
         $_sProprtyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;           
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;           
         $this->oProp = new $_sProprtyClassName( 
             $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
@@ -144,7 +144,7 @@ 
                 return $sCallerPath;
             }
             
-            if ( ! is_admin() ) {
+            if ( !is_admin() ) {
                 return null;
             }
             $_sPageNow = AdminPageFramework_Utility::getElement( $GLOBALS, 'pagenow' );

Indentation +77 added lines, -78 removed lines

@@ -25,84 +25,83 @@
     protected $_sStructureType = 'post_type';
     
     /**
-    * The constructor of the class object.
-    * 
-    * Registers necessary hooks and sets up internal properties.
-    * 
-    * <h4>Example</h4>
-    * <code>new APF_PostType( 
-    *     'apf_posts',     // post type slug
-    *       array( 
-    *           'labels' => array(
-    *               'name'               => 'Demo',
-    *               'all_items'          => __( 'Sample Posts', 'admin-page-framework-demo' ),
-    *               'singular_name'      => 'Demo',
-    *               'add_new'            => __( 'Add New', 'admin-page-framework-demo' ),
-    *               'add_new_item'       => __( 'Add New APF Post', 'admin-page-framework-demo' ),
-    *               'edit'               => __( 'Edit', 'admin-page-framework-demo' ),
-    *               'edit_item'          => __( 'Edit APF Post', 'admin-page-framework-demo' ),
-    *               'new_item'           => __( 'New APF Post', 'admin-page-framework-demo' ),
-    *               'view'               => __( 'View', 'admin-page-framework-demo' ),
-    *               'view_item'          => __( 'View APF Post', 'admin-page-framework-demo' ),
-    *               'search_items'       => __( 'Search APF Post', 'admin-page-framework-demo' ),
-    *               'not_found'          => __( 'No APF Post found', 'admin-page-framework-demo' ),
-    *               'not_found_in_trash' => __( 'No APF Post found in Trash', 'admin-page-framework-demo' ),
-    *               'parent'             => __( 'Parent APF Post', 'admin-page-framework-demo' ),
-    *               
-    *               // (framework specific)
-    *               'plugin_action_link' => __( 'APF Posts', 'admin-page-framework-demo' ), // framework specific key. [3.7.3+]
-    *           ),
-    *           'public'            => true,
-    *           'menu_position'     => 110,
-    *           'supports'          => array( 'title' ), // e.g. array( 'title', 'editor', 'comments', 'thumbnail', 'excerpt' ),    
-    *           'taxonomies'        => array( '' ),
-    *           'has_archive'       => true,
-    *           'show_admin_column' => true, // [3.5+ core] this is for custom taxonomies to automatically add the column in the listing table.
-    *           'menu_icon'         => $this->oProp->bIsAdmin 
-    *               ? ( 
-    *                   version_compare( $GLOBALS['wp_version'], '3.8', '>=' ) 
-    *                       ? 'dashicons-wordpress' 
-    *                       : plugins_url( 'asset/image/wp-logo_16x16.png', APFDEMO_FILE ) 
-    *               )
-    *               : null, // do not call the function in the front-end.
-    *               
-    *           // (framework specific) this sets the screen icon for the post type for WordPress v3.7.1 or below.
-    *           // a file path can be passed instead of a url, plugins_url( 'asset/image/wp-logo_32x32.png', APFDEMO_FILE )
-    *           'screen_icon' => dirname( APFDEMO_FILE  ) . '/asset/image/wp-logo_32x32.png', 
-    *           
-    *           // [3.5.10+] (framework specific) default: true
-    *           'show_submenu_add_new'  => true, 
-    *           
-    *           // [3.7.4+] (framework specific) default: 10
-    *           'submenu_order_manage'  => 20,
-    *           'submenu_order_addnew'  => 21,
-    * 
-    *       )     
-    * );</code>
-    * 
-    * <h4>Framework Specific Post Type Arguments</h4>
-    * In addition to the post type argument structure defined by the WordPress core, there are arguments defined by the framework.
-    * 
-    * - screen_icon - For WordPress 3.7.x or below, set an icon url or path for the 32x32 screen icon displayed in the post listing page.
-    * - show_submenu_add_new - [3.5.10+]    (boolean) Whether the sub-menu item of `Add New` should be displayed.
-    * - submenu_order_manage - [3.7.4+]     (numeric) The menu position of the `Manage` sub-menu item which gets automatically crated by the system when the admin ui is enabled. Default: `5`
-    * - submenu_order_addnew - [3.7.4+]     (numeric) The menu position of the `Manage` sub-menu item which gets automatically crated by the system when the admin ui is enabled. Default: `10`
-    *
-    * <h4>Framework Specific Post Type Label Arguments</h4>
-    * - plugin_listing_table_title_cell_link' - [3.0.6+] Deprecated [3.7.3] use the `plugin_action_link` argument instead.
-    * - plugin_action_link' - [3.7.3+] If the caller script is a plugin, this determines the label of the action link embedded in the plugin listing page (plugins.php).
-    * To disable the action link, set an empty string `''`. 
-
-    * 
-    * @since        2.0.0
-    * @since        2.1.6       Added the $sTextDomain parameter.
-    * @see          http://codex.wordpress.org/Function_Reference/register_post_type#Arguments
-    * @param        string      The post type slug.
-    * @param        array       The <a href="http://codex.wordpress.org/Function_Reference/register_post_type#Arguments">argument array</a> passed to register_post_type().
-    * @param        string      The path of the caller script. This is used to retrieve the script information to insert it into the footer. If not set, the framework tries to detect it.
-    * @param        string      The text domain of the caller script.
-    * @return       void
-    */
+     * The constructor of the class object.
+     * 
+     * Registers necessary hooks and sets up internal properties.
+     * 
+     * <h4>Example</h4>
+     * <code>new APF_PostType( 
+     *     'apf_posts',     // post type slug
+     *       array( 
+     *           'labels' => array(
+     *               'name'               => 'Demo',
+     *               'all_items'          => __( 'Sample Posts', 'admin-page-framework-demo' ),
+     *               'singular_name'      => 'Demo',
+     *               'add_new'            => __( 'Add New', 'admin-page-framework-demo' ),
+     *               'add_new_item'       => __( 'Add New APF Post', 'admin-page-framework-demo' ),
+     *               'edit'               => __( 'Edit', 'admin-page-framework-demo' ),
+     *               'edit_item'          => __( 'Edit APF Post', 'admin-page-framework-demo' ),
+     *               'new_item'           => __( 'New APF Post', 'admin-page-framework-demo' ),
+     *               'view'               => __( 'View', 'admin-page-framework-demo' ),
+     *               'view_item'          => __( 'View APF Post', 'admin-page-framework-demo' ),
+     *               'search_items'       => __( 'Search APF Post', 'admin-page-framework-demo' ),
+     *               'not_found'          => __( 'No APF Post found', 'admin-page-framework-demo' ),
+     *               'not_found_in_trash' => __( 'No APF Post found in Trash', 'admin-page-framework-demo' ),
+     *               'parent'             => __( 'Parent APF Post', 'admin-page-framework-demo' ),
+     *               
+     *               // (framework specific)
+     *               'plugin_action_link' => __( 'APF Posts', 'admin-page-framework-demo' ), // framework specific key. [3.7.3+]
+     *           ),
+     *           'public'            => true,
+     *           'menu_position'     => 110,
+     *           'supports'          => array( 'title' ), // e.g. array( 'title', 'editor', 'comments', 'thumbnail', 'excerpt' ),    
+     *           'taxonomies'        => array( '' ),
+     *           'has_archive'       => true,
+     *           'show_admin_column' => true, // [3.5+ core] this is for custom taxonomies to automatically add the column in the listing table.
+     *           'menu_icon'         => $this->oProp->bIsAdmin 
+     *               ? ( 
+     *                   version_compare( $GLOBALS['wp_version'], '3.8', '>=' ) 
+     *                       ? 'dashicons-wordpress' 
+     *                       : plugins_url( 'asset/image/wp-logo_16x16.png', APFDEMO_FILE ) 
+     *               )
+     *               : null, // do not call the function in the front-end.
+     *               
+     *           // (framework specific) this sets the screen icon for the post type for WordPress v3.7.1 or below.
+     *           // a file path can be passed instead of a url, plugins_url( 'asset/image/wp-logo_32x32.png', APFDEMO_FILE )
+     *           'screen_icon' => dirname( APFDEMO_FILE  ) . '/asset/image/wp-logo_32x32.png', 
+     *           
+     *           // [3.5.10+] (framework specific) default: true
+     *           'show_submenu_add_new'  => true, 
+     *           
+     *           // [3.7.4+] (framework specific) default: 10
+     *           'submenu_order_manage'  => 20,
+     *           'submenu_order_addnew'  => 21,
+     * 
+     *       )     
+     * );</code>
+     * 
+     * <h4>Framework Specific Post Type Arguments</h4>
+     * In addition to the post type argument structure defined by the WordPress core, there are arguments defined by the framework.
+     * 
+     * - screen_icon - For WordPress 3.7.x or below, set an icon url or path for the 32x32 screen icon displayed in the post listing page.
+     * - show_submenu_add_new - [3.5.10+]    (boolean) Whether the sub-menu item of `Add New` should be displayed.
+     * - submenu_order_manage - [3.7.4+]     (numeric) The menu position of the `Manage` sub-menu item which gets automatically crated by the system when the admin ui is enabled. Default: `5`
+     * - submenu_order_addnew - [3.7.4+]     (numeric) The menu position of the `Manage` sub-menu item which gets automatically crated by the system when the admin ui is enabled. Default: `10`
+     *
+     * <h4>Framework Specific Post Type Label Arguments</h4>
+     * - plugin_listing_table_title_cell_link' - [3.0.6+] Deprecated [3.7.3] use the `plugin_action_link` argument instead.
+     * - plugin_action_link' - [3.7.3+] If the caller script is a plugin, this determines the label of the action link embedded in the plugin listing page (plugins.php).
+     * To disable the action link, set an empty string `''`. 
+     * 
+     * @since        2.0.0
+     * @since        2.1.6       Added the $sTextDomain parameter.
+     * @see          http://codex.wordpress.org/Function_Reference/register_post_type#Arguments
+     * @param        string      The post type slug.
+     * @param        array       The <a href="http://codex.wordpress.org/Function_Reference/register_post_type#Arguments">argument array</a> passed to register_post_type().
+     * @param        string      The path of the caller script. This is used to retrieve the script information to insert it into the footer. If not set, the framework tries to detect it.
+     * @param        string      The text domain of the caller script.
+     * @return       void
+     */
     public function __construct( $sPostType, $aArguments=array(), $sCallerPath=null, $sTextDomain='admin-page-framework' ) {
         
         if ( empty( $sPostType ) ) { 

Braces +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
  * @since           2.0.0
  * @package         AdminPageFramework/Factory/PostType
  */
-abstract class AdminPageFramework_PostType extends AdminPageFramework_PostType_Controller {    
+abstract class AdminPageFramework_PostType extends AdminPageFramework_PostType_Controller {
       
     /**
      * Defines the class object structure type.
@@ -105,7 +105,7 @@ 
     */
     public function __construct( $sPostType, $aArguments=array(), $sCallerPath=null, $sTextDomain='admin-page-framework' ) {
         
-        if ( empty( $sPostType ) ) { 
+        if ( empty( $sPostType ) ) {
             return; 
         }
 

development/factory/_common/utility/AdminPageFramework_ArrayHandler.php

Doc Comments -14 removed lines

@@ -84,12 +84,6 @@ 
      * Returns the specified option value.
      * 
      * @since       3.6.0
-     * @param       string|array        $sKey1|$aKeys       The dimensional key or keys of an array of the element to extract its value.
-     * If a string is set to the first parameter, the second parameter is also expected to be a string serving as a second dimensional key. 
-     * For example, there is a multi-dimensional array, `array( 'a' => array( 'b' => 'bbb' ) )` to extract the value of the 'b' element in the second depth,
-     * set the parameter as follows. `get( 'a', 'b' )` or `get( array( 'a', 'b' ) );`
-     * @param       string|variant      $sKey2|$vDefault    The second dimensional key or the default value in case the value is not set.
-     * @param       string              $sKey3              The third dimensional key and so on.
      */
     public function get( /* $sKey1, $sKey2, $sKey3, ... OR $aKeys, $vDefault */ ) {
         
@@ -139,10 +133,6 @@ 
      * 
      * @since       3.6.0
      * @return      void
-     * @param       string|array        $asKeys       The key or keys of an array of the element to set its value.
-     * If a string is passed, it specifies the element with the set key. If an array is passed, it specifies the element with the dimensional keys.
-     * For example, `set( 'a', 'aaa' )` will set a value, `array( 'a' => 'aaa' )` and `set( array( 'a', 'b' ), 'bbb' )` will set `array( 'a' => array( 'b' => 'bbb' ) )`.
-     * @param       variant             $mValue       The value to be set.
      */
     public function set( /* $asKeys, $mValue */ ) {
         
@@ -168,10 +158,6 @@ 
      * Removes an element by dimensional keys.
      * @since       3.6.0
      * @return      void
-     * @param       string|array        $asKeys       The key or keys of an array of the element to set its value.
-     * If a string is passed, it specifies the element with the set key. If an array is passed, it specifies the element with the dimensional keys.
-     * For example, `delete( 'a' )` will unset an element of `a` in `array( 'a' => 'some value' )` so it becomes `array()`.
-     * `delete( array( 'a', 'b' ) )` will unset the element of `b` in `array( 'a' => array( 'b' => 'bbb' ) )`.
      */
     public function delete( /* $sKey1, $sKey2, $sKey3 ... OR $aKeys */ ) {
         

Spacing +9 added lines, -9 removed lines

@@ -51,14 +51,14 @@ 
      * @since       3.6.0
      * @var         array
      */
-    public $aData            = array();
+    public $aData = array();
     
     /**
      * Stores the default values.
      * @since       3.6.0
      * @var         array
      */
-    public $aDefault         = array();
+    public $aDefault = array();
     /**#@-*/
     
     /**
@@ -96,7 +96,7 @@ 
         $_aKeys     = func_get_args() + array( null );
         
         // If no key is specified, return the whole array.
-        if ( ! isset( $_aKeys[ 0 ] ) ) {
+        if ( !isset( $_aKeys[ 0 ] ) ) {
             return $this->uniteArrays(
                 $this->aData,
                 $this->aDefault
@@ -111,8 +111,8 @@ 
         
         // Now either the section ID or field ID is given. 
         return $this->getArrayValueByArrayKeys( 
-            $this->aData,   // subject array
-            $_aKeys,        // dimensional keys
+            $this->aData, // subject array
+            $_aKeys, // dimensional keys
             $this->_getDefaultValue( // default value
                 $_mDefault, 
                 $_aKeys 
@@ -145,8 +145,8 @@ 
      */
     public function set( /* $asKeys, $mValue */ ) {
         
-        $_aParameters   = func_get_args();
-        if ( ! isset( $_aParameters[ 0 ], $_aParameters[ 1 ] ) ) {
+        $_aParameters = func_get_args();
+        if ( !isset( $_aParameters[ 0 ], $_aParameters[ 1 ] ) ) {
             return;
         }
         $_asKeys        = $_aParameters[ 0 ];
@@ -174,8 +174,8 @@ 
      */
     public function delete( /* $sKey1, $sKey2, $sKey3 ... OR $aKeys */ ) {
         
-        $_aParameters   = func_get_args();
-        if ( ! isset( $_aParameters[ 0 ], $_aParameters[ 1 ] ) ) {
+        $_aParameters = func_get_args();
+        if ( !isset( $_aParameters[ 0 ], $_aParameters[ 1 ] ) ) {
             return;
         }
         $_asKeys        = $_aParameters[ 0 ];

development/factory/_common/utility/AdminPageFramework_RegisterClasses.php

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

Braces +10 added lines, -9 removed lines

@@ -181,7 +181,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 );
@@ -250,26 +251,26 @@ 
          * @internal
          * @return      void
          */        
-        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 ] );