View GitHub Repository admin-page-framework

Schedule Inspection
Subscribe
Completed
Branch — master (fd2290)
by
unknown
06:38
created
Status
Category
Toggle Patches
utility/base_utility/AdminPageFramework_Utility_SystemInformation.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -95,7 +95,7 @@
 
 block discarded – undo
95 95 
      * Returns an array of constants.
96 96 
      *
97 97 
      * @since       3.4.6
98 
-     * @param       array|string      $asCategory      The category key names of the returning array.
98 
+     * @param       array|string      $asCategories      The category key names of the returning array.
99 99 
      */
100 100 
     static public function getDefinedConstants( $asCategories=null, $asRemovingCategories=null ) {
101 101
Please login to merge, or discard this patch.
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch 
@@ -41,7 +41,7 @@  discard block
 
 block discarded – undo
41 41 
         $_sOutput = preg_replace(
42 42 
             array(
43 43 
                 '#^.*<body>(.*)</body>.*$#ms', '#<h2>PHP License</h2>.*$#ms',
44 
-                '#<h1>Configuration</h1>#',  "#\r?\n#", "#</(h1|h2|h3|tr)>#", '# +<#',
44 
+                '#<h1>Configuration</h1>#', "#\r?\n#", "#</(h1|h2|h3|tr)>#", '# +<#',
45 45 
                 "#[ \t]+#", '#&nbsp;#', '#  +#', '# class=".*?"#', '%&#039;%',
46 46 
                 '#<tr>(?:.*?)" src="(?:.*?)=(.*?)" alt="PHP Logo" /></a>'
47 47 
                     .'<h1>PHP Version (.*?)</h1>(?:\n+?)</td></tr>#',
 
@@ -52,11 +52,11 @@  discard block
 
 block discarded – undo
52 52 
                 '#</tr>#'
53 53 
             ),
54 54 
             array(
55 
-                '$1', '', '', '', '</$1>' . "\n", '<', ' ', ' ', ' ', '', ' ',
55 
+                '$1', '', '', '', '</$1>'."\n", '<', ' ', ' ', ' ', '', ' ',
56 56 
                 '<h2>PHP Configuration</h2>'."\n".'<tr><td>PHP Version</td><td>$2</td></tr>'.
57 57 
                 "\n".'<tr><td>PHP Egg</td><td>$1</td></tr>',
58 58 
                 '<tr><td>PHP Credits Egg</td><td>$1</td></tr>',
59 
-                '<tr><td>Zend Engine</td><td>$2</td></tr>' . "\n" . '<tr><td>Zend Egg</td><td>$1</td></tr>',
59 
+                '<tr><td>Zend Engine</td><td>$2</td></tr>'."\n".'<tr><td>Zend Egg</td><td>$1</td></tr>',
60 60 
                 ' ',
61 61 
                 '%S%',
62 62 
                 '%E%'
 
@@ -68,7 +68,7 @@  discard block
 
 block discarded – undo
68 68 
         unset( $_aSections[ 0 ] );
69 69
70 70 
         $_aOutput = array();
71 
-        foreach( $_aSections as $_sSection ) {
71 
+        foreach ( $_aSections as $_sSection ) {
72 72 
             $_iIndex = substr( $_sSection, 0, strpos( $_sSection, '</h2>' ) );
73 73 
             preg_match_all(
74 74 
                 '#%S%(?:<td>(.*?)</td>)?(?:<td>(.*?)</td>)?(?:<td>(.*?)</td>)?%E%#',
 
@@ -76,12 +76,12 @@  discard block
 
 block discarded – undo
76 76 
                 $_aAskApache,
77 77 
                 PREG_SET_ORDER
78 78 
             );
79 
-            foreach( $_aAskApache as $_aMatches ) {
80 
-                if ( ! isset( $_aMatches[ 1 ], $_aMatches[ 2 ] ) ) {
79 
+            foreach ( $_aAskApache as $_aMatches ) {
80 
+                if ( !isset( $_aMatches[ 1 ], $_aMatches[ 2 ] ) ) {
81 81 
                     array_slice( $_aMatches, 2 );
82 82 
                     continue;
83 83 
                 }
84 
-                $_aOutput[ $_iIndex ][ $_aMatches[ 1 ] ] = ! isset( $_aMatches[ 3 ] ) || $_aMatches[ 2 ] == $_aMatches[ 3 ]
84 
+                $_aOutput[ $_iIndex ][ $_aMatches[ 1 ] ] = !isset( $_aMatches[ 3 ] ) || $_aMatches[ 2 ] == $_aMatches[ 3 ]
85 85 
                     ? $_aMatches[ 2 ]
86 86 
                     : array_slice( $_aMatches, 2 );
87 87 
             }
 
@@ -97,7 +97,7 @@  discard block
 
 block discarded – undo
97 97 
      * @since       3.4.6
98 98 
      * @param       array|string      $asCategory      The category key names of the returning array.
99 99 
      */
100 
-    static public function getDefinedConstants( $asCategories=null, $asRemovingCategories=null ) {
100 
+    static public function getDefinedConstants( $asCategories = null, $asRemovingCategories = null ) {
101 101
102 102 
         $_aCategories           = is_array( $asCategories ) ? $asCategories : array( $asCategories );
103 103 
         $_aCategories           = array_filter( $_aCategories );
 
@@ -124,8 +124,8 @@  discard block
 
 block discarded – undo
124 124 
     static public function getPHPErrorLogPath() {
125 125
126 126 
         $_aPHPInfo = self::getPHPInfo();
127 
-        return isset( $_aPHPInfo['PHP Core']['error_log'] )
128 
-            ? $_aPHPInfo['PHP Core']['error_log']
127 
+        return isset( $_aPHPInfo[ 'PHP Core' ][ 'error_log' ] )
128 
+            ? $_aPHPInfo[ 'PHP Core' ][ 'error_log' ]
129 129 
             : '';
130 130
131 131 
     }
 
@@ -134,7 +134,7 @@  discard block
 
 block discarded – undo
134 134 
      * Returns a PHP error log.
135 135 
      * @since       3.4.6
136 136 
      */
137 
-    static public function getPHPErrorLog( $iLines=1 ) {
137 
+    static public function getPHPErrorLog( $iLines = 1 ) {
138 138
139 139 
         $_sLog = self::getFileTailContents( self::getPHPErrorLogPath(), $iLines );
140 140
Please login to merge, or discard this patch.
factory/_common/utility/base_utility/AdminPageFramework_Utility_URL.php 2 patches
Doc Comments   +2 added lines patch added patch discarded remove patch 
@@ -23,6 +23,7 @@  discard block
 
 block discarded – undo
23 23 
      *
24 24 
      * @since       2.0.0
25 25 
      * @since       3.5.7       Moved from `AdminPageFramework_Utility`.
26 
+     * @param string $sQueryKey
26 27 
      * @return      string|null
27 28 
      */
28 29 
     static public function getQueryValueInURLByKey( $sURL, $sQueryKey ) {
 
@@ -72,6 +73,7 @@  discard block
 
 block discarded – undo
72 73 
         /**
73 74 
          * Returns the port suffix in the currently loading url.
74 75 
          * @since       3.5.7
76 
+         * @param boolean $_bSSL
75 77 
          * @return      string
76 78 
          */
77 79 
         static private function _getURLPortSuffix( $_bSSL ) {
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -52,7 +52,7 @@  discard block
 
 block discarded – undo
52 52 
             0 => '',
53 53 
             1 => 's',
54 54 
         );
55 
-        $_sProtocol         = substr( $_sServerProtocol, 0, strpos( $_sServerProtocol, '/' ) )
55 
+        $_sProtocol = substr( $_sServerProtocol, 0, strpos( $_sServerProtocol, '/' ) )
56 56 
             . $_aProrocolSuffix[ ( int ) $_bSSL ];
57 57
58 58 
         // Port: e.g. :80
 
@@ -66,7 +66,7 @@  discard block
 
 block discarded – undo
66 66 
                 : $_SERVER[ 'SERVER_NAME' ]
67 67 
             );
68 68
69 
-        return $_sProtocol . '://' . $_sHost . $_sPort . $_SERVER[ 'REQUEST_URI' ];
69 
+        return $_sProtocol.'://'.$_sHost.$_sPort.$_SERVER[ 'REQUEST_URI' ];
70 70
71 71 
     }
72 72 
         /**
 
@@ -79,10 +79,10 @@  discard block
 
 block discarded – undo
79 79 
                 ? ( string ) $_SERVER[ 'SERVER_PORT' ]
80 80 
                 : '';
81 81 
             $_aPort     = array(
82 
-                0 => ':' . $_sPort,
82 
+                0 => ':'.$_sPort,
83 83 
                 1 => '',
84 84 
             );
85 
-            $_bPortSet  = ( ! $_bSSL && '80' === $_sPort ) || ( $_bSSL && '443' === $_sPort );
85 
+            $_bPortSet = ( !$_bSSL && '80' === $_sPort ) || ( $_bSSL && '443' === $_sPort );
86 86 
             return $_aPort[ ( int ) $_bPortSet ];
87 87 
         }
88 88
Please login to merge, or discard this patch.
factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_Hook.php 3 patches
Doc Comments   +1 added lines, -9 removed lines patch added patch discarded remove patch 
@@ -89,11 +89,6 @@  discard block
 
 block discarded – undo
89 89 
      * @access      public
90 90 
      * @since       2.0.0
91 91 
      * @remark      Accepts variadic parameters.
92 
-     * @param       object      $oCallerObject      the object that holds the callback method that matches the action hook name.
93 
-     * @param       array       $aActionHooks       a numerically index array consisting of action hook names that serve as the callback method names.
94 
-     * @param       mixed       $vArgs1             the argument to pass to the hook callback functions.
95 
-     * @param       mixed       $vArgs2             another argument to pass to the hook callback functions.
96 
-     * @param       mixed       $_and_more          add as many arguments as necessary to the next parameters.
97 92 
      * @return      void
98 93 
      */
99 94 
     static public function addAndDoActions( /* $oCallerObject, $aActionHooks, $vArgs1=null, $vArgs2=null, $_and_more=null */ ) {
 
@@ -192,10 +187,6 @@  discard block
 
 block discarded – undo
192 187 
      * `
193 188 
      *
194 189 
      * @since       2.0.0
195 
-     * @param       object      $oCallerObject
196 
-     * @param       string      $sFilter            The filter hook name.
197 
-     * @param       mixed       $vData              The filtering data
198 
-     * @param       mixed       $vArgs              The arguments.
199 190 
      */
200 191 
     static public function addAndApplyFilter( /* $oCallerObject, $sFilter, $vData, $vArgs... */ ) {
201 192 
         
@@ -233,6 +224,7 @@  discard block
 
 block discarded – undo
233 224 
      *
234 225 
      * @since       2.0.0
235 226 
      * @access      public
227 
+     * @param string $sPrefix
236 228 
      * @return      array       Returns an array consisting of the filters.
237 229 
      */
238 230 
     static public function getFilterArrayByPrefix( $sPrefix, $sClassName, $sPageSlug, $sTabSlug, $bReverse=false ) {
Please login to merge, or discard this patch.
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch 
@@ -26,7 +26,7 @@  discard block
 
 block discarded – undo
26 26 
      * @since       3.4.2
27 27 
      * @todo        Create a similar function for filters.
28 28 
      */
29 
-    static public function registerAction( $sActionHook, $oCallable, $iPriority=10 ) {
29 
+    static public function registerAction( $sActionHook, $oCallable, $iPriority = 10 ) {
30 30
31 31 
         if ( did_action( $sActionHook ) ) {
32 32 
             return call_user_func_array( $oCallable, array() );
 
@@ -63,13 +63,13 @@  discard block
 
 block discarded – undo
63 63 
      * @param       mixed       $_and_more      add as many arguments as necessary to the next parameters.
64 64 
      * @return      void        does not return a value.
65 65 
      */
66 
-    static public function doActions( $aActionHooks, $vArgs1=null, $vArgs2=null, $_and_more=null ) {
66 
+    static public function doActions( $aActionHooks, $vArgs1 = null, $vArgs2 = null, $_and_more = null ) {
67 67
68 68 
         $aArgs          = func_get_args();
69 69 
         $aActionHooks   = $aArgs[ 0 ];
70 
-        foreach( ( array ) $aActionHooks as $sActionHook  ) {
70 
+        foreach ( ( array ) $aActionHooks as $sActionHook ) {
71 71 
             $aArgs[ 0 ] = $sActionHook;
72 
-            call_user_func_array( 'do_action' , $aArgs );
72 
+            call_user_func_array( 'do_action', $aArgs );
73 73 
         }
74 74
75 75 
     }
 
@@ -101,12 +101,12 @@  discard block
 
 block discarded – undo
101 101 
         $aArgs          = func_get_args();
102 102 
         $oCallerObject  = $aArgs[ 0 ];
103 103 
         $aActionHooks   = $aArgs[ 1 ];
104 
-        foreach( ( array ) $aActionHooks as $sActionHook ) {
105 
-            if ( ! $sActionHook ) {
104 
+        foreach ( ( array ) $aActionHooks as $sActionHook ) {
105 
+            if ( !$sActionHook ) {
106 106 
                 continue;
107 107 
             }
108 108 
             $aArgs[ 1 ] = $sActionHook;
109 
-            call_user_func_array( array( get_class(), 'addAndDoAction' ) , $aArgs );
109 
+            call_user_func_array( array( get_class(), 'addAndDoAction' ), $aArgs );
110 110 
         }
111 111
112 112 
     }
 
@@ -125,7 +125,7 @@  discard block
 
 block discarded – undo
125 125 
         $_aArgs          = func_get_args();
126 126 
         $_oCallerObject  = $_aArgs[ 0 ];
127 127 
         $_sActionHook    = $_aArgs[ 1 ];
128 
-        if ( ! $_sActionHook ) {
128 
+        if ( !$_sActionHook ) {
129 129 
             return;
130 130 
         }
131 131 
         
@@ -137,7 +137,7 @@  discard block
 
 block discarded – undo
137 137
138 138 
         // Remove the first element, the caller object.
139 139 
         array_shift( $_aArgs );
140 
-        call_user_func_array( 'do_action' , $_aArgs );
140 
+        call_user_func_array( 'do_action', $_aArgs );
141 141
142 142 
     }
143 143 
     
@@ -165,12 +165,12 @@  discard block
 
 block discarded – undo
165 165 
         $_aFilters      = $_aArgs[ 1 ];
166 166 
         $_vInput        = $_aArgs[ 2 ];
167 167
168 
-        foreach( ( array ) $_aFilters as $_sFilter ) {
169 
-            if ( ! $_sFilter ) {
168 
+        foreach ( ( array ) $_aFilters as $_sFilter ) {
169 
+            if ( !$_sFilter ) {
170 170 
                 continue;
171 171 
             }
172 172 
             $_aArgs[ 1 ] = $_sFilter;
173 
-            $_aArgs[ 2 ] = $_vInput;    // assigns the updated value as it is filtered in previous iterations
173 
+            $_aArgs[ 2 ] = $_vInput; // assigns the updated value as it is filtered in previous iterations
174 174 
             $_vInput = call_user_func_array(
175 175 
                 array( get_class(), 'addAndApplyFilter' ),
176 176 
                 $_aArgs 
@@ -204,7 +204,7 @@  discard block
 
 block discarded – undo
204 204 
         $_aArgs          = func_get_args();
205 205 
         $_oCallerObject  = $_aArgs[ 0 ];
206 206 
         $_sFilter        = $_aArgs[ 1 ];
207 
-        if ( ! $_sFilter ) {
207 
+        if ( !$_sFilter ) {
208 208 
             return $_aArgs[ 2 ];
209 209 
         }
210 210 
         
@@ -235,17 +235,17 @@  discard block
 
 block discarded – undo
235 235 
      * @access      public
236 236 
      * @return      array       Returns an array consisting of the filters.
237 237 
      */
238 
-    static public function getFilterArrayByPrefix( $sPrefix, $sClassName, $sPageSlug, $sTabSlug, $bReverse=false ) {
238 
+    static public function getFilterArrayByPrefix( $sPrefix, $sClassName, $sPageSlug, $sTabSlug, $bReverse = false ) {
239 239
240 240 
         $_aFilters = array();
241 241 
         if ( $sTabSlug && $sPageSlug ) {
242 
-            $_aFilters[] = "{$sPrefix}{$sPageSlug}_{$sTabSlug}";
242 
+            $_aFilters[ ] = "{$sPrefix}{$sPageSlug}_{$sTabSlug}";
243 243 
         }
244 244 
         if ( $sPageSlug ) {
245 
-            $_aFilters[] = "{$sPrefix}{$sPageSlug}";
245 
+            $_aFilters[ ] = "{$sPrefix}{$sPageSlug}";
246 246 
         }
247 247 
         if ( $sClassName ) {
248 
-            $_aFilters[] = "{$sPrefix}{$sClassName}";
248 
+            $_aFilters[ ] = "{$sPrefix}{$sClassName}";
249 249 
         }
250 250 
         return $bReverse
251 251 
             ? array_reverse( $_aFilters )
Please login to merge, or discard this patch.
Braces   +6 added lines, -6 removed lines patch added patch discarded remove patch 
@@ -102,7 +102,7 @@  discard block
 
 block discarded – undo
102 102 
         $oCallerObject  = $aArgs[ 0 ];
103 103 
         $aActionHooks   = $aArgs[ 1 ];
104 104 
         foreach( ( array ) $aActionHooks as $sActionHook ) {
105 
-            if ( ! $sActionHook ) {
105 
+            if ( ! $sActionHook ) {
106 106 
                 continue;
107 107 
             }
108 108 
             $aArgs[ 1 ] = $sActionHook;    
@@ -125,7 +125,7 @@  discard block
 
 block discarded – undo
125 125 
         $_aArgs          = func_get_args();
126 126 
         $_oCallerObject  = $_aArgs[ 0 ];
127 127 
         $_sActionHook    = $_aArgs[ 1 ];
128 
-        if ( ! $_sActionHook ) {
128 
+        if ( ! $_sActionHook ) {
129 129 
             return;
130 130 
         }
131 131 
         
@@ -158,7 +158,7 @@  discard block
 
 block discarded – undo
158 158 
      *
159 159 
      * @since       2.0.0
160 160 
      */
161 
-    static public function addAndApplyFilters( /* $oCallerObject, $aFilters, $vInput, $vArgs... */ ) {
161 
+    static public function addAndApplyFilters( /* $oCallerObject, $aFilters, $vInput, $vArgs... */ ) {
162 162
163 163 
         $_aArgs         = func_get_args();
164 164 
         // $oCallerObject  = $_aArgs[ 0 ];
 
@@ -166,7 +166,7 @@  discard block
 
 block discarded – undo
166 166 
         $_vInput        = $_aArgs[ 2 ];
167 167
168 168 
         foreach( ( array ) $_aFilters as $_sFilter ) {
169 
-            if ( ! $_sFilter ) {
169 
+            if ( ! $_sFilter ) {
170 170 
                 continue;
171 171 
             }
172 172 
             $_aArgs[ 1 ] = $_sFilter;
 
@@ -197,14 +197,14 @@  discard block
 
 block discarded – undo
197 197 
      * @param       mixed       $vData              The filtering data
198 198 
      * @param       mixed       $vArgs              The arguments.
199 199 
      */
200 
-    static public function addAndApplyFilter( /* $oCallerObject, $sFilter, $vData, $vArgs... */ ) {
200 
+    static public function addAndApplyFilter( /* $oCallerObject, $sFilter, $vData, $vArgs... */ ) {
201 201
202 202 
         // Parameters
203 203 
         $_iArgs          = func_num_args();
204 204 
         $_aArgs          = func_get_args();
205 205 
         $_oCallerObject  = $_aArgs[ 0 ];
206 206 
         $_sFilter        = $_aArgs[ 1 ];
207 
-        if ( ! $_sFilter ) {
207 
+        if ( ! $_sFilter ) {
208 208 
             return $_aArgs[ 2 ];
209 209 
         }
210 210
Please login to merge, or discard this patch.
factory/_common/utility/wp_utility/AdminPageFramework_WPUtility_Page.php 3 patches
Doc Comments   +1 added lines patch added patch discarded remove patch 
@@ -284,6 +284,7 @@
 
 block discarded – undo
284 284 
              * Checkes whether the passed base url name is of the admin index page.
285 285 
              * @since       3.5.3
286 286 
              * return       boolean      Whether the passed base url name is of the admin index page.
287 
+             * @param string $sPageNow
287 288 
              */
288 289 
             static private function _isInAdminIndex( $sPageNow ) {
289 290 
                 return in_array(
Please login to merge, or discard this patch.
Braces   +7 added lines, -9 removed lines patch added patch discarded remove patch 
@@ -29,7 +29,7 @@  discard block
 
 block discarded – undo
29 29 
      */
30 30 
     static public function getCurrentPostType() {
31 31
32 
-        if ( isset( self::$_sCurrentPostType ) ) {
32 
+        if ( isset( self::$_sCurrentPostType ) ) {
33 33 
             return self::$_sCurrentPostType;
34 34 
         }
35 35 
         self::$_sCurrentPostType = self::_getCurrentPostType();
 
@@ -141,7 +141,7 @@  discard block
 
 block discarded – undo
141 141 
     static public function isPostDefinitionPage( $asPostTypes=array() ) {
142 142
143 143 
         // If it's not the post definition page,
144 
-        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) {
144 
+        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) {
145 145 
             return false;
146 146 
         }
147 147 
         return self::isCurrentPostTypeIn( $asPostTypes );
 
@@ -160,7 +160,7 @@  discard block
 
 block discarded – undo
160 160 
         $_aPostTypes = self::getAsArray( $asPostTypes );
161 161
162 162 
         // If the parameter is empty,
163 
-        if ( empty( $_aPostTypes ) ) {
163 
+        if ( empty( $_aPostTypes ) ) {
164 164 
             return true;
165 165 
         }
166 166 
 
@@ -176,13 +176,13 @@  discard block
 
 block discarded – undo
176 176 
      */
177 177 
     static public function isPostListingPage( $asPostTypes=array() ) {
178 178
179 
-        if ( 'edit.php' != self::getPageNow() ) {
179 
+        if ( 'edit.php' != self::getPageNow() ) {
180 180 
             return false;
181 181 
         }
182 182
183 183 
         $_aPostTypes = self::getAsArray( $asPostTypes );
184 184
185 
-        if ( ! isset( $_GET[ 'post_type' ] )  ) {
185 
+        if ( ! isset( $_GET[ 'post_type' ] )  ) {
186 186 
             return in_array( 'post', $_aPostTypes );
187 187 
         }
188 188 
 
@@ -273,11 +273,9 @@  discard block
 
 block discarded – undo
273 273
274 274 
                 if ( is_network_admin() ) {
275 275 
                     $_sNeedle = '#/wp-admin/network/?(.*?)$#i';
276 
-                }
277 
-                else if ( is_user_admin() ) {
276 
+                } else if ( is_user_admin() ) {
278 277 
                     $_sNeedle = '#/wp-admin/user/?(.*?)$#i';
279 
-                }
280 
-                else {
278 
+                } else {
281 279 
                     $_sNeedle = '#/wp-admin/?(.*?)$#i';
282 280 
                 }
283 281 
                 preg_match( $_sNeedle, $_SERVER[ 'PHP_SELF' ], $_aMatches );
Please login to merge, or discard this patch.
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch 
@@ -57,7 +57,7 @@  discard block
 
 block discarded – undo
57 57 
                 'getPostTypeByTypeNow',
58 58 
                 'getPostTypeByScreenObject',
59 59 
                 'getPostTypeByREQUEST',
60 
-                'getPostTypeByPostObject',  // 3.6.0+ Moved to the last as it is not reliable.
60 
+                'getPostTypeByPostObject', // 3.6.0+ Moved to the last as it is not reliable.
61 61 
             );
62 62 
             foreach ( $_aMethodsToTry as $_sMethodName ) {
63 63 
                 $_sPostType = call_user_func( array( __CLASS__, $_sMethodName ) );
 
@@ -121,9 +121,9 @@  discard block
 
 block discarded – undo
121 121 
      * @param       array|string        The post type slug(s) to check. If this is empty, the method just checks the current page is a taxonomy page.
122 122 
      * @return      boolean
123 123 
      */
124 
-    static public function isCustomTaxonomyPage( $asPostTypes=array() ) {
124 
+    static public function isCustomTaxonomyPage( $asPostTypes = array() ) {
125 125
126 
-        if ( ! in_array( self::getPageNow(), array( 'tags.php', 'edit-tags.php', 'term.php' ) ) ) {
126 
+        if ( !in_array( self::getPageNow(), array( 'tags.php', 'edit-tags.php', 'term.php' ) ) ) {
127 127 
             return false;
128 128 
         }
129 129 
         return self::isCurrentPostTypeIn( $asPostTypes );
 
@@ -138,10 +138,10 @@  discard block
 
 block discarded – undo
138 138 
      * Otherwise, it will check if the page belongs to the given post type(s).
139 139 
      * @return      boolean
140 140 
      */
141 
-    static public function isPostDefinitionPage( $asPostTypes=array() ) {
141 
+    static public function isPostDefinitionPage( $asPostTypes = array() ) {
142 142
143 143 
         // If it's not the post definition page,
144 
-        if ( ! in_array( self::getPageNow(), array( 'post.php', 'post-new.php', ) ) ) {
144 
+        if ( !in_array( self::getPageNow(), array( 'post.php', 'post-new.php',) ) ) {
145 145 
             return false;
146 146 
         }
147 147 
         return self::isCurrentPostTypeIn( $asPostTypes );
 
@@ -174,7 +174,7 @@  discard block
 
 block discarded – undo
174 174 
      *
175 175 
      * @since 3.0.0
176 176 
      */
177 
-    static public function isPostListingPage( $asPostTypes=array() ) {
177 
+    static public function isPostListingPage( $asPostTypes = array() ) {
178 178
179 179 
         if ( 'edit.php' != self::getPageNow() ) {
180 180 
             return false;
 
@@ -182,7 +182,7 @@  discard block
 
 block discarded – undo
182 182
183 183 
         $_aPostTypes = self::getAsArray( $asPostTypes );
184 184
185 
-        if ( ! isset( $_GET[ 'post_type' ] )  ) {
185 
+        if ( !isset( $_GET[ 'post_type' ] ) ) {
186 186 
             return in_array( 'post', $_aPostTypes );
187 187 
         }
188 188 
 
@@ -220,7 +220,7 @@  discard block
 
 block discarded – undo
220 220 
             0 => '_getPageNow_FrontEnd',
221 221 
             1 => '_getPageNow_BackEnd',
222 222 
         );
223 
-        $_sMethodName  = $_aMethodNames[ ( integer ) is_admin() ];
223 
+        $_sMethodName = $_aMethodNames[ ( integer ) is_admin() ];
224 224 
         self::$_sPageNow = self::$_sMethodName();
225 225 
         return self::$_sPageNow;
226 226 
         
@@ -315,7 +315,7 @@  discard block
 
 block discarded – undo
315 315
316 316 
         if ( isset( $GLBOALS[ 'page_hook' ] ) ) {
317 317 
             return is_network_admin()
318 
-                ? $GLBOALS[ 'page_hook' ] . '-network'
318 
+                ? $GLBOALS[ 'page_hook' ].'-network'
319 319 
                 : $GLBOALS[ 'page_hook' ];
320 320 
         }
321 321 
         
@@ -329,11 +329,11 @@  discard block
 
 block discarded – undo
329 329 
      * @since       3.7.0
330 330 
      * @return      boolean
331 331 
      */
332 
-    static public function doesMetaBoxExist( $sContext='' ) {
332 
+    static public function doesMetaBoxExist( $sContext = '' ) {
333 333
334 334 
         $_aDimensions = array( 'wp_meta_boxes', $GLOBALS[ 'page_hook' ] );
335 335 
         if ( $sContext ) {
336 
-            $_aDimensions[] = $sContext;
336 
+            $_aDimensions[ ] = $sContext;
337 337 
         }
338 338 
         $_aSideMetaBoxes = self::getElementAsArray(
339 339 
             $GLOBALS,
Please login to merge, or discard this patch.
admin_page/_model/delegate/AdminPageFramework_Model_Menu__RegisterMenu.php 3 patches
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -27,6 +27,7 @@  discard block
 
 block discarded – undo
27 27 
      * Sets up hooks
28 28 
      *
29 29 
      * @internal
30 
+     * @param AdminPageFramework_Model_Menu $oFactory
30 31 
      */
31 32 
     public function __construct( $oFactory, $sActionHook='admin_menu' ) {
32 33 
         
@@ -228,6 +229,7 @@  discard block
 
 block discarded – undo
228 229 
              * @since       3.3.0
229 230 
              * @since       3.1.1       Moved from `AdminPageFramework_Menu`.
230 231 
              * @since       3.7.4       Added the `$nOrder` parameter.
232 
+             * @param string $sMenuSlug
231 233 
              * @return      string      The page hook of the added page.
232 234 
              * @uses        add_submenu_page
233 235 
              */
 
@@ -340,7 +342,7 @@  discard block
 
 block discarded – undo
340 342 
                 /**
341 343 
                  * @since       3.7.4
342 344 
                  * @return      void
343 
-                 * @param       numeric     $$nOrder            A user set order (menu position, index).
345 
+                 * @param       numeric     $nOrder            A user set order (menu position, index).
344 346 
                  * @param       array       $aSubMenuItem       The sub menu item array set in the global `$submenu` array.
345 347 
                  */
346 348 
                 private function _setSubMenuPageByIndex( $nOrder, $aSubMenuItem, $sMenuSlug ) {
 
@@ -464,6 +466,7 @@  discard block
 
 block discarded – undo
464 466 
              * @since       3.1.1       Moved from `AdminPageFramework_Menu`.
465 467 
              * @since       3.5.3       Added the `$bShowInMenu` parameter.
466 468 
              * @since       3.7.4       Added the `$nOrder` parameter.
469 
+             * @param string $sMenuSlug
467 470 
              * @return      void
468 471 
              */
469 472 
             private function _addLinkSubmenuItem( $sMenuSlug, $sTitle, $sCapability, $sHref, $bShowInMenu, $nOrder ) {
Please login to merge, or discard this patch.
Spacing   +37 added lines, -37 removed lines patch added patch discarded remove patch 
@@ -28,7 +28,7 @@  discard block
 
 block discarded – undo
28 28 
      *
29 29 
      * @internal
30 30 
      */
31 
-    public function __construct( $oFactory, $sActionHook='admin_menu' ) {
31 
+    public function __construct( $oFactory, $sActionHook = 'admin_menu' ) {
32 32
33 33 
         $this->oFactory = $oFactory;
34 34 
         
@@ -76,7 +76,7 @@  discard block
 
 block discarded – undo
76 76 
     public function _replyToRegisterRootMenu() {
77 77
78 78 
         // If the root menu label is not set but the slug is set,
79 
-        if ( ! $this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
79 
+        if ( !$this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
80 80 
             return;
81 81 
         }
82 82 
         $this->_registerRootMenuPage();
 
@@ -93,12 +93,12 @@  discard block
 
 block discarded – undo
93 93 
         private function _registerRootMenuPage() {
94 94
95 95 
             $this->oFactory->oProp->aRootMenu[ '_page_hook' ] = add_menu_page(
96 
-                $this->oFactory->oProp->sClassName,                 // Page title - will be invisible anyway
97 
-                $this->oFactory->oProp->aRootMenu[ 'sTitle' ],      // Menu title - should be the root page title.
98 
-                $this->oFactory->oProp->sCapability,                // Capability - access right
99 
-                $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ],   // Menu ID
100 
-                '',                                       // Callback function for the page content output - the root page will be removed so no need to register a function.
101 
-                $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ],  // icon path
96 
+                $this->oFactory->oProp->sClassName, // Page title - will be invisible anyway
97 
+                $this->oFactory->oProp->aRootMenu[ 'sTitle' ], // Menu title - should be the root page title.
98 
+                $this->oFactory->oProp->sCapability, // Capability - access right
99 
+                $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ], // Menu ID
100 
+                '', // Callback function for the page content output - the root page will be removed so no need to register a function.
101 
+                $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ], // icon path
102 102 
                 $this->getElement(
103 103 
                     $this->oFactory->oProp->aRootMenu,
104 104 
                     'iPosition', 
@@ -120,8 +120,8 @@  discard block
 
 block discarded – undo
120 120
121 121 
         // Let external scripts add sub-menu pages.
122 122 
         $_aPages = $this->addAndApplyFilter(
123 
-            $this->oFactory,    // caller object
124 
-            "pages_{$this->oFactory->oProp->sClassName}",   // filter
123 
+            $this->oFactory, // caller object
124 
+            "pages_{$this->oFactory->oProp->sClassName}", // filter
125 125 
             $this->oFactory->oProp->aPages  // arguments
126 126 
         );
127 127 
         
@@ -131,7 +131,7 @@  discard block
 
 block discarded – undo
131 131 
         // Format the `$aPages` property and register the pages.
132 132 
         $_iParsedIndex    = 0;
133 133 
         $_aFormattedPages = array();
134 
-        foreach( $_aPages as $_aSubMenuItem ) {
134 
+        foreach ( $_aPages as $_aSubMenuItem ) {
135 135
136 136 
             // needs to be sanitized because there are hook filters applied to this array.
137 137 
             $_oFormatter = new AdminPageFramework_Format_SubMenuItem( 
@@ -166,8 +166,8 @@  discard block
 
 block discarded – undo
166 166 
          */
167 167 
         private function _getDefaultPageSlug( array $aPages ) {
168 168
169 
-            foreach( $aPages as $_aPage ) {
170 
-                if ( ! isset( $_aPage[ 'page_slug' ] ) ) {
169 
+            foreach ( $aPages as $_aPage ) {
170 
+                if ( !isset( $_aPage[ 'page_slug' ] ) ) {
171 171 
                     continue;
172 172 
                 }
173 173 
                 return $_aPage[ 'page_slug' ];                
@@ -189,7 +189,7 @@  discard block
 
 block discarded – undo
189 189 
          */
190 190 
         private function _registerSubMenuItem( array $aArgs ) {
191 191
192 
-            if ( ! current_user_can( $aArgs[ 'capability' ] ) ) {
192 
+            if ( !current_user_can( $aArgs[ 'capability' ] ) ) {
193 193 
                 return '';
194 194 
             }
195 195 
                                     
@@ -254,16 +254,16 @@  discard block
 
 block discarded – undo
254 254 
              */
255 255 
             private function _addPageSubmenuItem( $sRootPageSlug, $sMenuSlug, $sPageSlug, $sPageTitle, $sMenuTitle, $sCapability, $bShowInMenu, $nOrder ) {
256 256
257 
-                if ( ! $sPageSlug ) {
257 
+                if ( !$sPageSlug ) {
258 258 
                     return '';
259 259 
                 }
260 260
261 261 
                 $_sPageHook = add_submenu_page(
262 
-                    $sRootPageSlug,         // the root (parent) page slug
263 
-                    $sPageTitle,            // page title
264 
-                    $sMenuTitle,            // menu title
265 
-                    $sCapability,           // capability
266 
-                    $sPageSlug,             // menu slug
262 
+                    $sRootPageSlug, // the root (parent) page slug
263 
+                    $sPageTitle, // page title
264 
+                    $sMenuTitle, // menu title
265 
+                    $sCapability, // capability
266 
+                    $sPageSlug, // menu slug
267 267 
                     array( $this->oFactory, '_replyToRenderPage' )  // callback 3.7.10+
268 268 
                 );
269 269 
                 
@@ -283,14 +283,14 @@  discard block
 
 block discarded – undo
283 283 
                 $_aRemovedMenuItem = $this->_removePageSubmenuItem( $_nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle );
284 284
285 285 
                 // If the visibility option is `false`, remove the one just added from the sub-menu global array
286 
-                if ( ! $bShowInMenu && ! $this->_isCurrentPage( $sPageSlug ) ) {
286 
+                if ( !$bShowInMenu && !$this->_isCurrentPage( $sPageSlug ) ) {
287 287 
                     return $_sPageHook;
288 288 
                 }
289 289
290 290 
                 // Set the order index in the element of the `submenu` global array.
291 291 
                 $this->_setSubMenuPageByIndex(
292 
-                    $nOrder,                // user-set order
293 
-                    $_aRemovedMenuItem,     // will be reassign with a new index
292 
+                    $nOrder, // user-set order
293 
+                    $_aRemovedMenuItem, // will be reassign with a new index
294 294 
                     $sMenuSlug
295 295 
                 );
296 296 
 
@@ -329,7 +329,7 @@  discard block
 
 block discarded – undo
329 329 
                      */
330 330 
                     add_action(
331 331 
                         'current_screen',
332 
-                        array( $this->oFactory, "load_pre_" . $sPageSlug ),
332 
+                        array( $this->oFactory, "load_pre_".$sPageSlug ),
333 333 
                         20
334 334 
                     );
335 335 
                     
@@ -338,15 +338,15 @@  discard block
 
 block discarded – undo
338 338 
                      * Set a low priority because the user may add in-page tabs in their callback method of this action hook.
339 339 
                      * @since       3.6.3
340 340 
                      */
341 
-                    add_action( "load_" . $sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 );
341 
+                    add_action( "load_".$sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 );
342 342
343 343 
                     // 3.6.3+
344 
-                    add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) );
345 
-                    add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) );  // 3.7.10+
344 
+                    add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) );
345 
+                    add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) ); // 3.7.10+
346 346
347 347 
                     $this->oFactory->oProp->aPageHooks[ $sPageSlug ] = $this->getAOrB(
348 348 
                         is_network_admin(),
349 
-                        $sPageHook . '-network',
349 
+                        $sPageHook.'-network',
350 350 
                         $sPageHook
351 351 
                     );
352 352 
 
@@ -363,7 +363,7 @@  discard block
 
 block discarded – undo
363 363
364 364 
                     $_nNewIndex = $this->getUnusedNumericIndex(
365 365 
                         $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ), // subject array to parser
366 
-                        $nOrder,    // a desired menu position
366 
+                        $nOrder, // a desired menu position
367 367 
                         5           // offset
368 368 
                     );
369 369 
 
@@ -377,9 +377,9 @@  discard block
 
 block discarded – undo
377 377 
                  */
378 378 
                 private function _getSubMenuPageIndex( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug ) {
379 379
380 
-                    foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
380 
+                    foreach ( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
381 381
382 
-                        if ( ! isset( $_aSubMenu[ 3 ] ) ) {
382 
+                        if ( !isset( $_aSubMenu[ 3 ] ) ) {
383 383 
                             continue;
384 384 
                         }
385 385 
                                                
@@ -411,7 +411,7 @@  discard block
 
 block discarded – undo
411 411 
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`. Chagned the return type.
412 412 
                  * @return      array       removed menu item.
413 413 
                  */
414 
-                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){
414 
+                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) {
415 415
416 416 
                     $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex(
417 417 
                         $nSubMenuPageIndex,
 
@@ -484,7 +484,7 @@  discard block
 
 block discarded – undo
484 484 
              */
485 485 
             private function _addLinkSubmenuItem( $sMenuSlug, $sTitle, $sCapability, $sHref, $bShowInMenu, $nOrder ) {
486 486
487 
-                if ( ! $bShowInMenu ) {
487 
+                if ( !$bShowInMenu ) {
488 488 
                     return;
489 489 
                 }
490 490 
     
@@ -499,9 +499,9 @@  discard block
 
 block discarded – undo
499 499 
                     5   // offset
500 500 
                 );
501 501 
                 $_aSubMenuItems[ $_nIndex ] = array(
502 
-                    $sTitle,        // 0
503 
-                    $sCapability,   // 1
504 
-                    $sHref,         // 2
502 
+                    $sTitle, // 0
503 
+                    $sCapability, // 1
504 
+                    $sHref, // 2
505 505 
                 );
506 506 
                 $GLOBALS[ 'submenu' ][ $sMenuSlug ] = $_aSubMenuItems;
507 507 
                 
@@ -520,7 +520,7 @@  discard block
 
 block discarded – undo
520 520 
     public function _replyToRemoveRootMenuPage() {
521 521
522 522 
         // After adding the sub menus, if the root menu is created, remove the page that is automatically created when registering the root menu.
523 
-        if ( ! $this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
523 
+        if ( !$this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
524 524 
             return;
525 525 
         }
526 526 
         remove_submenu_page(
Please login to merge, or discard this patch.
Braces   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -167,7 +167,7 @@  discard block
 
 block discarded – undo
167 167 
         private function _getDefaultPageSlug( array $aPages ) {
168 168
169 169 
             foreach( $aPages as $_aPage ) {
170 
-                if ( ! isset( $_aPage[ 'page_slug' ] ) ) {
170 
+                if ( ! isset( $_aPage[ 'page_slug' ] ) ) {
171 171 
                     continue;
172 172 
                 }
173 173 
                 return $_aPage[ 'page_slug' ];                
@@ -379,7 +379,7 @@  discard block
 
 block discarded – undo
379 379
380 380 
                     foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) {
381 381
382 
-                        if ( ! isset( $_aSubMenu[ 3 ] ) ) {
382 
+                        if ( ! isset( $_aSubMenu[ 3 ] ) ) {
383 383 
                             continue;
384 384 
                         }
385 385 
                                                
@@ -394,7 +394,7 @@  discard block
 
 block discarded – undo
394 394 
                             $sPageTitle,
395 395 
                             $sPageSlug,
396 396 
                         );
397 
-                        if ( $_aA !== $_aB ) {
397 
+                        if ( $_aA !== $_aB ) {
398 398 
                             continue;
399 399 
                         }
400 400 
                         return $_iIndex;
 
@@ -411,7 +411,7 @@  discard block
 
 block discarded – undo
411 411 
                  * @since       3.1.1       Moved from `AdminPageFramework_Menu`. Chagned the return type.
412 412 
                  * @return      array       removed menu item.
413 413 
                  */
414 
-                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){
414 
+                private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) {
415 415
416 416 
                     $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex(
417 417 
                         $nSubMenuPageIndex,
 
@@ -517,7 +517,7 @@  discard block
 
 block discarded – undo
517 517 
      * @callback    action      admin_menu
518 518 
      * @internal
519 519 
      */
520 
-    public function _replyToRemoveRootMenuPage() {
520 
+    public function _replyToRemoveRootMenuPage() {
521 521
522 522 
         // After adding the sub menus, if the root menu is created, remove the page that is automatically created when registering the root menu.
523 523 
         if ( ! $this->oFactory->oProp->aRootMenu[ 'fCreateRoot' ] ) {
Please login to merge, or discard this patch.
post_type/_model/AdminPageFramework_PostType_Model__FlushRewriteRules.php 4 patches
Doc Comments   +1 added lines patch added patch discarded remove patch 
@@ -26,6 +26,7 @@
 
 block discarded – undo
26 26 
      * Sets up hooks and properties.
27 27 
      *
28 28 
      * @internal
29 
+     * @param AdminPageFramework_PostType_Model $oFactory
29 30 
      */
30 31 
     public function __construct( $oFactory ) {
31 32
Please login to merge, or discard this patch.
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -277,7 +277,7 @@
 
 block discarded – undo
277 277 
         );
278 278 
     }
279 279
280 
-       /**
280 
+        /**
281 281 
          * Extracts meta box form fields options array from the given options array of an admin page.
282 282 
          *
283 283 
          * @since       3.5.6
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch 
@@ -29,7 +29,7 @@  discard block
 
 block discarded – undo
29 29 
      */
30 30 
     public function __construct( $oFactory ) {
31 31
32 
-        if ( ! $this->_shouldProceed( $oFactory ) ) {
32 
+        if ( !$this->_shouldProceed( $oFactory ) ) {
33 33 
             return;
34 34 
         }
35 35 
         
@@ -54,10 +54,10 @@  discard block
 
 block discarded – undo
54 54 
          */
55 55 
         private function _shouldProceed( $oFactory ) {
56 56
57 
-            if ( ! $oFactory->oProp->bIsAdmin ) {
57 
+            if ( !$oFactory->oProp->bIsAdmin ) {
58 58 
                 return false;
59 59 
             }
60 
-            if ( ! $oFactory->oProp->sCallerPath ) {
60 
+            if ( !$oFactory->oProp->sCallerPath ) {
61 61 
                 return false;
62 62 
             }
63 63 
             return 'plugin' === $oFactory->oProp->sScriptType;
 
@@ -87,7 +87,7 @@  discard block
 
 block discarded – undo
87 87 
         }
88 88
89 89 
         // If the execution flow in the plugin activation hook, schedule flushing rewrite rules.
90 
-        if ( did_action( 'activate_' . plugin_basename( $this->oFactory->oProp->sCallerPath ) ) ) {
90 
+        if ( did_action( 'activate_'.plugin_basename( $this->oFactory->oProp->sCallerPath ) ) ) {
91 91 
             add_action( 'shutdown', array( $this, '_replyToFlushRewriteRules' ) );
92 92 
         }
93 93
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -72,7 +72,7 @@
 
 block discarded – undo
72 72 
      * @callback    action      activate_{plugin base name}
73 73 
      * @since       3.7.6
74 74 
      */
75 
-    public function _replyToSetUpPostType() {
75 
+    public function _replyToSetUpPostType() {
76 76 
         do_action( "set_up_{$this->oFactory->oProp->sClassName}", $this );
77 77 
     }
78 78
Please login to merge, or discard this patch.
post_type/_model/AdminPageFramework_PostType_Model__SubMenuOrder.php 3 patches
Doc Comments   +3 added lines patch added patch discarded remove patch 
@@ -26,6 +26,7 @@  discard block
 
 block discarded – undo
26 26 
      * Sets up hooks and properties.
27 27 
      *
28 28 
      * @internal
29 
+     * @param AdminPageFramework_PostType_Model $oFactory
29 30 
      */
30 31 
     public function __construct( $oFactory ) {
31 32 
           
@@ -82,6 +83,7 @@  discard block
 
 block discarded – undo
82 83
83 84 
         /**
84 85 
          * @since       3.7.4
86 
+         * @param string $sSubMenuSlug
85 87 
          */
86 88 
         private function _setSubMenuSlugForSorting( $sSubMenuSlug ) {
87 89 
                 
@@ -94,6 +96,7 @@  discard block
 
 block discarded – undo
94 96
95 97 
         /**
96 98 
          * @since       3.7.4
99 
+         * @param string $sSubMenuSlug
97 100 
          */
98 101 
         private function _setSubMenuItemIndex( $sSubMenuSlug ) {
99 102
Please login to merge, or discard this patch.
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch 
@@ -31,7 +31,7 @@  discard block
 
 block discarded – undo
31 31
32 32 
         $this->oFactory = $oFactory;
33 33
34 
-        if ( ! $oFactory->oProp->bIsAdmin ) {
34 
+        if ( !$oFactory->oProp->bIsAdmin ) {
35 35 
             return;
36 36 
         }
37 37 
         
@@ -47,7 +47,7 @@  discard block
 
 block discarded – undo
47 47
48 48 
         add_action(
49 49 
             'admin_menu',
50 
-            array( $this, 'sortAdminSubMenu' ),  // defined in the framework utility class.
50 
+            array( $this, 'sortAdminSubMenu' ), // defined in the framework utility class.
51 51 
             9999
52 52 
         );
53 53 
             
@@ -62,15 +62,15 @@  discard block
 
 block discarded – undo
62 62
63 63 
         // Check the post type `show_ui` and other related UI arguments.
64 64 
         $_bsShowInMeenu = $this->getShowInMenuPostTypeArgument( $this->oFactory->oProp->aPostTypeArgs );
65 
-        if ( ! $_bsShowInMeenu ) {
65 
+        if ( !$_bsShowInMeenu ) {
66 66 
             return;
67 67 
         }
68 68
69 69 
         // If the user sets a menu slug to the 'show_in_menu' argument, use that.
70 70 
         // It is used to set a custom post type sub-menu belong to another menu.
71 
-        $_sSubMenuSlug  = is_string( $_bsShowInMeenu )
71 
+        $_sSubMenuSlug = is_string( $_bsShowInMeenu )
72 72 
             ? $_bsShowInMeenu
73 
-            : 'edit.php?post_type=' . $this->oFactory->oProp->sPostType;
73 
+            : 'edit.php?post_type='.$this->oFactory->oProp->sPostType;
74 74
75 75 
         // Set the index to the framework specific global array for sorting.
76 76 
         $this->_setSubMenuSlugForSorting( $_sSubMenuSlug );
 
@@ -131,11 +131,11 @@  discard block
 
 block discarded – undo
131 131
132 132 
                 // @remark      This is the partial link url set in the third element ( index of 2 ) in the third dimension of submenu global array element.
133 133 
                 // This is not the submenu slug.
134 
-                $_sLinkSlugManage = 'edit.php?post_type=' . $this->oFactory->oProp->sPostType;
134 
+                $_sLinkSlugManage = 'edit.php?post_type='.$this->oFactory->oProp->sPostType;
135 135
136 136 
                 $_aLinkSlugs = array(
137 137 
                     $_sLinkSlugManage => $_nSubMenuOrderManage,
138 
-                    'post-new.php?post_type=' . $this->oFactory->oProp->sPostType => $_nSubMenuOrderAddNew,
138 
+                    'post-new.php?post_type='.$this->oFactory->oProp->sPostType => $_nSubMenuOrderAddNew,
139 139 
                 );
140 140
141 141 
                 // If the user does not set a custom value, unset it
 
@@ -144,8 +144,8 @@  discard block
 
 block discarded – undo
144 144 
                 }
145 145
146 146 
                 // If the user does not want to show the Add New sub menu, no need to change the order.
147 
-                if ( ! $_bShowAddNew || 10 == $_nSubMenuOrderAddNew ) {
148 
-                    unset( $_aLinkSlugs[ 'post-new.php?post_type=' . $this->oFactory->oProp->sPostType ] );
147 
+                if ( !$_bShowAddNew || 10 == $_nSubMenuOrderAddNew ) {
148 
+                    unset( $_aLinkSlugs[ 'post-new.php?post_type='.$this->oFactory->oProp->sPostType ] );
149 149 
                 }
150 150
151 151 
                 return $_aLinkSlugs;
 
@@ -165,9 +165,9 @@  discard block
 
 block discarded – undo
165 165 
          */
166 166 
         private function _setSubMenuIndexByLinksSlugs( $sSubMenuSlug, array $aLinkSlugs ) {
167 167
168 
-            foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sSubMenuSlug ) ) as $_nIndex => $_aSubMenuItem ) {
168 
+            foreach ( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sSubMenuSlug ) ) as $_nIndex => $_aSubMenuItem ) {
169 169
170 
-                foreach( $aLinkSlugs as $_sLinkSlug => $_nOrder ) {
170 
+                foreach ( $aLinkSlugs as $_sLinkSlug => $_nOrder ) {
171 171
172 172 
                     $_bIsSet = $this->_setSubMenuIndexByLinksSlug( $sSubMenuSlug, $_nIndex, $_aSubMenuItem, $_sLinkSlug, $_nOrder );
173 173 
                     
@@ -193,7 +193,7 @@  discard block
 
 block discarded – undo
193 193 
             private function _setSubMenuIndexByLinksSlug( $sSubMenuSlug, $nIndex, $aSubMenuItem, $sLinkSlug, $nOrder ) {
194 194
195 195 
                 // The third item is the link slug.
196 
-                if ( ! isset( $aSubMenuItem[ 2 ] ) ) {
196 
+                if ( !isset( $aSubMenuItem[ 2 ] ) ) {
197 197 
                     return false;
198 198 
                 }
199 199 
                 if ( $aSubMenuItem[ 2 ] !== $sLinkSlug ) {
Please login to merge, or discard this patch.
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch 
@@ -15,7 +15,7 @@  discard block
 
 block discarded – undo
15 15 
  * @subpackage      PostType
16 16 
  * @internal
17 17 
  */
18 
-class AdminPageFramework_PostType_Model__SubMenuOrder extends AdminPageFramework_FrameworkUtility {
18 
+class AdminPageFramework_PostType_Model__SubMenuOrder extends AdminPageFramework_FrameworkUtility {
19 19
20 20 
     /**
21 21 
      * Stores a post type factory object.
 
@@ -95,7 +95,7 @@  discard block
 
 block discarded – undo
95 95 
         /**
96 96 
          * @since       3.7.4
97 97 
          */
98 
-        private function _setSubMenuItemIndex( $sSubMenuSlug ) {
98 
+        private function _setSubMenuItemIndex( $sSubMenuSlug ) {
99 99
100 100 
             // Only if custom values are set, set them.
101 101 
             $this->_setSubMenuIndexByLinksSlugs( 
@@ -193,7 +193,7 @@  discard block
 
 block discarded – undo
193 193 
             private function _setSubMenuIndexByLinksSlug( $sSubMenuSlug, $nIndex, $aSubMenuItem, $sLinkSlug, $nOrder ) {
194 194
195 195 
                 // The third item is the link slug.
196 
-                if ( ! isset( $aSubMenuItem[ 2 ] ) ) {
196 
+                if ( ! isset( $aSubMenuItem[ 2 ] ) ) {
197 197 
                     return false;
198 198 
                 }
199 199 
                 if ( $aSubMenuItem[ 2 ] !== $sLinkSlug ) {
Please login to merge, or discard this patch.
development/utility/plugin_bootstrap/AdminPageFramework_PluginBootstrap.php 3 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -38,7 +38,7 @@
 
 block discarded – undo
38 38 
      * @param       string      $sPluginFilePath        The plugin file path.
39 39 
      * @param       string      $sPluginHookPrefix      The plugin hook slug without underscore. This will be used to construct hook names.
40 40 
      * @param       string      $sSetUpHook             The action hook name for the setUp callback. Default 'plugins_loaded'.
41 
-     * @param       string      $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
41 
+     * @param       integer      $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
42 42 
      */
43 43 
     public function __construct( $sPluginFilePath, $sPluginHookPrefix='', $sSetUpHook='plugins_loaded', $iPriority=10 ) {
44 44
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -40,7 +40,7 @@  discard block
 
 block discarded – undo
40 40 
      * @param       string      $sSetUpHook             The action hook name for the setUp callback. Default 'plugins_loaded'.
41 41 
      * @param       string      $iPriority              The priority. Set a lower number to get loader earlier. Default: `10`.
42 42 
      */
43 
-    public function __construct( $sPluginFilePath, $sPluginHookPrefix='', $sSetUpHook='plugins_loaded', $iPriority=10 ) {
43 
+    public function __construct( $sPluginFilePath, $sPluginHookPrefix = '', $sSetUpHook = 'plugins_loaded', $iPriority = 10 ) {
44 44
45 45 
         // Check if it has been loaded.
46 46 
         if ( $this->_hasLoaded() ) {
 
@@ -76,7 +76,7 @@  discard block
 
 block discarded – undo
76 76 
         register_deactivation_hook( $this->sFilePath, array( $this, 'replyToPluginDeactivation' ) );
77 77
78 78 
         // 8. Schedule to load plugin specific components.
79 
-        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) )  {
79 
+        if ( !$this->sSetUpHook || did_action( $this->sSetUpHook ) ) {
80 80 
             $this->_replyToLoadPluginComponents();
81 81 
         } else {
82 82 
             add_action( $this->sSetUpHook, array( $this, '_replyToLoadPluginComponents' ), $this->iPriority );
 
@@ -116,14 +116,14 @@  discard block
 
 block discarded – undo
116 116 
         protected function _registerClasses() {
117 117
118 118 
             // This class should be used in the framework bootstrap so disabling the auto-load option for performance.
119 
-            if ( ! class_exists( 'AdminPageFramework_RegisterClasses', false ) ) {
119 
+            if ( !class_exists( 'AdminPageFramework_RegisterClasses', false ) ) {
120 120 
                 return;
121 121 
             }
122 122
123 123 
             // Register classes
124 124 
             new AdminPageFramework_RegisterClasses(
125 
-                $this->getScanningDirs(),   // scanning directory paths
126 
-                array(),                    // autoloader options
125 
+                $this->getScanningDirs(), // scanning directory paths
126 
+                array(), // autoloader options
127 127 
                 $this->getClasses()         // pre-generated class list
128 128 
             );
129 129
Please login to merge, or discard this patch.
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -76,7 +76,7 @@  discard block
 
 block discarded – undo
76 76 
         register_deactivation_hook( $this->sFilePath, array( $this, 'replyToPluginDeactivation' ) );
77 77
78 78 
         // 8. Schedule to load plugin specific components.
79 
-        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) )  {
79 
+        if ( ! $this->sSetUpHook || did_action( $this->sSetUpHook ) ) {
80 80 
             $this->_replyToLoadPluginComponents();
81 81 
         } else {
82 82 
             add_action( $this->sSetUpHook, array( $this, '_replyToLoadPluginComponents' ), $this->iPriority );
 
@@ -100,7 +100,7 @@  discard block
 
 block discarded – undo
100 100 
         protected function _hasLoaded() {
101 101
102 102 
             static $_bLoaded = false;
103 
-            if ( $_bLoaded ) {
103 
+            if ( $_bLoaded ) {
104 104 
                 return true;
105 105 
             }
106 106 
             $_bLoaded = true;
Please login to merge, or discard this patch.
development/utility/pointer_tool_tip/AdminPageFramework_PointerToolTip.php 4 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -66,7 +66,7 @@
 
 block discarded – undo
66 66 
      *
67 67 
      * @since       DEVVER
68 68 
      * @see         https://codex.wordpress.org/Plugin_API/Admin_Screen_Reference
69 
-     * @param       array|strin     $asScreenIDs        Screen IDs or page slug.
69 
+     * @param       string[]     $asScreenIDs        Screen IDs or page slug.
70 70 
      * @param       string          $sPointerID         A unique pointer ID.
71 71 
      * @Param       array           $aPointerData       The pointer data.
72 72 
      */
Please login to merge, or discard this patch.
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -173,7 +173,7 @@
 
 block discarded – undo
173 173 
                     get_current_user_id(),
174 174 
                     'dismissed_wp_pointers',
175 175 
                     true
176 
-               )
176 
+                )
177 177 
             );
178 178 
             $_aValidPointers = array(
179 179 
                 'pointers'  => array(),
Please login to merge, or discard this patch.
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -62,7 +62,7 @@  discard block
 
 block discarded – undo
62 62 
     public function __construct( $asScreenIDs, $sPointerID, array $aPointerData ) {
63 63
64 64 
         // Bail if the WordPress version is less than 3.3,
65 
-        if ( version_compare( $GLOBALS[ 'wp_version' ], '3.3', '<' ) ) {
65 
+        if ( version_compare( $GLOBALS[ 'wp_version' ], '3.3', '<' ) ) {
66 66 
             return false;
67 67 
         }
68 68 
     
@@ -93,7 +93,7 @@  discard block
 
 block discarded – undo
93 93 
          *
94 94 
          */
95 95 
         private function _setHooks( $aScreenIDs ) {
96 
-            foreach( $aScreenIDs as $_sScreenID ) {
96 
+            foreach( $aScreenIDs as $_sScreenID ) {
97 97 
                 if ( ! $_sScreenID ) {
98 98 
                     continue;
99 99 
                 }
Please login to merge, or discard this patch.
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch 
@@ -112,18 +112,18 @@  discard block
 
 block discarded – undo
112 112 
          */
113 113 
         private function _setHooks( $aScreenIDs ) {
114 114
115 
-            foreach( $aScreenIDs as $_sScreenID ) {
116 
-                if ( ! $_sScreenID ) {
115 
+            foreach ( $aScreenIDs as $_sScreenID ) {
116 
+                if ( !$_sScreenID ) {
117 117 
                     continue;
118 118 
                 }
119 119 
                 add_filter(
120 
-                    get_class( $this ) . '-' . $_sScreenID,
120 
+                    get_class( $this ).'-'.$_sScreenID,
121 121 
                     array( $this, '_replyToSetPointer' )
122 122 
                 );
123 123
124 124 
             }
125 125
126 
-            if ( ! $this->_hasBeenCalled() ) {
126 
+            if ( !$this->_hasBeenCalled() ) {
127 127 
                 return;
128 128 
             }
129 129 
             
@@ -170,7 +170,7 @@  discard block
 
 block discarded – undo
170 170
171 171 
         $_aPointers = $this->_getValidPointers( $this->_getPointers() );
172 172
173 
-        if ( empty( $_aPointers ) || ! is_array( $_aPointers ) ) {
173 
+        if ( empty( $_aPointers ) || !is_array( $_aPointers ) ) {
174 174 
             return;
175 175 
         }
176 176 
         
@@ -190,11 +190,11 @@  discard block
 
 block discarded – undo
190 190 
             $_oScreen   = get_current_screen();
191 191 
             $_sScreenID = $_oScreen->id;
192 192 
             if ( in_array( $_sScreenID, $this->aScreenIDs ) ) {
193 
-                return apply_filters( get_class( $this ) . '-' . $_sScreenID, array() );
193 
+                return apply_filters( get_class( $this ).'-'.$_sScreenID, array() );
194 194 
             }
195 195
196 196 
             if ( isset( $_GET[ 'page' ] ) ) {
197 
-                return apply_filters( get_class( $this ) . '-' . $_GET[ 'page' ], array() );
197 
+                return apply_filters( get_class( $this ).'-'.$_GET[ 'page' ], array() );
198 198 
             }
199 199 
             return array();
200 200 
             
@@ -208,7 +208,7 @@  discard block
 
 block discarded – undo
208 208 
         private function _getValidPointers( $_aPointers ) {
209 209
210 210 
             // Get dismissed pointers
211 
-            $_aDismissed      = explode(
211 
+            $_aDismissed = explode(
212 212 
                 ',',
213 213 
                 ( string ) get_user_meta(
214 214 
                     get_current_user_id(), 
@@ -236,7 +236,7 @@  discard block
 
 block discarded – undo
236 236 
                 $_aPointer[ 'pointer_id' ] = $_iPointerID;
237 237
238 238 
                 // Add the pointer to $_aValidPointers array
239 
-                $_aValidPointers[] =  $_aPointer;
239 
+                $_aValidPointers[ ] = $_aPointer;
240 240
241 241 
             }
242 242 
             return $_aValidPointers;
 
@@ -312,9 +312,9 @@  discard block
 
 block discarded – undo
312 312 
          * @return      string
313 313 
          * @internal
314 314 
          */
315 
-        public function _getInternalScript( $aPointers=array() ) {
315 
+        public function _getInternalScript( $aPointers = array() ) {
316 316
317 
-            $_aJSArray      = json_encode( $aPointers );
317 
+            $_aJSArray = json_encode( $aPointers );
318 318
319 319 
             /**
320 320 
              * Checks check-boxes in siblings.
Please login to merge, or discard this patch.