Last modified: 2011-12-12 06:29:19 UTC
Created attachment 9655 [details] 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.
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!