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/field_type/AdminPageFramework_FieldType_import.php

Spacing +34 added lines, -34 removed lines

@@ -20,7 +20,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'import', );
+    public $aFieldTypeSlugs = array( 'import',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -91,9 +91,9 @@ 
     protected function getField( $aField ) {
         
         /* Set some required values */
-        $aField['attributes']['name']   = "__import[submit][{$aField['input_id']}]";
-        $aField['label']                = $aField['label'] 
-            ? $aField['label'] 
+        $aField[ 'attributes' ][ 'name' ]   = "__import[submit][{$aField[ 'input_id' ]}]";
+        $aField[ 'label' ]                = $aField[ 'label' ] 
+            ? $aField[ 'label' ] 
             : $this->oMsg->get( 'import' );
         return parent::getField( $aField );     
     }    
@@ -106,13 +106,13 @@ 
      * @since 3.0.0
      */    
     protected function _getExtraFieldsBeforeLabel( &$aField ) {
-        return "<input " . $this->getAttributes( 
+        return "<input ".$this->getAttributes( 
                 array(
-                    'id' => "{$aField['input_id']}_file",
+                    'id' => "{$aField[ 'input_id' ]}_file",
                     'type' => 'file',
-                    'name' => "__import[{$aField['input_id']}]",
-                ) + $aField['attributes']['file']     
-            ) . " />";
+                    'name' => "__import[{$aField[ 'input_id' ]}]",
+                ) + $aField[ 'attributes' ][ 'file' ]     
+            )." />";
     }    
     
     /**
@@ -121,44 +121,44 @@ 
      */
     protected function _getExtraInputFields( &$aField ) {
 
-        $aHiddenAttributes = array( 'type' => 'hidden', );     
+        $aHiddenAttributes = array( 'type' => 'hidden',);     
         return    
-            "<input " . $this->getAttributes( 
+            "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__import[{$aField['input_id']}][input_id]",
-                    'value' => $aField['input_id'],
+                    'name' => "__import[{$aField[ 'input_id' ]}][input_id]",
+                    'value' => $aField[ 'input_id' ],
                 ) + $aHiddenAttributes
-            ) . "/>"
-            . "<input " . $this->getAttributes( 
+            )."/>"
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__import[{$aField['input_id']}][field_id]",
-                    'value' => $aField['field_id'],
+                    'name' => "__import[{$aField[ 'input_id' ]}][field_id]",
+                    'value' => $aField[ 'field_id' ],
                 ) + $aHiddenAttributes
-            ) . "/>"
-            . "<input " . $this->getAttributes( 
+            )."/>"
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__import[{$aField['input_id']}][section_id]",
-                    'value' => isset( $aField['section_id'] ) && $aField['section_id'] != '_default' ? $aField['section_id'] : '',
+                    'name' => "__import[{$aField[ 'input_id' ]}][section_id]",
+                    'value' => isset( $aField[ 'section_id' ] ) && $aField[ 'section_id' ] != '_default' ? $aField[ 'section_id' ] : '',
                 ) + $aHiddenAttributes
-            ) . "/>"     
-            . "<input " . $this->getAttributes( 
+            )."/>"     
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__import[{$aField['input_id']}][is_merge]",
-                    'value' => $aField['is_merge'],
+                    'name' => "__import[{$aField[ 'input_id' ]}][is_merge]",
+                    'value' => $aField[ 'is_merge' ],
                 ) + $aHiddenAttributes
-            ) . "/>"    
-            . "<input " . $this->getAttributes( 
+            )."/>"    
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__import[{$aField['input_id']}][option_key]",
-                    'value' => $aField['option_key'],
+                    'name' => "__import[{$aField[ 'input_id' ]}][option_key]",
+                    'value' => $aField[ 'option_key' ],
                 ) + $aHiddenAttributes
-            ) . "/>"
-            . "<input " . $this->getAttributes( 
+            )."/>"
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__import[{$aField['input_id']}][format]",
-                    'value' => $aField['format'],
+                    'name' => "__import[{$aField[ 'input_id' ]}][format]",
+                    'value' => $aField[ 'format' ],
                 ) + $aHiddenAttributes
-            ) . "/>"
+            )."/>"
             ;
     }
         

Braces +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      * @since       2.1.5
      * @since       3.3.1       Changed from `_replyToGetStyles()`.
      */        
-    protected function getStyles() { 
+    protected function getStyles() {
         return <<<CSSRULES
 /* Text Field Type */
 .admin-page-framework-field.admin-page-framework-field-text > .admin-page-framework-input-label-container {

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

Spacing +35 added lines, -35 removed lines

@@ -20,7 +20,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'export', );
+    public $aFieldTypeSlugs = array( 'export',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -28,9 +28,9 @@ 
      * @remark $_aDefaultKeys holds shared default key-values defined in the base class.
      */
     protected $aDefaultKeys = array(
-        'data'          => null,        // ( array|string|object ) This is for the export field type. Do not set a value here.     
-        'format'        => 'json',      // ( string ) for the export field type. Do not set a default value here. Currently array, json, and text are supported.
-        'file_name'     => null,        // ( string ) for the export field type. Do not set a default value here.    
+        'data'          => null, // ( array|string|object ) This is for the export field type. Do not set a value here.     
+        'format'        => 'json', // ( string ) for the export field type. Do not set a default value here. Currently array, json, and text are supported.
+        'file_name'     => null, // ( string ) for the export field type. Do not set a default value here.    
         'attributes'    => array(
             'class' => 'button button-primary',
         ),    
@@ -71,14 +71,14 @@ 
     protected function getField( $aField ) {
             
         /* Set the transient data to export - If the value is not an array and the export data is set. */
-        if ( isset( $aField['data'] ) ) {
-            $this->setTransient( md5( "{$aField['class_name']}_{$aField['input_id']}" ), $aField['data'], 60*2 ); // 2 minutes.
+        if ( isset( $aField[ 'data' ] ) ) {
+            $this->setTransient( md5( "{$aField[ 'class_name' ]}_{$aField[ 'input_id' ]}" ), $aField[ 'data' ], 60 * 2 ); // 2 minutes.
         } 
         
         /* Set some required values */
-        $aField['attributes']['name']   = "__export[submit][{$aField['input_id']}]";
-        $aField['file_name']            = $aField['file_name'] ? $aField['file_name'] : $this->_generateExportFileName( $aField['option_key'] ? $aField['option_key'] : $aField['class_name'], $aField['format'] );
-        $aField['label']                = $aField['label'] ? $aField['label'] : $this->oMsg->get( 'export' );
+        $aField[ 'attributes' ][ 'name' ]   = "__export[submit][{$aField[ 'input_id' ]}]";
+        $aField[ 'file_name' ]            = $aField[ 'file_name' ] ? $aField[ 'file_name' ] : $this->_generateExportFileName( $aField[ 'option_key' ] ? $aField[ 'option_key' ] : $aField[ 'class_name' ], $aField[ 'format' ] );
+        $aField[ 'label' ]                = $aField[ 'label' ] ? $aField[ 'label' ] : $this->oMsg->get( 'export' );
         
         return parent::getField( $aField );
         
@@ -93,42 +93,42 @@ 
 
         $_aAttributes = array( 'type' => 'hidden' );
         return
-            "<input " . $this->getAttributes( 
+            "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__export[{$aField['input_id']}][input_id]",
-                    'value' => $aField['input_id'],
+                    'name' => "__export[{$aField[ 'input_id' ]}][input_id]",
+                    'value' => $aField[ 'input_id' ],
                 ) + $_aAttributes
-            ) . "/>"
-            . "<input " . $this->getAttributes( 
+            )."/>"
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__export[{$aField['input_id']}][field_id]",
-                    'value' => $aField['field_id'],
+                    'name' => "__export[{$aField[ 'input_id' ]}][field_id]",
+                    'value' => $aField[ 'field_id' ],
                 ) + $_aAttributes
-            ) . "/>"
-            . "<input " . $this->getAttributes( 
+            )."/>"
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__export[{$aField['input_id']}][section_id]",
-                    'value' => isset( $aField['section_id'] ) && $aField['section_id'] != '_default' ? $aField['section_id'] : '',
+                    'name' => "__export[{$aField[ 'input_id' ]}][section_id]",
+                    'value' => isset( $aField[ 'section_id' ] ) && $aField[ 'section_id' ] != '_default' ? $aField[ 'section_id' ] : '',
                 ) + $_aAttributes
-            ) . "/>"
-            . "<input " . $this->getAttributes( 
+            )."/>"
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__export[{$aField['input_id']}][file_name]",
-                    'value' => $aField['file_name'],
+                    'name' => "__export[{$aField[ 'input_id' ]}][file_name]",
+                    'value' => $aField[ 'file_name' ],
                 ) + $_aAttributes
-            ) . "/>"
-            . "<input " . $this->getAttributes( 
+            )."/>"
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__export[{$aField['input_id']}][format]",
-                    'value' => $aField['format'],
+                    'name' => "__export[{$aField[ 'input_id' ]}][format]",
+                    'value' => $aField[ 'format' ],
                 ) + $_aAttributes
-            ) . "/>"
-            . "<input " . $this->getAttributes( 
+            )."/>"
+            . "<input ".$this->getAttributes( 
                 array(
-                    'name' => "__export[{$aField['input_id']}][transient]",
-                    'value' => isset( $aField['data'] ),
+                    'name' => "__export[{$aField[ 'input_id' ]}][transient]",
+                    'value' => isset( $aField[ 'data' ] ),
                 ) + $_aAttributes
-            ) . "/>"
+            )."/>"
             ;
     }
             
@@ -141,7 +141,7 @@ 
          * @since 2.0.0
          * @since 2.1.5 Moved from the AdminPageFramework_FormField class.
          */ 
-        private function _generateExportFileName( $sOptionKey, $sExportFormat='json' ) {
+        private function _generateExportFileName( $sOptionKey, $sExportFormat = 'json' ) {
                 
             switch ( trim( strtolower( $sExportFormat ) ) ) {
                 case 'text': // for plain text.
@@ -156,7 +156,7 @@ 
                     break;
             }     
                 
-            return $sOptionKey . '_' . date("Ymd") . '.' . $sExt;
+            return $sOptionKey.'_'.date( "Ymd" ).'.'.$sExt;
             
         }
 

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

Braces +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
  * @since       DEVVER
  * @internal
  */
-class AdminPageFramework_Form_View___ToolTip extends AdminPageFramework_Form_View___Section_Base {            
+class AdminPageFramework_Form_View___ToolTip extends AdminPageFramework_Form_View___Section_Base {
   
     public $aArguments      = array(
         'attributes'    => array(), // attributes
@@ -99,7 +99,7 @@ 
          * @since       DEVVER
          * @return      string
          */
-        private function _getDescriptions() {         
+        private function _getDescriptions() {
 
             if ( isset( $this->aArguments[ 'content' ] ) ) {
                 return  "<span class='admin-page-framework-form-tool-tip-description'>"

Doc Comments +1 added lines, -2 removed lines

@@ -42,8 +42,6 @@ 
      * Sets up properties.
      * @since       3.6.0
      * @since       3.7.0           Changed the parameter structure.
-     * @param       array|string    $asArguments        The content output or the tooltip argument array.
-     * @param       string          $sTitleElementID    Not used at the moment.
      */
     public function __construct( /* $aArguments, $sTitleElementID */ ) {
 
@@ -170,6 +168,7 @@ 
 
     /**
      * Generates HTML attributes of the specified element.
+     * @param string $sElementKey
      * @return      string
      * @since       3.8.5
      */

Spacing +14 added lines, -14 removed lines

@@ -21,7 +21,7 @@ 
     /**
      * Stores the default argument values.
      */
-    public $aArguments      = array(
+    public $aArguments = array(
         'attributes'    => array(
             'container'   => array(),   
             'title'       => array(),
@@ -29,11 +29,11 @@ 
             'description' => array(),
             'icon'        => array(),
         ), 
-        'icon'          => null,    // the icon output to override
-        'dash-icon'     => 'dashicons-editor-help',   // the dash-icon class selector for the icon
-        'icon_alt_text' => '[?]',   // [3.8.5+] alternative text when icon is not available. For WP v3.7.x or below.
+        'icon'          => null, // the icon output to override
+        'dash-icon'     => 'dashicons-editor-help', // the dash-icon class selector for the icon
+        'icon_alt_text' => '[?]', // [3.8.5+] alternative text when icon is not available. For WP v3.7.x or below.
         'title'         => null,  
-        'content'       => null,    // will be assigned in the constructor
+        'content'       => null, // will be assigned in the constructor
     );
     
     public $sTitleElementID;
@@ -90,7 +90,7 @@ 
                 if ( is_string( $asContent ) ) {
                     return true;
                 }
-                if ( is_array( $asContent ) && ! $this->isAssociative( $asContent ) ) {
+                if ( is_array( $asContent ) && !$this->isAssociative( $asContent ) ) {
                     return true;
                 }
                 return false;
@@ -104,12 +104,12 @@ 
      * @return      string      The output.
      */
     public function get() {
-        if ( ! $this->aArguments[ 'content' ] ) {
+        if ( !$this->aArguments[ 'content' ] ) {
             return '';
         }
-        return "<a " . $this->_getElementAttributes( 'container', 'admin-page-framework-form-tooltip' ) . ">"
+        return "<a ".$this->_getElementAttributes( 'container', 'admin-page-framework-form-tooltip' ).">"
                 . $this->_getTipLinkIcon()
-                . "<span " . $this->_getElementAttributes( 'content', 'admin-page-framework-form-tooltip-content' ) . ">"
+                . "<span ".$this->_getElementAttributes( 'content', 'admin-page-framework-form-tooltip-content' ).">"
                     . $this->_getTipTitle()
                     . $this->_getDescriptions()
                 . "</span>"
@@ -127,7 +127,7 @@ 
                 return $this->aArguments[ 'icon' ];
             }
             if ( version_compare( $GLOBALS[ 'wp_version' ], '3.8', '>=' ) ) {
-                return "<span " . $this->_getElementAttributes( 
+                return "<span ".$this->_getElementAttributes( 
                         'icon', 
                         array(
                             'dashicons', 
@@ -145,7 +145,7 @@ 
          */
         private function _getTipTitle() {
             if ( isset( $this->aArguments[ 'title' ] ) ) {
-                return "<span " . $this->_getElementAttributes( 'title', 'admin-page-framework-form-tooltip-title' ) . ">"
+                return "<span ".$this->_getElementAttributes( 'title', 'admin-page-framework-form-tooltip-title' ).">"
                     . $this->aArguments[ 'title' ]
                     . "</span>";
             }
@@ -157,9 +157,9 @@ 
          */
         private function _getDescriptions() {         
             if ( isset( $this->aArguments[ 'content' ] ) ) {
-                return "<span " . $this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ) . ">"
+                return "<span ".$this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ).">"
                         . implode( 
-                            "</span><span " . $this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ) . ">", 
+                            "</span><span ".$this->_getElementAttributes( 'description', 'admin-page-framework-form-tooltip-description' ).">", 
                             $this->getAsArray( $this->aArguments[ 'content' ] )
                         )
                     . "</span>"
@@ -178,7 +178,7 @@ 
         $_aContainerAttributes = $this->getElementAsArray(
             $this->aArguments,
             array( 'attributes', $sElementKey )
-        ) + array( 'class' => '' ) ;
+        ) + array( 'class' => '' );
         $_aContainerAttributes[ 'class' ] = $this->getClassAttribute( 
             $_aContainerAttributes[ 'class' ], 
             $asClassSelectors 

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

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