Maniphest T220767

Some special pages are not properly displaying parenthesis and other seperators around user links
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Tks4Fish
	Apr 11 2019, 9:22 PM

Description

Today, after executing a CheckUser check at pt.wikipedia, I noticed that the parenthesis were not being displayed around the user links (USER talk contribs block INFO instead of USER (talk contribs block) INFO). After mentioning this on #wikimedia-tech on IRC, @Krenair said that it was not being displayed at Special:ListUsers too (at en.wikipedia). @Reedy tried to figure out what could have been causing it, but didn't find anything. After I mentioned that on the 6th it was being displayed, he mentioned that it probably is due to something that changed on .25.

Thanks in advance!

Details

Subject	Repo	Branch	Lines +/-
Revert "RecentChanges updated to use pseudo elements for presentation"	mediawiki/core	REL1_33	+46 -81
Linker: Use parentheses by default in userToolLinksRedContribs()	mediawiki/core	REL1_33	+7 -3
Linker: Use parentheses by default in userToolLinksRedContribs()	mediawiki/core	master	+7 -3
Make parentheses for user tools re-appear in several special pages	mediawiki/core	master	+2 -4
Make parentheses for user tools re-appear in several special pages	mediawiki/core	wmf/1.34.0-wmf.1	+2 -4

Customize query in gerrit

Related Objects

Mentioned In: T223871: Being unable to copy punctuation characters (defined in CSS) on certain special pages makes it harder to paste to text editor
T228565: Missing ( | ) around talk/contribs in previously deleted warning
T227190: VisualEditor doesn't show parenthesis in "this page is protected" notice
T224650: Parentheses and pipe separators missing on Special:AdvancedReviewLog
T222439: User links parentheses and separator broken on Mobile web
T219348: RecentChanges should be fully skinnable - use pseudo elements for separating elements
T220925: Missing characters causing unclear presentation on Special:ListUsers
Mentioned Here: T227190: VisualEditor doesn't show parenthesis in "this page is protected" notice
T223871: Being unable to copy punctuation characters (defined in CSS) on certain special pages makes it harder to paste to text editor
rMW5040b3f680cd: RecentChanges updated to use pseudo elements for presentation
T221253: Special:ListUsers displays tools without parentheses and pipes
T220885: Special:ListUsers is missing value separators
T206679: 1.33.0-wmf.25 deployment blockers

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 11 2019, 9:22 PM

Reedy triaged this task as High priority.Apr 11 2019, 9:22 PM

Ping T206679, but not really blocking (it's annoying but not overly broken)

Urbanecm subscribed.Apr 12 2019, 5:01 PM

Mainframe98 merged a task: T220885: Special:ListUsers is missing value separators.Apr 13 2019, 3:39 PM

Mainframe98 added subscribers: Xaosflux, Adithyak1997.

Notes from T220885

The following separators appear to be missing throughout the output:
word-separator
parentheses
pipe-separator

Xaosflux renamed this task from CU and special pages are not properly displaying parenthesis around user links to CU and special pages are not properly displaying parenthesis and other seperators around user links.Apr 13 2019, 3:43 PM

Xaosflux added projects: Regression, MW-1.33-notes (1.33.0-wmf.25; 2019-04-09).

As was noted in T220885 some special pages (e.g. https://test.wikipedia.org/wiki/Special:RecentChanges) are still using the separators.

Risker subscribed.Apr 13 2019, 4:35 PM

Ammarpad renamed this task from CU and special pages are not properly displaying parenthesis and other seperators around user links to Some special pages are not properly displaying parenthesis and other seperators around user links.Apr 13 2019, 4:39 PM

Mainframe98 added a project: MW-1.33-release.Apr 13 2019, 5:32 PM

Notes from enwiki discussion:

@Xaosflux: Looks like gerrit:499364 broke it by changing a method (Linker::userToolLinksRedContribs()) used by more than just the one special page that patch was trying to change. I'm not at liberty to copy that into Phabricator at the moment, if you could do so that might help. You might also CC the people involved with that change. Anomie⚔ 23:13, 13 April 2019 (UTC)

Xaosflux added subscribers: Jdlrobson, Catrope, kostajh and 2 others.Apr 13 2019, 11:40 PM

Amorymeltzer subscribed.Apr 14 2019, 2:00 AM

BethNaught mentioned this in T220925: Missing characters causing unclear presentation on Special:ListUsers.Apr 14 2019, 8:56 PM

BethNaught merged a task: T220925: Missing characters causing unclear presentation on Special:ListUsers.Apr 14 2019, 8:59 PM

BethNaught subscribed.

Reedy merged a task: T220929: Links to "talk" and "contribs" on Special:ListUsers no longer visually distinct from username links.Apr 14 2019, 11:56 PM

Reedy added a subscriber: Dcljr.

Anomie subscribed.Apr 15 2019, 11:10 AM

alaa subscribed.Apr 17 2019, 7:36 AM

Mainframe98 merged a task: T221253: Special:ListUsers displays tools without parentheses and pipes.Apr 17 2019, 3:46 PM

Mainframe98 added a subscriber: matej_suchanek.

Just wondering - is anyone working on this? Has anyone been able to validate what the breaking change was?

In T220767#5120700, @Xaosflux wrote:

Just wondering - is anyone working on this? Has anyone been able to validate what the breaking change was?

I'm guessing not, as it would have been identified/reverted..

I maybe did in T221253.

AmandaNP subscribed.Apr 18 2019, 7:48 AM

DoRD subscribed.Apr 18 2019, 11:44 AM

MusikAnimal subscribed.Apr 18 2019, 7:04 PM

I noticed that in CheckUser results, entries for IP addresses appear in the correct format, with parentheses and pipe characters, but entries with account names are missing those characters.

I'm sure some of these tags are wrong, but this is just being ignored after recently being broken and I'm trying to get someone's attention.

As @matej_suchanek noted, the breaking change is indeed 5040b3f680. I've tried to revert that change locally, everything started to work suddently. I can try to figure out which part of this change exactly was wrong.

In T220767#5121773, @matej_suchanek wrote:

I maybe did in T221253.

Yay dupe tasks ;)

In T220767#5125206, @Urbanecm wrote:

As @matej_suchanek noted, the breaking change is indeed 5040b3f680. I've tried to revert that change locally, everything started to work suddently. I can try to figure out which part of this change exactly was wrong.

Just revert it all then?

I guess no one pinged @Jdlrobson ?

Change 505242 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[mediawiki/core@master] Set $useParentheses to true in userToolLinksRedContribs

https://gerrit.wikimedia.org/r/505242

gerritbot added a project: Patch-For-Review.Apr 19 2019, 1:49 PM

In T220767#5125234, @gerritbot wrote:

Change 505242 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[mediawiki/core@master] Set $useParentheses to true in userToolLinksRedContribs

https://gerrit.wikimedia.org/r/505242

@Reedy This might be an easier solution. I'm not sure if this isn't breaking something else, but according to my local testing, it isn't causing anything like double parentheses or similar.

Reedy mentioned this in T219348: RecentChanges should be fully skinnable - use pseudo elements for separating elements.Apr 19 2019, 2:02 PM

In T220767#5125236, @Urbanecm wrote:

In T220767#5125234, @gerritbot wrote:

Change 505242 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[mediawiki/core@master] Set $useParentheses to true in userToolLinksRedContribs

https://gerrit.wikimedia.org/r/505242

@Reedy This might be an easier solution. I'm not sure if this isn't breaking something else, but according to my local testing, it isn't causing anything like double parentheses or similar.

Well, that looks a lot like a partial revert to me... :)

But if it's enough to solve the problem without nuking the whole patch, seems a better way forward to me

In T220767#5125236, @Urbanecm wrote:

In T220767#5125234, @gerritbot wrote:

Change 505242 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[mediawiki/core@master] Set $useParentheses to true in userToolLinksRedContribs

https://gerrit.wikimedia.org/r/505242

@Reedy This might be an easier solution. I'm not sure if this isn't breaking something else, but according to my local testing, it isn't causing anything like double parentheses or similar.

I do notice in LogFormatter.php in the original patch...

					// do not render parenthesises in the HTML markup (CSS will provide)

Was there a followup patch that wasn't merged?

As far as I can see, CSS provides parenthesises in Special:Log. That's visible when you try to copy a log item, it is copied without the parenthesis. For some reason, it doesn't provide parenthesis in some other places, like Special:ListUsers.

Of course, this also affects MW-1.33-release and we should be careful backports/fixes end up there too

Reedy edited projects, added MW-1.34-notes (1.34.0-wmf.1; 2019-04-16); removed MW-1.34-notes (1.34.0-wmf.3; 2019-04-30).Apr 19 2019, 3:10 PM

Hmm. So it seems there's no train next week... I think I'll cherry pick @Urbanecm's patch into .1, but leave it out of master, in the hope that @Catrope and/or @Jdlrobson (or anyone else on a responsible team) can deal with it in master before 1.34.0-wmf.3 is branched... There's about a week of working time after easter, but before the branch. Should be sufficient

Change 505262 had a related patch set uploaded (by Reedy; owner: Urbanecm):
[mediawiki/core@wmf/1.34.0-wmf.1] Make parentheses for user tools re-appear in several special pages

https://gerrit.wikimedia.org/r/505262

Change 505262 merged by jenkins-bot:
[mediawiki/core@wmf/1.34.0-wmf.1] Make parentheses for user tools re-appear in several special pages

https://gerrit.wikimedia.org/r/505262

Mentioned in SAL (#wikimedia-operations) [2019-04-19T15:54:07Z] <reedy@deploy1001> Synchronized php-1.34.0-wmf.1/includes/Linker.php: T220767 (duration: 00m 55s)

Mentioned in SAL (#wikimedia-operations) [2019-04-19T15:55:16Z] <reedy@deploy1001> Synchronized php-1.34.0-wmf.1/includes/logging/LogFormatter.php: T220767 (duration: 00m 53s)

Confirmed fixed on Special:CheckUser (get users) too

Leaving open as not fixed/merged in master, pending a possible better fix

Reedy added projects: Growth-Team, UI-Standardization.Apr 19 2019, 4:05 PM

Change 505324 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/core@master] Add interface.helpers.styles where UsersPager is used

https://gerrit.wikimedia.org/r/505324

Change 505326 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/core@master] Linker: Use parentheses by default in userToolLinksRedContribs()

https://gerrit.wikimedia.org/r/505326

Change 505242 abandoned by Urbanecm:
Make parentheses for user tools re-appear in several special pages

Reason:
replaced by I566d65e155258d3bd1a1a06bca9aa2b3a2d417ee

https://gerrit.wikimedia.org/r/505242

Change 505326 merged by jenkins-bot:
[mediawiki/core@master] Linker: Use parentheses by default in userToolLinksRedContribs()

https://gerrit.wikimedia.org/r/505326

Change 505335 had a related patch set uploaded (by Reedy; owner: Reedy):
[mediawiki/core@REL1_33] Revert "RecentChanges updated to use pseudo elements for presentation"

https://gerrit.wikimedia.org/r/505335

ReleaseTaggerBot edited projects, added MW-1.34-notes (1.34.0-wmf.3; 2019-04-30); removed MW-1.34-notes (1.34.0-wmf.1; 2019-04-16).Apr 19 2019, 11:00 PM

Change 505338 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/core@REL1_33] Linker: Use parentheses by default in userToolLinksRedContribs()

https://gerrit.wikimedia.org/r/505338

Change 505338 merged by jenkins-bot:
[mediawiki/core@REL1_33] Linker: Use parentheses by default in userToolLinksRedContribs()

https://gerrit.wikimedia.org/r/505338

Change 505335 abandoned by Reedy:
Revert "RecentChanges updated to use pseudo elements for presentation"

https://gerrit.wikimedia.org/r/505335

ReleaseTaggerBot edited projects, added MW-1.33-notes; removed MW-1.33-notes (1.33.0-wmf.25; 2019-04-09).Apr 20 2019, 12:00 AM

I'm back from vacation now. What's the status of this bug? How can I help.

In T220767#5129150, @Jdlrobson wrote:

I'm back from vacation now. What's the status of this bug? How can I help.

It was hotfixed by a patch I wrote and @Reedy backported to .1, and should be permanently fixed by https://gerrit.wikimedia.org/r/c/mediawiki/core/+/505326 (backported to REL1_33). Unless I'm overseeing something, all should be done - the only thing that needs to be done is https://gerrit.wikimedia.org/r/c/mediawiki/core/+/505324, which should just make the special page have parenthesis displayed by CSS and not hardcoded - which is not directly "fixing this bug". Correct me somebody if I'm wrong!

Should be taken care of now. Thanks all!

Jdlrobson closed this task as Resolved.Apr 25 2019, 6:20 AM

matej_suchanek removed a project: Patch-For-Review.Apr 28 2019, 9:07 AM

matej_suchanek unsubscribed.

Ammarpad mentioned this in T222439: User links parentheses and separator broken on Mobile web.May 3 2019, 10:53 AM

There is still a problem here. The patch caused the punctuation characters to be displayed again, but the characters cannot be selected in a browser. I haven't tested every browser, but on MacOS at least, this affects both Chrome and Firefox. In most cases, this isn't a problem, however for CheckUsers, it may be an issue.

When investigating complex sockpuppetry cases, I frequently copy/paste CU data into a text editor to make it easier to correctly format the results of the investigation for posting on-wiki. As it stands now, copying the data leaves out the parenthesis and pipe characters, so for this purpose, the fix didn't work. Please see the attached screenshots. [edit] The reason that the parens, etc., are needed for this purpose is because I use regex to find the username, and the first parens is what I search for.

What page is that? Can you open a new task with the URL and project you are seeing it on? Thanks in advance!

I can confirm that what was reported by DoRD is also happening on Chrome on Windows 10. I had that issue a few days ago too, which made pasting results a lot of work.

In T220767#5194044, @Jdlrobson wrote:

What page is that? Can you open a new task with the URL and project you are seeing it on? Thanks in advance!

This page is Special:Checkuser, specifically, from what I can see, "Get users" option when checking an IP.

List users does it as well, for example: https://en.wikipedia.org/wiki/Special:ListUsers?username=DoRD&group=checkuser&wpsubmit=&wpFormIdentifier=mw-listusers-form&limit=1

I've created a new task to avoid confusion since the reported problem is not about display but workflow. See T223871.

DannyS712 mentioned this in T224650: Parentheses and pipe separators missing on Special:AdvancedReviewLog.May 30 2019, 6:37 AM

Urbanecm mentioned this in T227190: VisualEditor doesn't show parenthesis in "this page is protected" notice.Jul 3 2019, 11:25 AM

Change 523806 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Add dependency on 'mediawiki.interface.helpers.styles'

https://gerrit.wikimedia.org/r/523806

gerritbot added a project: Patch-For-Review.Jul 16 2019, 8:58 PM

Oops sorry, I meant to tag that with T227190.

matmarex removed a project: Patch-For-Review.Jul 16 2019, 8:58 PM

Aklapper mentioned this in T228565: Missing ( | ) around talk/contribs in previously deleted warning.Jul 20 2019, 3:16 PM

Aklapper mentioned this in T223871: Being unable to copy punctuation characters (defined in CSS) on certain special pages makes it harder to paste to text editor.Feb 6 2020, 12:32 PM

Aklapper removed a subscriber: Anomie.Oct 16 2020, 5:40 PM

	F29111628: Screen Shot 2019-05-18 at 8.04.25 AM.png
	May 18 2019, 2:05 PM

	F29111629: Screen Shot 2019-05-18 at 8.55.18 AM.png
	May 18 2019, 2:05 PM

Some special pages are not properly displaying parenthesis and other seperators around user linksClosed, ResolvedPublicActions

Description

Details

Related Objects

Event Timeline

Some special pages are not properly displaying parenthesis and other seperators around user links
Closed, ResolvedPublic
Actions