Page MenuHomePhabricator

SpecialAllMessages: input form doesn't show and remember filter option
Closed, ResolvedPublic

Description

Patch against trunk

Goto for example https://en.wikipedia.org/wiki/Special:AllMessages. Next to "Filter by customisation state" no radiobutton is checked. Choose 'modified' or 'unmodified' and click go: again no radiobutton is checked. Clicking go again defaults to showing all messages instead of the selected filter.

This happens in 1.18 and 1.19.

Its because class AllmessagesTablePager uses $this->filter to set the radiobuttons instead of $this->custom.


Version: 1.20.x
Severity: normal

Attached:

Details

Reference
bz32958
TitleReferenceAuthorSource BranchDest Branch
jenkins-rel: remove GitHub pluginsrepos/releng/jenkins-deploy!44jnucheremove-github-pluginsmaster
Customize query in GitLab

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 12:00 AM
bzimport set Reference to bz32958.
bzimport added a subscriber: Unknown Object (MLST).

Done slightly differently on trunk in r105853 -- tagging for 1.18, 1.18wmf1 merge. Looks like something probably broke during a refactor and a var didn't get hoisted down to the tablepager class.

Good catch, and thanks for the report & patch!