r113589 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:< r113588‎ | r113589 | r113590 >
Date:21:01, 11 March 2012
Author:reedy
Status:reverted
Tags:gerritmigration 
Comment:
Few more types top flesh out the previous revisions
Modified paths:
  • /trunk/extensions/AbuseFilter/AbuseFilter.class.php (modified) (history)
  • /trunk/extensions/AbuseFilter/AbuseFilter.parser.php (modified) (history)

Diff [purge]

Index: trunk/extensions/AbuseFilter/AbuseFilter.parser.php
——@@ -74,6 +74,10 @@
7575 var $type;
7676 var $data;
7777
 78+ /**
 79+ * @param string $type
 80+ * @param null $val
 81+ */
7882 public function __construct( $type = self::DNull, $val = null ) {
7983 $this->type = $type;
8084 $this->data = $val;
——@@ -116,8 +120,7 @@
117121 }
118122
119123 /**
120 - * @static
121 - * @param $orig
 124+ * @param $orig AFPData
122125 * @param $target
123126 * @return AFPData
124127 */
——@@ -166,7 +169,7 @@
167170 }
168171
169172 /**
170 - * @param $value
 173+ * @param $value AFPData
171174 * @return AFPData
172175 */
173176 public static function boolInvert( $value ) {
——@@ -174,8 +177,8 @@
175178 }
176179
177180 /**
178 - * @param $base
179 - * @param $exponent
 181+ * @param $base AFPData
 182+ * @param $exponent AFPData
180183 * @return AFPData
181184 */
182185 public static function pow( $base, $exponent ) {
——@@ -183,8 +186,8 @@
184187 }
185188
186189 /**
187 - * @param $a
188 - * @param $b
 190+ * @param $a AFPData
 191+ * @param $b AFPData
189192 * @return AFPData
190193 */
191194 public static function keywordIn( $a, $b ) {
——@@ -199,8 +202,8 @@
200203 }
201204
202205 /**
203 - * @param $a
204 - * @param $b
 206+ * @param $a AFPData
 207+ * @param $b AFPData
205208 * @return AFPData
206209 */
207210 public static function keywordContains( $a, $b ) {
——@@ -230,8 +233,8 @@
231234 }
232235
233236 /**
234 - * @param $d1
235 - * @param $d2
 237+ * @param $d1 AFPData
 238+ * @param $d2 AFPData
236239 * @return bool
237240 */
238241 public static function equals( $d1, $d2 ) {
——@@ -240,8 +243,8 @@
241244 }
242245
243246 /**
244 - * @param $str
245 - * @param $pattern
 247+ * @param $str AFPData
 248+ * @param $pattern AFPData
246249 * @return AFPData
247250 */
248251 public static function keywordLike( $str, $pattern ) {
——@@ -254,8 +257,8 @@
255258 }
256259
257260 /**
258 - * @param $str
259 - * @param $regex
 261+ * @param $str AFPData
 262+ * @param $regex AFPData
260263 * @param $pos
261264 * @param $insensitive bool
262265 * @return AFPData
——@@ -295,7 +298,7 @@
296299 }
297300
298301 /**
299 - * @param $data
 302+ * @param $data AFPData
300303 * @return AFPData
301304 */
302305 public static function unaryMinus( $data ) {
——@@ -307,9 +310,9 @@
308311 }
309312
310313 /**
311 - * @param $a
312 - * @param $b
313 - * @param $op
 314+ * @param $a AFPData
 315+ * @param $b AFPData
 316+ * @param $op string
314317 * @return AFPData
315318 * @throws AFPException
316319 */
——@@ -329,9 +332,9 @@
330333 }
331334
332335 /**
333 - * @param $a
334 - * @param $b
335 - * @param $op
 336+ * @param $a AFPData
 337+ * @param $b AFPData
 338+ * @param $op string
336339 * @return AFPData
337340 * @throws AFPException
338341 */
——@@ -366,9 +369,9 @@
367370 }
368371
369372 /**
370 - * @param $a
371 - * @param $b
372 - * @param $op
 373+ * @param $a AFPData
 374+ * @param $b AFPData
 375+ * @param $op string
373376 * @param $pos
374377 * @return AFPData
375378 * @throws AFPUserVisibleException
——@@ -411,8 +414,8 @@
412415 }
413416
414417 /**
415 - * @param $a
416 - * @param $b
 418+ * @param $a AFPData
 419+ * @param $b AFPData
417420 * @return AFPData
418421 */
419422 public static function sum( $a, $b ) {
——@@ -426,8 +429,8 @@
427430 }
428431
429432 /**
430 - * @param $a
431 - * @param $b
 433+ * @param $a AFPData
 434+ * @param $b AFPData
432435 * @return AFPData
433436 */
434437 public static function sub( $a, $b ) {
——@@ -1002,7 +1005,7 @@
10031006 }
10041007
10051008 /**
1006 - * @param $result
 1009+ * @param $result AFPData
10071010 */
10081011 protected function doLevelBoolOps( &$result ) {
10091012 $this->doLevelCompares( $result );
——@@ -1849,7 +1852,7 @@
18501853 }
18511854
18521855 /**
1853 - * @param $args
 1856+ * @param $args array
18541857 * @return AFPData
18551858 * @throws AFPUserVisibleException
18561859 */
Index: trunk/extensions/AbuseFilter/AbuseFilter.class.php
——@@ -1035,9 +1035,9 @@
10361036 }
10371037
10381038 /**
1039 - * @param $action
1040 - * @param $parameters
1041 - * @param $title
 1039+ * @param $action string
 1040+ * @param $parameters array
 1041+ * @param $title Title
10421042 * @param $vars AbuseFilterVariableHolder
10431043 * @param $rule_desc
10441044 * @return string

Follow-up revisions

RevisionCommit summaryAuthorDate
r114396Revert r111217 (unreviewed rev in AbuseFilter) and its dependencies r113585, ...catrope19:41, 21 March 2012

Status & tagging log