michaeluno / admin-page-framework

development/factory/_abstract/form/_view/sectionset/AdminPageFramework_Form_View___Sectionsets.php

Spacing +42 added lines, -42 removed lines

@@ -126,16 +126,16 @@ 
      */
     public function get() {
         
-        $_oFormatSectionsetsByTab  = new AdminPageFramework_Form_View___Format_SectionsetsByTab(
+        $_oFormatSectionsetsByTab = new AdminPageFramework_Form_View___Format_SectionsetsByTab(
             $this->aStructure[ 'sectionsets' ],
             $this->aStructure[ 'fieldsets' ],
             $this->aArguments[ 'nested_depth' ]
         );
 
-        $_aOutput     = array();
-        foreach( $_oFormatSectionsetsByTab->getTabs() as $_sSectionTabSlug ) {
+        $_aOutput = array();
+        foreach ( $_oFormatSectionsetsByTab->getTabs() as $_sSectionTabSlug ) {
 
-            $_aOutput[] = $this->_getFormOutput(
+            $_aOutput[ ] = $this->_getFormOutput(
                 $_oFormatSectionsetsByTab->getSectionsets( $_sSectionTabSlug ),
                 $_oFormatSectionsetsByTab->getFieldsets( $_sSectionTabSlug ),
                 $_sSectionTabSlug,
@@ -149,7 +149,7 @@ 
 
         // Generate id for this output
         $_sOutput = implode( PHP_EOL, $_aOutput );
-        $_sElementID = "admin-page-framework-sectionsets-" . uniqid();
+        $_sElementID = "admin-page-framework-sectionsets-".uniqid();
         return $this->_getSpinnerOutput( $_sOutput )
             .   "<div id='{$_sElementID}' class='admin-page-framework-sctionsets admin-page-framework-form-js-on'>"
                 . $_sOutput
@@ -184,17 +184,17 @@ 
 
             // A sectionset is a set of sections.
             $_sSectionSet = $this->_getSectionsetsTables( 
-                $aSectionsets,  // sectionset definition (already devided by section tab)
-                $aFieldsets,    // fieldset definitions (already devided by section tab)
+                $aSectionsets, // sectionset definition (already devided by section tab)
+                $aFieldsets, // fieldset definitions (already devided by section tab)
                 $aCallbacks
             );
             return $_sSectionSet
-                ? "<div " . $this->getAttributes(
+                ? "<div ".$this->getAttributes(
                         array(
                             'class' => 'admin-page-framework-sectionset',
-                            'id'    => "sectionset-{$sSectionTabSlug}_" . md5( serialize( $aSectionsets ) ),
+                            'id'    => "sectionset-{$sSectionTabSlug}_".md5( serialize( $aSectionsets ) ),
                         ) 
-                    ) . ">"
+                    ).">"
                         . $_sSectionSet
                     . "</div>"
                 : '';
@@ -226,7 +226,7 @@ 
              * If no fields belonging to the section, return empty.
              * Otherwise, the sectionsets container gets rendered and its CSS rules such as margins give unwanted results.
              */
-            if ( ! count( $aFieldsets ) ) {
+            if ( !count( $aFieldsets ) ) {
                 return ''; 
             }           
 
@@ -238,12 +238,12 @@ 
                 'count_subsections' => 0,
             );
             $_sThisSectionID    = $_aFirstSectionset[ 'section_id' ];
-            $_sSectionsID       = 'sections-' . $_sThisSectionID;
+            $_sSectionsID       = 'sections-'.$_sThisSectionID;
             $_aCollapsible      = $this->_getCollapsibleArgumentForSections( 
                 $_aFirstSectionset 
             );
 
-            foreach( $aSectionsets as $_aSectionset ) {
+            foreach ( $aSectionsets as $_aSectionset ) {
            
                 $_sSectionTabSlug   = $_aSectionset[ 'section_tab_slug' ]; // needed outside the loop
                 $_aOutputs          = $this->_getSectionsetTable(
@@ -275,7 +275,7 @@ 
              * Changed the visibility scope to private. Changed the 1st parameter from `$aSection`.
              * @return      array
              */
-            private function _getCollapsibleArgumentForSections( array $aSectionset=array() ) {  
+            private function _getCollapsibleArgumentForSections( array $aSectionset = array() ) {  
                 
                 $_oArgumentFormater = new AdminPageFramework_Form_Model___Format_CollapsibleSection(
                     $aSectionset[ 'collapsible' ],
@@ -304,17 +304,17 @@ 
              */
             private function _getSectionsetTable( $_aOutputs, $_sSectionsID, array $_aSection, array $aFieldsInSections ) {
             
-                if ( ! $this->isSectionsetVisible( $_aSection ) ) {
+                if ( !$this->isSectionsetVisible( $_aSection ) ) {
 // var_dump( 'not visible: ' . $_aSection[ '_section_path' ] );
                     return $_aOutputs;
                 }
 
                 // If the 'save' argument is false, insert a flag that disables saving the section inputs.
-                $_aOutputs[ 'section_contents' ][] = $this->_getUnsetFlagSectionInputTag( $_aSection );
+                $_aOutputs[ 'section_contents' ][ ] = $this->_getUnsetFlagSectionInputTag( $_aSection );
                 
                 // For repeatable sections - sub-sections are divided field definition arrays by sub-section index, 
                 // not section definition arrays.
-                $_aSubSections      = $this->getIntegerKeyElements( 
+                $_aSubSections = $this->getIntegerKeyElements( 
                     $this->getElementAsArray(
                         $aFieldsInSections, // subject
                         $_aSection[ '_section_path' ], // $_aSection[ 'section_id' ],   // dimensional key
@@ -358,29 +358,29 @@ 
                 private function _getSubSections( $_aOutputs, $_sSectionsID, $_aSection, $_aSubSections ) {
                     
                     // Add the repeatable sections enabler script.
-                    if ( ! empty( $_aSection[ 'repeatable' ] ) ) {
-                        $_aOutputs[ 'section_contents' ][] = AdminPageFramework_Form_View___Script_RepeatableSection::getEnabler( 
+                    if ( !empty( $_aSection[ 'repeatable' ] ) ) {
+                        $_aOutputs[ 'section_contents' ][ ] = AdminPageFramework_Form_View___Script_RepeatableSection::getEnabler( 
                             $_sSectionsID, 
                             $_aOutputs[ 'count_subsections' ], 
                             $_aSection[ 'repeatable' ],
                             $this->oMsg
                         );
-                        $_aOutputs[ 'section_contents' ][] = $this->_getRepeatableSectionFlagTag( $_aSection );
+                        $_aOutputs[ 'section_contents' ][ ] = $this->_getRepeatableSectionFlagTag( $_aSection );
                     }
                     // Add the sortable sections enabler script. 3.6.0+
-                    if ( ! empty( $_aSection[ 'sortable' ] ) ) {
+                    if ( !empty( $_aSection[ 'sortable' ] ) ) {
 // @todo Change the name of the class to AdminPageFramework_Form_Script_...
-                        $_aOutputs[ 'section_contents' ][] = AdminPageFramework_Form_View___Script_SortableSection::getEnabler( 
+                        $_aOutputs[ 'section_contents' ][ ] = AdminPageFramework_Form_View___Script_SortableSection::getEnabler( 
                             $_sSectionsID, 
                             $_aSection[ 'sortable' ],
                             $this->oMsg
                         );
-                        $_aOutputs[ 'section_contents' ][] = $this->_getSortableSectionFlagTag( $_aSection );
+                        $_aOutputs[ 'section_contents' ][ ] = $this->_getSortableSectionFlagTag( $_aSection );
                     }
                     
                     // Get the section tables.
                     $_aSubSections = $this->numerizeElements( $_aSubSections ); // will include the main section as well.
-                    foreach( $_aSubSections as $_iIndex => $_aFields ) { 
+                    foreach ( $_aSubSections as $_iIndex => $_aFields ) { 
 
                         $_oEachSectionArguments = new AdminPageFramework_Form_Model___Format_EachSection(
                             $_aSection,
@@ -410,8 +410,8 @@ 
                         array(
                             'class'                     => 'element-address',
                             'type'                      => 'hidden',
-                            'name'                      => '__repeatable_elements_' . $aSection[ '_structure_type' ] 
-                                . '[ ' . $aSection[ 'section_id' ] . ' ]',
+                            'name'                      => '__repeatable_elements_'.$aSection[ '_structure_type' ] 
+                                . '[ '.$aSection[ 'section_id' ].' ]',
                             // @todo examine whether this value should include a section index.
                             'value' => $aSection[ 'section_id' ],                            
                         )
@@ -429,8 +429,8 @@ 
                         array(
                             'class'                     => 'element-address',
                             'type'                      => 'hidden',
-                            'name'                      => '__sortable_elements_' . $aSection[ '_structure_type' ] 
-                                . '[ ' . $aSection[ 'section_id' ] . ' ]',
+                            'name'                      => '__sortable_elements_'.$aSection[ '_structure_type' ] 
+                                . '[ '.$aSection[ 'section_id' ].' ]',
                             // @todo examine whether this value should include a section index.
                             'value' => $aSection[ 'section_id' ],                            
                         )
@@ -452,8 +452,8 @@ 
                         'input',
                         array(
                             'type'  => 'hidden',
-                            'name'  => '__unset_' .  $aSection[ '_structure_type' ] . '[ ' . $aSection[ 'section_id' ] . ' ]',
-                            'value' => "__dummy_option_key|" . $aSection[ 'section_id' ],
+                            'name'  => '__unset_'.$aSection[ '_structure_type' ].'[ '.$aSection[ 'section_id' ].' ]',
+                            'value' => "__dummy_option_key|".$aSection[ 'section_id' ],
                             'class' => 'unset-element-names element-address',
                         )
                     );            
@@ -469,7 +469,7 @@ 
                 private function _getSectionTableWithTabList( array $_aOutputs, array $aSectionset, $aFieldsetsPerSection ) {
                                         
                     // Tab list
-                    $_aOutputs[ 'section_tab_list' ][] = $this->_getTabList( 
+                    $_aOutputs[ 'section_tab_list' ][ ] = $this->_getTabList( 
                         $aSectionset, 
                         $aFieldsetsPerSection, 
                         $this->aCallbacks[ 'fieldset_output' ]
@@ -478,7 +478,7 @@ 
 
                     // Section container
                     $_oSectionTable = new AdminPageFramework_Form_View___Section(
-                        $this->aArguments,  // for nested sections
+                        $this->aArguments, // for nested sections
                         $aSectionset,
                         $this->aStructure,
                         $aFieldsetsPerSection,
@@ -488,7 +488,7 @@ 
                         $this->aCallbacks,
                         $this->oMsg
                     );
-                    $_aOutputs[ 'section_contents' ][] = $_oSectionTable->get();                    
+                    $_aOutputs[ 'section_contents' ][ ] = $_oSectionTable->get();                    
                    
                     return $_aOutputs;
                  
@@ -516,9 +516,9 @@ 
                         'section_index'     => null,
                         'collapsible'       => $aCollapsible,
                         'container_type'    => 'sections', // section or sections                    
-                        'sectionset'        => $aSectionset,    // 3.7.0+ for tooltip
+                        'sectionset'        => $aSectionset, // 3.7.0+ for tooltip
                     ),
-                    array(),            // fieldsets
+                    array(), // fieldsets
                     $this->aSavedData,   
                     $this->aFieldErrors, 
                     $this->aStructure[ 'field_type_definitions' ], 
@@ -535,7 +535,7 @@ 
                     $aOutputs[ 'count_subsections' ]
                 );
                 return $_oCollapsibleSectionTitle->get()
-                    . "<div " . $_oSectionsTablesContainerAttributes->get() . ">"
+                    . "<div ".$_oSectionsTablesContainerAttributes->get().">"
                         . $this->_getSectionTabList( $sSectionTabSlug, $aOutputs[ 'section_tab_list' ] )
                         . implode( PHP_EOL, $aOutputs[ 'section_contents' ] )
                     . "</div>";
@@ -566,21 +566,21 @@ 
              */
             private function _getTabList( array $aSection, array $aFields, $hfFieldCallback ) {
                                 
-                if ( ! $aSection[ 'section_tab_slug' ] ) {
+                if ( !$aSection[ 'section_tab_slug' ] ) {
                     return '';
                 }
                 
                 $iSectionIndex      = $aSection[ '_index' ];
 
-                $_sSectionTagID     = 'section-' . $aSection[ 'section_id' ] . '__' . $iSectionIndex;
+                $_sSectionTagID     = 'section-'.$aSection[ 'section_id' ].'__'.$iSectionIndex;
                 $_aTabAttributes    = $aSection[ 'attributes' ][ 'tab' ]
                     + array(
                         'class' => 'admin-page-framework-section-tab nav-tab',
                         'id'    => "section_tab-{$_sSectionTagID}",
                         'style' => null
                     );
-                $_aTabAttributes[ 'class' ] = $this->getClassAttribute( $_aTabAttributes[ 'class' ], $aSection[ 'class' ][ 'tab' ] );  // 3.3.1+
-                $_aTabAttributes[ 'style' ] = $this->getStyleAttribute( $_aTabAttributes[ 'style' ], $aSection[ 'hidden' ] ? 'display:none' : null );  // 3.3.1+
+                $_aTabAttributes[ 'class' ] = $this->getClassAttribute( $_aTabAttributes[ 'class' ], $aSection[ 'class' ][ 'tab' ] ); // 3.3.1+
+                $_aTabAttributes[ 'style' ] = $this->getStyleAttribute( $_aTabAttributes[ 'style' ], $aSection[ 'hidden' ] ? 'display:none' : null ); // 3.3.1+
                 
                 $_oSectionTitle = new AdminPageFramework_Form_View___SectionTitle(                    
                     array(
@@ -588,7 +588,7 @@ 
                         'tag'           => 'h4',
                         'section_index' => $iSectionIndex,
                         
-                        'sectionset'    => $aSection,   // 3.7.0+      for tooltip
+                        'sectionset'    => $aSection, // 3.7.0+      for tooltip
                     ),
                     $aFields,            
                     $this->aSavedData,   
@@ -598,7 +598,7 @@ 
                     $this->aCallbacks // field output element callables.                    
                 );                        
                 
-                return "<li " . $this->getAttributes( $_aTabAttributes ) . ">"
+                return "<li ".$this->getAttributes( $_aTabAttributes ).">"
                     . "<a href='#{$_sSectionTagID}'>"
                         . $_oSectionTitle->get()
                     ."</a>"

development/factory/_abstract/form/_view/sectionset/AdminPageFramework_Form_View___SectionCaption.php

Spacing +10 added lines, -10 removed lines

@@ -79,7 +79,7 @@ 
          */
         private function _getCaption( array $aSectionset, $iSectionIndex, $aFieldsets, $aFieldErrors, $aFieldTypeDefinitions, $aCallbacks, $oMsg ) {
             
-            if ( ! $aSectionset[ 'description' ] && ! $aSectionset[ 'title' ] ) {
+            if ( !$aSectionset[ 'description' ] && !$aSectionset[ 'title' ] ) {
                 return "<caption class='admin-page-framework-section-caption' style='display:none;'></caption>";
             }    
 
@@ -102,7 +102,7 @@ 
                     'collapsible'       => $_abCollapsible,
                     'container_type'    => 'section', // section or sections                    
                     
-                    'sectionset'        => $aSectionset,    // 3.7.0+ for tooltip
+                    'sectionset'        => $aSectionset, // 3.7.0+ for tooltip
                 ),
                 $aFieldsets,            
                 $this->aSavedData,   
@@ -112,15 +112,15 @@ 
                 $aCallbacks // field output element callables.                
             );            
             
-            $_bShowTitle    = empty( $_abCollapsible ) && ! $aSectionset[ 'section_tab_slug' ];
+            $_bShowTitle = empty( $_abCollapsible ) && !$aSectionset[ 'section_tab_slug' ];
             return 
-                "<caption " . $this->getAttributes( 
+                "<caption ".$this->getAttributes( 
                     array(
                         'class'             => 'admin-page-framework-section-caption',
                         // data-section_tab is referred by the repeater script to hide/show the title and the description
                         'data-section_tab'  => $aSectionset[ 'section_tab_slug' ],
                     ) 
-                ) . ">"
+                ).">"
                     . $_oCollapsibleSectionTitle->get()
                     . $this->getAOrB(
                         $_bShowTitle,
@@ -174,7 +174,7 @@ 
                         'tag'           => 'h3',
                         'section_index' => $iSectionIndex,
                         
-                        'sectionset'    => $aSectionset,    // 3.7.0+ for tooltip
+                        'sectionset'    => $aSectionset, // 3.7.0+ for tooltip
                     ),
                     $aFieldsets,            
                     $this->aSavedData,   
@@ -183,7 +183,7 @@ 
                     $this->oMsg,
                     $this->aCallbacks // field output element callables.
                 );
-                return "<div " . $this->getAttributes(
+                return "<div ".$this->getAttributes(
                         array(
                             'class' => 'admin-page-framework-section-title',
                             'style' => $this->getAOrB(
@@ -192,7 +192,7 @@ 
                                 'display: none;'
                             ),
                         )
-                    ). ">" 
+                    ).">" 
                         . $_oSectionTitle->get()
                     . "</div>";                
             }
@@ -208,7 +208,7 @@ 
                 if ( $aSectionset[ 'collapsible' ] ) {
                     return '';
                 }
-                if ( ! is_callable( $hfSectionCallback ) ) {
+                if ( !is_callable( $hfSectionCallback ) ) {
                     return '';
                 }
                 
@@ -240,7 +240,7 @@ 
              */
             private function _shouldShowCaptionTitle( $aSectionset, $iSectionIndex ) {
                 
-                if ( ! $aSectionset[ 'title' ] ){
+                if ( !$aSectionset[ 'title' ] ) {
                     return false;
                 }
                 if ( $aSectionset[ 'collapsible' ] ) {

development/factory/_abstract/form/_view/sectionset/AdminPageFramework_Form_View___CollapsibleSectionTitle.php

Spacing +8 added lines, -8 removed lines

@@ -18,17 +18,17 @@ 
  */
 class AdminPageFramework_Form_View___CollapsibleSectionTitle extends AdminPageFramework_Form_View___SectionTitle {
             
-    public $aArguments      = array(
+    public $aArguments = array(
         'title'             => null,
         'tag'               => null,
         'section_index'     => null,
         'collapsible'       => array(),
         'container_type'    => 'section', // section or sections
         
-        'sectionset'        => array(),  // 3.7.0+ sectionset definition array
+        'sectionset'        => array(), // 3.7.0+ sectionset definition array
         
     );
-    public $aFieldsets               = array();
+    public $aFieldsets = array();
     public $aSavedData              = array();
     public $aFieldErrors            = array();
     public $aFieldTypeDefinitions   = array();
@@ -62,7 +62,7 @@ 
          * @param       array|boolean   $aCollapsible       The collapsible argument.
          * @param       string          $sContainer          The position context. Accepts either 'sections' or 'section'. If the set position in the argument array does not match this value, the method will return an empty string.
          */
-        private function _getCollapsibleSectionTitleBlock( array $aCollapsible, $sContainer='sections', $iSectionIndex=null ) {
+        private function _getCollapsibleSectionTitleBlock( array $aCollapsible, $sContainer = 'sections', $iSectionIndex = null ) {
 
             if ( $sContainer !== $aCollapsible[ 'container' ] ) { 
                 return ''; 
@@ -78,10 +78,10 @@ 
             );
             
             $_aSectionset        = $this->aArguments[ 'sectionset' ];
-            $_sSectionTitleTagID = str_replace( '|', '_', $_aSectionset[ '_section_path' ]  ) . '_' . $iSectionIndex;
+            $_sSectionTitleTagID = str_replace( '|', '_', $_aSectionset[ '_section_path' ] ).'_'.$iSectionIndex;
             
             return $this->_getCollapsibleSectionsEnablerScript()
-                . "<div " . $this->getAttributes(
+                . "<div ".$this->getAttributes(
                     array(
                         'id'    => $_sSectionTitleTagID,
                         'class' => $this->getClassAttribute(
@@ -102,11 +102,11 @@ 
                                 'collapsed',
                                 ''
                             ),                            
-                            'admin-page-framework-collapsible-type-' . $aCollapsible[ 'type' ]
+                            'admin-page-framework-collapsible-type-'.$aCollapsible[ 'type' ]
                         ),
                     ) 
                     + $this->getDataAttributeArray( $aCollapsible )
-                ) . ">"  
+                ).">"  
                         . $_sSectionTitle
                     . "</div>";
             

development/factory/_abstract/form/_view/css/AdminPageFramework_Form_View___CSS_Section.php

Indentation +7 added lines, -7 removed lines

@@ -24,13 +24,13 @@
     protected function _get() {
         return $this->_getFormSectionRules();
     }
-         /**
-         * Returns the CSS rules for form fields.
-         * 
-         * @since       3.4.0
-         * @since       3.7.0      Moved from `AdminPageFramework_CSS`.
-         * @internal
-         */    
+            /**
+             * Returns the CSS rules for form fields.
+             * 
+             * @since       3.4.0
+             * @since       3.7.0      Moved from `AdminPageFramework_CSS`.
+             * @internal
+             */    
         private function _getFormSectionRules() {
             
             return <<<CSSRULES

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

Spacing +13 added lines, -13 removed lines

@@ -23,12 +23,12 @@ 
      * @since       3.6.0       Moved from `AdminPageFramework_FormDefinition`.
      */
     static public $aStructure = array(
-        'title'                     => null,        // (string)  the section title will be assigned by default in the section formatting method.
-        'is_collapsed'              => true,        // (boolean) whether it is already collapsed or expanded
-        'toggle_all_button'         => null,        // (boolean|string|array) the position of where to display the toggle-all button that toggles the folding state of all collapsible sections. Accepts the following values. 'top-right', 'top-left', 'bottom-right', 'bottom-left'. If true is passed, the default 'top-right' will be used. To not to display, do not set any or pass `false` or `null`.
-        'collapse_others_on_expand' => true,        // (boolean) whether the other collapsible sections should be folded when the section is unfolded.
-        'container'                 => 'sections',  // (string) the container element that collapsible styling gets applied to. Either 'sections' or 'section' is accepted.
-        'type'                      => 'box',       // 3.7.0+  (string)  supported types 'box', 'button' Default: `box`. The `button` type is only supported when the `container` argument is `section`.
+        'title'                     => null, // (string)  the section title will be assigned by default in the section formatting method.
+        'is_collapsed'              => true, // (boolean) whether it is already collapsed or expanded
+        'toggle_all_button'         => null, // (boolean|string|array) the position of where to display the toggle-all button that toggles the folding state of all collapsible sections. Accepts the following values. 'top-right', 'top-left', 'bottom-right', 'bottom-left'. If true is passed, the default 'top-right' will be used. To not to display, do not set any or pass `false` or `null`.
+        'collapse_others_on_expand' => true, // (boolean) whether the other collapsible sections should be folded when the section is unfolded.
+        'container'                 => 'sections', // (string) the container element that collapsible styling gets applied to. Either 'sections' or 'section' is accepted.
+        'type'                      => 'box', // 3.7.0+  (string)  supported types 'box', 'button' Default: `box`. The `button` type is only supported when the `container` argument is `section`.
     );   
     
     public $abCollapsible = false;
@@ -41,7 +41,7 @@ 
      * @remark      The framework will not pass this parameter when formatting a section definition array.
      * It will be passed when the framework is rendering a form table to generate collapsible elements.
      */
-    public $aSection      = array();
+    public $aSection = array();
     
     /**
      * Sets up properties.
@@ -84,14 +84,14 @@ 
             
             $_aCollapsible = $this->getAsArray( $this->abCollapsible ) + array(
                 'title'     => $sTitle,
-            ) +  self::$aStructure;
+            ) + self::$aStructure;
             
             $_aCollapsible[ 'toggle_all_button' ] = implode( 
                 ',', 
                 $this->getAsArray( $_aCollapsible[ 'toggle_all_button' ] ) 
             );
             
-            if ( ! empty( $aSection ) ) {
+            if ( !empty( $aSection ) ) {
                 $_aCollapsible[ 'toggle_all_button' ] = $this->_getToggleAllButtonArgument( 
                     $_aCollapsible[ 'toggle_all_button' ], 
                     $aSection
@@ -115,7 +115,7 @@ 
              */
             private function _getToggleAllButtonArgument( $sToggleAll, array $aSection ) {
                 
-                if ( ! $aSection[ 'repeatable' ] ) {            
+                if ( !$aSection[ 'repeatable' ] ) {            
                     return $sToggleAll;
                 }
                 
@@ -125,13 +125,13 @@ 
                 }
                 
                 // Disable the toggle all button for middle sub-sections in repeatable sections.
-                if ( ! $aSection[ '_is_first_index' ] && ! $aSection[ '_is_last_index' ] ) {
+                if ( !$aSection[ '_is_first_index' ] && !$aSection[ '_is_last_index' ] ) {
                     return 0;
                 }            
                 
                 $_aToggleAll = $this->getAOrB(
-                    true === $sToggleAll || 1 ===  $sToggleAll, // evaluate
-                    array( 'top-right', 'bottom-right' ),   // if true
+                    true === $sToggleAll || 1 === $sToggleAll, // evaluate
+                    array( 'top-right', 'bottom-right' ), // if true
                     explode( ',', $sToggleAll ) // if false
                 );            
                 $_aToggleAll = $this->getAOrB(

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

Spacing +22 added lines, -22 removed lines

@@ -35,41 +35,41 @@ 
         // 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.
         
-        'tip'               => null,     // 3.7.0  (string) Tool tip HTML strings. 
+        'tip'               => null, // 3.7.0  (string) Tool tip HTML strings. 
         
         // Internal
-        '_fields_type'      => null,     // @deprecated 3.7.0+ 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,     // 3.7.0+
-        '_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 3.7.0+ 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, // 3.7.0+
+        '_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'         => '',       // 3.7.0+ (string) e.g. my_section|nested_section       
-        '_section_path_array'   => '',       // 3.7.0+ (array) an array version of the above section_path argument. Numerically indexed.
-        '_nested_depth'         => 0,        // 3.7.0+ (integer) the nested level of the section
+        '_section_path'         => '', // 3.7.0+ (string) e.g. my_section|nested_section       
+        '_section_path_array'   => '', // 3.7.0+ (array) an array version of the above section_path argument. Numerically indexed.
+        '_nested_depth'         => 0, // 3.7.0+ (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.
         // 3.7.0+  It no longer stores a factory object but a form object.
@@ -124,11 +124,11 @@ 
         $_aSectionPath = explode( '|', $this->sSectionPath );
         $_aSectionset  = $this->uniteArrays(
             array( 
-                '_fields_type'          => $this->sStructureType,   // @deprecated  3.7.0+
-                '_structure_type'       => $this->sStructureType,   // 3.7.0+
-                '_section_path'         => $this->sSectionPath,     // 3.7.0+
+                '_fields_type'          => $this->sStructureType, // @deprecated  3.7.0+
+                '_structure_type'       => $this->sStructureType, // 3.7.0+
+                '_section_path'         => $this->sSectionPath, // 3.7.0+
                 '_section_path_array'   => $_aSectionPath,
-                '_nested_depth'         => count( $_aSectionPath ) - 1,    // 3.7.0+ - zero base
+                '_nested_depth'         => count( $_aSectionPath ) - 1, // 3.7.0+ - zero base
             ) 
             + $this->aSectionset
             + array(

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

Spacing +32 added lines, -32 removed lines

@@ -29,36 +29,36 @@ 
     static public $aStructure = array(
     
         // Required Keys
-        'field_id'                  => null,    // (string)
-        'type'                      => null,    // (string)
-        'section_id'                => null,    // (string)
+        'field_id'                  => null, // (string)
+        'type'                      => null, // (string)
+        'section_id'                => null, // (string)
 
         // Optional Keys        
-        'section_title'             => null,    // This will be assigned automatically in the formatting method.
-        'page_slug'                 => null,    // This will be assigned automatically in the formatting method.
-        'tab_slug'                  => null,    // This will be assigned automatically in the formatting method.
-        'option_key'                => null,    // This will be assigned automatically in the formatting method.
-        'class_name'                => null,    // Stores the instantiated class name. Used by the export field type. Also a third party custom field type uses it.
+        'section_title'             => null, // This will be assigned automatically in the formatting method.
+        'page_slug'                 => null, // This will be assigned automatically in the formatting method.
+        'tab_slug'                  => null, // This will be assigned automatically in the formatting method.
+        'option_key'                => null, // This will be assigned automatically in the formatting method.
+        'class_name'                => null, // Stores the instantiated class name. Used by the export field type. Also a third party custom field type uses it.
         'capability'                => null,        
         'title'                     => null,    
         'tip'                       => null,    
         'description'               => null,    
-        'error_message'             => null,    // error message for the field
+        'error_message'             => null, // error message for the field
         'before_label'              => null,    
         'after_label'               => null,    
         'if'                        => true,    
-        'order'                     => null,    // do not set the default number here for this key.     
+        'order'                     => null, // do not set the default number here for this key.     
         'default'                   => null,
         'value'                     => null,
-        'help'                      => null,    // 2.1.0+
-        'help_aside'                => null,    // 2.1.0+
-        'repeatable'                => null,    // 2.1.3+
-        'sortable'                  => null,    // 2.1.3+
-        'show_title_column'         => true,    // 3.0.0+
-        'hidden'                    => null,    // 3.0.0+
+        'help'                      => null, // 2.1.0+
+        'help_aside'                => null, // 2.1.0+
+        'repeatable'                => null, // 2.1.3+
+        'sortable'                  => null, // 2.1.3+
+        'show_title_column'         => true, // 3.0.0+
+        'hidden'                    => null, // 3.0.0+
 
         // @todo    Examine why an array is not set but null here for the attributes argument.
-        'attributes'                => null,    // 3.0.0+ - the array represents the attributes of input tag
+        'attributes'                => null, // 3.0.0+ - the array represents the attributes of input tag
         'class'                     => array(   // 3.3.1+
             'fieldrow'  =>  array(),
             'fieldset'  =>  array(),
@@ -66,21 +66,21 @@ 
             'field'     =>  array(),
         ), 
 
-        'save'                      => true,    // 3.6.0+
-        'content'                   => null,    // 3.6.1+ - (string) An overriding field-set output.
+        'save'                      => true, // 3.6.0+
+        'content'                   => null, // 3.6.1+ - (string) An overriding field-set output.
         
         // Internal Keys
-        '_fields_type'              => null,    // @deprecated  3.7.0++, 3.0.0+ - an internal key that indicates the fields type such as page, meta box for pages, meta box for posts, or taxonomy.
-        '_structure_type'           => null,    // DEVVEr+
-        '_caller_object'            => null,    // 3.4.0+ - (object) stores the object of the caller class. The object is referenced when creating nested fields.
+        '_fields_type'              => null, // @deprecated  3.7.0++, 3.0.0+ - an internal key that indicates the fields type such as page, meta box for pages, meta box for posts, or taxonomy.
+        '_structure_type'           => null, // DEVVEr+
+        '_caller_object'            => null, // 3.4.0+ - (object) stores the object of the caller class. The object is referenced when creating nested fields.
                 
-        '_section_path'             => '',      // 3.7.0+ (string) Stores the section path that indicates the structural address of the nested section. e.g. my_section|nested_one
-        '_section_path_array'       => '',      // 3.7.0+ (array) An array version of the above section path.
-        '_nested_depth'             => 0,       // 3.4.0+ - (integer) stores the level of the nesting depth. This is mostly used for debugging by checking if the field is a nested field or not.        
-        '_subsection_index'         => null,    // 3.7.0+  Passed to the `field_definition_{...}` filter hook callbacks.
+        '_section_path'             => '', // 3.7.0+ (string) Stores the section path that indicates the structural address of the nested section. e.g. my_section|nested_one
+        '_section_path_array'       => '', // 3.7.0+ (array) An array version of the above section path.
+        '_nested_depth'             => 0, // 3.4.0+ - (integer) stores the level of the nesting depth. This is mostly used for debugging by checking if the field is a nested field or not.        
+        '_subsection_index'         => null, // 3.7.0+  Passed to the `field_definition_{...}` filter hook callbacks.
         
-        '_field_path'               => '',      // 3.7.0+ (string Stores the field path that indicates the structural location of the field. This is relative to the belonging section.
-        '_field_path_array'         => '',      // 3.7.0+ (array) An array version of the above field path.
+        '_field_path'               => '', // 3.7.0+ (string Stores the field path that indicates the structural location of the field. This is relative to the belonging section.
+        '_field_path_array'         => '', // 3.7.0+ (array) An array version of the above field path.
         
     );        
     
@@ -157,8 +157,8 @@ 
             array( 
                 '_fields_type'          => $this->sStructureType, // @deprecated 3.7.0 backward-compatibility
                 '_structure_type'       => $this->sStructureType,  
-                '_caller_object'        => $this->oCallerObject,  // 3.4.1+ Stores the caller framework factory object. 
-                '_subsection_index'     => $this->iSubSectionIndex,  // 3.7.0+
+                '_caller_object'        => $this->oCallerObject, // 3.4.1+ Stores the caller framework factory object. 
+                '_subsection_index'     => $this->iSubSectionIndex, // 3.7.0+
             )
             + $this->aFieldset,
             array( 
@@ -177,9 +177,9 @@ 
 // @todo when nested fields are supported, set an appropriate value here.
 $_aFieldset[ '_field_path' ]        = $this->getFormElementPath( $_aFieldset[ 'field_id' ] );
 $_aFieldset[ '_field_path_array' ]  = explode( '|', $_aFieldset[ '_field_path' ] );
-$_aFieldset[ '_nested_depth' ]      = count( $_aFieldset[ '_field_path_array' ] ) - 1 ;
+$_aFieldset[ '_nested_depth' ]      = count( $_aFieldset[ '_field_path_array' ] ) - 1;
         
-        $_aFieldset[ 'order' ]       = $this->getAOrB(
+        $_aFieldset[ 'order' ] = $this->getAOrB(
             is_numeric( $_aFieldset[ 'order' ] ),
             $_aFieldset[ 'order' ],
             $this->iCountOfElements + 10

development/factory/admin_page/form/AdminPageFramework_Form_admin_page.php

Spacing +19 added lines, -19 removed lines

@@ -52,16 +52,16 @@ 
     public function getPageOnlyOptions( $aOptions, $sPageSlug ) {
 
         $_aStoredOptionsOfThePage = array();
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
             
             // Check the section
-            if ( ! $this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
+            if ( !$this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
                 continue;
             }
 
             // At this point, the element belongs the given page slug as the section is of the given page slug's.
             $this->_setPageOnlyOptions( 
-                $_aStoredOptionsOfThePage,  // by reference - gets updated in the method.
+                $_aStoredOptionsOfThePage, // by reference - gets updated in the method.
                 $aOptions, 
                 $_aSubSectionsOrFields, 
                 $sPageSlug,
@@ -81,7 +81,7 @@ 
          * @internal
          */
         private function _setPageOnlyOptions( array &$_aStoredOptionsOfThePage, array $aOptions, array $_aSubSectionsOrFields, $sPageSlug, $_sSectionID ) {
-            foreach( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
                 
                 // If it's a sub-section array,
                 if ( $this->isNumericInteger( $_sFieldID ) ) {
@@ -153,7 +153,7 @@ 
     public function getOtherPageOptions( $aOptions, $sPageSlug ) {
 
         $_aStoredOptionsNotOfThePage = array();
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
             
             // Check the section
             if ( $this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
@@ -182,7 +182,7 @@ 
          * @internal
          */
         private function _setOtherPageOptions( array &$_aStoredOptionsNotOfThePage, array $aOptions, array $_aSubSectionsOrFields, $sPageSlug ) {
-            foreach( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
 
                 // It's a sub-section array. 
                 if ( $this->isNumericInteger( $_sFieldID ) ) {
@@ -245,7 +245,7 @@ 
     public function getOtherTabOptions( $aOptions, $sPageSlug, $sTabSlug ) {
 
         $_aStoredOptionsNotOfTheTab = array();
-        foreach( $this->aFieldsets as $_sSectionPath => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionPath => $_aSubSectionsOrFields ) {
                         
             // If the section is of the given page and the given tab, skip.
             if ( $this->_isThisSectionSetToThisTab( $_sSectionPath, $sPageSlug, $sTabSlug ) ) {
@@ -276,7 +276,7 @@ 
         private function _setOtherTabOptions( array &$_aStoredOptionsNotOfTheTab, array $aOptions, array $_aSubSectionsOrFields, $sSectionPath ) {
             
             // At this point, the passed element belongs to the other tabs since the section of the given tab is skipped.
-            foreach ( $_aSubSectionsOrFields as $_isSubSectionIndexOrFieldID => $_aSubSectionOrField  ) {
+            foreach ( $_aSubSectionsOrFields as $_isSubSectionIndexOrFieldID => $_aSubSectionOrField ) {
                 
                 // If it's a sub section
                 if ( $this->isNumericInteger( $_isSubSectionIndexOrFieldID ) ) {
@@ -333,7 +333,7 @@ 
      * @since       3.7.0      Moved from `AdminPageFramework_FormDefinition_Page`.
      * @return      array
      */
-    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug='' ) {     
+    public function getTabOptions( $aOptions, $sPageSlug, $sTabSlug = '' ) {     
         $_aOtherTabOptions = $this->getOtherTabOptions( $aOptions, $sPageSlug, $sTabSlug );
         return $this->invertCastArrayContents( $aOptions, $_aOtherTabOptions );     
     }
@@ -348,17 +348,17 @@ 
      * @since       3.7.0      Moved from `AdminPageFramework_FormDefinition_Page`.
      * @return      array
      */
-    public function getTabOnlyOptions( array $aOptions, $sPageSlug, $sTabSlug='' ) {
+    public function getTabOnlyOptions( array $aOptions, $sPageSlug, $sTabSlug = '' ) {
         
         $_aStoredOptionsOfTheTab = array();
-        if ( ! $sTabSlug ) { 
+        if ( !$sTabSlug ) { 
             return $_aStoredOptionsOfTheTab; 
         }
         
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
              
             // Check the section
-            if ( ! $this->_isThisSectionSetToThisTab( $_sSectionID, $sPageSlug, $sTabSlug ) ) {
+            if ( !$this->_isThisSectionSetToThisTab( $_sSectionID, $sPageSlug, $sTabSlug ) ) {
                 continue;
             }
             
@@ -384,7 +384,7 @@ 
          */
         private function _setTabOnlyOptions( array &$_aStoredOptionsOfTheTab, array $aOptions, array $_aSubSectionsOrFields, $_sSectionID ) {
             
-            foreach( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aSubSectionsOrFields as $_sFieldID => $_aFieldset ) {
                                 
                 // if it's a sub-section array.
                 if ( $this->isNumericInteger( $_sFieldID ) ) {
@@ -441,7 +441,7 @@ 
      */
     private function _isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) {
         
-        if ( ! isset( $this->aSectionsets[ $_sSectionID ][ 'page_slug' ] ) ) {
+        if ( !isset( $this->aSectionsets[ $_sSectionID ][ 'page_slug' ] ) ) {
             return false;
         }
         return ( 
@@ -459,10 +459,10 @@ 
      */
     private function _isThisSectionSetToThisTab( $_sSectionID, $sPageSlug, $sTabSlug ) {
         
-        if ( ! $this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
+        if ( !$this->_isThisSectionSetToThisPage( $_sSectionID, $sPageSlug ) ) {
             return false;
         }
-        if ( ! isset( $this->aSectionsets[ $_sSectionID ][ 'tab_slug' ] ) ) {
+        if ( !isset( $this->aSectionsets[ $_sSectionID ][ 'tab_slug' ] ) ) {
             return false;
         }
         return (
@@ -479,9 +479,9 @@ 
      */
     private function _setOptionValue( &$aSubject, $asDimensionalPath, $aOptions ) {
         $_aDimensionalPath = $this->getAsArray( $asDimensionalPath );
-        $_mValue     = $this->getElement(
+        $_mValue = $this->getElement(
             $aOptions,
-            $_aDimensionalPath,    // as of 3.7.0, it can be an array or string
+            $_aDimensionalPath, // as of 3.7.0, it can be an array or string
             null
         );
         if ( isset( $_mValue ) ) {

development/factory/admin_page/_model/delegate/validaor/AdminPageFramework_Model__FormSubmission__Validator__Import.php

Spacing +11 added lines, -11 removed lines

@@ -32,7 +32,7 @@ 
      * @callback    action      try_validation_after_{class name}
      */
     public function _replyToCallback( $aInputs, $aRawInputs, array $aSubmits, $aSubmitInformation, $oFactory ) {
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return;
         }
         $this->_doImportOptions(
@@ -102,9 +102,9 @@ 
                 // Check the uploaded file MIME type.
                 $_aMIMEType = $this->_getImportMIMEType( $_aArguments );
                 $_sType     = $_oImport->getType();
-                if ( ! in_array( $_sType, $_aMIMEType ) ) {        
+                if ( !in_array( $_sType, $_aMIMEType ) ) {        
                     $this->oFactory->setSettingNotice( sprintf( $this->oFactory->oMsg->get( 'uploaded_file_type_not_supported' ), $_sType ) );
-                    return $aStoredOptions;        // do not change the framework's options.
+                    return $aStoredOptions; // do not change the framework's options.
                 }
 
                 // Retrieve the importing data.
@@ -169,7 +169,7 @@ 
                         array( 
                             'text/plain', 
                             'application/octet-stream', // .json file is dealt as a binary file.
-                            'application/json',  // 3.7.0+ some servers cannot upload json files without this
+                            'application/json', // 3.7.0+ some servers cannot upload json files without this
                         ), 
                         $aArguments
                     ); 
@@ -258,18 +258,18 @@ 
     protected function _getPortFilterHookNames( $sPrefix, array $aArguments ) {
         
         return array(
-            $sPrefix . $aArguments[ 'class_name' ] . '_' . $aArguments[ 'pressed_input_id' ],
+            $sPrefix.$aArguments[ 'class_name' ].'_'.$aArguments[ 'pressed_input_id' ],
             $aArguments[ 'section_id' ] 
-                ? $sPrefix . $aArguments[ 'class_name' ] . '_' . $aArguments[ 'section_id' ] .'_' . $aArguments[ 'pressed_field_id' ]
-                : $sPrefix . $aArguments[ 'class_name' ] . '_' . $aArguments[ 'pressed_field_id' ],
+                ? $sPrefix.$aArguments[ 'class_name' ].'_'.$aArguments[ 'section_id' ].'_'.$aArguments[ 'pressed_field_id' ]
+                : $sPrefix.$aArguments[ 'class_name' ].'_'.$aArguments[ 'pressed_field_id' ],
             $aArguments[ 'section_id' ] 
-                ? $sPrefix . $aArguments[ 'class_name' ] . '_' . $aArguments[ 'section_id' ] 
+                ? $sPrefix.$aArguments[ 'class_name' ].'_'.$aArguments[ 'section_id' ] 
                 : null,
             $aArguments[ 'tab_slug' ] 
-                ? $sPrefix . $aArguments[ 'page_slug' ] . '_' . $aArguments[ 'tab_slug' ]
+                ? $sPrefix.$aArguments[ 'page_slug' ].'_'.$aArguments[ 'tab_slug' ]
                 : null,
-            $sPrefix . $aArguments[ 'page_slug' ],
-            $sPrefix . $aArguments[ 'class_name' ]
+            $sPrefix.$aArguments[ 'page_slug' ],
+            $sPrefix.$aArguments[ 'class_name' ]
         );            
         
     }