Page MenuHomePhabricator

Wrong curid for history link in watchlist for category changes with wpusenewrc enabled
Closed, ResolvedPublic

Event Timeline

It seems this has been quietly resolved. Now categories with grouped changes are followed only by the plain text "(n changes)", no "history" link. Can someone identify the commit that implemented this change and close this task?

However, wrong curids still appear in permalinks (timestamps) in collapsed lines.

The relevant line of code that hides the wrong "history" link is this one here:$595. It exists since 2016. Nothing changed, as far as I can see.

I suggest to close this ticket and open a new one for the wrong curid in collapsed lines.

Nardog closed this task as Resolved.EditedDec 23 2020, 4:40 PM

@thiemowmde Alright, created T270774: Wrong curid and title (tooltip) in grouped category changes on Watchlist/RecentChanges.

T147770 wasn't really a duplicate, by the way, because that one was about diff links and this one history links. But both tasks indeed seem to have been squashed by the same change nevertheless.

It's weird you say it was fixed in 2016. I thought I'd seen those broken diff and history links just months ago, or my brain is failing me.