| Index: trunk/phase3/includes/LogEventsList.php |
| — | — | @@ -27,10 +27,27 @@ |
| 28 | 28 | const NO_ACTION_LINK = 1; |
| 29 | 29 | const NO_EXTRA_USER_LINKS = 2; |
| 30 | 30 | |
| | 31 | + /** |
| | 32 | + * @var Skin |
| | 33 | + */ |
| 31 | 34 | private $skin; |
| | 35 | + |
| | 36 | + /** |
| | 37 | + * @var OutputPage |
| | 38 | + */ |
| 32 | 39 | private $out; |
| 33 | 40 | public $flags; |
| 34 | 41 | |
| | 42 | + /** |
| | 43 | + * @var Array |
| | 44 | + */ |
| | 45 | + protected $message; |
| | 46 | + |
| | 47 | + /** |
| | 48 | + * @var Array |
| | 49 | + */ |
| | 50 | + protected $mDefaultQuery; |
| | 51 | + |
| 35 | 52 | public function __construct( $skin, $out, $flags = 0 ) { |
| 36 | 53 | $this->skin = $skin; |
| 37 | 54 | $this->out = $out; |
| — | — | @@ -82,8 +99,7 @@ |
| 83 | 100 | * @param $tagFilter: array? |
| 84 | 101 | */ |
| 85 | 102 | public function showOptions( $types=array(), $user='', $page='', $pattern='', $year='', |
| 86 | | - $month = '', $filter = null, $tagFilter='' ) |
| 87 | | - { |
| | 103 | + $month = '', $filter = null, $tagFilter='' ) { |
| 88 | 104 | global $wgScript, $wgMiserMode; |
| 89 | 105 | |
| 90 | 106 | $action = $wgScript; |
| — | — | @@ -357,7 +373,15 @@ |
| 358 | 374 | return $comment; |
| 359 | 375 | } |
| 360 | 376 | |
| 361 | | - // @TODO: split up! |
| | 377 | + /** |
| | 378 | + * @TODO: split up! |
| | 379 | + * |
| | 380 | + * @param $row |
| | 381 | + * @param Title $title |
| | 382 | + * @param Array $paramArray |
| | 383 | + * @param $comment |
| | 384 | + * @return String |
| | 385 | + */ |
| 362 | 386 | private function logActionLinks( $row, $title, $paramArray, &$comment ) { |
| 363 | 387 | global $wgUser; |
| 364 | 388 | if( ( $this->flags & self::NO_ACTION_LINK ) // we don't want to see the action |
| — | — | @@ -506,8 +530,7 @@ |
| 507 | 531 | private function getShowHideLinks( $row ) { |
| 508 | 532 | global $wgUser; |
| 509 | 533 | if( ( $this->flags & self::NO_ACTION_LINK ) // we don't want to see the links |
| 510 | | - || $row->log_type == 'suppress' ) // no one can hide items from the suppress log |
| 511 | | - { |
| | 534 | + || $row->log_type == 'suppress' ) { // no one can hide items from the suppress log |
| 512 | 535 | return ''; |
| 513 | 536 | } |
| 514 | 537 | $del = ''; |
| — | — | @@ -758,8 +781,7 @@ |
| 759 | 782 | * @param $tagFilter String: tag |
| 760 | 783 | */ |
| 761 | 784 | public function __construct( $list, $types = array(), $user = '', $title = '', $pattern = '', |
| 762 | | - $conds = array(), $year = false, $month = false, $tagFilter = '' ) |
| 763 | | - { |
| | 785 | + $conds = array(), $year = false, $month = false, $tagFilter = '' ) { |
| 764 | 786 | parent::__construct(); |
| 765 | 787 | $this->mConds = $conds; |
| 766 | 788 | |
| — | — | @@ -878,8 +900,9 @@ |
| 879 | 901 | global $wgMiserMode, $wgUser; |
| 880 | 902 | |
| 881 | 903 | $title = Title::newFromText( $page ); |
| 882 | | - if( strlen( $page ) == 0 || !$title instanceof Title ) |
| | 904 | + if( strlen( $page ) == 0 || !$title instanceof Title ) { |
| 883 | 905 | return false; |
| | 906 | + } |
| 884 | 907 | |
| 885 | 908 | $this->title = $title->getPrefixedText(); |
| 886 | 909 | $ns = $title->getNamespace(); |