Page MenuHomePhabricator
Create Task
Maniphest T16638

Block log below block form limited to 50 entries
Closed, ResolvedPublic
Actions

Description

As surprising as it is, some users on en.wiki have more than 50 block log entries, causing a confusing discrepancy between Special:Log and the log that appears below the block form. It would seem that the block log that appears below the block form is limited to 50 entries.

I would recommend not limiting the log below the block form. Or, alternately, it could limit it to 10 entries or something and then provide a link to the full block log if there are more entries.

Version: unspecified
Severity: enhancement

Details

Reference
bz14638

Event Timeline

• bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:09 PM
• bzimport added a project: MediaWiki-User-management.
• bzimport set Reference to bz14638.
• MZMcBride created this task.Jun 25 2008, 8:53 AM
Nakon added a comment.Jul 4 2008, 5:58 AM

Checks to see if there are more than 50 rows and displays a message

attachment log.diff ignored as obsolete

• bzimport added a comment.Jul 4 2008, 8:42 AM

Bryan.TongMinh wrote:

The message should point to the page where the other log entries can be viewed.

• brooke added a comment.Jul 6 2008, 7:36 PM

Displaying up to 10 with a link to the full log page for more would probably be great, yes.

siebrand added a comment.Aug 13 2008, 5:58 PM

+need-review

• bzimport added a comment.Sep 12 2008, 9:44 AM

mattj wrote:

Proposed patch

I've done a new patch based on Brion's post, displays 10 then if its more generates a link.

attachment blocklogfragmentlimit.patch ignored as obsolete

• bzimport added a comment.Sep 15 2008, 1:26 AM

soxred93 wrote:

Done in r40838

• brooke added a comment.Sep 15 2008, 6:34 PM

Reverted in r40864; construction of the link looks very bizarre. We have regular linking functions for this sort of thing...

• bzimport added a comment.Sep 16 2008, 11:40 PM

mattj wrote:

Proposed patch v2

This uses the proper Linker function link(), as well as updating the message name (since the old one was incorrectly named, since it was at the bottom.

Attached:

blocklogfragmentlimit.patch1 KBDownload

• bzimport added a comment.Sep 17 2008, 11:48 PM

mattj wrote:

Patch committed in r40944 by Soxred93

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits