I mean, it should do exactly that,

Double-slam accept

Do as Valerio said. Tested before and after, two errors less in PHPStan output.

I saw that in my searching but I just end up with Argument 1 passed to PhabricatorCustomField::getObjectField() must implement interface PhabricatorCustomFieldInterface whenever I try to use it, perhaps because I'm unsure what to actually pass it as the $object other than $this, though I second-guess my second-guessing because ManifestCustomField, which I'm extending, does seem to have the interface the error is claiming I'm missing?

In your opinion, why is the current code not working? I mean, it should do exactly that, checking the text before, and the text after, so, to only mention newly-introduced mentions.

Eh, there is a trade-off: Editing the description of an existing task to add a @mention will not add this newly mentioned user as a task subscriber anymore.

Sorry for the delay with this. I've moved the null check as suggested.

Move null check to earlier in function

Make lint happy

Only one fix remains. I set "unclear" in the test plan accordingly to T15536. If you still want to merge, just:

git rebase master

rebase

feeex typo (theenks aklapper)

Either INVITEED_* is a typo or I don't get what the second E stands for. :)

Not sure what "similar working links" means... Something like https://we.phorge.it/diviner/find/?name=PhutilSafeHTML ?

Is this also the reason why links to e.g. https://we.phorge.it/diviner/find/?name=JX.behavior&type=function&jump=1 on https://we.phorge.it/book/javelin/article/behaviors/ or to http://we.phorge.it/diviner/find/?name=JX.Stratcom&type=class&jump=1 are 404s, or do I misunderstand and that's a separate issue?

Urgh. In calling https://we.phorge.it/source/phorge/browse/master/src/applications/diviner/query/DivinerAtomQuery.php$340 , $this->titles is an array which has a length of 1, with its only member being null. Sounds like there's already something fishy before.

Where I can find similar working links? I've tried generating Diviner books for both Arcanist and Phorge but http://phorge.localhost/diviner/find/?name=JX.Stratcom is still giving me 404

Thanks for finding a way to reproduce! I still fail to find the right timing here to produce a testcase, so this is another untested shot in the dark.

No, as project membership is unrelated to task object policy, as a task can associated to more than one project.

Just for the glory of the archive. I've tested this feature in production before landing.

I guess this is the time. Feel free to land as usual with

Well, I doubt we can dig more than this.

I think that is complex because some admins are not well active on their platforms, so as said before it can be underdimensioned

@aklapper I see. What do you think about D25301#15795 ?

• updated screenshot access to All Users
• thanks, that task would solve the problem!
• alternatively, "Project Members" option could be made available as an object policy for tasks

@valerio.bozzolan also, triggers don't offer an option to change task space or task access.

OK. Full immersion mode 🌈

I'm trying to reproduce, creating a repository, stopping phlog, and pushing, and starting and immediately randomly stopping phlog.

At the moment we have a multi-thread command existence checker, and it early dies if a command is not existing.