| Ciencia_Al_Poder | |
| Nov 15 2017, 10:23 AM |
| F10803976: Captura el 2017-11-15 a las 11.21.03-fullpage.png | |
| Nov 15 2017, 10:23 AM |
I've even changed the filter to show last 5000 changes, in the past 30 days, and it doesn't display more than the default 50
See screenshot when not logged-in (being logged-in or not doesn't seem to make a difference):
It was working correctly yesterday at least
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Addshore | T173818 [Epic] Kill the Wikidata build step | |||
| Resolved | Release | • demon | T178635 1.31.0-wmf.8 deployment blockers | ||
| Duplicate | None | T180549 [wmf.8-regression] New filters do not display records according to selected options for "Number of edits to show..." | |||
| Resolved | • Catrope | T180577 [Regression] Recent Changes on MediaWiki.org doesn't display more than 50 past edits |
I'm gonna be bold and mark it as UBN since it's a regression (apparently) of 1.31.0-wmf.8, being deployed to major wikipedias today, to attract attention at least
It is also broken on test wiki, which means it may be in the deployment train. Clearly UBN.
The Global collaboration team has been informed about it.
This only affects the new filters, not the old filters. The issue seems that the URL is not updating when the limit is changed. If I add it manually to the URL it works.
git bisect says b8a10e6dcf00da3519ccb9e43d1c2ce0db422557 is the first bad commit.
I don't know how it breaks things. It seems to revert clearly, so that could be done as a quick fix.
I talked to Moriel and we discovered that it breaks things by removing "excluded" parameters (which includes "days" and "limit") from the URL both in the address bar and in the AJAX request. The latter was not intended.
She says there should be no negative side effects from reverting that commit, so I'm going to SWAT a revert in half an hour.
Change 391613 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/core@master] Revert "RCFilters: Remove excluded params from URL"
Change 391614 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/core@wmf/1.31.0-wmf.8] Revert "RCFilters: Remove excluded params from URL"
Change 391614 merged by jenkins-bot:
[mediawiki/core@wmf/1.31.0-wmf.8] Revert "RCFilters: Remove excluded params from URL"
Change 391613 merged by jenkins-bot:
[mediawiki/core@master] Revert "RCFilters: Remove excluded params from URL"
Mentioned in SAL (#wikimedia-operations) [2017-11-15T19:19:49Z] <catrope@tin> Synchronized php-1.31.0-wmf.8/resources/src/mediawiki.rcfilters/mw.rcfilters.UriProcessor.js: T180577 (duration: 00m 49s)
Thanks for your participation @Ciencia_Al_Poder. But it's safer to let our QA person check for sure that this is completely fixed, so I'm reopening to let that process play out.
Given the UBN! priority and the fact that blockers to the train, well, block the train, can that happen sooner than later? ASAP would be good (again, UBN!).