michaeluno / admin-page-framework

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

Spacing +26 added lines, -26 removed lines

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

Braces +6 added lines, -6 removed lines

@@ -320,14 +320,14 @@ 
         wp_enqueue_script( 'thickbox' );
         wp_enqueue_style( 'thickbox' );
     
-        if ( function_exists( 'wp_enqueue_media' ) ) {     
+        if ( function_exists( 'wp_enqueue_media' ) ) {
             // If the WordPress version is 3.5 or above,
             new AdminPageFramework_Form_View___Script_MediaUploader( $this->oMsg );
         } else {
             wp_enqueue_script( 'media-upload' );    
         }
 
-        if ( in_array( $this->getPageNow(), array( 'media-upload.php', 'async-upload.php', ) ) ) {     
+        if ( in_array( $this->getPageNow(), array( 'media-upload.php', 'async-upload.php', ) ) ) {
             add_filter( 'gettext', array( $this, '_replyToReplaceThickBoxText' ) , 1, 2 );     
         }
         
@@ -343,17 +343,17 @@ 
         public function _replyToReplaceThickBoxText( $sTranslated, $sText ) {
 
             // Replace the button label in the media thick box.
-            if ( ! in_array( $this->getPageNow(), array( 'media-upload.php', 'async-upload.php' ) ) ) { 
+            if ( ! in_array( $this->getPageNow(), array( 'media-upload.php', 'async-upload.php' ) ) ) {
                 return $sTranslated; 
             }
-            if ( $sText !== 'Insert into Post' ) { 
+            if ( $sText !== 'Insert into Post' ) {
                 return $sTranslated; 
             }
-            if ( $this->getQueryValueInURLByKey( wp_get_referer(), 'referrer' ) !== 'admin_page_framework' ) { 
+            if ( $this->getQueryValueInURLByKey( wp_get_referer(), 'referrer' ) !== 'admin_page_framework' ) {
                 return $sTranslated; 
             }
             
-            if ( isset( $_GET['button_label'] ) ) { 
+            if ( isset( $_GET['button_label'] ) ) {
                 return $_GET['button_label']; 
             }
 

Doc Comments +1 added lines

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

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

Spacing +13 added lines, -13 removed lines

@@ -90,16 +90,16 @@ 
                 'AdminPageFramework_Form_View___CSS_FieldError',
                 'AdminPageFramework_Form_View___CSS_ToolTip',
             );
-            foreach( $_aClassNames as $_sClassName ) {
+            foreach ( $_aClassNames as $_sClassName ) {
                 $_oCSS = new $_sClassName;
-                $this->aResources[ 'internal_styles' ][] = $_oCSS->get();
+                $this->aResources[ 'internal_styles' ][ ] = $_oCSS->get();
             }
             $_aClassNamesForIE = array(
                 'AdminPageFramework_Form_View___CSS_CollapsibleSectionIE',
             );
-            foreach( $_aClassNames as $_sClassName ) {
+            foreach ( $_aClassNames as $_sClassName ) {
                 $_oCSS = new $_sClassName;
-                $this->aResources[ 'internal_styles_ie' ][] = $_oCSS->get();
+                $this->aResources[ 'internal_styles_ie' ][ ] = $_oCSS->get();
             }
             
         }
@@ -117,7 +117,7 @@ 
          * @return      void
          */
         private function ___set( $aAllFieldsets ) {
-            foreach( $aAllFieldsets as $_sSecitonID => $_aFieldsets ) {
+            foreach ( $aAllFieldsets as $_sSecitonID => $_aFieldsets ) {
                 $this->___setFieldResourcesBySection( $_aFieldsets );
             }
         }
@@ -129,7 +129,7 @@ 
             private function ___setFieldResourcesBySection( $_aFieldsets ) {
 
                 $_bIsSubSectionLoaded = false;
-                foreach( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField )  {
+                foreach ( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField ) {
 // @todo Examine if this structure is correct or not.
 // It may not be necessary to check the sub-section dimensions as this is not the saved options array.
                     // if it's a sub-section
@@ -140,7 +140,7 @@ 
                             continue;
                         }
                         $_bIsSubSectionLoaded = true;
-                        foreach( $_aSubSectionOrField as $_aField ) {
+                        foreach ( $_aSubSectionOrField as $_aField ) {
                             $this->___setFieldResources( $_aField );
                         }
                         continue;
@@ -165,7 +165,7 @@ 
                 private function ___setFieldResources( $aFieldset ) {
 
                     // Check the field conditions.
-                    if ( ! $this->___isFieldsetAllowed( $aFieldset ) ) {
+                    if ( !$this->___isFieldsetAllowed( $aFieldset ) ) {
                         return;
                     }
 
@@ -196,7 +196,7 @@ 
                         return $this->callBack(
                             $this->aCallbacks[ 'is_fieldset_registration_allowed' ],
                             array(
-                                true,   // 1st parameter
+                                true, // 1st parameter
                                 $aFieldset, // 2nd parameter
                             )
                         );
@@ -208,7 +208,7 @@ 
                      */
                     private function ___setResourcesOfNestedFields( $aFieldset ) {
 
-                        if ( ! $this->hasFieldDefinitionsInContent( $aFieldset ) ) {
+                        if ( !$this->hasFieldDefinitionsInContent( $aFieldset ) ) {
                             return;
                         }
                         foreach ( $aFieldset[ 'content' ] as $_asNestedFieldset ) {
@@ -248,7 +248,7 @@ 
                         $this->callBack(
                             $this->aCallbacks[ 'load_fieldset_resource' ],
                             array(
-                                $aFieldset,   // 1st parameter
+                                $aFieldset, // 1st parameter
                             )
                         );
 
@@ -256,7 +256,7 @@ 
         // Be careful not to add duplicate items as currently the sub-field items are parsed.
 
                         // Is already registered?
-                        if ( $this->hasBeenCalled( 'registered_' . $_sFieldtype . '_' . $this->aArguments[ 'structure_type' ] ) ) {
+                        if ( $this->hasBeenCalled( 'registered_'.$_sFieldtype.'_'.$this->aArguments[ 'structure_type' ] ) ) {
                             return;
                         }
 
@@ -271,7 +271,7 @@ 
                             $_aFieldTypeDefinition,
                             $this->aResources
                         );
-                        $this->aResources     = $_oFieldTypeResources->get();
+                        $this->aResources = $_oFieldTypeResources->get();
 
                     }
 

Braces +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
             private function ___setFieldResourcesBySection( $_aFieldsets ) {
 
                 $_bIsSubSectionLoaded = false;
-                foreach( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField )  {
+                foreach( $_aFieldsets as $_iSubSectionIndexOrFieldID => $_aSubSectionOrField ) {
 // @todo Examine if this structure is correct or not.
 // It may not be necessary to check the sub-section dimensions as this is not the saved options array.
                     // if it's a sub-section

development/factory/_common/_abstract/AdminPageFramework_Factory_Router.php

Spacing +17 added lines, -17 removed lines

@@ -164,8 +164,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 );
         
     }    
         /**
@@ -174,8 +174,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;
         }
@@ -189,15 +189,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();         
         }
         
@@ -211,10 +211,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();
@@ -226,7 +226,7 @@ 
      */
     protected function _setUp() {
         $aActions = array(
-            'set_up_' . $this->oProp->sClassName,
+            'set_up_'.$this->oProp->sClassName,
         );
         $this->setUp();
         $this->oUtil->addAndDoActions( $this, $aActions, $this );
@@ -269,7 +269,7 @@ 
      */
     public function _replyToDetermineToLoad() {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( !$this->_isInThePage() ) { 
             return; 
         }
 
@@ -333,7 +333,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}" )
             );
         }
         
@@ -367,7 +367,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;
@@ -432,7 +432,7 @@ 
      * 
      * @internal
      */
-    public function __call( $sMethodName, $aArguments=null ) {    
+    public function __call( $sMethodName, $aArguments = null ) {    
          
         $_mFirstArg = $this->oUtil->getElement( $aArguments, 0 );
         
@@ -462,7 +462,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. 
@@ -473,7 +473,7 @@ 
                     array( $this, $_sAutoCallbackMethodName ),
                     $aArguments
                 )
-                : $this->oUtil->getElement( $aArguments, 0 );   // the first argument
+                : $this->oUtil->getElement( $aArguments, 0 ); // the first argument
             
         }
         
@@ -483,7 +483,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 

Braces +4 added lines, -4 removed lines

@@ -189,7 +189,7 @@ 
      */
     public function _replyToLoadComponents( /* $oScreen */ ) {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( ! $this->_isInThePage() ) {
             return; 
         }
                     
@@ -240,7 +240,7 @@ 
      * @since       3.1.0
      * @internal
      */ 
-    protected function _isInstantiatable() { 
+    protected function _isInstantiatable() {
         return true; 
     }
     
@@ -269,7 +269,7 @@ 
      */
     public function _replyToDetermineToLoad() {
 
-        if ( ! $this->_isInThePage() ) { 
+        if ( ! $this->_isInThePage() ) {
             return; 
         }
 
@@ -432,7 +432,7 @@ 
      * 
      * @internal
      */
-    public function __call( $sMethodName, $aArguments=null ) {    
+    public function __call( $sMethodName, $aArguments=null ) {
          
         $_mFirstArg = $this->oUtil->getElement( $aArguments, 0 );
         

development/factory/_common/_abstract/_view/AdminPageFramework_Factory_View__SettingNotice.php

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
      * @since       3.7.9       Added the second parameter to accept an action hook name.
      */
         
-    public function __construct( $oFactory, $sActionHookName='admin_notices' ) {
+    public function __construct( $oFactory, $sActionHookName = 'admin_notices' ) {
         
         $this->oFactory = $oFactory;
            
@@ -45,7 +45,7 @@ 
      */
     public function _replyToPrintSettingNotice() {
             
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return;
         }
         $this->oFactory->oForm->printSubmitNotices();
@@ -59,7 +59,7 @@ 
          */
         private function _shouldProceed() {
             
-            if ( ! $this->oFactory->isInThePage() ) {
+            if ( !$this->oFactory->isInThePage() ) {
                 return false; 
             }
                 

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

Spacing +9 added lines, -9 removed lines

@@ -53,7 +53,7 @@ 
      * @since 2.1.0
      * @internal
      */  
-    protected function _setHelpTab( $sID, $sTitle, $aContents, $aSideBarContents=array() ) {
+    protected function _setHelpTab( $sID, $sTitle, $aContents, $aSideBarContents = array() ) {
         
         if ( empty( $aContents ) ) { 
             return; 
@@ -70,7 +70,7 @@ 
             ) 
         );     
         
-        if ( ! empty( $aSideBarContents ) ) {
+        if ( !empty( $aSideBarContents ) ) {
             $this->_oScreen->set_help_sidebar( implode( PHP_EOL, $aSideBarContents ) );
         }
             
@@ -82,7 +82,7 @@ 
      * @internal
      */ 
     protected function _formatHelpDescription( $sHelpDescription ) {
-        return "<div class='contextual-help-description'>" . $sHelpDescription . "</div>";
+        return "<div class='contextual-help-description'>".$sHelpDescription."</div>";
     }
     
     
@@ -106,10 +106,10 @@ 
      * @remark      This method just adds the given text into the class property. The actual registration will be performed with the <em>replyToRegisterHelpTabTextForMetaBox()</em> method.
      * @internal
      */ 
-    public function _addHelpText( $sHTMLContent, $sHTMLSidebarContent="" ) {
+    public function _addHelpText( $sHTMLContent, $sHTMLSidebarContent = "" ) {
         
-        $this->oProp->aHelpTabText[]        = "<div class='contextual-help-description'>" . $sHTMLContent . "</div>";
-        $this->oProp->aHelpTabTextSide[]    = "<div class='contextual-help-description'>" . $sHTMLSidebarContent . "</div>";
+        $this->oProp->aHelpTabText[ ]        = "<div class='contextual-help-description'>".$sHTMLContent."</div>";
+        $this->oProp->aHelpTabTextSide[ ]    = "<div class='contextual-help-description'>".$sHTMLSidebarContent."</div>";
         
     }
     
@@ -125,9 +125,9 @@ 
      * @remark      This method just adds the given text into the class property. The actual registration will be performed with the `replyToRegisterHelpTabTextForMetaBox()` method.
      * @internal
      */     
-    public function _addHelpTextForFormFields( $sFieldTitle, $sHelpText, $sHelpTextSidebar="" ) {
+    public function _addHelpTextForFormFields( $sFieldTitle, $sHelpText, $sHelpTextSidebar = "" ) {
         $this->_addHelpText(
-            "<span class='contextual-help-tab-title'>" . $sFieldTitle . "</span> - " . PHP_EOL
+            "<span class='contextual-help-tab-title'>".$sFieldTitle."</span> - ".PHP_EOL
                 . $sHelpText,     
             $sHelpTextSidebar
         );     
@@ -145,7 +145,7 @@ 
     public function _replyToRegisterHelpTabText() {
 
         // Check if the currently loaded page is of meta box page.
-        if ( ! $this->oProp->oCaller->isInThePage() ) {
+        if ( !$this->oProp->oCaller->isInThePage() ) {
             return false; 
         }
 

Braces +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
      */  
     protected function _setHelpTab( $sID, $sTitle, $aContents, $aSideBarContents=array() ) {
         
-        if ( empty( $aContents ) ) { 
+        if ( empty( $aContents ) ) {
             return; 
         }
         

development/factory/admin_page/AdminPageFramework_Router.php

Spacing +20 added lines, -20 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' ) {
         
         $_sProprtyClassName = isset( $this->aSubClassNames[ 'oProp' ] )
             ? $this->aSubClassNames[ 'oProp' ]
-            : 'AdminPageFramework_Property_' . $this->_sStructureType;
+            : 'AdminPageFramework_Property_'.$this->_sStructureType;
             
         $this->oProp = new $_sProprtyClassName( 
             $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,7 +64,7 @@ 
      * @callback        add_action      wp_loaded
      */
     public function _replyToLoadComponentsForAjax() {
-        if ( ! $this->oProp->bIsAdminAjax ) {
+        if ( !$this->oProp->bIsAdminAjax ) {
             return;
         }
         new AdminPageFramework_Model_Menu__RegisterMenu( $this, 'pseudo_admin_menu' );
@@ -111,7 +111,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 );
@@ -122,15 +122,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_':
@@ -148,9 +148,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;
                 }
@@ -174,7 +174,7 @@ 
          */ 
         protected function _doPageLoadCall( $sMethodName, $sPageSlug, $sTabSlug, $oScreen ) {
 
-            if ( ! $this->_isPageLoadCall( $sMethodName, $sPageSlug, $oScreen ) ) {
+            if ( !$this->_isPageLoadCall( $sMethodName, $sPageSlug, $oScreen ) ) {
                 return;
             }
 
@@ -198,10 +198,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.
                 );         
             }
@@ -226,10 +226,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' ),
@@ -274,7 +274,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 )
@@ -302,7 +302,7 @@ 
 //        }
         
         // Nothing to do in the network admin area.
-        return ! is_network_admin();
+        return !is_network_admin();
         
     }
     
@@ -323,11 +323,11 @@ 
         }
 
         // If the setUp method is not loaded yet,
-        if ( ! did_action( 'set_up_' . $this->oProp->sClassName ) ) {
+        if ( !did_action( 'set_up_'.$this->oProp->sClassName ) ) {
             return true;
         }    
 
-        if ( ! isset( $_GET[ 'page' ] ) ) { 
+        if ( !isset( $_GET[ 'page' ] ) ) { 
             return false; 
         }        
         

Braces +3 added lines, -3 removed lines

@@ -111,7 +111,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 );
@@ -197,7 +197,7 @@ 
             // Note that if the tab is the first item and the user arrives the page by clicking on the sidebar menu,
             // the tab slug will be empty unless an in-page tab is added.
             $sTabSlug = $this->oProp->getCurrentTabSlug( $sPageSlug );
-            if ( strlen( $sTabSlug ) ) { 
+            if ( strlen( $sTabSlug ) ) {
                 $this->_setShowDebugInfoProperty( $sPageSlug, $sTabSlug );  // 3.8.8+
                 $this->oUtil->addAndDoActions( 
                     $this, // the caller object
@@ -327,7 +327,7 @@ 
             return true;
         }    
 
-        if ( ! isset( $_GET[ 'page' ] ) ) { 
+        if ( ! isset( $_GET[ 'page' ] ) ) {
             return false; 
         }        
         

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

Spacing +7 added lines, -7 removed lines

@@ -28,7 +28,7 @@ 
      */
     public function __construct( $oFactory ) {
        
-        $this->oFactory         = $oFactory;        
+        $this->oFactory = $oFactory;        
         
         // wp_mail() will be loaded by the time 'after_setup_theme' is loaded.
         // @deprecated      3.7.0
@@ -52,18 +52,18 @@ 
      */
     public function _replyToCheckRedirects() {
   
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return;
         }
 
         // The redirect transient key.
-        $_sTransient = 'apf_rurl' . md5( 
-            trim( "redirect_{$this->oFactory->oProp->sClassName}_{$_GET['page']}" )
+        $_sTransient = 'apf_rurl'.md5( 
+            trim( "redirect_{$this->oFactory->oProp->sClassName}_{$_GET[ 'page' ]}" )
         );
         
         // Check the settings error transient.
         $_aError = $this->oFactory->getFieldErrors();
-        if ( ! empty( $_aError ) ) {
+        if ( !empty( $_aError ) ) {
             $this->deleteTransient( $_sTransient ); // we don't need it any more.
             return;
         }
@@ -90,7 +90,7 @@ 
         private function _shouldProceed() {
 
             // Check if it's one of the plugin's added page. If not, do nothing.
-            if ( ! $this->oFactory->isInThePage() ) {
+            if ( !$this->oFactory->isInThePage() ) {
                 return false;
             }
             
@@ -103,7 +103,7 @@ 
                 'settings-updated',
                 false
             );            
-            if ( ! $_bsSettingsUpdatedFlag ) {
+            if ( !$_bsSettingsUpdatedFlag ) {
                 return false;
             }
             

development/factory/admin_page/_model/AdminPageFramework_Property_admin_page.php

Spacing +14 added lines, -14 removed lines

@@ -217,7 +217,7 @@ 
      */
     public $aDisallowedQueryKeys = array( 
         'settings-updated', 
-        'confirmation',     // 3.3.0+
+        'confirmation', // 3.3.0+
         'field_errors'      // 3.4.1+
     );
         
@@ -263,7 +263,7 @@ 
      * Stores the cache lifetime of the transient used for the form options when the user passes an integer to the option key parameter.
      * @since       3.5.9
      */
-    public $iOptionTransientDuration  = 0; 
+    public $iOptionTransientDuration = 0; 
      
     /**
      * Constructs the instance of AdminPageFramework_Property_admin_page class object.
@@ -272,10 +272,10 @@ 
      * @since       2.0.0
      * @since       2.1.5   The $oCaller parameter was added.
      */ 
-    public function __construct( $oCaller, $sCallerPath, $sClassName, $aisOptionKey, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
+    public function __construct( $oCaller, $sCallerPath, $sClassName, $aisOptionKey, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework' ) {
 
         // 3.7.0+ This must be set before the parent constructor. As the form arguments array uses this value.
-        $this->_sFormRegistrationHook = 'load_after_' . $sClassName;
+        $this->_sFormRegistrationHook = 'load_after_'.$sClassName;
 
         parent::__construct( 
             $oCaller, 
@@ -351,7 +351,7 @@ 
                 }
                 // The user want to save options in a transient.
                 if ( in_array( $_sType, array( 'integer' ) ) ) {
-                    return 'apf_' . md5( site_url() . '_' . $sClassName . '_' . get_current_user_id() );
+                    return 'apf_'.md5( site_url().'_'.$sClassName.'_'.get_current_user_id() );
                 }
                 
                 // Unknown type - maybe the user is trying to do something advanced.
@@ -378,7 +378,7 @@ 
          */
         protected function _isAdminPage() {
             
-            if ( ! is_admin() ) {
+            if ( !is_admin() ) {
                 return false;
             }
             return isset( $_GET[ 'page' ] );
@@ -418,7 +418,7 @@ 
                 case 'transient':
                     return $this->getAsArray(
                         $this->getTransient( 
-                            $this->sOptionKey,  // transient key
+                            $this->sOptionKey, // transient key
                             array() // default
                         )
                     );
@@ -432,7 +432,7 @@ 
      * @since       3.1.1       Made it return a value.
      * @return      boolean     True if saved; otherwise, false.
      */
-    public function updateOption( $aOptions=null ) {
+    public function updateOption( $aOptions = null ) {
 
         if ( $this->_bDisableSavingOptions ) {
             return false;
@@ -460,7 +460,7 @@ 
                     );
                 case 'transient':
                     return $this->setTransient( 
-                        $this->sOptionKey,  // transient key
+                        $this->sOptionKey, // transient key
                         $aOptions,
                         $this->iOptionTransientDuration
                     );
@@ -475,7 +475,7 @@ 
      * @since       2.1.0       Set the default value to the parameter and if the parameter value is empty, it applies the current $_GET[ 'page' ] value.
      * @return      boolean     Returns true if it is of framework's added page; otherwise, false.
      */
-    public function isPageAdded( $sPageSlug='' ) {    
+    public function isPageAdded( $sPageSlug = '' ) {    
         
         $sPageSlug = trim( $sPageSlug );
         $sPageSlug = $sPageSlug 
@@ -512,7 +512,7 @@ 
      * @return      string      The found tab slug. An empty string if not found.
      * @remark      Do not return `null` when not found as some framework methods check the returned value with `isset()` and if null is given, `isset()` yields `false` while it does `true` for an empty string ('').
      */    
-    public function getCurrentTabSlug( $sCurrentPageSlug='' ) {
+    public function getCurrentTabSlug( $sCurrentPageSlug = '' ) {
 
         // It is possible that the tab slug is not set if it is the default tab.
         $_sTabSlug = $this->getElement( $this->aQuery, 'tab', '' );
@@ -532,7 +532,7 @@ 
          * 
          * @deprecated  3.5.3
          */
-        public function getCurrentTab( $sCurrentPageSlug='' ) {
+        public function getCurrentTab( $sCurrentPageSlug = '' ) {
             return $this->getCurrentTabSlug( $sCurrentPageSlug );            
         }
     
@@ -547,11 +547,11 @@ 
      */
     public function getDefaultInPageTab( $sPageSlug ) {
 
-        if ( ! $sPageSlug ) {
+        if ( !$sPageSlug ) {
             return ''; 
         }
         return $this->getElement( 
-            $this->aDefaultInPageTabs,  // subject array
+            $this->aDefaultInPageTabs, // subject array
             $sPageSlug, // key
             ''    // default
         );

Braces +3 added lines, -3 removed lines

@@ -312,7 +312,7 @@ 
          * @since       3.5.9
          * @return      void
          */
-        private function _setOptionsProperties( $aisOptionKey, $sClassName ) {            
+        private function _setOptionsProperties( $aisOptionKey, $sClassName ) {
                         
             $_aArguments = is_array( $aisOptionKey ) 
                 ? $aisOptionKey
@@ -475,7 +475,7 @@ 
      * @since       2.1.0       Set the default value to the parameter and if the parameter value is empty, it applies the current $_GET[ 'page' ] value.
      * @return      boolean     Returns true if it is of framework's added page; otherwise, false.
      */
-    public function isPageAdded( $sPageSlug='' ) {    
+    public function isPageAdded( $sPageSlug='' ) {
         
         $sPageSlug = trim( $sPageSlug );
         $sPageSlug = $sPageSlug 
@@ -516,7 +516,7 @@ 
 
         // It is possible that the tab slug is not set if it is the default tab.
         $_sTabSlug = $this->getElement( $this->aQuery, 'tab', '' );
-        if ( $_sTabSlug ) { 
+        if ( $_sTabSlug ) {
             return $_sTabSlug;
         }
         $sCurrentPageSlug = $sCurrentPageSlug

development/factory/meta_box/_controller/AdminPageFramework_HelpPane_post_meta_box.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     public function _replyToRegisterHelpTabText() {
 
         // Check if the currently loaded page is of meta box page.
-        if ( ! $this->oProp->oCaller->isInThePage() ) {
+        if ( !$this->oProp->oCaller->isInThePage() ) {
             return false; 
         }