michaeluno / admin-page-framework

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/admin-page-framework.php

Spacing +10 added lines, -10 removed lines

@@ -10,7 +10,7 @@ 
  * @package     AdminPageFramework
  */
 
-if ( ! class_exists( 'AdminPageFramework_Registry', false ) ) :
+if ( !class_exists( 'AdminPageFramework_Registry', false ) ) :
 /**
  * Facilitates WordPress plugin and theme development.
  *
@@ -61,7 +61,7 @@ 
 final class AdminPageFramework_Registry extends AdminPageFramework_Registry_Base {
 
     const TEXT_DOMAIN        = 'admin-page-framework';
-    const TEXT_DOMAIN_PATH   = '/language';  // not used at the moment
+    const TEXT_DOMAIN_PATH   = '/language'; // not used at the moment
 
     /**
      * Indicates whether the framework is loaded from the minified version or not.
@@ -103,11 +103,11 @@ 
      * Sets up static properties.
      * @return      void
      */
-    static public function setUp( $sFilePath=__FILE__ ) {
+    static public function setUp( $sFilePath = __FILE__ ) {
 
         self::$sFilePath                = $sFilePath;
         self::$sDirPath                 = dirname( self::$sFilePath );
-        self::$sIncludeClassListPath    = self::$sDirPath . '/admin-page-framework-include-class-list.php';
+        self::$sIncludeClassListPath    = self::$sDirPath.'/admin-page-framework-include-class-list.php';
         self::$aClassFiles              = self::_getClassFilePathList( self::$sIncludeClassListPath );
         self::$sAutoLoaderPath          = isset( self::$aClassFiles[ 'AdminPageFramework_RegisterClasses' ] )
             ? self::$aClassFiles[ 'AdminPageFramework_RegisterClasses' ]
@@ -122,7 +122,7 @@ 
          * @return      array
          */
         static private function _getClassFilePathList( $sInclusionClassListPath ) {
-            $aClassFiles = array();    // this will be updated if the inclusion below is successful.
+            $aClassFiles = array(); // this will be updated if the inclusion below is successful.
             include( $sInclusionClassListPath );
             return $aClassFiles;
         }
@@ -135,11 +135,11 @@ 
      */
     static public function getVersion() {
 
-        if ( ! isset( self::$sAutoLoaderPath ) ) {
-            trigger_error( self::NAME . ': ' . ' : ' . sprintf( __( 'The method is called too early. Perform <code>%2$s</code> earlier.', 'admin-page-framework' ), __METHOD__, 'setUp()' ), E_USER_WARNING );
+        if ( !isset( self::$sAutoLoaderPath ) ) {
+            trigger_error( self::NAME.': '.' : '.sprintf( __( 'The method is called too early. Perform <code>%2$s</code> earlier.', 'admin-page-framework' ), __METHOD__, 'setUp()' ), E_USER_WARNING );
             return self::VERSION;
         }
-        $_aMinifiedVesionSuffix     = array(
+        $_aMinifiedVesionSuffix = array(
             0 => '',
             1 => '.min',
         );
@@ -168,7 +168,7 @@ 
 }
 endif;
 
-if ( ! class_exists( 'AdminPageFramework_Bootstrap', false ) ) :
+if ( !class_exists( 'AdminPageFramework_Bootstrap', false ) ) :
 /**
  * Loads the Admin Page Framework library.
  *
@@ -189,7 +189,7 @@ 
      */
     public function __construct( $sLibraryPath ) {
 
-        if ( ! $this->_isLoadable() ) {
+        if ( !$this->_isLoadable() ) {
             return;
         }
 

development/factory/_common/form/field_type/AdminPageFramework_FieldType_inline_mixed.php

Spacing +2 added lines, -2 removed lines

@@ -218,8 +218,8 @@
      * @remark $_aDefaultKeys holds shared default key-values defined in the base class.
      */
     protected $aDefaultKeys = array(
-        'label_min_width'  => '',       // disabled as the embedded elements are all inline.
-        'show_debug_info'  => false,    // 3.8.8+ @todo Examine why this value does not override the default value of field definition arguments and if possible and appropriate, override it.
+        'label_min_width'  => '', // disabled as the embedded elements are all inline.
+        'show_debug_info'  => false, // 3.8.8+ @todo Examine why this value does not override the default value of field definition arguments and if possible and appropriate, override it.
     );
 
     

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

development/factory/_common/form/_view/script/AdminPageFramework_Form_View___Script_RepeatableSection.php

Indentation +1 added lines, -1 removed lines

@@ -520,7 +520,7 @@
             return self::getAttributes(
                     array(
                     'class'     => 'repeatable-section-remove-button button-secondary '
-                                   . 'repeatable-section-button button button-large',
+                                    . 'repeatable-section-button button button-large',
                     'title'     => $oMsg->get( 'remove_section' ),
                     'style'     => $iSectionCount <= 1
                         ? 'display:none'

Spacing +12 added lines, -12 removed lines

@@ -446,7 +446,7 @@ 
         if ( empty( $asArguments ) ) {
             return '';
         }
-        if ( self::hasBeenCalled( 'repeatable_section_' . $sContainerTagID ) ) {
+        if ( self::hasBeenCalled( 'repeatable_section_'.$sContainerTagID ) ) {
             return '';
         }
 
@@ -454,7 +454,7 @@ 
         $_oFormatter    = new AdminPageFramework_Form_Model___Format_RepeatableSection( $asArguments, $oMsg );
         $_aArguments    = $_oFormatter->get();
         $_sButtons      = self::___getRepeatableSectionButtons( $_aArguments, $oMsg, $sContainerTagID, $iSectionCount );
-        $_sButtonsHTML  = '"' . $_sButtons . '"';
+        $_sButtonsHTML  = '"'.$_sButtons.'"';
         $_aJSArray      = json_encode( $_aArguments );
         $_sScript       = <<<JAVASCRIPTS
 jQuery( document ).ready( function() {
@@ -500,17 +500,17 @@ 
                 $_sIconAdd      = "<span class='dashicons dashicons-plus-alt2'></span>";
             }
             return "<div class='admin-page-framework-repeatable-section-buttons-outer-container'>"
-                . "<div " . self::___getContainerAttributes( $_aArguments, $oMsg ) . ' >'
-                    . "<a " . self::___getRemoveButtonAttributes( $sContainerTagID, $oMsg, $iSectionCount ) . ">"
+                . "<div ".self::___getContainerAttributes( $_aArguments, $oMsg ).' >'
+                    . "<a ".self::___getRemoveButtonAttributes( $sContainerTagID, $oMsg, $iSectionCount ).">"
                         . $_sIconRemove
                     . "</a>"
-                    . "<a " . self::___getAddButtonAttributes( $sContainerTagID, $oMsg, $_aArguments ) . ">"
+                    . "<a ".self::___getAddButtonAttributes( $sContainerTagID, $oMsg, $_aArguments ).">"
                         . $_sIconAdd
                     . "</a>"
                 . "</div>"
             . "</div>"
             . AdminPageFramework_Form_Utility::getModalForDisabledRepeatableElement(
-                    'repeatable_section_disabled_' . $sContainerTagID,
+                    'repeatable_section_disabled_'.$sContainerTagID,
                     $_aArguments[ 'disabled' ]
                 );
         }
@@ -526,12 +526,12 @@ 
                     empty( $aArguments[ 'disabled' ] ) ? '' : 'disabled'
                 ),
             );
-            unset( $aArguments[ 'disabled' ][ 'message' ] );    // this element can contain HTML tags.
+            unset( $aArguments[ 'disabled' ][ 'message' ] ); // this element can contain HTML tags.
             // Needs to remove it if it is empty as its data attribute will be checked in the JavaScript script.
             if ( empty( $aArguments[ 'disabled' ] ) ) {
                 unset( $aArguments[ 'disabled' ] );
             }
-            return self::getAttributes( $_aAttriubtes ) . ' ' . self::getDataAttributes( $aArguments );
+            return self::getAttributes( $_aAttriubtes ).' '.self::getDataAttributes( $aArguments );
         }
         /**
          * @return  string
@@ -562,10 +562,10 @@ 
                         . 'repeatable-section-button button button-large',
                     'title'     => $oMsg->get( 'add_section' ),
                     'data-id'   => $sContainerTagID,
-                    'href'      => ! empty( $aArguments[ 'disabled' ] )
-                        ? '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                            . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                            . '&inlineId=' . 'repeatable_section_disabled_' . $sContainerTagID
+                    'href'      => !empty( $aArguments[ 'disabled' ] )
+                        ? '#TB_inline?width='.$aArguments[ 'disabled' ][ 'box_width' ]
+                            . '&height='.$aArguments[ 'disabled' ][ 'box_height' ]
+                            . '&inlineId='.'repeatable_section_disabled_'.$sContainerTagID
                         : null,
                 )
             );

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

Spacing +5 added lines, -5 removed lines

@@ -36,8 +36,8 @@ 
      * @since   3.8.13
      */
     static protected $_aStructure_Disabled = array(
-        'message'    => 'The ability of repeating sections is disabled.',   // will be reassigned
-        'caption'    => 'Warning',  // will be reassigned
+        'message'    => 'The ability of repeating sections is disabled.', // will be reassigned
+        'caption'    => 'Warning', // will be reassigned
         'box_width'  => 300,
         'box_height' => 72,
     );
@@ -54,7 +54,7 @@ 
      */
     public function __construct( /* array $asArguments, $oMsg */ ) {
 
-        $_aParameters       = func_get_args() + array(
+        $_aParameters = func_get_args() + array(
             $this->_aArguments,
             null
         );
@@ -73,8 +73,8 @@ 
     public function get() {
 
         $_aArguments = $this->_aArguments + self::$_aStructure;
-        unset( $_aArguments[ 0 ] );   // remove the 0 index element converted from `'repeatable   => 'true',`.
-        if ( ! empty( $_aArguments[ 'disabled' ] ) ) {
+        unset( $_aArguments[ 0 ] ); // remove the 0 index element converted from `'repeatable   => 'true',`.
+        if ( !empty( $_aArguments[ 'disabled' ] ) ) {
             $_aArguments[ 'disabled' ] = $_aArguments[ 'disabled' ] + array(
                 'message'    => $this->_getDefaultMessage(),
                 'caption'    => $this->_oMsg->get( 'warning_caption' ),

development/factory/_common/_abstract/AdminPageFramework_Factory_Model.php

Braces +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
      * @since       3.7.0
      * @return      array       The modified section-sets definition array.
      */    
-    public function _replyToModifySectionsets( $aSectionsets ) {    
+    public function _replyToModifySectionsets( $aSectionsets ) {
         
         return $this->oUtil->addAndApplyFilter( 
             $this,  // caller factory object
@@ -364,7 +364,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }    
 

Spacing +13 added lines, -13 removed lines

@@ -31,7 +31,7 @@ 
         
         add_filter(
             // 'field_types_admin_page_framework',
-            'field_types_' . $oProp->sClassName,
+            'field_types_'.$oProp->sClassName,
             array( $this, '_replyToFilterFieldTypeDefinitions' )
         );
         
@@ -53,7 +53,7 @@ 
             'title'      => null,
             'help_aside' => null,
         );
-        if ( ! $aFieldset[ 'help' ] ) {
+        if ( !$aFieldset[ 'help' ] ) {
             return;
         }
         $this->oHelpPane->_addHelpTextForFormFields( 
@@ -73,9 +73,9 @@ 
     public function _replyToFilterFieldTypeDefinitions( $aFieldTypeDefinitions ) {
         
         // Not triggering `__call()` as the filter is fired manually in the form class.
-        if ( method_exists( $this, 'field_types_' . $this->oProp->sClassName ) ) {
+        if ( method_exists( $this, 'field_types_'.$this->oProp->sClassName ) ) {
             return call_user_func_array(
-                array( $this, 'field_types_' . $this->oProp->sClassName ),
+                array( $this, 'field_types_'.$this->oProp->sClassName ),
                 array( $aFieldTypeDefinitions )
             );
         }
@@ -96,7 +96,7 @@ 
     public function _replyToModifySectionsets( $aSectionsets ) {    
         
         return $this->oUtil->addAndApplyFilter( 
-            $this,  // caller factory object
+            $this, // caller factory object
             "sections_{$this->oProp->sClassName}", 
             $aSectionsets
         );
@@ -116,7 +116,7 @@ 
     public function _replyToModifyFieldsets( $aFieldsets, $aSectionsets ) {
 
         // Apply filters to added field-sets
-        foreach( $aFieldsets as $_sSectionPath => $_aFields ) {
+        foreach ( $aFieldsets as $_sSectionPath => $_aFields ) {
             $_aSectionPath  = explode( '|', $_sSectionPath );
             $_sFilterSuffix = implode( '_', $_aSectionPath );
             $aFieldsets[ $_sSectionPath ] = $this->oUtil->addAndApplyFilter(
@@ -125,7 +125,7 @@ 
                 $_aFields
             ); 
         }
-        $aFieldsets =  $this->oUtil->addAndApplyFilter( 
+        $aFieldsets = $this->oUtil->addAndApplyFilter( 
             $this,
             "fields_{$this->oProp->sClassName}",
             $aFieldsets
@@ -195,14 +195,14 @@ 
          */
         private function _getHookNameByFieldsetAndPrefix( $sPrefix, $aFieldset ) {
             
-            $_sFieldPart    = '_' . implode( '_', $aFieldset[ '_field_path_array' ] );
+            $_sFieldPart    = '_'.implode( '_', $aFieldset[ '_field_path_array' ] );
             $_sSectionPart  = implode( '_', $aFieldset[ '_section_path_array' ] );
             $_sSectionPart  = $this->oUtil->getAOrB(
                 '_default' === $_sSectionPart,
                 '',
-                '_' . $_sSectionPart
+                '_'.$_sSectionPart
             );
-            return $sPrefix . $this->oProp->sClassName . $_sSectionPart . $_sFieldPart;
+            return $sPrefix.$this->oProp->sClassName.$_sSectionPart.$_sFieldPart;
             
         }
                
@@ -272,9 +272,9 @@ 
     public function _replyToGetSavedFormData() {
         
         // Must update the property with the filtered value.
-        $this->oProp->aOptions =  $this->oUtil->addAndApplyFilter(
+        $this->oProp->aOptions = $this->oUtil->addAndApplyFilter(
             $this, // the caller factory object
-            'options_' . $this->oProp->sClassName,
+            'options_'.$this->oProp->sClassName,
             $this->oProp->aOptions      // subject value to be filtered
         );
         return $this->oProp->aOptions;
@@ -364,7 +364,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }