Page MenuHomePhabricator

PATCH: Special:Export: prevent option selector flying back to top of list
Closed, ResolvedPublic

Description

Special:Export, when doing 'add pages from namespace',
passes the form parameter "nsindex" back to the server.

This "nsindex" is primarily used to pick pages, but should be used to
set that <option selected> when the form is returned to the user.

Unfortunately this information is thrown away, and the selector flies
back up to the default (first, NS_MAIN) value. The user has to figure
out where he was and what he was doing all over again.

This patch fixes it.

Note, the form still lacks a reset button.
If one wants to start over clean, and is not a power user.


Version: 1.15.x
Severity: normal

Details

Reference
bz18031

Event Timeline

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

Created attachment 5939
patch

Attached:

<b>Notice</b>: Undefined variable: nsindex in <b>/Library/WebServer/Documents/trunk/includes/specials/SpecialExport.php</b> on line <b>154</b><br />