bug 18472 Suppression log filtered by "offender", as Oversight log can be
[lhc/web/wiklou.git] / includes / LogEventsList.php
index 93085d4..7c209f9 100644 (file)
@@ -77,27 +77,53 @@ class LogEventsList {
                $month = '', $filter = null, $tagFilter='' )
        {
                global $wgScript, $wgMiserMode;
-               $action = htmlspecialchars( $wgScript );
+               
+               $action = $wgScript;
                $title = SpecialPage::getTitleFor( 'Log' );
-               $special = htmlspecialchars( $title->getPrefixedDBkey() );
+               $special = $title->getPrefixedDBkey();
+               
                // For B/C, we take strings, but make sure they are converted...
                $types = ($types === '') ? array() : (array)$types;
 
                $tagSelector = ChangeTags::buildTagFilterSelector( $tagFilter );
-
-               $this->out->addHTML( "<form action=\"$action\" method=\"get\"><fieldset>" .
-                       Xml::element( 'legend', array(), wfMsg( 'log' ) ) .
-                       Xml::hidden( 'title', $special ) . "\n" .
-                       $this->getTypeMenu( $types ) . "\n" .
-                       $this->getUserInput( $user ) . "\n" .
-                       $this->getTitleInput( $page ) . "\n" .
-                       ( !$wgMiserMode ? ($this->getTitlePattern( $pattern )."\n") : "" ) .
-                       "<p>" . Xml::dateMenu( $year, $month ) . "\n" .
-                       ( $tagSelector ? Xml::tags( 'p', null, implode( '&nbsp;', $tagSelector ) ) :'' ). "\n" .
-                       ( $filter ? "</p><p>".$this->getFilterLinks( $filter )."\n" : "" ) . "\n" .
-                       Xml::submitButton( wfMsg( 'allpagessubmit' ) ) . "</p>\n" .
-                       "</fieldset></form>"
-               );
+               
+               $html = '';
+               $html .= Xml::hidden( 'title', $special );
+               
+               // Basic selectors
+               $html .= $this->getTypeMenu( $types ) . "\n";
+               $html .= $this->getUserInput( $user ) . "\n";
+               $html .= $this->getTitleInput( $page ) . "\n";
+               $html .= $this->getExtraInputs( $types ) . "\n";
+
+               // Title pattern, if allowed
+               if (!$wgMiserMode) {
+                       $html .= $this->getTitlePattern( $pattern ) . "\n";
+               }
+               
+               // date menu
+               $html .= Xml::tags( 'p', null, Xml::dateMenu( $year, $month ) );
+               
+               // Tag filter
+               if ($tagSelector) {
+                       $html .= Xml::tags( 'p', null, implode( '&nbsp;', $tagSelector ) );
+               }
+               
+               // Filter links
+               if ($filter) {
+                       $html .= Xml::tags( 'p', null, $this->getFilterLinks( $filter ) );
+               }
+               
+               // Submit button
+               $html .= Xml::submitButton( wfMsg( 'allpagessubmit' ) );
+               
+               // Fieldset
+               $html = Xml::fieldset( wfMsg( 'log' ), $html );
+               
+               // Form wrapping
+               $html = Xml::tags( 'form', array( 'action' => $action, 'method' => 'get' ), $html );
+
+               $this->out->addHTML( $html );
        }
        
        /**
@@ -213,6 +239,15 @@ class LogEventsList {
                        Xml::checkLabel( wfMsg( 'log-title-wildcard' ), 'pattern', 'pattern', $pattern ) .
                        '</span>';
        }
+       
+       private function getExtraInputs( $types ) {
+               global $wgRequest;
+               if( count($types) == 1 && $types[0] == 'suppress' ) {
+                       return Xml::inputLabel( wfMsg('revdelete-offender'), 'offender',
+                               'mw-log-offender', 20, $wgRequest->getVal('offender') );
+               }
+               return '';
+       }
 
        public function beginLogEventsList() {
                return "<ul>\n";
@@ -531,29 +566,62 @@ class LogEventsList {
        }
 
        /**
-        * Quick function to show a short log extract
-        * @param $out OutputPage
+        * Show log extract. Either with text and a box (set $msgKey) or without (don't set $msgKey)
+        * @param $out OutputPage or String-by-reference
         * @param $types String or Array
         * @param $page String
         * @param $user String
-        * @param $lim Integer
+        * @param $lim Integer Limit of items to show, default is 50
         * @param $conds Array
+        * @param $showIfEmpty boolean Set to false if you don't want any output in case the loglist is empty
+        *              if set to true (default), "No matching items in log" is displayed if loglist is empty
+        * @param $msgKey String if you want a nice box with a message, set this to the key of the message
+        * @return Integer Number of total log items (not limited by $lim)
         */
-       public static function showLogExtract( $out, $types=array(), $page='', $user='', $lim=0, $conds=array() ) {
-               global $wgUser;
+       public static function showLogExtract( &$out, $types=array(), $page='', $user='', $lim=0, $conds=array(), $showIfEmpty = true, $msgKey = '' ) {
+               global $wgUser, $wgOut;
                # Insert list of top 50 or so items
-               $loglist = new LogEventsList( $wgUser->getSkin(), $out, 0 );
+               $loglist = new LogEventsList( $wgUser->getSkin(), $wgOut, 0 );
                $pager = new LogPager( $loglist, $types, $user, $page, '', $conds );
                if( $lim > 0 ) $pager->mLimit = $lim;
                $logBody = $pager->getBody();
+               $s = '';
                if( $logBody ) {
-                       $out->addHTML(
-                               $loglist->beginLogEventsList() .
-                               $logBody .
-                               $loglist->endLogEventsList()
+                       if ( $msgKey )
+                               $s = '<div class="mw-warning-with-logexcerpt">' . wfMsgExt( $msgKey, array('parse') ) ;
+                       $s .= $loglist->beginLogEventsList() .
+                                $logBody .
+                                $loglist->endLogEventsList();
+               } else {
+                       if ( $showIfEmpty )
+                               $s = wfMsgExt( 'logempty', array('parse') );
+               }
+               if( $pager->getNumRows() > $pager->mLimit ) { # Show "Full log" link
+                       $urlParam = array();
+                       if ( $page != '')
+                               $urlParam['page'] = $page;
+                       if ( $user != '')
+                               $urlParam['user'] = $user;
+                       if ( !is_array( $types ) ) # Make it an array, if it isn't
+                               $types = array( $types );
+                       # If there is exactly one log type, we can link to Special:Log?type=foo
+                       if ( count( $types ) == 1 )
+                               $urlParam['type'] = $types[0];
+                       $s .= $wgUser->getSkin()->link(
+                               SpecialPage::getTitleFor( 'Log' ),
+                               wfMsgHtml( 'log-fulllog' ),
+                               array(),
+                               $urlParam
                        );
+
+               }
+               if ( $logBody && $msgKey )
+                       $s .= '</div>';
+
+               if( $out instanceof OutputPage ){
+                       $out->addHTML( $s );
                } else {
-                       $out->addWikiMsg( 'logempty' );
+                       $out = $s;
                }
                return $pager->getNumRows();
        }
@@ -699,8 +767,11 @@ class LogPager extends ReverseChronologicalPager {
                        global $wgUser;
                        $this->mConds['log_user'] = $userid;
                        // Paranoia: avoid brute force searches (bug 17342)
-                       if( !$wgUser->isAllowed( 'suppressrevision' ) ) {
+                       if( !$wgUser->isAllowed( 'deleterevision' ) ) {
                                $this->mConds[] = $this->mDb->bitAnd('log_deleted', LogPage::DELETED_USER) . ' = 0';
+                       } else if( !$wgUser->isAllowed( 'suppressrevision' ) ) {
+                               $this->mConds[] = $this->mDb->bitAnd('log_deleted', LogPage::SUPPRESSED_USER) .
+                                       ' != ' . LogPage::SUPPRESSED_USER;
                        }
                        $this->user = $usertitle->getText();
                }
@@ -743,8 +814,11 @@ class LogPager extends ReverseChronologicalPager {
                        $this->mConds['log_title'] = $title->getDBkey();
                }
                // Paranoia: avoid brute force searches (bug 17342)
-               if( !$wgUser->isAllowed( 'suppressrevision' ) ) {
+               if( !$wgUser->isAllowed( 'deleterevision' ) ) {
                        $this->mConds[] = $this->mDb->bitAnd('log_deleted', LogPage::DELETED_ACTION) . ' = 0';
+               } else if( !$wgUser->isAllowed( 'suppressrevision' ) ) {
+                       $this->mConds[] = $this->mDb->bitAnd('log_deleted', LogPage::SUPPRESSED_ACTION) .
+                               ' != ' . LogPage::SUPPRESSED_ACTION;
                }
        }