michaeluno / admin-page-framework

development/factory/_common/_abstract/AdminPageFramework_Factory_Router.php

Spacing +20 added lines, -20 removed lines

@@ -169,8 +169,8 @@ 
         }
 
         // Call the user constructor.
-        $this->start();     // defined in the controller class.
-        $this->oUtil->addAndDoAction( $this, 'start_' . $this->oProp->sClassName, $this );
+        $this->start(); // defined in the controller class.
+        $this->oUtil->addAndDoAction( $this, 'start_'.$this->oProp->sClassName, $this );
 
     }
         /**
@@ -179,8 +179,8 @@ 
          * @return      array
          */
         private function ___getSubClassNames() {
-            foreach( $this->_aSubClassPrefixes as $_sObjectVariableName => $_sPrefix ) {
-                $this->aSubClassNames[ $_sObjectVariableName ] = $_sPrefix . $this->_sStructureType;
+            foreach ( $this->_aSubClassPrefixes as $_sObjectVariableName => $_sPrefix ) {
+                $this->aSubClassNames[ $_sObjectVariableName ] = $_sPrefix.$this->_sStructureType;
             }
             return $this->aSubClassNames + $this->_aSubClassNames;
         }
@@ -194,15 +194,15 @@ 
      */
     public function _replyToLoadComponents( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) {
+        if ( !$this->_isInThePage() ) {
             return;
         }
 
-        if ( ! isset( $this->oResource ) ) {
+        if ( !isset( $this->oResource ) ) {
             $this->oResource = $this->_replyTpSetAndGetInstance_oResource();
         }
 
-        if ( ! isset(  $this->oLink ) ) {
+        if ( !isset( $this->oLink ) ) {
             $this->oLink = $this->_replyTpSetAndGetInstance_oLink();
         }
 
@@ -216,10 +216,10 @@ 
      * Calls the load method and callbacks.
      * @since       3.8.14
      */
-    protected function _load( $aActions=array() ) {
+    protected function _load( $aActions = array() ) {
         $aActions = empty( $aActions )
             ? array(
-                'load_' . $this->oProp->sClassName,
+                'load_'.$this->oProp->sClassName,
             )
             : $aActions;
         $this->load();
@@ -231,7 +231,7 @@ 
      */
     protected function _setUp() {
         $aActions = array(
-            'set_up_' . $this->oProp->sClassName,
+            'set_up_'.$this->oProp->sClassName,
         );
         $this->setUp();
         $this->oUtil->addAndDoActions( $this, $aActions, $this );
@@ -258,14 +258,14 @@ 
          * @since   3.8.19
          */
         protected function _isWordPressCoreAjaxRequest() {
-            if ( ! isset( $GLOBALS[ 'pagenow' ] ) ) {
+            if ( !isset( $GLOBALS[ 'pagenow' ] ) ) {
                 return false;
             }
             if ( 'admin-ajax.php' !== $GLOBALS[ 'pagenow' ] ) {
                 return false;
             }
             return in_array(
-                isset( $_POST[ 'action' ] ) ? $_POST[ 'action' ] : '',  // sanitization unnecessary as just checking
+                isset( $_POST[ 'action' ] ) ? $_POST[ 'action' ] : '', // sanitization unnecessary as just checking
                 array( 'heartbeat', 'closed-postboxes', 'meta-box-order' ),
                 true
             );
@@ -293,7 +293,7 @@ 
      * @return  boolean
      */
     protected function _isValidAjaxReferrer() {
-        if ( ! $this->oProp->bIsAdminAjax ) {
+        if ( !$this->oProp->bIsAdminAjax ) {
             return false;
         }
         return true;
@@ -309,7 +309,7 @@ 
      */
     public function _replyToDetermineToLoad() {
 
-        if ( ! $this->_isInThePage() ) {
+        if ( !$this->_isInThePage() ) {
             return;
         }
 
@@ -375,7 +375,7 @@ 
         // Set and return the sub class object instance.
         if ( isset( $this->aSubClassNames[ $sPropertyName ] ) ) {
             return call_user_func(
-                array( $this, "_replyTpSetAndGetInstance_{$sPropertyName}"  )
+                array( $this, "_replyTpSetAndGetInstance_{$sPropertyName}" )
             );
         }
 
@@ -409,7 +409,7 @@ 
          */
         public function _replyTpSetAndGetInstance_oMsg() {
             $this->oMsg = call_user_func_array(
-                array( $this->aSubClassNames[ 'oMsg' ], 'getInstance'),
+                array( $this->aSubClassNames[ 'oMsg' ], 'getInstance' ),
                 array( $this->oProp->sTextDomain )  // parameters
             );
             return $this->oMsg;
@@ -474,7 +474,7 @@ 
      *
      * @internal
      */
-    public function __call( $sMethodName, $aArguments=null ) {
+    public function __call( $sMethodName, $aArguments = null ) {
 
         $_mFirstArg = $this->oUtil->getElement( $aArguments, 0 );
 
@@ -504,7 +504,7 @@ 
 
             // Check if the method name does not contain a backslash.
             if ( false === strpos( $sMethodName, "\\" ) ) {
-                return $this->oUtil->getElement( $aArguments, 0 );  // the first element - the filter value
+                return $this->oUtil->getElement( $aArguments, 0 ); // the first element - the filter value
             }
 
             // If the method name contains a backslash, the user may be using a name space.
@@ -515,7 +515,7 @@ 
                     array( $this, $_sAutoCallbackMethodName ),
                     $aArguments
                 )
-                : $this->oUtil->getElement( $aArguments, 0 );   // the first argument
+                : $this->oUtil->getElement( $aArguments, 0 ); // the first argument
 
         }
 
@@ -525,7 +525,7 @@ 
          */
         private function _triggerUndefinedMethodWarning( $sMethodName ) {
             trigger_error(
-                AdminPageFramework_Registry::NAME . ': '
+                AdminPageFramework_Registry::NAME.': '
                     . sprintf(
                         __( 'The method is not defined: %1$s', $this->oProp->sTextDomain ),
                         $sMethodName

development/factory/_common/_abstract/_model/AdminPageFramework_Property_Base.php

Spacing +16 added lines, -16 removed lines

@@ -320,7 +320,7 @@ 
         'caller_id'                         => '',
         'structure_type'                    => '',
         'action_hook_form_registration'     => '',
-        'autoload_min_resource'             => true,    // will be overridden by the $bAutoloadMinifiedResource property.
+        'autoload_min_resource'             => true, // will be overridden by the $bAutoloadMinifiedResource property.
     );
 
     /**
@@ -328,14 +328,14 @@ 
      * @since       3.7.0
      */
     public $aFormCallbacks = array(
-        'hfID'              => null,    // the input id attribute
-        'hfTagID'           => null,    // the field container id attribute
-        'hfName'            => null,    // the field name attribute
-        'hfNameFlat'        => null,    // the flat field name attribute
+        'hfID'              => null, // the input id attribute
+        'hfTagID'           => null, // the field container id attribute
+        'hfName'            => null, // the field name attribute
+        'hfNameFlat'        => null, // the flat field name attribute
         // @todo Document the differences between `hfName` and `hfInputName`
-        'hfInputName'       => null,    // 3.6.0+   the field input name attribute
-        'hfInputNameFlat'   => null,    // 3.6.0+   the flat field input name
-        'hfClass'           => null,    // the class attribute
+        'hfInputName'       => null, // 3.6.0+   the field input name attribute
+        'hfInputNameFlat'   => null, // 3.6.0+   the flat field input name
+        'hfClass'           => null, // the class attribute
     );
 
     /**
@@ -468,7 +468,7 @@ 
              * @internal
              */
             private function ___getURLQuery() {
-                if ( ! $this->bIsAdminAjax ) {
+                if ( !$this->bIsAdminAjax ) {
                     return $this->getHTTPQueryGET( array(), array() );
                 }
                 parse_str(
@@ -483,7 +483,7 @@ 
          * @since       3.7.9
          */
         private function ___setGlobals() {
-            if ( ! isset( $GLOBALS[ 'aAdminPageFramework' ] ) ) {
+            if ( !isset( $GLOBALS[ 'aAdminPageFramework' ] ) ) {
                 $GLOBALS[ 'aAdminPageFramework' ] = array(
                     'aFieldFlags' => array()
                 );
@@ -511,7 +511,7 @@ 
             'action_hook_form_registration'     => $this->bIsAdminAjax
                 ? 'admin_init'  // 3.8.14
                 : $this->_sFormRegistrationHook,
-            'autoload_min_resource'             => $this->bAutoloadMinifiedResource,    // [3.8.31]
+            'autoload_min_resource'             => $this->bAutoloadMinifiedResource, // [3.8.31]
         ) + $this->aFormArguments;
     }
     /**
@@ -555,8 +555,8 @@ 
             'hfTagID'                           => array( $this->oCaller, '_replyToGetInputTagIDAttribute' ), // the fields & fieldset & field row container id attribute
             'hfName'                            => array( $this->oCaller, '_replyToGetFieldNameAttribute' ), // the input name attribute
             'hfNameFlat'                        => array( $this->oCaller, '_replyToGetFlatFieldName' ), // the flat input name attribute
-            'hfInputName'                       => array( $this->oCaller, '_replyToGetInputNameAttribute' ),    // 3.6.0+   the field input name attribute
-            'hfInputNameFlat'                   => array( $this->oCaller, '_replyToGetFlatInputName' ),    // 3.6.0+   the flat field input name
+            'hfInputName'                       => array( $this->oCaller, '_replyToGetInputNameAttribute' ), // 3.6.0+   the field input name attribute
+            'hfInputNameFlat'                   => array( $this->oCaller, '_replyToGetFlatInputName' ), // 3.6.0+   the flat field input name
             'hfClass'                           => array( $this->oCaller, '_replyToGetInputClassAttribute' ), // the class attribute
             'hfSectionName'                     => array( $this->oCaller, '_replyToGetSectionName' ), // 3.6.0+
         ) + $this->aFormCallbacks;
@@ -630,7 +630,7 @@ 
      * @remark      The information can be used to embed into the footer etc.
      * @return      array       The information of the script.
      */
-    protected function getCallerInfo( $sCallerPath='' ) {
+    protected function getCallerInfo( $sCallerPath = '' ) {
 
         if ( isset( self::$_aScriptDataCaches[ $sCallerPath ] ) ) {
             return self::$_aScriptDataCaches[ $sCallerPath ];
@@ -748,11 +748,11 @@ 
 
         // 3.7.9 Moved from the constructor to make it lighter.
         if ( 'sClassHash' === $sName ) {
-            $this->sClassHash       = md5( $this->sClassName );
+            $this->sClassHash = md5( $this->sClassName );
             return $this->sClassHash;
         }
         if ( 'sScriptType' === $sName ) {
-            $this->sScriptType      = $this->_getCallerType( $this->sCallerPath );    // 3.7.6+
+            $this->sScriptType = $this->_getCallerType( $this->sCallerPath ); // 3.7.6+
             return $this->sScriptType;
         }
         if ( 'oUtil' === $sName ) {

development/factory/_common/_abstract/_model/AdminPageFramework_Message.php

Spacing +17 added lines, -17 removed lines

@@ -83,11 +83,11 @@ 
         'nonce_verification_failed'             => 'A problem occurred while processing the form data. Please try again.',
         'check_max_input_vars'                  => 'Not all form fields could not be sent. '
             . 'Please check your server settings of PHP <code>max_input_vars</code> and consult the server administrator to increase the value. '
-            . '<code>max input vars</code>: %1$s. <code>$_POST</code> count: %2$s',  // 3.5.11+ // sanitization unnecessary as it is just a literal string
-        'send_email'                            => 'Is it okay to send the email?',     // 3.3.0+
-        'email_sent'                            => 'The email has been sent.',  // 3.3.0+, 3.3.5+ deprecated, 3.8.32 Re-added
+            . '<code>max input vars</code>: %1$s. <code>$_POST</code> count: %2$s', // 3.5.11+ // sanitization unnecessary as it is just a literal string
+        'send_email'                            => 'Is it okay to send the email?', // 3.3.0+
+        'email_sent'                            => 'The email has been sent.', // 3.3.0+, 3.3.5+ deprecated, 3.8.32 Re-added
         'email_scheduled'                       => 'The email has been scheduled.', // 3.3.5+, 3.8.32 deprecated
-        'email_could_not_send'                  => 'There was a problem sending the email',     // 3.3.0+
+        'email_could_not_send'                  => 'There was a problem sending the email', // 3.3.0+
 
         // AdminPageFramework_PostType
         'title'                                 => 'Title',
@@ -114,14 +114,14 @@ 
         'upload_file'                           => 'Upload File',
         'use_this_file'                         => 'Use This File',
         'select_file'                           => 'Select File',
-        'remove_value'                          => 'Remove Value',  // 3.2.0+
-        'select_all'                            => 'Select All',    // 3.3.0+
-        'select_none'                           => 'Select None',   // 3.3.0+
+        'remove_value'                          => 'Remove Value', // 3.2.0+
+        'select_all'                            => 'Select All', // 3.3.0+
+        'select_none'                           => 'Select None', // 3.3.0+
         'no_term_found'                         => 'No term found.', // 3.3.2+
 
         // AdminPageFramework_Form_View___Script_{...}
         'select'                                => 'Select', // 3.4.2+
-        'insert'                                => 'Insert',  // 3.4.2+
+        'insert'                                => 'Insert', // 3.4.2+
         'use_this'                              => 'Use This', // 3.4.2+
         'return_to_library'                     => 'Return to Library', // 3.4.2+
 
@@ -133,8 +133,8 @@ 
 
         // Repeatable sections & fields
         'repeatable_section_is_disabled'        => 'The ability to repeat sections is disabled.', // 3.8.13+
-        'repeatable_field_is_disabled'          => 'The ability to repeat fields is disabled.',   // 3.8.13+
-        'warning_caption'                       => 'Warning',   // 3.8.13+
+        'repeatable_field_is_disabled'          => 'The ability to repeat fields is disabled.', // 3.8.13+
+        'warning_caption'                       => 'Warning', // 3.8.13+
 
         // AdminPageFramework_FormField
         'allowed_maximum_number_of_fields'      => 'The allowed maximum number of fields is {0}.',
@@ -216,7 +216,7 @@ 
      * @remark      This class should be instantiated via this method.
      * @return      AdminPageFramework_Message
      */
-    public static function getInstance( $sTextDomain='admin-page-framework' ) {
+    public static function getInstance( $sTextDomain = 'admin-page-framework' ) {
 
         $_oInstance = isset( self::$_aInstancesByTextDomain[ $sTextDomain ] ) && ( self::$_aInstancesByTextDomain[ $sTextDomain ] instanceof AdminPageFramework_Message )
             ? self::$_aInstancesByTextDomain[ $sTextDomain ]
@@ -229,7 +229,7 @@ 
          * Ensures that only one instance of this class object exists. ( no multiple instances of this object )
          * @deprecated  3.2.0
          */
-        public static function instantiate( $sTextDomain='admin-page-framework' ) {
+        public static function instantiate( $sTextDomain = 'admin-page-framework' ) {
             return self::getInstance( $sTextDomain );
         }
 
@@ -237,14 +237,14 @@ 
      * Sets up properties.
      * @param string $sTextDomain
      */
-    public function __construct( $sTextDomain='admin-page-framework' ) {
+    public function __construct( $sTextDomain = 'admin-page-framework' ) {
 
         $this->_sTextDomain = $sTextDomain;
 
         // Fill the $aMessages property with the keys extracted from the $aDefaults property
         // with the value of null.  The null is set to let it trigger the __get() method
         // so that each translation item gets processed individually.
-        $this->aMessages    = array_fill_keys(
+        $this->aMessages = array_fill_keys(
             array_keys( $this->aDefaults ),
             null
         );
@@ -279,13 +279,13 @@ 
      * @param       string       $sKey
      * @return      string|array
      */
-    public function get( $sKey='' ) {
-        if ( ! $sKey ) {
+    public function get( $sKey = '' ) {
+        if ( !$sKey ) {
             return $this->_getAllMessages();
         }
         return isset( $this->aMessages[ $sKey ] )
             ? __( $this->aMessages[ $sKey ], $this->_sTextDomain )
-            : __( $this->{$sKey}, $this->_sTextDomain );     // triggers __get()
+            : __( $this->{$sKey}, $this->_sTextDomain ); // triggers __get()
     }
         /**
          * Returns the all registered messag items.

development/factory/_common/form/notice/AdminPageFramework_Form___SubmitNotice.php

Spacing +9 added lines, -9 removed lines

@@ -40,7 +40,7 @@ 
          * @return      string
          */
         private function _getTransientKey() {
-            return 'apf_ntc_' . get_current_user_id();
+            return 'apf_ntc_'.get_current_user_id();
         }
     
     /**
@@ -52,14 +52,14 @@ 
      * @param       string      $sType If empty, the method will check if a message exists in all types. Otherwise, it checks the existence of a message of the specified type.
      * @return      boolean     True if a setting notice is set; otherwise, false.
      */
-    public function hasNotice( $sType='' ) {
+    public function hasNotice( $sType = '' ) {
                 
-        if ( ! $sType ) {
+        if ( !$sType ) {
             return ( boolean ) count( self::$_aNotices );
         }
         
         // Check if there is a message of the type.
-        foreach( self::$_aNotices as $_aNotice ) {
+        foreach ( self::$_aNotices as $_aNotice ) {
             $_sClassAttribute = $this->getElement( 
                 $_aNotice, 
                 array( 
@@ -98,7 +98,7 @@ 
      * @param        boolean     $bOverride      (optional) If true, only one message will be shown in the next page load. false: do not override when there is a message of the same id. true: override the previous one.
      * @return       void
      */
-    public function set( $sMessage, $sType='error', $asAttributes=array(), $bOverride=true ) {
+    public function set( $sMessage, $sType = 'error', $asAttributes = array(), $bOverride = true ) {
         
         // If the array is empty, schedule the task of saving the array at shutdown.
         if ( empty( self::$_aNotices ) ) {
@@ -108,7 +108,7 @@ 
         $_sID = md5( trim( $sMessage ) );
             
         // If the override is false and a message is already set, do not add.
-        if ( ! $bOverride && isset( self::$_aNotices[ $_sID ] ) ) {
+        if ( !$bOverride && isset( self::$_aNotices[ $_sID ] ) ) {
             return;
         }
 
@@ -118,14 +118,14 @@ 
         }
         
         $_aAttributes = $this->getAsArray( $asAttributes );
-        if ( is_string( $asAttributes ) && ! empty( $asAttributes ) ) {
+        if ( is_string( $asAttributes ) && !empty( $asAttributes ) ) {
             $_aAttributes[ 'id' ] = $asAttributes;
         }
         self::$_aNotices[ $_sID ] = array(
             'sMessage'      => $sMessage,
             'aAttributes'   => $_aAttributes + array(
                 'class'     => $sType,
-                'id'        => 'form_submit_notice_' . $_sID,
+                'id'        => 'form_submit_notice_'.$_sID,
             ),
         );
 
@@ -166,7 +166,7 @@ 
         }
             
         // By setting false to the 'settings-notice' key, it's possible to disable the notifications set with the framework.
-        if ( isset( $_GET[ 'settings-notice' ] ) && ! $_GET[ 'settings-notice' ] ) {    // sanitization unnecessary
+        if ( isset( $_GET[ 'settings-notice' ] ) && !$_GET[ 'settings-notice' ] ) {    // sanitization unnecessary
             return; 
         }
             

Braces +4 added lines, -3 removed lines

@@ -141,7 +141,7 @@ 
          * @return      void
          */
         public function _replyToSaveNotices() {
-            if ( empty( self::$_aNotices ) ) { 
+            if ( empty( self::$_aNotices ) ) {
                 return; 
             }            
             $_bResult = $this->setTransient( 
@@ -161,12 +161,13 @@ 
         
         // Retrieve the notifications set in a transient.
         $_aNotices = $this->_getNotices();
-        if ( false === $_aNotices ) { 
+        if ( false === $_aNotices ) {
             return; 
         }
             
         // By setting false to the 'settings-notice' key, it's possible to disable the notifications set with the framework.
-        if ( isset( $_GET[ 'settings-notice' ] ) && ! $_GET[ 'settings-notice' ] ) {    // sanitization unnecessary
+        if ( isset( $_GET[ 'settings-notice' ] ) && ! $_GET[ 'settings-notice' ] ) {
+// sanitization unnecessary
             return; 
         }
             

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

Spacing +5 added lines, -5 removed lines

@@ -56,7 +56,7 @@ 
             $_sPageSlug = $this->getHTTPQueryGET( 'page', '' );
             $_sTabSlug  = $this->getHTTPQueryGET( 'tab', '' );
             $_sUserID   = get_current_user_id();
-            return "apf_li_" . md5(
+            return "apf_li_".md5(
                 $_sPageNow
                 . $_sPageSlug
                 . $_sTabSlug
@@ -94,13 +94,13 @@ 
          * @return      void
          */
         public function _replyToSave() {
-            if ( ! isset( self::$_aLastInputs ) ) {
+            if ( !isset( self::$_aLastInputs ) ) {
                 return;
             }
             $this->setTransient(
                 $this->sTransientKey,
                 self::$_aLastInputs,
-                60*60     // store it for 1 hour
+                60 * 60     // store it for 1 hour
             );
         }
 
@@ -120,9 +120,9 @@ 
             $_aLastInputs = self::$_aCaches[ $this->sTransientKey ];
         } else {
             $_aLastInputs = $this->getTransient( $this->sTransientKey );
-            self::$_aCaches[ $this->sTransientKey ]  = $_aLastInputs;
+            self::$_aCaches[ $this->sTransientKey ] = $_aLastInputs;
             if ( false !== $_aLastInputs ) {
-                $this->delete();    // deletes at the end of the script.
+                $this->delete(); // deletes at the end of the script.
             }
         }
 

development/factory/_common/form/error/AdminPageFramework_Form___FieldError.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
             $_sPageSlug = $this->getHTTPQueryGET( 'page', '' );
             $_sTabSlug  = $this->getHTTPQueryGET( 'tab', '' );
             $_sUserID   = get_current_user_id();
-            return "apf_fe_" . md5(
+            return "apf_fe_".md5(
                 $this->getPageNow()
                 . $_sPageSlug
                 . $_sTabSlug
@@ -133,7 +133,7 @@ 
     public function get() {
 
         // Use a cache if exists.
-        self::$_aFieldErrorCaches[ $this->sTransientKey ]  = isset( self::$_aFieldErrorCaches[ $this->sTransientKey ] )
+        self::$_aFieldErrorCaches[ $this->sTransientKey ] = isset( self::$_aFieldErrorCaches[ $this->sTransientKey ] )
             ? self::$_aFieldErrorCaches[ $this->sTransientKey ]
             : $this->getTransient( $this->sTransientKey );
 
@@ -150,7 +150,7 @@ 
      * @since       3.7.0
      */
     public function delete() {
-        if ( $this->hasBeenCalled( 'delete_' . $this->sTransientKey ) ) {
+        if ( $this->hasBeenCalled( 'delete_'.$this->sTransientKey ) ) {
             return;
         }
         add_action( 'shutdown', array( $this, '_replyToDelete' ) );

development/factory/_common/form/AdminPageFramework_Form_Model.php

Braces +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
                 array( $this, '_replyToRegisterFormItems' ),
                 100 // priority - low value is set as meta boxes use the `current_screen` action hook for `setUp()`.
             );
-        } else {                
+        } else {
             add_action(
                 $this->aArguments[ 'action_hook_form_registration' ],
                 array( $this, '_replyToRegisterFormItems' )
@@ -160,7 +160,7 @@ 
      * Changed the name from `getFieldsModel()`.
      * @return      array
      */
-    public function getDataStructureFromAddedFieldsets()  {
+    public function getDataStructureFromAddedFieldsets() {
                     
         $_aFormDataStructure  = array();
         foreach ( $this->getAsArray( $this->aFieldsets ) as $_sSectionID => $_aFieldsets ) {
@@ -198,7 +198,7 @@ 
      * @param       array       $aSubject       The subject array to modify. Usually the saved option data.
      * @return      array       The modified options array.
      */
-    public function dropRepeatableElements( array $aSubject ) {        
+    public function dropRepeatableElements( array $aSubject ) {
         $_oFilterRepeatableElements = new AdminPageFramework_Form_Model___Modifier_FilterRepeatableElements( 
             $aSubject,
             $this->getHTTPRequestSanitized( $this->getElementAsArray(

Spacing +19 added lines, -19 removed lines

@@ -64,10 +64,10 @@ 
      * @return array
      * @since  3.7.0
      */
-    public function getSubmittedData( array $aDataToParse, $bExtractFromFieldStructure=true, $bStripSlashes=true ) {
+    public function getSubmittedData( array $aDataToParse, $bExtractFromFieldStructure = true, $bStripSlashes = true ) {
                 
         // Extracts the form data from the subject data for parsing
-        $_aSubmittedFormData    = $bExtractFromFieldStructure
+        $_aSubmittedFormData = $bExtractFromFieldStructure
             ? $this->castArrayContents( 
                 $this->getDataStructureFromAddedFieldsets(), // form data (options) structure
                 $aDataToParse   // the subject data array, usually `$_POST`.
@@ -95,12 +95,12 @@ 
             array_merge(
                 $this->getElementAsArray( 
                     $_POST, // sanitization done
-                    '__repeatable_elements_' . $this->aArguments[ 'structure_type' ],
+                    '__repeatable_elements_'.$this->aArguments[ 'structure_type' ],
                     array()
                 ),
                 $this->getElementAsArray( 
                     $_POST, // sanitization done
-                    '__sortable_elements_' . $this->aArguments[ 'structure_type' ],
+                    '__sortable_elements_'.$this->aArguments[ 'structure_type' ],
                     array()
                 )
             )
@@ -160,9 +160,9 @@ 
      * Changed the name from `getFieldsModel()`.
      * @return      array
      */
-    public function getDataStructureFromAddedFieldsets()  {
+    public function getDataStructureFromAddedFieldsets() {
                     
-        $_aFormDataStructure  = array();
+        $_aFormDataStructure = array();
         foreach ( $this->getAsArray( $this->aFieldsets ) as $_sSectionID => $_aFieldsets ) {
 
             if ( $_sSectionID !== '_default' ) {
@@ -171,7 +171,7 @@ 
             }
             
             // For default field items.
-            foreach( $_aFieldsets as $_sFieldID => $_aFieldset ) {
+            foreach ( $_aFieldsets as $_sFieldID => $_aFieldset ) {
                 $_aFormDataStructure[ $_aFieldset[ 'field_id' ] ] = $_aFieldset;
             }
 
@@ -203,7 +203,7 @@ 
             $aSubject,
             $this->getHTTPRequestSanitized( $this->getElementAsArray(
                 $_POST, // sanitization done
-                '__repeatable_elements_' . $this->aArguments[ 'structure_type' ]
+                '__repeatable_elements_'.$this->aArguments[ 'structure_type' ]
             ) )
         );
         return $_oFilterRepeatableElements->get();
@@ -216,13 +216,13 @@ 
     public function _replyToRegisterFormItems() {
 
         // Check if the form should be created or not.
-        if ( ! $this->isInThePage() ) {
+        if ( !$this->isInThePage() ) {
             return;
         }
 
         // Load field type definitions.
-        $this->_setFieldTypeDefinitions( 'admin_page_framework' );    // site-wide
-        $this->_setFieldTypeDefinitions( $this->aArguments[ 'caller_id' ] );   // per class
+        $this->_setFieldTypeDefinitions( 'admin_page_framework' ); // site-wide
+        $this->_setFieldTypeDefinitions( $this->aArguments[ 'caller_id' ] ); // per class
         
         // Set the options array
         $this->aSavedData = $this->_getSavedData(
@@ -250,7 +250,7 @@ 
             $this->aArguments,
             $this->aFieldsets,
             self::$_aResources,
-            $this->aFieldTypeDefinitions,   // must be called after performing `_setFieldTypeDefinitions()`.
+            $this->aFieldTypeDefinitions, // must be called after performing `_setFieldTypeDefinitions()`.
             $this->aCallbacks,
             $this->oMsg
         );
@@ -268,10 +268,10 @@ 
         $this->callBack(
             $this->aCallbacks[ 'handle_form_data' ],
             array(
-                $this->aSavedData,      // 1st parameter
-                $this->aArguments,      // 2nd parameter
-                $this->aSectionsets,    // 3rd parameter
-                $this->aFieldsets,      // 4th parameter
+                $this->aSavedData, // 1st parameter
+                $this->aArguments, // 2nd parameter
+                $this->aSectionsets, // 3rd parameter
+                $this->aFieldsets, // 4th parameter
             )
         );        
 
@@ -285,15 +285,15 @@ 
             $this->aSectionsets = $this->callBack(
                 $this->aCallbacks[ 'sectionsets_before_registration' ],
                 array(
-                    $this->aSectionsets,    // 1st parameter
+                    $this->aSectionsets, // 1st parameter
                 )
             );
             // Let the main routine modify the fieldsets definition array.
             $this->aFieldsets = $this->callBack(
                 $this->aCallbacks[ 'fieldsets_before_registration' ],
                 array(
-                    $this->aFieldsets,    // 1st parameter
-                    $this->aSectionsets,  // 2nd parameter
+                    $this->aFieldsets, // 1st parameter
+                    $this->aSectionsets, // 2nd parameter
                 )
             );
 

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_Page.php

Spacing +13 added lines, -13 removed lines

@@ -56,7 +56,7 @@ 
                 'getPostTypeByTypeNow',
                 'getPostTypeByScreenObject',
                 'getPostTypeByREQUEST',
-                'getPostTypeByPostObject',  // 3.6.0+ Moved to the last as it is not reliable.
+                'getPostTypeByPostObject', // 3.6.0+ Moved to the last as it is not reliable.
             );
             foreach ( $_aMethodsToTry as $_sMethodName ) {
                 $_sPostType = call_user_func( array( __CLASS__, $_sMethodName ) );
@@ -96,7 +96,7 @@ 
                 }
                 if ( isset( $_GET[ 'post' ] ) && $_GET[ 'post' ] ) {    // sanitization unnecessary
                     // It will perform a database query.
-                    return get_post_type( absint( self::getHTTPQueryGET( 'post', 0 ) ) );  // sanitization done
+                    return get_post_type( absint( self::getHTTPQueryGET( 'post', 0 ) ) ); // sanitization done
                 }
             }
 
@@ -120,9 +120,9 @@ 
      * @param       array|string        The post type slug(s) to check. If this is empty, the method just checks the current page is a taxonomy page.
      * @return      boolean
      */
-    static public function isCustomTaxonomyPage( $asPostTypes=array() ) {
+    static public function isCustomTaxonomyPage( $asPostTypes = array() ) {
 
-        if ( ! in_array( self::getPageNow(), array( 'tags.php', 'edit-tags.php', 'term.php' ) ) ) {
+        if ( !in_array( self::getPageNow(), array( 'tags.php', 'edit-tags.php', 'term.php' ) ) ) {
             return false;
         }
         return self::isCurrentPostTypeIn( $asPostTypes );
@@ -137,10 +137,10 @@ 
      * Otherwise, it will check if the page belongs to the given post type(s).
      * @return      boolean
      */
-    static public function isPostDefinitionPage( $asPostTypes=array() ) {
+    static public function isPostDefinitionPage( $asPostTypes = array() ) {
 
         // If it's not the post definition page,
-        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) {
+        if ( !in_array( self::getPageNow(), array( 'post.php', 'post-new.php',) ) ) {
             return false;
         }
         return self::isCurrentPostTypeIn( $asPostTypes );
@@ -173,17 +173,17 @@ 
      *
      * @since 3.0.0
      */
-    static public function isPostListingPage( $asPostTypes=array() ) {
+    static public function isPostListingPage( $asPostTypes = array() ) {
 
         if ( 'edit.php' != self::getPageNow() ) {
             return false;
         }
 
         $_aPostTypes = self::getAsArray( $asPostTypes );
-        if ( ! isset( $_GET[ 'post_type' ] )  ) {   // sanitization unnecessary
+        if ( !isset( $_GET[ 'post_type' ] ) ) {   // sanitization unnecessary
             return in_array( 'post', $_aPostTypes, true );
         }
-        return in_array( $_GET[ 'post_type' ], $_aPostTypes, true );    // sanitization unnecessary
+        return in_array( $_GET[ 'post_type' ], $_aPostTypes, true ); // sanitization unnecessary
 
     }
 
@@ -217,7 +217,7 @@ 
             0 => '_getPageNow_FrontEnd',
             1 => '_getPageNow_BackEnd',
         );
-        $_sMethodName  = $_aMethodNames[ ( integer ) is_admin() ];
+        $_sMethodName = $_aMethodNames[ ( integer ) is_admin() ];
         self::$_sPageNow = self::$_sMethodName();
         return self::$_sPageNow;
 
@@ -312,7 +312,7 @@ 
 
         if ( isset( $GLBOALS[ 'page_hook' ] ) ) {
             return is_network_admin()
-                ? $GLBOALS[ 'page_hook' ] . '-network'
+                ? $GLBOALS[ 'page_hook' ].'-network'
                 : $GLBOALS[ 'page_hook' ];
         }
 
@@ -326,11 +326,11 @@ 
      * @since       3.7.0
      * @return      boolean
      */
-    static public function doesMetaBoxExist( $sContext='' ) {
+    static public function doesMetaBoxExist( $sContext = '' ) {
 
         $_aDimensions = array( 'wp_meta_boxes', $GLOBALS[ 'page_hook' ] );
         if ( $sContext ) {
-            $_aDimensions[] = $sContext;
+            $_aDimensions[ ] = $sContext;
         }
         $_aSideMetaBoxes = self::getElementAsArray(
             $GLOBALS,

Braces +8 added lines, -7 removed lines

@@ -91,10 +91,12 @@ 
              * Tries to find the post type from the URL query for type.
              */
             static public function getPostTypeByREQUEST() {
-                if ( isset( $_REQUEST[ 'post_type' ] ) ) {  // sanitization unnecessary
+                if ( isset( $_REQUEST[ 'post_type' ] ) ) {
+// sanitization unnecessary
                     return sanitize_key( sanitize_text_field( $_REQUEST[ 'post_type' ] ) ); // sanitization done
                 }
-                if ( isset( $_GET[ 'post' ] ) && $_GET[ 'post' ] ) {    // sanitization unnecessary
+                if ( isset( $_GET[ 'post' ] ) && $_GET[ 'post' ] ) {
+// sanitization unnecessary
                     // It will perform a database query.
                     return get_post_type( absint( self::getHTTPQueryGET( 'post', 0 ) ) );  // sanitization done
                 }
@@ -180,7 +182,8 @@ 
         }
 
         $_aPostTypes = self::getAsArray( $asPostTypes );
-        if ( ! isset( $_GET[ 'post_type' ] )  ) {   // sanitization unnecessary
+        if ( ! isset( $_GET[ 'post_type' ] )  ) {
+// sanitization unnecessary
             return in_array( 'post', $_aPostTypes, true );
         }
         return in_array( $_GET[ 'post_type' ], $_aPostTypes, true );    // sanitization unnecessary
@@ -270,11 +273,9 @@ 
 
                 if ( is_network_admin() ) {
                     $_sNeedle = '#/wp-admin/network/?(.*?)$#i';
-                }
-                else if ( is_user_admin() ) {
+                } else if ( is_user_admin() ) {
                     $_sNeedle = '#/wp-admin/user/?(.*?)$#i';
-                }
-                else {
+                } else {
                     $_sNeedle = '#/wp-admin/?(.*?)$#i';
                 }
                 preg_match( $_sNeedle, $_SERVER[ 'PHP_SELF' ], $_aMatches );

development/factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_URL.php

Spacing +16 added lines, -16 removed lines

@@ -29,10 +29,10 @@ 
      * @return string|array
      * @since  3.8.32
      */
-    static public function getHTTPQueryGET( $asKeys=array(), $mDefault=null ) {
+    static public function getHTTPQueryGET( $asKeys = array(), $mDefault = null ) {
         self::$___aGET = isset( self::$___aGET )
             ? self::$___aGET
-            : self::getArrayMappedRecursive( 'sanitize_text_field', $_GET );    // sanitization done
+            : self::getArrayMappedRecursive( 'sanitize_text_field', $_GET ); // sanitization done
         if ( empty( $asKeys ) ) {
             return self::$___aGET;
         }
@@ -51,9 +51,9 @@ 
         $sPageURL       = 'on' == @$_SERVER[ "HTTPS" ] ? "https://" : "http://";
 
         if ( "80" != $_SERVER[ "SERVER_PORT" ] ) {
-            $sPageURL .= $_SERVER[ "SERVER_NAME" ] . ":" . $_SERVER[ "SERVER_PORT" ] . $sRequestURI;
+            $sPageURL .= $_SERVER[ "SERVER_NAME" ].":".$_SERVER[ "SERVER_PORT" ].$sRequestURI;
         } else {
-            $sPageURL .= $_SERVER[ "SERVER_NAME" ] . $sRequestURI;
+            $sPageURL .= $_SERVER[ "SERVER_NAME" ].$sRequestURI;
         }
         return $sPageURL;
 
@@ -70,7 +70,7 @@ 
      * @param string $sSubjectURL ( optional ) The subject url to modify
      * @return string The modified url.
      */
-    static public function getQueryAdminURL( $aAddingQueries=array(), $aRemovingQueryKeys=array(), $sSubjectURL='' ) {
+    static public function getQueryAdminURL( $aAddingQueries = array(), $aRemovingQueryKeys = array(), $sSubjectURL = '' ) {
 
         $_sAdminURL = is_network_admin()
             ? network_admin_url( AdminPageFramework_WPUtility_Page::getPageNow() )
@@ -116,16 +116,16 @@ 
      * @remark  The parsable path is limited to under the WP_CONTENT_DIR directory.
      */
     static public function getSRCFromPath( $sFilePath ) {
-        $sFilePath        = str_replace('\\', '/', $sFilePath );
-        $_sContentDirPath = str_replace('\\', '/', WP_CONTENT_DIR );
+        $sFilePath        = str_replace( '\\', '/', $sFilePath );
+        $_sContentDirPath = str_replace( '\\', '/', WP_CONTENT_DIR );
         if ( false !== strpos( $sFilePath, $_sContentDirPath ) ) {
-            $_sRelativePath = AdminPageFramework_Utility::getRelativePath( WP_CONTENT_DIR , $sFilePath );
-            $_sRelativePath = preg_replace("/^\.[\/\\\]/", '', $_sRelativePath, 1 );
+            $_sRelativePath = AdminPageFramework_Utility::getRelativePath( WP_CONTENT_DIR, $sFilePath );
+            $_sRelativePath = preg_replace( "/^\.[\/\\\]/", '', $_sRelativePath, 1 );
             return content_url( $_sRelativePath );
         }
-        $_sRelativePath = AdminPageFramework_Utility::getRelativePath( ABSPATH , $sFilePath );
-        $_sRelativePath = preg_replace("/^\.[\/\\\]/", '', $_sRelativePath, 1 );
-        return trailingslashit( get_bloginfo( 'url' ) ) . $_sRelativePath;
+        $_sRelativePath = AdminPageFramework_Utility::getRelativePath( ABSPATH, $sFilePath );
+        $_sRelativePath = preg_replace( "/^\.[\/\\\]/", '', $_sRelativePath, 1 );
+        return trailingslashit( get_bloginfo( 'url' ) ).$_sRelativePath;
     }
 
     /**
@@ -141,9 +141,9 @@ 
      * @param       string      $sSRC
      * @param       boolean     $bReturnNullIfNotExist
      */
-    static public function getResolvedSRC( $sSRC, $bReturnNullIfNotExist=false ) {
+    static public function getResolvedSRC( $sSRC, $bReturnNullIfNotExist = false ) {
 
-        if ( ! self::isResourcePath( $sSRC ) ) {
+        if ( !self::isResourcePath( $sSRC ) ) {
             return $bReturnNullIfNotExist
                 ? null
                 : $sSRC;
@@ -156,7 +156,7 @@ 
 
         // If the file exists, it means it is an absolute path. If so, calculate the URL from the path.
         if ( file_exists( realpath( $sSRC ) ) ) {
-            return self::getSRCFromPath( $sSRC );   // url escaping is done in the method
+            return self::getSRCFromPath( $sSRC ); // url escaping is done in the method
         }
 
         if ( $bReturnNullIfNotExist ) {
@@ -173,7 +173,7 @@ 
          * @param boolean $bReturnNullIfNotExist
          * @return string
          */
-        static public function resolveSRC( $sSRC, $bReturnNullIfNotExist=false ) {
+        static public function resolveSRC( $sSRC, $bReturnNullIfNotExist = false ) {
             return self::getResolvedSRC( $sSRC, $bReturnNullIfNotExist );
         }