michaeluno / admin-page-framework

development/factory/admin_page/_model/AdminPageFramework_ExportOptions.php

Spacing +5 added lines, -5 removed lines

@@ -99,9 +99,9 @@ 
      * @since 3.5.4       Added the `$aHeader` parameter. Deprecated the `$sFileName` parameter as it is included in the $aHeader definition.
      * @since 2.0.0
      */
-    public function doExport( $vData, $sFormatType=null, array $aHeader=array() ) {
+    public function doExport( $vData, $sFormatType = null, array $aHeader = array() ) {
 
-        $sFormatType    = isset( $sFormatType )
+        $sFormatType = isset( $sFormatType )
             ? $sFormatType
             : $this->sFormatType;
 
@@ -129,9 +129,9 @@ 
          * @param       string $sKey
          * @return      void
          */
-        private function _outputHTTPHeader( array $aHeader, $sKey='' ) {
+        private function _outputHTTPHeader( array $aHeader, $sKey = '' ) {
 
-            foreach( $aHeader as $_sKey => $_asValue ) {
+            foreach ( $aHeader as $_sKey => $_asValue ) {
 
                 // Nested items. Set the key to overrider array keys.
                 if ( is_array( $_asValue ) ) {
@@ -173,7 +173,7 @@ 
                     return;
                 case 'json': // for json.
                     echo json_encode( ( array ) $vData );
-                    return ;
+                    return;
                 case 'array': // for serialized PHP array.
                 default: // for anything else,
                     echo serialize( ( array ) $vData );

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

Spacing +11 added lines, -11 removed lines

@@ -25,7 +25,7 @@ 
      * @since  3.8.24
      */
     static public function getHTTPRequestSanitized( array $aRequest, $bStripSlashes ) {
-        foreach( $aRequest as $_isIndex => $_mValue ) {
+        foreach ( $aRequest as $_isIndex => $_mValue ) {
             if ( is_array( $_mValue ) ) {
                 $aRequest[ $_isIndex ] = self::getHTTPRequestSanitized( $_mValue, $bStripSlashes );
                 continue;
@@ -47,10 +47,10 @@ 
          * @sicne  3.8.25
          * @see    _sanitize_text_fields()
          */
-        static private function ___getHTTPRequestTextValueSanitized( $sString, $bKeepLineFeeds=true ) {
+        static private function ___getHTTPRequestTextValueSanitized( $sString, $bKeepLineFeeds = true ) {
             
             $_sFiltered = wp_check_invalid_utf8( $sString );
-            if ( ! $bKeepLineFeeds ) {
+            if ( !$bKeepLineFeeds ) {
                 $_sFiltered = preg_replace( '/[\r\n\t ]+/', ' ', $_sFiltered );
             }
             $_sFiltered = trim( $_sFiltered );
@@ -99,7 +99,7 @@ 
      * @return mixed
      * @since  3.8.24
      */
-    static public function getObjectCache( $asName, $mDefault=null ) {
+    static public function getObjectCache( $asName, $mDefault = null ) {
         return self::getArrayValueByArrayKeys( self::$___aObjectCache, self::getAsArray( $asName ), $mDefault );
     }
 
@@ -114,9 +114,9 @@ 
      * @param   string $sProgramName
      * @since   3.8.8
      */
-    static public function showDeprecationNotice( $sDeprecated, $sAlternative='', $sProgramName='Admin Page Framework' ) {
+    static public function showDeprecationNotice( $sDeprecated, $sAlternative = '', $sProgramName = 'Admin Page Framework' ) {
         trigger_error(
-            $sProgramName . ': ' . sprintf(
+            $sProgramName.': '.sprintf(
                 $sAlternative
                     ? '<code>%1$s</code> has been deprecated. Use <code>%2$s</code> instead.'
                     : '<code>%1$s</code> has been deprecated.',
@@ -139,8 +139,8 @@ 
      * @param       string|array        $asParameters       Parameters to pass to the callback function.
      * @return      mixed
      */
-    public function callBack( $oCallable, $asParameters=array() ) {
-        $_aParameters   = self::getAsArray(
+    public function callBack( $oCallable, $asParameters = array() ) {
+        $_aParameters = self::getAsArray(
             $asParameters,
             true // preserve empty
         );
@@ -179,7 +179,7 @@ 
      * @param       array       $aParameters
      * @return      string      The captured output buffer.
      */
-    static public function getOutputBuffer( $cCallable, array $aParameters=array() ) {
+    static public function getOutputBuffer( $cCallable, array $aParameters = array() ) {
 
         ob_start();
         echo call_user_func_array( $cCallable, $aParameters );
@@ -201,7 +201,7 @@ 
 
         $_iCount     = count( get_object_vars( $oInstance ) );
         $_sClassName = get_class( $oInstance );
-        return '(object) ' . $_sClassName . ': ' . $_iCount . ' properties.';
+        return '(object) '.$_sClassName.': '.$_iCount.' properties.';
 
     }
 
@@ -218,7 +218,7 @@ 
      * @param       boolean|integer|double|string|array|object|resource|NULL        $mFalse     The value to return when the first parameter value yields false.
      * @return      mixed
      */
-    static public function getAOrB( $mValue, $mTrue=null, $mFalse=null ) {
+    static public function getAOrB( $mValue, $mTrue = null, $mFalse = null ) {
         return $mValue ? $mTrue : $mFalse;
     }
 

admin-page-framework-loader.php

Spacing +22 added lines, -22 removed lines

@@ -16,9 +16,9 @@ 
  */
 class AdminPageFrameworkLoader_Registry_Base {
 
-    const VERSION        = '3.8.26b01';    // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
+    const VERSION        = '3.8.26b01'; // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
     const NAME           = 'Admin Page Framework - Loader'; // the name is not 'Admin Page Framework' because warning messages gets confusing.
-    const SHORTNAME      = 'Admin Page Framework';  // used for a menu title etc.
+    const SHORTNAME      = 'Admin Page Framework'; // used for a menu title etc.
     const DESCRIPTION    = 'Loads Admin Page Framework which facilitates WordPress plugin and theme development.';
     const URI            = 'http://admin-page-framework.michaeluno.jp/';
     const AUTHOR         = 'miunosoft (Michael Uno)';
@@ -55,14 +55,14 @@ 
      * @remark      This is also accessed from `uninstall.php` so do not remove.
      * @remark      Do not exceed 8 characters as a transient name allows 45 characters or less ( 40 for site transients ) so that md5 (32 characters) can be added.
      */
-    const TRANSIENT_PREFIX         = 'APFL_';
+    const TRANSIENT_PREFIX = 'APFL_';
 
     /**
      * The hook slug used for the prefix of action and filter hook names.
      *
      * @remark      The ending underscore is not necessary.
      */
-    const HOOK_SLUG                = 'admin_page_framework_loader';
+    const HOOK_SLUG = 'admin_page_framework_loader';
 
     /**
      * The text domain slug and its path.
@@ -116,7 +116,7 @@ 
      */
     static public $aAdminPages = array(
         // key => 'page slug'
-        'about'     => 'apfl_about',        // the welcome page
+        'about'     => 'apfl_about', // the welcome page
         'addon'     => 'apfl_addons',
         'tool'      => 'apfl_tools',
         'help'      => 'apfl_contact',
@@ -154,12 +154,12 @@ 
      * @since       3.5.0
      * @return      string
      */
-    public static function getPluginURL( $sRelativePath='' ) {
+    public static function getPluginURL( $sRelativePath = '' ) {
         if ( isset( self::$_sPluginURLCache ) ) {
-            return self::$_sPluginURLCache . $sRelativePath;
+            return self::$_sPluginURLCache.$sRelativePath;
         }
         self::$_sPluginURLCache = trailingslashit( plugins_url( '', self::$sFilePath ) );
-        return self::$_sPluginURLCache . $sRelativePath;
+        return self::$_sPluginURLCache.$sRelativePath;
     }
         /**
          * @since       3.7.9
@@ -189,13 +189,13 @@ 
      * @since       3.5.0
      * @return      void
      */
-    static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
-        if ( ! is_admin() ) {
+    static public function setAdminNotice( $sMessage, $sClassAttribute = 'error' ) {
+        if ( !is_admin() ) {
             return;
         }
-        self::$_aAdminNotices[] = array(
+        self::$_aAdminNotices[ ] = array(
             'message'           => $sMessage,
-            'class_attribute'   => trim( $sClassAttribute ) . ' notice is-dismissible',
+            'class_attribute'   => trim( $sClassAttribute ).' notice is-dismissible',
         );
         add_action( 'admin_notices', array( __CLASS__, '_replyToSetAdminNotice' ) );
     }
@@ -205,12 +205,12 @@ 
          * @return      void
          */
         static public function _replyToSetAdminNotice() {
-            foreach( self::$_aAdminNotices as $_aAdminNotice ) {
-                echo "<div class='" . esc_attr( $_aAdminNotice['class_attribute'] ) . " notice is-dismissible'>"
+            foreach ( self::$_aAdminNotices as $_aAdminNotice ) {
+                echo "<div class='".esc_attr( $_aAdminNotice[ 'class_attribute' ] )." notice is-dismissible'>"
                         ."<p>"
                             . sprintf(
-                                '<strong>%1$s</strong>: ' . $_aAdminNotice['message'],
-                                self::NAME . ' ' . self::VERSION
+                                '<strong>%1$s</strong>: '.$_aAdminNotice[ 'message' ],
+                                self::NAME.' '.self::VERSION
                             )
                         . "</p>"
                     . "</div>";
@@ -222,7 +222,7 @@ 
 AdminPageFrameworkLoader_Registry::setUp( __FILE__ );
 
 // Initial checks. - Do no load if accessed directly, not exiting because the 'uninstall.php' and inclusion list generator will load this file.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( !defined( 'ABSPATH' ) ) {
     return;
 }
 if ( defined( 'DOING_UNINSTALL' ) && DOING_UNINSTALL ) {
@@ -234,8 +234,8 @@ 
 
     $_bFrameworkLoaded = class_exists( 'AdminPageFramework_Registry', false );
     if (
-        ! $_bFrameworkLoaded
-        || ! defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
+        !$_bFrameworkLoaded
+        || !defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
         || version_compare( AdminPageFramework_Registry::VERSION, AdminPageFrameworkLoader_Registry::VERSION, '<' )
     ) {
         AdminPageFrameworkLoader_Registry::setAdminNotice(
@@ -253,18 +253,18 @@ 
 add_action( 'admin_init', 'AdminPageFrameworkLoader_Warning' );
 
 // Include the library file - the development version will be available if you cloned the GitHub repository.
-$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/development/admin-page-framework.php';
+$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/development/admin-page-framework.php';
 $_bDebugMode              = defined( 'WP_DEBUG' ) && WP_DEBUG;
 $_bLoadDevelopmentVersion = $_bDebugMode && file_exists( $_sDevelopmentVersionPath );
 include(
     $_bLoadDevelopmentVersion
         ? $_sDevelopmentVersionPath
-        : AdminPageFrameworkLoader_Registry::$sDirPath . '/library/apf/admin-page-framework.php'
+        : AdminPageFrameworkLoader_Registry::$sDirPath.'/library/apf/admin-page-framework.php'
 );
 
 // Include the framework loader plugin components.
 include( AdminPageFramework_Registry::$aClassFiles[ 'AdminPageFramework_PluginBootstrap' ] );
-include( AdminPageFrameworkLoader_Registry::$sDirPath . '/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
+include( AdminPageFrameworkLoader_Registry::$sDirPath.'/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
 new AdminPageFrameworkLoader_Bootstrap(
     AdminPageFrameworkLoader_Registry::$sFilePath,
     AdminPageFrameworkLoader_Registry::HOOK_SLUG    // hook prefix

development/factory/taxonomy_field/AdminPageFramework_TaxonomyField_Model.php

Spacing +12 added lines, -12 removed lines

@@ -53,8 +53,8 @@ 
         return $this->_getFilteredColumnsByFilterPrefix(
             $this->oUtil->getAsArray( $aColumns ),
             'columns_',
-            isset( $_GET['taxonomy'] )  // in ajax, $_GET is not even set.
-                ? $_GET['taxonomy']
+            isset( $_GET[ 'taxonomy' ] )  // in ajax, $_GET is not even set.
+                ? $_GET[ 'taxonomy' ]
                 : ''
         );
     }
@@ -71,8 +71,8 @@ 
         return $this->_getFilteredColumnsByFilterPrefix(
             $this->oUtil->getAsArray( $aSortableColumns ),
             'sortable_columns_',
-            isset( $_GET['taxonomy'] )  // in ajax, $_GET is not even set.
-                ? $_GET['taxonomy']
+            isset( $_GET[ 'taxonomy' ] )  // in ajax, $_GET is not even set.
+                ? $_GET[ 'taxonomy' ]
                 : ''
         );
     }
@@ -86,7 +86,7 @@ 
             if ( $sTaxonomy ) {
                 $aColumns = $this->oUtil->addAndApplyFilter(
                     $this,
-                    "{$sFilterPrefix}{$_GET['taxonomy']}",
+                    "{$sFilterPrefix}{$_GET[ 'taxonomy' ]}",
                     $aColumns
                 );
             }
@@ -125,7 +125,7 @@ 
      * @param       string|null  $sOptionKey
      * @internal
      */
-    protected function _setOptionArray( $iTermID=null, $sOptionKey=null ) {
+    protected function _setOptionArray( $iTermID = null, $sOptionKey = null ) {
         $this->oForm->aSavedData = $this->_getSavedFormData( $iTermID, $sOptionKey );
     }
         /**
@@ -138,7 +138,7 @@ 
 
             return $this->oUtil->addAndApplyFilter(
                 $this, // the caller factory object
-                'options_' . $this->oProp->sClassName,
+                'options_'.$this->oProp->sClassName,
                 $this->_getSavedTermFormData( $iTermID, $sOptionKey )
                 // @todo maybe pass the term id because the user will not know whihch form data it is
             );
@@ -172,7 +172,7 @@ 
      */
     public function _replyToValidateOptions( $iTermID ) {
 
-        if ( ! $this->_shouldProceedValidation() ) {
+        if ( !$this->_shouldProceedValidation() ) {
             return;
         }
 
@@ -181,7 +181,7 @@ 
         $_aSubmittedFormData    = $this->oForm->getSubmittedData( $_POST );
         $_aSubmittedFormData    = $this->oUtil->addAndApplyFilters(
             $this,
-            'validation_' . $this->oProp->sClassName,
+            'validation_'.$this->oProp->sClassName,
             call_user_func_array(
                 array( $this, 'validate' ), // triggers __call()
                 array( $_aSubmittedFormData, $_aSavedFormData, $this )
@@ -192,7 +192,7 @@ 
 
         // @todo Examine whether it is appropriate to merge recursivly
         // as some fields will have a problem such as select with multiple options.
-        $_aTaxonomyFormData[ $iTermID ]  = $this->oUtil->uniteArrays(
+        $_aTaxonomyFormData[ $iTermID ] = $this->oUtil->uniteArrays(
             $_aSubmittedFormData,
             $_aSavedFormData
         );
@@ -214,10 +214,10 @@ 
          */
         protected function _shouldProceedValidation() {
 
-            if ( ! isset( $_POST[ $this->oProp->sClassHash ] ) ) {
+            if ( !isset( $_POST[ $this->oProp->sClassHash ] ) ) {
                 return false;
             }
-            if ( ! wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {
+            if ( !wp_verify_nonce( $_POST[ $this->oProp->sClassHash ], $this->oProp->sClassHash ) ) {
                 return false;
             }
             return true;

development/factory/widget/_model/AdminPageFramework_Property_widget.php

Spacing +4 added lines, -4 removed lines

@@ -115,7 +115,7 @@ 
      * Stores method names of the `WP_Widget` class, referred when the class is assumed as a WP_Widget subclass.
      * @since       3.8.17
      */
-    public $aWPWidgetMethods    = array();
+    public $aWPWidgetMethods = array();
     /**
      * Stores property names of the `WP_Widget` class, , referred when the class is assumed as a WP_Widget subclass.
      * @since       3.8.17
@@ -126,13 +126,13 @@ 
      * Sets up properties.
      * @since       3.7.0
      */
-    public function __construct( $oCaller, $sCallerPath, $sClassName, $sCapability='manage_options', $sTextDomain='admin-page-framework', $sStructureType='' ) {
+    public function __construct( $oCaller, $sCallerPath, $sClassName, $sCapability = 'manage_options', $sTextDomain = 'admin-page-framework', $sStructureType = '' ) {
 
         // 3.7.0+
-        $this->_sFormRegistrationHook   = 'load_' . $sClassName;
+        $this->_sFormRegistrationHook   = 'load_'.$sClassName;
 
         // 3.7.9+ - setting a custom action hook for admin notices prevents the form object from being instantiated unnecessarily.
-        $this->sSettingNoticeActionHook = 'load_' . $sClassName;
+        $this->sSettingNoticeActionHook = 'load_'.$sClassName;
 
         parent::__construct(
             $oCaller,

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

Spacing +7 added lines, -7 removed lines

@@ -27,7 +27,7 @@ 
      */
     public function __construct( $oProp, $oMsg ) {
 
-        if ( ! $this->_shouldProceed( $oProp ) ) {
+        if ( !$this->_shouldProceed( $oProp ) ) {
             return;
         }
 
@@ -44,7 +44,7 @@ 
          */
         private function _shouldProceed( $oProp ) {
 
-            if ( $oProp->bIsAdminAjax || ! $oProp->bIsAdmin ) {
+            if ( $oProp->bIsAdminAjax || !$oProp->bIsAdmin ) {
                 return false;
             }
             return ( boolean ) $oProp->bShowDebugInfo;
@@ -71,7 +71,7 @@ 
     public function _replyToGetPageLoadInfo( $sFooterHTML ) {
 
         // 3.8.8+ The `bShowDebugInfo` property may be updated by the user during the page load.
-        if ( ! $this->oProp->bShowDebugInfo ) {
+        if ( !$this->oProp->bShowDebugInfo ) {
             return $sFooterHTML;
         }
 
@@ -105,10 +105,10 @@ 
             $_sInitialMemoryUsage = $this->getReadableBytes( $_nInitialMemoryUsage );
             return "<div id='admin-page-framework-page-load-stats'>"
                     . "<ul>"
-                        . "<li>" . sprintf( $this->oMsg->get( 'queries_in_seconds' ), $_nQueryCount, $_nSeconds ) . "</li>"
-                        . "<li>" . sprintf( $this->oMsg->get( 'out_of_x_memory_used' ), $_sMemoryUsage, $_sMemoryLimit, round( ( $_nMemoryUsage / $_nMemoryLimit ), 2 ) * 100 . '%' ) . "</li>"
-                        . "<li>" . sprintf( $this->oMsg->get( 'peak_memory_usage' ), $_sMemoryPeakUsage ) . "</li>"
-                        . "<li>" . sprintf( $this->oMsg->get( 'initial_memory_usage' ), $_sInitialMemoryUsage ) . "</li>"
+                        . "<li>".sprintf( $this->oMsg->get( 'queries_in_seconds' ), $_nQueryCount, $_nSeconds )."</li>"
+                        . "<li>".sprintf( $this->oMsg->get( 'out_of_x_memory_used' ), $_sMemoryUsage, $_sMemoryLimit, round( ( $_nMemoryUsage / $_nMemoryLimit ), 2 ) * 100.'%' )."</li>"
+                        . "<li>".sprintf( $this->oMsg->get( 'peak_memory_usage' ), $_sMemoryPeakUsage )."</li>"
+                        . "<li>".sprintf( $this->oMsg->get( 'initial_memory_usage' ), $_sInitialMemoryUsage )."</li>"
                     . "</ul>"
                 . "</div>";