michaeluno / admin-page-framework

development/factory/_common/utility/AdminPageFramework_Debug.php

Doc Comments +5 added lines

@@ -140,6 +140,7 @@ 
          * Determines the log file path.
          * @since       3.5.3 
          * @internal    
+         * @param string|null $bsFilePath
          * @return      string      The path of the file to log the contents.
          */
         static private function _getLogFilePath( $bsFilePath, $sCallerClass ) {
@@ -194,6 +195,7 @@ 
              * Returns a length of a value.
              * @since       3.5.3
              * @internal
+             * @param string $sVariableType
              * @return      integer|null        For string or integer, the string length. For array, the element lengths. For other types, null.
              */
             static private function _getValueLength( $mValue, $sVariableType ) {
@@ -211,6 +213,8 @@ 
          * Returns the heading part of a log item.
          * @since       3.5.3
          * @internal
+         * @param double $fCurrentTimeStamp
+         * @param double $nElapsed
          * @return      string      the heading part of a log item.
          */
         static private function _getLogHeadingLine( $fCurrentTimeStamp, $nElapsed, $sCallerClass, $sCallerFunction ) {
@@ -418,6 +422,7 @@ 
             }
         /**
          * Performs `array_map()` recursively.
+         * @param string[] $oCallable
          * @return      array.
          * @since       3.8.9
          */

Spacing +42 added lines, -42 removed lines

@@ -33,7 +33,7 @@ 
      * @param       string          $sFilePath      The file path for a log file.
      * @return      void
      */
-    static public function dump( $asArray, $sFilePath=null ) {
+    static public function dump( $asArray, $sFilePath = null ) {
         echo self::get( $asArray, $sFilePath );
     }    
         /**
@@ -44,7 +44,7 @@ 
          * @since unknown
          * @deprecated      3.2.0
          */
-        static public function dumpArray( $asArray, $sFilePath=null ) {
+        static public function dumpArray( $asArray, $sFilePath = null ) {
             self::dump( $asArray, $sFilePath );
         }    
         
@@ -59,7 +59,7 @@ 
      * @param       string          $sFilePath      The file path for a log file.
      * @param       boolean         $bEscape        Whether to escape characters.
      */
-    static public function get( $asArray, $sFilePath=null, $bEscape=true ) {
+    static public function get( $asArray, $sFilePath = null, $bEscape = true ) {
 
         if ( $sFilePath ) {
             self::log( $asArray, $sFilePath );     
@@ -81,8 +81,8 @@ 
          * @since       3.0.0 Changed the $bEncloseInTag parameter to bEscape.
          * @deprecated  3.2.0
          */
-        static public function getArray( $asArray, $sFilePath=null, $bEscape=true ) {
-            self::showDeprecationNotice( __CLASS__ .'::' . __FUNCTION__,  __CLASS__ .'::get()' );
+        static public function getArray( $asArray, $sFilePath = null, $bEscape = true ) {
+            self::showDeprecationNotice( __CLASS__.'::'.__FUNCTION__, __CLASS__.'::get()' );
             return self::get( $asArray, $sFilePath, $bEscape );
         }      
             
@@ -104,18 +104,18 @@ 
      * @param       string      $sFilePath      The log file path.
      * @return      void
      **/
-    static public function log( $mValue, $sFilePath=null ) {
+    static public function log( $mValue, $sFilePath = null ) {
                 
         static $_fPreviousTimeStamp = 0;
         
         $_oCallerInfo       = debug_backtrace();
         $_sCallerFunction   = self::getElement(
-            $_oCallerInfo,  // subject array
+            $_oCallerInfo, // subject array
             array( 1, 'function' ), // key
             ''      // default
         );                        
-        $_sCallerClass      = self::getElement(
-            $_oCallerInfo,  // subject array
+        $_sCallerClass = self::getElement(
+            $_oCallerInfo, // subject array
             array( 1, 'class' ), // key
             ''      // default
         );           
@@ -125,10 +125,10 @@ 
             self::_getLogFilePath( $sFilePath, $_sCallerClass ), 
             self::_getLogHeadingLine( 
                 $_fCurrentTimeStamp,
-                round( $_fCurrentTimeStamp - $_fPreviousTimeStamp, 3 ),     // elapsed time
+                round( $_fCurrentTimeStamp - $_fPreviousTimeStamp, 3 ), // elapsed time
                 $_sCallerClass,
                 $_sCallerFunction
-            ) . PHP_EOL
+            ).PHP_EOL
             . self::_getLogContents( $mValue ),
             FILE_APPEND 
         );     
@@ -150,9 +150,9 @@ 
             }
             // Return a generated default log path.
             if ( true === $bsFilePath ) {
-                return WP_CONTENT_DIR . DIRECTORY_SEPARATOR . basename( get_class() ) . '_' . date( "Ymd" ) . '.log';
+                return WP_CONTENT_DIR.DIRECTORY_SEPARATOR.basename( get_class() ).'_'.date( "Ymd" ).'.log';
             }
-            return WP_CONTENT_DIR . DIRECTORY_SEPARATOR . basename( get_class() ) . '_' . basename( $sCallerClass ) . '_' . date( "Ymd" ) . '.log';
+            return WP_CONTENT_DIR.DIRECTORY_SEPARATOR.basename( get_class() ).'_'.basename( $sCallerClass ).'_'.date( "Ymd" ).'.log';
             
         }
             /**
@@ -161,7 +161,7 @@ 
              * @internal
              */
             static private function _createFile( $sFilePath ) {
-                if ( ! $sFilePath || true === $sFilePath ) {
+                if ( !$sFilePath || true === $sFilePath ) {
                     return false;
                 }
                 if ( file_exists( $sFilePath ) ) {
@@ -184,10 +184,10 @@ 
             $_iLengths  = self::_getValueLength( $mValue, $_sType );
             return '(' 
                 . $_sType
-                . ( null !== $_iLengths ? ', length: ' . $_iLengths : '' )
+                . ( null !== $_iLengths ? ', length: '.$_iLengths : '' )
             . ') '
             . self::getAsString( $mValue ) 
-            . PHP_EOL . PHP_EOL;
+            . PHP_EOL.PHP_EOL;
         
         }      
             /**
@@ -228,11 +228,11 @@ 
             $_nMicroseconds     = str_pad( round( ( $_nNow - floor( $_nNow ) ) * 10000 ), 4, '0' );
             
             $_aOutput           = array(
-                date( "Y/m/d H:i:s", $_nNow ) . '.' . $_nMicroseconds,
+                date( "Y/m/d H:i:s", $_nNow ).'.'.$_nMicroseconds,
                 self::_getFormattedElapsedTime( $nElapsed ),
                 $_iPageLoadID,
                 AdminPageFramework_Registry::getVersion(),
-                $sCallerClass . '::' . $sCallerFunction,
+                $sCallerClass.'::'.$sCallerFunction,
                 current_filter(),
                 self::getCurrentURL(),
             );
@@ -250,22 +250,22 @@ 
                 $_aElapsedParts     = explode( ".", ( string ) $nElapsed );
                 $_sElapsedFloat     = str_pad(
                     self::getElement(
-                        $_aElapsedParts,  // subject array
+                        $_aElapsedParts, // subject array
                         1, // key
                         0      // default
                     ),      
                     3, 
                     '0'
                 );
-                $_sElapsed          = self::getElement(
-                    $_aElapsedParts,  // subject array
-                    0,  // key
+                $_sElapsed = self::getElement(
+                    $_aElapsedParts, // subject array
+                    0, // key
                     0   // default
                 );                                   
-                $_sElapsed          = strlen( $_sElapsed ) > 1 
-                    ? '+' . substr( $_sElapsed, -1, 2 ) 
-                    : ' ' . $_sElapsed;
-                return $_sElapsed . '.' . $_sElapsedFloat;
+                $_sElapsed = strlen( $_sElapsed ) > 1 
+                    ? '+'.substr( $_sElapsed, -1, 2 ) 
+                    : ' '.$_sElapsed;
+                return $_sElapsed.'.'.$_sElapsedFloat;
             
             }
         /**
@@ -275,8 +275,8 @@ 
          * @since       3.0.3   Changed the default log location and file name.
          * @deprecated  3.1.0   Use the `log()` method instead.
          */
-        static public function logArray( $asArray, $sFilePath=null ) {
-            self::showDeprecationNotice( __CLASS__ .'::' . __FUNCTION__,  __CLASS__ .'::log()' );
+        static public function logArray( $asArray, $sFilePath = null ) {
+            self::showDeprecationNotice( __CLASS__.'::'.__FUNCTION__, __CLASS__.'::log()' );
             self::log( $asArray, $sFilePath );     
         }      
         
@@ -300,20 +300,20 @@ 
          */
         static private function _getLegibleCallable( $asoCallable ) {
             
-            if ( ! is_callable( $asoCallable ) ) {
+            if ( !is_callable( $asoCallable ) ) {
                 return $asoCallable;
             }
             if ( is_string( $asoCallable ) ) {
-                return '(callable) ' . $asoCallable;
+                return '(callable) '.$asoCallable;
             }
             if ( is_object( $asoCallable ) ) {
-                return '(callable) ' . get_class( $asoCallable );
+                return '(callable) '.get_class( $asoCallable );
             }
             $_sSubject = is_object( $asoCallable[ 0 ] )
                 ? get_class( $asoCallable[ 0 ] )
                 : ( string ) $asoCallable[ 0 ];
 
-            return '(callable) ' . $_sSubject . '::' . ( string ) $asoCallable[ 1 ];
+            return '(callable) '.$_sSubject.'::'.( string ) $asoCallable[ 1 ];
             
         }
         
@@ -323,7 +323,7 @@ 
          */
         static public function _getLegibleObject( $oObject ) {
             
-            if ( ! is_object( $oObject ) ) {
+            if ( !is_object( $oObject ) ) {
                 return $oObject;
             }
             
@@ -331,8 +331,8 @@ 
                 return ( string ) $oObject;
             }
             
-            return '(object) ' . get_class( $oObject ) . ' ' 
-                . count( get_object_vars( $oObject ) ) . ' properties.';
+            return '(object) '.get_class( $oObject ).' ' 
+                . count( get_object_vars( $oObject ) ).' properties.';
             
         }
         
@@ -342,7 +342,7 @@ 
          */
         static public function _getLegibleArray( $aArray ) {
             
-            if ( ! is_array( $aArray ) ) {
+            if ( !is_array( $aArray ) ) {
                 return $aArray;
             }
             return self::_getArrayMappedRecursive( 
@@ -360,7 +360,7 @@ 
      * @return      array
      * @internal
      */
-    static public function getSliceByDepth( array $aSubject, $iDepth=0 ) {
+    static public function getSliceByDepth( array $aSubject, $iDepth = 0 ) {
 
         foreach ( $aSubject as $_sKey => $_vValue ) {
             if ( is_array( $_vValue ) ) {
@@ -384,13 +384,13 @@ 
         static private function _getLegibleValue( $mItem ) {
             
             if ( is_object( $mItem ) ) {
-                return '(object) ' . get_class( $mItem );
+                return '(object) '.get_class( $mItem );
             }
             if ( is_null( $mItem ) ) {
                 return '(null)';
             }
             if ( is_bool( $mItem ) ) {
-                return '(boolean) ' . ( $mItem ? 'true' : 'false' );
+                return '(boolean) '.( $mItem ? 'true' : 'false' );
             }
             // At this point, the value is a scalar.
             return self::_getLegibleString( $mItem );
@@ -400,7 +400,7 @@ 
              * @return      string
              * @since       3.8.9
              */
-            static private function _getLegibleString( $sScalar, $iCharLimit=300 ) {
+            static private function _getLegibleString( $sScalar, $iCharLimit = 300 ) {
                 
                 static $_iMBSupport;
                 $_iMBSupport = isset( $_iMBSupport ) ? $_iMBSupport : function_exists( 'mb_strlen' );      
@@ -409,9 +409,9 @@ 
                 
                 $_iCharLength = call_user_func_array( $_aStrLenMethod[ $_iMBSupport ], array( $sScalar ) );
                 if ( $_iCharLength <= $iCharLimit ) {
-                    return '(' . gettype( $sScalar ) . ') ' . $sScalar;
+                    return '('.gettype( $sScalar ).') '.$sScalar;
                 }
-                return '(' . gettype( $sScalar ) . ') '
+                return '('.gettype( $sScalar ).') '
                     . call_user_func_array( $_aSubstrMethod[ $_iMBSupport ], array( $sScalar, 0, $iCharLimit ) ) 
                     . '...';
                     

Braces +1 added lines, -1 removed lines

@@ -435,7 +435,7 @@
              * @return      mixed       A modified value.
              * @since       3.8.9
              */
-            static private function _getArrayMappedNested( $mItem ) {            
+            static private function _getArrayMappedNested( $mItem ) {
                 return is_array( $mItem ) 
                     ? array_map( array( __CLASS__, '_getArrayMappedNested' ), $mItem ) 
                     : call_user_func( self::$oCurrentCallable, $mItem );            

admin-page-framework-loader.php

Spacing +22 added lines, -22 removed lines

@@ -16,9 +16,9 @@ 
  */
 class AdminPageFrameworkLoader_Registry_Base {
 
-    const VERSION        = '3.8.9b01';    // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
+    const VERSION        = '3.8.9b01'; // <--- 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' ) ) {
@@ -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