From 88ce2bf36c083dbcf2984b5b565d4d9ef9360cc7 Mon Sep 17 00:00:00 2001 From: Tim Starling Date: Tue, 4 Nov 2008 00:33:09 +0000 Subject: [PATCH] Fixes for r42642: * Fix excessively abbreviated variable names $y and $m * Don't use empty() unless you want to suppress warnings * More documentation for $wgFilterLogTypes * hide_patrol_log instead of hidepatrollog, camelCase for variable and function names, hyphens for message names * More spaces, break long lines --- includes/DefaultSettings.php | 21 +++++++- includes/LogEventsList.php | 66 +++++++++++++----------- languages/messages/MessagesAr.php | 2 +- languages/messages/MessagesArz.php | 2 +- languages/messages/MessagesAst.php | 2 +- languages/messages/MessagesBcc.php | 2 +- languages/messages/MessagesBe_tarask.php | 2 +- languages/messages/MessagesCs.php | 2 +- languages/messages/MessagesDe.php | 2 +- languages/messages/MessagesDsb.php | 2 +- languages/messages/MessagesEn.php | 2 +- languages/messages/MessagesEo.php | 2 +- languages/messages/MessagesEs.php | 2 +- languages/messages/MessagesFa.php | 2 +- languages/messages/MessagesFi.php | 2 +- languages/messages/MessagesFr.php | 2 +- languages/messages/MessagesGl.php | 2 +- languages/messages/MessagesHe.php | 2 +- languages/messages/MessagesHr.php | 2 +- languages/messages/MessagesHsb.php | 2 +- languages/messages/MessagesHu.php | 2 +- languages/messages/MessagesIa.php | 2 +- languages/messages/MessagesId.php | 2 +- languages/messages/MessagesIt.php | 2 +- languages/messages/MessagesJa.php | 2 +- languages/messages/MessagesKsh.php | 2 +- languages/messages/MessagesMs.php | 2 +- languages/messages/MessagesNl.php | 2 +- languages/messages/MessagesNo.php | 2 +- languages/messages/MessagesOc.php | 2 +- languages/messages/MessagesPl.php | 2 +- languages/messages/MessagesPt.php | 2 +- languages/messages/MessagesQqq.php | 2 +- languages/messages/MessagesQu.php | 2 +- languages/messages/MessagesRu.php | 2 +- languages/messages/MessagesSi.php | 2 +- languages/messages/MessagesSk.php | 2 +- languages/messages/MessagesSv.php | 2 +- languages/messages/MessagesUk.php | 2 +- languages/messages/MessagesVec.php | 2 +- languages/messages/MessagesVi.php | 2 +- 41 files changed, 93 insertions(+), 72 deletions(-) diff --git a/includes/DefaultSettings.php b/includes/DefaultSettings.php index e7263265cc..a86c40c680 100644 --- a/includes/DefaultSettings.php +++ b/includes/DefaultSettings.php @@ -2755,8 +2755,25 @@ $wgLogRestrictions = array( ); /** - * Show/hide links on Special:Log will be shown for these log types - * This is associative array of log => (hidden by default) + * Show/hide links on Special:Log will be shown for these log types. Intended + * for extensions only. + * + * This is associative array of log type => boolean "hide by default" + * + * See $wgLogTypes for a list of available log types. + * + * For example: + * $wgFilterLogTypes => array( + * 'move' => true, + * 'import' => false, + * ); + * + * Will display show/hide links for the move and import logs. Move logs will be + * hidden by default unless the link is clicked. Import logs will be shown by + * default, and hidden when the link is clicked. + * + * A message of the form log-show-hide- should be added, and will be used + * for the link text. */ $wgFilterLogTypes = array( 'patrol' => true diff --git a/includes/LogEventsList.php b/includes/LogEventsList.php index 13f80df0f7..0de92b6b98 100644 --- a/includes/LogEventsList.php +++ b/includes/LogEventsList.php @@ -39,7 +39,7 @@ class LogEventsList { // Precache various messages if( !isset( $this->message ) ) { $messages = 'revertmerge protect_change unblocklink revertmove undeletelink revdel-restore rev-delundel hist'; - foreach( explode(' ', $messages ) as $msg ) { + foreach( explode( ' ', $messages ) as $msg ) { $this->message[$msg] = wfMsgExt( $msg, array( 'escape') ); } } @@ -62,11 +62,13 @@ class LogEventsList { * @param string $user, * @param string $page, * @param string $pattern - * @param int $y year - * @param int $y month + * @param int $year year + * @param int $month month * @param bool $filter */ - public function showOptions( $type='', $user='', $page='', $pattern='', $y='', $m='', $filter=null ) { + public function showOptions( $type = '', $user = '', $page = '', $pattern = '', $year = '', + $month = '', $filter = null ) + { global $wgScript, $wgMiserMode; $action = htmlspecialchars( $wgScript ); $title = SpecialPage::getTitleFor( 'Log' ); @@ -79,25 +81,25 @@ class LogEventsList { $this->getUserInput( $user ) . "\n" . $this->getTitleInput( $page ) . "\n" . ( !$wgMiserMode ? ($this->getTitlePattern( $pattern )."\n") : "" ) . - "

" . $this->getDateMenu( $y, $m ) . "\n" . - ( !empty($filter) ? "

".$this->getFilterLinks( $type, $filter )."\n" : "" ) . + "

" . $this->getDateMenu( $year, $month ) . "\n" . + ( $filter ? "

".$this->getFilterLinks( $type, $filter )."\n" : "" ) . Xml::submitButton( wfMsg( 'allpagessubmit' ) ) . "

\n" . "" ); } - private function getFilterLinks( $logtype, $filter ) { + private function getFilterLinks( $logType, $filter ) { global $wgTitle; // show/hide links - $showhide = array( wfMsgHtml( 'show' ), wfMsgHtml( 'hide' ) ); + $messages = array( wfMsgHtml( 'show' ), wfMsgHtml( 'hide' ) ); // Option value -> message mapping $links = array(); foreach( $filter as $type => $val ) { - $onoff = 1 - intval($val); - $link = $this->skin->makeKnownLinkObj( $wgTitle, $showhide[$onoff], - wfArrayToCGI( array( "hide{$type}log" => $onoff ), $this->getDefaultQuery() ) + $hideVal = 1 - intval($val); + $link = $this->skin->makeKnownLinkObj( $wgTitle, $messages[$hideVal], + wfArrayToCGI( array( "hide_{$type}_log" => $hideVal ), $this->getDefaultQuery() ) ); - $links[$type] = wfMsgHtml( "logshowhide-{$type}", $link ); + $links[$type] = wfMsgHtml( "log-show-hide-{$type}", $link ); } // Build links return implode( ' | ', $links ); @@ -127,19 +129,19 @@ class LogEventsList { $html = "