| Index: trunk/phase3/includes/revisiondelete/RevisionDelete.php |
| — | — | @@ -10,6 +10,10 @@ |
| 11 | 11 | var $authorIdField = 'rev_user'; |
| 12 | 12 | var $authorNameField = 'rev_user_text'; |
| 13 | 13 | |
| | 14 | + /** |
| | 15 | + * @param $db DatabaseBase |
| | 16 | + * @return mixed |
| | 17 | + */ |
| 14 | 18 | public function doQuery( $db ) { |
| 15 | 19 | $ids = array_map( 'intval', $this->ids ); |
| 16 | 20 | return $db->select( array('revision','page'), '*', |
| — | — | @@ -186,6 +190,10 @@ |
| 187 | 191 | var $authorIdField = 'ar_user'; |
| 188 | 192 | var $authorNameField = 'ar_user_text'; |
| 189 | 193 | |
| | 194 | + /** |
| | 195 | + * @param $db DatabaseBase |
| | 196 | + * @return mixed |
| | 197 | + */ |
| 190 | 198 | public function doQuery( $db ) { |
| 191 | 199 | $timestamps = array(); |
| 192 | 200 | foreach ( $this->ids as $id ) { |
| — | — | @@ -348,6 +356,10 @@ |
| 349 | 357 | * Item class for an oldimage table row |
| 350 | 358 | */ |
| 351 | 359 | class RevDel_FileItem extends RevDel_Item { |
| | 360 | + |
| | 361 | + /** |
| | 362 | + * @var File |
| | 363 | + */ |
| 352 | 364 | var $file; |
| 353 | 365 | |
| 354 | 366 | public function __construct( $list, $row ) { |
| Index: trunk/phase3/includes/revisiondelete/RevisionDeleteAbstracts.php |
| — | — | @@ -4,7 +4,13 @@ |
| 5 | 5 | * Abstract base class for a list of deletable items |
| 6 | 6 | */ |
| 7 | 7 | abstract class RevDel_List { |
| 8 | | - var $special, $title, $ids, $res, $current; |
| | 8 | + |
| | 9 | + /** |
| | 10 | + * @var Title |
| | 11 | + */ |
| | 12 | + var $title; |
| | 13 | + |
| | 14 | + var $special, $ids, $res, $current; |
| 9 | 15 | var $type = null; // override this |
| 10 | 16 | var $idField = null; // override this |
| 11 | 17 | var $dateField = false; // override this |
| — | — | @@ -329,7 +335,7 @@ |
| 330 | 336 | |
| 331 | 337 | /** |
| 332 | 338 | * Do the DB query to iterate through the objects. |
| 333 | | - * @param $db Database object to use for the query |
| | 339 | + * @param $db DatabaseBase object to use for the query |
| 334 | 340 | */ |
| 335 | 341 | abstract public function doQuery( $db ); |
| 336 | 342 | |
| Index: trunk/phase3/includes/job/JobQueue.php |
| — | — | @@ -16,8 +16,13 @@ |
| 17 | 17 | * @ingroup JobQueue |
| 18 | 18 | */ |
| 19 | 19 | abstract class Job { |
| | 20 | + |
| | 21 | + /** |
| | 22 | + * @var Title |
| | 23 | + */ |
| | 24 | + var $title; |
| | 25 | + |
| 20 | 26 | var $command, |
| 21 | | - $title, |
| 22 | 27 | $params, |
| 23 | 28 | $id, |
| 24 | 29 | $removeDuplicates, |
| Index: trunk/phase3/includes/job/DoubleRedirectJob.php |
| — | — | @@ -13,6 +13,10 @@ |
| 14 | 14 | */ |
| 15 | 15 | class DoubleRedirectJob extends Job { |
| 16 | 16 | var $reason, $redirTitle, $destTitleText; |
| | 17 | + |
| | 18 | + /** |
| | 19 | + * @var User |
| | 20 | + */ |
| 17 | 21 | static $user; |
| 18 | 22 | |
| 19 | 23 | /** |
| — | — | @@ -53,6 +57,7 @@ |
| 54 | 58 | } |
| 55 | 59 | Job::batchInsert( $jobs ); |
| 56 | 60 | } |
| | 61 | + |
| 57 | 62 | function __construct( $title, $params = false, $id = 0 ) { |
| 58 | 63 | parent::__construct( 'fixDoubleRedirect', $title, $params, $id ); |
| 59 | 64 | $this->reason = $params['reason']; |