Last modified: 2008-09-17 23:48:17 UTC

Wikimedia Bugzilla is closed!

Wikimedia has migrated from Bugzilla to Phabricator. Bug reports should be created and updated in Wikimedia Phabricator instead. Please create an account in Phabricator and add your Bugzilla email address to it.
Wikimedia Bugzilla is read-only. If you try to edit or create any bug report in Bugzilla you will be shown an intentional error message.
In order to access the Phabricator task corresponding to a Bugzilla report, just remove "static-" from its URL.
You could still run searches in Bugzilla or access your list of votes but bug reports will obviously not be up-to-date in Bugzilla.
Bug 14638 - Block log below block form limited to 50 entries
Block log below block form limited to 50 entries
Product: MediaWiki
Classification: Unclassified
User blocking (Other open bugs)
All All
: Normal enhancement (vote)
: ---
Assigned To: Matt Johnston
: patch, patch-need-review
Depends on:
  Show dependency treegraph
Reported: 2008-06-25 08:53 UTC by MZMcBride
Modified: 2008-09-17 23:48 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---

Checks to see if there are more than 50 rows and displays a message (1.13 KB, patch)
2008-07-04 05:58 UTC, Nakon
Proposed patch (1.83 KB, patch)
2008-09-12 09:44 UTC, Matt Johnston
Proposed patch v2 (1.83 KB, patch)
2008-09-16 23:40 UTC, Matt Johnston

Description MZMcBride 2008-06-25 08:53:19 UTC
As surprising as it is, some users on 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.
Comment 1 Nakon 2008-07-04 05:58:46 UTC
Created attachment 5044 [details]
Checks to see if there are more than 50 rows and displays a message
Comment 2 Bryan Tong Minh 2008-07-04 08:42:03 UTC
The message should point to the page where the other log entries can be viewed.
Comment 3 Brion Vibber 2008-07-06 19:36:38 UTC
Displaying up to 10 with a link to the full log page for more would probably be great, yes.
Comment 4 Siebrand Mazeland 2008-08-13 17:58:24 UTC
Comment 5 Matt Johnston 2008-09-12 09:44:03 UTC
Created attachment 5322 [details]
Proposed patch

I've done a new patch based on Brion's post, displays 10 then if its more generates a link.
Comment 6 X! 2008-09-15 01:26:46 UTC
Done in r40838
Comment 7 Brion Vibber 2008-09-15 18:34:08 UTC
Reverted in r40864; construction of the link looks very bizarre. We have regular linking functions for this sort of thing...
Comment 8 Matt Johnston 2008-09-16 23:40:40 UTC
Created attachment 5336 [details]
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.
Comment 9 Matt Johnston 2008-09-17 23:48:17 UTC
Patch committed in r40944 by Soxred93

Note You need to log in before you can comment on or make changes to this bug.