michaeluno / admin-page-framework

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

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
      * </code>
      * @access       public      This must be public as accessed from outside.
      */    
-    public $aFieldTypeSlugs = array( 'default', );
+    public $aFieldTypeSlugs = array( 'default',);
     
     /**
      * Defines the default key-values of this field type. 

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

Braces +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
     protected function getField( $aField ) {
             
         $_oSelectInput = new AdminPageFramework_Input_select( $aField[ 'attributes' ] );
-        if ( $aField[ 'is_multiple' ]  ) {            
+        if ( $aField[ 'is_multiple' ]  ) {
             $_oSelectInput->setAttribute( array( 'select', 'multiple' ), 'multiple' );
         }
         return

Spacing +4 added lines, -4 removed lines

@@ -103,7 +103,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'select', );
+    public $aFieldTypeSlugs = array( 'select',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -115,7 +115,7 @@ 
             'select'    => array(
                 'size'          => 1,
                 'autofocusNew'  => null,
-                'multiple'      => null,    // set 'multiple' for multiple selections. If 'is_multiple' is set, it takes the precedence.
+                'multiple'      => null, // set 'multiple' for multiple selections. If 'is_multiple' is set, it takes the precedence.
                 'required'      => null,     
             ),
             'optgroup'  => array(),
@@ -155,12 +155,12 @@ 
     protected function getField( $aField ) {
             
         $_oSelectInput = new AdminPageFramework_Input_select( $aField[ 'attributes' ] );
-        if ( $aField[ 'is_multiple' ]  ) {            
+        if ( $aField[ 'is_multiple' ] ) {            
             $_oSelectInput->setAttribute( array( 'select', 'multiple' ), 'multiple' );
         }
         return
             $aField[ 'before_label' ]
-            . "<div " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-container admin-page-framework-select-label' ) . ">" 
+            . "<div ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-container admin-page-framework-select-label' ).">" 
                 . "<label for='{$aField[ 'input_id' ]}'>"
                     . $aField[ 'before_input' ]
                     . $_oSelectInput->get( $aField[ 'label' ] )

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

Braces +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
          */
         private function _getCollapsibleSectionTitleBlock( array $aCollapsible, $sContainer='sections', $iSectionIndex=null ) {
 
-            if ( $sContainer !== $aCollapsible[ 'container' ] ) { 
+            if ( $sContainer !== $aCollapsible[ 'container' ] ) {
                 return ''; 
             }
               

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>"

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 ) {

Spacing +44 added lines, -44 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,
@@ -152,7 +152,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
@@ -187,17 +187,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>"
                 : '';
@@ -229,7 +229,7 @@ 
              * If there is no field overall to the section and its section tab, return an empty string.
              * Otherwise, the section-sets container gets rendered and its CSS rules such as margins give unwanted results.
              */
-            if ( ! count( $aFieldsets ) ) {
+            if ( !count( $aFieldsets ) ) {
                 return ''; 
             }
 
@@ -241,13 +241,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,
@@ -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' ],
@@ -303,19 +303,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
                     )
                 );
@@ -358,28 +358,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,
@@ -409,8 +409,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' ],                            
                         )
@@ -428,8 +428,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' ],                            
                         )
@@ -451,8 +451,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',
                         )
                     );            
@@ -468,7 +468,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;
                  
@@ -514,9 +514,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' ], 
@@ -533,7 +533,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>";
@@ -564,21 +564,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(
@@ -586,7 +586,7 @@ 
                         'tag'           => 'h4',
                         'section_index' => $iSectionIndex,
                         
-                        'sectionset'    => $aSection,   // 3.7.0+      for tooltip
+                        'sectionset'    => $aSection, // 3.7.0+      for tooltip
                     ),
                     $aFields,            
                     $this->aSavedData,   
@@ -596,7 +596,7 @@ 
                     $this->aCallbacks // field output element callables.                    
                 );                        
                 
-                return "<li " . $this->getAttributes( $_aTabAttributes ) . ">"
+                return "<li ".$this->getAttributes( $_aTabAttributes ).">"
                     . "<a href='#{$_sSectionTagID}'>"
                         . $_oSectionTitle->get()
                     ."</a>"

Braces +4 added lines, -4 removed lines

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

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/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();

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;
         }        
         

Braces +1 added lines, -1 removed lines

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

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

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
         private function _getFieldError( $aErrors, $aSectionPath, $aFieldPath, $sHeadingMessage ) {
 
             // If this field has a section and the error element is set
-            $_aErrorPath   = array_merge( $aSectionPath, $aFieldPath );
+            $_aErrorPath = array_merge( $aSectionPath, $aFieldPath );
             if ( $this->_hasFieldError( $aErrors, $_aErrorPath ) ) {
                 return "<span class='field-error'>*&nbsp;"
                         . $sHeadingMessage

Doc Comments +1 added lines

@@ -75,6 +75,7 @@
          * 
          * @since       3.1.0
          * @since       3.7.0      Moved from `AdminPageFramework_Form_View___Fieldset`.
+         * @param string $sHeadingMessage
          * @return      string      The error string message. An empty value if not found.
          */
         private function _getFieldError( $aErrors, $aSectionPath, $aFieldPath, $sHeadingMessage ) {