michaeluno / admin-page-framework

development/factory/term_meta/form/AdminPageFramework_Form_term_meta.php

Spacing +3 added lines, -3 removed lines

@@ -54,7 +54,7 @@ 
             '' // default value
         );    
     
-        if ( ! $this->canUserView( $this->sCapability ) ) {
+        if ( !$this->canUserView( $this->sCapability ) ) {
             return '';
         }    
 
@@ -82,10 +82,10 @@ 
         );        
         
         $_sAddNewTerm     = $bEditTerm ? '' : ' add-new-term';
-        $_sClassSelectors = 'admin-page-framework-form-table-term_meta' . $_sAddNewTerm;
+        $_sClassSelectors = 'admin-page-framework-form-table-term_meta'.$_sAddNewTerm;
         return '<tr class="admin-page-framework-form-table-outer-row-term_meta">'
             . '<td colspan=2>'
-                . '<table class="' . $_sClassSelectors . '">'
+                . '<table class="'.$_sClassSelectors.'">'
                     . '<tbody>'
                         . '<tr>'
                             . '<td>'

Braces +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
      * @since       3.8.0
      * @return      void
      */
-    public function construct() {        
+    public function construct() {
         $this->_addDefaultResources();
     }
         /**

development/factory/term_meta/AdminPageFramework_TermMeta_Model.php

Spacing +7 added lines, -7 removed lines

@@ -56,10 +56,10 @@ 
      * @since       3.8.0
      * @internal
      */
-    protected function _setOptionArray( $iTermID=null, $_deprecated=null ) {
+    protected function _setOptionArray( $iTermID = null, $_deprecated = null ) {
         $this->oForm->aSavedData = $this->oUtil->addAndApplyFilter(
             $this, // the caller factory object
-            'options_' . $this->oProp->sClassName,
+            'options_'.$this->oProp->sClassName,
             $this->_getSavedTermMetas( $iTermID, $this->oForm->aFieldsets )
             // @todo maybe pass the term id because the user will not know whihch form data is
         );        
@@ -91,7 +91,7 @@ 
      */
     public function _replyToValidateOptions( $iTermID ) {
 
-        if ( ! $this->_shouldProceedValidation() ) {
+        if ( !$this->_shouldProceedValidation() ) {
             return;
         }              
 
@@ -99,19 +99,19 @@ 
         $_aSubmittedFormData    = $this->oForm->getSubmittedData( $_POST ); 
         $_aSubmittedFormData    = $this->oUtil->addAndApplyFilters( 
             $this, 
-            'validation_' . $this->oProp->sClassName, 
+            'validation_'.$this->oProp->sClassName, 
             call_user_func_array(       // 1st param
                 array( $this, 'validate' ), // triggers __call()
                 array( $_aSubmittedFormData, $_aSavedFormData, $this )
             ), // 3.5.10+
-            $_aSavedFormData,   // 2nd param
+            $_aSavedFormData, // 2nd param
             $this   // 3rd param
         );
         
         // @todo Update term metas
         $this->oForm->updateMetaDataByType( 
-            $iTermID,  // object id
-            $_aSubmittedFormData,  // user submit form data
+            $iTermID, // object id
+            $_aSubmittedFormData, // user submit form data
             $this->oForm->dropRepeatableElements( $_aSavedFormData ), // Drop repeatable section elements from the saved meta array.
             $this->oForm->sStructureType   // fields type
         );             

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

Braces +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
                     $aField[ '_structure_type' ], 
                     array( 'widget', 'post_meta_box', 'page_meta_box' ) 
                 ) 
-            ){
+            ) {
                 return "<span class='title-colon'>:</span>" ;
             }                                                 
             

Indentation +1 added lines, -1 removed lines

@@ -172,7 +172,7 @@
                             . $this->getFrameworkNameVersion()
                             . '  ('
                                 . $this->oMsg->get( 'debug_info_will_be_disabled' )
-                              . ')'
+                                . ')'
                         . '</span>',
                     'attributes'    => array(
                         'container' => array(

Spacing +15 added lines, -15 removed lines

@@ -69,7 +69,7 @@ 
         
         $aField = $this->aFieldset;
         
-        if ( ! $aField[ 'show_title_column' ] ) {
+        if ( !$aField[ 'show_title_column' ] ) {
             return '';
         }          
             
@@ -83,7 +83,7 @@ 
             'for'   => $_oInputTagIDGenerator->get(),
         );
         $_sOutput .= $aField[ 'title' ]
-            ? "<label " . $this->getAttributes( $_aLabelAttributes ) . "'>"
+            ? "<label ".$this->getAttributes( $_aLabelAttributes )."'>"
                     . "<a id='{$aField[ 'field_id' ]}'></a>"  // to allow the browser to link to the element.
                     . "<span title='" 
                             . esc_attr( 
@@ -114,12 +114,12 @@ 
          */
         private function _getFieldOutputsInFieldTitleAreaFromNestedFields( $aField ) {
             
-            if ( ! $this->hasNestedFields( $aField ) ) {
+            if ( !$this->hasNestedFields( $aField ) ) {
                 return '';
             }
             
             $_sOutput = '';
-            foreach( $aField[ 'content' ] as $_aNestedField ) {
+            foreach ( $aField[ 'content' ] as $_aNestedField ) {
                 
                 if ( 'field_title' !== $_aNestedField[ 'placement' ] ) {
                     continue;
@@ -127,13 +127,13 @@ 
 
                 $_oFieldset = new AdminPageFramework_Form_View___Fieldset( 
                     $_aNestedField, 
-                    $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,
                     $this->aCallbacks // field output element callables.
                 );
-                $_sOutput   .= $_oFieldset->get(); // field output                
+                $_sOutput .= $_oFieldset->get(); // field output                
 
             }
             return $_sOutput;
@@ -145,7 +145,7 @@ 
          * @since       3.7.0
          */
         private function _getToolTip( $asTip, $sElementID ) {
-            $_oToolTip           = new AdminPageFramework_Form_View___ToolTip(
+            $_oToolTip = new AdminPageFramework_Form_View___ToolTip(
                 $asTip,
                 $sElementID
             );            
@@ -159,14 +159,14 @@ 
          */
         private function _getDebugInfo( $aField ) {
             
-            if ( ! $this->_shouldShowDebugInfo( $aField ) ) {
+            if ( !$this->_shouldShowDebugInfo( $aField ) ) {
                 return '';
             }
-            $_oToolTip           = new AdminPageFramework_Form_View___ToolTip(
+            $_oToolTip = new AdminPageFramework_Form_View___ToolTip(
                 array(
                     'title'         => $this->oMsg->get( 'field_arguments' ),
                     'dash-icon'     => 'dashicons-info',
-                    'icon_alt_text' => '[' . $this->oMsg->get( 'debug' ) . ' ]',
+                    'icon_alt_text' => '['.$this->oMsg->get( 'debug' ).' ]',
                     'content'       => AdminPageFramework_Debug::getDetails( $aField )
                         . '<span class="admin-page-framework-info">'
                             . $this->getFrameworkNameVersion()
@@ -180,7 +180,7 @@ 
                         ),
                     )
                 ),
-                $aField[ 'field_id' ] . '_debug'
+                $aField[ 'field_id' ].'_debug'
             );            
             return $_oToolTip->get();                    
             
@@ -191,7 +191,7 @@ 
              */
             private function _shouldShowDebugInfo( $aField ) {
                 
-                if ( ! $aField[ 'show_debug_info' ] ) {
+                if ( !$aField[ 'show_debug_info' ] ) {
                     return false;
                 }
                 if ( strlen( $aField[ '_parent_field_path' ] ) ) {
@@ -207,7 +207,7 @@ 
          */
         private function _getTitleColon( $aField ) {
             
-            if ( ! isset( $aField[ 'title' ] ) || '' === $aField[ 'title' ] ) {
+            if ( !isset( $aField[ 'title' ] ) || '' === $aField[ 'title' ] ) {
                 return '';
             }                    
             if ( 
@@ -215,8 +215,8 @@ 
                     $aField[ '_structure_type' ], 
                     array( 'widget', 'post_meta_box', 'page_meta_box' ) 
                 ) 
-            ){
-                return "<span class='title-colon'>:</span>" ;
+            ) {
+                return "<span class='title-colon'>:</span>";
             }                                                 
             
         }

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

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,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.        
@@ -60,7 +60,7 @@ 
         
         $aFieldset = $this->aFieldset;
         
-        if ( ! $this->isNormalPlacement( $aFieldset ) ) {
+        if ( !$this->isNormalPlacement( $aFieldset ) ) {
             return '';
         }
    
@@ -71,7 +71,7 @@ 
         return $this->_getFieldByContainer( 
             $aFieldset, 
             array(
-                'open_main'     => "<div " . $_oFieldrowAttribute->get() . ">",
+                'open_main'     => "<div ".$_oFieldrowAttribute->get().">",
                 'close_main'    => "</div>",
             )
         ); 

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

Spacing +5 added lines, -5 removed lines

@@ -67,16 +67,16 @@ 
      */
     protected function getField( $aField ) {
 
-        $_oCallerForm   = $aField[ '_caller_object' ];
+        $_oCallerForm = $aField[ '_caller_object' ];
 
         $_aInlineMixedOutput = array();
-        foreach( $this->getAsArray( $aField[ 'content' ] ) as $_aChildFieldset ) {
+        foreach ( $this->getAsArray( $aField[ 'content' ] ) as $_aChildFieldset ) {
          
             if ( is_scalar( $_aChildFieldset ) ) {
                 continue;
             }
             
-            if ( ! $this->isNormalPlacement( $_aChildFieldset ) ) {
+            if ( !$this->isNormalPlacement( $_aChildFieldset ) ) {
                 continue;
             }
             
@@ -92,12 +92,12 @@ 
             $_oFieldset = new AdminPageFramework_Form_View___Fieldset(
                 $_aChildFieldset,  
                 $_oCallerForm->aSavedData,
-                $_oCallerForm->getFieldErrors(),    // @todo Generate field errors. $this->aErrors, 
+                $_oCallerForm->getFieldErrors(), // @todo Generate field errors. $this->aErrors, 
                 $_oCallerForm->aFieldTypeDefinitions, 
                 $_oCallerForm->oMsg,
                 $_oCallerForm->aCallbacks // field output element callables.
             );
-            $_aInlineMixedOutput[] = $_oFieldset->get(); // field output            
+            $_aInlineMixedOutput[ ] = $_oFieldset->get(); // field output            
             
          
         }

Braces +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
      * 
      * @since       3.8.0
      */        
-    protected function getStyles() { 
+    protected function getStyles() {
         return <<<CSSRULES
 /* Nested fieldsets 3.8.0+ */
 .admin-page-framework-fieldset > .admin-page-framework-fields > .admin-page-framework-field.with-nested-fields > .admin-page-framework-fieldset.multiple-nesting {

development/utility/pointer_tool_tip/AdminPageFramework_PointerToolTip.php

Spacing +11 added lines, -11 removed lines

@@ -112,18 +112,18 @@ 
          */
         private function _setHooks( $aScreenIDs ) {
             
-            foreach( $aScreenIDs as $_sScreenID ) {            
-                if ( ! $_sScreenID ) {
+            foreach ( $aScreenIDs as $_sScreenID ) {            
+                if ( !$_sScreenID ) {
                     continue;
                 }
                 add_filter( 
-                    get_class( $this ) . '-' . $_sScreenID, 
+                    get_class( $this ).'-'.$_sScreenID, 
                     array( $this, '_replyToSetPointer' )
                 );
                                 
             }       
             
-            if ( ! $this->_hasBeenCalled() ) {
+            if ( !$this->_hasBeenCalled() ) {
                 return;
             } 
             
@@ -170,7 +170,7 @@ 
     
         $_aPointers = $this->_getValidPointers( $this->_getPointers() );
              
-        if ( empty( $_aPointers ) || ! is_array( $_aPointers ) ) {
+        if ( empty( $_aPointers ) || !is_array( $_aPointers ) ) {
             return;
         }
         
@@ -190,11 +190,11 @@ 
             $_oScreen   = get_current_screen();
             $_sScreenID = $_oScreen->id;    
             if ( in_array( $_sScreenID, $this->aScreenIDs ) ) {
-                return apply_filters( get_class( $this ) . '-' . $_sScreenID, array() );
+                return apply_filters( get_class( $this ).'-'.$_sScreenID, array() );
             } 
             
             if ( isset( $_GET[ 'page' ] ) ) {
-                return apply_filters( get_class( $this ) . '-' . $_GET[ 'page' ], array() );
+                return apply_filters( get_class( $this ).'-'.$_GET[ 'page' ], array() );
             }
             return array();
             
@@ -208,7 +208,7 @@ 
         private function _getValidPointers( $_aPointers ) {
         
             // Get dismissed pointers
-            $_aDismissed      = explode( 
+            $_aDismissed = explode( 
                 ',', 
                 ( string ) get_user_meta( 
                     get_current_user_id(), 
@@ -236,7 +236,7 @@ 
                 $_aPointer[ 'pointer_id' ] = $_iPointerID;
                 
                 // Add the pointer to $_aValidPointers array
-                $_aValidPointers[] =  $_aPointer;
+                $_aValidPointers[ ] = $_aPointer;
                 
             }            
             return $_aValidPointers;
@@ -312,9 +312,9 @@ 
          * @return      string  
          * @internal
          */
-        public function _getInternalScript( $aPointers=array() ) {
+        public function _getInternalScript( $aPointers = array() ) {
 
-            $_aJSArray      = json_encode( $aPointers );
+            $_aJSArray = json_encode( $aPointers );
 
             /**
              * Checks check-boxes in siblings.

Doc Comments +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
      * 
      * @since       3.7.0
      * @see         https://codex.wordpress.org/Plugin_API/Admin_Screen_Reference
-     * @param       array|strin     $asScreenIDs        Screen IDs or page slug.
+     * @param       string[]     $asScreenIDs        Screen IDs or page slug.
      * @param       string          $sPointerID         A unique pointer ID.
      * @param       array           $aPointerData       The pointer data.
      * <h4>Arguments</h4>

Indentation +1 added lines, -1 removed lines

@@ -213,7 +213,7 @@
                     get_current_user_id(), 
                     'dismissed_wp_pointers', 
                     true 
-               )
+                )
             );
             $_aValidPointers = array();
          

Braces +2 added lines, -2 removed lines

@@ -91,7 +91,7 @@ 
     public function __construct( $asScreenIDs, $sPointerID, array $aPointerData ) {
 
         // Bail if the WordPress version is less than 3.3,
-        if ( version_compare( $GLOBALS[ 'wp_version' ], '3.3', '<' ) ) {        
+        if ( version_compare( $GLOBALS[ 'wp_version' ], '3.3', '<' ) ) {
             return false;
         }       
     
@@ -111,7 +111,7 @@ 
          */
         private function _setHooks( $aScreenIDs ) {
             
-            foreach( $aScreenIDs as $_sScreenID ) {            
+            foreach( $aScreenIDs as $_sScreenID ) {
                 if ( ! $_sScreenID ) {
                     continue;
                 }

development/factory/_common/form/_model/AdminPageFramework_Form_Model___FieldTypeResource.php

Spacing +7 added lines, -7 removed lines

@@ -59,23 +59,23 @@ 
      */
     public function get() {
 
-        $this->aResources[ 'internal_scripts' ]      = $this->_getUpdatedInternalItemsByCallback( 
+        $this->aResources[ 'internal_scripts' ] = $this->_getUpdatedInternalItemsByCallback( 
             $this->aResources[ 'internal_scripts' ], 
             'hfGetScripts' 
         );
-        $this->aResources[ 'internal_styles' ]       = $this->_getUpdatedInternalItemsByCallback( 
+        $this->aResources[ 'internal_styles' ] = $this->_getUpdatedInternalItemsByCallback( 
             $this->aResources[ 'internal_styles' ], 
             'hfGetStyles'
         );
-        $this->aResources[ 'internal_styles_ie' ]    = $this->_getUpdatedInternalItemsByCallback( 
+        $this->aResources[ 'internal_styles_ie' ] = $this->_getUpdatedInternalItemsByCallback( 
             $this->aResources[ 'internal_styles_ie' ], 
             'hfGetIEStyles'
         );        
-        $this->aResources[ 'src_styles' ]          = $this->_getUpdatedEnqueuingItemsByCallback( 
+        $this->aResources[ 'src_styles' ] = $this->_getUpdatedEnqueuingItemsByCallback( 
             $this->aResources[ 'src_styles' ], 
             'aEnqueueStyles' 
         );
-        $this->aResources[ 'src_scripts' ]         = $this->_getUpdatedEnqueuingItemsByCallback( 
+        $this->aResources[ 'src_scripts' ] = $this->_getUpdatedEnqueuingItemsByCallback( 
             $this->aResources[ 'src_scripts' ], 
             'aEnqueueScripts' 
         );
@@ -89,10 +89,10 @@ 
          */
         private function _getUpdatedInternalItemsByCallback( array $aSubject, $sKey ) {
             $_oCallable = $this->getElement( $this->aFieldTypeDefinition, $sKey );
-            if ( ! is_callable( $_oCallable ) ) {
+            if ( !is_callable( $_oCallable ) ) {
                 return $aSubject;
             }
-            $aSubject[] = call_user_func_array( 
+            $aSubject[ ] = call_user_func_array( 
                 $_oCallable, 
                 array() 
             );            

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

Spacing +31 added lines, -31 removed lines

@@ -59,7 +59,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'image', );
+    public $aFieldTypeSlugs = array( 'image',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -68,8 +68,8 @@ 
      */
     protected $aDefaultKeys = array(
         'attributes_to_store'       => array(), // ( array ) This is for the image and media field type. The attributes to save besides URL. e.g. ( for the image field type ) array( 'title', 'alt', 'width', 'height', 'caption', 'id', 'align', 'link' ).
-        'show_preview'              => true,    // ( boolean ) Indicates whether the image preview should be displayed or not.
-        'allow_external_source'     => true,    // ( boolean ) Indicates whether the media library box has the From URL tab.
+        'show_preview'              => true, // ( boolean ) Indicates whether the image preview should be displayed or not.
+        'allow_external_source'     => true, // ( boolean ) Indicates whether the media library box has the From URL tab.
         'attributes'                => array(
             'input'     => array(
                 'size'      => 40,
@@ -99,7 +99,7 @@ 
         return // $this->_getScript_CustomMediaUploaderObject() . PHP_EOL    
             $this->_getScript_ImageSelector( 
                 "admin_page_framework"
-            )  . PHP_EOL
+            ).PHP_EOL
             . $this->_getScript_RegisterCallbacks();
     }
         /**
@@ -217,7 +217,7 @@ 
             $_sInsertFromURL         = esc_js( $this->oMsg->get( 'insert_from_url' ) );
 
             // if the WordPress version is 3.4.x or below
-            if ( ! function_exists( 'wp_enqueue_media' ) ) {
+            if ( !function_exists( 'wp_enqueue_media' ) ) {
             
                 return <<<JAVASCRIPTS
 /**
@@ -605,13 +605,13 @@ 
                 . "<label for='{$aField[ 'input_id' ]}'>"
                     . $aField[ 'before_input' ]
                     . $this->getAOrB(
-                        $aField[ 'label' ] && ! $aField[ 'repeatable' ],
-                        "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
+                        $aField[ 'label' ] && !$aField[ 'repeatable' ],
+                        "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
                             . $aField[ 'label' ] 
                         . "</span>",                        
                         ''                        
                     )
-                    . "<input " . $this->getAttributes( $this->_getImageInputAttributes( $aField, $_iCountAttributes, $_sImageURL, $_aBaseAttributes ) ) . " />" 
+                    . "<input ".$this->getAttributes( $this->_getImageInputAttributes( $aField, $_iCountAttributes, $_sImageURL, $_aBaseAttributes ) )." />" 
                     . $aField[ 'after_input' ]
                     . "<div class='repeatable-field-buttons'></div>" // the repeatable field buttons will be replaced with this element.
                     . $this->getExtraInputFields( $aField )
@@ -651,7 +651,7 @@ 
          */
         private function _getBaseAttributes( array $aField ) {
             
-            $_aBaseAttributes   = $aField[ 'attributes' ] + array( 'class' => null );
+            $_aBaseAttributes = $aField[ 'attributes' ] + array( 'class' => null );
             unset( 
                 $_aBaseAttributes[ 'input' ], 
                 $_aBaseAttributes[ 'button' ], 
@@ -720,8 +720,8 @@ 
         protected function getExtraInputFields( array $aField ) {
             
             $_aOutputs = array();
-            foreach( $this->getElementAsArray( $aField, 'attributes_to_store' ) as $sAttribute ) {
-                $_aOutputs[] = "<input " . $this->getAttributes( 
+            foreach ( $this->getElementAsArray( $aField, 'attributes_to_store' ) as $sAttribute ) {
+                $_aOutputs[ ] = "<input ".$this->getAttributes( 
                     array(
                         'id'        => "{$aField[ 'input_id' ]}_{$sAttribute}",
                         'type'      => 'hidden',
@@ -737,7 +737,7 @@ 
                             ''
                         ),
                     )
-                ) . "/>";
+                )."/>";
             }
             return implode( PHP_EOL, $_aOutputs );
             
@@ -750,17 +750,17 @@ 
          */
         protected function _getPreviewContainer( $aField, $sImageURL, $aPreviewAtrributes ) {
 
-            if ( ! $aField[ 'show_preview' ] ) { 
+            if ( !$aField[ 'show_preview' ] ) { 
                 return ''; 
             }
             
             $sImageURL = esc_url( $this->getResolvedSRC( $sImageURL, true ) );
             return 
-                "<div " . $this->getAttributes( 
+                "<div ".$this->getAttributes( 
                         array(
                             'id'    => "image_preview_container_{$aField[ 'input_id' ]}",     
-                            'class' => 'image_preview ' . $this->getElement( $aPreviewAtrributes, 'class', '' ),
-                            'style' => $this->getAOrB( $sImageURL, '', "display: none; "  ) 
+                            'class' => 'image_preview '.$this->getElement( $aPreviewAtrributes, 'class', '' ),
+                            'style' => $this->getAOrB( $sImageURL, '', "display: none; " ) 
                                 . $this->getElement( $aPreviewAtrributes, 'style', '' ),
                         ) + $aPreviewAtrributes
                     )
@@ -783,12 +783,12 @@ 
          */
         protected function _getUploaderButtonScript( $sInputID, $abRepeatable, $bExternalSource, array $aButtonAttributes ) {
             
-            $_bRepeatable     = ! empty( $abRepeatable );
+            $_bRepeatable = !empty( $abRepeatable );
             
             // Do not include the escaping character (backslash) in the heredoc variable declaration 
             // because the minifier script will parse it and the <<<JAVASCRIPTS and JAVASCRIPTS; parts are converted to double quotes (")
             // which causes the PHP syntax error.                
-            $_sButtonHTML     = '"' . $this->_getUploaderButtonHTML( $sInputID, $aButtonAttributes, $_bRepeatable, $bExternalSource ) . '"';
+            $_sButtonHTML     = '"'.$this->_getUploaderButtonHTML( $sInputID, $aButtonAttributes, $_bRepeatable, $bExternalSource ).'"';
             $_sRepeatable     = $this->getAOrB( $_bRepeatable, 'true', 'false' );
             $_bExternalSource = $this->getAOrB( $bExternalSource, 'true', 'false' );
             $_sScript = <<<JAVASCRIPTS
@@ -804,7 +804,7 @@ 
                     . '/* <![CDATA[ */'
                     . $_sScript 
                     . '/* ]]> */'
-                . "</script>". PHP_EOL;
+                . "</script>".PHP_EOL;
 
         }
             /**
@@ -823,7 +823,7 @@ 
                     $bRepeatable, 
                     $bExternalSource 
                 );
-                return "<a " . $this->getAttributes( $_aAttributes ) . ">"
+                return "<a ".$this->getAttributes( $_aAttributes ).">"
                         . ( $_bIsLabelSet
                             ? $_aAttributes[ 'data-label' ] 
                             : ( strrpos( $_aAttributes[ 'class' ], 'dashicons' )
@@ -842,7 +842,7 @@ 
                  */
                 private function _getFormattedUploadButtonAttributes( $sInputID, array $aButtonAttributes, $_bIsLabelSet, $bRepeatable, $bExternalSource ) {
                         
-                    $_aAttributes           = array(
+                    $_aAttributes = array(
                             'id'        => "select_image_{$sInputID}",
                             'href'      => '#',            
                             'data-uploader_type'            => ( string ) function_exists( 'wp_enqueue_media' ),
@@ -855,7 +855,7 @@ 
                                 : $this->oMsg->get( 'select_image' ),
                             'data-label' => null,
                         );
-                    $_aAttributes[ 'class' ]  = $this->getClassAttribute( 
+                    $_aAttributes[ 'class' ] = $this->getClassAttribute( 
                         'select_image button button-small ',
                         $this->getAOrB(
                             trim( $aButtonAttributes[ 'class' ] ),
@@ -883,16 +883,16 @@ 
          * @return      string
          * @internal
          */
-        protected function _getRemoveButtonScript( $sInputID, array $aButtonAttributes, $sType='image' ) {
+        protected function _getRemoveButtonScript( $sInputID, array $aButtonAttributes, $sType = 'image' ) {
            
-            if ( ! function_exists( 'wp_enqueue_media' ) ) {
+            if ( !function_exists( 'wp_enqueue_media' ) ) {
                 return '';
             }
     
             // Do not include the escaping character (backslash) in the heredoc variable declaration 
             // because the minifier script will parse it and the <<<JAVASCRIPTS and JAVASCRIPTS; parts are converted to double quotes (")
             // which causes the PHP syntax error.                
-            $_sButtonHTML  = '"' . $this->_getRemoveButtonHTMLByType( $sInputID, $aButtonAttributes, $sType ) . '"';
+            $_sButtonHTML = '"'.$this->_getRemoveButtonHTMLByType( $sInputID, $aButtonAttributes, $sType ).'"';
             $_sScript = <<<JAVASCRIPTS
                 if ( 0 === jQuery( 'a#remove_{$sType}_{$sInputID}' ).length ) {
                     jQuery( 'input#{$sInputID}' ).after( $_sButtonHTML );
@@ -903,7 +903,7 @@ 
                     . '/* <![CDATA[ */'
                     . $_sScript 
                     . '/* ]]> */'
-                . "</script>". PHP_EOL;
+                . "</script>".PHP_EOL;
            
         }
         
@@ -913,11 +913,11 @@ 
          * @return      string      The generated HTML remove button output.
          * @internal
          */
-        protected function _getRemoveButtonHTMLByType( $sInputID, array $aButtonAttributes, $sType='image' ) {
+        protected function _getRemoveButtonHTMLByType( $sInputID, array $aButtonAttributes, $sType = 'image' ) {
         
             $_bIsLabelSet   = isset( $aButtonAttributes[ 'data-label' ] ) && $aButtonAttributes[ 'data-label' ];
             $_aAttributes   = $this->_getFormattedRemoveButtonAttributesByType( $sInputID, $aButtonAttributes, $_bIsLabelSet, $sType );
-            return "<a " . $this->getAttributes( $_aAttributes ) . ">"
+            return "<a ".$this->getAttributes( $_aAttributes ).">"
                     . ( $_bIsLabelSet
                         ? $_aAttributes[ 'data-label' ] 
                         : $this->getAOrB(
@@ -936,9 +936,9 @@ 
              * @return      array       The formatted remove button attributes array.
              * @internal
              */
-            protected function _getFormattedRemoveButtonAttributesByType( $sInputID, array $aButtonAttributes, $_bIsLabelSet, $sType='image' ) {
+            protected function _getFormattedRemoveButtonAttributesByType( $sInputID, array $aButtonAttributes, $_bIsLabelSet, $sType = 'image' ) {
 
-                $_sOnClickFunctionName  = 'removeInputValuesFor' . ucfirst( $sType );
+                $_sOnClickFunctionName  = 'removeInputValuesFor'.ucfirst( $sType );
                 $_aAttributes           = array(
                         'id'        => "remove_{$sType}_{$sInputID}",
                         'href'      => '#',            
@@ -950,7 +950,7 @@ 
                             ? $aButtonAttributes[ 'data-label' ] 
                             : $this->oMsg->get( 'remove_value' ),
                     );
-                $_aAttributes[ 'class' ]  = $this->getClassAttribute( 
+                $_aAttributes[ 'class' ] = $this->getClassAttribute( 
                     "remove_value remove_{$sType} button button-small", 
                     $this->getAOrB(
                         trim( $aButtonAttributes[ 'class' ] ),

Indentation +1 added lines, -1 removed lines

@@ -625,7 +625,7 @@
             return $_aBaseAttributes;
             
         }    
-       /**
+        /**
          * Returns the set image url.
          * 
          * When the 'attributes_to_store' argument is present, there will be sub elements to the field value.

Braces +2 added lines, -2 removed lines

@@ -476,7 +476,7 @@ 
      * Returns the field type specific CSS rules.
      * @internal
      */ 
-    protected function getStyles() { 
+    protected function getStyles() {
         return <<<CSSRULES
 /* Image Field Preview Container */
 .admin-page-framework-field .image_preview {
@@ -714,7 +714,7 @@ 
          */
         protected function _getPreviewContainer( $aField, $sImageURL, $aPreviewAtrributes ) {
 
-            if ( ! $aField[ 'show_preview' ] ) { 
+            if ( ! $aField[ 'show_preview' ] ) {
                 return ''; 
             }
             

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

Spacing +29 added lines, -29 removed lines

@@ -65,7 +65,7 @@ 
     /**
      * Defines the field type slugs used for this field type.
      */
-    public $aFieldTypeSlugs = array( 'size', );
+    public $aFieldTypeSlugs = array( 'size',);
     
     /**
      * Defines the default key-values of this field type. 
@@ -74,7 +74,7 @@ 
      */
     protected $aDefaultKeys = array(
         'is_multiple'           => false, // indicates whether the select tag alloes multiple selections.
-        'units'                 => null,  // do not define units here since this will be merged with the user defined field array.
+        'units'                 => null, // do not define units here since this will be merged with the user defined field array.
         'attributes'            => array(
             'size'      => array(
                 'min'           => null,
@@ -102,7 +102,7 @@ 
      */
     protected $aDefaultUnits = array(
         'px'    => 'px', // pixel
-        '%'     => '%',  // percentage
+        '%'     => '%', // percentage
         'em'    => 'em', // font size
         'ex'    => 'ex', // font height
         'in'    => 'in', // inch
@@ -162,8 +162,8 @@ 
         );
 
         $_aOutput = array();
-        foreach( ( array ) $aField[ 'label' ] as $_isKey => $_sLabel ) {
-            $_aOutput[] = $this->_getFieldOutputByLabel( 
+        foreach ( ( array ) $aField[ 'label' ] as $_isKey => $_sLabel ) {
+            $_aOutput[ ] = $this->_getFieldOutputByLabel( 
                 $_isKey, 
                 $_sLabel,
                 $aField
@@ -186,8 +186,8 @@ 
 
             $_aBaseAttributes   = $_bMultiLabels
                 ? array(
-                        'name'  => $aField[ 'attributes' ][ 'name' ] . "[{$isKey}]",
-                        'id'    => $aField[ 'attributes' ][ 'id' ] . "_{$isKey}",
+                        'name'  => $aField[ 'attributes' ][ 'name' ]."[{$isKey}]",
+                        'id'    => $aField[ 'attributes' ][ 'id' ]."_{$isKey}",
                         'value' => $aField[ 'value' ],
                     ) 
                     + $aField[ 'attributes' ]
@@ -199,9 +199,9 @@ 
                      
             $_aOutput = array(
                 $this->getElementByLabel( $aField[ 'before_label' ], $isKey, $aField[ 'label' ] ),
-                    "<div " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-container admin-page-framework-select-label' ) . ">",
-                        $this->_getNumberInputPart( $aField, $_aBaseAttributes, $isKey, is_array( $aField[ 'label' ] ) ),  // The size (number) part
-                        $this->_getUnitSelectInput( $aField, $_aBaseAttributes, $isKey, is_array( $aField[ 'label' ] ) ),  // The unit (select) part
+                    "<div ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-container admin-page-framework-select-label' ).">",
+                        $this->_getNumberInputPart( $aField, $_aBaseAttributes, $isKey, is_array( $aField[ 'label' ] ) ), // The size (number) part
+                        $this->_getUnitSelectInput( $aField, $_aBaseAttributes, $isKey, is_array( $aField[ 'label' ] ) ), // The unit (select) part
                     "</div>",
                 $this->getElementByLabel( $aField[ 'after_label' ], $isKey, $aField[ 'label' ] )
             );
@@ -218,7 +218,7 @@ 
             private function _getNumberInputPart( array $aField, array $aBaseAttributes, $isKey, $bMultiLabels ) {
                 
                 // Size and Size Label
-                $_aSizeAttributes       = $this->_getSizeAttributes( 
+                $_aSizeAttributes = $this->_getSizeAttributes( 
                     $aField, 
                     $aBaseAttributes,
                     $bMultiLabels
@@ -226,32 +226,32 @@ 
                         : ''
                 );
 
-                $_aSizeLabelAttributes  = array(
+                $_aSizeLabelAttributes = array(
                     'for'   => $_aSizeAttributes[ 'id' ],
                     'class' => $_aSizeAttributes[ 'disabled' ] 
                         ? 'disabled' 
                         : null,
                 );                  
                 
-                $_sLabel                = $this->getElementByLabel( 
+                $_sLabel = $this->getElementByLabel( 
                     $aField[ 'label' ], 
                     $isKey, 
                     $aField[ 'label' ]
                 );
-                return "<label " . $this->getAttributes( $_aSizeLabelAttributes ) . ">"
+                return "<label ".$this->getAttributes( $_aSizeLabelAttributes ).">"
                     . $this->getElement( 
                         $aField, 
                         $bMultiLabels
                             ? array( 'before_label', $isKey, 'size' ) 
                             : array( 'before_label', 'size' ) 
                     )
-                    . ( $aField[ 'label' ] && ! $aField[ 'repeatable' ]
-                        ? "<span " . $this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ) . ">" 
+                    . ( $aField[ 'label' ] && !$aField[ 'repeatable' ]
+                        ? "<span ".$this->getLabelContainerAttributes( $aField, 'admin-page-framework-input-label-string' ).">" 
                                 . $_sLabel 
                             . "</span>"
                         : "" 
                     )
-                    . "<input " . $this->getAttributes( $_aSizeAttributes ) . " />" 
+                    . "<input ".$this->getAttributes( $_aSizeAttributes )." />" 
                     . $this->getElement( 
                         $aField, 
                         $bMultiLabels
@@ -294,7 +294,7 @@ 
                     )
                     : $aField[ 'units' ];
                 
-                return "<label " . $this->getAttributes( 
+                return "<label ".$this->getAttributes( 
                         array(
                             'for'       => $_aUnitAttributes[ 'id' ],
                             'class'     => $_aUnitAttributes[ 'disabled' ] 
@@ -326,9 +326,9 @@ 
                  * @return      array       an unit attribute array
                  * @internal
                  */
-                private function _getUnitAttributes( array $aField, array $aBaseAttributes, $isLabelKey='' ) {
+                private function _getUnitAttributes( array $aField, array $aBaseAttributes, $isLabelKey = '' ) {
                     
-                    $_bIsMultiple    = $aField[ 'is_multiple' ] 
+                    $_bIsMultiple = $aField[ 'is_multiple' ] 
                         ? true 
                         : $this->getElement( 
                             $aField,
@@ -340,13 +340,13 @@ 
               
                     $_aSelectAttributes = array(
                         'type'      => 'select',
-                        'id'        => $aField[ 'input_id' ] . ( '' === $isLabelKey ? '' : '_' . $isLabelKey ) . '_' . 'unit',
+                        'id'        => $aField[ 'input_id' ].( '' === $isLabelKey ? '' : '_'.$isLabelKey ).'_'.'unit',
                         'multiple'  => $_bIsMultiple 
                             ? 'multiple' 
                             : null,
                         'name'      => $_bIsMultiple 
-                            ? "{$aField['_input_name']}" . ( '' === $isLabelKey ? '' : '[' . $isLabelKey . ']' ) . "[unit][]" 
-                            : "{$aField['_input_name']}" . ( '' === $isLabelKey ? '' : '[' . $isLabelKey . ']' ) . "[unit]",
+                            ? "{$aField[ '_input_name' ]}".( '' === $isLabelKey ? '' : '['.$isLabelKey.']' )."[unit][]" 
+                            : "{$aField[ '_input_name' ]}".( '' === $isLabelKey ? '' : '['.$isLabelKey.']' )."[unit]",
                         'value'     => $this->getElement( 
                             $aField, 
                             array( 'value', 'unit' ),
@@ -358,7 +358,7 @@ 
                         '' === $isLabelKey
                             ? array( 'attributes', 'unit' )
                             : array( 'attributes', $isLabelKey, 'unit' ),
-                        $this->aDefaultKeys['attributes']['unit'] 
+                        $this->aDefaultKeys[ 'attributes' ][ 'unit' ] 
                     )
                     + $aBaseAttributes;       
                     return $_aSelectAttributes;
@@ -372,15 +372,15 @@ 
              * @return      array       an size attribute array
              * @internal
              */
-            private function _getSizeAttributes( array $aField, array $aBaseAttributes, $sLabelKey='' ) {
+            private function _getSizeAttributes( array $aField, array $aBaseAttributes, $sLabelKey = '' ) {
 
                 return array(
                         'type'  => 'number',
-                        'id'    => $aField['input_id'] . '_' . ( '' !== $sLabelKey ? $sLabelKey . '_' : '' ) . 'size',
-                        'name'  => $aField[ '_input_name' ] . ( '' !== $sLabelKey ? "[{$sLabelKey}]" : '' ) . '[size]',
+                        'id'    => $aField[ 'input_id' ].'_'.( '' !== $sLabelKey ? $sLabelKey.'_' : '' ).'size',
+                        'name'  => $aField[ '_input_name' ].( '' !== $sLabelKey ? "[{$sLabelKey}]" : '' ).'[size]',
                         'value' => $this->getElement(
-                            $aField,        // subject
-                            array( 'value', 'size' ),   // dimensional keys
+                            $aField, // subject
+                            array( 'value', 'size' ), // dimensional keys
                             ''  // default
                         ),
                     ) 

Doc Comments +2 added lines

@@ -211,6 +211,7 @@ 
              * Returns the HTML output of the number input part.
              * 
              * @since       3.5.3
+             * @param boolean $bMultiLabels
              * @return      string      The number input output.
              * @internal
              */
@@ -265,6 +266,7 @@ 
              * Returns the HTML output of the unit select input part.
              * 
              * @since       3.5.3
+             * @param boolean $bMultiLabels
              * @return      string      The unit select input output.
              * @internal
              */