Passed
Branch dev (b1003b)
by Michael
21:22
created
development/utility/plugin_bootstrap/AdminPageFramework_PluginBootstrap.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
      * @param string        $sSetUpHook             The action hook name for the setUp callback. Default 'plugins_loaded'.
128 128
      * @param integer|float $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
129 129
      */
130
-    public function __construct( $sPluginFilePath, $sPluginHookPrefix='', $sSetUpHook='plugins_loaded', $iPriority=10 ) {
130
+    public function __construct( $sPluginFilePath, $sPluginHookPrefix = '', $sSetUpHook = 'plugins_loaded', $iPriority = 10 ) {
131 131
 
132 132
         // Check if it has been loaded.
133 133
         if ( $this->_hasLoaded() ) {
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
         register_deactivation_hook( $this->sFilePath, array( $this, 'replyToPluginDeactivation' ) );
164 164
 
165 165
         // Schedule to load plugin specific components.
166
-        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) )  {
166
+        if ( !$this->sSetUpHook || did_action( $this->sSetUpHook ) ) {
167 167
             $this->_replyToLoadPluginComponents();
168 168
         } else {
169 169
             add_action( $this->sSetUpHook, array( $this, '_replyToLoadPluginComponents' ), $this->iPriority );
@@ -204,14 +204,14 @@  discard block
 block discarded – undo
204 204
         protected function _registerClasses() {
205 205
 
206 206
             // This class should be used in the framework bootstrap so disabling the auto-load option for performance.
207
-            if ( ! class_exists( 'AdminPageFramework_RegisterClasses', false ) ) {
207
+            if ( !class_exists( 'AdminPageFramework_RegisterClasses', false ) ) {
208 208
                 return;
209 209
             }
210 210
 
211 211
             // Register classes
212 212
             new AdminPageFramework_RegisterClasses(
213
-                $this->getScanningDirs(),   // scanning directory paths
214
-                array(),                    // autoloader options
213
+                $this->getScanningDirs(), // scanning directory paths
214
+                array(), // autoloader options
215 215
                 $this->getClasses()         // pre-generated class list
216 216
             );
217 217
 
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -163,7 +163,7 @@
 block discarded – undo
163 163
         register_deactivation_hook( $this->sFilePath, array( $this, 'replyToPluginDeactivation' ) );
164 164
 
165 165
         // Schedule to load plugin specific components.
166
-        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) )  {
166
+        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) ) {
167 167
             $this->_replyToLoadPluginComponents();
168 168
         } else {
169 169
             add_action( $this->sSetUpHook, array( $this, '_replyToLoadPluginComponents' ), $this->iPriority );
Please login to merge, or discard this patch.
development/cli/AdminPageFramework_ClassMapHeader.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -11,7 +11,7 @@
 block discarded – undo
11 11
  * If accessed from a console, include the registry class to load 'AdminPageFramework_Registry_Base'.
12 12
  */
13 13
 if ( php_sapi_name() === 'cli' ) {
14
-    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ) . '/admin-page-framework.php';
14
+    $_sFrameworkFilePath = dirname( dirname( __FILE__ ) ).'/admin-page-framework.php';
15 15
     if ( file_exists( $_sFrameworkFilePath ) ) {
16 16
         include_once( $_sFrameworkFilePath );
17 17
     }
Please login to merge, or discard this patch.
development/admin-page-framework.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@  discard block
 block discarded – undo
10 10
  * @package     AdminPageFramework
11 11
  */
12 12
 
13
-if ( ! class_exists( 'AdminPageFramework_Registry', false ) ) :
13
+if ( !class_exists( 'AdminPageFramework_Registry', false ) ) :
14 14
 /**
15 15
  * Facilitates WordPress plugin and theme development.
16 16
  *
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 final class AdminPageFramework_Registry extends AdminPageFramework_Registry_Base {
62 62
 
63 63
     const TEXT_DOMAIN        = 'admin-page-framework';
64
-    const TEXT_DOMAIN_PATH   = '/language';  // not used at the moment
64
+    const TEXT_DOMAIN_PATH   = '/language'; // not used at the moment
65 65
 
66 66
     /**
67 67
      * Indicates whether the framework is loaded from the minified version or not.
@@ -108,10 +108,10 @@  discard block
 block discarded – undo
108 108
     /**
109 109
      * Sets up static properties.
110 110
      */
111
-    static public function setUp( $sFilePath=__FILE__ ) {
111
+    static public function setUp( $sFilePath = __FILE__ ) {
112 112
         self::$sFilePath             = $sFilePath;
113 113
         self::$sDirPath              = dirname( self::$sFilePath );
114
-        self::$sClassMapPath         = self::$sDirPath . '/admin-page-framework-class-map.php';
114
+        self::$sClassMapPath         = self::$sDirPath.'/admin-page-framework-class-map.php';
115 115
         self::$aClassFiles           = include( self::$sClassMapPath );
116 116
         self::$sAutoLoaderPath       = isset( self::$aClassFiles[ 'AdminPageFramework_RegisterClasses' ] )
117 117
             ? self::$aClassFiles[ 'AdminPageFramework_RegisterClasses' ]
@@ -128,11 +128,11 @@  discard block
 block discarded – undo
128 128
      */
129 129
     static public function getVersion() {
130 130
 
131
-        if ( ! isset( self::$sAutoLoaderPath ) ) {
132
-            trigger_error( self::NAME . ': ' . ' : ' . sprintf( __( 'The method is called too early. Perform <code>%2$s</code> earlier.', 'admin-page-framework' ), __METHOD__, 'setUp()' ), E_USER_WARNING );
131
+        if ( !isset( self::$sAutoLoaderPath ) ) {
132
+            trigger_error( self::NAME.': '.' : '.sprintf( __( 'The method is called too early. Perform <code>%2$s</code> earlier.', 'admin-page-framework' ), __METHOD__, 'setUp()' ), E_USER_WARNING );
133 133
             return self::VERSION;
134 134
         }
135
-        $_aMinifiedVersionSuffix    = array(
135
+        $_aMinifiedVersionSuffix = array(
136 136
             0 => '',
137 137
             1 => '.min',
138 138
         );
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
 }
162 162
 endif;
163 163
 
164
-if ( ! class_exists( 'AdminPageFramework_Bootstrap', false ) ) :
164
+if ( !class_exists( 'AdminPageFramework_Bootstrap', false ) ) :
165 165
 /**
166 166
  * Loads the Admin Page Framework library.
167 167
  *
@@ -182,7 +182,7 @@  discard block
 block discarded – undo
182 182
     public function __construct( $sLibraryPath ) {
183 183
 
184 184
         // Singleton
185
-        if ( ! $this->___isLoadable() ) {
185
+        if ( !$this->___isLoadable() ) {
186 186
             return;
187 187
         }
188 188
 
Please login to merge, or discard this patch.
development/factory/_common/utility/debug/AdminPageFramework_Debug.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
      * @param       integer         $iArrayDepthLimit
36 36
      * @return      void
37 37
      */
38
-    static public function dump( $asArray, $sFilePath=null, $bStackTrace=false, $iStringLengthLimit=0, $iArrayDepthLimit=0 ) {
38
+    static public function dump( $asArray, $sFilePath = null, $bStackTrace = false, $iStringLengthLimit = 0, $iArrayDepthLimit = 0 ) {
39 39
         echo self::get( $asArray, $sFilePath, true, $bStackTrace, $iStringLengthLimit, $iArrayDepthLimit );
40 40
     }
41 41
 
@@ -44,12 +44,12 @@  discard block
 block discarded – undo
44 44
      * @since       3.8.9
45 45
      * @return      string
46 46
      */
47
-    static public function getDetails( $mValue, $bEscape=true, $bStackTrace=false, $iStringLengthLimit=0, $iArrayDepthLimit=0 ) {
47
+    static public function getDetails( $mValue, $bEscape = true, $bStackTrace = false, $iStringLengthLimit = 0, $iArrayDepthLimit = 0 ) {
48 48
         $_sValueWithDetails = self::_getArrayRepresentationSanitized(
49 49
             self::_getLegibleDetails( $mValue, $iStringLengthLimit, $iArrayDepthLimit )
50 50
         );
51 51
         $_sValueWithDetails = $bStackTrace
52
-            ? $_sValueWithDetails . PHP_EOL . self::getStackTrace()
52
+            ? $_sValueWithDetails.PHP_EOL.self::getStackTrace()
53 53
             : $_sValueWithDetails;
54 54
         return $bEscape
55 55
             ? "<pre class='dump-array'>"
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
      * @param       integer         $iArrayDepthLimit
76 76
      * @return      string
77 77
      */
78
-    static public function get( $asArray, $sFilePath=null, $bEscape=true, $bStackTrace=false, $iStringLengthLimit=0, $iArrayDepthLimit=0 ) {
78
+    static public function get( $asArray, $sFilePath = null, $bEscape = true, $bStackTrace = false, $iStringLengthLimit = 0, $iArrayDepthLimit = 0 ) {
79 79
 
80 80
         if ( $sFilePath ) {
81 81
             self::log( $asArray, $sFilePath );
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
      * @param       integer     $iArrayDepthLimit   The depth limit for arrays.*
118 118
      * @return      void
119 119
      **/
120
-    static public function log( $mValue, $sFilePath=null, $bStackTrace=false, $iTrace=0, $iStringLengthLimit=99999, $iArrayDepthLimit=50 ) {
120
+    static public function log( $mValue, $sFilePath = null, $bStackTrace = false, $iTrace = 0, $iStringLengthLimit = 99999, $iArrayDepthLimit = 50 ) {
121 121
         self::_log( $mValue, $sFilePath, $bStackTrace, $iTrace, $iStringLengthLimit, $iArrayDepthLimit );
122 122
     }
123 123
 
@@ -131,8 +131,8 @@  discard block
 block discarded – undo
131 131
      * @since unknown
132 132
      * @deprecated      3.2.0
133 133
      */
134
-    static public function dumpArray( $asArray, $sFilePath=null ) {
135
-        self::showDeprecationNotice( 'AdminPageFramework_Debug::' . __FUNCTION__, 'AdminPageFramework_Debug::dump()' );
134
+    static public function dumpArray( $asArray, $sFilePath = null ) {
135
+        self::showDeprecationNotice( 'AdminPageFramework_Debug::'.__FUNCTION__, 'AdminPageFramework_Debug::dump()' );
136 136
         AdminPageFramework_Debug::dump( $asArray, $sFilePath );
137 137
     }
138 138
 
@@ -145,8 +145,8 @@  discard block
 block discarded – undo
145 145
      * @since       3.0.0 Changed the $bEncloseInTag parameter to bEscape.
146 146
      * @deprecated  3.2.0
147 147
      */
148
-    static public function getArray( $asArray, $sFilePath=null, $bEscape=true ) {
149
-        self::showDeprecationNotice( 'AdminPageFramework_Debug::' . __FUNCTION__, 'AdminPageFramework_Debug::get()' );
148
+    static public function getArray( $asArray, $sFilePath = null, $bEscape = true ) {
149
+        self::showDeprecationNotice( 'AdminPageFramework_Debug::'.__FUNCTION__, 'AdminPageFramework_Debug::get()' );
150 150
         return AdminPageFramework_Debug::get( $asArray, $sFilePath, $bEscape );
151 151
     }
152 152
 
@@ -157,8 +157,8 @@  discard block
 block discarded – undo
157 157
      * @since       3.0.3   Changed the default log location and file name.
158 158
      * @deprecated  3.1.0   Use the `log()` method instead.
159 159
      */
160
-    static public function logArray( $asArray, $sFilePath=null ) {
161
-        self::showDeprecationNotice( 'AdminPageFramework_Debug::' . __FUNCTION__, 'AdminPageFramework_Debug::log()' );
160
+    static public function logArray( $asArray, $sFilePath = null ) {
161
+        self::showDeprecationNotice( 'AdminPageFramework_Debug::'.__FUNCTION__, 'AdminPageFramework_Debug::log()' );
162 162
         AdminPageFramework_Debug::log( $asArray, $sFilePath );
163 163
     }
164 164
 
Please login to merge, or discard this patch.
admin-page-framework-loader.php 1 patch
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -16,9 +16,9 @@  discard block
 block discarded – undo
16 16
  */
17 17
 class AdminPageFrameworkLoader_Registry_Base {
18 18
 
19
-    const VERSION        = '3.9.0b18';    // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
19
+    const VERSION        = '3.9.0b18'; // <--- DON'T FORGET TO CHANGE THIS AS WELL!!
20 20
     const NAME           = 'Admin Page Framework - Loader'; // the name is not 'Admin Page Framework' because warning messages gets confusing.
21
-    const SHORTNAME      = 'Admin Page Framework';  // used for a menu title etc.
21
+    const SHORTNAME      = 'Admin Page Framework'; // used for a menu title etc.
22 22
     const DESCRIPTION    = 'Loads Admin Page Framework which facilitates WordPress plugin and theme development.';
23 23
     const URI            = 'https://admin-page-framework.michaeluno.jp/';
24 24
     const AUTHOR         = 'miunosoft (Michael Uno)';
@@ -55,14 +55,14 @@  discard block
 block discarded – undo
55 55
      * @remark      This is also accessed from `uninstall.php` so do not remove.
56 56
      * @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.
57 57
      */
58
-    const TRANSIENT_PREFIX         = 'APFL_';
58
+    const TRANSIENT_PREFIX = 'APFL_';
59 59
 
60 60
     /**
61 61
      * The hook slug used for the prefix of action and filter hook names.
62 62
      *
63 63
      * @remark      The ending underscore is not necessary.
64 64
      */
65
-    const HOOK_SLUG                = 'admin_page_framework_loader';
65
+    const HOOK_SLUG = 'admin_page_framework_loader';
66 66
 
67 67
     /**
68 68
      * The text domain slug and its path.
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
      */
117 117
     static public $aAdminPages = array(
118 118
         // key => 'page slug'
119
-        'about'     => 'apfl_about',        // the welcome page
119
+        'about'     => 'apfl_about', // the welcome page
120 120
         'addon'     => 'apfl_addons',
121 121
         'tool'      => 'apfl_tools',
122 122
         'help'      => 'apfl_contact',
@@ -154,12 +154,12 @@  discard block
 block discarded – undo
154 154
      * @since       3.5.0
155 155
      * @return      string
156 156
      */
157
-    public static function getPluginURL( $sRelativePath='' ) {
157
+    public static function getPluginURL( $sRelativePath = '' ) {
158 158
         if ( isset( self::$_sPluginURLCache ) ) {
159
-            return self::$_sPluginURLCache . $sRelativePath;
159
+            return self::$_sPluginURLCache.$sRelativePath;
160 160
         }
161 161
         self::$_sPluginURLCache = trailingslashit( plugins_url( '', self::$sFilePath ) );
162
-        return self::$_sPluginURLCache . $sRelativePath;
162
+        return self::$_sPluginURLCache.$sRelativePath;
163 163
     }
164 164
         /**
165 165
          * @since       3.7.9
@@ -189,13 +189,13 @@  discard block
 block discarded – undo
189 189
      * @since       3.5.0
190 190
      * @return      void
191 191
      */
192
-    static public function setAdminNotice( $sMessage, $sClassAttribute='error' ) {
193
-        if ( ! is_admin() ) {
192
+    static public function setAdminNotice( $sMessage, $sClassAttribute = 'error' ) {
193
+        if ( !is_admin() ) {
194 194
             return;
195 195
         }
196
-        self::$_aAdminNotices[] = array(
196
+        self::$_aAdminNotices[ ] = array(
197 197
             'message'           => $sMessage,
198
-            'class_attribute'   => trim( $sClassAttribute ) . ' notice is-dismissible',
198
+            'class_attribute'   => trim( $sClassAttribute ).' notice is-dismissible',
199 199
         );
200 200
         add_action( 'admin_notices', array( __CLASS__, '_replyToSetAdminNotice' ) );
201 201
     }
@@ -205,12 +205,12 @@  discard block
 block discarded – undo
205 205
          * @return      void
206 206
          */
207 207
         static public function _replyToSetAdminNotice() {
208
-            foreach( self::$_aAdminNotices as $_aAdminNotice ) {
209
-                echo "<div class='" . esc_attr( $_aAdminNotice[ 'class_attribute' ] ) . " notice is-dismissible'>"
208
+            foreach ( self::$_aAdminNotices as $_aAdminNotice ) {
209
+                echo "<div class='".esc_attr( $_aAdminNotice[ 'class_attribute' ] )." notice is-dismissible'>"
210 210
                         ."<p>"
211 211
                             . sprintf(
212
-                                '<strong>%1$s</strong>: ' . $_aAdminNotice[ 'message' ],
213
-                                self::NAME . ' ' . self::VERSION
212
+                                '<strong>%1$s</strong>: '.$_aAdminNotice[ 'message' ],
213
+                                self::NAME.' '.self::VERSION
214 214
                             )
215 215
                         . "</p>"
216 216
                     . "</div>";
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
 AdminPageFrameworkLoader_Registry::setUp( __FILE__ );
223 223
 
224 224
 // Initial checks. - Do no load if accessed directly, not exiting because the 'uninstall.php' and inclusion list generator will load this file.
225
-if ( ! defined( 'ABSPATH' ) ) {
225
+if ( !defined( 'ABSPATH' ) ) {
226 226
     return;
227 227
 }
228 228
 if ( defined( 'DOING_UNINSTALL' ) && DOING_UNINSTALL ) {
@@ -234,8 +234,8 @@  discard block
 block discarded – undo
234 234
 
235 235
     $_bFrameworkLoaded = class_exists( 'AdminPageFramework_Registry', false );
236 236
     if (
237
-        ! $_bFrameworkLoaded
238
-        || ! defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
237
+        !$_bFrameworkLoaded
238
+        || !defined( 'AdminPageFramework_Registry::VERSION' ) // backward compatibility
239 239
         || version_compare( AdminPageFramework_Registry::VERSION, AdminPageFrameworkLoader_Registry::VERSION, '<' )
240 240
     ) {
241 241
         AdminPageFrameworkLoader_Registry::setAdminNotice(
@@ -253,18 +253,18 @@  discard block
 block discarded – undo
253 253
 add_action( 'admin_init', 'AdminPageFrameworkLoader_Warning' );
254 254
 
255 255
 // Include the library file - the development version will be available if you cloned the GitHub repository.
256
-$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath . '/development/admin-page-framework.php';
256
+$_sDevelopmentVersionPath = AdminPageFrameworkLoader_Registry::$sDirPath.'/development/admin-page-framework.php';
257 257
 $_bDebugMode              = defined( 'WP_DEBUG' ) && WP_DEBUG;
258 258
 $_bLoadDevelopmentVersion = $_bDebugMode && file_exists( $_sDevelopmentVersionPath );
259 259
 include(
260 260
     $_bLoadDevelopmentVersion
261 261
         ? $_sDevelopmentVersionPath
262
-        : AdminPageFrameworkLoader_Registry::$sDirPath . '/library/apf/admin-page-framework.php'
262
+        : AdminPageFrameworkLoader_Registry::$sDirPath.'/library/apf/admin-page-framework.php'
263 263
 );
264 264
 
265 265
 // Include the framework loader plugin components.
266 266
 include( AdminPageFramework_Registry::$aClassFiles[ 'AdminPageFramework_PluginBootstrap' ] );
267
-include( AdminPageFrameworkLoader_Registry::$sDirPath . '/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
267
+include( AdminPageFrameworkLoader_Registry::$sDirPath.'/include/class/AdminPageFrameworkLoader_Bootstrap.php' );
268 268
 new AdminPageFrameworkLoader_Bootstrap(
269 269
     AdminPageFrameworkLoader_Registry::$sFilePath,
270 270
     AdminPageFrameworkLoader_Registry::HOOK_SLUG    // hook prefix
Please login to merge, or discard this patch.