michaeluno / admin-page-framework

development/factory/_abstract/form/_model/AdminPageFramework_Form_Model___SetFieldResources.php

Spacing +12 added lines, -12 removed lines

@@ -89,16 +89,16 @@ 
                 'AdminPageFramework_Form_View___CSS_CollapsibleSection',
                 'AdminPageFramework_Form_View___CSS_FieldError',
             );
-            foreach( $_aClassNames as $_sClassName ) {
+            foreach ( $_aClassNames as $_sClassName ) {
                 $_oCSS = new $_sClassName;
-                $this->aResources[ 'inline_styles' ][] = $_oCSS->get();
+                $this->aResources[ 'inline_styles' ][ ] = $_oCSS->get();
             }
             $_aClassNamesForIE = array(
                 'AdminPageFramework_Form_View___CSS_CollapsibleSectionIE',
             );
-            foreach( $_aClassNames as $_sClassName ) {
+            foreach ( $_aClassNames as $_sClassName ) {
                 $_oCSS = new $_sClassName;
-                $this->aResources[ 'inline_styles_ie' ][] = $_oCSS->get();
+                $this->aResources[ 'inline_styles_ie' ][ ] = $_oCSS->get();
             }
             
         }
@@ -118,10 +118,10 @@ 
 
             // Parse all added fieldsets and check associated resources.
 // @todo Find a way to retrieve the section id for nested sections and fields.            
-            foreach( $this->aFieldsets as $_sSecitonID => $_aFieldsets ) {
+            foreach ( $this->aFieldsets as $_sSecitonID => $_aFieldsets ) {
                 
                 $_bIsSubSectionLoaded = false;
-                foreach( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField )  {
+                foreach ( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField ) {
 // @todo Examine if this structure is correct or not. 
 // It may not be necessary to check the sub-section dimensions as this is not the saved options array.
                     // if it's a sub-section
@@ -132,7 +132,7 @@ 
                             continue;
                         }
                         $_bIsSubSectionLoaded = true;
-                        foreach( $_aSubSectionOrField as $_aField ) {
+                        foreach ( $_aSubSectionOrField as $_aField ) {
                             $this->_setFieldResources( $_aField );     
                         }
                         continue;
@@ -157,7 +157,7 @@ 
             private function _setFieldResources( array $aFieldset ) {
                                 
                 // Check the field conditions here.
-                if ( ! $this->_isFieldsetAllowed( $aFieldset ) ) {
+                if ( !$this->_isFieldsetAllowed( $aFieldset ) ) {
                     return;
                 }
                 
@@ -185,7 +185,7 @@ 
                 $this->callBack(
                     $this->aCallbacks[ 'load_fieldset_resource' ], 
                     array(
-                        $aFieldset,   // 1st parameter 
+                        $aFieldset, // 1st parameter 
                     )
                 );
                 
@@ -218,10 +218,10 @@ 
                  * @return      boolean
                  */
                 private function _isAlreadyRegistered( $sFieldtype, $sStructureType ) {
-                    if ( isset( self::$_aRegisteredFieldTypes[ $sFieldtype . '_' .$sStructureType ] ) ) {
+                    if ( isset( self::$_aRegisteredFieldTypes[ $sFieldtype.'_'.$sStructureType ] ) ) {
                         return true;
                     }
-                    self::$_aRegisteredFieldTypes[ $sFieldtype . '_' .$sStructureType ] = true;
+                    self::$_aRegisteredFieldTypes[ $sFieldtype.'_'.$sStructureType ] = true;
                     return false;
                 }
                     /**
@@ -240,7 +240,7 @@ 
             return $this->callBack(
                 $this->aCallbacks[ 'is_fieldset_registration_allowed' ], 
                 array(
-                    true,   // 1st parameter 
+                    true, // 1st parameter 
                     $aFieldset, // 2nd parameter
                 )
             );

Braces +2 added lines, -2 removed lines

@@ -121,14 +121,14 @@
             foreach( $this->aFieldsets as $_sSecitonID => $_aFieldsets ) {
                 
                 $_bIsSubSectionLoaded = false;
-                foreach( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField )  {
+                foreach( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField ) {
 // @todo Examine if this structure is correct or not. 
 // It may not be necessary to check the sub-section dimensions as this is not the saved options array.
                     // if it's a sub-section
                     if ( $this->isNumericInteger( $_iSubSectionIndexOrFieldID ) ) {
 
                         // no need to repeat the same set of fields
-                        if ( $_bIsSubSectionLoaded ) { 
+                        if ( $_bIsSubSectionLoaded ) {
                             continue;
                         }
                         $_bIsSubSectionLoaded = true;

development/factory/_abstract/form/_model/element_definition/AdminPageFramework_Form_Model___FieldConditioner.php

Spacing +8 added lines, -8 removed lines

@@ -60,18 +60,18 @@ 
 
             // Drop keys of fields-array which do not exist in the sections-array. 
             // For this reasons, the sections-array should be conditioned first before applying this method.
-            $aFields    = $this->castArrayContents( $aSections, $aFields );
+            $aFields = $this->castArrayContents( $aSections, $aFields );
 
             $_aNewFields = array();
-            foreach( $aFields as $_sSectionID => $_aSubSectionOrFields ) {
+            foreach ( $aFields as $_sSectionID => $_aSubSectionOrFields ) {
                 
                 // This type check is important as the parsing field array is content-cast, which can set null value to elements.
-                if ( ! is_array( $_aSubSectionOrFields ) ) { 
+                if ( !is_array( $_aSubSectionOrFields ) ) { 
                     continue; 
                 }
                             
                 $this->_setConditionedFields( 
-                    $_aNewFields,   // by reference - gets updated in the method.
+                    $_aNewFields, // by reference - gets updated in the method.
                     $_aSubSectionOrFields, 
                     $_sSectionID
                 );
@@ -91,14 +91,14 @@ 
              */
             private function _setConditionedFields( array &$_aNewFields, $_aSubSectionOrFields, $_sSectionID ) {
                 
-                foreach( $_aSubSectionOrFields as $_sIndexOrFieldID => $_aSubSectionOrField ) {
+                foreach ( $_aSubSectionOrFields as $_sIndexOrFieldID => $_aSubSectionOrField ) {
                     
                     // If it is a sub-section array.
                     if ( $this->isNumericInteger( $_sIndexOrFieldID ) ) {
                         $_sSubSectionIndex  = $_sIndexOrFieldID;
                         $_aFields           = $_aSubSectionOrField;
-                        foreach( $_aFields as $_aField ) {
-                            if ( ! $this->_isAllowed( $_aField ) ) {
+                        foreach ( $_aFields as $_aField ) {
+                            if ( !$this->_isAllowed( $_aField ) ) {
                                 continue;
                             }
                             $_aNewFields[ $_sSectionID ][ $_sSubSectionIndex ][ $_aField[ 'field_id' ] ] = $_aField;
@@ -109,7 +109,7 @@ 
                     
                     // Otherwise, insert the formatted field definition array.
                     $_aField = $_aSubSectionOrField;
-                    if ( ! $this->_isAllowed( $_aField ) ) {
+                    if ( !$this->_isAllowed( $_aField ) ) {
                         continue;
                     }
                     $_aNewFields[ $_sSectionID ][ $_aField[ 'field_id' ] ] = $_aField;

Braces +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
             foreach( $aFields as $_sSectionID => $_aSubSectionOrFields ) {
                 
                 // This type check is important as the parsing field array is content-cast, which can set null value to elements.
-                if ( ! is_array( $_aSubSectionOrFields ) ) { 
+                if ( ! is_array( $_aSubSectionOrFields ) ) {
                     continue; 
                 }
                             

development/factory/_abstract/form/_model/element_definition/AdminPageFramework_Form_Model___SectionConditioner.php

Spacing +7 added lines, -7 removed lines

@@ -17,7 +17,7 @@ 
  */
 class AdminPageFramework_Form_Model___SectionConditioner extends AdminPageFramework_WPUtility {
     
-    public $aSectionsets  = array();
+    public $aSectionsets = array();
 
     /**
      * Sets up hooks.
@@ -28,7 +28,7 @@ 
         $_aParameters = func_get_args() + array( 
             $this->aSectionsets, 
         );
-        $this->aSectionsets  = $_aParameters[ 0 ];                    
+        $this->aSectionsets = $_aParameters[ 0 ];                    
         
     }
 
@@ -50,11 +50,11 @@ 
      * @since       DEVVER      Moved from `AdminPageFramework_FormDefinition`. Changed the name from `getConditionedSections()`.
      * @return      array       The conditioned sectionsets array.
      */
-    private function _getSectionsConditioned( array $aSections=array() ) {
+    private function _getSectionsConditioned( array $aSections = array() ) {
         
-        $_aNewSections  = array();
-        foreach( $aSections as $_sSectionID => $_aSection ) {
-            if ( ! $this->_isAllowed( $_aSection ) ) {
+        $_aNewSections = array();
+        foreach ( $aSections as $_sSectionID => $_aSection ) {
+            if ( !$this->_isAllowed( $_aSection ) ) {
                 continue;                
             }
             $_aNewSections[ $_sSectionID ] = $_aSection;
@@ -73,7 +73,7 @@ 
     protected function _isAllowed( array $aDefinition ) {
         
         // Check capability. If the access level is not sufficient, skip.
-        if ( ! current_user_can( $aDefinition[ 'capability' ] ) ) { 
+        if ( !current_user_can( $aDefinition[ 'capability' ] ) ) { 
             return false;
         }
         return ( boolean ) $aDefinition[ 'if' ];

Braces +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
     protected function _isAllowed( array $aDefinition ) {
         
         // Check capability. If the access level is not sufficient, skip.
-        if ( ! current_user_can( $aDefinition[ 'capability' ] ) ) { 
+        if ( ! current_user_can( $aDefinition[ 'capability' ] ) ) {
             return false;
         }
         return ( boolean ) $aDefinition[ 'if' ];

development/factory/_abstract/form/_model/formatter/AdminPageFramework_Form_Model___FormatDynamicElements.php

Spacing +6 added lines, -6 removed lines

@@ -68,7 +68,7 @@ 
                 $aOptions // data source
             );
 
-            foreach( $aOptions as $_sSectionID => $_aSubSectionOrFields ) {
+            foreach ( $aOptions as $_sSectionID => $_aSubSectionOrFields ) {
                 
                 $_aSubSection = $this->_getSubSectionFromOptions(   
                     $_sSectionID,
@@ -105,10 +105,10 @@ 
                 
                 $_aSubSection = array();
                 $_iPrevIndex  = null;
-                foreach( $_aSubSectionOrFields as $_isIndexOrFieldID => $_aSubSectionOrFieldOptions ) {
+                foreach ( $_aSubSectionOrFields as $_isIndexOrFieldID => $_aSubSectionOrFieldOptions ) {
                 
                     // If it is not a sub-section array, skip.
-                    if ( ! $this->isNumericInteger( $_isIndexOrFieldID ) ) { 
+                    if ( !$this->isNumericInteger( $_isIndexOrFieldID ) ) { 
                         continue; 
                     }
                     
@@ -122,7 +122,7 @@ 
                     );
        
                     // Update the internal section index key
-                    foreach( $_aSubSection[ $_iIndex ] as &$_aField ) {
+                    foreach ( $_aSubSection[ $_iIndex ] as &$_aField ) {
                         $_aField[ '_section_index' ] = $_iIndex;
                     }
                     unset( $_aField ); // to be safe in PHP
@@ -147,7 +147,7 @@ 
                  */
                 private function _getSubSectionItemsFromOptions( array $_aSubSection, $_sSectionID, $_iIndex, $_iPrevIndex ) {
                     
-                    if ( ! isset( $this->aFieldsets[ $_sSectionID ] ) ) {
+                    if ( !isset( $this->aFieldsets[ $_sSectionID ] ) ) {
                         return array();
                     }
                     
@@ -156,7 +156,7 @@ 
                         : $this->getNonIntegerKeyElements( $this->aFieldsets[ $_sSectionID ] );
                         
                     // if empty, merge with the previous element.
-                    return ! empty( $_aFields )
+                    return !empty( $_aFields )
                         ? $_aFields
                         : $this->getElementAsArray(
                             $_aSubSection,

Braces +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
                 foreach( $_aSubSectionOrFields as $_isIndexOrFieldID => $_aSubSectionOrFieldOptions ) {
                 
                     // If it is not a sub-section array, skip.
-                    if ( ! $this->isNumericInteger( $_isIndexOrFieldID ) ) { 
+                    if ( ! $this->isNumericInteger( $_isIndexOrFieldID ) ) {
                         continue; 
                     }
                     

development/factory/_abstract/form/_model/formatter/AdminPageFramework_Form_Model___FormatSectionset.php

Spacing +21 added lines, -21 removed lines

@@ -35,39 +35,39 @@ 
         // Optional
         'page_slug'         => null,
         'tab_slug'          => null,
-        'section_tab_slug'  => null,    // 3.0.0+
+        'section_tab_slug'  => null, // 3.0.0+
         'title'             => null,
         'description'       => null,
         'capability'        => null,
         'if'                => true,    
-        'order'             => null,    // do not set the default number here because incremented numbers will be added when registering the sections.
+        'order'             => null, // do not set the default number here because incremented numbers will be added when registering the sections.
         'help'              => null,
         'help_aside'        => null,
-        'repeatable'        => false,   // (boolean|array) 3.0.0+
-        'sortable'          => false,   // (boolean|array) 3.6.0+
+        'repeatable'        => false, // (boolean|array) 3.0.0+
+        'sortable'          => false, // (boolean|array) 3.6.0+
         'attributes'        => array(   // 3.3.1+
-            'class'         => null,    // set null to avoid undefined index warnings.
-            'style'         => null,    // set null to avoid undefined index warnings.
+            'class'         => null, // set null to avoid undefined index warnings.
+            'style'         => null, // set null to avoid undefined index warnings.
             'tab'           => array(),
         ),
         'class'             => array(    // 3.3.1+
             'tab'           => array(),
         ),
-        'hidden'            => false,    // 3.3.1+
-        'collapsible'       => false,    // 3.4.0+ (boolean|array) For the array structure see the $aStructure_CollapsibleArguments property.
-        'save'              => true,     // 3.6.0+
+        'hidden'            => false, // 3.3.1+
+        'collapsible'       => false, // 3.4.0+ (boolean|array) For the array structure see the $aStructure_CollapsibleArguments property.
+        'save'              => true, // 3.6.0+
         
-        'content'           => null,     // 3.6.1+  (string) An overriding section-set output.
+        'content'           => null, // 3.6.1+  (string) An overriding section-set output.
         
         // Internal
-        '_fields_type'      => null,     // @deprecated DEVVER+ Use the `_structure_type` instead. 3.0.0+ - same as the one of the field definition array. Used to insert debug info at the bottom of sections.        
-        '_structure_type'   => null,     // DEVVER+
-        '_is_first_index'   => false,    // 3.4.0+ (boolean) indicates whether it is the first item of the sub-sections (for repeatable sections).
-        '_is_last_index'    => false,    // 3.4.0+ (boolean) indicates whether it is the last item of the sub-sections (for repeatable sections).
+        '_fields_type'      => null, // @deprecated DEVVER+ Use the `_structure_type` instead. 3.0.0+ - same as the one of the field definition array. Used to insert debug info at the bottom of sections.        
+        '_structure_type'   => null, // DEVVER+
+        '_is_first_index'   => false, // 3.4.0+ (boolean) indicates whether it is the first item of the sub-sections (for repeatable sections).
+        '_is_last_index'    => false, // 3.4.0+ (boolean) indicates whether it is the last item of the sub-sections (for repeatable sections).
         
-        '_section_path'         => '',       // DEVVER+ (string) e.g. my_section|nested_section       
-        '_section_path_array'   => '',       // DEVVER+ (array) an array version of the above section_path argument. Numerically indexed.
-        '_nested_depth'         => 0,        // DEVVER+ (integer) the nested level of the section
+        '_section_path'         => '', // DEVVER+ (string) e.g. my_section|nested_section       
+        '_section_path_array'   => '', // DEVVER+ (array) an array version of the above section_path argument. Numerically indexed.
+        '_nested_depth'         => 0, // DEVVER+ (integer) the nested level of the section
         
         // 3.6.0+ - (object) the caller framework factory object. This allows the framework to access the factory property when rendering the section.
         // DEVVER+  It no longer stores a factory object but a form object.
@@ -122,11 +122,11 @@ 
         $_aSectionPath = explode( '|', $this->sSectionPath );
         $_aSectionset  = $this->uniteArrays(
             array( 
-                '_fields_type'          => $this->sStructureType,   // @deprecated  DEVVER+
-                '_structure_type'       => $this->sStructureType,   // DEVVER+
-                '_section_path'         => $this->sSectionPath,     // DEVVER+
+                '_fields_type'          => $this->sStructureType, // @deprecated  DEVVER+
+                '_structure_type'       => $this->sStructureType, // DEVVER+
+                '_section_path'         => $this->sSectionPath, // DEVVER+
                 '_section_path_array'   => $_aSectionPath,
-                '_nested_depth'         => count( $_aSectionPath ) - 1,    // DEVVER+ - zero base
+                '_nested_depth'         => count( $_aSectionPath ) - 1, // DEVVER+ - zero base
             ) 
             + $this->aSectionset
             + array(

development/factory/_abstract/form/_model/formatter/AdminPageFramework_Form_Model___FormatSectionsets.php

Spacing +12 added lines, -12 removed lines

@@ -17,7 +17,7 @@ 
 class AdminPageFramework_Form_Model___FormatSectionsets extends AdminPageFramework_Form_Base {
     
     public $sStructureType = '';
-    public $aSectionsets  = array();
+    public $aSectionsets = array();
     public $sCapability = '';
     public $aCallbacks = array(
         'sectionset_before_output' => null
@@ -62,9 +62,9 @@ 
         }
 
         $_aSectionsets = $this->_getSectionsetsFormatted(
-            array(),                // sectionsets array to modify - new formatted items will be stored here
-            $this->aSectionsets,    // parsing sectionsets
-            array(),                // section path - empty for root 
+            array(), // sectionsets array to modify - new formatted items will be stored here
+            $this->aSectionsets, // parsing sectionsets
+            array(), // section path - empty for root 
             $this->sCapability      // capability
         );
 
@@ -81,10 +81,10 @@ 
          */
         private function _getSectionsetsFormatted( $_aNewSectionsets, $aSectionsetsToParse, $aSectionPath, $sCapability ) {
 
-            foreach( $aSectionsetsToParse as $_sSectionPath => $_aSectionset ) {
+            foreach ( $aSectionsetsToParse as $_sSectionPath => $_aSectionset ) {
 
                 // The '_default' section can be empty so do not check `if ( empty( $_aSectionset ) )` here.
-                if ( ! is_array( $_aSectionset ) ) { 
+                if ( !is_array( $_aSectionset ) ) { 
                     continue; 
                 }
                 
@@ -111,7 +111,7 @@ 
                 
                 // DEVVER+ For nested sections         
                 $_aNewSectionsets = $this->_getNestedSections( 
-                    $_aNewSectionsets,  // sectionset array to modify
+                    $_aNewSectionsets, // sectionset array to modify
                     $_aSectionset, 
                     $_aSectionPath, // section path
                     $_aSectionset[ 'capability' ]
@@ -128,15 +128,15 @@ 
              */
             private function _getNestedSections( $aSectionsetsToEdit, $aSectionset, $aSectionPath, $sCapability ) {
 
-                if ( ! $this->_hasNestedSections( $aSectionset ) ) {
+                if ( !$this->_hasNestedSections( $aSectionset ) ) {
                     return $aSectionsetsToEdit;
                 }
 
                 // Reccursive call
                 return $this->_getSectionsetsFormatted(
-                        $aSectionsetsToEdit,          // sectionsets array to modify - new formatted items will be stored here
-                        $aSectionset[ 'content' ],    // parsing sectionsets
-                        $aSectionPath,                // section path - empty for root 
+                        $aSectionsetsToEdit, // sectionsets array to modify - new formatted items will be stored here
+                        $aSectionset[ 'content' ], // parsing sectionsets
+                        $aSectionPath, // section path - empty for root 
                         $sCapability                  // capability
                     );                          
 
@@ -149,7 +149,7 @@ 
                 private function _hasNestedSections( $aSectionset ) {
                     
                     $aSectionset = $aSectionset + array( 'content' => null );
-                    if ( ! is_array( $aSectionset[ 'content' ] ) ) {
+                    if ( !is_array( $aSectionset[ 'content' ] ) ) {
                         return false;
                     }
                     $_aContents  = $aSectionset[ 'content' ];

Braces +2 added lines, -2 removed lines

@@ -84,7 +84,7 @@ 
             foreach( $aSectionsetsToParse as $_sSectionPath => $_aSectionset ) {
 
                 // The '_default' section can be empty so do not check `if ( empty( $_aSectionset ) )` here.
-                if ( ! is_array( $_aSectionset ) ) { 
+                if ( ! is_array( $_aSectionset ) ) {
                     continue; 
                 }
                 
@@ -103,7 +103,7 @@ 
                     $this->aCallbacks[ 'sectionset_before_output' ], 
                     array( $_aSectionsetFormatter->get() )
                 );
-                if ( empty( $_aSectionset ) ) { 
+                if ( empty( $_aSectionset ) ) {
                     continue; 
                 }
                 

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

Spacing +7 added lines, -7 removed lines

@@ -24,7 +24,7 @@ 
      */
     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
+        '_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.
     );
     
     /**
@@ -83,7 +83,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
@@ -127,7 +127,7 @@ 
         // ); 
                             
         // used in the attribute below plus it is also used in the sample custom field type.
-        $_aField[ '_field_container_id' ]      = "field-{$_aField[ 'input_id' ]}";
+        $_aField[ '_field_container_id' ] = "field-{$_aField[ 'input_id' ]}";
 
 // @todo for issue #158 https://github.com/michaeluno/admin-page-framework/issues/158               
 // These models are for generating ids and names dynamically.
@@ -165,8 +165,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/_abstract/form/_model/formatter/AdminPageFramework_Form_Model___Format_EachSection.php

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
     /**
      * Stores the section definition.
      */
-    public $aSection            = array();
+    public $aSection = array();
     
     /**
      * Indicates the sub section index.
@@ -91,8 +91,8 @@ 
         
         $_aSection[ '_is_collapsible' ]    = $_aSection[ 'collapsible' ] && 'section' === $_aSection[ 'collapsible' ][ 'container' ];
         
-        $_aSection[ '_tag_id' ]            = 'section-' . $_aSection[ 'section_id' ] . '__' . $this->iIndex;
-        $_aSection[ '_tag_id_model' ]      = 'section-' . $_aSection[ 'section_id' ] . '__' . '___i___';
+        $_aSection[ '_tag_id' ]            = 'section-'.$_aSection[ 'section_id' ].'__'.$this->iIndex;
+        $_aSection[ '_tag_id_model' ]      = 'section-'.$_aSection[ 'section_id' ].'__'.'___i___';
         
         return $_aSection;
         

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

Indentation +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
             // $aSubFields gets updated in the method
             $this->_fillRepeatableElements( $aField, $aSubFields, $mSavedValue );
                             
-             // $aSubFields gets updated in the method
+                // $aSubFields gets updated in the method
             $this->_fillSubFields( $aSubFields, $aFirstField );
 
             // Put them together

Spacing +10 added lines, -10 removed lines

@@ -53,7 +53,7 @@ 
     public function get() {
 
         // Get the set value(s)
-        $_mSavedValue    = $this->_getStoredInputFieldValue( 
+        $_mSavedValue = $this->_getStoredInputFieldValue( 
             $this->aField, 
             $this->aOptions 
         );
@@ -113,9 +113,9 @@ 
              * @return      void
              */
             private function _divideMainAndSubFields( array $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;
                     }
@@ -133,12 +133,12 @@ 
              * @return      void
              */
             private function _fillRepeatableElements( array $aField, array &$aSubFields, $mSavedValue ) {
-                if ( ! $aField[ 'repeatable' ] ) {
+                if ( !$aField[ 'repeatable' ] ) {
                     return;
                 }
                 $_aSavedValues = ( array ) $mSavedValue;
                 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();     
@@ -153,7 +153,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( 
@@ -181,13 +181,13 @@ 
          
             // Determine whether the elements are saved in an array.
             $_bHasSubFields = count( $aFields ) > 1 || $aField[ 'repeatable' ] || $aField[ 'sortable' ];
-            if ( ! $_bHasSubFields ) {
+            if ( !$_bHasSubFields ) {
                 $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[ '_is_multiple_fields' ] = true;
             }
@@ -201,7 +201,7 @@ 
          * @since       3.6.0       Moved from `AdminPageFramework_FieldDefinition`.
          */
         private function _setFieldsValue( array &$aFields ) {
-            foreach( $aFields as &$_aField ) {
+            foreach ( $aFields as &$_aField ) {
                 $_aField[ '_is_value_set_by_user' ] = isset( $_aField[ 'value' ] );
                 $_aField[ 'value' ]                 = $this->_getSetFieldValue( $_aField );
             }
@@ -242,7 +242,7 @@ 
         private function _getStoredInputFieldValue( $aField, $aOptions ) {    
 
             // 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, 
                     $aField[ 'field_id' ],

Braces +1 added lines, -1 removed lines

@@ -239,7 +239,7 @@
          * @since       3.6.0       Moved from `AdminPageFramework_FieldDefinition`.
          * @since       DEVVER      Changed the `_field_type` element to `_structure_type`.
          */
-        private function _getStoredInputFieldValue( $aField, $aOptions ) {    
+        private function _getStoredInputFieldValue( $aField, $aOptions ) {
 
             // If a section is not set, check the first dimension element.
             if ( ! isset( $aField[ 'section_id' ] ) || '_default' === $aField[ 'section_id' ] ) {