| Index: trunk/phase3/maintenance/namespaceDupes.php |
| — | — | @@ -62,8 +62,8 @@ |
| 63 | 63 | |
| 64 | 64 | /** |
| 65 | 65 | * @todo Document |
| 66 | | - * @param $fix bool Whether or not to fix broken entries |
| 67 | | - * @param $suffix String Suffix to append to renamed articles |
| | 66 | + * @param $fix Boolean: whether or not to fix broken entries |
| | 67 | + * @param $suffix String: suffix to append to renamed articles |
| 68 | 68 | */ |
| 69 | 69 | private function checkAll( $fix, $suffix = '' ) { |
| 70 | 70 | global $wgContLang, $wgNamespaceAliases, $wgCanonicalNamespaceNames; |
| — | — | @@ -132,8 +132,9 @@ |
| 133 | 133 | |
| 134 | 134 | /** |
| 135 | 135 | * Get the interwiki list |
| | 136 | + * |
| 136 | 137 | * @todo Needs to respect interwiki cache! |
| 137 | | - * @return array |
| | 138 | + * @return Array |
| 138 | 139 | */ |
| 139 | 140 | private function getInterwikiList() { |
| 140 | 141 | $result = $this->db->select( 'interwiki', array( 'iw_prefix' ) ); |
| — | — | @@ -147,10 +148,10 @@ |
| 148 | 149 | |
| 149 | 150 | /** |
| 150 | 151 | * @todo Document |
| 151 | | - * @param $ns int A namespace id |
| | 152 | + * @param $ns Integer: a namespace id |
| 152 | 153 | * @param $name String |
| 153 | | - * @param $fix bool Whether to fix broken entries |
| 154 | | - * @param $suffix String Suffix to append to renamed articles |
| | 154 | + * @param $fix Boolean: whether to fix broken entries |
| | 155 | + * @param $suffix String: suffix to append to renamed articles |
| 155 | 156 | */ |
| 156 | 157 | private function checkNamespace( $ns, $name, $fix, $suffix = '' ) { |
| 157 | 158 | $conflicts = $this->getConflicts( $ns, $name ); |
| — | — | @@ -181,8 +182,9 @@ |
| 182 | 183 | /** |
| 183 | 184 | * Find pages in mainspace that have a prefix of the new namespace |
| 184 | 185 | * so we know titles that will need migrating |
| 185 | | - * @param $ns int Namespace id (id for new namespace?) |
| 186 | | - * @param $name String Prefix that is being made a namespace |
| | 186 | + * |
| | 187 | + * @param $ns Integer: namespace id (id for new namespace?) |
| | 188 | + * @param $name String: prefix that is being made a namespace |
| 187 | 189 | */ |
| 188 | 190 | private function getConflicts( $ns, $name ) { |
| 189 | 191 | $page = 'page'; |
| — | — | @@ -252,9 +254,10 @@ |
| 253 | 255 | |
| 254 | 256 | /** |
| 255 | 257 | * Resolve any conflicts |
| 256 | | - * @param $row Row from the page table to fix |
| 257 | | - * @param $resolveable bool |
| 258 | | - * @param $suffix String Suffix to append to the fixed page |
| | 258 | + * |
| | 259 | + * @param $row Object: row from the page table to fix |
| | 260 | + * @param $resolvable Boolean |
| | 261 | + * @param $suffix String: suffix to append to the fixed page |
| 259 | 262 | */ |
| 260 | 263 | private function resolveConflict( $row, $resolvable, $suffix ) { |
| 261 | 264 | if ( !$resolvable ) { |
| — | — | @@ -281,9 +284,10 @@ |
| 282 | 285 | |
| 283 | 286 | /** |
| 284 | 287 | * Resolve a given conflict |
| 285 | | - * @param $row Row from the old broken entry |
| 286 | | - * @param $table String Table to update |
| 287 | | - * @param $prefix String Prefix for column name, like page or ar |
| | 288 | + * |
| | 289 | + * @param $row Object: row from the old broken entry |
| | 290 | + * @param $table String: table to update |
| | 291 | + * @param $prefix String: prefix for column name, like page or ar |
| 288 | 292 | */ |
| 289 | 293 | private function resolveConflictOn( $row, $table, $prefix ) { |
| 290 | 294 | $this->output( "... resolving on $table... " ); |