michaeluno / admin-page-framework

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

Doc Comments +1 added lines

@@ -391,6 +391,7 @@
      * This is used for element that `label_min_width` is applied.
      * 
      * @since       3.8.0
+     * @param string $asClassAttributes
      * @return      string
      */
     protected function getLabelContainerAttributes( array $aField, $asClassAttributes, array $aAttributes=array() ) {

Spacing +27 added lines, -27 removed lines

@@ -60,7 +60,7 @@ 
         'after_label'       => null,    
         'before_field'      => null,
         'after_field'       => null,
-        'label_min_width'   => 0,   // (string|integer) min-width applied to the input label in pixels. 3.8.0+ Changed the default value from 140 to 0 as it is now applie through embedded CSS. When this value is set by the user, it is set inline and the value will be overriden.
+        'label_min_width'   => 0, // (string|integer) min-width applied to the input label in pixels. 3.8.0+ Changed the default value from 140 to 0 as it is now applie through embedded CSS. When this value is set by the user, it is set inline and the value will be overriden.
         'before_fieldset'   => null, // 3.1.1+
         'after_fieldset'    => null, // 3.1.1+
         
@@ -100,7 +100,7 @@ 
      * @param       object              $oMsg                   The framework message object.
      * @param       boolean             $bAutoRegister          Whether or not to register the field type(s).
      */
-    public function __construct( $asClassName='admin_page_framework', $asFieldTypeSlug=null, $oMsg=null, $bAutoRegister=true ) {
+    public function __construct( $asClassName = 'admin_page_framework', $asFieldTypeSlug = null, $oMsg = null, $bAutoRegister = true ) {
             
         $this->aFieldTypeSlugs  = empty( $asFieldTypeSlug ) 
             ? $this->aFieldTypeSlugs 
@@ -111,9 +111,9 @@ 
         
         // This automatically registers the field type. The build-in ones will be registered manually so it will be skipped.
         if ( $bAutoRegister ) {
-            foreach( ( array ) $asClassName as $_sClassName ) {
+            foreach ( ( array ) $asClassName as $_sClassName ) {
                 add_filter( 
-                    'field_types_' . $_sClassName, 
+                    'field_types_'.$_sClassName, 
                     array( $this, '_replyToRegisterInputFieldType' )
                 );
             }
@@ -159,8 +159,8 @@ 
         }
         return is_array( $asLabel ) // if the user stes multiple items
             ? $this->getElement( 
-                $asElement,         // subject
-                $this->getAsArray( $asKey, true /* preserve empty */ ),     // dimensional path 
+                $asElement, // subject
+                $this->getAsArray( $asKey, true /* preserve empty */ ), // dimensional path 
                 ''                  // default - if the element is not found, return an empty
             )
             : $asElement;
@@ -175,7 +175,7 @@ 
      */
     protected function getFieldOutput( array $aFieldset ) {
         
-        if ( ! is_object( $aFieldset[ '_caller_object' ] ) ) {
+        if ( !is_object( $aFieldset[ '_caller_object' ] ) ) {
             return '';
         }
 
@@ -183,14 +183,14 @@ 
         $aFieldset[ '_parent_field_object' ] = $aFieldset[ '_field_object' ]; // 3.6.0+
         
         // 3.7.0+ The caller object is no longer a factory object but a form object.
-        $_oCallerForm   = $aFieldset[ '_caller_object' ];
+        $_oCallerForm = $aFieldset[ '_caller_object' ];
 
         $_oFieldset = new AdminPageFramework_Form_View___Fieldset( 
-            $aFieldset,                          // the field definition array
-            $_oCallerForm->aSavedData,               // the stored form data
-            $_oCallerForm->getFieldErrors(),         // the field error array.
-            $_oCallerForm->aFieldTypeDefinitions,    // the field type definition array.
-            $_oCallerForm->oMsg,                     // the system message object
+            $aFieldset, // the field definition array
+            $_oCallerForm->aSavedData, // the stored form data
+            $_oCallerForm->getFieldErrors(), // the field error array.
+            $_oCallerForm->aFieldTypeDefinitions, // the field type definition array.
+            $_oCallerForm->oMsg, // the system message object
             $_oCallerForm->aCallbacks                // field output element callables.
         );           
         return $_oFieldset->get();
@@ -230,13 +230,13 @@ 
      * @since       3.0.3       Tweaked it to have better execution speed.
      * @internal
      */
-    public function getDefinitionArray( $sFieldTypeSlug='' ) {
+    public function getDefinitionArray( $sFieldTypeSlug = '' ) {
         
         // The uniteArrays() method resulted in somewhat being slow due to overhead on checking array keys for recursive array merges.
         $_aDefaultKeys = $this->aDefaultKeys + self::$_aDefaultKeys;
-        $_aDefaultKeys['attributes'] = isset( $this->aDefaultKeys['attributes'] ) && is_array( $this->aDefaultKeys['attributes'] )
-            ? $this->aDefaultKeys['attributes'] + self::$_aDefaultKeys['attributes'] 
-            : self::$_aDefaultKeys['attributes'];
+        $_aDefaultKeys[ 'attributes' ] = isset( $this->aDefaultKeys[ 'attributes' ] ) && is_array( $this->aDefaultKeys[ 'attributes' ] )
+            ? $this->aDefaultKeys[ 'attributes' ] + self::$_aDefaultKeys[ 'attributes' ] 
+            : self::$_aDefaultKeys[ 'attributes' ];
         
         return array(
             'sFieldTypeSlug'        => $sFieldTypeSlug,
@@ -274,7 +274,7 @@ 
      * @since       3.0.0
      * @internal
      */
-    public function _replyToFieldTypeSetter( $sFieldSetType='' ) {
+    public function _replyToFieldTypeSetter( $sFieldSetType = '' ) {
         $this->_sFieldSetType = $sFieldSetType;
     }
     
@@ -328,8 +328,8 @@ 
             wp_enqueue_script( 'media-upload' );    
         }
 
-        if ( in_array( $this->getPageNow(), array( 'media-upload.php', 'async-upload.php', ) ) ) {     
-            add_filter( 'gettext', array( $this, '_replyToReplaceThickBoxText' ) , 1, 2 );     
+        if ( in_array( $this->getPageNow(), array( 'media-upload.php', 'async-upload.php',) ) ) {     
+            add_filter( 'gettext', array( $this, '_replyToReplaceThickBoxText' ), 1, 2 );     
         }
         
     }
@@ -344,7 +344,7 @@ 
         public function _replyToReplaceThickBoxText( $sTranslated, $sText ) {
 
             // Replace the button label in the media thick box.
-            if ( ! in_array( $this->getPageNow(), array( 'media-upload.php', 'async-upload.php' ) ) ) { 
+            if ( !in_array( $this->getPageNow(), array( 'media-upload.php', 'async-upload.php' ) ) ) { 
                 return $sTranslated; 
             }
             if ( $sText !== 'Insert into Post' ) { 
@@ -354,8 +354,8 @@ 
                 return $sTranslated; 
             }
             
-            if ( isset( $_GET['button_label'] ) ) { 
-                return $_GET['button_label']; 
+            if ( isset( $_GET[ 'button_label' ] ) ) { 
+                return $_GET[ 'button_label' ]; 
             }
 
             return $this->oProp->sThickBoxButtonUseThis 
@@ -374,12 +374,12 @@ 
          */
         public function _replyToRemovingMediaLibraryTab( $aTabs ) {
             
-            if ( ! isset( $_REQUEST['enable_external_source'] ) ) { 
+            if ( !isset( $_REQUEST[ 'enable_external_source' ] ) ) { 
                 return $aTabs; 
             }
             
-            if ( ! $_REQUEST['enable_external_source'] ) {
-                unset( $aTabs['type_url'] ); // removes the 'From URL' tab in the thick box.
+            if ( !$_REQUEST[ 'enable_external_source' ] ) {
+                unset( $aTabs[ 'type_url' ] ); // removes the 'From URL' tab in the thick box.
             }
             return $aTabs;
             
@@ -393,7 +393,7 @@ 
      * @since       3.8.0
      * @return      string
      */
-    protected function getLabelContainerAttributes( array $aField, $asClassAttributes, array $aAttributes=array() ) {
+    protected function getLabelContainerAttributes( array $aField, $asClassAttributes, array $aAttributes = array() ) {
 
         $aAttributes[ 'class' ] = $this->getClassAttribute( $asClassAttributes, $this->getElement( $aAttributes, 'class' ) );
         $aAttributes[ 'style' ] = $this->getStyleAttribute(

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

Doc Comments +2 added lines

@@ -183,6 +183,8 @@
     }
     /**
      * Trims a traling sub-string if exists.
+     * @param string $sString
+     * @param string $sSuffix
      * @return      string
      * @since       3.7.2
      */

Spacing +10 added lines, -10 removed lines

@@ -27,15 +27,15 @@ 
      * @since       3.8.0   Renamed from `sanitizeLength()`.
      * @return      string
      */
-    static public function getLengthSanitized( $sLength, $sUnit='px' ) {
+    static public function getLengthSanitized( $sLength, $sUnit = 'px' ) {
         return is_numeric( $sLength ) 
-            ? $sLength . $sUnit
+            ? $sLength.$sUnit
             : $sLength;
     }    
         /**
          * @deprecated  3.8.0       Use `getLengthSanitized()` instead.
          */
-        static public function sanitizeLength( $sLength, $sUnit='px' ) {
+        static public function sanitizeLength( $sLength, $sUnit = 'px' ) {
             return self::getLengthSanitized( $sLength, $sUnit );
         }
   
@@ -78,9 +78,9 @@ 
      * @since       2.0.0
      * @return      string|integer      A numeric value will be returned. 
      */ 
-    static public function fixNumber( $nToFix, $nDefault, $nMin='', $nMax='' ) {
+    static public function fixNumber( $nToFix, $nDefault, $nMin = '', $nMax = '' ) {
 
-        if ( ! is_numeric( trim( $nToFix ) ) ) { 
+        if ( !is_numeric( trim( $nToFix ) ) ) { 
             return $nDefault; 
         }
         if ( $nMin !== '' && $nToFix < $nMin ) { 
@@ -102,7 +102,7 @@ 
      */
     static public function getCSSMinified( $sCSSRules ) {
         return str_replace( 
-            array( "\r\n", "\r", "\n", "\t", '  ', '    ', '    '),  // remove line breaks, tab, and white sspaces.
+            array( "\r\n", "\r", "\n", "\t", '  ', '    ', '    ' ), // remove line breaks, tab, and white sspaces.
             '', 
             preg_replace( '!/\*[^*]*\*+([^/][^*]*\*+)*/!', '', $sCSSRules ) // remove comments
         );
@@ -112,7 +112,7 @@ 
          */
         static public function minifyCSS( $sCSSRules ) {
             trigger_error( 
-                AdminPageFramework_Registry::NAME . ': ' . sprintf(
+                AdminPageFramework_Registry::NAME.': '.sprintf(
                     'The method, %1$s, is deprecated. Use %2$s instead.',
                     'minifyCSS()',
                     'getCSSMinified()'
@@ -141,8 +141,8 @@ 
      */     
     static public function getNumberOfReadableSize( $nSize ) {
         
-        $_nReturn     = substr( $nSize, 0, -1 );
-        switch( strtoupper( substr( $nSize, -1 ) ) ) {
+        $_nReturn = substr( $nSize, 0, -1 );
+        switch ( strtoupper( substr( $nSize, -1 ) ) ) {
             case 'P':
                 $_nReturn *= 1024;
             case 'T':
@@ -168,7 +168,7 @@ 
         $_nLog      = log( $nBytes, 1024 );
         $_iPower    = ( int ) $_nLog;
         $_iSize     = pow( 1024, $_nLog - $_iPower );
-        return $_iSize . $_aUnits[ $_iPower ];
+        return $_iSize.$_aUnits[ $_iPower ];
     }
 
     /**

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.

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

Spacing +11 added lines, -11 removed lines

@@ -36,11 +36,11 @@ 
      */
     static public function getInlineCSS( array $aCSSRules ) {
         $_aOutput = array();
-        foreach( $aCSSRules as $_sProperty => $_sValue ) {
+        foreach ( $aCSSRules as $_sProperty => $_sValue ) {
             if ( is_null( $_sValue ) ) {
                 continue;
             }
-            $_aOutput[] = $_sProperty . ': ' . $_sValue;
+            $_aOutput[ ] = $_sProperty.': '.$_sValue;
         }
         return implode( '; ', $_aOutput );
     }    
@@ -71,7 +71,7 @@ 
     static public function getStyleAttribute( $asInlineCSSes ) {
         
         $_aCSSRules = array();
-        foreach( array_reverse( func_get_args() ) as $_asCSSRules ) {
+        foreach ( array_reverse( func_get_args() ) as $_asCSSRules ) {
             
             // For array, store in the container.
             if ( is_array( $_asCSSRules ) ) {
@@ -81,9 +81,9 @@ 
             
             // At this point, it is a string. Break them down to array elements.
             $__aCSSRules = explode( ';', $_asCSSRules );
-            foreach( $__aCSSRules as $_sPair ) {
+            foreach ( $__aCSSRules as $_sPair ) {
                 $_aCSSPair = explode( ':', $_sPair );
-                if ( ! isset( $_aCSSPair[ 0 ], $_aCSSPair[ 1 ] ) ) {
+                if ( !isset( $_aCSSPair[ 0 ], $_aCSSPair[ 1 ] ) ) {
                     continue;
                 }
                 $_aCSSRules[ $_aCSSPair[ 0 ] ] = $_aCSSPair[ 1 ];
@@ -120,9 +120,9 @@ 
      */
     static public function getClassAttribute( /* $asClassSelectors1, $asClassSelectors12, ... */ ) {
         
-        $_aClasses  = array();
-        foreach( func_get_args() as $_asClasses ) {
-            if ( ! in_array( gettype( $_asClasses ), array( 'array', 'string' ) ) ) {
+        $_aClasses = array();
+        foreach ( func_get_args() as $_asClasses ) {
+            if ( !in_array( gettype( $_asClasses ), array( 'array', 'string' ) ) ) {
                 continue;
             }            
             $_aClasses = array_merge( 
@@ -132,7 +132,7 @@ 
                     : explode( ' ', $_asClasses )
             );
         }
-        $_aClasses  = array_unique( array_filter( $_aClasses ) );
+        $_aClasses = array_unique( array_filter( $_aClasses ) );
         
         // @todo examine if it is okay to remove the trim() function below.
         return trim( implode( ' ', $_aClasses ) );
@@ -148,7 +148,7 @@ 
         static public function generateClassAttribute( /* $asClassSelectors1, $asClassSelectors12 ... */ ) {
             $_aParams = func_get_args();
             return call_user_func_array(
-                array( __CLASS__ , 'getClassAttribute' ), 
+                array( __CLASS__, 'getClassAttribute' ), 
                 $_aParams
             );        
         }
@@ -162,7 +162,7 @@ 
     static public function getDataAttributeArray( array $aArray ) {
         
         $_aNewArray = array();
-        foreach( $aArray as $sKey => $v ) {
+        foreach ( $aArray as $sKey => $v ) {
             if ( in_array( gettype( $v ), array( 'array', 'object' ) ) ) {
                 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/_model/AdminPageFramework_Form_Model___SetFieldResources.php

Spacing +13 added lines, -13 removed lines

@@ -92,16 +92,16 @@ 
                 'AdminPageFramework_Form_View___CSS_FieldError',
                 'AdminPageFramework_Form_View___CSS_ToolTip',
             );
-            foreach( $_aClassNames as $_sClassName ) {
+            foreach ( $_aClassNames as $_sClassName ) {
                 $_oCSS = new $_sClassName;
-                $this->aResources[ 'internal_styles' ][] = $_oCSS->get();
+                $this->aResources[ 'internal_styles' ][ ] = $_oCSS->get();
             }
             $_aClassNamesForIE = array(
                 'AdminPageFramework_Form_View___CSS_CollapsibleSectionIE',
             );
-            foreach( $_aClassNames as $_sClassName ) {
+            foreach ( $_aClassNames as $_sClassName ) {
                 $_oCSS = new $_sClassName;
-                $this->aResources[ 'internal_styles_ie' ][] = $_oCSS->get();
+                $this->aResources[ 'internal_styles_ie' ][ ] = $_oCSS->get();
             }
             
         }
@@ -120,10 +120,10 @@ 
         protected function _set( ) {
 
             // Parse all added fieldsets and check associated resources.
-            foreach( $this->aFieldsets as $_sSecitonID => $_aFieldsets ) {
+            foreach ( $this->aFieldsets as $_sSecitonID => $_aFieldsets ) {
                 
                 $_bIsSubSectionLoaded = false;
-                foreach( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField )  {
+                foreach ( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField ) {
 // @todo Examine if this structure is correct or not. 
 // It may not be necessary to check the sub-section dimensions as this is not the saved options array.
                     // if it's a sub-section
@@ -134,7 +134,7 @@ 
                             continue;
                         }
                         $_bIsSubSectionLoaded = true;
-                        foreach( $_aSubSectionOrField as $_aField ) {
+                        foreach ( $_aSubSectionOrField as $_aField ) {
                             $this->_setFieldResources( $_aField );     
                         }
                         continue;
@@ -159,13 +159,13 @@ 
             private function _setFieldResources( array $aFieldset ) {
                                 
                 // Check the field conditions here.
-                if ( ! $this->_isFieldsetAllowed( $aFieldset ) ) {
+                if ( !$this->_isFieldsetAllowed( $aFieldset ) ) {
                     return;
                 }               
                 
                 // 3.8.0+ Support nested fields.
                 if ( $this->hasFieldDefinitionsInContent( $aFieldset ) ) {
-                    foreach( $aFieldset[ 'content' ] as $_asNestedFieldset ) {
+                    foreach ( $aFieldset[ 'content' ] as $_asNestedFieldset ) {
                         if ( is_scalar( $_asNestedFieldset ) ) {
                             continue;
                         }
@@ -202,7 +202,7 @@ 
                 $this->callBack(
                     $this->aCallbacks[ 'load_fieldset_resource' ], 
                     array(
-                        $aFieldset,   // 1st parameter 
+                        $aFieldset, // 1st parameter 
                     )
                 );
                 
@@ -235,10 +235,10 @@ 
                  * @return      boolean
                  */
                 private function _isAlreadyRegistered( $sFieldtype, $sStructureType ) {
-                    if ( isset( self::$_aRegisteredFieldTypes[ $sFieldtype . '_' .$sStructureType ] ) ) {
+                    if ( isset( self::$_aRegisteredFieldTypes[ $sFieldtype.'_'.$sStructureType ] ) ) {
                         return true;
                     }
-                    self::$_aRegisteredFieldTypes[ $sFieldtype . '_' .$sStructureType ] = true;
+                    self::$_aRegisteredFieldTypes[ $sFieldtype.'_'.$sStructureType ] = true;
                     return false;
                 }
                     /**
@@ -257,7 +257,7 @@ 
             return $this->callBack(
                 $this->aCallbacks[ 'is_fieldset_registration_allowed' ], 
                 array(
-                    true,   // 1st parameter 
+                    true, // 1st parameter 
                     $aFieldset, // 2nd parameter
                 )
             );

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

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

Spacing +4 added lines, -4 removed lines

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

development/factory/_common/form/field_type/AdminPageFramework_FieldType_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
                         ),
                     )