Once we remove this, we'll need to handle this edge-case:

• I've closed a ticket as "duplicate of T1", but I was wrong and want to mark it as "duplicate of T2" instead (and say it's not a dup of T1).

lol

I was able to reproduce this on a test instance even though the task is closed long ago:

Remove unused final protected methods in PhabricatorEditorExtension

But it's designed to accept an array of futures, not a single one. Maybe just wrap in array($result).

Test plan needed. But I don't know what an "hardpoint" is.

In D25709#19636, @valerio.bozzolan wrote:

OK.

@avivey what do you think about a new silly commodity method PhabricatorTransactions::findOneByType(array, string) ?

Thanks. I'm quite sure that everything is just calling writeCache().

Please wait at least for another +1, or, a week, before landing this. Since others may have different opinions on PhabricatorTransactions::findOneByType().

Updates after comments

Updates after comments

@avivey what do you think about a new silly commodity method PhabricatorTransactions::findOneByType(array, string) ?

Partially related to T15822: When a phriction page is moved, history should follow the page move

Thanks

In D25709#19616, @valerio.bozzolan wrote:

P.S. Why does Phriction have the field text_metadata? And why does Conpherence have content_metadata? 🤔

Or, to just download the current version of D25705 to eventually add some local extra changes, just:

If you would like to download the current version of D25705 to maybe land that, just:

In D25705#19508, @valerio.bozzolan wrote:

OK try this to restore your desired situation (so, two diffs)

git stash save

git checkout -b just_phriction
git reset --hard origin/master
arc patch --nobranch --diff 2173
arc diff --update D25705

git checkout -b just_conpherence
git reset --hard origin/master
arc patch --nobranch --diff 2178
arc diff --create

Then you have this benefit that you can repeat as many times you want:

git checkout just_phriction
LOL DO CHANGES
arc diff

git checkout just_conpherence
LOL DO CHANGES
arc diff

Or, get fresh master again:

git checkout master

Ideally, the default assumes that people do not touch master but work in branches, each branch a different feature

Remove unused save() method in PhabricatorUserCache

OK. Giving that also the parent::save() will fail, I think yes, feel free to nuke this method to see what happens. lol

In D25645#19609, @valerio.bozzolan wrote:

It would be nice to understand why nothing calls save() 🤔 since I guess nothing calls it, or we would have a crash report.

or am I supposed to $result = id(new ArcanistHardpointFutureList())->newFromFutures($result);? Yeez

P.S. Why does Phriction have the field text_metadata? And why does Conpherence have content_metadata? 🤔

We can avoid phutil_nonempty_string() since PHP pitfalls "Check for non-empty strings" are under control here.

shorter like in D25705

It would be nice to understand why nothing calls save() 🤔 since I guess nothing calls it, or we would have a crash report.

OK. Well, thaanks \o/

I'm really impressed. Tested and it's super-nice, with implicit middle fingers pointed to JavaScript.

Screenshots of {desktop, tablet, phone} width, exposing an arrow only on tablet and phone as we previously made people scroll large distances anyway and to increase discoverability of the feature in these viewports:

Apply phone style also to tablet style because same rendering (only one column)

In T15872#18251, @valerio.bozzolan wrote:

About this:

unset LC_ALL + LC_MESSAGES=C

I think it's not safe since other LC_STUFF can still be read 🤔

About this:

Set cursor: pointer in CSS

This [LANG=C] is supposed to be the default for missing LC_stuff

final defuser \o/ thanks

OK thanks! you persuaded me