michaeluno / admin-page-framework

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

Spacing +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
 
         $_aParameters = func_get_args() + array( 
             $this->aFieldset, 
-            $this->aSavedData,    // passed by reference. @todo: examine why it needs to be passed by reference.
+            $this->aSavedData, // passed by reference. @todo: examine why it needs to be passed by reference.
             $this->aFieldErrors, 
             $this->aFieldTypeDefinitions, 
             $this->aCallbacks, // field output element callables.        
@@ -71,7 +71,7 @@ 
         return $this->_getFieldByContainer( 
             $aFieldset, 
             array(
-                'open_main'     => "<div " . $_oFieldrowAttribute->get() . ">",
+                'open_main'     => "<div ".$_oFieldrowAttribute->get().">",
                 'close_main'    => "</div>",
             )
         ); 

Braces +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
         
         $aFieldset = $this->aFieldset;
 
-        if ( 'section_title' === $aFieldset[ 'type' ] ) { 
+        if ( 'section_title' === $aFieldset[ 'type' ] ) {
             return ''; 
         }
         

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

Spacing +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
      * @return      string
      * @since       DEVVER
      */
-    public function get( $bTableRow=true ) {
+    public function get( $bTableRow = true ) {
         
         $_sMethodName = $this->getAOrB(
             $bTableRow,
@@ -66,7 +66,7 @@ 
         );
         
         $_aOutput = array();
-        foreach( $this->aFieldsetsPerSection as $_aFieldset ) {
+        foreach ( $this->aFieldsetsPerSection as $_aFieldset ) {
 
             $_oFieldsetOutputFormatter = new AdminPageFramework_Form_Model___Format_FieldsetOutput(
                 $_aFieldset, 
@@ -74,7 +74,7 @@ 
                 $this->aFieldTypeDefinitions
             );        
             
-            $_aOutput[] = call_user_func_array(
+            $_aOutput[ ] = call_user_func_array(
                 array( $this, $_sMethodName ),
                 array( $_oFieldsetOutputFormatter->get() )
             );

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

Spacing +16 added lines, -16 removed lines

@@ -33,7 +33,7 @@ 
 
         $_aParameters = func_get_args() + array( 
             $this->aFieldset, 
-            $this->aSavedData,    // passed by reference. @todo: examine why it needs to be passed by reference.
+            $this->aSavedData, // passed by reference. @todo: examine why it needs to be passed by reference.
             $this->aFieldErrors, 
             $this->aFieldTypeDefinitions, 
             $this->aCallbacks, // field output element callables.        
@@ -65,10 +65,10 @@ 
             return ''; 
         }
         
-        $_oFieldrowAttribute   = new AdminPageFramework_Form_View___Attribute_Fieldrow( 
+        $_oFieldrowAttribute = new AdminPageFramework_Form_View___Attribute_Fieldrow( 
             $aFieldset,
             array( 
-                'id'        => 'fieldrow-' . $aFieldset[ 'tag_id' ],
+                'id'        => 'fieldrow-'.$aFieldset[ 'tag_id' ],
                 'valign'    => 'top',
                 'class'     => 'admin-page-framework-fieldrow',
             )                
@@ -77,7 +77,7 @@ 
         return $this->_getFieldByContainer( 
             $aFieldset, 
             array(
-                'open_container'    => "<tr " . $_oFieldrowAttribute->get() . ">",
+                'open_container'    => "<tr ".$_oFieldrowAttribute->get().">",
                 'close_container'   => "</tr>",
                 'open_title'        => "<th>",
                 'close_title'       => "</th>",
@@ -117,13 +117,13 @@ 
                 'close_main'        => '',
             );
             
-            $_aOutput   = array();
+            $_aOutput = array();
             if ( $aFieldset[ 'show_title_column' ] ) {
-                $_aOutput[] = $aOpenCloseTags[ 'open_title' ]
+                $_aOutput[ ] = $aOpenCloseTags[ 'open_title' ]
                         . $this->_getFieldTitle( $aFieldset )
                     . $aOpenCloseTags[ 'close_title' ];
             }
-            $_aOutput[] = $aOpenCloseTags[ 'open_main' ]
+            $_aOutput[ ] = $aOpenCloseTags[ 'open_main' ]
                     // . call_user_func_array( $hfCallback, array( $aFieldset ) )
                     . $this->getFieldsetOutput( $aFieldset )
                 . $aOpenCloseTags[ 'close_main' ];
@@ -147,22 +147,22 @@ 
                     $aField,
                     0   // the first item
                 );
-                return "<label for='" . $_oInputTagIDGenerator->get() . "'>"
-                        . "<a id='{$aField['field_id']}'></a>"  // to allow the browser to link to the element.
+                return "<label for='".$_oInputTagIDGenerator->get()."'>"
+                        . "<a id='{$aField[ 'field_id' ]}'></a>"  // to allow the browser to link to the element.
                         . "<span title='" 
                                 . esc_attr( strip_tags( 
-                                    isset( $aField['tip'] ) 
-                                        ? $aField['tip'] 
+                                    isset( $aField[ 'tip' ] ) 
+                                        ? $aField[ 'tip' ] 
                                         : ( 
-                                            is_array( $aField['description'] 
-                                                ? implode( '&#10;', $aField['description'] )
-                                                : $aField['description'] 
+                                            is_array( $aField[ 'description' ] 
+                                                ? implode( '&#10;', $aField[ 'description' ] )
+                                                : $aField[ 'description' ] 
                                             ) 
                                         ) 
                                 ) ) 
                             . "'>"
-                                . $aField['title'] 
-                            . ( in_array( $aField[ '_structure_type' ], array( 'widget', 'post_meta_box', 'page_meta_box' ) ) && isset( $aField['title'] ) && '' !== $aField['title']
+                                . $aField[ 'title' ] 
+                            . ( in_array( $aField[ '_structure_type' ], array( 'widget', 'post_meta_box', 'page_meta_box' ) ) && isset( $aField[ 'title' ] ) && '' !== $aField[ 'title' ]
                                 ? "<span class='title-colon'>:</span>" 
                                 : ''
                             )

Braces +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
         
         $aFieldset = $this->aFieldset;
 
-        if ( 'section_title' === $aFieldset[ 'type' ] ) { 
+        if ( 'section_title' === $aFieldset[ 'type' ] ) {
             return ''; 
         }
         

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

Spacing +8 added lines, -8 removed lines

@@ -76,15 +76,15 @@ 
         $_oSectionTableBodyAttributes = new AdminPageFramework_Form_View___Attribute_SectionTableBody( $this->aSectionset );
         
         $_aOutput       = array();
-        $_aOutput[]     = "<table " . $_oSectionTableAttributes->get() . ">"
+        $_aOutput[ ]     = "<table ".$_oSectionTableAttributes->get().">"
                 . $_oTableCaption->get()
-                . "<tbody " . $_oSectionTableBodyAttributes->get() . ">"
+                . "<tbody ".$_oSectionTableBodyAttributes->get().">"
                     . $this->_getSectionContent( $_iSectionIndex )
                 . "</tbody>"
             . "</table>";
 
-        $_oSectionTableContainerAttributes  = new AdminPageFramework_Form_View___Attribute_SectionTableContainer( $this->aSectionset );
-        return "<div " . $_oSectionTableContainerAttributes->get() . ">"
+        $_oSectionTableContainerAttributes = new AdminPageFramework_Form_View___Attribute_SectionTableContainer( $this->aSectionset );
+        return "<div ".$_oSectionTableContainerAttributes->get().">"
                 . implode( PHP_EOL, $_aOutput )
             . "</div>";    
         
@@ -144,7 +144,7 @@ 
                 // Retrieve the formatted sectionsets of the content.
                 $_sSectionPath = $this->aSectionset[ '_section_path' ];
                 $_aSectionsets = $this->aStructure[ 'sectionsets' ];
-                if ( ! isset( $_aSectionsets[ $_sSectionPath ] ) ) {    // @todo    not sure what this check is for
+                if ( !isset( $_aSectionsets[ $_sSectionPath ] ) ) {    // @todo    not sure what this check is for
                     return '';
                 }          
                 
@@ -195,14 +195,14 @@ 
                     $_aNestedSectionPaths = array();
                     
                     // List the section paths of the direct children
-                    foreach( $aNestedSctionsets as $_aNestedSectionset ) {
-                        $_sThisSectionPath = $sSubjectSectionPath . '|' . $_aNestedSectionset[ 'section_id' ];
+                    foreach ( $aNestedSctionsets as $_aNestedSectionset ) {
+                        $_sThisSectionPath = $sSubjectSectionPath.'|'.$_aNestedSectionset[ 'section_id' ];
                         $_aNestedSectionPaths[ $_sThisSectionPath ] = $_sThisSectionPath;
                     }                    
                     
                     // Now we need children's children.
                     $_aChildSectionPaths = array();
-                    foreach( $_aNestedSectionPaths as $_sNestedSectionPath ) {
+                    foreach ( $_aNestedSectionPaths as $_sNestedSectionPath ) {
                         $_aNestedSectionsets = $this->getElementAsArray( 
                             $aSectionsets, 
                             array( $_sNestedSectionPath, 'content' )

Braces +2 added lines, -1 removed lines

@@ -144,7 +144,8 @@
                 // Retrieve the formatted sectionsets of the content.
                 $_sSectionPath = $this->aSectionset[ '_section_path' ];
                 $_aSectionsets = $this->aStructure[ 'sectionsets' ];
-                if ( ! isset( $_aSectionsets[ $_sSectionPath ] ) ) {    // @todo    not sure what this check is for
+                if ( ! isset( $_aSectionsets[ $_sSectionPath ] ) ) {
+// @todo    not sure what this check is for
                     return '';
                 }          
                 

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

Spacing +8 added lines, -8 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>";
             }    
 
@@ -110,15 +110,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,
@@ -179,7 +179,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(
@@ -188,7 +188,7 @@ 
                                 'display: none;'
                             ),
                         )
-                    ). ">" 
+                    ).">" 
                         . $_oSectionTitle->get()
                     . "</div>";                
             }
@@ -204,7 +204,7 @@ 
                 if ( $aSectionset[ 'collapsible' ] ) {
                     return '';
                 }
-                if ( ! is_callable( $hfSectionCallback ) ) {
+                if ( !is_callable( $hfSectionCallback ) ) {
                     return '';
                 }
                 
@@ -236,7 +236,7 @@ 
              */
             private function _shouldShowCaptionTitle( $aSectionset, $iSectionIndex ) {
                 
-                if ( ! $aSectionset[ 'title' ] ){
+                if ( !$aSectionset[ 'title' ] ) {
                     return false;
                 }
                 if ( $aSectionset[ 'collapsible' ] ) {

Braces +1 added lines, -1 removed lines

@@ -236,7 +236,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___SectionTitle.php

Spacing +4 added lines, -4 removed lines

@@ -18,12 +18,12 @@ 
  */
 class AdminPageFramework_Form_View___SectionTitle extends AdminPageFramework_Form_View___Section_Base {            
   
-    public $aArguments      = array(
+    public $aArguments = array(
         'title'         => null,
         'tag'           => null,
         'section_index' => null,
     );
-    public $aFieldsets               = array();
+    public $aFieldsets = array();
     public $aSavedData              = array();
     public $aFieldErrors            = array();
     public $aFieldTypeDefinitions   = array();
@@ -86,7 +86,7 @@ 
          * @since       DEVVER      Moved from `AdminPageFramework_FormPart_SectionTitle`.
          * @return      string      The section title output. 
          */
-        protected function _getSectionTitle( $sTitle, $sTag, $aFieldsets, $iSectionIndex=null, $aFieldTypeDefinitions=array() ) {
+        protected function _getSectionTitle( $sTitle, $sTag, $aFieldsets, $iSectionIndex = null, $aFieldTypeDefinitions = array() ) {
        
             $_aSectionTitleField = $this->_getSectionTitleField( $aFieldsets, $iSectionIndex, $aFieldTypeDefinitions );
             return $_aSectionTitleField
@@ -106,7 +106,7 @@ 
              */
             private function _getSectionTitleField( array $aFieldsetsets, $iSectionIndex, $aFieldTypeDefinitions ) {   
             
-                foreach( $aFieldsetsets as $_aFieldsetset ) {
+                foreach ( $aFieldsetsets as $_aFieldsetset ) {
                     
                     if ( 'section_title' !== $_aFieldsetset[ 'type' ] ) {
                         continue;

Braces +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
  * @since       DEVVER      Changed the name from `AdminPageFramework_FormPart_SectionTitle`.
  * @internal
  */
-class AdminPageFramework_Form_View___SectionTitle extends AdminPageFramework_Form_View___Section_Base {            
+class AdminPageFramework_Form_View___SectionTitle extends AdminPageFramework_Form_View___Section_Base {
   
     public $aArguments      = array(
         'title'         => null,
@@ -104,7 +104,7 @@ 
              * @since       3.6.0       Added the `$iSectionIndex` parameter. Added the `$aFieldTypeDefinitions` parameter.
              * @since       DEVVER      Moved from `AdminPageFramework_FormPart_SectionTitle`.
              */
-            private function _getSectionTitleField( array $aFieldsetsets, $iSectionIndex, $aFieldTypeDefinitions ) {   
+            private function _getSectionTitleField( array $aFieldsetsets, $iSectionIndex, $aFieldTypeDefinitions ) {
             
                 foreach( $aFieldsetsets as $_aFieldsetset ) {
                     

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

Spacing +2 added lines, -2 removed lines

@@ -54,13 +54,13 @@
     public function getFieldsetOutput( $aFieldset ) {
 
         // Check if the field is visible
-        if ( ! $this->isFieldsetVisible( $aFieldset ) ) {          
+        if ( !$this->isFieldsetVisible( $aFieldset ) ) {          
             return '';
         }
 
         $_oFieldset = new AdminPageFramework_Form_View___Fieldset( 
             $aFieldset, 
-            $this->aSavedData,    // passed by reference. @todo: examine why it needs to be passed by reference.
+            $this->aSavedData, // passed by reference. @todo: examine why it needs to be passed by reference.
             $this->aFieldErrors, 
             $this->aFieldTypeDefinitions, 
             $this->oMsg,

Braces +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
     public function getFieldsetOutput( $aFieldset ) {
 
         // Check if the field is visible
-        if ( ! $this->isFieldsetVisible( $aFieldset ) ) {          
+        if ( ! $this->isFieldsetVisible( $aFieldset ) ) {
             return '';
         }
 

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

Indentation +1 added lines, -1 removed lines

@@ -531,7 +531,7 @@
                  * @return      string      The generated section tab list as HTML.
                  */
                 private function _getSectionTabList( $sSectionTabSlug, array $aSectionTabList ) {
-                   return $sSectionTabSlug 
+                    return $sSectionTabSlug 
                         ? "<ul class='admin-page-framework-section-tabs nav-tab-wrapper'>" 
                             . implode( PHP_EOL, $aSectionTabList ) 
                             . "</ul>"

Braces +4 added lines, -4 removed lines

@@ -204,7 +204,7 @@ 
         private function _getSectionsetsTables( array $aSectionsets, array $aFieldsets, array $aCallbacks ) {
 
 
-            if ( empty( $aSectionsets ) ) { 
+            if ( empty( $aSectionsets ) ) {
                 return ''; 
             } 
             
@@ -258,7 +258,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' ],
@@ -363,7 +363,7 @@ 
                     
                     // 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,
@@ -428,7 +428,7 @@ 
                  */
                 private function _getUnsetFlagSectionInputTag( array $aSection ) {
                     
-                    if ( false !== $aSection[ 'save' ] ) {                
+                    if ( false !== $aSection[ 'save' ] ) {
                         return '';
                     }
                     return $this->getHTMLTag( 

Doc Comments +4 added lines, -1 removed lines

@@ -161,6 +161,7 @@ 
     }
         /**
          * @since       DEVVER
+         * @param string $_sOutput
          * @return      string
          */
         private function _getSpinnerOutput( $_sOutput ) {
@@ -297,7 +298,7 @@ 
              * @param       array       $_aOutputs      Holds output elements - contents, section tab list, count of subsections
              * @param       string      $_sSectionsID   The container id of secitons.
              * @param       array       $_aSection
-             * @param       array       $_aFieldsInSections     A fieldsets array already divided by section tab.
+             * @param       array       $aFieldsInSections     A fieldsets array already divided by section tab.
              * @return      array       The updated sections table output array.
              */
             private function _getSectionsetTable( $_aOutputs, $_sSectionsID, array $_aSection, array $aFieldsInSections ) {
@@ -351,6 +352,7 @@ 
             }
                 /**
                  * @since       DEVVER
+                 * @param string $_sSectionsID
                  * @return      array
                  */
                 private function _getSubSections( $_aOutputs, $_sSectionsID, $_aSection, $_aSubSections ) {
@@ -499,6 +501,7 @@ 
              * @since       3.5.3
              * @since       3.6.0       Removed the `$sSectionID` parameter. Added the `$aSectionset` parameter.
              * @since       DEVVER      Moved from `AdminPageFramework_FormPart_Table`.
+             * @param string $sSectionsID
              * @return      string      The formatted sections table HTML output.
              */
             private function _getFormattedSectionsTablesOutput( array $aOutputs, $aSectionset, $sSectionsID, array $aCollapsible, $sSectionTabSlug ) {

Spacing +39 added lines, -39 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>"
                 : '';
@@ -236,12 +236,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(
@@ -273,7 +273,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' ],
@@ -302,17 +302,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
@@ -356,29 +356,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,
@@ -408,8 +408,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' ],                            
                         )
@@ -427,8 +427,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' ],                            
                         )
@@ -450,8 +450,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',
                         )
                     );            
@@ -467,7 +467,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' ]
@@ -476,7 +476,7 @@ 
 
                     // Section container
                     $_oSectionTable = new AdminPageFramework_Form_View___Section(
-                        $this->aArguments,  // for nested sections
+                        $this->aArguments, // for nested sections
                         $aSectionset,
                         $this->aStructure,
                         $aFieldsetsPerSection,
@@ -486,7 +486,7 @@ 
                         $this->aCallbacks,
                         $this->oMsg
                     );
-                    $_aOutputs[ 'section_contents' ][] = $_oSectionTable->get();                    
+                    $_aOutputs[ 'section_contents' ][ ] = $_oSectionTable->get();                    
                    
                     return $_aOutputs;
                  
@@ -515,7 +515,7 @@ 
                         'collapsible'       => $aCollapsible,
                         'container_type'    => 'sections', // section or sections                    
                     ),
-                    array(),            // fieldsets
+                    array(), // fieldsets
                     $this->aSavedData,   
                     $this->aFieldErrors, 
                     $this->aStructure[ 'field_type_definitions' ], 
@@ -532,7 +532,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>";
@@ -563,21 +563,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(
@@ -593,7 +593,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/format/AdminPageFramework_Form_View___Format_SectionsetsByTab.php

Spacing +5 added lines, -5 removed lines

@@ -24,7 +24,7 @@ 
      * 
      * Only one depth at a time will be parsed.
      */
-    public $iNestedDepth  = 0;
+    public $iNestedDepth = 0;
     
     /**
      * Stores the generated section tab slugs.
@@ -120,11 +120,11 @@ 
             $_aFieldsBySectionTab   = array();
             $_iIndex                = 0;
 
-            foreach( $aSectionsets as $_sSectionPath => $_aSectionset ) {
+            foreach ( $aSectionsets as $_sSectionPath => $_aSectionset ) {
 
                 // If no fields for the section, no need to add the section 
                 // unless the custom sectionset output is defined.
-                if ( ! isset( $aFieldsets[ $_sSectionPath ] ) && ! $this->_isCustomContentSet( $_aSectionset ) ) {
+                if ( !isset( $aFieldsets[ $_sSectionPath ] ) && !$this->_isCustomContentSet( $_aSectionset ) ) {
                     continue;
                 }
                        
@@ -136,7 +136,7 @@ 
                 $_sSectionTaqbSlug = $this->getAOrB(
                     $_aSectionset[ 'section_tab_slug' ],
                     $_aSectionset[ 'section_tab_slug' ],
-                    '_default_' . $this->iNestedDepth . '_' . ( ++$_iIndex )
+                    '_default_'.$this->iNestedDepth.'_'.( ++$_iIndex )
                 );
                 $_aSectionsBySectionTab[ $_sSectionTaqbSlug ][ $_sSectionPath ] = $_aSectionset;
                 $_aFieldsBySectionTab[ $_sSectionTaqbSlug ][ $_sSectionPath ]   = $this->getElement(
@@ -157,7 +157,7 @@ 
              * @since       DEVVER      Moved from `AdminPageFramework_FormPart_Table`.
              * @return      boolean     True if a custom content value is set.
              */
-            private function _isCustomContentSet( array $aSectionset, array $aKeys=array( 'content' ) ) {
+            private function _isCustomContentSet( array $aSectionset, array $aKeys = array( 'content' ) ) {
                 return isset( $aSectionset[ 'content' ] );
                 // @deprecated
                 // foreach( $aKeys as $_sKey ) {