r40865 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r40864‎ | r40865 | r40866 >
Date:18:50, 15 September 2008
Author:aaron
Status:old
Tags:
Comment:
Revert r40808. No need to make log_params and output uglier due to edit comment limitations.
Modified paths:
  • /trunk/phase3/includes/LogPage.php (modified) (history)
  • /trunk/phase3/includes/Title.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/Title.php
@@ -1403,8 +1403,7 @@
14041404 $log = new LogPage( 'protect' );
14051405
14061406 if( $create_perm ) {
1407 - $params = array("[create=$create_perm] $expiry_description",'');
1408 - $log->addEntry( $this->mRestrictions['create'] ? 'modify' : 'protect', $this, trim( $reason ), $params );
 1407+ $log->addEntry( $this->mRestrictions['create'] ? 'modify' : 'protect', $this, trim( $reason . " [create=$create_perm] $expiry_description" ) );
14091408 } else {
14101409 $log->addEntry( 'unprotect', $this, $reason );
14111410 }
Index: trunk/phase3/includes/LogPage.php
@@ -204,15 +204,10 @@
205205 $params[2] = isset( $params[2] ) ?
206206 self::formatBlockFlags( $params[2], is_null( $skin ) ) : '';
207207 } else if ( $type == 'protect' && count($params) == 3 ) {
208 - $details .= "{$params[1]}"; // restrictions and expiries
 208+ $details .= " {$params[1]}"; // restrictions and expiries
209209 if( $params[2] ) {
210210 $details .= ' ['.wfMsg('protect-summary-cascade').']';
211211 }
212 - # If this is going to be displayed, make it somewhat consistent with page history
213 - # and older log entries where everything is treated as a comment
214 - if (!is_null($skin)) {
215 - $details = $skin->commentBlock($details);
216 - }
217212 }
218213 $rv = wfMsgReal( $wgLogActions[$key], $params, true, !$skin ) . $details;
219214 }

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r40808* Update title protection to be more consistent with regular protection...mrzman03:02, 14 September 2008