@@ -122,7 +122,7 @@ discard block |
||
122 | 122 | * @internal |
123 | 123 | * @param string $sInputID The input ID that will be the base of each generated option tag ID. |
124 | 124 | * @param array $aLabels The array holding labels. |
125 | - * @param array $aAttributes The attribute arrays. Accepts the following arguments. |
|
125 | + * @param array $aBaseAttributes The attribute arrays. Accepts the following arguments. |
|
126 | 126 | * - optgroup |
127 | 127 | * - option |
128 | 128 | */ |
@@ -160,6 +160,7 @@ discard block |
||
160 | 160 | /** |
161 | 161 | * Returns an HTML output of optgroup tag. |
162 | 162 | * @since 3.5.3 |
163 | + * @param string $sInputID |
|
163 | 164 | * @return string an HTML output of optgroup tag. |
164 | 165 | */ |
165 | 166 | private function _getOptGroup( array $aBaseAttributes, $sInputID, $sKey, $asLabel ) { |
@@ -179,6 +180,7 @@ discard block |
||
179 | 180 | /** |
180 | 181 | * |
181 | 182 | * @since 3.5.3 |
183 | + * @param string $sInputID |
|
182 | 184 | */ |
183 | 185 | private function _getOptionTagAttributes( array $aBaseAttributes, $sInputID, $sKey, $aValues ) { |
184 | 186 |
@@ -69,14 +69,14 @@ discard block |
||
69 | 69 | ); |
70 | 70 | |
71 | 71 | return |
72 | - "<{$this->aOptions[ 'input_container_tag' ]} " . $this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ) . ">" |
|
73 | - . "<select " . $this->getAttributes( $this->_getSelectAttributes( $_aAttributes ) ) . " >" |
|
72 | + "<{$this->aOptions[ 'input_container_tag' ]} ".$this->getAttributes( $this->aOptions[ 'input_container_attributes' ] ).">" |
|
73 | + . "<select ".$this->getAttributes( $this->_getSelectAttributes( $_aAttributes ) )." >" |
|
74 | 74 | . $this->_getDropDownList( |
75 | 75 | $this->getAttribute( 'id' ), |
76 | 76 | $this->getAsArray( |
77 | 77 | isset( $_aLabels ) |
78 | 78 | ? $_aLabels |
79 | - : $this->aField[ 'label' ], // backward compatibility |
|
79 | + : $this->aField[ 'label' ], // backward compatibility |
|
80 | 80 | true |
81 | 81 | ), |
82 | 82 | $_aAttributes |
@@ -104,9 +104,9 @@ discard block |
||
104 | 104 | ? 'multiple' |
105 | 105 | : null, |
106 | 106 | 'name' => $_bIsMultiple |
107 | - ? $this->getAttribute( 'name' ) . '[]' |
|
107 | + ? $this->getAttribute( 'name' ).'[]' |
|
108 | 108 | : $this->getAttribute( 'name' ), |
109 | - 'data-id' => $this->getAttribute( 'id' ), // referenced by the JavaScript scripts such as the revealer script. |
|
109 | + 'data-id' => $this->getAttribute( 'id' ), // referenced by the JavaScript scripts such as the revealer script. |
|
110 | 110 | ) |
111 | 111 | ); |
112 | 112 | |
@@ -128,12 +128,12 @@ discard block |
||
128 | 128 | */ |
129 | 129 | private function _getDropDownList( $sInputID, array $aLabels, array $aBaseAttributes ) { |
130 | 130 | |
131 | - $_aOutput = array(); |
|
132 | - foreach( $aLabels as $__sKey => $__asLabel ) { |
|
131 | + $_aOutput = array(); |
|
132 | + foreach ( $aLabels as $__sKey => $__asLabel ) { |
|
133 | 133 | |
134 | 134 | // For an optgroup tag, |
135 | 135 | if ( is_array( $__asLabel ) ) { |
136 | - $_aOutput[] = $this->_getOptGroup( |
|
136 | + $_aOutput[ ] = $this->_getOptGroup( |
|
137 | 137 | $aBaseAttributes, |
138 | 138 | $sInputID, |
139 | 139 | $__sKey, |
@@ -143,8 +143,8 @@ discard block |
||
143 | 143 | } |
144 | 144 | |
145 | 145 | // A normal option tag, |
146 | - $_aOutput[] = $this->_getOptionTag( |
|
147 | - $__asLabel, // the text label the user sees to be selected |
|
146 | + $_aOutput[ ] = $this->_getOptionTag( |
|
147 | + $__asLabel, // the text label the user sees to be selected |
|
148 | 148 | $this->_getOptionTagAttributes( |
149 | 149 | $aBaseAttributes, |
150 | 150 | $sInputID, |
@@ -170,7 +170,7 @@ discard block |
||
170 | 170 | $_aOptGroupAttributes = array( |
171 | 171 | 'label' => $sKey, |
172 | 172 | ) + $_aOptGroupAttributes; |
173 | - return "<optgroup " . $this->getAttributes( $_aOptGroupAttributes ) . ">" |
|
173 | + return "<optgroup ".$this->getAttributes( $_aOptGroupAttributes ).">" |
|
174 | 174 | . $this->_getDropDownList( $sInputID, $asLabel, $aBaseAttributes ) |
175 | 175 | . "</optgroup>"; |
176 | 176 | |
@@ -188,7 +188,7 @@ discard block |
||
188 | 188 | $aValues |
189 | 189 | ); |
190 | 190 | return array( |
191 | - 'id' => $sInputID . '_' . $sKey, |
|
191 | + 'id' => $sInputID.'_'.$sKey, |
|
192 | 192 | 'value' => $sKey, |
193 | 193 | 'selected' => in_array( ( string ) $sKey, $aValues ) |
194 | 194 | ? 'selected' |
@@ -204,8 +204,8 @@ discard block |
||
204 | 204 | * @sicne 3.4.0 |
205 | 205 | * @return string The generated option tag HTML output. |
206 | 206 | */ |
207 | - private function _getOptionTag( $sLabel, array $aOptionTagAttributes=array() ) { |
|
208 | - return "<option " . $this->getAttributes( $aOptionTagAttributes ) . " >" |
|
207 | + private function _getOptionTag( $sLabel, array $aOptionTagAttributes = array() ) { |
|
208 | + return "<option ".$this->getAttributes( $aOptionTagAttributes )." >" |
|
209 | 209 | . $sLabel |
210 | 210 | . "</option>"; |
211 | 211 | } |
@@ -95,7 +95,7 @@ |
||
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 |
@@ -41,7 +41,7 @@ discard block |
||
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]+#", '# #', '# +#', '# class=".*?"#', '%'%', |
46 | 46 | '#<tr>(?:.*?)" src="(?:.*?)=(.*?)" alt="PHP Logo" /></a>' |
47 | 47 | .'<h1>PHP Version (.*?)</h1>(?:\n+?)</td></tr>#', |
@@ -52,11 +52,11 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -23,6 +23,7 @@ discard block |
||
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 |
||
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 ) { |
@@ -52,7 +52,7 @@ discard block |
||
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 |
||
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 |
||
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 |
@@ -89,11 +89,6 @@ discard block |
||
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 |
||
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 |
||
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 ) { |
@@ -26,7 +26,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 ) |
@@ -102,7 +102,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
@@ -284,6 +284,7 @@ |
||
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( |
@@ -29,7 +29,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 ); |
@@ -57,7 +57,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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, |
@@ -27,6 +27,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 ) { |
@@ -28,7 +28,7 @@ discard block |
||
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 | |
@@ -71,8 +71,8 @@ discard block |
||
71 | 71 | |
72 | 72 | // Apply filters to let other scripts add sub menu pages. |
73 | 73 | $this->oFactory->oProp->aPages = $this->addAndApplyFilter( |
74 | - $this->oFactory, // caller object |
|
75 | - "pages_{$this->oFactory->oProp->sClassName}", // filter |
|
74 | + $this->oFactory, // caller object |
|
75 | + "pages_{$this->oFactory->oProp->sClassName}", // filter |
|
76 | 76 | $this->oFactory->oProp->aPages // arguments |
77 | 77 | ); |
78 | 78 | |
@@ -120,7 +120,7 @@ discard block |
||
120 | 120 | |
121 | 121 | foreach ( $this->oFactory->oProp->aPages as $_aPage ) { |
122 | 122 | |
123 | - if ( ! isset( $_aPage[ 'page_slug' ] ) ) { |
|
123 | + if ( !isset( $_aPage[ 'page_slug' ] ) ) { |
|
124 | 124 | continue; |
125 | 125 | } |
126 | 126 | $this->oFactory->oProp->sDefaultPageSlug = $_aPage[ 'page_slug' ]; |
@@ -140,12 +140,12 @@ discard block |
||
140 | 140 | private function _registerRootMenuPage() { |
141 | 141 | |
142 | 142 | $this->oFactory->oProp->aRootMenu[ '_page_hook' ] = add_menu_page( |
143 | - $this->oFactory->oProp->sClassName, // Page title - will be invisible anyway |
|
144 | - $this->oFactory->oProp->aRootMenu[ 'sTitle' ], // Menu title - should be the root page title. |
|
145 | - $this->oFactory->oProp->sCapability, // Capability - access right |
|
146 | - $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ], // Menu ID |
|
147 | - '', // Callback function for the page content output - the root page will be removed so no need to register a function. |
|
148 | - $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ], // icon path |
|
143 | + $this->oFactory->oProp->sClassName, // Page title - will be invisible anyway |
|
144 | + $this->oFactory->oProp->aRootMenu[ 'sTitle' ], // Menu title - should be the root page title. |
|
145 | + $this->oFactory->oProp->sCapability, // Capability - access right |
|
146 | + $this->oFactory->oProp->aRootMenu[ 'sPageSlug' ], // Menu ID |
|
147 | + '', // Callback function for the page content output - the root page will be removed so no need to register a function. |
|
148 | + $this->oFactory->oProp->aRootMenu[ 'sIcon16x16' ], // icon path |
|
149 | 149 | $this->getElement( |
150 | 150 | $this->oFactory->oProp->aRootMenu, |
151 | 151 | 'iPosition', |
@@ -168,7 +168,7 @@ discard block |
||
168 | 168 | */ |
169 | 169 | private function _registerSubMenuItem( array $aArgs ) { |
170 | 170 | |
171 | - if ( ! current_user_can( $aArgs[ 'capability' ] ) ) { |
|
171 | + if ( !current_user_can( $aArgs[ 'capability' ] ) ) { |
|
172 | 172 | return ''; |
173 | 173 | } |
174 | 174 | |
@@ -233,16 +233,16 @@ discard block |
||
233 | 233 | */ |
234 | 234 | private function _addPageSubmenuItem( $sRootPageSlug, $sMenuSlug, $sPageSlug, $sPageTitle, $sMenuTitle, $sCapability, $bShowInMenu, $nOrder ) { |
235 | 235 | |
236 | - if ( ! $sPageSlug ) { |
|
236 | + if ( !$sPageSlug ) { |
|
237 | 237 | return ''; |
238 | 238 | } |
239 | 239 | |
240 | 240 | $_sPageHook = add_submenu_page( |
241 | - $sRootPageSlug, // the root (parent) page slug |
|
242 | - $sPageTitle, // page title |
|
243 | - $sMenuTitle, // menu title |
|
244 | - $sCapability, // capability |
|
245 | - $sPageSlug, // menu slug |
|
241 | + $sRootPageSlug, // the root (parent) page slug |
|
242 | + $sPageTitle, // page title |
|
243 | + $sMenuTitle, // menu title |
|
244 | + $sCapability, // capability |
|
245 | + $sPageSlug, // menu slug |
|
246 | 246 | /** |
247 | 247 | * In admin.php ( line 149 of WordPress v3.6.1 ), do_action($page_hook) ( where $page_hook is $_sPageHook ) |
248 | 248 | * will be executed and it triggers the __call() magic method with the method name of "md5 class hash + _page_ + this page slug". |
@@ -267,14 +267,14 @@ discard block |
||
267 | 267 | $_aRemovedMenuItem = $this->_removePageSubmenuItem( $_nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ); |
268 | 268 | |
269 | 269 | // If the visibility option is `false`, remove the one just added from the sub-menu global array |
270 | - if ( ! $bShowInMenu && ! $this->_isCurrentPage( $sPageSlug ) ) { |
|
270 | + if ( !$bShowInMenu && !$this->_isCurrentPage( $sPageSlug ) ) { |
|
271 | 271 | return $_sPageHook; |
272 | 272 | } |
273 | 273 | |
274 | 274 | // Set the order index in the element of the `submenu` global array. |
275 | 275 | $this->_setSubMenuPageByIndex( |
276 | - $nOrder, // user-set order |
|
277 | - $_aRemovedMenuItem, // will be reassign with a new index |
|
276 | + $nOrder, // user-set order |
|
277 | + $_aRemovedMenuItem, // will be reassign with a new index |
|
278 | 278 | $sMenuSlug |
279 | 279 | ); |
280 | 280 | |
@@ -313,7 +313,7 @@ discard block |
||
313 | 313 | */ |
314 | 314 | add_action( |
315 | 315 | 'current_screen', |
316 | - array( $this->oFactory, "load_pre_" . $sPageSlug ), |
|
316 | + array( $this->oFactory, "load_pre_".$sPageSlug ), |
|
317 | 317 | 20 |
318 | 318 | ); |
319 | 319 | |
@@ -322,15 +322,15 @@ discard block |
||
322 | 322 | * Set a low priority because the user may add in-page tabs in their callback method of this action hook. |
323 | 323 | * @since 3.6.3 |
324 | 324 | */ |
325 | - add_action( "load_" . $sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 ); |
|
325 | + add_action( "load_".$sPageSlug, array( $this->oFactory, '_replyToFinalizeInPageTabs' ), 9999 ); |
|
326 | 326 | |
327 | 327 | // 3.6.3+ |
328 | - add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) ); |
|
329 | - add_action( "load_after_" . $sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) ); // 3.7.10+ |
|
328 | + add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnqueuePageAssets' ) ); |
|
329 | + add_action( "load_after_".$sPageSlug, array( $this->oFactory, '_replyToEnablePageMetaBoxes' ) ); // 3.7.10+ |
|
330 | 330 | |
331 | 331 | $this->oFactory->oProp->aPageHooks[ $sPageSlug ] = $this->getAOrB( |
332 | 332 | is_network_admin(), |
333 | - $sPageHook . '-network', |
|
333 | + $sPageHook.'-network', |
|
334 | 334 | $sPageHook |
335 | 335 | ); |
336 | 336 | |
@@ -347,7 +347,7 @@ discard block |
||
347 | 347 | |
348 | 348 | $_nNewIndex = $this->getUnusedNumericIndex( |
349 | 349 | $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ), // subject array to parser |
350 | - $nOrder, // a desired menu position |
|
350 | + $nOrder, // a desired menu position |
|
351 | 351 | 5 // offset |
352 | 352 | ); |
353 | 353 | |
@@ -361,9 +361,9 @@ discard block |
||
361 | 361 | */ |
362 | 362 | private function _getSubMenuPageIndex( $sMenuSlug, $sMenuTitle, $sPageTitle, $sPageSlug ) { |
363 | 363 | |
364 | - foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) { |
|
364 | + foreach ( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) { |
|
365 | 365 | |
366 | - if ( ! isset( $_aSubMenu[ 3 ] ) ) { |
|
366 | + if ( !isset( $_aSubMenu[ 3 ] ) ) { |
|
367 | 367 | continue; |
368 | 368 | } |
369 | 369 | |
@@ -395,7 +395,7 @@ discard block |
||
395 | 395 | * @since 3.1.1 Moved from `AdminPageFramework_Menu`. Chagned the return type. |
396 | 396 | * @return array removed menu item. |
397 | 397 | */ |
398 | - private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){ |
|
398 | + private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) { |
|
399 | 399 | |
400 | 400 | $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex( |
401 | 401 | $nSubMenuPageIndex, |
@@ -468,7 +468,7 @@ discard block |
||
468 | 468 | */ |
469 | 469 | private function _addLinkSubmenuItem( $sMenuSlug, $sTitle, $sCapability, $sHref, $bShowInMenu, $nOrder ) { |
470 | 470 | |
471 | - if ( ! $bShowInMenu ) { |
|
471 | + if ( !$bShowInMenu ) { |
|
472 | 472 | return; |
473 | 473 | } |
474 | 474 | |
@@ -483,9 +483,9 @@ discard block |
||
483 | 483 | 5 // offset |
484 | 484 | ); |
485 | 485 | $_aSubMenuItems[ $_nIndex ] = array( |
486 | - $sTitle, // 0 |
|
487 | - $sCapability, // 1 |
|
488 | - $sHref, // 2 |
|
486 | + $sTitle, // 0 |
|
487 | + $sCapability, // 1 |
|
488 | + $sHref, // 2 |
|
489 | 489 | ); |
490 | 490 | $GLOBALS[ 'submenu' ][ $sMenuSlug ] = $_aSubMenuItems; |
491 | 491 |
@@ -120,7 +120,7 @@ discard block |
||
120 | 120 | |
121 | 121 | foreach ( $this->oFactory->oProp->aPages as $_aPage ) { |
122 | 122 | |
123 | - if ( ! isset( $_aPage[ 'page_slug' ] ) ) { |
|
123 | + if ( ! isset( $_aPage[ 'page_slug' ] ) ) { |
|
124 | 124 | continue; |
125 | 125 | } |
126 | 126 | $this->oFactory->oProp->sDefaultPageSlug = $_aPage[ 'page_slug' ]; |
@@ -363,7 +363,7 @@ discard block |
||
363 | 363 | |
364 | 364 | foreach( $this->getElementAsArray( $GLOBALS, array( 'submenu', $sMenuSlug ) ) as $_iIndex => $_aSubMenu ) { |
365 | 365 | |
366 | - if ( ! isset( $_aSubMenu[ 3 ] ) ) { |
|
366 | + if ( ! isset( $_aSubMenu[ 3 ] ) ) { |
|
367 | 367 | continue; |
368 | 368 | } |
369 | 369 | |
@@ -378,7 +378,7 @@ discard block |
||
378 | 378 | $sPageTitle, |
379 | 379 | $sPageSlug, |
380 | 380 | ); |
381 | - if ( $_aA !== $_aB ) { |
|
381 | + if ( $_aA !== $_aB ) { |
|
382 | 382 | continue; |
383 | 383 | } |
384 | 384 | return $_iIndex; |
@@ -395,7 +395,7 @@ discard block |
||
395 | 395 | * @since 3.1.1 Moved from `AdminPageFramework_Menu`. Chagned the return type. |
396 | 396 | * @return array removed menu item. |
397 | 397 | */ |
398 | - private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ){ |
|
398 | + private function _removePageSubmenuItem( $nSubMenuPageIndex, $sMenuSlug, $sPageSlug, $sMenuTitle ) { |
|
399 | 399 | |
400 | 400 | $_aRemovedMenuItem = $this->_removePageSubMenuItemByIndex( |
401 | 401 | $nSubMenuPageIndex, |
@@ -26,6 +26,7 @@ |
||
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 |
@@ -277,7 +277,7 @@ |
||
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 |
@@ -29,7 +29,7 @@ discard block |
||
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 |
||
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 |
||
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 |
@@ -72,7 +72,7 @@ |
||
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 |
@@ -26,6 +26,7 @@ discard block |
||
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 |
||
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 |
||
94 | 96 | |
95 | 97 | /** |
96 | 98 | * @since 3.7.4 |
99 | + * @param string $sSubMenuSlug |
|
97 | 100 | */ |
98 | 101 | private function _setSubMenuItemIndex( $sSubMenuSlug ) { |
99 | 102 |
@@ -31,7 +31,7 @@ discard block |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 |
||
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 ) { |
@@ -15,7 +15,7 @@ discard block |
||
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 |
||
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 |
||
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 ) { |
@@ -38,7 +38,7 @@ |
||
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 |
@@ -40,7 +40,7 @@ discard block |
||
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 |
||
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 |
||
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 |
@@ -76,7 +76,7 @@ discard block |
||
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 |
||
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; |