michaeluno / admin-page-framework

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/_model/AdminPageFramework_Factory_Model___Meta_Base.php

Spacing +9 added lines, -9 removed lines

@@ -23,17 +23,17 @@ 
     /**
      * The callback function name or the callable object to retrieve meta data.
      */
-    protected $osCallable   = 'get_post_meta';
+    protected $osCallable = 'get_post_meta';
 
     /**
      * The object ID such as post ID, user ID, and term ID.
      */
-    public $iObjectID       = 0;
+    public $iObjectID = 0;
     
     /**
      * A form fieldsets array.
      */
-    public $aFieldsets      = array();
+    public $aFieldsets = array();
     
     /**
      * Sets up properties.
@@ -57,7 +57,7 @@ 
      */
     public function get() {
 
-        if ( ! $this->iObjectID ) {
+        if ( !$this->iObjectID ) {
             return array();
         }
         return $this->_getSavedDataFromFieldsets(
@@ -76,11 +76,11 @@ 
             
             $_aMetaKeys = $this->_getMetaKeys( $iObjectID );
             $_aMetaData = array();
-            foreach( $aFieldsets as $_sSectionID => $_aFieldsets ) {
+            foreach ( $aFieldsets as $_sSectionID => $_aFieldsets ) {
                 
-                if ( '_default' == $_sSectionID  ) {
-                    foreach( $_aFieldsets as $_aFieldset ) {
-                        if ( ! in_array( $_aFieldset[ 'field_id' ], $_aMetaKeys ) ) {
+                if ( '_default' == $_sSectionID ) {
+                    foreach ( $_aFieldsets as $_aFieldset ) {
+                        if ( !in_array( $_aFieldset[ 'field_id' ], $_aMetaKeys ) ) {
                             continue;
                         }
                         $_aMetaData[ $_aFieldset[ 'field_id' ] ] = call_user_func_array(
@@ -93,7 +93,7 @@ 
                         );
                     }
                 }
-                if ( ! in_array( $_sSectionID, $_aMetaKeys ) ) {
+                if ( !in_array( $_sSectionID, $_aMetaKeys ) ) {
                     continue;
                 }                
                 $_aMetaData[ $_sSectionID ] = call_user_func_array(

development/factory/_common/form/_view/generator/section/AdminPageFramework_Form_View___Generate_Section_Base.php

Spacing +2 added lines, -2 removed lines

@@ -93,8 +93,8 @@
         
         // Extract the first part as it does not have braces
         $_sName = array_shift( $aParts );
-        foreach( $aParts as $_sPart ) {
-            $_sName .= '[' . $_sPart . ']';
+        foreach ( $aParts as $_sPart ) {
+            $_sName .= '['.$_sPart.']';
         }
         return $_sName;
         

development/factory/_common/form/_model/formatter/AdminPageFramework_Form_Model___Format_EachField.php

Spacing +7 added lines, -7 removed lines

@@ -24,8 +24,8 @@ 
      * Represents the structure of the sub-field definition array.
      */
     static public $aStructure = array(
-        '_is_sub_field'                 => false,   // @todo change this key name as all the parsed field is technically a sub-field.
-        '_index'                        => 0,       // indicates the field index
+        '_is_sub_field'                 => false, // @todo change this key name as all the parsed field is technically a sub-field.
+        '_index'                        => 0, // indicates the field index
         '_is_multiple_fields'           => false,
         '_saved_value'                  => null,
         '_is_value_set_by_user'         => false,
@@ -39,8 +39,8 @@ 
         '_fields_container_id'          => '',
         '_fieldset_container_id'        => '',
         
-        '_field_object'                 => null,        // 3.6.0+
-        '_parent_field_object'          => null,        // 3.6.0+ Stores the parent field object to be accessed from the nested fields to generate id and name attribute models.
+        '_field_object'                 => null, // 3.6.0+
+        '_parent_field_object'          => null, // 3.6.0+ Stores the parent field object to be accessed from the nested fields to generate id and name attribute models.
     );
     
     /**
@@ -87,7 +87,7 @@ 
 
         $_aField = $this->aField + self::$aStructure;
         
-        $_aField[ '_is_sub_field' ]            = is_numeric( $this->isIndex ) && 0 < $this->isIndex;      // 3.5.3+
+        $_aField[ '_is_sub_field' ]            = is_numeric( $this->isIndex ) && 0 < $this->isIndex; // 3.5.3+
         $_aField[ '_index' ]                   = $this->isIndex;
 
         // 'input_id' - something like ({section id}_){field_id}__{index} e.g. my_section_id_my_field_id__0
@@ -134,8 +134,8 @@ 
                     'value'             => $_aField[ 'value' ],
                     'type'              => $_aField[ 'type' ], // text, password, etc.
                     'disabled'          => null,
-                    'data-id_model'     => $_aField[ '_input_id_model' ],    // 3.3.1+
-                    'data-name_model'   => $_aField[ '_input_name_model' ],  // 3.3.1+
+                    'data-id_model'     => $_aField[ '_input_id_model' ], // 3.3.1+
+                    'data-name_model'   => $_aField[ '_input_name_model' ], // 3.3.1+
                 )
             ),
             // this allows sub-fields with different field types to set the default key-values for the sub-field.

development/factory/_common/form/_model/formatter/AdminPageFramework_Form_Model___Format_FieldsetOutput.php

Spacing +12 added lines, -12 removed lines

@@ -33,23 +33,23 @@ 
      * @internal
      */ 
     static public $aStructure = array(       
-        '_section_index'            => null,    // 3.0.0+ - internally set to indicate the section index for repeatable sections.        
+        '_section_index'            => null, // 3.0.0+ - internally set to indicate the section index for repeatable sections.        
         
         'tag_id'                    => null,
-        '_tag_id_model'             => '',      // 3.6.0+   
+        '_tag_id_model'             => '', // 3.6.0+   
         
-        '_field_name'               => '',      // 3.6.0+   
-        '_field_name_model'         => '',      // 3.6.0+           
+        '_field_name'               => '', // 3.6.0+   
+        '_field_name_model'         => '', // 3.6.0+           
         
-        '_field_name_flat'          => '',      // 3.6.0+
-        '_field_name_flat_model'    => '',      // 3.6.0+   
+        '_field_name_flat'          => '', // 3.6.0+
+        '_field_name_flat_model'    => '', // 3.6.0+   
                 
-        '_field_address'            => '',      // 3.6.0+
-        '_field_address_model'      => '',      // 3.6.0+
+        '_field_address'            => '', // 3.6.0+
+        '_field_address_model'      => '', // 3.6.0+
                
-        '_parent_field_object'      => null,    // 3.6.0+   Assigned when a field creates a nested field.
+        '_parent_field_object'      => null, // 3.6.0+   Assigned when a field creates a nested field.
         
-        '_parent_tag_id'            => null,    // 3.8.0+   Set outside the class.
+        '_parent_tag_id'            => null, // 3.8.0+   Set outside the class.
         
     );        
     
@@ -92,7 +92,7 @@ 
         $_aFieldset = $this->aFieldset + self::$aStructure;
 
         // The section index must be set before generating a field tag id as it uses a section index.
-        $_aFieldset[ '_section_index' ]   = $this->iSectionIndex;
+        $_aFieldset[ '_section_index' ] = $this->iSectionIndex;
         $_oFieldTagIDGenerator = new AdminPageFramework_Form_View___Generate_FieldTagID( 
             $_aFieldset,
             $_aFieldset[ '_caller_object' ]->aCallbacks[ 'hfTagID' ]
@@ -126,7 +126,7 @@ 
         
         // 3.8.0+ Format nested fieldsets.
         if ( $this->hasFieldDefinitionsInContent( $_aFieldset ) ) {
-            foreach( $_aFieldset[ 'content' ] as &$_aNestedFieldset ) {
+            foreach ( $_aFieldset[ 'content' ] as &$_aNestedFieldset ) {
                 // The inline-mixed type has a string element.
                 if ( is_scalar( $_aNestedFieldset ) ) {
                     continue;

development/factory/_common/utility/AdminPageFramework_RegisterClasses.php

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/factory/_common/form/_view/AdminPageFramework_Form_View___DebugInfo.php

Spacing +4 added lines, -4 removed lines

@@ -19,7 +19,7 @@ 
  */
 class AdminPageFramework_Form_View___DebugInfo extends AdminPageFramework_FrameworkUtility {            
 
-    public $sStructureType   = '';
+    public $sStructureType = '';
     
     /**
      * @since       3.8.5
@@ -57,12 +57,12 @@ 
      */
     public function get() {
         
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return '';
         }
         
         return "<div class='admin-page-framework-info'>" 
-                . $this->oMsg->get( 'debug_info' ) . ': '
+                . $this->oMsg->get( 'debug_info' ).': '
                     . $this->getFrameworkNameVersion()
             . "</div>";
         
@@ -73,7 +73,7 @@ 
          */
         private function _shouldProceed() {
 
-            if ( ! $this->callBack( $this->aCallbacks[ 'show_debug_info' ], true ) ) {
+            if ( !$this->callBack( $this->aCallbacks[ 'show_debug_info' ], true ) ) {
                 return false;
             }        
             // For the generic admin pages, do no show debug information for each section.

Braces +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
  * @extends     AdminPageFramework_FrameworkUtility
  * @internal
  */
-class AdminPageFramework_Form_View___DebugInfo extends AdminPageFramework_FrameworkUtility {            
+class AdminPageFramework_Form_View___DebugInfo extends AdminPageFramework_FrameworkUtility {
 
     public $sStructureType   = '';
     

development/factory/admin_page/AdminPageFramework_Controller_Menu.php

Indentation +14 added lines, -14 removed lines

@@ -250,20 +250,20 @@
     }
 
     /**
-    * Adds the given link into the menu on the left sidebar of the administration panel.
-    * 
-    * @since        2.0.0
-    * @since        3.0.0       Changed the scope to public from protected.
-    * @since        3.5.0       Changed the scope to public as it was still protected.
-    * @param        string      the menu title.
-    * @param        string      the URL linked to the menu.
-    * @param        string      (optional) the <a href="http://codex.wordpress.org/Roles_and_Capabilities" target="_blank">access level</a>.
-    * @param        string      (optional) the order number. The larger it is, the lower the position it gets.
-    * @param        string      (optional) if set to false, the menu title will not be listed in the tab navigation menu at the top of the page.
-    * @access       public
-    * @return       void
-    * @internal
-    */    
+     * Adds the given link into the menu on the left sidebar of the administration panel.
+     * 
+     * @since        2.0.0
+     * @since        3.0.0       Changed the scope to public from protected.
+     * @since        3.5.0       Changed the scope to public as it was still protected.
+     * @param        string      the menu title.
+     * @param        string      the URL linked to the menu.
+     * @param        string      (optional) the <a href="http://codex.wordpress.org/Roles_and_Capabilities" target="_blank">access level</a>.
+     * @param        string      (optional) the order number. The larger it is, the lower the position it gets.
+     * @param        string      (optional) if set to false, the menu title will not be listed in the tab navigation menu at the top of the page.
+     * @access       public
+     * @return       void
+     * @internal
+     */    
     public function addSubMenuLink( array $aSubMenuLink ) {
         
         // If required keys are not set, return.

Spacing +6 added lines, -6 removed lines

@@ -76,7 +76,7 @@ 
      * @param       string      (optional) the position number that is passed to the <var>$position</var> parameter of the <a href="http://codex.wordpress.org/Function_Reference/add_menu_page">add_menu_page()</a> function.
      * @return      void
      */
-    public function setRootMenuPage( $sRootMenuLabel, $sIcon16x16=null, $iMenuPosition=null ) {
+    public function setRootMenuPage( $sRootMenuLabel, $sIcon16x16 = null, $iMenuPosition = null ) {
 
         $sRootMenuLabel = trim( $sRootMenuLabel );
         $_sSlug         = $this->_isBuiltInMenuItem( $sRootMenuLabel ); // if true, this method returns the slug
@@ -267,16 +267,16 @@ 
     public function addSubMenuLink( array $aSubMenuLink ) {
         
         // If required keys are not set, return.
-        if ( ! isset( $aSubMenuLink[ 'href' ], $aSubMenuLink[ 'title' ] ) ) { 
+        if ( !isset( $aSubMenuLink[ 'href' ], $aSubMenuLink[ 'title' ] ) ) { 
             return; 
         }
         
         // If the set URL is not valid, return.
-        if ( ! filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) { 
+        if ( !filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) { 
             return; 
         }
 
-        $_oFormatter   = new AdminPageFramework_Format_SubMenuLink( 
+        $_oFormatter = new AdminPageFramework_Format_SubMenuLink( 
             $aSubMenuLink, 
             $this,
             count( $this->oProp->aPages ) + 1
@@ -334,11 +334,11 @@ 
      */ 
     public function addSubMenuPage( array $aSubMenuPage ) {
 
-        if ( ! isset( $aSubMenuPage[ 'page_slug' ] ) ) { 
+        if ( !isset( $aSubMenuPage[ 'page_slug' ] ) ) { 
             return; 
         }
             
-        $_oFormatter   = new AdminPageFramework_Format_SubMenuPage( 
+        $_oFormatter = new AdminPageFramework_Format_SubMenuPage( 
             $aSubMenuPage,
             $this,
             count( $this->oProp->aPages ) + 1

Braces +3 added lines, -3 removed lines

@@ -267,12 +267,12 @@ 
     public function addSubMenuLink( array $aSubMenuLink ) {
         
         // If required keys are not set, return.
-        if ( ! isset( $aSubMenuLink[ 'href' ], $aSubMenuLink[ 'title' ] ) ) { 
+        if ( ! isset( $aSubMenuLink[ 'href' ], $aSubMenuLink[ 'title' ] ) ) {
             return; 
         }
         
         // If the set URL is not valid, return.
-        if ( ! filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) { 
+        if ( ! filter_var( $aSubMenuLink[ 'href' ], FILTER_VALIDATE_URL ) ) {
             return; 
         }
 
@@ -334,7 +334,7 @@ 
      */ 
     public function addSubMenuPage( array $aSubMenuPage ) {
 
-        if ( ! isset( $aSubMenuPage[ 'page_slug' ] ) ) { 
+        if ( ! isset( $aSubMenuPage[ 'page_slug' ] ) ) {
             return; 
         }
             

development/factory/_common/form/_model/formatter/AdminPageFramework_Form_Model___Format_Fields.php

Spacing +11 added lines, -11 removed lines

@@ -56,7 +56,7 @@ 
     public function get() {
 
         // Get the set value(s)
-        $_mSavedValue    = $this->_getStoredInputFieldValue( 
+        $_mSavedValue = $this->_getStoredInputFieldValue( 
             $this->aField, 
             $this->aOptions 
         );
@@ -116,9 +116,9 @@ 
              * @return      void
              */
             private function _divideMainAndSubFields( $aField, array &$aFirstField, array &$aSubFields ) {
-                foreach( $aField as $_nsIndex => $_mFieldElement ) {
+                foreach ( $aField as $_nsIndex => $_mFieldElement ) {
                     if ( is_numeric( $_nsIndex ) ) {
-                        $aSubFields[] = $_mFieldElement;
+                        $aSubFields[ ] = $_mFieldElement;
                     } else {
                         $aFirstField[ $_nsIndex ] = $_mFieldElement;
                     }
@@ -144,7 +144,7 @@ 
                 // We are collecting elements from the second sub-field.
                 unset( $_aSavedValues[ 0 ] ); 
                    
-                foreach( $_aSavedValues as $_iIndex => $vValue ) {
+                foreach ( $_aSavedValues as $_iIndex => $vValue ) {
                     $aSubFields[ $_iIndex - 1 ] = isset( $aSubFields[ $_iIndex - 1 ] ) && is_array( $aSubFields[ $_iIndex - 1 ] )
                         ? $aSubFields[ $_iIndex - 1 ] 
                         : array();     
@@ -159,7 +159,7 @@ 
              */
             private function _fillSubFields( array &$aSubFields, array $aFirstField ) {
                         
-                foreach( $aSubFields as &$_aSubField ) {
+                foreach ( $aSubFields as &$_aSubField ) {
                     
                     // Evacuate the label element which should not be merged.
                     $_aLabel = $this->getElement( 
@@ -190,15 +190,15 @@ 
          
             // Determine whether the elements are saved in an array.
             // $_bHasSubFields = count( $aFields ) > 1 || $aField[ 'repeatable' ] || $aField[ 'sortable' ];
-            if ( ! $this->hasSubFields( $aFields, $aField ) ) {
+            if ( !$this->hasSubFields( $aFields, $aField ) ) {
                 $aFields[ 0 ][ '_saved_value' ] = $mSavedValue;
                 $aFields[ 0 ][ '_is_multiple_fields' ] = false;
                 return;                    
             }
      
-            foreach( $aFields as $_iIndex => &$_aThisField ) {
+            foreach ( $aFields as $_iIndex => &$_aThisField ) {
                 $_aThisField[ '_saved_value' ]        = $this->getElement( $mSavedValue, $_iIndex, null );
-                $_aThisField[ '_subfield_index' ]     = $_iIndex;   // 3.8.0+
+                $_aThisField[ '_subfield_index' ]     = $_iIndex; // 3.8.0+
                 $_aThisField[ '_is_multiple_fields' ] = true;
             }
     
@@ -212,7 +212,7 @@ 
          * @since       3.6.0       Moved from `AdminPageFramework_FieldDefinition`.
          */
         private function _setFieldsValue( &$aFields ) {
-            foreach( $aFields as &$_aField ) {
+            foreach ( $aFields as &$_aField ) {
                 $_aField[ '_is_value_set_by_user' ] = isset( $_aField[ 'value' ] );
                 $_aField[ 'value' ]                 = $this->_getSetFieldValue( $_aField );
             }
@@ -257,10 +257,10 @@ 
          */
         private function _getStoredInputFieldValue( $aField, $aOptions ) {    
 
-            $_aFieldPath   = $aField[ '_field_path_array' ];
+            $_aFieldPath = $aField[ '_field_path_array' ];
         
             // If a section is not set, check the first dimension element.
-            if ( ! isset( $aField[ 'section_id' ] ) || '_default' === $aField[ 'section_id' ] ) {
+            if ( !isset( $aField[ 'section_id' ] ) || '_default' === $aField[ 'section_id' ] ) {
                 return $this->getElement( 
                     $aOptions, 
                     $_aFieldPath, // $aField[ 'field_id' ],  // @todo this may have to be a field path instead of field id.

Braces +1 added lines, -1 removed lines

@@ -254,7 +254,7 @@
          * @since       3.7.0       Changed the `_field_type` element to `_structure_type`.
          * @return      null|string|array
          */
-        private function _getStoredInputFieldValue( $aField, $aOptions ) {    
+        private function _getStoredInputFieldValue( $aField, $aOptions ) {
 
             $_aFieldPath   = $aField[ '_field_path_array' ];