Ouch. Indeed my last comment had no sense. I was meaning T15941 that should be kept opened.

Ops I've closed this by mistake. We closed just T15564 👍

Can this be resolved? If not, what exactly is left in this task (apart from separate T15803)?

P.S. - XACT exists, but it does not support the newObject():

Additionally, I don't know if we are able here to show the "supported types" in line 391 (that are less than the existing types, needed in the other if case).

In D25988#26522, @aklapper wrote:

Should ideally use the same error message as in https://we.phorge.it/source/phorge/browse/master/src/applications/transactions/conduit/TransactionSearchConduitAPIMethod.php;4862eada5cd05236b81487b261668f2a2d72fad7$382-387 so one string less to translate.

Ironically though, that existing message will list XACT used to trigger this issue:

Ops I've closed this by mistake. We closed just T15564 👍

Uploading the patch that works on my computer now®

Thanks for this patch that unfortunately it's revealing even more weird legacy ghosts.

Mark as "some comments"

Should ideally use the same error message as in https://we.phorge.it/source/phorge/browse/master/src/applications/transactions/conduit/TransactionSearchConduitAPIMethod.php;4862eada5cd05236b81487b261668f2a2d72fad7$382-387 so one string less to translate.

Covered, thanks for review :3

I need to sleep

transaction.search: handle unmanaged exception when objectType is not supported

Thaanks. Can reproduce in latest master. Premising that uhmmmm probably it seems we are not using the API correctly as the correct usage seems with objectType=TASK. Example to get my transactions:

add more edge cases in unit tests

Don't think this is needed for a standard install. The gerritbot colors could be implemented only on phabricator.wikimedia.org.

The colors indicate patch status (the bolded words in the comment) – blue for new, green for merged, red for abandoned.

To put it another way:
I think the described use-case is too narrow, and a naive expansion of the use-case isn't scalable.
The described use-case fails for at least one possible use-case ("some bots have something useful to say").

In D25987#26487, @valerio.bozzolan wrote:

having "User Bot" possibly everywhere? 🌈

Personally I'd decline T16052 (data duplication) and I do not yet see a need for potential new transaction types either (a comment is a comment is a comment no matter who/what made it).

Yes, this is where my first comment enters - I feel this is a rabbit-hole we shouldn't venture into, etc.
The current script appears to have 3 names, and referring to "legacy data" implies that there won't be any new names to add.

If you mean this kind of hardcoded CSS rules, yes, it's possible for local installs:

the legacy data can be handled by the already-existing hard-coded names...

Yessss, I agree and we explored a bit the creation of new transactions as good long-term direction, unfortunately it seems still necessary a bit of CSS help from the backend to cover the legacy data (e.g. 10 years old bot-generated comments, with traditional comments)

I feel this is a rabbit-hole we shouldn't venture into.
The slippery-slope argument will make us adding a custom class for each individual user, so css extensions can be used to hide/highlight comments from boss/intern/etc.
It's also probably not enough to remove the hard-coded requirement either - in some environments, one "bot" user is copying comments from another platform, and another is making statistical updates about a jira ticket, so you'd still need a better filter.

Uhm... can we sleep with the very possible possibility of having "User Bot" possibly everywhere? 🌈

removed nonsense comment (last change.zip)

always set a bool so we have an explicit value and don't just fallback to "false"

Don't bool with the bool otherwise it bools

fix damn 81 lines lint

add destroy unit test for A > B > C and milestone (I love computers)

The docs are surprisingly less jokey than I feared. Left a few minor change suggestions, as well as in the code comments.

Note: the same issue appears to also exist for images:

look mum, I'm learning English

improve comments (again) - thanks