Page MenuHomePhabricator

wrap stand alone message 'logempty'
Closed, ResolvedPublic

Description

When there is a semiprotected page, and MediaWiki found no log entry for the page the message 'logempty' is used. But the message stand there without any informationen. See url for example.

Please wrap 'logempty' always in a warning message, in this case it that message 'semiprotectedpagewarning', because 'logempty' as a stand alone message is confused. Thanks.


Version: unspecified
Severity: enhancement
URL: http://de.wikipedia.org/w/index.php?title=Hilfe:Benutzerkonto_anlegen&action=edit&uselang=en

Details

Reference
bz21559

Event Timeline

bzimport raised the priority of this task from to Low.Nov 21 2014, 10:47 PM
bzimport set Reference to bz21559.
bzimport added a subscriber: Unknown Object (MLST).

Created attachment 6807
page from url in edit mode (skin vector)

pic to show, which text I mean.

Attached:

logempty.PNG (269×1 px, 25 KB)

That was not my request.

The message 'logempty' is stand alone without any other message and so the user does not know for which log there were no matching items found. That is very useless and confuse.

I came across this bug at http://nl.wikipedia.org/wiki/Sjabloon:Infobox/breedte.

This page was protected before the protection log existed.

The relevant code from http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/includes/EditPage.php?view=markup

1438 if ( $this->mTitle->getNamespace() != NS_MEDIAWIKI && $this->mTitle->isProtected( 'edit' ) ) {
1439 # Is the title semi-protected?
1440 if ( $this->mTitle->isSemiProtected() ) {
1441 $noticeMsg = 'semiprotectedpagewarning';
1442 } else {
1443 # Then it must be protected based on static groups (regular)
1444 $noticeMsg = 'protectedpagewarning';
1445 }
1446 LogEventsList::showLogExtract( $wgOut, 'protect', $this->mTitle->getPrefixedText(), '',
1447 array( 'lim' => 1, 'msgKey' => array( $noticeMsg ) ) );

Here LogEventsList::showLogExtract is used to produce the message. This code is in http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/includes/LogEventsList.php?&view=markup

LogEventsList::showLogExtract will return the message 'Logempty' ("No matching items in log.") when no matching items are found in the log.

'showIfEmpty' should probably set to False and a test should be added to see if the number of returned items is 1 and if not, $noticeMsg should be added to the output.

  • This bug has been marked as a duplicate of bug 24511 ***