michaeluno / admin-page-framework

development/factory/_common/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( 

Spacing +44 added lines, -44 removed lines

@@ -127,16 +127,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,
@@ -150,7 +150,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
@@ -185,17 +185,17 @@ 
 
             // A sectionset is a set of sections.
             $_sSectionSet = $this->_getSectionsetsTables( 
-                $aSectionsets,  // section-set definition (already divided by section tab)
-                $aFieldsets,    // field-set definitions (already divided by section tab)
+                $aSectionsets, // section-set definition (already divided by section tab)
+                $aFieldsets, // field-set definitions (already divided 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>"
                 : '';
@@ -227,7 +227,7 @@ 
              * If there is no field overall to the section and its section tab, return an empty string.
              * Otherwise, the sectionsets container gets rendered and its CSS rules such as margins give unwanted results.
              */
-            if ( ! count( $aFieldsets ) ) {
+            if ( !count( $aFieldsets ) ) {
                 return ''; 
             }
 
@@ -239,13 +239,13 @@ 
             );
             $_sSectionTabSlug   = $_aFirstSectionset[ 'section_tab_slug' ];
             $_sThisSectionID    = $_aFirstSectionset[ 'section_id' ];
-            $_sSectionsID       = 'sections-' . $_sThisSectionID;
+            $_sSectionsID       = 'sections-'.$_sThisSectionID;
             $_aCollapsible      = $this->_getCollapsibleArgumentForSections( 
                 $_aFirstSectionset 
             );
                         
-            foreach( $aSectionsets as $_aSectionset ) {
-                $_aOutputs          = $this->_getSectionsetTable(
+            foreach ( $aSectionsets as $_aSectionset ) {
+                $_aOutputs = $this->_getSectionsetTable(
                     $_aOutputs,
                     $_sSectionsID,
                     $_aSectionset,
@@ -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' ],
@@ -301,19 +301,19 @@ 
              */
             private function _getSectionsetTable( $_aOutputs, $_sSectionsID, array $_aSection, array $aFieldsInSections ) {
             
-                if ( ! $this->isSectionsetVisible( $_aSection ) ) {
+                if ( !$this->isSectionsetVisible( $_aSection ) ) {
                     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 array
-                        $_aSection[ '_section_path' ],  // dimensional path
+                        $_aSection[ '_section_path' ], // dimensional path
                         array() // default
                     )
                 );
@@ -356,28 +356,28 @@ 
                 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' ] ) ) {
-                        $_aOutputs[ 'section_contents' ][] = AdminPageFramework_Form_View___Script_SortableSection::getEnabler( 
+                    if ( !empty( $_aSection[ 'sortable' ] ) ) {
+                        $_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,
@@ -407,8 +407,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' ],                            
                         )
@@ -426,8 +426,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' ],                            
                         )
@@ -449,8 +449,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',
                         )
                     );            
@@ -466,7 +466,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' ]
@@ -474,7 +474,7 @@ 
 
                     // Section container
                     $_oSectionTable = new AdminPageFramework_Form_View___Section(
-                        $this->aArguments,  // for nested sections
+                        $this->aArguments, // for nested sections
                         $aSectionset,
                         $this->aStructure,
                         $aFieldsetsPerSection,
@@ -484,7 +484,7 @@ 
                         $this->aCallbacks,
                         $this->oMsg
                     );
-                    $_aOutputs[ 'section_contents' ][] = $_oSectionTable->get();                    
+                    $_aOutputs[ 'section_contents' ][ ] = $_oSectionTable->get();                    
                    
                     return $_aOutputs;
                  
@@ -512,9 +512,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' ], 
@@ -531,7 +531,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>";
@@ -562,21 +562,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(
@@ -584,7 +584,7 @@ 
                         'tag'           => 'h4',
                         'section_index' => $iSectionIndex,
                         
-                        'sectionset'    => $aSection,   // 3.7.0+      for tooltip
+                        'sectionset'    => $aSection, // 3.7.0+      for tooltip
                     ),
                     $aFields,            
                     $this->aSavedData,   
@@ -594,7 +594,7 @@ 
                     $this->aCallbacks // field output element callables.                    
                 );                        
                 
-                return "<li " . $this->getAttributes( $_aTabAttributes ) . ">"
+                return "<li ".$this->getAttributes( $_aTabAttributes ).">"
                     . "<a href='#{$_sSectionTagID}'>"
                         . $_oSectionTitle->get()
                     ."</a>"

Doc Comments +4 added lines, -1 removed lines

@@ -165,6 +165,7 @@ 
     }
         /**
          * @since       3.7.0
+         * @param string $_sOutput
          * @return      string
          */
         private function _getSpinnerOutput( $_sOutput ) {
@@ -299,7 +300,7 @@ 
              * @param       array       $_aOutputs      Holds output elements - contents, section tab list, count of subsections.
              * @param       string      $_sSectionsID   The container id of sections.
              * @param       array       $_aSection
-             * @param       array       $_aFieldsInSections     A field-sets array already divided by section tab.
+             * @param       array       $aFieldsInSections     A field-sets array already divided by section tab.
              * @return      array       The updated sections table output array.
              */
             private function _getSectionsetTable( $_aOutputs, $_sSectionsID, array $_aSection, array $aFieldsInSections ) {
@@ -354,6 +355,7 @@ 
                  * Returns the output of sub-sections for repeatable and sortable sections.
                  * 
                  * @since       3.7.0
+                 * @param string $_sSectionsID
                  * @return      array
                  */
                 private function _getSubSections( $_aOutputs, $_sSectionsID, $_aSection, $_aSubSections ) {
@@ -500,6 +502,7 @@ 
              * @since       3.5.3
              * @since       3.6.0       Removed the `$sSectionID` parameter. Added the `$aSectionset` parameter.
              * @since       3.7.0       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 ) {

development/factory/_common/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' ] ) {

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/_common/form/_view/sectionset/AdminPageFramework_Form_View___Section.php

Spacing +9 added lines, -9 removed lines

@@ -77,15 +77,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>";    
         
@@ -135,7 +135,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 '';
                 }          
                 
@@ -190,22 +190,22 @@ 
                     $_aNestedSectionPaths = array();
                     
                     // List the section paths of the direct children
-                    foreach( $aNestedSctionsets as $_aNestedSectionset ) {
+                    foreach ( $aNestedSctionsets as $_aNestedSectionset ) {
                         
                         // 3.7.6+ There were cases non array gets passed (like the FAQ page in the demo)
                         // and caused warnings in PHP 7.
-                        if ( ! is_array( $_aNestedSectionset ) ) {
+                        if ( !is_array( $_aNestedSectionset ) ) {
                             continue;
                         }
                         
-                        $_sThisSectionPath = $sSubjectSectionPath . '|' . $_aNestedSectionset[ 'section_id' ];
+                        $_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/_common/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 ) {

development/factory/_common/form/_view/AdminPageFramework_Form_View___DebugInfo.php

Braces +1 added lines, -1 removed lines

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

Spacing +4 added lines, -4 removed lines

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

development/factory/_common/form/_view/resource/AdminPageFramework_Form_View__Resource__Head.php

Spacing +4 added lines, -4 removed lines

@@ -29,7 +29,7 @@ 
      * @param       object      $oForm
      * @param       string      $sHeadActionHook        The action hook triggered inside the `<head>` tag. For front-end forms, use `wp_head`.
      */
-    public function __construct( $oForm, $sHeadActionHook='admin_head' ) {
+    public function __construct( $oForm, $sHeadActionHook = 'admin_head' ) {
     
         $this->oForm = $oForm;
     
@@ -53,7 +53,7 @@ 
          * as multiple instances of the factory class is loaded among separate pages, 
          * one gets denied here and if it can load earlier than the one which should insert the below script.
          */
-        if ( ! $this->oForm->isInThePage() ) {
+        if ( !$this->oForm->isInThePage() ) {
             return;
         }    
     
@@ -76,8 +76,8 @@ 
         private function _getScripts_RequiredInHead() {
             return 'document.write( "<style class=\'admin-page-framework-js-embedded-inline-style\'>'
                     . str_replace(
-                        '\\n',   // search
-                        '',     // replace
+                        '\\n', // search
+                        '', // replace
                         esc_js( $this->_getInlineCSS() )    // subject
                     )
                 . '</style>" );';            

Braces +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
     
         $this->oForm = $oForm;
     
-        if ( in_array( $this->oForm->aArguments[ 'structure_type' ], array( 'widget' ) ) ) {       
+        if ( in_array( $this->oForm->aArguments[ 'structure_type' ], array( 'widget' ) ) ) {
             return;
         }
     
@@ -97,7 +97,7 @@ 
                  * @return      string
                  * @since       DEVVER
                  */
-                private function _getScriptElementConcealerCSSRules() {                    
+                private function _getScriptElementConcealerCSSRules() {
                     
                     // Avoid hiding framework forms in widgets. In some cases, the user sets a form in any page,
                     // which causes a different structure form type including `admin_page` gets loaded even in widgets.php (or any page builder pages dealing with widgets)

development/factory/_common/form/_view/resource/AdminPageFramework_Form_View__Resource.php

Indentation +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         // Widgets can be called multiple times for the number of user-created widget instances for one class instance 
         // so make sure it is processed only once per page.
         if ( $this->hasBeenCalled( 'resource_' . $oForm->aArguments[ 'caller_id' ] ) ) {
-             return;
+                return;
         }
 
         $this->_setHooks();

Braces +1 added lines, -1 removed lines

@@ -230,7 +230,7 @@
         );
         
         $_sID = $this->sanitizeSlug( strtolower( $this->oForm->aArguments[ 'caller_id' ] ) );
-        if ( $_sCSSRules ) {            
+        if ( $_sCSSRules ) {
             echo "<style type='text/css' id='inline-style-{$_sID}' class='admin-page-framework-form-style'>"
                     . $_sCSSRules
                 . "</style>";

Spacing +10 added lines, -10 removed lines

@@ -40,7 +40,7 @@ 
        
         // Widgets can be called multiple times for the number of user-created widget instances for one class instance 
         // so make sure it is processed only once per page.
-        if ( $this->hasBeenCalled( 'resource_' . $oForm->aArguments[ 'caller_id' ] ) ) {
+        if ( $this->hasBeenCalled( 'resource_'.$oForm->aArguments[ 'caller_id' ] ) ) {
              return;
         }
 
@@ -113,12 +113,12 @@ 
      * @since       3.7.0
      */
     public function _replyToEnqueueScripts() {
-        if ( ! $this->oForm->isInThePage() ) {
+        if ( !$this->oForm->isInThePage() ) {
             return;
         }
-        foreach( $this->oForm->getResources( 'src_scripts' ) as $_isIndex => $_asEnqueue ) {
+        foreach ( $this->oForm->getResources( 'src_scripts' ) as $_isIndex => $_asEnqueue ) {
             $this->_enqueueScript( $_asEnqueue );
-            $this->oForm->unsetResources( array( 'src_scripts', $_isIndex ) );  // no longer needed
+            $this->oForm->unsetResources( array( 'src_scripts', $_isIndex ) ); // no longer needed
         }       
     }
         /**
@@ -165,7 +165,7 @@ 
             private function _getFormattedEnqueueScript( $asEnqueue ) {
                 static $_iCallCount = 1;
                 $_aEnqueueItem = $this->getAsArray( $asEnqueue ) + array(
-                    'handle_id'     => 'script_form_' . $this->oForm->aArguments[ 'caller_id' ] . '_' . $_iCallCount,
+                    'handle_id'     => 'script_form_'.$this->oForm->aArguments[ 'caller_id' ].'_'.$_iCallCount,
                     'src'           => null,
                     'dependencies'  => null,
                     'version'       => null,
@@ -188,10 +188,10 @@ 
      */    
     public function _replyToEnqueueStyles() {
 
-        if ( ! $this->oForm->isInThePage() ) {
+        if ( !$this->oForm->isInThePage() ) {
             return;
         }
-        foreach( $this->oForm->getResources( 'src_styles' ) as $_isIndex => $_asEnqueueItem ) {
+        foreach ( $this->oForm->getResources( 'src_styles' ) as $_isIndex => $_asEnqueueItem ) {
             $this->_enqueueStyle( $_asEnqueueItem );
             $this->oForm->unsetResources( array( 'src_styles', $_isIndex ) ); // no longer needed
         }           
@@ -213,7 +213,7 @@ 
             private function _getFormattedEnqueueStyle( $asEnqueue ) {
                 static $_iCallCount = 1;
                 $_aEnqueueItem = $this->getAsArray( $asEnqueue ) + array(
-                    'handle_id'     => 'style_form_' . $this->oForm->aArguments[ 'caller_id' ] . '_' . $_iCallCount,
+                    'handle_id'     => 'style_form_'.$this->oForm->aArguments[ 'caller_id' ].'_'.$_iCallCount,
                     'src'           => null,
                     'dependencies'  => null,
                     'version'       => null,
@@ -237,7 +237,7 @@ 
      */    
     public function _replyToAddStyle() {
         
-        if ( ! $this->oForm->isInThePage() ) {
+        if ( !$this->oForm->isInThePage() ) {
             return;
         }   
         $_sCSSRules = $this->_getFormattedInternalStyles( 
@@ -279,7 +279,7 @@ 
      */    
     public function _replyToAddScript() {
         
-        if ( ! $this->oForm->isInThePage() ) {
+        if ( !$this->oForm->isInThePage() ) {
             return;
         }        
         

development/factory/_common/form/_view/fieldset/AdminPageFramework_Form_View___Fieldset.php

Braces +1 added lines, -1 removed lines

@@ -139,7 +139,7 @@
                  */
                 private function _getUnsetFlagFieldInputTag( array $aField ) {
                     
-                    if ( false !== $aField[ 'save' ] ) {                
+                    if ( false !== $aField[ 'save' ] ) {
                         return '';
                     }
                     return $this->getHTMLTag( 

Doc Comments +2 added lines

@@ -324,6 +324,7 @@ 
                  * Retrieves a field output.
                  * 
                  * @since       3.8.0
+                 * @param boolean $bIsLastElement
                  * @return      string
                  */ 
                 private function _getFieldOutput( $sContent, array $aField, $bIsLastElement ) {
@@ -402,6 +403,7 @@ 
          * Returns the final fields output.
          * 
          * @since       3.1.0
+         * @param integer $iFieldsCount
          * @return      string
          */
         private function _getFinalOutput( array $aFieldset, array $aFieldsOutput, $iFieldsCount ) {

Spacing +29 added lines, -29 removed lines

@@ -38,7 +38,7 @@ 
             $this->aFieldset[ '_field_path_array' ],
             $this->aFieldset[ 'error_message' ]
         );
-        $_aOutputs[]     = $_oFieldError->get();
+        $_aOutputs[ ] = $_oFieldError->get();
 
         // 2. Construct fields array for sub-fields.
         $_oFieldsFormatter = new AdminPageFramework_Form_Model___Format_Fields(
@@ -48,7 +48,7 @@ 
         $_aFields = $_oFieldsFormatter->get();
             
         // 3. Get the field and its sub-fields output.
-        $_aOutputs[] = $this->_getFieldsOutput( 
+        $_aOutputs[ ] = $this->_getFieldsOutput( 
             $this->aFieldset,
             $_aFields, 
             $this->aCallbacks 
@@ -71,11 +71,11 @@ 
          * @since   3.8.0   Added the `$aFieldset` parameter
          * @return  string
          */ 
-        private function _getFieldsOutput( array $aFieldset, array $aFields, array $aCallbacks=array() ) {
+        private function _getFieldsOutput( array $aFieldset, array $aFields, array $aCallbacks = array() ) {
 
             $_aOutput = array();
-            foreach( $aFields as $_isIndex => $_aField ) {
-                $_aOutput[] = $this->_getEachFieldOutput( 
+            foreach ( $aFields as $_isIndex => $_aField ) {
+                $_aOutput[ ] = $this->_getEachFieldOutput( 
                     $_aField, 
                     $_isIndex, 
                     $aCallbacks,
@@ -94,11 +94,11 @@ 
              * @since       3.5.3
              * @return      string      the HTML output of the given field.
              */
-            private function _getEachFieldOutput( array $aField, $isIndex, array $aCallbacks, $bIsLastElement=false ) {
+            private function _getEachFieldOutput( array $aField, $isIndex, array $aCallbacks, $bIsLastElement = false ) {
                 
                 // Field type definition - allows mixed field types in sub-fields 
                 $_aFieldTypeDefinition = $this->_getFieldTypeDefinition( $aField[ 'type' ] );
-                if ( ! is_callable( $_aFieldTypeDefinition[ 'hfRenderField' ] ) ) {
+                if ( !is_callable( $_aFieldTypeDefinition[ 'hfRenderField' ] ) ) {
                     return '';
                 }     
 
@@ -132,7 +132,7 @@ 
                 private function _getFieldOutput( $sContent, array $aField, $bIsLastElement ) {
                     $_oFieldAttribute = new AdminPageFramework_Form_View___Attribute_Field( $aField );
                     return $aField[ 'before_field' ]
-                        . "<div " . $_oFieldAttribute->get() . ">"
+                        . "<div ".$_oFieldAttribute->get().">"
                             . $sContent
                             . $this->_getUnsetFlagFieldInputTag( $aField )
                             . $this->_getDelimiter( $aField, $bIsLastElement )
@@ -154,7 +154,7 @@ 
                         'input',
                         array(
                             'type'  => 'hidden',
-                            'name'  => '__unset_' . $aField[ '_fields_type' ] . '[' . $aField[ '_input_name_flat' ] . ']',
+                            'name'  => '__unset_'.$aField[ '_fields_type' ].'['.$aField[ '_input_name_flat' ].']',
                             'value' => $aField[ '_input_name_flat' ],
                             'class' => 'unset-element-names element-address',
                         )
@@ -185,7 +185,7 @@ 
                  */
                 private function _getDelimiter( array $aField, $bIsLastElement ) {
                     return $aField[ 'delimiter' ]
-                        ? "<div " . $this->getAttributes( 
+                        ? "<div ".$this->getAttributes( 
                                 array(
                                     'class' => 'delimiter',
                                     'id'    => "delimiter-{$aField[ 'input_id' ]}",
@@ -195,7 +195,7 @@ 
                                         ""
                                     ),
                                 ) 
-                            ) . ">"
+                            ).">"
                                 . $aField[ 'delimiter' ]
                             . "</div>"
                         : '';
@@ -209,9 +209,9 @@ 
          */
         private function _getFinalOutput( array $aFieldset, array $aFieldsOutput, $iFieldsCount ) {
                             
-            $_oFieldsetAttributes   = new AdminPageFramework_Form_View___Attribute_Fieldset( $aFieldset );
+            $_oFieldsetAttributes = new AdminPageFramework_Form_View___Attribute_Fieldset( $aFieldset );
             return $aFieldset[ 'before_fieldset' ]
-                . "<fieldset " . $_oFieldsetAttributes->get() . ">"
+                . "<fieldset ".$_oFieldsetAttributes->get().">"
                     . $this->_getEmbeddedFieldTitle( $aFieldset )
                     . $this->_getChildFieldTitle( $aFieldset )                
                     . $this->_getFieldsetContent( $aFieldset, $aFieldsOutput, $iFieldsCount )
@@ -228,7 +228,7 @@ 
              */
             private function _getEmbeddedFieldTitle( array $aFieldset ) {
                 
-                if ( ! $aFieldset[ '_is_title_embedded' ] ) {
+                if ( !$aFieldset[ '_is_title_embedded' ] ) {
                     return '';
                 }
 
@@ -253,7 +253,7 @@ 
              */
             private function _getChildFieldTitle( array $aFieldset ) {
 
-                if ( ! $aFieldset[ '_nested_depth' ] ) {
+                if ( !$aFieldset[ '_nested_depth' ] ) {
                     return '';
                 }
                 if ( $aFieldset[ '_is_title_embedded' ] ) {
@@ -282,12 +282,12 @@ 
                     return $aFieldset[ 'content' ];
                 }
             
-                $_oFieldsAttributes     = new AdminPageFramework_Form_View___Attribute_Fields( 
+                $_oFieldsAttributes = new AdminPageFramework_Form_View___Attribute_Fields( 
                     $aFieldset, 
-                    array(),    // attribute array
+                    array(), // attribute array
                     $iFieldsCount
                 );
-                return "<div " . $_oFieldsAttributes->get() . ">"
+                return "<div ".$_oFieldsAttributes->get().">"
                         . $aFieldset[ 'before_fields' ]
                             . implode( PHP_EOL, $aFieldsOutput )
                         . $aFieldset[ 'after_fields' ]
@@ -310,13 +310,13 @@ 
                     $aField[ 'description' ],
                     'admin-page-framework-fields-description'   // class selector
                 );
-                $_aOutput[] = $_oFieldDescription->get();
+                $_aOutput[ ] = $_oFieldDescription->get();
                     
                 // Dimensional keys of repeatable and sortable fields
-                $_aOutput[] = $this->_getDynamicElementFlagFieldInputTag( $aField );
+                $_aOutput[ ] = $this->_getDynamicElementFlagFieldInputTag( $aField );
                     
                 // Repeatable and sortable scripts 
-                $_aOutput[] = $this->_getFieldScripts( $aField, $iFieldsCount );
+                $_aOutput[ ] = $this->_getFieldScripts( $aField, $iFieldsCount );
                 
                 return implode( PHP_EOL, array_filter( $_aOutput ) );
                 
@@ -346,8 +346,8 @@ 
                             'input',
                             array(
                                 'type'                      => 'hidden',
-                                'name'                      => '__repeatable_elements_' . $aFieldset[ '_structure_type' ] 
-                                    . '[' . $aFieldset[ '_field_address' ] . ']',
+                                'name'                      => '__repeatable_elements_'.$aFieldset[ '_structure_type' ] 
+                                    . '['.$aFieldset[ '_field_address' ].']',
                                 'class'                     => 'element-address',
                                 'value'                     => $aFieldset[ '_field_address' ],
                                 'data-field_address_model'  => $aFieldset[ '_field_address_model' ],
@@ -363,8 +363,8 @@ 
                             'input',
                             array(
                                 'type'                      => 'hidden',
-                                'name'                      => '__sortable_elements_' . $aFieldset[ '_structure_type' ] 
-                                    . '[' . $aFieldset[ '_field_address' ] . ']',
+                                'name'                      => '__sortable_elements_'.$aFieldset[ '_structure_type' ] 
+                                    . '['.$aFieldset[ '_field_address' ].']',
                                 'class'                     => 'element-address',
                                 'value'                     => $aFieldset[ '_field_address' ],
                                 'data-field_address_model'  => $aFieldset[ '_field_address_model' ],
@@ -383,14 +383,14 @@ 
                     $_aOutput   = array();
                     
                     // Add the repeater script 
-                    $_aOutput[] = $aField[ 'repeatable' ]
-                        ? $this->_getRepeaterFieldEnablerScript( 'fields-' . $aField['tag_id'], $iFieldsCount, $aField['repeatable'] )
+                    $_aOutput[ ] = $aField[ 'repeatable' ]
+                        ? $this->_getRepeaterFieldEnablerScript( 'fields-'.$aField[ 'tag_id' ], $iFieldsCount, $aField[ 'repeatable' ] )
                         : '';
 
                     // Add the sortable script - if the number of fields is only one, no need to sort the field. 
                     // Repeatable fields can make the number increase so here it checks the repeatability.
-                    $_aOutput[] = $aField['sortable'] && ( $iFieldsCount > 1 || $aField['repeatable'] )
-                        ? $this->_getSortableFieldEnablerScript( 'fields-' . $aField['tag_id'] )
+                    $_aOutput[ ] = $aField[ 'sortable' ] && ( $iFieldsCount > 1 || $aField[ 'repeatable' ] )
+                        ? $this->_getSortableFieldEnablerScript( 'fields-'.$aField[ 'tag_id' ] )
                         : '';     
                     
                     return implode( PHP_EOL, $_aOutput );

development/factory/_common/form/_view/attribute/AdminPageFramework_Form_View___Attribute_SectionTableContainer.php

Spacing +6 added lines, -6 removed lines

@@ -27,8 +27,8 @@ 
      */
     protected function _getAttributes() {
         
-        $_aSectionAttributes    = $this->uniteArrays(
-            $this->dropElementsByType( $this->aArguments[ 'attributes' ] ),   // remove elements of an array.
+        $_aSectionAttributes = $this->uniteArrays(
+            $this->dropElementsByType( $this->aArguments[ 'attributes' ] ), // remove elements of an array.
             array( 
                 'id'            => $this->aArguments[ '_tag_id' ], // section-{section id}__{index}
                 'class'         => $this->getClassAttribute( 
@@ -49,19 +49,19 @@ 
             )     
         );
         
-        $_aSectionAttributes[ 'class' ]   = $this->getClassAttribute( 
+        $_aSectionAttributes[ 'class' ] = $this->getClassAttribute( 
             $_aSectionAttributes[ 'class' ], 
             $this->dropElementsByType( $this->aArguments[ 'class' ] )
-        );  // 3.3.1+
+        ); // 3.3.1+
         
-        $_aSectionAttributes[ 'style' ]   = $this->getStyleAttribute( 
+        $_aSectionAttributes[ 'style' ] = $this->getStyleAttribute( 
             $_aSectionAttributes[ 'style' ], 
             $this->getAOrB(
                 $this->aArguments[ 'hidden' ],
                 'display:none',
                 null
             )
-        );  // 3.3.1+        
+        ); // 3.3.1+        
 
         return $_aSectionAttributes;