Index: trunk/phase3/includes/revisiondelete/RevisionDelete.php |
— | — | @@ -292,6 +292,10 @@ |
293 | 293 | var $authorNameField = 'oi_user_text'; |
294 | 294 | var $storeBatch, $deleteBatch, $cleanupBatch; |
295 | 295 | |
| 296 | + /** |
| 297 | + * @param $db DatabaseBase |
| 298 | + * @return mixed |
| 299 | + */ |
296 | 300 | public function doQuery( $db ) { |
297 | 301 | $archiveNames = array(); |
298 | 302 | foreach( $this->ids as $timestamp ) { |
— | — | @@ -515,7 +519,11 @@ |
516 | 520 | var $dateField = 'fa_timestamp'; |
517 | 521 | var $authorIdField = 'fa_user'; |
518 | 522 | var $authorNameField = 'fa_user_text'; |
519 | | - |
| 523 | + |
| 524 | + /** |
| 525 | + * @param $db DatabaseBase |
| 526 | + * @return mixed |
| 527 | + */ |
520 | 528 | public function doQuery( $db ) { |
521 | 529 | $ids = array_map( 'intval', $this->ids ); |
522 | 530 | return $db->select( 'filearchive', '*', |
— | — | @@ -593,6 +601,10 @@ |
594 | 602 | var $authorIdField = 'log_user'; |
595 | 603 | var $authorNameField = 'log_user_text'; |
596 | 604 | |
| 605 | + /** |
| 606 | + * @param $db DatabaseBase |
| 607 | + * @return mixed |
| 608 | + */ |
597 | 609 | public function doQuery( $db ) { |
598 | 610 | $ids = array_map( 'intval', $this->ids ); |
599 | 611 | return $db->select( 'logging', '*', |
Index: trunk/phase3/includes/revisiondelete/RevisionDeleter.php |
— | — | @@ -125,10 +125,17 @@ |
126 | 126 | return null; |
127 | 127 | } |
128 | 128 | } |
129 | | - |
130 | | - // Checks if a revision still exists in the revision table. |
131 | | - // If it doesn't, returns the corresponding ar_timestamp field |
132 | | - // so that this key can be used instead. |
| 129 | + |
| 130 | + /** |
| 131 | + * Checks if a revision still exists in the revision table. |
| 132 | + * If it doesn't, returns the corresponding ar_timestamp field |
| 133 | + * so that this key can be used instead. |
| 134 | + * |
| 135 | + * @static |
| 136 | + * @param $title Title |
| 137 | + * @param $revid |
| 138 | + * @return bool|mixed |
| 139 | + */ |
133 | 140 | public static function checkRevisionExistence( $title, $revid ) { |
134 | 141 | $dbr = wfGetDB( DB_SLAVE ); |
135 | 142 | $exists = $dbr->selectField( 'revision', '1', |
— | — | @@ -145,8 +152,17 @@ |
146 | 153 | |
147 | 154 | return $timestamp; |
148 | 155 | } |
149 | | - |
150 | | - // Creates utility links for log entries. |
| 156 | + |
| 157 | + /** |
| 158 | + * Creates utility links for log entries. |
| 159 | + * |
| 160 | + * @static |
| 161 | + * @param $title Title |
| 162 | + * @param $paramArray Array |
| 163 | + * @param $skin Skin |
| 164 | + * @param $messages |
| 165 | + * @return String |
| 166 | + */ |
151 | 167 | public static function getLogLinks( $title, $paramArray, $skin, $messages ) { |
152 | 168 | global $wgLang; |
153 | 169 | |
— | — | @@ -215,7 +231,7 @@ |
216 | 232 | } |
217 | 233 | |
218 | 234 | // View/modify link... |
219 | | - if ( count($undeletedRevisions) ) { |
| 235 | + if ( count( $undeletedRevisions ) ) { |
220 | 236 | // FIXME THIS IS A HORRIBLE HORRIBLE HACK AND SHOULD DIE |
221 | 237 | // It's not possible to pass a list of both deleted and |
222 | 238 | // undeleted revisions to SpecialRevisionDelete, so we're |
— | — | @@ -229,7 +245,7 @@ |
230 | 246 | array( |
231 | 247 | 'target' => $title->getPrefixedText(), |
232 | 248 | 'type' => $originalKey, |
233 | | - 'ids' => implode(',', $undeletedRevisions), |
| 249 | + 'ids' => implode( ',', $undeletedRevisions ), |
234 | 250 | ), |
235 | 251 | array( 'known', 'noclasses' ) |
236 | 252 | ); |