| Index: trunk/phase3/includes/ChangesList.php |
| — | — | @@ -339,7 +339,6 @@ |
| 340 | 340 | wfProfileIn( __METHOD__ ); |
| 341 | 341 | |
| 342 | 342 | # Extract DB fields into local scope |
| 343 | | - // FIXME: Would be good to replace this extract() call with something that explicitly initializes local variables. |
| 344 | 343 | extract( $rc->mAttribs ); |
| 345 | 344 | |
| 346 | 345 | # Should patrol-related stuff be shown? |
| — | — | @@ -433,7 +432,6 @@ |
| 434 | 433 | $rc = RCCacheEntry::newFromParent( $baseRC ); |
| 435 | 434 | |
| 436 | 435 | # Extract fields from DB into the function scope (rc_xxxx variables) |
| 437 | | - // FIXME: Would be good to replace this extract() call with something that explicitly initializes local variables. |
| 438 | 436 | extract( $rc->mAttribs ); |
| 439 | 437 | $curIdEq = 'curid=' . $rc_cur_id; |
| 440 | 438 | |
| — | — | @@ -722,7 +720,6 @@ |
| 723 | 721 | $r .= '<table cellpadding="0" cellspacing="0" border="0" style="background: none">'; |
| 724 | 722 | foreach( $block as $rcObj ) { |
| 725 | 723 | # Get rc_xxxx variables |
| 726 | | - // FIXME: Would be good to replace this extract() call with something that explicitly initializes local variables. |
| 727 | 724 | extract( $rcObj->mAttribs ); |
| 728 | 725 | |
| 729 | 726 | #$r .= '<tr><td valign="top">'.$this->spacerArrow(); |
| — | — | @@ -842,7 +839,6 @@ |
| 843 | 840 | protected function recentChangesBlockLine( $rcObj ) { |
| 844 | 841 | global $wgContLang, $wgRCShowChangedSize; |
| 845 | 842 | # Get rc_xxxx variables |
| 846 | | - // FIXME: Would be good to replace this extract() call with something that explicitly initializes local variables. |
| 847 | 843 | extract( $rcObj->mAttribs ); |
| 848 | 844 | $curIdEq = "curid={$rc_cur_id}"; |
| 849 | 845 | |