michaeluno / admin-page-framework

development/factory/_common/form/AdminPageFramework_Form_Controller.php

Braces +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
      */
     public function removeSection( $sSectionID ) {
         
-        if ( '_default' === $sSectionID ){
+        if ( '_default' === $sSectionID ) {
             return;
         }
         unset(

Doc Comments +2 added lines

@@ -148,6 +148,8 @@
     }
     /**
      * @since       3.7.0
+     * @param string $sKey
+     * @param string $sValue
      * @return      void
      */
     public function addResource( $sKey, $sValue ) {

Spacing +9 added lines, -9 removed lines

@@ -44,7 +44,7 @@ 
      * @param       string      $sType If empty, the method will check if a message exists in all types. Otherwise, it checks the existence of a message of the specified type.
      * @return      boolean     True if a setting notice is set; otherwise, false.
      */
-    public function hasSubmitNotice( $sType='' ) {
+    public function hasSubmitNotice( $sType = '' ) {
         return $this->oSubmitNotice->hasNotice( $sType );
     }
 
@@ -70,7 +70,7 @@ 
      * @param        boolean     $bOverride      (optional) If true, only one message will be shown in the next page load. false: do not override when there is a message of the same id. true: override the previous one.
      * @return       void
      */
-    public function setSubmitNotice( $sMessage, $sType='error', $asAttributes=array(), $bOverride=true ) {
+    public function setSubmitNotice( $sMessage, $sType = 'error', $asAttributes = array(), $bOverride = true ) {
         $this->oSubmitNotice->set(
             $sMessage,
             $sType,
@@ -97,7 +97,7 @@ 
 
         $this->aSectionsets[ $aSectionset[ 'section_id' ] ] = $aSectionset;
         $this->aFieldsets[ $aSectionset[ 'section_id' ] ]   = $this->getElement(
-            $this->aFieldsets,  // subject array
+            $this->aFieldsets, // subject array
             $aSectionset[ 'section_id' ], // key
             array()      // default
         );
@@ -112,7 +112,7 @@ 
      */
     public function removeSection( $sSectionID ) {
 
-        if ( '_default' === $sSectionID ){
+        if ( '_default' === $sSectionID ) {
             return;
         }
         unset(
@@ -151,7 +151,7 @@ 
      * @return      void
      */
     public function addResource( $sKey, $sValue ) {
-        self::$_aResources[ $sKey ][] = $sValue;
+        self::$_aResources[ $sKey ][ ] = $sValue;
     }
 
     /**
@@ -174,7 +174,7 @@ 
     public function addField( $asFieldset ) {
 
         // If it is a target section, update the property and return.
-        if ( ! $this->_isFieldsetDefinition( $asFieldset ) ) {
+        if ( !$this->_isFieldsetDefinition( $asFieldset ) ) {
             $this->_asTargetSectionID = $this->_getTargetSectionID( $asFieldset );
             return $this->_asTargetSectionID;
         }
@@ -183,13 +183,13 @@ 
 
         // Set the target section ID
         $this->_asTargetSectionID = $this->getElement(
-            $_aFieldset,  // subject array
+            $_aFieldset, // subject array
             'section_id', // key
             $this->_asTargetSectionID // default
         );
 
         // Required Keys - 3.8.0+ Now 'type' can be omitted.
-        if ( ! isset( $_aFieldset[ 'field_id' ] ) ) {
+        if ( !isset( $_aFieldset[ 'field_id' ] ) ) {
             return null;
         }
 
@@ -292,7 +292,7 @@ 
      */
     public function removeField( $sFieldID ) {
 
-        foreach( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
+        foreach ( $this->aFieldsets as $_sSectionID => $_aSubSectionsOrFields ) {
 
             if ( array_key_exists( $sFieldID, $_aSubSectionsOrFields ) ) {
                 unset( $this->aFieldsets[ $_sSectionID ][ $sFieldID ] );

development/factory/_common/form/_model/formatter/AdminPageFramework_Form_Model___FormatDynamicElements.php

Spacing +6 added lines, -6 removed lines

@@ -70,7 +70,7 @@ 
                 $aOptions // data source
             );
 
-            foreach( $aOptions as $_sSectionID => $_aSubSectionOrFields ) {
+            foreach ( $aOptions as $_sSectionID => $_aSubSectionOrFields ) {
                 
                 $_aSubSection = $this->_getSubSectionFromOptions(
                     $_sSectionID,
@@ -107,10 +107,10 @@ 
                 
                 $_aSubSection = array();
                 $_iPrevIndex  = null;
-                foreach( $_aSubSectionOrFields as $_isIndexOrFieldID => $_aSubSectionOrFieldOptions ) {
+                foreach ( $_aSubSectionOrFields as $_isIndexOrFieldID => $_aSubSectionOrFieldOptions ) {
                 
                     // If it is not a sub-section array, skip.
-                    if ( ! $this->isNumericInteger( $_isIndexOrFieldID ) ) {
+                    if ( !$this->isNumericInteger( $_isIndexOrFieldID ) ) {
                         continue;
                     }
                     
@@ -124,7 +124,7 @@ 
                     );
        
                     // Update the internal section index key
-                    foreach( $_aSubSection[ $_iIndex ] as &$_aField ) {
+                    foreach ( $_aSubSection[ $_iIndex ] as &$_aField ) {
                         $_aField[ '_section_index' ] = $_iIndex;
                     }
                     unset( $_aField ); // to be safe in PHP
@@ -150,7 +150,7 @@ 
                  */
                 private function _getSubSectionItemsFromOptions( array $_aSubSection, $_sSectionID, $_iIndex, $_iPrevIndex ) {
                     
-                    if ( ! isset( $this->aFieldsets[ $_sSectionID ] ) ) {
+                    if ( !isset( $this->aFieldsets[ $_sSectionID ] ) ) {
                         return array();
                     }
                     
@@ -159,7 +159,7 @@ 
                         : $this->getNonIntegerKeyElements( $this->aFieldsets[ $_sSectionID ] );
                         
                     // if empty, merge with the previous element.
-                    return ! empty( $_aFields )
+                    return !empty( $_aFields )
                         ? $_aFields
                         : $this->getElementAsArray(
                             $_aSubSection,

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

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
             $this->aAttributes + $this->_getAttributes()
         );
                     
-        $_aAttributes[ 'class' ]   = $this->getClassAttribute(
+        $_aAttributes[ 'class' ] = $this->getClassAttribute(
             $this->getElement( $_aAttributes, 'class', array() ),
             $this->getElement( $this->aArguments, array( 'class', $this->sContext ), array() )
         );

development/factory/_common/form/_view/script/AdminPageFramework_Form_View___Script_CollapsibleSection.php

Spacing +5 added lines, -5 removed lines

@@ -43,7 +43,7 @@ 
         
         $_aParams               = func_get_args() + array( null );
         $_oMsg                  = $_aParams[ 0 ];
-        $_sToggleAllButtonHTML  = '"' . self::_getToggleAllButtonHTML( $_oMsg ) . '"';
+        $_sToggleAllButtonHTML  = '"'.self::_getToggleAllButtonHTML( $_oMsg ).'"';
            
         return <<<JAVASCRIPTS
 ( function( $ ) {
@@ -212,19 +212,19 @@ 
             $_sLabelToggleAll           = $oMsg->get( 'toggle_all' );
             $_sLabelToggleAllSections   = $oMsg->get( 'toggle_all_collapsible_sections' );
             $_sDashIconSort             = self::getAOrB(
-                version_compare( $GLOBALS['wp_version'], '3.8', '<' ),  // evaluate
+                version_compare( $GLOBALS[ 'wp_version' ], '3.8', '<' ), // evaluate
                 '', // true
                 'dashicons dashicons-sort' // false
             );
-            $_sText                     = self::getAOrB(
+            $_sText = self::getAOrB(
                 $_sDashIconSort, // evaluate
                 '', // true
                 $_sLabelToggleAll // false
             );
 
             return "<div class='admin-page-framework-collapsible-toggle-all-button-container'>"
-                    . "<span class='admin-page-framework-collapsible-toggle-all-button button " . $_sDashIconSort . "'"
-                        . " title='" . esc_attr( $_sLabelToggleAllSections )
+                    . "<span class='admin-page-framework-collapsible-toggle-all-button button ".$_sDashIconSort."'"
+                        . " title='".esc_attr( $_sLabelToggleAllSections )
                     . "'>"
                         . $_sText
                     . "</span>"

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();
@@ -63,7 +63,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 '';
@@ -79,10 +79,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(
@@ -103,11 +103,11 @@ 
                                 'collapsed',
                                 ''
                             ),
-                            'admin-page-framework-collapsible-type-' . $aCollapsible[ 'type' ]
+                            'admin-page-framework-collapsible-type-'.$aCollapsible[ 'type' ]
                         ),
                     )
                     + $this->getDataAttributeArray( $aCollapsible )
-                ) . ">"
+                ).">"
                         . $_sSectionTitle
                     . "</div>";
             

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

Spacing +2 added lines, -2 removed lines

@@ -56,13 +56,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,

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

Indentation +1 added lines, -1 removed lines

@@ -556,7 +556,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

@@ -164,6 +164,7 @@ 
     }
         /**
          * @since       3.7.0
+         * @param string $_sOutput
          * @return      string
          */
         private function _getSpinnerOutput( $_sOutput ) {
@@ -298,7 +299,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 ) {
@@ -353,6 +354,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 ) {
@@ -499,6 +501,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>"

development/factory/_common/form/input/AdminPageFramework_Input_radio.php

Spacing +10 added lines, -10 removed lines

@@ -36,12 +36,12 @@ 
         
         // Output
         return
-            "<{$this->aOptions['input_container_tag']} " . $this->getAttributes( $this->aOptions['input_container_attributes'] ) . ">"
-                . "<input " . $this->getAttributes( $_aAttributes ) . " />"
-            . "</{$this->aOptions['input_container_tag']}>"
-            . "<{$this->aOptions['label_container_tag']} " . $this->getAttributes( $this->aOptions['label_container_attributes'] ) . ">"
+            "<{$this->aOptions[ 'input_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ).">"
+                . "<input ".$this->getAttributes( $_aAttributes )." />"
+            . "</{$this->aOptions[ 'input_container_tag' ]}>"
+            . "<{$this->aOptions[ 'label_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'label_container_attributes' ] ).">"
                 . $_sLabel
-            . "</{$this->aOptions['label_container_tag']}>"
+            . "</{$this->aOptions[ 'label_container_tag' ]}>"
             ;
         
     }
@@ -58,22 +58,22 @@ 
     public function getAttributesByKey( /* $sKey */ ) {
         
         // Parameters
-        $_aParams       = func_get_args() + array( 0 => '', );
-        $_sKey           = $_aParams[ 0 ];
+        $_aParams = func_get_args() + array( 0 => '',);
+        $_sKey = $_aParams[ 0 ];
         
         // Result
         return $this->getElementAsArray( $this->aAttributes, $_sKey, array() )
             + array(
                 'type'          => 'radio',
-                'checked'       => isset( $this->aAttributes['value'] ) && $this->aAttributes['value'] == $_sKey
+                'checked'       => isset( $this->aAttributes[ 'value' ] ) && $this->aAttributes[ 'value' ] == $_sKey
                     ? 'checked'
                     : null,
                 'value'         => $_sKey,
                 // 'id'            => $this->aField['input_id'] . '_' . $_sKey,
-                'id'            => $this->getAttribute( 'id' ) . '_' . $_sKey,
+                'id'            => $this->getAttribute( 'id' ).'_'.$_sKey,
                 // 'data-default'  => $this->aField['default'],        // refered by the repeater script
                 // 'data-id'       => $this->aField['input_id'],       // refered by the JavaScript scripts such as the revealer script.
-                'data-id'       => $this->getAttribute( 'id' ),       // refered by the JavaScript scripts such as the revealer script.
+                'data-id'       => $this->getAttribute( 'id' ), // refered by the JavaScript scripts such as the revealer script.
             )
             + $this->aAttributes;
             

Doc Comments -4 removed lines

@@ -20,8 +20,6 @@ 
      * Returns the output of the input element.
      *
      * @since       3.4.0
-     * @param       string      $sLabel         The label text.
-     * @param       array       $aAttributes    (optional) The attribute array. If set, it will be merged with the attribute set in the constructor.
      */
     public function get( /* $sLabel, $aAttributes=array() */ ) {
 
@@ -51,8 +49,6 @@ 
      *
      * @return      array       The updated attribute array.
      * @since       3.5.3
-     * @param       string      $sKey       The array element key of the radio button.
-     * It is assumed that there is an array that holds multiple radio buttons and each of them has an array key.
      */
     public function getAttributesByKey( /* $sKey */ ) {
 

development/factory/admin_page/AdminPageFramework_Model_Page.php

Spacing +5 added lines, -5 removed lines

@@ -37,13 +37,13 @@ 
      */
     public function _replyToFinalizeInPageTabs() {
 
-        if ( ! $this->oProp->isPageAdded() ) {
+        if ( !$this->oProp->isPageAdded() ) {
             return;
         }
 
-        foreach( $this->oProp->aPages as $_sPageSlug => $_aPage ) {
+        foreach ( $this->oProp->aPages as $_sPageSlug => $_aPage ) {
             
-            if ( ! isset( $this->oProp->aInPageTabs[ $_sPageSlug ] ) ) {
+            if ( !isset( $this->oProp->aInPageTabs[ $_sPageSlug ] ) ) {
                 continue;
             }
             
@@ -80,8 +80,8 @@ 
          * @since       3.6.0
          */
         private function _getDefaultInPageTab( $sPageSlug, $aInPageTabs ) {
-            foreach( $aInPageTabs as $_aInPageTab ) {
-                if ( ! isset( $_aInPageTab[ 'tab_slug' ] ) ) {
+            foreach ( $aInPageTabs as $_aInPageTab ) {
+                if ( !isset( $_aInPageTab[ 'tab_slug' ] ) ) {
                     continue;
                 }
                 // Regardless of whether it's a hidden tab, it is stored as the default in-page tab.