Page MenuHomePhabricator

Closing a revision via a commit generates superfluous "mention" in Task
Closed, ResolvedPublic

Description

Probably introduced in rP3dd92a78ac2a (other commits since our last update don't seem to touch this code).

When someone pushes their commit, the revision gets closed, which gets mentioned in the task:

xxx added a commit: rXXabcdefabcdef: Use xx for xx to fix xx

However, it now also generates a mention:

xxx mentioned this in: rXXabcdefabcdef: Use xx for xx to fix xx

It seems a bit double. It doesn't bother me, but some coworkers were somewhat confused and thought they did something wrong.

(The commit(s) was not changed in any way after running arc amend)

Event Timeline

richardvanvelzen raised the priority of this task from to Needs Triage.
richardvanvelzen updated the task description. (Show Details)
richardvanvelzen updated the task description. (Show Details)
richardvanvelzen added a subscriber: richardvanvelzen.
chad added a subscriber: btrahan.
chad added a subscriber: chad.

Nice catch.

I am not reproducing this in my test environment.

I've now seen the same issue over several commits, leading me to believe
that it's not a fluke.

Can you show me a raw commit message involved in these shenanigans?

rP3dd92a78ac2a4931b7a673cc94fa61eae05f3bd1 basically starts parsing the commit message to generate mentions. "Vanilla" Phabricator does not put any references to differential objects - just links to the actual differential - which is why its not happening in my test install.

(This didn't reproduce on this install just now. See D10712 and corresponding commit rPac83ff706023e266907ed625954ef9b988daf014 )

Oh, I notice I forgot to mention it was about a task :/ My apologies for
that.

If you look at T2497 you see the mention I meant.

Ah, okay. Thanks! I'll get a patch out later today pr early tomorrow.

(If someone else wants to take a look, the phids from task fixes, task ref, and probably auditors need to get passed to setUnmentionablePHIDs in the editor. Challenge is probably getting all those relationships. Can likely be done just for TYPE_COMMIT transactions.)

richardvanvelzen renamed this task from Closing a revision via a commit generates superfluous "mention" to Closing a revision via a commit generates superfluous "mention" in Task.Oct 16 2014, 10:07 PM
richardvanvelzen updated the task description. (Show Details)
chad triaged this task as Normal priority.Oct 16 2014, 11:15 PM