michaeluno / admin-page-framework

development/factory/_abstract/view/generator/field/AdminPageFramework_Generate_FieldInputName.php

Doc Comments +1 added lines

@@ -60,6 +60,7 @@
         /**
          * Applies the subject string to the set callback filter function.
          * @since       3.6.0
+         * @param string $sSubject
          */
         protected function _getFiltered( $sSubject ) {
             return is_callable( $this->hfCallback )

Spacing +2 added lines, -2 removed lines

@@ -51,9 +51,9 @@
         $_sIndex = $this->getAOrB(
             '0' !== $this->sIndex && empty( $this->sIndex ),
             '',
-            "[" . $this->sIndex . "]"
+            "[".$this->sIndex."]"
         );        
-        return $this->_getFiltered( $this->_getFieldName() . $_sIndex );
+        return $this->_getFiltered( $this->_getFieldName().$_sIndex );
         
     }
     

development/factory/_abstract/view/generator/section/AdminPageFramework_Generate_Section_Base.php

Doc Comments +1 added lines

@@ -51,6 +51,7 @@
     /**
      * Applies the subject string to the set callback filter function.
      * @since       3.6.0
+     * @param string $sSubject
      */
     protected function _getFiltered( $sSubject ) {
         return is_callable( $this->hfCallback )

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
             $this->hfCallback,
         );
         $this->aArguments = $_aParameters[ 0 ];        
-        $this->hfCallback  = $_aParameters[ 1 ];
+        $this->hfCallback = $_aParameters[ 1 ];
 
     }    
     

development/factory/admin_page/model/AdminPageFramework_FormEmail.php

Doc Comments +1 added lines

@@ -202,6 +202,7 @@
          * 
          * @since       3.3.0
          * @since       3.4.2       Moved from the validation class.
+         * @param string $sKey
          */
         private function _getEmailArgument( $aInput, array $aEmailOptions, $sKey, $sSectionID ) {
             

Indentation +1 added lines, -1 removed lines

@@ -277,7 +277,7 @@
         );        
     }    
     
-       /**
+        /**
          * Extracts meta box form fields options array from the given options array of an admin page.
          * 
          * @since       3.5.6

Spacing +4 added lines, -4 removed lines

@@ -85,7 +85,7 @@ 
         }
 
         // Send mail.
-        $_bSent         = wp_mail( 
+        $_bSent = wp_mail( 
             $this->_getEmailArgument( $aInput, $aEmailOptions, 'to', $sSubmitSectionID ),
             $this->_getEmailArgument( $aInput, $aEmailOptions, 'subject', $sSubmitSectionID ),
             $_bIsHTML 
@@ -100,7 +100,7 @@ 
         remove_filter( 'wp_mail_from_name', array( $this, '_replyToSetEmailSenderAddress' ) );
         
         // Clean up.
-        foreach( $this->_aPathsToDelete as $_sPath ) {
+        foreach ( $this->_aPathsToDelete as $_sPath ) {
             unlink( $_sPath );
         }
         
@@ -121,7 +121,7 @@ 
             }
             
             $_aAttachments = $this->getAsArray( $asAttachments );          
-            foreach( $_aAttachments as $_iIndex => $_sPathORURL ) {
+            foreach ( $_aAttachments as $_iIndex => $_sPathORURL ) {
                 
                 // If it is a file path, fine.
                 if ( is_file( $_sPathORURL ) ) {
@@ -211,7 +211,7 @@ 
             }
             
             // If the key element is empty, search the corresponding item in the same section. 
-            if ( ! $aEmailOptions[ $sKey ] ) {
+            if ( !$aEmailOptions[ $sKey ] ) {
                 return $this->getArrayValueByArrayKeys( $aInput, array( $sSectionID, $sKey ) );
             }
             

Braces +1 added lines, -1 removed lines

@@ -159,7 +159,7 @@
                 
                 // Download the file. It returns a string value on success. WP Error object on failure.
                 $_sPath = $this->download( $sURL, 10 );
-                if ( is_string( $_sPath ) ) {                    
+                if ( is_string( $_sPath ) ) {
                     $this->_aPathsToDelete[ $_sPath ] = $_sPath;
                     return $_sPath;
                 }

development/factory/admin_page/model/AdminPageFramework_ImportOptions.php

Doc Comments +5 added lines

@@ -89,6 +89,11 @@
         return $vData;
         
     }
+
+    /**
+     * @param string $vData
+     * @param string $sFormatType
+     */
     public function formatImportData( &$vData, $sFormatType=null ) {
         
         $sFormatType = isset( $sFormatType ) ? $sFormatType : $this->getFormatType();

Spacing +3 added lines, -3 removed lines

@@ -57,11 +57,11 @@ 
         
     }
     
-    private function getElementInFilesArray( array $aFilesImport, $sInputID, $sElementKey='error' ) {
+    private function getElementInFilesArray( array $aFilesImport, $sInputID, $sElementKey = 'error' ) {
 
         $sElementKey = strtolower( $sElementKey );
         return $this->getElement( 
-            $aFilesImport,  // subject array
+            $aFilesImport, // subject array
             array( $sElementKey, $sInputID ), // dimensional keys
             null    // default
         );
@@ -89,7 +89,7 @@ 
         return $vData;
         
     }
-    public function formatImportData( &$vData, $sFormatType=null ) {
+    public function formatImportData( &$vData, $sFormatType = null ) {
         
         $sFormatType = isset( $sFormatType ) ? $sFormatType : $this->getFormatType();
         switch ( strtolower( $sFormatType ) ) {

development/factory/admin_page/model/AdminPageFramework_Property_Page.php

Doc Comments +5 added lines

@@ -281,6 +281,8 @@ 
      * @remark      Used by the setInPageTabsVisibility() method.
      * @since       2.0.0
      * @since       2.1.5   The $oCaller parameter was added.
+     * @param AdminPageFramework_Router $oCaller
+     * @param string $sClassName
      */ 
     public function __construct( $oCaller, $sCallerPath, $sClassName, $aisOptionKey, $sCapability='manage_options', $sTextDomain='admin-page-framework' ) {
 
@@ -419,6 +421,7 @@ 
          * 
          * This is to support transient form data which disappears with the set timeout.
          * 
+         * @param string $sOptionType
          * @return      array       The retrieved options array.
          */ 
         private function _getOptionsByType( $sOptionType ) {
@@ -486,6 +489,7 @@ 
         /**
          * 
          * @since       3.5.9
+         * @param string $sOptionType
          * @return      boolean     True if saved; otherwise, false.
          */
         private function _updateOptionsByType( $aOptions, $sOptionType ) {
@@ -581,6 +585,7 @@ 
      * @since 2.1.5 Made it public and moved from the AdminPageFramework_Page class since this method is used by the AdminPageFramework_HeadTab class as well.
      * @internal
      * @remark Used in the __call() method in the main class.
+     * @param string $sPageSlug
      * @return string The default in-page tab slug if found; otherwise, an empty string.
      */         
     public function getDefaultInPageTab( $sPageSlug ) {

Spacing +35 added lines, -35 removed lines

@@ -216,7 +216,7 @@ 
      */
     public $aDisallowedQueryKeys = array( 
         'settings-updated', 
-        'confirmation',     // 3.3.0+
+        'confirmation', // 3.3.0+
         'field_errors'      // 3.4.1+
     );
         
@@ -271,7 +271,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_Page class object.
@@ -280,7 +280,7 @@ 
      * @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' ) {
         
         parent::__construct( 
             $oCaller, 
@@ -299,10 +299,10 @@ 
         );
                 
         /* Store the page class objects in the global storage. These will be referred by the meta box class to determine if the passed page slug's screen ID (hook suffix). */
-        $GLOBALS['aAdminPageFramework']['aPageClasses'] = isset( $GLOBALS['aAdminPageFramework']['aPageClasses'] ) && is_array( $GLOBALS['aAdminPageFramework']['aPageClasses'] )
-            ? $GLOBALS['aAdminPageFramework']['aPageClasses']
+        $GLOBALS[ 'aAdminPageFramework' ][ 'aPageClasses' ] = isset( $GLOBALS[ 'aAdminPageFramework' ][ 'aPageClasses' ] ) && is_array( $GLOBALS[ 'aAdminPageFramework' ][ 'aPageClasses' ] )
+            ? $GLOBALS[ 'aAdminPageFramework' ][ 'aPageClasses' ]
             : array();
-        $GLOBALS['aAdminPageFramework']['aPageClasses'][ $sClassName ] = $oCaller; // The meta box class for pages needs to access the object.
+        $GLOBALS[ 'aAdminPageFramework' ][ 'aPageClasses' ][ $sClassName ] = $oCaller; // The meta box class for pages needs to access the object.
                 
         // The capability for the settings. $this->sOptionKey is the part that is set in the settings_fields() function.
         // This prevents the "Cheatin' huh?" message.
@@ -353,7 +353,7 @@ 
                 }
                 // The user want to save options in a transient.
                 if ( in_array( $_sType, array( 'integer' ) ) ) {
-                    return 'apf_' . $sClassName . '_' . get_current_user_id();
+                    return 'apf_'.$sClassName.'_'.get_current_user_id();
                 }
                 
                 // Unknown type - maybe the user is trying to do something advanced.
@@ -380,10 +380,10 @@ 
          */
         protected function _isAdminPage() {
             
-            if ( ! is_admin() ) {
+            if ( !is_admin() ) {
                 return false;
             }
-            return isset( $_GET['page'] );
+            return isset( $_GET[ 'page' ] );
             
         }    
     
@@ -399,7 +399,7 @@ 
     
         $_aOptions = $this->oUtil->addAndApplyFilter( 
             $this->oCaller, // 3.4.1+ changed from $GLOBALS['aAdminPageFramework']['aPageClasses'][ $this->sClassName ], // the caller object
-            'options_' . $this->sClassName, // options_{instantiated class name}
+            'options_'.$this->sClassName, // options_{instantiated class name}
             $this->_getOptionsByType( $this->sOptionType ) // filtering item
         );
 
@@ -408,7 +408,7 @@ 
 // while getSavedOptions() method is called every time a field is processed for outputs.
 // However, in getSavedOptions, also the last input array is merged when the 'confirmation' query key is set,
 // that should be done here.
-        $_aLastInput = isset( $_GET['field_errors'] ) && $_GET['field_errors'] 
+        $_aLastInput = isset( $_GET[ 'field_errors' ] ) && $_GET[ 'field_errors' ] 
             ? $this->_getLastInput() 
             : array();
         $_aOptions   = $_aLastInput + $this->oUtil->getAsArray( $_aOptions );
@@ -433,7 +433,7 @@ 
                 case 'transient':
                     return $this->oUtil->getAsArray(
                         $this->oUtil->getTransient( 
-                            $this->sOptionKey,  // transient key
+                            $this->sOptionKey, // transient key
                             array() // default
                         )
                     );
@@ -467,7 +467,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;
@@ -495,7 +495,7 @@ 
                     );
                 case 'transient':
                     return $this->oUtil->setTransient( 
-                        $this->sOptionKey,  // transient key
+                        $this->sOptionKey, // transient key
                         $aOptions,
                         $this->iOptionTransientDuration
                     );
@@ -510,7 +510,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 
@@ -529,7 +529,7 @@ 
      */
     public function getCurrentPageSlug() {
         return $this->oUtil->getElement( 
-            $_GET,  // subject array
+            $_GET, // subject array
             'page', // key
             ''      // default
         );            
@@ -547,10 +547,10 @@ 
      * @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 = '' ) {
         
-        if ( isset( $_GET['tab'] ) && $_GET['tab'] ) { 
-            return $_GET['tab'];
+        if ( isset( $_GET[ 'tab' ] ) && $_GET[ 'tab' ] ) { 
+            return $_GET[ 'tab' ];
         }
         $sCurrentPageSlug = $sCurrentPageSlug
             ? $sCurrentPageSlug
@@ -565,7 +565,7 @@ 
          * 
          * @deprecated  3.5.3
          */
-        public function getCurrentTab( $sCurrentPageSlug='' ) {
+        public function getCurrentTab( $sCurrentPageSlug = '' ) {
             return $this->getCurrentTabSlug( $sCurrentPageSlug );            
         }
     
@@ -580,11 +580,11 @@ 
      */         
     public function getDefaultInPageTab( $sPageSlug ) {
     
-        if ( ! $sPageSlug ) { 
+        if ( !$sPageSlug ) { 
             return ''; 
         }
         return $this->oUtil->getElement( 
-            $this->aDefaultInPageTabs,  // subject array
+            $this->aDefaultInPageTabs, // subject array
             $sPageSlug, // key
             ''    // default
         );
@@ -599,14 +599,14 @@ 
     public function getDefaultOptions( $aFields ) {
         
         $_aDefaultOptions = array();
-        foreach( $aFields as $_sSectionID => $_aFields  ) {
+        foreach ( $aFields as $_sSectionID => $_aFields ) {
             
-            foreach( $_aFields as $_sFieldID => $_aField ) {
+            foreach ( $_aFields as $_sFieldID => $_aField ) {
                 
                 $_vDefault = $this->_getDefautValue( $_aField );
                 
-                if ( isset( $_aField['section_id'] ) && $_aField['section_id'] != '_default' ) {
-                    $_aDefaultOptions[ $_aField['section_id'] ][ $_sFieldID ] = $_vDefault;
+                if ( isset( $_aField[ 'section_id' ] ) && $_aField[ 'section_id' ] != '_default' ) {
+                    $_aDefaultOptions[ $_aField[ 'section_id' ] ][ $_sFieldID ] = $_vDefault;
                 } else {
                     $_aDefaultOptions[ $_sFieldID ] = $_vDefault;
                 }
@@ -633,11 +633,11 @@ 
             // If there are no sub-fields     
             if ( count( $_aSubFields ) == 0 ) {
                 return $this->oUtil->getElement(
-                    $aField,   // subject
-                    'value',    // key
+                    $aField, // subject
+                    'value', // key
                     $this->oUtil->getElement(   // default value
-                        $aField,   // subject  
-                        'default',  // key
+                        $aField, // subject  
+                        'default', // key
                         null        // default value
                     )
                 );
@@ -646,13 +646,13 @@ 
             // Otherwise, there are sub-fields
             $_aDefault = array();
             array_unshift( $_aSubFields, $aField ); // insert the main field into the very first index.
-            foreach( $_aSubFields as $_iIndex => $_aField ) {
+            foreach ( $_aSubFields as $_iIndex => $_aField ) {
                 $_aDefault[ $_iIndex ] = $this->oUtil->getElement( 
-                    $_aField,   // subject
-                    'value',    // key
+                    $_aField, // subject
+                    'value', // key
                     $this->oUtil->getElement(   // default value
-                        $_aField,   // subject  
-                        'default',  // key
+                        $_aField, // subject  
+                        'default', // key
                         null        // default value
                     )
                 ); 

Braces +4 added lines, -4 removed lines

@@ -314,7 +314,7 @@ 
          * @since       3.5.9
          * @return      void
          */
-        private function _setOptionsProperties( $aisOptionKey, $sClassName ) {            
+        private function _setOptionsProperties( $aisOptionKey, $sClassName ) {
                         
             $_aArguments = is_array( $aisOptionKey ) 
                 ? $aisOptionKey
@@ -510,7 +510,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 
@@ -549,7 +549,7 @@ 
      */    
     public function getCurrentTabSlug( $sCurrentPageSlug='' ) {
         
-        if ( isset( $_GET['tab'] ) && $_GET['tab'] ) { 
+        if ( isset( $_GET['tab'] ) && $_GET['tab'] ) {
             return $_GET['tab'];
         }
         $sCurrentPageSlug = $sCurrentPageSlug
@@ -580,7 +580,7 @@ 
      */         
     public function getDefaultInPageTab( $sPageSlug ) {
     
-        if ( ! $sPageSlug ) { 
+        if ( ! $sPageSlug ) {
             return ''; 
         }
         return $this->oUtil->getElement( 

development/factory/admin_page/model/delegate/AdminPageFramework_Model_FormSubmission_Base.php

Doc Comments +1 added lines

@@ -79,6 +79,7 @@
      * @since       3.5.3
      * @since       3.6.3       Moved from `AdminPageFramework_Form_Model_Validation_Option`.
      * @internal
+     * @param boolean $bIsInputEmtpy
      * @return      void
      * @remark      Accessed from some of the parent/child classes.
      */

Spacing +3 added lines, -3 removed lines

@@ -46,12 +46,12 @@
      * @return      null|string     Returns `null` if no value is found and the associated link url if found. 
      * Otherwise, the found value.
      */ 
-    protected function _getPressedSubmitButtonData( array $aPostElements, $sTargetKey='field_id' ) {    
+    protected function _getPressedSubmitButtonData( array $aPostElements, $sTargetKey = 'field_id' ) {    
 
-        foreach( $aPostElements as $_sInputID => $_aSubElements ) {
+        foreach ( $aPostElements as $_sInputID => $_aSubElements ) {
             
             // The 'name' key must be set.
-            if ( ! isset( $_aSubElements[ 'name' ] ) ) {
+            if ( !isset( $_aSubElements[ 'name' ] ) ) {
                 continue;
             }
             $_aNameKeys = explode( '|', $_aSubElements[ 'name' ] ); 

Braces +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
      * @return      null|string     Returns `null` if no value is found and the associated link url if found. 
      * Otherwise, the found value.
      */ 
-    protected function _getPressedSubmitButtonData( array $aPostElements, $sTargetKey='field_id' ) {    
+    protected function _getPressedSubmitButtonData( array $aPostElements, $sTargetKey='field_id' ) {
 
         foreach( $aPostElements as $_sInputID => $_aSubElements ) {
             
@@ -84,7 +84,7 @@ 
      */
     protected function _setSettingNoticeAfterValidation( $bIsInputEmtpy ) {
      
-        if ( $this->oFactory->hasSettingNotice() ) {     
+        if ( $this->oFactory->hasSettingNotice() ) {
             return;
         }
         $this->oFactory->setSettingNotice(  

development/factory/admin_page/model/delegate/validaor/AdminPageFramework_Model_FormSubmission_Validator_Export.php

Doc Comments +2 added lines

@@ -92,6 +92,7 @@ 
              * 
              * @since       3.5.4
              * @since       3.6.3       Moved from `AdminPageFramework_Form_Model_Export`.
+             * @param string $sFileName
              * @return      array
              */
             private function _getExportHeaderArray( array $aArguments, $sFileName, $mData ) {
@@ -132,6 +133,7 @@ 
              * @since       3.5.3
              * @since       3.6.3       Moved from `AdminPageFramework_Form_Model_Export`.
              * @internal   
+             * @param string $mData
              * @return      string      The export file name.
              */        
             private function _getExportFileName( array $aArguments, $sExportFileName, $mData ) {        

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * Admin Page Framework
- * 
- * http://en.michaeluno.jp/admin-page-framework/
- * Copyright (c) 2013-2015 Michael Uno; Licensed MIT
- * 
- */
+         * Admin Page Framework
+         * 
+         * http://en.michaeluno.jp/admin-page-framework/
+         * Copyright (c) 2013-2015 Michael Uno; Licensed MIT
+         * 
+         */
 
 /**
  * Provides methods to handle importing options.

Spacing +4 added lines, -4 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function _replyToCallback( $aInputs, $aRawInputs, array $aSubmits, $aSubmitInformation, $oFactory ) {
         
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return;
         }        
         $this->_exportOptions(
@@ -65,11 +65,11 @@ 
          */
         protected function _exportOptions( $mData, $sPageSlug, $sTabSlug ) {
 
-            $_oExport           = new AdminPageFramework_ExportOptions( 
+            $_oExport = new AdminPageFramework_ExportOptions( 
                 $_POST[ '__export' ], 
                 $this->oFactory->oProp->sClassName 
             );
-            $_aArguments        = array(
+            $_aArguments = array(
                 'class_name'        => $this->oFactory->oProp->sClassName,
                 'page_slug'         => $sPageSlug,
                 'tab_slug'          => $sTabSlug,
@@ -141,7 +141,7 @@ 
                     $sExportFileName, 
                     $aArguments[ 'pressed_field_id' ],
                     $aArguments[ 'pressed_input_id' ],
-                    $mData,     // 3.4.6+
+                    $mData, // 3.4.6+
                     $this->oFactory       // 3.4.6+
                 );    
             }

Braces +3 added lines, -3 removed lines

@@ -120,7 +120,7 @@ 
              * @internal   
              * @return      string      the filtered export data.
              */
-            private function _getFilteredExportingData( array $aArguments, $mData ) {                   
+            private function _getFilteredExportingData( array $aArguments, $mData ) {
                 return $this->_getFilteredItemForPortByPrefix(
                     'export_',
                     $mData,    
@@ -134,7 +134,7 @@ 
              * @internal   
              * @return      string      The export file name.
              */        
-            private function _getExportFileName( array $aArguments, $sExportFileName, $mData ) {        
+            private function _getExportFileName( array $aArguments, $sExportFileName, $mData ) {
                 return $this->addAndApplyFilters(
                     $this->oFactory,
                     $this->_getPortFilterHookNames( 'export_name_', $aArguments ),
@@ -152,7 +152,7 @@ 
              * @internal   
              * @return      string      The export format type.
              */        
-            private function _getExportFormatType( array $aArguments, $sExportFileFormat ) {  
+            private function _getExportFormatType( array $aArguments, $sExportFileFormat ) {
                 return $this->_getFilteredItemForPortByPrefix(
                     'export_format_',
                     $sExportFileFormat,    

development/factory/admin_page/model/delegate/validaor/AdminPageFramework_Model_FormSubmission_Validator_Import.php

Doc Comments +4 added lines

@@ -143,6 +143,7 @@ 
                 /**
                  * Sets update admin notice.
                  * @since       3.5.3
+                 * @param boolean $bEmpty
                  * @return      void
                  */
                 private function _setImportAdminNotice( $bEmpty ) {
@@ -204,6 +205,8 @@ 
                  * @since       3.5.3
                  * @since       3.6.3       Moved from `AdminPageFramework_Form_Model_Import`.
                  * @internal   
+                 * @param string $sFormatType
+                 * @param string $sImportOptionKey
                  * @return      string      The filtered import data.
                  */    
                 private function _getFilteredImportData( array $aArguments, $mData, $aStoredOptions, $sFormatType, $sImportOptionKey ) {
@@ -232,6 +235,7 @@ 
      * @param       string      The filter prefix.
      * @param       mixed       The subject filtering value.
      * @param       array       An argument array holding submit information. Part of this will be passed to the filter callbacks but this itself does not mean is for the filter callbacks.
+     * @param string $sPrefix
      */
     protected function _getFilteredItemForPortByPrefix( $sPrefix, $mFilteringValue, array $aArguments ) {
         return $this->addAndApplyFilters(

Spacing +10 added lines, -10 removed lines

@@ -32,7 +32,7 @@ 
      * @callback    action      try_validation_after_{class name}
      */
     public function _replyToCallback( $aInputs, $aRawInputs, array $aSubmits, $aSubmitInformation, $oFactory ) {
-        if ( ! $this->_shouldProceed() ) {
+        if ( !$this->_shouldProceed() ) {
             return;
         }
         $this->_doImportOptions(
@@ -102,9 +102,9 @@ 
                 // Check the uploaded file MIME type.
                 $_aMIMEType = $this->_getImportMIMEType( $_aArguments );
                 $_sType     = $_oImport->getType();
-                if ( ! in_array( $_sType, $_aMIMEType ) ) {        
+                if ( !in_array( $_sType, $_aMIMEType ) ) {        
                     $this->oFactory->setSettingNotice( sprintf( $this->oFactory->oMsg->get( 'uploaded_file_type_not_supported' ), $_sType ) );
-                    return $aStoredOptions;        // do not change the framework's options.
+                    return $aStoredOptions; // do not change the framework's options.
                 }
 
                 // Retrieve the importing data.
@@ -254,18 +254,18 @@ 
     protected function _getPortFilterHookNames( $sPrefix, array $aArguments ) {
         
         return array(
-            $sPrefix . $aArguments[ 'class_name' ] . '_' . $aArguments[ 'pressed_input_id' ],
+            $sPrefix.$aArguments[ 'class_name' ].'_'.$aArguments[ 'pressed_input_id' ],
             $aArguments[ 'section_id' ] 
-                ? $sPrefix . $aArguments[ 'class_name' ] . '_' . $aArguments[ 'section_id' ] .'_' . $aArguments[ 'pressed_field_id' ]
-                : $sPrefix . $aArguments[ 'class_name' ] . '_' . $aArguments[ 'pressed_field_id' ],
+                ? $sPrefix.$aArguments[ 'class_name' ].'_'.$aArguments[ 'section_id' ].'_'.$aArguments[ 'pressed_field_id' ]
+                : $sPrefix.$aArguments[ 'class_name' ].'_'.$aArguments[ 'pressed_field_id' ],
             $aArguments[ 'section_id' ] 
-                ? $sPrefix . $aArguments[ 'class_name' ] . '_' . $aArguments[ 'section_id' ] 
+                ? $sPrefix.$aArguments[ 'class_name' ].'_'.$aArguments[ 'section_id' ] 
                 : null,
             $aArguments[ 'tab_slug' ] 
-                ? $sPrefix . $aArguments[ 'page_slug' ] . '_' . $aArguments[ 'tab_slug' ]
+                ? $sPrefix.$aArguments[ 'page_slug' ].'_'.$aArguments[ 'tab_slug' ]
                 : null,
-            $sPrefix . $aArguments[ 'page_slug' ],
-            $sPrefix . $aArguments[ 'class_name' ]
+            $sPrefix.$aArguments[ 'page_slug' ],
+            $sPrefix.$aArguments[ 'class_name' ]
         );            
         
     }        

Braces +2 added lines, -2 removed lines

@@ -102,7 +102,7 @@ 
                 // Check the uploaded file MIME type.
                 $_aMIMEType = $this->_getImportMIMEType( $_aArguments );
                 $_sType     = $_oImport->getType();
-                if ( ! in_array( $_sType, $_aMIMEType ) ) {        
+                if ( ! in_array( $_sType, $_aMIMEType ) ) {
                     $this->oFactory->setSettingNotice( sprintf( $this->oFactory->oMsg->get( 'uploaded_file_type_not_supported' ), $_sType ) );
                     return $aStoredOptions;        // do not change the framework's options.
                 }
@@ -163,7 +163,7 @@ 
                  * @since       3.5.3
                  * @return      array       An array holding processable MIME types.       
                  */
-                private function _getImportMIMEType( array $aArguments ) {  
+                private function _getImportMIMEType( array $aArguments ) {
                     return $this->_getFilteredItemForPortByPrefix(
                         'import_mime_types_',
                         array( 'text/plain', 'application/octet-stream' ), // .json file is dealt as a binary file.

development/factory/admin_page/model/delegate/validaor/AdminPageFramework_Model_FormSubmission_Validator_Redirect.php

Doc Comments +1 added lines

@@ -66,6 +66,7 @@
          * Sets the given URL's transient.
          * @since       unknown
          * @since       3.6.3       Moved from `AdminPageFramework_Validation`.
+         * @param string $sURL
          */
         private function _setRedirectTransients( $sURL, $sPageSlug ) {
             if ( empty( $sURL ) ) { 

Spacing +3 added lines, -3 removed lines

@@ -35,7 +35,7 @@ 
             $aSubmits, 
             'redirect_url' 
         );
-        if ( ! $_sRedirectURL ) {
+        if ( !$_sRedirectURL ) {
             return;
         }
         
@@ -71,8 +71,8 @@ 
             if ( empty( $sURL ) ) { 
                 return; 
             }
-            $_sTransient = 'apf_rurl' . md5( trim( "redirect_{$this->oFactory->oProp->sClassName}_{$sPageSlug}" ) );
-            return $this->setTransient( $_sTransient, $sURL , 60*2 );
+            $_sTransient = 'apf_rurl'.md5( trim( "redirect_{$this->oFactory->oProp->sClassName}_{$sPageSlug}" ) );
+            return $this->setTransient( $_sTransient, $sURL, 60 * 2 );
         }
  
 }
\ No newline at end of file

Braces +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
          * @since       3.6.3       Moved from `AdminPageFramework_Validation`.
          */
         private function _setRedirectTransients( $sURL, $sPageSlug ) {
-            if ( empty( $sURL ) ) { 
+            if ( empty( $sURL ) ) {
                 return; 
             }
             $_sTransient = 'apf_rurl' . md5( trim( "redirect_{$this->oFactory->oProp->sClassName}_{$sPageSlug}" ) );