What happens?:
You get log entries for all renames
What should have happened instead?:
It should ignore the extraneous whitespace after the ":"
Description
Description
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | BUG REPORT | Func | T337740 Special:Log mis-parses target with tab in it | ||
| Resolved | BUG REPORT | Func | T338042 Special:Log should show error when the input is invalid |
Event Timeline
Comment Actions
A tab character when URL encoded gives %09 and when used as part of a page name/title, it's normalized to a bad title from my tests with User:%09Foobar etc. Maybe we should validate the input and error that the username is invalid rather than try to normalize to a name that doesn't match the input?
I think the output you're getting now isn't supposed to even work that way since that would normalize to a bad title instead of showing the list of all renames.
@daniel / @Krinkle, what do you think should be a proper behavior here? Are we accepting %09 as part of a page name?
Comment Actions
Seems that the invalid title is not reported correctly to the form and internally using no title which results in showing the whole result (T338042)
The title is invalid in mediawiki: https://en.wikipedia.org/wiki/User%3A%09Foo