michaeluno / admin-page-framework

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/utility/base_utility/AdminPageFramework_Utility_String.php

Spacing +10 added lines, -10 removed lines

@@ -35,10 +35,10 @@ 
      * @since       3.8.4   When non-true value is passed, `0px` will be returned.
      * @return      string
      */
-    static public function getLengthSanitized( $sLength, $sUnit='px' ) {
+    static public function getLengthSanitized( $sLength, $sUnit = 'px' ) {
         $sLength = $sLength ? $sLength : 0;
         return is_numeric( $sLength )
-            ? $sLength . $sUnit
+            ? $sLength.$sUnit
             : $sLength;
     }
 
@@ -80,9 +80,9 @@ 
      * @since       3.8.11      Renamed from `fixNumber()`.
      * @return      string|integer      A numeric value will be returned.
      */
-    static public function getNumberFixed( $nToFix, $nDefault, $nMin='', $nMax='' ) {
+    static public function getNumberFixed( $nToFix, $nDefault, $nMin = '', $nMax = '' ) {
 
-        if ( ! is_numeric( trim( $nToFix ) ) ) {
+        if ( !is_numeric( trim( $nToFix ) ) ) {
             return $nDefault;
         }
         if ( $nMin !== '' && $nToFix < $nMin ) {
@@ -100,7 +100,7 @@ 
          * @return      string|integer      A numeric value will be returned.
          * @deprecated  3.8.11
          */
-        static public function fixNumber( $nToFix, $nDefault, $nMin='', $nMax='' ) {
+        static public function fixNumber( $nToFix, $nDefault, $nMin = '', $nMax = '' ) {
             return self::getNumberFixed( $nToFix, $nDefault, $nMin, $nMax );
         }
 
@@ -113,7 +113,7 @@ 
      */
     static public function getCSSMinified( $sCSSRules ) {
         return str_replace(
-            array( "\r\n", "\r", "\n", "\t", '  ', '    ', '    '),  // remove line breaks, tab, and white sspaces.
+            array( "\r\n", "\r", "\n", "\t", '  ', '    ', '    ' ), // remove line breaks, tab, and white sspaces.
             '',
             preg_replace( '!/\*[^*]*\*+([^/][^*]*\*+)*/!', '', $sCSSRules ) // remove comments
         );
@@ -150,8 +150,8 @@ 
      */
     static public function getNumberOfReadableSize( $nSize ) {
 
-        $_nReturn     = substr( $nSize, 0, -1 );
-        switch( strtoupper( substr( $nSize, -1 ) ) ) {
+        $_nReturn = substr( $nSize, 0, -1 );
+        switch ( strtoupper( substr( $nSize, -1 ) ) ) {
             case 'P':
                 $_nReturn *= 1024;
             case 'T':
@@ -175,13 +175,13 @@ 
      * @since       3.8.26 Added the `$iRoundPrecision` parameter.
      * @return      string
      */
-    static public function getReadableBytes( $nBytes, $iRoundPrecision=2 ) {
+    static public function getReadableBytes( $nBytes, $iRoundPrecision = 2 ) {
         $_aUnits = array( 0 => 'B', 1 => 'kB', 2 => 'MB', 3 => 'GB' );
         $_nLog   = log( $nBytes, 1024 );
         $_iPower = ( int ) $_nLog;
         $_ifSize = pow( 1024, $_nLog - $_iPower );
         $_ifSize = round( $_ifSize, $iRoundPrecision );
-        return $_ifSize . $_aUnits[ $_iPower ];
+        return $_ifSize.$_aUnits[ $_iPower ];
     }
 
     /**

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>";
 

development/factory/admin_page/AdminPageFramework_Controller_Form.php

Spacing +17 added lines, -17 removed lines

@@ -84,7 +84,7 @@ 
      */
     public function addSettingSections( /* $aSection1, $aSection2=null, $_and_more=null */ ) {
 
-        foreach( func_get_args() as $asSection ) {
+        foreach ( func_get_args() as $asSection ) {
             $this->addSettingSection( $asSection );
         }
 
@@ -107,7 +107,7 @@ 
      */
     public function addSettingSection( $asSection ) {
 
-        if ( ! is_array( $asSection ) ) {
+        if ( !is_array( $asSection ) ) {
             $this->_sTargetPageSlug = is_string( $asSection )
                 ? $asSection
                 : $this->_sTargetPageSlug;
@@ -129,10 +129,10 @@ 
             )
         );
 
-        $aSection[ 'section_tab_slug' ]   = $this->oUtil->sanitizeSlug( $aSection[ 'section_tab_slug' ] );
+        $aSection[ 'section_tab_slug' ] = $this->oUtil->sanitizeSlug( $aSection[ 'section_tab_slug' ] );
 
         // A page slug is required.
-        if ( ! $aSection[ 'page_slug' ] ) {
+        if ( !$aSection[ 'page_slug' ] ) {
             return;
         }
         $this->oForm->addSection( $aSection );
@@ -148,8 +148,8 @@ 
         private function _getTargetPageSlug( $aSection ) {
 
             $_sTargetPageSlug = $this->oUtil->getElement(
-                $aSection,      // subject
-                'page_slug',    // key
+                $aSection, // subject
+                'page_slug', // key
                 $this->_sTargetPageSlug     // default
             );
 
@@ -169,8 +169,8 @@ 
          */
         private function _getTargetTabSlug( $aSection ) {
             $_sTargetTabSlug = $this->oUtil->getElement(
-                $aSection,              // subject
-                'tab_slug',             // key
+                $aSection, // subject
+                'tab_slug', // key
                 $this->_sTargetTabSlug  // default
             );
             $_sTargetTabSlug = $_sTargetTabSlug
@@ -199,7 +199,7 @@ 
     * @return       void
     */
     public function removeSettingSections( /* $sSectionID1=null, $sSectionID2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $_sSectionID ) {
+        foreach ( func_get_args() as $_sSectionID ) {
             $this->oForm->removeSection( $_sSectionID );
         }
     }
@@ -257,7 +257,7 @@ 
      * @remark      The actual registration will be performed in the <em>_replyToRegisterSettings()</em> method with the <em>admin_menu</em> hook.
      */
     public function addSettingFields( /* $aField1, $aField2=null, $_and_more=null */ ) {
-        foreach( func_get_args() as $aField ) {
+        foreach ( func_get_args() as $aField ) {
             $this->addSettingField( $aField );
         }
     }
@@ -294,8 +294,8 @@ 
     * @param        string      $_and_more      (optional) add more field IDs to the next parameters as many as necessary.
     * @return void
     */
-    public function removeSettingFields( $sFieldID1, $sFieldID2=null, $_and_more=null ) {
-        foreach( func_get_args() as $_sFieldID ) {
+    public function removeSettingFields( $sFieldID1, $sFieldID2 = null, $_and_more = null ) {
+        foreach ( func_get_args() as $_sFieldID ) {
             $this->oForm->removeField( $_sFieldID );
         }
     }
@@ -342,7 +342,7 @@ 
             $this->oProp->sOptionKey,
             empty( $_aParams )
                 ? null                  // will return the entire options array
-                : $_aDimensionalKeys,   // dimensional keys
+                : $_aDimensionalKeys, // dimensional keys
             $_mDefault, // default
             $this->getSavedOptions() + $this->oForm->getDefaultFormValues()
         );
@@ -361,21 +361,21 @@ 
      * If the user has not submitted the form, the framework will try to return the default value set in the field definition array.
      * @deprecated  3.3.0
      */
-    public function getFieldValue( $sFieldID, $sSectionID='' ) {
+    public function getFieldValue( $sFieldID, $sSectionID = '' ) {
 
         $this->oUtil->showDeprecationNotice(
-            'The method,' . __METHOD__ . ',', // deprecated item
+            'The method,'.__METHOD__.',', // deprecated item
             'getValue()' // alternative
         );
 
         $_aOptions = $this->oUtil->uniteArrays( $this->oProp->aOptions, $this->oForm->getDefaultFormValues() );
         /* If it's saved, return it */
-        if ( ! $sSectionID ) {
+        if ( !$sSectionID ) {
             if ( array_key_exists( $sFieldID, $_aOptions ) ) {
                 return $_aOptions[ $sFieldID ];
             }
             // loop through section elements
-            foreach( $_aOptions as $aOptions ) {
+            foreach ( $_aOptions as $aOptions ) {
                 if ( array_key_exists( $sFieldID, $aOptions ) ) {
                     return $aOptions[ $sFieldID ];
                 }