michaeluno / admin-page-framework

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

development/utility/class_tester/AdminPageFramework_ClassTester.php

Doc Comments +3 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
      * Creates an object instance with dynamic parameters.
      * 
      * @since       3.7.10
-     * @param       string      $sCalssName     The class name for testing.
+     * @param       string      $sClassName     The class name for testing.
      * @param       array       $aParameters    The parameters to pass to the constructor of the class set in the first parameter.
      * @return      object      An object instance of the class specified in the first parameter.
      */
@@ -61,7 +61,7 @@ 
      * 
      * @since       3.7.10
      * @param       object      $oClass         The subject class object.
-     * @param       string      $sMathodName    The subject method name.
+     * @param       string      $sMethodName    The subject method name.
      * @param       array       $aParameters    The parameters to pass to the method set in the second parameter.    
      * @remark      This supports private methods to be executed.
      */
@@ -87,6 +87,7 @@ 
     }
         /**
          * @since       3.7.10
+         * @param string $sClassName
          * @return      object
          * @internal
          */

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
      * @param       array       $aParameters    The parameters to pass to the constructor of the class set in the first parameter.
      * @return      object      An object instance of the class specified in the first parameter.
      */
-    static public function getInstance( $sClassName, array $aParameters=array() ) {
+    static public function getInstance( $sClassName, array $aParameters = array() ) {
         
         $_oReflection = new ReflectionClass( $sClassName );
         return $_oReflection->newInstanceArgs( $aParameters );               
@@ -69,7 +69,7 @@ 
         // For PHP 5.2.x or below
         if ( version_compare( phpversion(), '<', '5.3.0' ) ) {
             trigger_error(
-                'Program Name' . ': ' 
+                'Program Name'.': ' 
                     . sprintf( 
                         'The method cannot run with your PHP version: %1$s',
                         phpversion()

development/document/factory/AdminPageFramework_Documentation.php

Indentation -1 removed lines

@@ -196,7 +196,6 @@
  * For common hooks throughout the other factory components, see [Base Factory](./package-AdminPageFramework.Common.Factory.html).
  * 
  * <h3>Factory Specific Hooks</h3>
-
  * <h4> Action Hooks</h4>
  * <ul>
  *     <li>**load_{page slug}** – [2.1.0+] triggered when the framework's page is loaded before the header gets sent. This will not be triggered in the admin pages that are not registered by the framework. The first parameter: class object [3.1.2+].</li>

development/factory/user_meta/_model/AdminPageFramework_UserMeta_Model___UserMeta.php

Spacing +1 added lines, -1 removed lines

@@ -21,6 +21,6 @@
     /**
      * The callback function name or the callable object to retrieve meta data.
      */
-    protected $osCallable   = 'get_user_meta'; 
+    protected $osCallable = 'get_user_meta'; 
         
 }

development/factory/_common/_abstract/AdminPageFramework_Factory_View.php

Spacing +12 added lines, -12 removed lines

@@ -41,7 +41,7 @@ 
          * @return      string      the input id attribute
          */    
         public function _replyToGetSectionName( /* $sSectionName, $aSectionset */ ) {
-            $_aParams = func_get_args() + array( null, null, );
+            $_aParams = func_get_args() + array( null, null,);
             return $_aParams[ 0 ];
         }
         
@@ -69,8 +69,8 @@ 
          * @since       3.6.0
          * @return      string
          */
-        public function _replyToGetFieldNameAttribute( /* $sFieldName, $aFieldset */ )  {            
-            $_aParams = func_get_args() + array( null, null, );
+        public function _replyToGetFieldNameAttribute( /* $sFieldName, $aFieldset */ ) {            
+            $_aParams = func_get_args() + array( null, null,);
             return $_aParams[ 0 ];                        
         }
         /**
@@ -80,7 +80,7 @@ 
          * @return      string
          */        
         public function _replyToGetFlatFieldName( /* $sFieldName, $aFieldset */ ) {            
-            $_aParams = func_get_args() + array( null, null, );
+            $_aParams = func_get_args() + array( null, null,);
             return $_aParams[ 0 ];            
         }
         
@@ -102,7 +102,7 @@ 
          * @return      string      the flat input name attribute
          */    
         public function _replyToGetFlatInputName( /* $sFlatNameAttribute, $aField, $sKey */ ) {
-            $_aParams   = func_get_args() + array( null, null, null );
+            $_aParams = func_get_args() + array( null, null, null );
             return $_aParams[ 0 ];
         }
 
@@ -144,14 +144,14 @@ 
                 'if'            => true,
                 'capability'    => '',
             );
-            if ( ! $aElementDefinition[ 'if' ] ) {
+            if ( !$aElementDefinition[ 'if' ] ) {
                 return false;
             }
             // For front-end forms that allow guests, the capability level can be empty. In that case, return true.
-            if ( ! $aElementDefinition[ 'capability' ] ) {
+            if ( !$aElementDefinition[ 'capability' ] ) {
                 return true;
             }
-            if ( ! current_user_can( $aElementDefinition[ 'capability' ] ) ) {
+            if ( !current_user_can( $aElementDefinition[ 'capability' ] ) ) {
                 return false;
             }            
             return $bDefault;
@@ -187,7 +187,7 @@ 
             $this,
             array( 
                 // section_{instantiated class name}_{section id}
-                'section_head_' . $this->oProp->sClassName . '_' . $aSectionset[ 'section_id' ] 
+                'section_head_'.$this->oProp->sClassName.'_'.$aSectionset[ 'section_id' ] 
             ), 
             $sSectionDescription
         );
@@ -204,15 +204,15 @@ 
      */
     public function _replyToGetFieldOutput( $sFieldOutput, $aFieldset ) {
 
-        $_sSectionPart  = $this->oUtil->getAOrB(
+        $_sSectionPart = $this->oUtil->getAOrB(
             isset( $aFieldset[ 'section_id' ] ) && '_default' !== $aFieldset[ 'section_id' ],
-            '_' . $aFieldset[ 'section_id' ],
+            '_'.$aFieldset[ 'section_id' ],
             ''
         );
         return $this->oUtil->addAndApplyFilters(
             $this,
             array( 
-                'field_' . $this->oProp->sClassName . $_sSectionPart . '_' . $aFieldset[ 'field_id' ]
+                'field_'.$this->oProp->sClassName.$_sSectionPart.'_'.$aFieldset[ 'field_id' ]
             ),
             $sFieldOutput,
             $aFieldset // the field array

Braces +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
          * @since       3.6.0
          * @return      string
          */
-        public function _replyToGetFieldNameAttribute( /* $sFieldName, $aFieldset */ )  {            
+        public function _replyToGetFieldNameAttribute( /* $sFieldName, $aFieldset */ ) {
             $_aParams = func_get_args() + array( null, null, );
             return $_aParams[ 0 ];                        
         }
@@ -79,7 +79,7 @@ 
          * @since       3.6.0
          * @return      string
          */        
-        public function _replyToGetFlatFieldName( /* $sFieldName, $aFieldset */ ) {            
+        public function _replyToGetFlatFieldName( /* $sFieldName, $aFieldset */ ) {
             $_aParams = func_get_args() + array( null, null, );
             return $_aParams[ 0 ];            
         }

development/factory/_common/_abstract/_controller/AdminPageFramework_Link_Base.php

Spacing +19 added lines, -19 removed lines

@@ -26,9 +26,9 @@ 
     /**
      * Sets up hooks and properties.
      */ 
-    public function __construct( $oProp, $oMsg=null ) {
+    public function __construct( $oProp, $oMsg = null ) {
         
-        if ( ! $this->_isLoadable( $oProp ) ) {
+        if ( !$this->_isLoadable( $oProp ) ) {
             return;
         }
                 
@@ -40,7 +40,7 @@ 
         // Add an action link in the plugin listing page
         if ( $this->_shouldSetPluginActionLinks() ) {            
             add_filter( 
-                'plugin_action_links_' . plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] ),
+                'plugin_action_links_'.plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] ),
                 array( $this, '_replyToAddSettingsLinkInPluginListingPage' ), 
                 20     // set a lower priority so that the link will be embedded at the beginning ( the most left hand side ).
             );     
@@ -55,13 +55,13 @@ 
          * @return      boolean
          */
         private function _isLoadable( $oProp ) {
-            if ( ! $oProp->bIsAdmin ) { 
+            if ( !$oProp->bIsAdmin ) { 
                 return false;
             }
             if ( $oProp->bIsAdminAjax ) {
                 return false;
             }
-            return ! $this->hasBeenCalled( 'links_' . $oProp->sClassName );
+            return !$this->hasBeenCalled( 'links_'.$oProp->sClassName );
         }
         
     /**
@@ -72,11 +72,11 @@ 
     protected function _shouldSetPluginActionLinks() {
         
         // It is possible that the sub-objects are not set when the class is considered not loadable.
-        if ( ! isset( $this->oProp ) ) {
+        if ( !isset( $this->oProp ) ) {
             return false;
         }
         
-        if ( ! in_array( $this->oProp->sPageNow, array( 'plugins.php' ) ) ) {
+        if ( !in_array( $this->oProp->sPageNow, array( 'plugins.php' ) ) ) {
             return false;
         }
         return 'plugin' === $this->oProp->aScriptInfo[ 'sType' ];
@@ -143,7 +143,7 @@ 
                         array(
                             'href'      => $aScriptInfo[ 'sURI' ],
                             'target'    => '_blank',
-                            'title'     => $aScriptInfo[ 'sName' ] . $_sVersion . $_sDescription 
+                            'title'     => $aScriptInfo[ 'sName' ].$_sVersion.$_sDescription 
                         ), 
                         $aScriptInfo[ 'sName' ] 
                     )    
@@ -165,13 +165,13 @@ 
                 $_sAuthorInfo = $this->getAOrB(
                     empty( $aScriptInfo[ 'sAuthor' ] ),
                     $_sAuthorInfo,
-                    ' by ' . $_sAuthorInfo
+                    ' by '.$_sAuthorInfo
                 );
                 
                 // Enclosing the output in a span tag as the outer element is a '<p>' tag. So this cannot be div.
                 // 3.5.7+ Added the class attribute for acceptance testing
                 return "<span class='apf-script-info'>"  
-                        . $_sPluginInfo . $_sAuthorInfo
+                        . $_sPluginInfo.$_sAuthorInfo
                     . "</span>";
           
             }
@@ -202,7 +202,7 @@ 
                         array(
                             'href'      => $aScriptInfo[ 'sURI' ],
                             'target'    => '_blank',
-                            'title'     => $aScriptInfo[ 'sName' ] . $_sVersion . $_sDescription,
+                            'title'     => $aScriptInfo[ 'sName' ].$_sVersion.$_sDescription,
                         ), 
                         $aScriptInfo[ 'sName' ]
                     )                   
@@ -212,16 +212,16 @@ 
                 // 3.5.7+ added the 'apf-credit' class attribute for acceptance testing
                 // 3.7.0+  added the footer-thankyou id attribute.
                 return "<span class='apf-credit' id='footer-thankyou'>" 
-                    . $this->oMsg->get( 'powered_by' ) . '&nbsp;' 
+                    . $this->oMsg->get( 'powered_by' ).'&nbsp;' 
                     . $_sLibraryInfo
                     . ",&nbsp;"
-                    . $this->oMsg->get( 'and' ) . '&nbsp;' 
+                    . $this->oMsg->get( 'and' ).'&nbsp;' 
                     . $this->getHTMLTag( 
                         'a', 
                         array(
                             'href'      => 'https://wordpress.org',
                             'target'    => '_blank',
-                            'title'     => 'WordPress ' . $GLOBALS[ 'wp_version' ]
+                            'title'     => 'WordPress '.$GLOBALS[ 'wp_version' ]
                         ), 
                         'WordPress'
                     )
@@ -237,7 +237,7 @@ 
          */
         protected function _setFooterHooks() {
             
-            add_filter( 'admin_footer_text' , array( $this, '_replyToAddInfoInFooterLeft' ) );
+            add_filter( 'admin_footer_text', array( $this, '_replyToAddInfoInFooterLeft' ) );
             add_filter( 'update_footer', array( $this, '_replyToAddInfoInFooterRight' ), 11 );
             
         }             
@@ -249,7 +249,7 @@ 
              * @callback    filter      admin_footer_text
              * @internal
              */ 
-            public function _replyToAddInfoInFooterLeft( $sLinkHTML='' ) {
+            public function _replyToAddInfoInFooterLeft( $sLinkHTML = '' ) {
 
                 $sLinkHTML = empty( $this->oProp->aScriptInfo[ 'sName' ] )
                     ? $sLinkHTML
@@ -257,7 +257,7 @@ 
              
                 return $this->addAndApplyFilters( 
                     $this->oProp->oCaller, 
-                    'footer_left_' . $this->oProp->sClassName, 
+                    'footer_left_'.$this->oProp->sClassName, 
                     $sLinkHTML
                 );
              
@@ -270,10 +270,10 @@ 
              * @callback    filter      admin_footer_text
              * @internal
              */     
-            public function _replyToAddInfoInFooterRight( $sLinkHTML='' ) {
+            public function _replyToAddInfoInFooterRight( $sLinkHTML = '' ) {
                 return $this->addAndApplyFilters( 
                     $this->oProp->oCaller, 
-                    'footer_right_' . $this->oProp->sClassName, 
+                    'footer_right_'.$this->oProp->sClassName, 
                     $this->oProp->aFooterInfo[ 'sRight' ]
                 );                
             }       

Braces +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
         add_action( 'in_admin_footer', array( $this, '_replyToSetFooterInfo' ) );
         
         // Add an action link in the plugin listing page
-        if ( $this->_shouldSetPluginActionLinks() ) {            
+        if ( $this->_shouldSetPluginActionLinks() ) {
             add_filter( 
                 'plugin_action_links_' . plugin_basename( $this->oProp->aScriptInfo[ 'sPath' ] ),
                 array( $this, '_replyToAddSettingsLinkInPluginListingPage' ), 
@@ -55,7 +55,7 @@ 
          * @return      boolean
          */
         private function _isLoadable( $oProp ) {
-            if ( ! $oProp->bIsAdmin ) { 
+            if ( ! $oProp->bIsAdmin ) {
                 return false;
             }
             if ( $oProp->bIsAdminAjax ) {