michaeluno / admin-page-framework

development/factory/_common/form/_view/fieldset/AdminPageFramework_Form_View___Fieldset_Base.php

Doc Comments +4 added lines, -1 removed lines

@@ -173,6 +173,7 @@ 
      * Returns the repeatable fields script.
      * 
      * @since 2.1.3
+     * @param string $sFieldsContainerID
      */
     protected function _getRepeaterFieldEnablerScript( $sFieldsContainerID, $iFieldCount, $aSettings ) {
         
@@ -261,6 +262,7 @@ 
             /**
              * @since       3.8.13
              * @param       array       $aArguments
+             * @param string $sSmallButtonSelector
              * @return      string
              */
             private function ___getAddButtonAttribtes( $aArguments, $sFieldsContainerID, $sSmallButtonSelector ) {
@@ -279,7 +281,7 @@ 
             }
             /**
              * @since       3.8.13
-             * @param       array       $aArguments
+             * @param string $sSmallButtonSelector
              * @return      string
              */
             private function ___getRemoveButtonAttributes( $sFieldsContainerID, $sSmallButtonSelector, $iFieldCount ) {
@@ -323,6 +325,7 @@ 
      * Returns the sortable fields script.
      * 
      * @since 3.0.0
+     * @param string $sFieldsContainerID
      */    
     protected function _getSortableFieldEnablerScript( $sFieldsContainerID ) {        
     

Indentation +3 added lines, -3 removed lines

@@ -272,8 +272,8 @@ 
                     'href'      => empty( $aArguments[ 'disabled' ] )
                         ? null
                         : '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                          . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                          . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
+                            . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
+                            . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
                 );
                 return $this->getAttributes( $_sPlusButtonAttributes );
             }
@@ -285,7 +285,7 @@ 
             private function ___getRemoveButtonAttributes( $sFieldsContainerID, $sSmallButtonSelector, $iFieldCount ) {
                 $_aMinusButtonAttributes = array(
                     'class'     => 'repeatable-field-remove-button button-secondary repeatable-field-button button'
-                         . $sSmallButtonSelector,
+                            . $sSmallButtonSelector,
                     'title'     => $this->oMsg->get( 'remove' ),
                     'style'     => $iFieldCount <= 1
                         ? 'visibility: hidden'

Spacing +25 added lines, -25 removed lines

@@ -67,7 +67,7 @@ 
      * @param       object      $oMsg                   An object storing the system messages.
      * @param       array       $aCallbacks             An array storing the form-field specific callbacks.     
      */
-    public function __construct( $aFieldset, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks=array() ) {
+    public function __construct( $aFieldset, $aOptions, $aErrors, &$aFieldTypeDefinitions, &$oMsg, array $aCallbacks = array() ) {
 
         // Set up the properties that will be accessed later in the methods.
         $this->aFieldset                = $this->_getFormatted( $aFieldset, $aFieldTypeDefinitions );
@@ -76,13 +76,13 @@ 
         $this->aErrors                  = $this->getAsArray( $aErrors );
         $this->oMsg                     = $oMsg;
         $this->aCallbacks               = $aCallbacks + array(
-            'hfID'              => null,    // the input id attribute
-            'hfTagID'           => null,    // the fieldset/field row container id attribute
-            'hfName'            => null,    // the input name attribute
-            'hfNameFlat'        => null,    // the flat input name attribute                
+            'hfID'              => null, // the input id attribute
+            'hfTagID'           => null, // the fieldset/field row container id attribute
+            'hfName'            => null, // the input name attribute
+            'hfNameFlat'        => null, // the flat input name attribute                
             'hfInputName'       => null,
             'hfInputNameFlat'   => null,
-            'hfClass'           => null,    // the class attribute
+            'hfClass'           => null, // the class attribute
         );        
         
         // 2. Load necessary JavaScript scripts.
@@ -149,9 +149,9 @@ 
          * @since   3.2.0   Added the `$sFieldsType` parameter.
          * @internal
          */
-        private function _loadScripts( $sStructureType='' ) {
+        private function _loadScripts( $sStructureType = '' ) {
 
-            if ( 'widget' === $sStructureType && ! self::$_bIsLoadedJSScripts_Widget ) {
+            if ( 'widget' === $sStructureType && !self::$_bIsLoadedJSScripts_Widget ) {
                 new AdminPageFramework_Form_View___Script_Widget;
                 self::$_bIsLoadedJSScripts_Widget = true;
             }
@@ -176,8 +176,8 @@ 
      */
     protected function _getRepeaterFieldEnablerScript( $sFieldsContainerID, $iFieldCount, $aSettings ) {
         
-        $_sSmallButtons         = '"' . $this->___getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, true ) . '"';
-        $_sNestedFieldsButtons  = '"' . $this->___getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, false ) . '"';
+        $_sSmallButtons         = '"'.$this->___getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, true ).'"';
+        $_sNestedFieldsButtons  = '"'.$this->___getRepeatableButtonHTML( $sFieldsContainerID, ( array ) $aSettings, $iFieldCount, false ).'"';
         $_aJSArray              = json_encode( $aSettings );
         $_sScript               = <<<JAVASCRIPTS
 jQuery( document ).ready( function() {
@@ -223,36 +223,36 @@ 
          * @since       3.8.0
          * @return      string
          */
-        private function ___getRepeatableButtonHTML( $sFieldsContainerID, array $aArguments, $iFieldCount, $bSmall=true ) {
+        private function ___getRepeatableButtonHTML( $sFieldsContainerID, array $aArguments, $iFieldCount, $bSmall = true ) {
 
             // @todo Move this formatting routine to the field-set formatter class.
             $_oFormatter             = new AdminPageFramework_Form_Model___Format_RepeatableField( $aArguments, $this->oMsg );
             $_aArguments             = $_oFormatter->get();
             $_sSmallButtonSelector   = $bSmall ? ' button-small' : '';
             if ( version_compare( $GLOBALS[ 'wp_version' ], '5.3', '>=' ) ) {
-                return "<div " . $this->___getContainerAttributes( $_aArguments ) . " >"
-                        . "<a " . $this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ) . ">"
+                return "<div ".$this->___getContainerAttributes( $_aArguments )." >"
+                        . "<a ".$this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ).">"
                             . "<span class='dashicons dashicons-minus'></span>"
                         . "</a>"
-                        . "<a " . $this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ) . ">"
+                        . "<a ".$this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ).">"
                             . "<span class='dashicons dashicons-plus-alt2'></span>"
                         ."</a>"
                     . "</div>"
                     . $this->getModalForDisabledRepeatableElement(
-                        'repeatable_field_disabled_' . $sFieldsContainerID,
+                        'repeatable_field_disabled_'.$sFieldsContainerID,
                         $_aArguments[ 'disabled' ]
                     );
             }
-            return "<div " . $this->___getContainerAttributes( $_aArguments ) . " >"
-                    . "<a " . $this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ) . ">"
+            return "<div ".$this->___getContainerAttributes( $_aArguments )." >"
+                    . "<a ".$this->___getRemoveButtonAttributes( $sFieldsContainerID, $_sSmallButtonSelector, $iFieldCount ).">"
                         . "-"
                     . "</a>"
-                    . "<a " . $this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ) . ">"
+                    . "<a ".$this->___getAddButtonAttribtes( $_aArguments, $sFieldsContainerID, $_sSmallButtonSelector ).">"
                         . "+"
                     ."</a>"
                 . "</div>"
                 . $this->getModalForDisabledRepeatableElement(
-                    'repeatable_field_disabled_' . $sFieldsContainerID,
+                    'repeatable_field_disabled_'.$sFieldsContainerID,
                     $_aArguments[ 'disabled' ]
                 );
 
@@ -271,9 +271,9 @@ 
                     'data-id'   => $sFieldsContainerID,
                     'href'      => empty( $aArguments[ 'disabled' ] )
                         ? null
-                        : '#TB_inline?width=' . $aArguments[ 'disabled' ][ 'box_width' ]
-                          . '&height=' . $aArguments[ 'disabled' ][ 'box_height' ]
-                          . '&inlineId=' . 'repeatable_field_disabled_' . $sFieldsContainerID,
+                        : '#TB_inline?width='.$aArguments[ 'disabled' ][ 'box_width' ]
+                          . '&height='.$aArguments[ 'disabled' ][ 'box_height' ]
+                          . '&inlineId='.'repeatable_field_disabled_'.$sFieldsContainerID,
                 );
                 return $this->getAttributes( $_sPlusButtonAttributes );
             }
@@ -300,10 +300,10 @@ 
              * @return      string
              */
             private function ___getContainerAttributes( $aArguments ) {
-                $_aContainerAttributes   = array(
+                $_aContainerAttributes = array(
                     'class' => $this->getClassAttribute(
                         'admin-page-framework-repeatable-field-buttons',
-                        ! empty( $aArguments[ 'disabled' ] )
+                        !empty( $aArguments[ 'disabled' ] )
                             ? 'disabled'
                             : ''
                     ),
@@ -316,7 +316,7 @@ 
                     unset( $aArguments[ 'disabled' ] );
                 }
                 return $this->getAttributes( $_aContainerAttributes )
-                    . ' ' . $this->getDataAttributes( $aArguments );
+                    . ' '.$this->getDataAttributes( $aArguments );
             }
 
     /**

development/factory/_common/utility/debug/AdminPageFramework_Debug_Base.php

Doc Comments +2 added lines

@@ -146,6 +146,7 @@ 
             /**
              * @since       3.8.9
              * @since       3.8.22  Renamed from `_getLegibleValue()`.
+             * @param integer $iStringLengthLimit
              * @return      string
              */
             static private function ___getLegibleDetailedValue( $mItem, $iStringLengthLimit ) {
@@ -314,6 +315,7 @@ 
     /**
      * Performs `array_map()` recursively.
      * @remark      Accepts arguments.
+     * @param string[] $oCallable
      * @return      array
      * @since       3.8.9
      */

Indentation +1 added lines, -1 removed lines

@@ -266,7 +266,7 @@
      * @since  3.8.22
      */
     static public function getSlicedByDepth( array $aSubject, $iDepth=0, $sMore='(array truncated) ...' ) {
-       return self::_getSlicedByDepth( $aSubject, $iDepth, $sMore );
+        return self::_getSlicedByDepth( $aSubject, $iDepth, $sMore );
     }
 
     /**

Spacing +37 added lines, -37 removed lines

@@ -34,10 +34,10 @@ 
      * @since       3.8.9
      * @return      string
      */
-    static protected function _getLegibleDetails( $mValue, $iStringLengthLimit=0, $iArrayDepthLimit=0 ) {
+    static protected function _getLegibleDetails( $mValue, $iStringLengthLimit = 0, $iArrayDepthLimit = 0 ) {
         if ( is_array( $mValue ) ) {
-            return '(array, length: ' . count( $mValue ).') '
-                . print_r( self::___getLegibleDetailedArray( $mValue, $iStringLengthLimit, $iArrayDepthLimit ) , true );
+            return '(array, length: '.count( $mValue ).') '
+                . print_r( self::___getLegibleDetailedArray( $mValue, $iStringLengthLimit, $iArrayDepthLimit ), true );
         }
         return print_r( self::___getLegibleDetailedValue( $mValue, $iStringLengthLimit ), true );
     }
@@ -49,7 +49,7 @@ 
      * @since       3.8.22  Added the `$sStringLengthLimit` and `$iArrayDepthLimit` parameters.
      * @return      string
      */
-    static protected function _getLegible( $mValue, $iStringLengthLimit=0, $iArrayDepthLimit=0 ) {
+    static protected function _getLegible( $mValue, $iStringLengthLimit = 0, $iArrayDepthLimit = 0 ) {
 
         $iArrayDepthLimit = $iArrayDepthLimit ? $iArrayDepthLimit : self::$iLegibleArrayDepthLimit;
         $mValue           = is_object( $mValue )
@@ -66,7 +66,7 @@ 
             )
             : $mValue;
         $mValue = is_string( $mValue )
-            ? self::___getLegibleString( $mValue,  $iStringLengthLimit, false )
+            ? self::___getLegibleString( $mValue, $iStringLengthLimit, false )
             : $mValue;
         return self::_getArrayRepresentationSanitized( print_r( $mValue, true ) );
 
@@ -80,7 +80,7 @@ 
          */
         static private function ___getObjectName( $mItem ) {
             if ( is_object( $mItem ) ) {
-                return '(object) ' . get_class( $mItem );
+                return '(object) '.get_class( $mItem );
             }
             return $mItem;
         }
@@ -91,7 +91,7 @@ 
          * @return      string
          */
         static private function ___getLegibleDetailedCallable( $asoCallable ) {
-            return '(callable) ' . self::___getCallableName( $asoCallable );
+            return '(callable) '.self::___getCallableName( $asoCallable );
         }
             /**
              * @since       3.8.9
@@ -109,7 +109,7 @@ 
                 $_sSubject = is_object( $asoCallable[ 0 ] )
                     ? get_class( $asoCallable[ 0 ] )
                     : ( string ) $asoCallable[ 0 ];
-                return $_sSubject . '::' . ( string ) $asoCallable[ 1 ];
+                return $_sSubject.'::'.( string ) $asoCallable[ 1 ];
 
             }
 
@@ -123,7 +123,7 @@ 
             if ( method_exists( $oObject, '__toString' ) ) {
                 return ( string ) $oObject;
             }
-            return '(object) ' . get_class( $oObject ) . ' ' . count( get_object_vars( $oObject ) ) . ' properties.';
+            return '(object) '.get_class( $oObject ).' '.count( get_object_vars( $oObject ) ).' properties.';
 
         }
         /**
@@ -135,7 +135,7 @@ 
          * @since       3.8.22  Renamed from `_getLegibleArray()`.
          * @return      array
          */
-        static private function ___getLegibleDetailedArray( array $aArray, $iStringLengthLimit=0, $iDepthLimit=0 ) {
+        static private function ___getLegibleDetailedArray( array $aArray, $iStringLengthLimit = 0, $iDepthLimit = 0 ) {
             $_iDepthLimit = $iDepthLimit ? $iDepthLimit : self::$iLegibleArrayDepthLimit;
             return self::___getArrayMappedRecursive(
                 self::_getSlicedByDepth( $aArray, $_iDepthLimit ),
@@ -171,9 +171,9 @@ 
                         return self::___getLegibleDetailedObject( $mNonScalar );
                     }
                     if ( is_array( $mNonScalar ) ) {
-                        return '(' . $_sType . ') ' . count( $mNonScalar ) . ' elements';
+                        return '('.$_sType.') '.count( $mNonScalar ).' elements';
                     }
-                    return '(' . $_sType . ') ' . ( string ) $mNonScalar;
+                    return '('.$_sType.') '.( string ) $mNonScalar;
 
                 }
                 /**
@@ -185,11 +185,11 @@ 
                  */
                 static private function ___getLegibleDetailedScalar( $sScalar, $iStringLengthLimit ) {
                     if ( is_bool( $sScalar ) ) {
-                        return '(boolean) ' . ( $sScalar ? 'true' : 'false' );
+                        return '(boolean) '.( $sScalar ? 'true' : 'false' );
                     }
                     return is_string( $sScalar )
                         ? self::___getLegibleString( $sScalar, $iStringLengthLimit, true )
-                        : '(' . gettype( $sScalar ) . ', length: ' . self::___getValueLength( $sScalar ) .  ') ' . $sScalar;
+                        : '('.gettype( $sScalar ).', length: '.self::___getValueLength( $sScalar ).') '.$sScalar;
                 }
                     /**
                      * Returns a length of a value.
@@ -213,7 +213,7 @@ 
                      * @param       boolean     $bShowDetails
                      * @return      string
                      */
-                    static private function ___getLegibleString( $sString, $iLengthLimit, $bShowDetails=true ) {
+                    static private function ___getLegibleString( $sString, $iLengthLimit, $bShowDetails = true ) {
 
                         static $_iMBSupport;
                         $_iMBSupport    = isset( $_iMBSupport ) ? $_iMBSupport : ( integer ) function_exists( 'mb_strlen' );
@@ -224,8 +224,8 @@ 
 
                         if ( $bShowDetails ) {
                             return $_iCharLength <= $iCharLimit
-                                ? '(string, length: ' . $_iCharLength . ') ' . $sString
-                                : '(string, length: ' . $_iCharLength . ') ' . call_user_func_array( $_aSubstrMethod[ $_iMBSupport ], array( $sString, 0, $iCharLimit ) )
+                                ? '(string, length: '.$_iCharLength.') '.$sString
+                                : '(string, length: '.$_iCharLength.') '.call_user_func_array( $_aSubstrMethod[ $_iMBSupport ], array( $sString, 0, $iCharLimit ) )
                                     . '...';
                         }
                         return $_iCharLength <= $iCharLimit
@@ -265,7 +265,7 @@ 
      * @return array
      * @since  3.8.22
      */
-    static public function getSlicedByDepth( array $aSubject, $iDepth=0, $sMore='(array truncated) ...' ) {
+    static public function getSlicedByDepth( array $aSubject, $iDepth = 0, $sMore = '(array truncated) ...' ) {
        return self::_getSlicedByDepth( $aSubject, $iDepth, $sMore );
     }
 
@@ -285,7 +285,7 @@ 
      * @since       3.8.22      Added the `$sMore` parameter.
      * @internal
      */
-    static private function _getSlicedByDepth( array $aSubject, $iDepth=0, $sMore='(array truncated) ...' ) {
+    static private function _getSlicedByDepth( array $aSubject, $iDepth = 0, $sMore = '(array truncated) ...' ) {
 
         foreach ( $aSubject as $_sKey => $_vValue ) {
 
@@ -317,7 +317,7 @@ 
      * @return      array
      * @since       3.8.9
      */
-    static private function ___getArrayMappedRecursive( array $aArray, $oCallable, array $aArguments=array() ) {
+    static private function ___getArrayMappedRecursive( array $aArray, $oCallable, array $aArguments = array() ) {
 
         self::$___oCurrentCallableForArrayMapRecursive = $oCallable;
         self::$___aArgumentsForArrayMapRecursive       = $aArguments;
@@ -350,29 +350,29 @@ 
      * @return  string
      * @since   3.8.22
      */
-    static public function getStackTrace( Exception $oException, $iSkip=0 ) {
+    static public function getStackTrace( Exception $oException, $iSkip = 0 ) {
 
         $_aTraces    = array();
         $_aFrames    = $oException->getTrace();
         $_aFrames    = array_slice( $_aFrames, $iSkip );
         foreach ( array_reverse( $_aFrames ) as $_iIndex => $_aFrame ) {
 
-            $_aFrame     = $_aFrame + array(
+            $_aFrame = $_aFrame + array(
                 'file'  => null, 'line' => null, 'function' => null,
                 'class' => null, 'args' => array(),
             );
             $_sArguments = self::___getArgumentsOfEachStackTrace( $_aFrame[ 'args' ] );
-            $_aTraces[]  = sprintf(
+            $_aTraces[ ]  = sprintf(
                 "#%s %s(%s): %s(%s)",
                 $_iIndex + 1,
                 $_aFrame[ 'file' ],
                 $_aFrame[ 'line' ],
-                isset( $_aFrame[ 'class' ] ) ? $_aFrame[ 'class' ] . '->' . $_aFrame[ 'function' ] : $_aFrame[ 'function' ],
+                isset( $_aFrame[ 'class' ] ) ? $_aFrame[ 'class' ].'->'.$_aFrame[ 'function' ] : $_aFrame[ 'function' ],
                 $_sArguments
             );
 
         }
-        return implode( PHP_EOL, $_aTraces ) . PHP_EOL;
+        return implode( PHP_EOL, $_aTraces ).PHP_EOL;
 
     }
         /**
@@ -387,16 +387,16 @@ 
             foreach ( $aTraceArguments as $_mArgument ) {
                 $_sType        = gettype( $_mArgument );
                 $_sType        = str_replace(
-                    array( 'resource (closed)', 'unknown type', 'integer', 'double', ),
-                    array( 'resource', 'unknown', 'scalar', 'scalar', ),
+                    array( 'resource (closed)', 'unknown type', 'integer', 'double',),
+                    array( 'resource', 'unknown', 'scalar', 'scalar',),
                     $_sType
                 );
                 $_sMethodName  = "___getStackTraceArgument_{$_sType}";
-                $_aArguments[] = method_exists( __CLASS__, $_sMethodName )
+                $_aArguments[ ] = method_exists( __CLASS__, $_sMethodName )
                     ? self::{$_sMethodName}( $_mArgument )
                     : $_sType;
             }
-            return join(", ",  $_aArguments );
+            return join( ", ", $_aArguments );
         }
             /**
              * @since 3.8.22
@@ -405,7 +405,7 @@ 
              * @return string
              */
             static private function ___getStackTraceArgument_string( $mArgument ) {
-                return "'" . $mArgument . "'";
+                return "'".$mArgument."'";
             }
             static private function ___getStackTraceArgument_scalar( $mArgument ) {
                 return $mArgument;
@@ -417,7 +417,7 @@ 
                 return 'NULL';
             }
             static private function ___getStackTraceArgument_object( $mArgument ) {
-                return 'Object(' . get_class( $mArgument ) . ')';
+                return 'Object('.get_class( $mArgument ).')';
             }
             static private function ___getStackTraceArgument_resource( $mArgument ) {
                 return get_resource_type( $mArgument );
@@ -430,18 +430,18 @@ 
                 $_iMax    = 10;
                 $_iTotal  = count( $mArgument );
                 $_iIndex  = 0;
-                foreach( $mArgument as $_sKey => $_mValue ) {
+                foreach ( $mArgument as $_sKey => $_mValue ) {
                     $_iIndex++;
-                    $_mValue   = is_scalar( $_mValue )
+                    $_mValue = is_scalar( $_mValue )
                         ? $_mValue
-                        : ucfirst( gettype( $_mValue ) ) . (
+                        : ucfirst( gettype( $_mValue ) ).(
                             is_object( $_mValue )
-                                ? ' (' . get_class( $_mValue ) . ')'
+                                ? ' ('.get_class( $_mValue ).')'
                                 : ''
                         );
-                    $_sOutput .= $_sKey . ': ' . $_mValue . ',';
+                    $_sOutput .= $_sKey.': '.$_mValue.',';
                     if ( $_iIndex > $_iMax && $_iTotal > $_iMax ) {
-                        $_sOutput  = rtrim( $_sOutput, ','  ) . '...';
+                        $_sOutput = rtrim( $_sOutput, ',' ).'...';
                         break;
                     }
                 }

development/factory/admin_page/_model/delegate/AdminPageFramework_Model__FormSubmission.php

Spacing +14 added lines, -14 removed lines

@@ -28,7 +28,7 @@ 
      */
     public function __construct( $oFactory, $aSavedData, $aArguments, $aSectionsets, $aFieldsets ) {
        
-        $this->oFactory         = $oFactory;        
+        $this->oFactory = $oFactory;        
                 
         // add_action
         // @deprecated      3.7.0
@@ -71,7 +71,7 @@ 
      */    
     public function _handleFormData() {
         
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return;
         }
 
@@ -126,7 +126,7 @@ 
         new AdminPageFramework_Model__FormSubmission__Validator( $this->oFactory );
         
         // [3.6.3+] Apply filters. All the sub-routines of handling form submit data use this filter hook.
-        $_aInputs    = $this->addAndApplyFilters(    
+        $_aInputs = $this->addAndApplyFilters(    
             $this->oFactory, 
             "validation_pre_{$this->oFactory->oProp->sClassName}", 
             $_aInputs,
@@ -137,7 +137,7 @@ 
 
         // Save the data.
         $_bUpdated = false;
-        if ( ! $this->oFactory->oProp->_bDisableSavingOptions ) {  
+        if ( !$this->oFactory->oProp->_bDisableSavingOptions ) {  
             $_bUpdated = $this->oFactory->oProp->updateOption( $_aInputs );
         }
 
@@ -176,7 +176,7 @@ 
         private function _shouldProceed() {
             
             if ( 
-                ! isset( 
+                !isset( 
                     $_POST[ 'admin_page_framework_start' ], // indicates the framework form is started
                     $_POST[ '_wp_http_referer' ]
                 ) 
@@ -193,7 +193,7 @@ 
             
             // Check if all the form fields are sent. 
             if (
-                ! isset(
+                !isset(
                     // these keys are supposed to be embedded at the end of the form.
                     // if the server truncates the form input values for `max_input_vars`, these will be lost in PHP 5.3.9 or above.
                     $_POST[ '_is_admin_page_framework' ], // holds the form nonce
@@ -213,11 +213,11 @@ 
                 return false;
             }
 
-            $_bVerifyNonce       = wp_verify_nonce(
+            $_bVerifyNonce = wp_verify_nonce(
                 $_POST[ '_is_admin_page_framework' ],
-                'form_' . md5( $this->oFactory->oProp->sClassName . get_current_user_id() )
+                'form_'.md5( $this->oFactory->oProp->sClassName.get_current_user_id() )
             );
-            if ( ! $_bVerifyNonce ) {
+            if ( !$_bVerifyNonce ) {
                 $this->oFactory->setAdminNotice( $this->oFactory->oMsg->get( 'nonce_verification_failed' ) );
                 return false;
             }
@@ -231,7 +231,7 @@ 
          */
         private function _getUserInputsFromPOST() {
 
-            $_aInputs     = $this->getElementAsArray( 
+            $_aInputs = $this->getElementAsArray( 
                 $_POST, 
                 $this->oFactory->oProp->sOptionKey, 
                 array() 
@@ -266,7 +266,7 @@ 
                 $this->oFactory,
                 array( 
                     // @todo deprecate the hook with the input ID
-                    "submit_{$this->oFactory->oProp->sClassName}_{$_sPressedInputID}",  // will be deprecated in near future release
+                    "submit_{$this->oFactory->oProp->sClassName}_{$_sPressedInputID}", // will be deprecated in near future release
                     $_sSubmitSectionID 
                         ? "submit_{$this->oFactory->oProp->sClassName}_{$_sSubmitSectionID}_{$_sPressedFieldID}" 
                         : "submit_{$this->oFactory->oProp->sClassName}_{$_sPressedFieldID}",
@@ -349,9 +349,9 @@ 
             
             // Drop the 'field_errors' key.
             $_aRemoveQueries = array();
-            if ( ! isset( $aStatus[ 'field_errors' ] ) || ! $aStatus[ 'field_errors' ] ) {
+            if ( !isset( $aStatus[ 'field_errors' ] ) || !$aStatus[ 'field_errors' ] ) {
                 unset( $aStatus[ 'field_errors' ] );
-                $_aRemoveQueries[] = 'field_errors';
+                $_aRemoveQueries[ ] = 'field_errors';
             }        
          
             return $this->addAndApplyFilters(    // 3.4.4+
@@ -376,7 +376,7 @@ 
          */
         private function _removePageElements( $aOptions, $sPageSlug, $sTabSlug ) {
             
-            if ( ! $sPageSlug && ! $sTabSlug ) {
+            if ( !$sPageSlug && !$sTabSlug ) {
                 return $aOptions;
             }
             

Braces +4 added lines, -3 removed lines

@@ -137,7 +137,7 @@ 
 
         // Save the data.
         $_bUpdated = false;
-        if ( ! $this->oFactory->oProp->_bDisableSavingOptions ) {  
+        if ( ! $this->oFactory->oProp->_bDisableSavingOptions ) {
             $_bUpdated = $this->oFactory->oProp->updateOption( $_aInputs );
         }
 
@@ -180,14 +180,15 @@ 
                     $_POST[ 'admin_page_framework_start' ], // indicates the framework form is started
                     $_POST[ '_wp_http_referer' ]
                 ) 
-            ) {     
+            ) {
                 return false;
             }
             
             // Referrer
             $_sRequestURI   = remove_query_arg( array( 'settings-updated', 'confirmation', 'field_errors' ), wp_unslash( $_SERVER[ 'REQUEST_URI' ] ) );
             $_sRefererURI   = remove_query_arg( array( 'settings-updated', 'confirmation', 'field_errors' ), $_POST[ '_wp_http_referer' ] );
-            if ( $_sRequestURI != $_sRefererURI ) { // see the function definition of wp_referer_field() in functions.php.
+            if ( $_sRequestURI != $_sRefererURI ) {
+// see the function definition of wp_referer_field() in functions.php.
                 return false;
             }
             

development/factory/admin_page/_model/delegate/AdminPageFramework_Model_Menu__RegisterMenu.php

Spacing +51 added lines, -51 removed lines

@@ -27,7 +27,7 @@ 
      *
      * @internal
      */
-    public function __construct( $oFactory, $sActionHook='admin_menu' ) {
+    public function __construct( $oFactory, $sActionHook = 'admin_menu' ) {
 
         $this->oFactory = $oFactory;
         add_action(
@@ -74,7 +74,7 @@ 
     public function _replyToRegisterRootMenu() {
 
         // If the root menu label is not set but the slug is set,
-        if ( ! $this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
+        if ( !$this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
             return;
         }
         $this->_registerRootMenuPage();
@@ -96,12 +96,12 @@ 
             }
 
             $this->oFactory->oProp->aRootMenu[ '_page_hook' ] = add_menu_page(
-                $this->oFactory->oProp->sClassName,                 // Page title - will be invisible anyway
-                $this->oFactory->oProp->aRootMenu[ 'sTitle' ],      // Menu title - should be the root page title.
-                $this->oFactory->oProp->sCapability,                // Capability - access right
-                $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ],   // Menu ID
-                '',                                       // Callback function for the page content output - the root page will be removed so no need to register a function.
-                $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ],  // icon path
+                $this->oFactory->oProp->sClassName, // Page title - will be invisible anyway
+                $this->oFactory->oProp->aRootMenu[ 'sTitle' ], // Menu title - should be the root page title.
+                $this->oFactory->oProp->sCapability, // Capability - access right
+                $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ], // Menu ID
+                '', // Callback function for the page content output - the root page will be removed so no need to register a function.
+                $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ], // icon path
                 $this->getElement(
                     $this->oFactory->oProp->aRootMenu,
                     'iPosition',
@@ -123,8 +123,8 @@ 
 
         // Let external scripts add sub-menu pages.
         $_aPages = $this->addAndApplyFilter(
-            $this->oFactory,    // caller object
-            "pages_{$this->oFactory->oProp->sClassName}",   // filter
+            $this->oFactory, // caller object
+            "pages_{$this->oFactory->oProp->sClassName}", // filter
             $this->oFactory->oProp->aPages  // arguments
         );
 
@@ -134,7 +134,7 @@ 
         // Format the `$aPages` property and register the pages.
         $_iParsedIndex    = 0;
         $_aFormattedPages = array();
-        foreach( $_aPages as $_aSubMenuItem ) {
+        foreach ( $_aPages as $_aSubMenuItem ) {
 
             // needs to be sanitized because there are hook filters applied to this array.
             $_oFormatter = new AdminPageFramework_Format_SubMenuItem(
@@ -169,8 +169,8 @@ 
          */
         private function _getDefaultPageSlug( array $aPages ) {
 
-            foreach( $aPages as $_aPage ) {
-                if ( ! isset( $_aPage[ 'page_slug' ] ) ) {
+            foreach ( $aPages as $_aPage ) {
+                if ( !isset( $_aPage[ 'page_slug' ] ) ) {
                     continue;
                 }
                 return $_aPage[ 'page_slug' ];
@@ -192,7 +192,7 @@ 
          */
         private function _registerSubMenuItem( array $aArgs ) {
 
-            if ( ! current_user_can( $aArgs[ 'capability' ] ) ) {
+            if ( !current_user_can( $aArgs[ 'capability' ] ) ) {
                 return '';
             }
 
@@ -256,14 +256,14 @@ 
              */
             private function _addPageSubmenuItem( $sRootPageSlug, $sMenuSlug, $sPageSlug, $sPageTitle, $sMenuTitle, $sCapability, $bShowInMenu, $nOrder ) {
 
-                if ( ! $sPageSlug ) {
+                if ( !$sPageSlug ) {
                     return '';
                 }
                 $_sPageHook = $this->___addSubMenuPage(
-                    $sRootPageSlug,         // the root (parent) page slug
-                    $sPageTitle,            // page title
-                    $sMenuTitle,            // menu title
-                    $sCapability,           // capability
+                    $sRootPageSlug, // the root (parent) page slug
+                    $sPageTitle, // page title
+                    $sMenuTitle, // menu title
+                    $sCapability, // capability
                     $sPageSlug              // menu slug
                 );
 
@@ -291,14 +291,14 @@ 
                 $_aRemovedMenuItem = $this->_removePageSubmenuItem( $_nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle );
 
                 // If the visibility option is `false`, remove the one just added from the sub-menu global array
-                if ( ! $bShowInMenu && ! $this->_isCurrentPage( $sPageSlug ) ) {
+                if ( !$bShowInMenu && !$this->_isCurrentPage( $sPageSlug ) ) {
                     return $_sPageHook;
                 }
 
                 // Set the order index in the element of the `submenu` global array.
                 $this->_setSubMenuPageByIndex(
-                    $nOrder,                // user-set order
-                    $_aRemovedMenuItem,     // will be reassign with a new index
+                    $nOrder, // user-set order
+                    $_aRemovedMenuItem, // will be reassign with a new index
                     $sMenuSlug
                 );
 
@@ -319,21 +319,21 @@ 
                  * @uses        add_submenu_page
                  */
                 private function ___addSubMenuPage(
-                    $sRootPageSlug,         // the root (parent) page slug
-                    $sPageTitle,            // page title
-                    $sMenuTitle,            // menu title
-                    $sCapability,           // capability
+                    $sRootPageSlug, // the root (parent) page slug
+                    $sPageTitle, // page title
+                    $sMenuTitle, // menu title
+                    $sCapability, // capability
                     $sPageSlug              // menu slug
                 ) {
                     if ( $this->oFactory->oProp->bIsAdminAjax ) {
                         return $sPageSlug;
                     }
                     return add_submenu_page(
-                        $sRootPageSlug,         // the root (parent) page slug
-                        $sPageTitle,            // page title
-                        $sMenuTitle,            // menu title
-                        $sCapability,           // capability
-                        $sPageSlug,             // menu slug
+                        $sRootPageSlug, // the root (parent) page slug
+                        $sPageTitle, // page title
+                        $sMenuTitle, // menu title
+                        $sCapability, // capability
+                        $sPageSlug, // menu slug
                         array( $this->oFactory, '_replyToRenderPage' )  // callback 3.7.10+
                     );
                 }
@@ -367,7 +367,7 @@ 
                      */
                     add_action(
                         'current_screen',
-                        array( $this->oFactory, "load_pre_" . $sPageSlug ),
+                        array( $this->oFactory, "load_pre_".$sPageSlug ),
                         20
                     );
                     /**
@@ -382,15 +382,15 @@ 
                      * Set a low priority because the user may add in-page tabs in their callback method of this action hook.
                      * @since       3.6.3
                      */
-                    add_action( "load_" . $sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 );
+                    add_action( "load_".$sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 );
 
                     // 3.6.3+
-                    add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) );
-                    add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) );  // 3.7.10+
+                    add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) );
+                    add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) ); // 3.7.10+
 
                     $this->oFactory->oProp->aPageHooks[ $sPageSlug ] = $this->getAOrB(
                         is_network_admin(),
-                        $sPageHook . '-network',
+                        $sPageHook.'-network',
                         $sPageHook
                     );
 
@@ -407,7 +407,7 @@ 
 
                     $_nNewIndex = $this->getUnusedNumericIndex(
                         $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ), // subject array to parser
-                        $nOrder,    // a desired menu position
+                        $nOrder, // a desired menu position
                         5           // offset
                     );
 
@@ -421,9 +421,9 @@ 
                  */
                 private function _getSubMenuPageIndex( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug ) {
 
-                    foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
+                    foreach ( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
 
-                        if ( ! isset( $_aSubMenu[ 3 ] ) ) {
+                        if ( !isset( $_aSubMenu[ 3 ] ) ) {
                             continue;
                         }
 
@@ -455,7 +455,7 @@ 
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`. Chagned the return type.
                  * @return      array       removed menu item.
                  */
-                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){
+                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) {
 
                     $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex(
                         $nSubMenuPageIndex,
@@ -528,7 +528,7 @@ 
              */
             private function _addLinkSubmenuItem( $sMenuSlug, $sTitle, $sCapability, $sHref, $bShowInMenu, $nOrder ) {
 
-                if ( ! $bShowInMenu ) {
+                if ( !$bShowInMenu ) {
                     return;
                 }
 
@@ -543,9 +543,9 @@ 
                     5   // offset
                 );
                 $_aSubMenuItems[ $_nIndex ] = array(
-                    $sTitle,        // 0
-                    $sCapability,   // 1
-                    $sHref,         // 2
+                    $sTitle, // 0
+                    $sCapability, // 1
+                    $sHref, // 2
                 );
                 $GLOBALS[ 'submenu' ][ $sMenuSlug ] = $_aSubMenuItems;
 
@@ -562,17 +562,17 @@ 
      */
     public function _replyToLoadPageForAjax() {
         $_sCurrentPageSlug = $this->oFactory->oProp->getCurrentPageSlugIfAdded();
-        if ( ! $_sCurrentPageSlug ) {
+        if ( !$_sCurrentPageSlug ) {
             return;
         }
-        $_sCurrentTabSlug  = $this->oFactory->oProp->getCurrentInPageTabSlugIfAdded( $_sCurrentPageSlug );
+        $_sCurrentTabSlug = $this->oFactory->oProp->getCurrentInPageTabSlugIfAdded( $_sCurrentPageSlug );
         if (
-            ! empty( $this->oFactory->oProp->aInPageTabs )      // it means at least one in-page tab is added
-            && ! $_sCurrentTabSlug // the current tab could not be retrieved
+            !empty( $this->oFactory->oProp->aInPageTabs )      // it means at least one in-page tab is added
+            && !$_sCurrentTabSlug // the current tab could not be retrieved
         ) {
             return;
         }
-        call_user_func_array( array( $this->oFactory, "load_pre_" . $_sCurrentPageSlug ), array() );
+        call_user_func_array( array( $this->oFactory, "load_pre_".$_sCurrentPageSlug ), array() );
     }
 
     /**
@@ -585,7 +585,7 @@ 
     public function _replyToRemoveRootMenuPage() {
 
         // After adding the sub menus, if the root menu is created, remove the page that is automatically created when registering the root menu.
-        if ( ! $this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
+        if ( !$this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
             return;
         }
         if ( $this->oFactory->oProp->bIsAdminAjax ) {
@@ -596,7 +596,7 @@ 
          * Namespace slugs have backslashes but the registered slugs backslashes are all converted to forward-slashes.
          * @since       3.5.16
          */
-        $_sMenuSlug =             str_replace(
+        $_sMenuSlug = str_replace(
             '\\',
             '/',
             $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ]

development/factory/admin_page/_view/AdminPageFramework_View__PageRenderer.php

Spacing +19 added lines, -19 removed lines

@@ -47,7 +47,7 @@ 
 
         // Do actions before rendering the page. In this order, global -> page -> in-page tab
         $this->addAndDoActions(
-            $this->oFactory,  // the caller object
+            $this->oFactory, // the caller object
             $this->getFilterArrayByPrefix( 'do_before_', $this->oFactory->oProp->sClassName, $_sPageSlug, $_sTabSlug, true ), // the action hooks
             $this->oFactory   // the argument 1
         );
@@ -72,7 +72,7 @@ 
                     </div><!-- #post-body -->
                 </div><!-- #poststuff -->
 
-            <?php echo $this->_printFormClosingTag( $_sPageSlug, $_sTabSlug, $this->oFactory->oProp->bEnableForm );  // </form> ?>
+            <?php echo $this->_printFormClosingTag( $_sPageSlug, $_sTabSlug, $this->oFactory->oProp->bEnableForm ); // </form> ?>
             </div><!-- .admin-page-framework-container -->
 
             <?php
@@ -83,7 +83,7 @@ 
         <?php
         // Do actions after rendering the page.
         $this->addAndDoActions(
-            $this->oFactory,  // the caller object
+            $this->oFactory, // the caller object
             $this->getFilterArrayByPrefix( 'do_after_', $this->oFactory->oProp->sClassName, $_sPageSlug, $_sTabSlug, true ), // the action hooks
             $this->oFactory   // the argument 1
         );
@@ -100,14 +100,14 @@ 
          */
         private function _getNumberOfColumns() {
 
-            if ( ! $this->doesMetaBoxExist( 'side' ) ) {
+            if ( !$this->doesMetaBoxExist( 'side' ) ) {
                 return 1;
             }
 
             $_iColumns = $this->getNumberOfScreenColumns();
             return $_iColumns
                 ? $_iColumns
-                : 1;    // default - this is because generic pages do not have meta boxes.
+                : 1; // default - this is because generic pages do not have meta boxes.
         }
             // @deprecated
             // Make sure if no side meta box exists, set it 1.
@@ -126,7 +126,7 @@ 
         private function _getContentTop() {
 
             // Screen icon, page heading tabs(page title), and in-page tabs.
-            $_oScreenIcon       = new AdminPageFramework_View__PageRenderer__ScreenIcon(
+            $_oScreenIcon = new AdminPageFramework_View__PageRenderer__ScreenIcon(
                 $this->oFactory,
                 $this->sPageSlug,
                 $this->sTabSlug
@@ -142,7 +142,7 @@ 
                 $this->oFactory,
                 $this->sPageSlug
             );
-            $_sContentTop      .= $_oInPageTabs->get();
+            $_sContentTop .= $_oInPageTabs->get();
 
             // Apply filters in this order, in-page tab -> page -> global.
             return $this->addAndApplyFilters(
@@ -214,7 +214,7 @@ 
              */
             private function _getFormOutput( $sPageSlug ) {
 
-                if ( ! $this->oFactory->oProp->bEnableForm ) {
+                if ( !$this->oFactory->oProp->bEnableForm ) {
                     return '';
                 }
                 return $this->oFactory->oForm->get();
@@ -244,9 +244,9 @@ 
          * @internal
          * @return      void
          */
-        private function _printFormOpeningTag( $fEnableForm=true ) {
+        private function _printFormOpeningTag( $fEnableForm = true ) {
 
-            if ( ! $fEnableForm ) {
+            if ( !$fEnableForm ) {
                 return;
             }
 
@@ -259,12 +259,12 @@ 
                             'action'    => wp_unslash( remove_query_arg( 'settings-updated', $this->oFactory->oProp->sTargetFormPage ) ),
                         )
                     )
-                . " >" . PHP_EOL;
+                . " >".PHP_EOL;
 
             // [3.5.11+] Insert a mark that indicates the framework form has started.
             // This will be checked in the validation method with the `is_admin_page_framework` input value which gets inserted at the end of the form
             // in order to determine all the fields are sent for the PHP max_input_vars limitation set in the server configuration.
-            echo "<input type='hidden' name='admin_page_framework_start' value='1' />" . PHP_EOL;
+            echo "<input type='hidden' name='admin_page_framework_start' value='1' />".PHP_EOL;
 
             // Embed the '_wp_http_referer' hidden field that is checked in the submit data processing method.
             settings_fields( $this->oFactory->oProp->sOptionKey );
@@ -280,16 +280,16 @@ 
          * @internal
          * @return      void
          */
-        private function _printFormClosingTag( $sPageSlug, $sTabSlug, $fEnableForm=true ) {
+        private function _printFormClosingTag( $sPageSlug, $sTabSlug, $fEnableForm = true ) {
 
-            if ( ! $fEnableForm ) {
+            if ( !$fEnableForm ) {
                 return;
             }
-            $_sNonce = wp_create_nonce( 'form_' . md5( $this->oFactory->oProp->sClassName . get_current_user_id() ) );
-            echo "<input type='hidden' name='page_slug' value='{$sPageSlug}' />" . PHP_EOL
-                . "<input type='hidden' name='tab_slug' value='{$sTabSlug}' />" . PHP_EOL
-                . "<input type='hidden' name='_is_admin_page_framework' value='{$_sNonce}' />" . PHP_EOL
-                . "</form><!-- End Form -->" . PHP_EOL;
+            $_sNonce = wp_create_nonce( 'form_'.md5( $this->oFactory->oProp->sClassName.get_current_user_id() ) );
+            echo "<input type='hidden' name='page_slug' value='{$sPageSlug}' />".PHP_EOL
+                . "<input type='hidden' name='tab_slug' value='{$sTabSlug}' />".PHP_EOL
+                . "<input type='hidden' name='_is_admin_page_framework' value='{$_sNonce}' />".PHP_EOL
+                . "</form><!-- End Form -->".PHP_EOL;
 
         }
 

development/factory/admin_page/_view/AdminPageFramework_View__Resource.php

Spacing +13 added lines, -13 removed lines

@@ -67,9 +67,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.
@@ -83,23 +83,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' )
             );
@@ -116,10 +116,10 @@ 
              * @since       3.6.3
              * @return      void
              */
-            private function _enqueuePageAssets( array $aPageAssets, $sType='style' ) {
-                $_sMethodName = "_enqueueAsset_" . $sType;
-                foreach( $aPageAssets as $_asPageAsset ) {
-                    $this->{$_sMethodName}( $_asPageAsset);
+            private function _enqueuePageAssets( array $aPageAssets, $sType = 'style' ) {
+                $_sMethodName = "_enqueueAsset_".$sType;
+                foreach ( $aPageAssets as $_asPageAsset ) {
+                    $this->{$_sMethodName}( $_asPageAsset );
                 }
             }
 
@@ -144,7 +144,7 @@ 
                     }
 
                     // Insert the CSS rule in the head tag.
-                    $this->aCSSRules[] = $_sSRC;
+                    $this->aCSSRules[ ] = $_sSRC;
 
                 }
 
@@ -169,7 +169,7 @@ 
                     }
 
                     // Insert the scripts in the head tag.
-                    $this->aScripts[] = $_sSRC;
+                    $this->aScripts[ ] = $_sSRC;
 
                 }
 

development/factory/admin_page/AdminPageFramework_Router.php

Spacing +21 added lines, -21 removed lines

@@ -26,11 +26,11 @@ 
      *
      * @since       3.3.0
      */
-    public function __construct( $sOptionKey=null, $sCallerPath=null, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sOptionKey = null, $sCallerPath = null, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         $_sPropertyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
 
         $this->oProp = new $_sPropertyClassName(
             $this,
@@ -47,12 +47,12 @@ 
 //        if ( $this->oProp->bIsAdminAjax ) {
 //            return;
 //        }
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return;
         }
 
         add_action( 'wp_loaded', array( $this, '_replyToDetermineToLoad' ) );
-        add_action( 'set_up_' . $this->oProp->sClassName, array( $this, '_replyToLoadComponentsForAjax' ), 100 );
+        add_action( 'set_up_'.$this->oProp->sClassName, array( $this, '_replyToLoadComponentsForAjax' ), 100 );
 
     }
 
@@ -64,13 +64,13 @@ 
      * @callback        add_action      set_up_{extended class name}
      */
     public function _replyToLoadComponentsForAjax() {
-        if ( ! $this->oProp->bIsAdminAjax ) {
+        if ( !$this->oProp->bIsAdminAjax ) {
             return;
         }
 
         new AdminPageFramework_Model_Menu__RegisterMenu( $this, 'pseudo_admin_menu' );
         do_action( 'pseudo_admin_menu', '' );
-        do_action( 'pseudo_current_screen' );       // @deprecated 3.8.22 Kept for backward-compatibility
+        do_action( 'pseudo_current_screen' ); // @deprecated 3.8.22 Kept for backward-compatibility
 
         $_sPageSlug = $this->oProp->getCurrentPageSlug();
         if ( $this->oProp->isPageAdded( $_sPageSlug ) ) {
@@ -117,7 +117,7 @@ 
      * @since       3.3.1       Moved from `AdminPageFramework_Base`.
      * @internal
      */
-    public function __call( $sMethodName, $aArgs=null ) {
+    public function __call( $sMethodName, $aArgs = null ) {
 
         $_sPageSlug             = $this->oProp->getCurrentPageSlug();
         $_sTabSlug              = $this->oProp->getCurrentTabSlug( $_sPageSlug );
@@ -128,15 +128,15 @@ 
             'load_pre_',
         );
 
-        switch( $this->_getCallbackName( $sMethodName, $_sPageSlug, $_aKnownMethodPrefixes ) ) {
+        switch ( $this->_getCallbackName( $sMethodName, $_sPageSlug, $_aKnownMethodPrefixes ) ) {
 
             // add_settings_section() callback
             case 'section_pre_':
-                return $this->_renderSectionDescription( $sMethodName );    // defined in AdminPageFramework_Setting
+                return $this->_renderSectionDescription( $sMethodName ); // defined in AdminPageFramework_Setting
 
             // add_settings_field() callback
             case 'field_pre_':
-                return $this->_renderSettingField( $_mFirstArg, $_sPageSlug );  // defined in AdminPageFramework_Setting
+                return $this->_renderSettingField( $_mFirstArg, $_sPageSlug ); // defined in AdminPageFramework_Setting
 
             // load-{page} callback
             case 'load_pre_':
@@ -154,9 +154,9 @@ 
          * @return      string      The found callback method name or the prefix of a known callback method name. An empty string if not found.
          * @internal
          */
-        private function _getCallbackName( $sMethodName, $sPageSlug, array $aKnownMethodPrefixes=array() ) {
+        private function _getCallbackName( $sMethodName, $sPageSlug, array $aKnownMethodPrefixes = array() ) {
 
-            foreach( $aKnownMethodPrefixes as $_sMethodPrefix ) {
+            foreach ( $aKnownMethodPrefixes as $_sMethodPrefix ) {
                 if ( $this->oUtil->hasPrefix( $_sMethodPrefix, $sMethodName ) ) {
                     return $_sMethodPrefix;
                 }
@@ -180,7 +180,7 @@ 
          */
         protected function _doPageLoadCall( $sMethodName, $sPageSlug, $sTabSlug, $oScreen ) {
 
-            if ( ! $this->_isPageLoadCall( $sMethodName, $sPageSlug, $oScreen ) ) {
+            if ( !$this->_isPageLoadCall( $sMethodName, $sPageSlug, $oScreen ) ) {
                 return;
             }
 
@@ -204,10 +204,10 @@ 
             // the tab slug will be empty unless an in-page tab is added.
             $sTabSlug = $this->oProp->getCurrentTabSlug( $sPageSlug );
             if ( strlen( $sTabSlug ) ) {
-                $this->_setShowDebugInfoProperty( $sPageSlug, $sTabSlug );  // 3.8.8+
+                $this->_setShowDebugInfoProperty( $sPageSlug, $sTabSlug ); // 3.8.8+
                 $this->oUtil->addAndDoActions(
                     $this, // the caller object
-                    array( "load_{$sPageSlug}_" . $sTabSlug ),
+                    array( "load_{$sPageSlug}_".$sTabSlug ),
                     $this // the admin page object - this lets third-party scripts use the framework methods.
                 );
             }
@@ -232,10 +232,10 @@ 
              * @since       3.8.8
              * @return      void
              */
-            private function _setShowDebugInfoProperty( $sPageSlug, $sTabSlug='' ) {
+            private function _setShowDebugInfoProperty( $sPageSlug, $sTabSlug = '' ) {
 
                 // For the page,
-                if ( ! strlen( $sTabSlug ) ) {
+                if ( !strlen( $sTabSlug ) ) {
                     $this->oProp->bShowDebugInfo = $this->oUtil->getElement(
                         $this->oProp->aPages,
                         array( $sPageSlug, 'show_debug_info' ),
@@ -280,7 +280,7 @@ 
                 if ( substr( $sMethodName, strlen( 'load_pre_' ) ) !== $sPageSlug ) {
                     return false;
                 }
-                if ( ! isset( $this->oProp->aPageHooks[ $sPageSlug ] ) ) {
+                if ( !isset( $this->oProp->aPageHooks[ $sPageSlug ] ) ) {
                     return false;
                 }
                 $_sPageHook = is_object( $osScreenORPageHook )
@@ -306,7 +306,7 @@ 
         }
 
         // Nothing to do in the network admin area.
-        return ! is_network_admin();
+        return !is_network_admin();
 
     }
 
@@ -321,13 +321,13 @@ 
      */
     protected function _isInThePage() {
 
-        if ( ! $this->oProp->bIsAdmin ) {
+        if ( !$this->oProp->bIsAdmin ) {
             return false;
         }
 
         // If the `setUp()` method is not loaded yet, nothing can be checked
         // as there is not page is added.
-        if ( ! did_action( 'set_up_' . $this->oProp->sClassName ) ) {
+        if ( !did_action( 'set_up_'.$this->oProp->sClassName ) ) {
             return true;
         }
 

development/factory/_common/_abstract/_controller/AdminPageFramework_Resource_Base.php

Indentation +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
     /*
      * Shared methods
      */
-    	/**
+        /**
          * Checks the src url of the enqueued script/style to determine whether or not to set up a attribute modification callback.
          *
          * If it is one of the framework added item, the method sets up a hook to modify the url to add custom attributes.

Spacing +42 added lines, -42 removed lines

@@ -36,19 +36,19 @@ 
 
         /* The system internal keys. */
         'sSRC'          => null,
-        'aPostTypes'    => array(),     // for meta box class
+        'aPostTypes'    => array(), // for meta box class
         'sPageSlug'     => null,
         'sTabSlug'      => null,
-        'sType'         => null,        // script or style
+        'sType'         => null, // script or style
 
         /* The below keys are for users. */
         'handle_id'     => null,
         'dependencies'  => array(),
-        'version'       => false,       // although the type should be string, the wp_enqueue_...() functions want false as the default value.
-        'translation'   => array(),     // only for scripts
-        'in_footer'     => false,       // only for scripts
-        'media'         => 'all',       // only for styles
-        'attributes'    => array(),     // 3.3.0+ - the attribute array
+        'version'       => false, // although the type should be string, the wp_enqueue_...() functions want false as the default value.
+        'translation'   => array(), // only for scripts
+        'in_footer'     => false, // only for scripts
+        'media'         => 'all', // only for styles
+        'attributes'    => array(), // 3.3.0+ - the attribute array
 
     );
 
@@ -59,7 +59,7 @@ 
      * @remark      This value should be overridden in an extended class.
      * @internal
      */
-    protected $_sClassSelector_Style    = 'admin-page-framework-style';
+    protected $_sClassSelector_Style = 'admin-page-framework-style';
 
     /**
      * Stores the class selector used to the class-specific script.
@@ -68,7 +68,7 @@ 
      * @remark      This value should be overridden in an extended class.
      * @internal
      */
-    protected $_sClassSelector_Script   = 'admin-page-framework-script';
+    protected $_sClassSelector_Script = 'admin-page-framework-script';
 
     /**
      * Stores hand IDs by resource url to look up handle id and add custom arguments.
@@ -143,8 +143,8 @@ 
      * @internal
      */
 
-    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs=array() ) {}
-    public function _forceToEnqueueScript( $sSRC, $aCustomArgs=array() ) {}
+    public function _forceToEnqueueStyle( $sSRC, $aCustomArgs = array() ) {}
+    public function _forceToEnqueueScript( $sSRC, $aCustomArgs = array() ) {}
 
     /**
      * A helper function for the _replyToEnqueueScripts() and the `_replyToEnqueueStyle()` methods.
@@ -204,8 +204,8 @@ 
                         return $sSanitizedURL;
                     }
 
-                    $_sAttributes   = $this->getAttributes( $_aAttributes );
-                    return $sSanitizedURL . "' " . rtrim( $_sAttributes, "'\"" );
+                    $_sAttributes = $this->getAttributes( $_aAttributes );
+                    return $sSanitizedURL."' ".rtrim( $_sAttributes, "'\"" );
 
                 }
 
@@ -225,7 +225,7 @@ 
      */
     protected function _printCommonStyles( $sIDPrefix, $sClassName ) {
 
-        if ( $this->hasBeenCalled( 'COMMON_STYLES: ' . get_class( $this ) . '::' . __METHOD__ ) ) {
+        if ( $this->hasBeenCalled( 'COMMON_STYLES: '.get_class( $this ).'::'.__METHOD__ ) ) {
             return;
         }
         $_oCaller = $this->oProp->oCaller;
@@ -241,10 +241,10 @@ 
          */
         private function ___getCommonStyleTag( $oCaller, $sIDPrefix ) {
 
-            $_sStyle     = $this->addAndApplyFilters(
+            $_sStyle = $this->addAndApplyFilters(
                 $oCaller,
                 array(
-                    "style_common_admin_page_framework",            // 3.2.1+
+                    "style_common_admin_page_framework", // 3.2.1+
                     "style_common_{$this->oProp->sClassName}",
                 ),
                 AdminPageFramework_CSS::getDefaultCSS()
@@ -252,7 +252,7 @@ 
             $_sStyle     = $this->isDebugMode() ? $_sStyle : $this->getCSSMinified( $_sStyle );
             $_sStyle     = trim( $_sStyle );
             if ( $_sStyle ) {
-                echo "<style type='text/css' id='" . esc_attr( strtolower( $sIDPrefix ) ) . "'>"
+                echo "<style type='text/css' id='".esc_attr( strtolower( $sIDPrefix ) )."'>"
                         . $_sStyle
                     . "</style>";
             }
@@ -267,10 +267,10 @@ 
          */
         private function ___getCommonIEStyleTag( $oCaller, $sIDPrefix ) {
 
-            $_sStyleIE   = $this->addAndApplyFilters(
+            $_sStyleIE = $this->addAndApplyFilters(
                 $oCaller,
                 array(
-                    "style_ie_common_admin_page_framework",         // 3.2.1+
+                    "style_ie_common_admin_page_framework", // 3.2.1+
                     "style_ie_common_{$this->oProp->sClassName}",
                 ),
                 AdminPageFramework_CSS::getDefaultCSSIE()
@@ -278,7 +278,7 @@ 
             $_sStyleIE  = $this->isDebugMode() ? $_sStyleIE : $this->getCSSMinified( $_sStyleIE );
             $_sStyleIE  = trim( $_sStyleIE );
             return $_sStyleIE
-                ? "<!--[if IE]><style type='text/css' id='" . esc_attr( strtolower( $sIDPrefix . "-ie" ) ) . "'>"
+                ? "<!--[if IE]><style type='text/css' id='".esc_attr( strtolower( $sIDPrefix."-ie" ) )."'>"
                         . $_sStyleIE
                     . "</style><![endif]-->"
                 : '';
@@ -297,23 +297,23 @@ 
      */
     protected function _printCommonScripts( $sIDPrefix, $sClassName ) {
 
-        if ( $this->hasBeenCalled( 'COMMON_SCRIPT: ' . get_class( $this ) . '::' . __METHOD__ ) ) {
+        if ( $this->hasBeenCalled( 'COMMON_SCRIPT: '.get_class( $this ).'::'.__METHOD__ ) ) {
             return;
         }
 
         $_sScript = $this->addAndApplyFilters(
             $this->oProp->oCaller,
             array(
-                "script_common_admin_page_framework",       // 3.2.1+
+                "script_common_admin_page_framework", // 3.2.1+
                 "script_common_{$this->oProp->sClassName}",
             ),
             AdminPageFramework_Property_Base::$_sDefaultScript
         );
         $_sScript = trim( $_sScript );
-        if ( ! $_sScript ) {
+        if ( !$_sScript ) {
             return;
         }
-        echo "<script type='text/javascript' id='" . esc_attr( strtolower( $sIDPrefix ) ) . "'>"
+        echo "<script type='text/javascript' id='".esc_attr( strtolower( $sIDPrefix ) )."'>"
                 . '/* <![CDATA[ */'
                 . $_sScript
                 . '/* ]]> */'
@@ -331,7 +331,7 @@ 
      */
     protected function _printClassSpecificStyles( $sIDPrefix ) {
 
-        $_oCaller   = $this->oProp->oCaller;
+        $_oCaller = $this->oProp->oCaller;
         echo $this->_getClassSpecificStyleTag( $_oCaller, $sIDPrefix );
         echo $this->_getClassSpecificIEStyleTag( $_oCaller, $sIDPrefix );
 
@@ -352,18 +352,18 @@ 
             static $_iCallCount = 0;
 
             $_sFilterName = "style_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: '.$_sFilterName ) ) { // 3.8.22
                 return '';
             }
             $_sStyle = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyle );
             $_sStyle = $this->isDebugMode() ? $_sStyle : $this->getCSSMinified( $_sStyle );
             $_sStyle = trim( $_sStyle );
-            if ( ! $_sStyle ) {
+            if ( !$_sStyle ) {
                 return '';
             }
             $_iCallCount++;
-            $_sID = strtolower( "{$sIDPrefix}-" . $this->oProp->sClassName . "_{$_iCallCount}" );
-            return "<style type='text/css' id='" . esc_attr( $_sID ) . "'>"
+            $_sID = strtolower( "{$sIDPrefix}-".$this->oProp->sClassName."_{$_iCallCount}" );
+            return "<style type='text/css' id='".esc_attr( $_sID )."'>"
                     . $_sStyle
                 . "</style>";
 
@@ -379,18 +379,18 @@ 
             static $_iCallCountIE = 1;
 
             $_sFilterName = "style_ie_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: '.$_sFilterName ) ) { // 3.8.22
                 return '';
             }
             $_sStyleIE = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyleIE );
             $_sStyleIE = $this->isDebugMode() ? $_sStyleIE : $this->getCSSMinified( $_sStyleIE );
             $_sStyleIE = trim( $_sStyleIE );
-            if ( ! $_sStyleIE ) {
+            if ( !$_sStyleIE ) {
                 return '';
             }
             $_iCallCountIE++;
-            $_sID  = strtolower( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" );
-            return "<!--[if IE]><style type='text/css' id='" . esc_attr( $_sID ) . "'>"
+            $_sID = strtolower( "{$sIDPrefix}-ie-{$this->oProp->sClassName}_{$_iCallCountIE}" );
+            return "<!--[if IE]><style type='text/css' id='".esc_attr( $_sID )."'>"
                     . $_sStyleIE
                 . "</style><![endif]-->";
 
@@ -407,18 +407,18 @@ 
 
         static $_iCallCount = 1;
         $_sFilterName = "script_{$this->oProp->sClassName}";
-        if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+        if ( $this->hasBeenCalled( 'FILTER: '.$_sFilterName ) ) { // 3.8.22
             return '';
         }
         $_sScript = $this->addAndApplyFilters( $this->oProp->oCaller, $_sFilterName, $this->oProp->sScript );
         $_sScript = trim( $_sScript );
-        if ( ! $_sScript ) {
+        if ( !$_sScript ) {
             return '';
         }
 
         $_iCallCount++;
         $_sID = strtolower( "{$sIDPrefix}-{$this->oProp->sClassName}_{$_iCallCount}" );
-        echo "<script type='text/javascript' id='" . esc_attr( $_sID ) . "'>"
+        echo "<script type='text/javascript' id='".esc_attr( $_sID )."'>"
                 . '/* <![CDATA[ */'
                 . $_sScript
                 . '/* ]]> */'
@@ -442,12 +442,12 @@ 
     public function _replyToAddStyle() {
 
         $_oCaller = $this->oProp->oCaller;
-        if ( ! $_oCaller->isInThePage() ) {
+        if ( !$_oCaller->isInThePage() ) {
             return;
         }
 
         $this->_printCommonStyles( 'admin-page-framework-style-common', get_class() );
-        $this->_printClassSpecificStyles( $this->_sClassSelector_Style . '-' . $this->oProp->sStructureType );
+        $this->_printClassSpecificStyles( $this->_sClassSelector_Style.'-'.$this->oProp->sStructureType );
 
     }
     /**
@@ -462,12 +462,12 @@ 
     public function _replyToAddScript() {
 
         $_oCaller = $this->oProp->oCaller;
-        if ( ! $_oCaller->isInThePage() ) {
+        if ( !$_oCaller->isInThePage() ) {
             return;
         }
 
         $this->_printCommonScripts( 'admin-page-framework-script-common', get_class() );
-        $this->_printClassSpecificScripts( $this->_sClassSelector_Script . '-' . $this->oProp->sStructureType );
+        $this->_printClassSpecificScripts( $this->_sClassSelector_Script.'-'.$this->oProp->sStructureType );
 
     }
 
@@ -519,7 +519,7 @@ 
      * @internal
      */
     public function _replyToEnqueueStyles() {
-        foreach( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
+        foreach ( $this->oProp->aEnqueuingStyles as $_sKey => $_aEnqueuingStyle ) {
             $this->_enqueueSRCByCondition( $_aEnqueuingStyle );
             unset( $this->oProp->aEnqueuingStyles[ $_sKey ] );
         }
@@ -536,7 +536,7 @@ 
      * @internal
      */
     public function _replyToEnqueueScripts() {
-        foreach( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
+        foreach ( $this->oProp->aEnqueuingScripts as $_sKey => $_aEnqueuingScript ) {
             $this->_enqueueSRCByCondition( $_aEnqueuingScript );
             unset( $this->oProp->aEnqueuingScripts[ $_sKey ] );
         }

Braces +6 added lines, -3 removed lines

@@ -352,7 +352,8 @@ 
             static $_iCallCount = 0;
 
             $_sFilterName = "style_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) {
+// 3.8.22
                 return '';
             }
             $_sStyle = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyle );
@@ -379,7 +380,8 @@ 
             static $_iCallCountIE = 1;
 
             $_sFilterName = "style_ie_{$this->oProp->sClassName}";
-            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+            if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) {
+// 3.8.22
                 return '';
             }
             $_sStyleIE = $this->addAndApplyFilters( $_oCaller, $_sFilterName, $this->oProp->sStyleIE );
@@ -407,7 +409,8 @@ 
 
         static $_iCallCount = 1;
         $_sFilterName = "script_{$this->oProp->sClassName}";
-        if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) { // 3.8.22
+        if ( $this->hasBeenCalled( 'FILTER: ' . $_sFilterName ) ) {
+// 3.8.22
             return '';
         }
         $_sScript = $this->addAndApplyFilters( $this->oProp->oCaller, $_sFilterName, $this->oProp->sScript );

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

Spacing +7 added lines, -7 removed lines

@@ -27,9 +27,9 @@ 
      * @return      void
      * @since       3.8.8
      */
-    static public function showDeprecationNotice( $sDeprecated, $sAlternative='', $sProgramName='Admin Page Framework' ) {
+    static public function showDeprecationNotice( $sDeprecated, $sAlternative = '', $sProgramName = 'Admin Page Framework' ) {
         trigger_error(
-            $sProgramName . ': ' . sprintf(
+            $sProgramName.': '.sprintf(
                 $sAlternative
                     ? '<code>%1$s</code> has been deprecated. Use <code>%2$s</code> instead.'
                     : '<code>%1$s</code> has been deprecated.',
@@ -52,8 +52,8 @@ 
      * @param       string|array        $asParameters       Parameters to pass to the callback function.
      * @return      mixed
      */
-    public function callBack( $oCallable, $asParameters=array() ) {
-        $_aParameters   = self::getAsArray(
+    public function callBack( $oCallable, $asParameters = array() ) {
+        $_aParameters = self::getAsArray(
             $asParameters,
             true // preserve empty
         );
@@ -90,7 +90,7 @@ 
      * @since       3.6.3
      * @return      string      The captured output buffer.
      */
-    static public function getOutputBuffer( $oCallable, array $aParameters=array() ) {
+    static public function getOutputBuffer( $oCallable, array $aParameters = array() ) {
 
         ob_start();
         echo call_user_func_array( $oCallable, $aParameters );
@@ -111,7 +111,7 @@ 
 
         $_iCount     = count( get_object_vars( $oInstance ) );
         $_sClassName = get_class( $oInstance );
-        return '(object) ' . $_sClassName . ': ' . $_iCount . ' properties.';
+        return '(object) '.$_sClassName.': '.$_iCount.' properties.';
 
     }
 
@@ -128,7 +128,7 @@ 
      * @param       boolean|integer|double|string|array|object|resource|NULL        $mTrue      The value to return when the first parameter value yields false.
      * @return      mixed
      */
-    static public function getAOrB( $mValue, $mTrue=null, $mFalse=null ) {
+    static public function getAOrB( $mValue, $mTrue = null, $mFalse = null ) {
         return $mValue ? $mTrue : $mFalse;
     }