Last modified: 2008-10-04 18:10:35 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T12323, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 10323 - Special:Undelete should have "select all" checkbox or "inverse selection" button
Special:Undelete should have "select all" checkbox or "inverse selection" button
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
unspecified
All All
: Normal enhancement with 2 votes (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-06-20 19:56 UTC by Danny B.
Modified: 2008-10-04 18:10 UTC (History)
3 users (show)

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


Attachments
Preliminary patch to add the "invert selection" feature (2.13 KB, patch)
2007-11-23 13:15 UTC, Huji
Details
"Invert selection" button acting on the server-side (3.65 KB, patch)
2007-11-24 20:09 UTC, Huji
Details

Description Danny B. 2007-06-20 19:56:56 UTC
Special:Undelete should have "select all" checkbox or "inverse selection" button (or both), because sometimes is necessary to undelete many revisions except for some, therefore is very annoying to click on every single checkbox.

Using the first or second solution would make it much more comfortable.

Thanks
Comment 1 Raimond Spekking 2007-06-20 20:05:59 UTC

*** This bug has been marked as a duplicate of bug 10056 ***
Comment 2 Danny B. 2007-06-20 20:40:40 UTC
This is not a dupe of bug 10056, it takes different approach.

There's no request for changing of default behavior (unchecked boxes), there's a request for adding of small javascript (I can provide) to handle the selection.

However, regarding to the bug 10056, the "select all" checkbox can be omitted in favour of "inverse selection" which will do the same job.

Also the script can not let the user submit the form, if all items checked.
Comment 3 Rob Church 2007-06-20 21:09:30 UTC
This should not be done until bug 10056 comment 3 has been addressed, so that we don't have to add irritating and confusing restrictions on when the user can submit the form; the handler should be altered such that the user can cause no accidental increase in complexity by operating the user interface in an apparently intuitive manner.
Comment 4 Danny B. 2007-06-20 21:42:37 UTC
Actually, the script can check befor submit if all items checked and if yes then uncheck them all, which would solve it.
Comment 5 Rob Church 2007-06-21 10:17:59 UTC
That doesn't seem like a particularly clever solution either, to be honest, because it will look extremely strange to most users who don't know what's going on, or why.
Comment 6 Brion Vibber 2007-06-21 15:58:25 UTC
I could have sworn there were already "select all" and "clear" buttons, I guess I was thinking of a different form. :)

Nothing additional is required, just add standard JS-friendly buttons. (If sufficiently adventurous, the 'check all' could be done in a non-JS-requiring way as well, where submitting with it will cause the resubmit to check the buttons instead of performing an action, while JS can still do the checks without submission. But, eh, details.)
Comment 7 AlexSm 2007-06-29 15:07:06 UTC
Danny B: the script is already there, hold Shift while clicking the checkbox on the other end. Of course, the problem is that very few people know about this feature. (Well, at least I managed to insert the info into http://en.wikipedia.org/wiki/MediaWiki:Watcheditlist/Check_all )

So adding "inverse selection" Javascript link makes sense.
Comment 8 Huji 2007-11-23 13:15:44 UTC
Created attachment 4363 [details]
Preliminary patch to add the "invert selection" feature

I created a patch which adds "invert selection" feature. I added a new message for the link, and added the relevant lines of code to Wikibits.js to make sure it is available in all skins.

This is a preliminary patch, and your comments are needed to improve it, in terms of a better look in user interface, etc.
Comment 9 Huji 2007-11-24 20:09:17 UTC
Created attachment 4373 [details]
"Invert selection" button acting on the server-side

I created a new patch to solve the selection inversion issue. This time, I've added a button (rather than a link) which doesn't rely on Javascript (acts on serer-side).

Hopefully, it can be applied after being reviewed by a more experienced developer.
Comment 10 Huji 2007-11-25 18:04:52 UTC
In order to put less load on the servers, a possible solution will be to mix the patches I provided above, so if JS is enabled, the client-side script is run, and if JS is disabled, the server-side code is run. What do you think about this?
Comment 11 spiritia 2008-10-03 08:45:04 UTC
These features -- select all, select inverse, clear -- are badly needed in Special:Undelete for merging page histories. In these situations one always have to restore virtually all deleted versions. 
Comment 12 Huji 2008-10-04 18:01:12 UTC
I committed the server-side version of the "Invert selection" button with r41659. If a client-side solution is also desirable, it can be added separately. A "select all" button is not needed; if it is intended to restore all revesions, then all checkboxes can be left unchecked; if the intent is to use "select all" and then uncheck a few items, this can be done now by selecting those items first and then inverting the selection.
Comment 13 Huji 2008-10-04 18:10:35 UTC
I had to revert r41659 and re-commit the changes in r41661 due to accidental breakage.

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


Navigation
Links