michaeluno / admin-page-framework

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

Spacing +11 added lines, -11 removed lines

@@ -63,7 +63,7 @@ 
     protected function setUp() {
         
         // If the WordPress version is greater than or equal to 3.5, then load the new WordPress color picker.
-        if ( version_compare( $GLOBALS['wp_version'], '3.5', '>=' ) ) {
+        if ( version_compare( $GLOBALS[ 'wp_version' ], '3.5', '>=' ) ) {
             //Both the necessary css and javascript have been registered already by WordPress, so all we have to do is load them with their handle.
             wp_enqueue_style( 'wp-color-picker' );
             wp_enqueue_script( 'wp-color-picker' );
@@ -235,9 +235,9 @@ 
     protected function getField( $aField ) {
 
         // If the value is not set, apply the default value, 'transparent'.
-        $aField['value'] = is_null( $aField['value'] ) 
+        $aField[ 'value' ] = is_null( $aField[ 'value' ] ) 
             ? 'transparent' 
-            : $aField['value'];    
+            : $aField[ 'value' ];    
             
         $aField[ 'attributes' ] = $this->_getInputAttributes( $aField );
         
@@ -246,20 +246,20 @@ 
             . "<div class='admin-page-framework-input-label-container'>"
                 . "<label for='{$aField[ 'input_id' ]}'>"
                     . $aField[ 'before_input' ]
-                    . ( $aField[ 'label' ] && ! $aField[ 'repeatable' ]
-                        ? "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
-                                . $aField['label'] 
+                    . ( $aField[ 'label' ] && !$aField[ 'repeatable' ]
+                        ? "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
+                                . $aField[ 'label' ] 
                             . "</span>"
                         : "" 
                     )
-                    . "<input " . $this->getAttributes( $aField[ 'attributes' ] ) . " />" 
+                    . "<input ".$this->getAttributes( $aField[ 'attributes' ] )." />" 
                     . $aField[ 'after_input' ]
                     . "<div class='repeatable-field-buttons'></div>" // the repeatable field buttons will be replaced with this element.
                 . "</label>"
                 . "<div class='colorpicker' id='color_{$aField[ 'input_id' ]}'></div>" // this div element with this class selector becomes a farbtastic color picker. ( below 3.4.x ) // rel='{$aField['input_id']}'
                 . $this->_getColorPickerEnablerScript( "{$aField[ 'input_id' ]}" )
             . "</div>"
-            . $aField['after_label'];
+            . $aField[ 'after_label' ];
         
     }
         /**
@@ -271,13 +271,13 @@ 
         private function _getInputAttributes( array $aField ) {
                                
             return array(
-                'color'        => $aField['value'],    
-                'value'        => $aField['value'],
+                'color'        => $aField[ 'value' ],    
+                'value'        => $aField[ 'value' ],
                 'data-default' => isset( $aField[ 'default' ] )
                     ? $aField[ 'default' ]
                     : 'transparent', // used by the repeatable script
                 'type'         => 'text', // it must be text
-                'class'        => trim( 'input_color ' . $aField['attributes']['class'] ),
+                'class'        => trim( 'input_color '.$aField[ 'attributes' ][ 'class' ] ),
             ) + $aField[ 'attributes' ];
             
         }    

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

Spacing +23 added lines, -23 removed lines

@@ -554,8 +554,8 @@ 
     protected function getField( $aField ) {
 
         $_aOutput = array();
-        foreach( ( array ) $aField[ 'label' ] as $_sKey => $_sLabel ) {
-            $_aOutput[] = $this->_getFieldOutputByLabel( 
+        foreach ( ( array ) $aField[ 'label' ] as $_sKey => $_sLabel ) {
+            $_aOutput[ ] = $this->_getFieldOutputByLabel( 
                 $_sKey, 
                 $_sLabel, 
                 $aField
@@ -563,7 +563,7 @@ 
         }
         
         // the repeatable field buttons will be replaced with this element.
-        $_aOutput[] = "<div class='repeatable-field-buttons'></div>";
+        $_aOutput[ ] = "<div class='repeatable-field-buttons'></div>";
         return implode( '', $_aOutput );
         
     }
@@ -584,27 +584,27 @@ 
             $aField[ 'rich' ]       = $this->getElementByLabel( $aField[ 'rich' ], $sKey, $aField[ 'label' ] );
             $aField[ 'attributes' ] = $_bIsArray
                 ? array(
-                        'name'  => $aField[ 'attributes' ][ 'name' ] . "[{$sKey}]",
-                        'id'    => $aField[ 'attributes' ][ 'id' ] . "_{$sKey}",
+                        'name'  => $aField[ 'attributes' ][ 'name' ]."[{$sKey}]",
+                        'id'    => $aField[ 'attributes' ][ 'id' ]."_{$sKey}",
                         'value' => $aField[ 'value' ],
                     ) 
                     + $aField[ 'attributes' ]
                 : $aField[ 'attributes' ];        
-            $_aOutput           = array(
-                $this->getElementByLabel( $aField['before_label'], $sKey, $aField[ 'label' ] ),
+            $_aOutput = array(
+                $this->getElementByLabel( $aField[ 'before_label' ], $sKey, $aField[ 'label' ] ),
                 "<div class='admin-page-framework-input-label-container {$_sClassSelector}'>",
-                    "<label for='" . $aField[ 'attributes' ][ 'id' ] . "'>",
-                        $this->getElementByLabel( $aField['before_input'], $sKey, $aField[ 'label' ] ),
+                    "<label for='".$aField[ 'attributes' ][ 'id' ]."'>",
+                        $this->getElementByLabel( $aField[ 'before_input' ], $sKey, $aField[ 'label' ] ),
                         $_sLabel 
-                            ? "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
+                            ? "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
                                     . $_sLabel
                                 . "</span>"
                             : '',
                         $this->_getEditor( $aField ),
-                        $this->getElementByLabel( $aField['after_input'], $sKey, $aField[ 'label' ] ),
+                        $this->getElementByLabel( $aField[ 'after_input' ], $sKey, $aField[ 'label' ] ),
                     "</label>",
                 "</div>",
-                $this->getElementByLabel( $aField['after_label'], $sKey, $aField[ 'label' ] ),
+                $this->getElementByLabel( $aField[ 'after_label' ], $sKey, $aField[ 'label' ] ),
             );
             return implode( '', $_aOutput );
   
@@ -619,31 +619,31 @@ 
          */
         private function _getEditor( $aField ) {
                         
-            unset( $aField['attributes']['value'] );
+            unset( $aField[ 'attributes' ][ 'value' ] );
             
             // For no TinyMCE
-            if ( empty( $aField['rich'] ) || ! $this->isTinyMCESupported() ) {
-                return "<textarea " . $this->getAttributes( $aField['attributes'] ) . " >" // this method is defined in the base class
-                            . esc_textarea( $aField['value'] )
+            if ( empty( $aField[ 'rich' ] ) || !$this->isTinyMCESupported() ) {
+                return "<textarea ".$this->getAttributes( $aField[ 'attributes' ] )." >" // this method is defined in the base class
+                            . esc_textarea( $aField[ 'value' ] )
                         . "</textarea>";
             }
 
             // Rich editor
             ob_start();
             wp_editor( 
-                $aField['value'],
-                $aField['attributes']['id'],  
+                $aField[ 'value' ],
+                $aField[ 'attributes' ][ 'id' ],  
                 $this->uniteArrays( 
-                    ( array ) $aField['rich'],
+                    ( array ) $aField[ 'rich' ],
                     array(
                         'wpautop'           => true, // use wpautop?
                         'media_buttons'     => true, // show insert/upload button(s)
-                        'textarea_name'     => $aField['attributes']['name'],
-                        'textarea_rows'     => $aField['attributes']['rows'],
+                        'textarea_name'     => $aField[ 'attributes' ][ 'name' ],
+                        'textarea_rows'     => $aField[ 'attributes' ][ 'rows' ],
                         'tabindex'          => '',
                         'tabfocus_elements' => ':prev,:next', // the previous and next element ID to move the focus to when pressing the Tab key in TinyMCE
                         'editor_css'        => '', // intended for extra styles for both visual and Text editors buttons, needs to include the <style> tags, can use "scoped".
-                        'editor_class'      => $aField['attributes']['class'], // add extra class(es) to the editor textarea
+                        'editor_class'      => $aField[ 'attributes' ][ 'class' ], // add extra class(es) to the editor textarea
                         'teeny'             => false, // output the minimal editor config used in Press This
                         'dfw'               => false, // replace the default fullscreen with DFW (needs specific DOM elements and css)
                         'tinymce'           => true, // load TinyMCE, can be used to pass settings directly to TinyMCE using an array()
@@ -655,7 +655,7 @@ 
             ob_end_clean();
             
             return $_sContent
-                . $this->_getScriptForRichEditor( $aField['attributes']['id'] );
+                . $this->_getScriptForRichEditor( $aField[ 'attributes' ][ 'id' ] );
             
         }
     

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

Spacing +8 added lines, -8 removed lines

@@ -37,21 +37,21 @@
      */
     public function _replyToGetField( $aField ) {
         return 
-            $aField['before_label']
+            $aField[ 'before_label' ]
             . "<div class='admin-page-framework-input-label-container'>"
-                . "<label for='{$aField['input_id']}'>"
-                    . $aField['before_input']
-                    . ( $aField['label'] && ! $aField['repeatable']
-                        ? "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
+                . "<label for='{$aField[ 'input_id' ]}'>"
+                    . $aField[ 'before_input' ]
+                    . ( $aField[ 'label' ] && !$aField[ 'repeatable' ]
+                        ? "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
                                 . $aField[ 'label' ]
                             . "</span>"
                         : "" 
                     )
-                    . $aField['value']
-                    . $aField['after_input']
+                    . $aField[ 'value' ]
+                    . $aField[ 'after_input' ]
                 . "</label>"
             . "</div>"
-            . $aField['after_label']
+            . $aField[ 'after_label' ]
         ;     
     }
 

development/factory/admin_page/_view/AdminPageFramework_View__Resource.php

Spacing +13 added lines, -13 removed lines

@@ -68,9 +68,9 @@ 
                  */
                 public function _appendInternalAssets( $sInternal, &$aContainer ) {
                     $_aInternals = array_unique( $aContainer );
-                    $sInternal   = PHP_EOL . $sInternal;
-                    foreach( $_aInternals as $_iIndex => $_sInternal ) {
-                        $sInternal .= $_sInternal . PHP_EOL;
+                    $sInternal   = PHP_EOL.$sInternal;
+                    foreach ( $_aInternals as $_iIndex => $_sInternal ) {
+                        $sInternal .= $_sInternal.PHP_EOL;
                         unset( $_aInternals[ $_iIndex ] );
                     }
                     $aContainer = $_aInternals; // update the container array.
@@ -84,23 +84,23 @@ 
         private function _parseAssets( $oFactory ) {
             
             // page
-            $_aPageStyles      = $this->getElementAsArray(
+            $_aPageStyles = $this->getElementAsArray(
                 $oFactory->oProp->aPages,
                 array( $this->sCurrentPageSlug, 'style' )
             );               
             $this->_enqueuePageAssets( $_aPageStyles, 'style' );
             
-            $_aPageScripts     = $this->getElementAsArray(
+            $_aPageScripts = $this->getElementAsArray(
                 $oFactory->oProp->aPages,
                 array( $this->sCurrentPageSlug, 'script' )
             );
             $this->_enqueuePageAssets( $_aPageScripts, 'script' );
             
             // In-page tabs 
-            if ( ! $this->sCurrentTabSlug ) {
+            if ( !$this->sCurrentTabSlug ) {
                 return;
             }        
-            $_aInPageTabStyles  = $this->getElementAsArray(
+            $_aInPageTabStyles = $this->getElementAsArray(
                 $oFactory->oProp->aInPageTabs,
                 array( $this->sCurrentPageSlug, $this->sCurrentTabSlug, 'style' )
             );              
@@ -117,10 +117,10 @@ 
              * @since       3.6.3
              * @return      void
              */    
-            private function _enqueuePageAssets( array $aPageAssets, $sType='style' ) {
-                $_sMathodName = "_enqueueAsset_" . $sType;
-                foreach( $aPageAssets as $_asPageAsset ) {
-                    $this->{$_sMathodName}( $_asPageAsset);
+            private function _enqueuePageAssets( array $aPageAssets, $sType = 'style' ) {
+                $_sMathodName = "_enqueueAsset_".$sType;
+                foreach ( $aPageAssets as $_asPageAsset ) {
+                    $this->{$_sMathodName}( $_asPageAsset );
                 }
             }
 
@@ -145,7 +145,7 @@ 
                     }
                     
                     // Insert the CSS rule in the head tag.
-                    $this->aCSSRules[] = $_sSRC;
+                    $this->aCSSRules[ ] = $_sSRC;
                     
                 }                    
      
@@ -170,7 +170,7 @@ 
                     }                
                     
                     // Insert the scripts in the head tag.                
-                    $this->aScripts[] = $_sSRC;
+                    $this->aScripts[ ] = $_sSRC;
                     
                 }        
                 

development/factory/_common/utility/base_utility/AdminPageFramework_Utility_Path.php

Spacing +7 added lines, -7 removed lines

@@ -29,8 +29,8 @@ 
     static public function getRelativePath( $from, $to ) {
         
         // some compatibility fixes for Windows paths
-        $from = is_dir( $from ) ? rtrim( $from, '\/') . '/' : $from;
-        $to   = is_dir( $to )   ? rtrim( $to, '\/') . '/'   : $to;
+        $from = is_dir( $from ) ? rtrim( $from, '\/' ).'/' : $from;
+        $to   = is_dir( $to ) ? rtrim( $to, '\/' ).'/' : $to;
         $from = str_replace( '\\', '/', $from );
         $to   = str_replace( '\\', '/', $to );
 
@@ -38,21 +38,21 @@ 
         $to       = explode( '/', $to );
         $relPath  = $to;
 
-        foreach( $from as $depth => $dir ) {
+        foreach ( $from as $depth => $dir ) {
             // find first non-matching dir
-            if( $dir === $to[ $depth ] ) {
+            if ( $dir === $to[ $depth ] ) {
                 // ignore this directory
                 array_shift( $relPath );
             } else {
                 // get number of remaining dirs to $from
                 $remaining = count( $from ) - $depth;
-                if( $remaining > 1 ) {
+                if ( $remaining > 1 ) {
                     // add traversals up to first matching dir
                     $padLength = ( count( $relPath ) + $remaining - 1 ) * -1;
                     $relPath = array_pad( $relPath, $padLength, '..' );
                     break;
                 } else {
-                    $relPath[ 0 ] = './' . $relPath[ 0 ];
+                    $relPath[ 0 ] = './'.$relPath[ 0 ];
                 }
             }
         }
@@ -81,7 +81,7 @@ 
                 6, // the second parameter: limit
             )
         );
-        foreach( $_aBackTrace as $_aDebugInfo )  {     
+        foreach ( $_aBackTrace as $_aDebugInfo ) {     
             $_sCallerFilePath = $_aDebugInfo[ 'file' ];
             if ( in_array( $_sCallerFilePath, $_aRedirectedFilePaths ) ) { 
                 continue; 

Braces +2 added lines, -2 removed lines

@@ -81,9 +81,9 @@
                 6, // the second parameter: limit
             )
         );
-        foreach( $_aBackTrace as $_aDebugInfo )  {     
+        foreach( $_aBackTrace as $_aDebugInfo ) {
             $_sCallerFilePath = $_aDebugInfo[ 'file' ];
-            if ( in_array( $_sCallerFilePath, $_aRedirectedFilePaths ) ) { 
+            if ( in_array( $_sCallerFilePath, $_aRedirectedFilePaths ) ) {
                 continue; 
             }
             break; // catch the first found item.

development/factory/_common/_abstract/AdminPageFramework_Factory_Model.php

Doc Comments +1 added lines, -2 removed lines

@@ -203,6 +203,7 @@ 
          * Constructs a WordPress filter hook name.
          * @internal
          * @since       3.8.4
+         * @param string $sPrefix
          * @return      string
          */
         private function _getHookNameByFieldsetAndPrefix( $sPrefix, $aFieldset ) {
@@ -339,8 +340,6 @@ 
      * @since       3.7.0      Changed back the visibility scope to protected as there is the `getFieldErrors()` public method.
      * @access      protected
      * @internal
-     * @param       string      $sID        deprecated
-     * @param       boolean     $bDelete    whether or not the transient should be deleted after retrieving it. 
      * @return      array
      * @deprecated  3.7.0      Use `getFieldErrors()` instead. Kept for backward compatibility.
      */

Spacing +13 added lines, -13 removed lines

@@ -32,7 +32,7 @@ 
         
         add_filter(
             // 'field_types_admin_page_framework',
-            'field_types_' . $oProp->sClassName,
+            'field_types_'.$oProp->sClassName,
             array( $this, '_replyToFilterFieldTypeDefinitions' )
         );
         
@@ -65,7 +65,7 @@ 
             'title'      => null,
             'help_aside' => null,
         );
-        if ( ! $aFieldset[ 'help' ] ) {
+        if ( !$aFieldset[ 'help' ] ) {
             return;
         }
         $this->oHelpPane->_addHelpTextForFormFields( 
@@ -85,9 +85,9 @@ 
     public function _replyToFilterFieldTypeDefinitions( $aFieldTypeDefinitions ) {
         
         // Not triggering `__call()` as the filter is fired manually in the form class.
-        if ( method_exists( $this, 'field_types_' . $this->oProp->sClassName ) ) {
+        if ( method_exists( $this, 'field_types_'.$this->oProp->sClassName ) ) {
             return call_user_func_array(
-                array( $this, 'field_types_' . $this->oProp->sClassName ),
+                array( $this, 'field_types_'.$this->oProp->sClassName ),
                 array( $aFieldTypeDefinitions )
             );
         }
@@ -108,7 +108,7 @@ 
     public function _replyToModifySectionsets( $aSectionsets ) {    
         
         return $this->oUtil->addAndApplyFilter( 
-            $this,  // caller factory object
+            $this, // caller factory object
             "sections_{$this->oProp->sClassName}", 
             $aSectionsets
         );
@@ -128,7 +128,7 @@ 
     public function _replyToModifyFieldsets( $aFieldsets, $aSectionsets ) {
 
         // Apply filters to added fieldsets
-        foreach( $aFieldsets as $_sSectionPath => $_aFields ) {
+        foreach ( $aFieldsets as $_sSectionPath => $_aFields ) {
             $_aSectionPath  = explode( '|', $_sSectionPath );
             $_sFilterSuffix = implode( '_', $_aSectionPath );
             $aFieldsets[ $_sSectionPath ] = $this->oUtil->addAndApplyFilter(
@@ -137,7 +137,7 @@ 
                 $_aFields
             ); 
         }
-        $aFieldsets =  $this->oUtil->addAndApplyFilter( 
+        $aFieldsets = $this->oUtil->addAndApplyFilter( 
             $this,
             "fields_{$this->oProp->sClassName}",
             $aFieldsets
@@ -207,14 +207,14 @@ 
          */
         private function _getHookNameByFieldsetAndPrefix( $sPrefix, $aFieldset ) {
             
-            $_sFieldPart    = '_' . implode( '_', $aFieldset[ '_field_path_array' ] );
+            $_sFieldPart    = '_'.implode( '_', $aFieldset[ '_field_path_array' ] );
             $_sSectionPart  = implode( '_', $aFieldset[ '_section_path_array' ] );
             $_sSectionPart  = $this->oUtil->getAOrB(
                 '_default' === $_sSectionPart,
                 '',
-                '_' . $_sSectionPart
+                '_'.$_sSectionPart
             );
-            return $sPrefix . $this->oProp->sClassName . $_sSectionPart . $_sFieldPart;
+            return $sPrefix.$this->oProp->sClassName.$_sSectionPart.$_sFieldPart;
             
         }
                
@@ -284,9 +284,9 @@ 
     public function _replyToGetSavedFormData() {
         
         // Must update the property with the filtered value.
-        $this->oProp->aOptions =  $this->oUtil->addAndApplyFilter(
+        $this->oProp->aOptions = $this->oUtil->addAndApplyFilter(
             $this, // the caller factory object
-            'options_' . $this->oProp->sClassName,
+            'options_'.$this->oProp->sClassName,
             $this->oProp->aOptions      // subject value to be filtered
         );
         return $this->oProp->aOptions;
@@ -366,7 +366,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }    
 

Braces +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
      * but it requires to call the setUp() method in the overridden method so it's not that useful.
      * @internal
      */
-    protected function _setUp() { 
+    protected function _setUp() {
         $this->setUp();
     }    
     
@@ -105,7 +105,7 @@ 
      * @since       3.7.0
      * @return      array       The modified sectionsets definition array.
      */    
-    public function _replyToModifySectionsets( $aSectionsets ) {    
+    public function _replyToModifySectionsets( $aSectionsets ) {
         
         return $this->oUtil->addAndApplyFilter( 
             $this,  // caller factory object
@@ -366,7 +366,7 @@ 
          * An alias of `_setLastInputs()`.
          * @deprecated      3.7.0
          */
-        public function _setLastInput( $aLastInputs )  {
+        public function _setLastInput( $aLastInputs ) {
             return $this->setLastInputs( $aLastInputs );
         }    
 

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

Spacing +8 added lines, -8 removed lines

@@ -33,7 +33,7 @@ 
 
         $_aParameters = func_get_args() + array( 
             $this->aFieldset, 
-            $this->aSavedData,    // passed by reference. @todo: examine why it needs to be passed by reference.
+            $this->aSavedData, // passed by reference. @todo: examine why it needs to be passed by reference.
             $this->aFieldErrors, 
             $this->aFieldTypeDefinitions, 
             $this->aCallbacks, // field output element callables.        
@@ -61,14 +61,14 @@ 
         
         $aFieldset = $this->aFieldset;
 
-        if ( ! $this->isNormalPlacement( $aFieldset ) ) {
+        if ( !$this->isNormalPlacement( $aFieldset ) ) {
             return '';
         }
 
-        $_oFieldrowAttribute   = new AdminPageFramework_Form_View___Attribute_Fieldrow( 
+        $_oFieldrowAttribute = new AdminPageFramework_Form_View___Attribute_Fieldrow( 
             $aFieldset,
             array( 
-                'id'        => 'fieldrow-' . $aFieldset[ 'tag_id' ],
+                'id'        => 'fieldrow-'.$aFieldset[ 'tag_id' ],
                 'valign'    => 'top',
                 'class'     => 'admin-page-framework-fieldrow',
             )                
@@ -77,7 +77,7 @@ 
         return $this->_getFieldByContainer( 
             $aFieldset, 
             array(
-                'open_container'    => "<tr " . $_oFieldrowAttribute->get() . ">",
+                'open_container'    => "<tr ".$_oFieldrowAttribute->get().">",
                 'close_container'   => "</tr>",
                 'open_title'        => "<th>",
                 'close_title'       => "</th>",
@@ -126,13 +126,13 @@ 
                 $this->aCallbacks,
                 $this->oMsg
             );
-            $_aOutput   = array();
+            $_aOutput = array();
             if ( $aFieldset[ 'show_title_column' ] ) {
-                $_aOutput[] = $aOpenCloseTags[ 'open_title' ]
+                $_aOutput[ ] = $aOpenCloseTags[ 'open_title' ]
                         . $_oFieldTitle->get()
                     . $aOpenCloseTags[ 'close_title' ];
             }
-            $_aOutput[] = $aOpenCloseTags[ 'open_main' ]
+            $_aOutput[ ] = $aOpenCloseTags[ 'open_main' ]
                     // . call_user_func_array( $hfCallback, array( $aFieldset ) )
                     . $this->getFieldsetOutput( $aFieldset )
                 . $aOpenCloseTags[ 'close_main' ];

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

Spacing +16 added lines, -16 removed lines

@@ -77,9 +77,9 @@ 
      * Defines the default key-values of this field type. 
      */
     protected $aDefaultKeys = array(
-        'select_all_button'     => false,        // 3.3.0+   to change the label, set the label here
-        'select_none_button'    => false,        // 3.3.0+   to change the label, set the label here
-        'save_unchecked'        => true,        // (optional, boolean) 3.8.8+   Whether to store the values of unchecked items.
+        'select_all_button'     => false, // 3.3.0+   to change the label, set the label here
+        'select_none_button'    => false, // 3.3.0+   to change the label, set the label here
+        'save_unchecked'        => true, // (optional, boolean) 3.8.8+   Whether to store the values of unchecked items.
     );
         
     /**
@@ -91,7 +91,7 @@ 
      */ 
     protected function getScripts() {
         new AdminPageFramework_Form_View___Script_CheckboxSelector;
-        $_sClassSelectorSelectAll  = $this->_getSelectButtonClassSelectors( 
+        $_sClassSelectorSelectAll = $this->_getSelectButtonClassSelectors( 
             $this->aFieldTypeSlugs, 
             'select_all_button' // data attribute
         );
@@ -118,14 +118,14 @@ 
          * @return      string
          * @internal
          */
-        private function _getSelectButtonClassSelectors( array $aFieldTypeSlugs, $sDataAttribute='select_all_button' ) {
+        private function _getSelectButtonClassSelectors( array $aFieldTypeSlugs, $sDataAttribute = 'select_all_button' ) {
             
             $_aClassSelectors = array();
             foreach ( $aFieldTypeSlugs as $_sSlug ) {
-                if ( ! is_scalar( $_sSlug ) ) {
+                if ( !is_scalar( $_sSlug ) ) {
                     continue;
                 }
-                $_aClassSelectors[] = '.admin-page-framework-checkbox-container-' . $_sSlug . "[data-{$sDataAttribute}]";
+                $_aClassSelectors[ ] = '.admin-page-framework-checkbox-container-'.$_sSlug."[data-{$sDataAttribute}]";
             }
             return implode( ',', $_aClassSelectors );
             
@@ -186,8 +186,8 @@ 
 
         $_aOutput       = array();
         $_bIsMultiple   = is_array( $aField[ 'label' ] );
-        foreach( $this->getAsArray( $aField[ 'label' ], true ) as $_sKey => $_sLabel ) {
-            $_aOutput[] = $this->_getEachCheckboxOutput( 
+        foreach ( $this->getAsArray( $aField[ 'label' ], true ) as $_sKey => $_sLabel ) {
+            $_aOutput[ ] = $this->_getEachCheckboxOutput( 
                 $aField, 
                 $_bIsMultiple 
                     ? $_sKey 
@@ -195,7 +195,7 @@ 
                 $_sLabel
             );
         }        
-        return "<div " . $this->getAttributes( $this->_getCheckboxContainerAttributes( $aField ) ) . ">"
+        return "<div ".$this->getAttributes( $this->_getCheckboxContainerAttributes( $aField ) ).">"
                 . "<div class='repeatable-field-buttons'></div>" // the repeatable field buttons will be replaced with this element.
                 . implode( PHP_EOL, $_aOutput )
             . "</div>";
@@ -210,12 +210,12 @@ 
          */
         protected function _getCheckboxContainerAttributes( array $aField ) {
             return array(
-                'class'                     => 'admin-page-framework-checkbox-container-' . $aField[ 'type' ],
+                'class'                     => 'admin-page-framework-checkbox-container-'.$aField[ 'type' ],
                 'data-select_all_button'    => $aField[ 'select_all_button' ] 
-                    ? ( ! is_string( $aField[ 'select_all_button' ] ) ? $this->oMsg->get( 'select_all' ) : $aField[ 'select_all_button' ] )
+                    ? ( !is_string( $aField[ 'select_all_button' ] ) ? $this->oMsg->get( 'select_all' ) : $aField[ 'select_all_button' ] )
                     : null,
                 'data-select_none_button'   => $aField[ 'select_none_button' ] 
-                    ? ( ! is_string( $aField[ 'select_none_button' ] ) ? $this->oMsg->get( 'select_none' ) : $aField[ 'select_none_button' ] )
+                    ? ( !is_string( $aField[ 'select_none_button' ] ) ? $this->oMsg->get( 'select_none' ) : $aField[ 'select_none_button' ] )
                     : null,
             );            
         }
@@ -230,7 +230,7 @@ 
         private function _getEachCheckboxOutput( array $aField, $sKey, $sLabel ) {
 
             $_aInputAttributes = array(
-                'data-key'  => $sKey,   // 3.8.8+ For the `post_type_taxonomy` field type.
+                'data-key'  => $sKey, // 3.8.8+ For the `post_type_taxonomy` field type.
             ) + $aField[ 'attributes' ];
             $_oCheckbox = new AdminPageFramework_Input_checkbox( 
                 $_aInputAttributes,
@@ -241,8 +241,8 @@ 
             $_oCheckbox->setAttributesByKey( $sKey );
             $_oCheckbox->addClass( $this->_sCheckboxClassSelector );
             return $this->getElementByLabel( $aField[ 'before_label' ], $sKey, $aField[ 'label' ] )
-                . "<div " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-container admin-page-framework-checkbox-label' ) . ">" 
-                    . "<label " . $this->getAttributes( 
+                . "<div ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-container admin-page-framework-checkbox-label' ).">" 
+                    . "<label ".$this->getAttributes( 
                         array(
                             'for'   => $_oCheckbox->getAttribute( 'id' ),
                             'class' => $_oCheckbox->getAttribute( 'disabled' )

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

Spacing +9 added lines, -9 removed lines

@@ -63,16 +63,16 @@ 
      *  - use_desc_for_title    (boolean|int) default is 1 - Whether to use the category description as the title attribute. side note: the framework enables this by default.
      * @param       integer     $iCurrentObjectID
      */
-    function start_el( &$sOutput, $oTerm, $iDepth=0, $aArgs=array(), $iCurrentObjectID=0 ) {
+    function start_el( &$sOutput, $oTerm, $iDepth = 0, $aArgs = array(), $iCurrentObjectID = 0 ) {
        
         $aArgs = $aArgs + array(
             '_name_prefix'      => null,
             '_input_id_prefix'  => null,
             '_attributes'       => array(),
             '_selected_items'   => array(),
-            'taxonomy'          => null,    // parsed by the core function to perform the database query.
-            'disabled'          => null,    // not sure what this was for
-            '_save_unchecked'   => true,    // 3.8.8+
+            'taxonomy'          => null, // parsed by the core function to perform the database query.
+            'disabled'          => null, // not sure what this was for
+            '_save_unchecked'   => true, // 3.8.8+
         );
         
         // Local variables
@@ -84,7 +84,7 @@ 
 
         // Post count
         $_sPostCount     = $aArgs[ 'show_post_count' ] 
-            ? " <span class='font-lighter'>(" . $oTerm->count . ")</span>" 
+            ? " <span class='font-lighter'>(".$oTerm->count.")</span>" 
             : '';
         
         // Attributes
@@ -103,7 +103,7 @@ 
           + array(
             'class'     => null,
         );
-        $_aInputAttributes['class'] .= ' apf_checkbox';
+        $_aInputAttributes[ 'class' ] .= ' apf_checkbox';
         
         $_aLiTagAttributes = array(
             'id'        => "list-{$_sID}",
@@ -112,15 +112,15 @@ 
         );
 
         $_sHiddenInputForUnchecked = $aArgs[ '_save_unchecked' ]
-            ? "<input value='0' type='hidden' name='" . $_aInputAttributes[ 'name' ] . "' class='apf_checkbox' />"
+            ? "<input value='0' type='hidden' name='".$_aInputAttributes[ 'name' ]."' class='apf_checkbox' />"
             : '';
             
         // Output - the variable is by reference so the modification takes effect
         $sOutput .= "\n"
-            . "<li " . AdminPageFramework_WPUtility::getAttributes( $_aLiTagAttributes ) . ">" 
+            . "<li ".AdminPageFramework_WPUtility::getAttributes( $_aLiTagAttributes ).">" 
                 . "<label for='{$_sID}' class='taxonomy-checklist-label'>"
                     . $_sHiddenInputForUnchecked    // 3.8.8+
-                    . "<input " . AdminPageFramework_WPUtility::getAttributes( $_aInputAttributes ) . " />"
+                    . "<input ".AdminPageFramework_WPUtility::getAttributes( $_aInputAttributes )." />"
                     . esc_html( apply_filters( 'the_category', $oTerm->name ) ) 
                     . $_sPostCount
                 . "</label>";