Rebase; improve PhpDoc

I hope that something is better than nothing I think. Then we can move if we don't like it.

If Harbormaster is fine I better be fine too, dont want no problems with them for reals

Those were just for the looks, but this is of course personal.
I can remove them if you want.

even simpler

Ah! yes - undo ConpherenceViewController

(I don't get the comment about D25784) thanks for feedback

I guess D25784 shouldn't be mashed up in here thus requesting changes.

Uh!

I fail to reproduce a problem locally (after commenting out entire ArcanistBundleTestCase::testGitRepository due to a different issue I'm running into locally).

In D25807#21516, @valerio.bozzolan wrote:

P.S. try applying D25809 to check if the red unit test becomes green

@mturdus: Thanks for the patch! I tested this locally on a wiki page with a bunch of sections and on http://phorge.localhost/book/phorge/article/remarkup/ ; and on Differential's /D1 where it does not affect the ToC panel (good).

fix typo

Deprecated. See wojak_depressed macro instead - https://we.phorge.it/macro/view/14/

LOL don't take this too seriously but sometime this cursed thing makes sense

From the text body:

IMPORTANT: Share that patch to your DBA <snip>

Moah fixees

Nice catch, thanks! Sorry for this regression in unit tests

Does appear fixed in latest master!

@valerio.bozzolan: Thank you! I know this is super boring work, but it allows PHPStan to show other type/class issues which are currently hidden due to PHPStan barking earlier about that non-standard question mark when parsing those lines. :-/

Abandoning in favor of T15890 and D25774 per previous discussion

Abandoning in favor of T15890 and D25774 per previous discussion

I confirm with my hand on my heart that I've consumed my eyes inspecting every single line to be absolutely sure that this PHPDoc probably has a better meaning and will not cause nuclear implosions.

In T15848#19430, @valerio.bozzolan wrote:

I would like to better understand the root problem.

Why changing the current user agent should improve the situation?

OK thanks (I need some rest)

In D25758#21773, @valerio.bozzolan wrote:

Maybe better to discuss a little bit in T15848: newOAuth1Future doesn't set a User-Agent since I'm curious but also confused

Not exactly sure what the confusion is. The patch and task are clear to me and solve the issue.

Moreover traditionally, the user agent is the name of the software, not the origin. So this could also deserve another comment about why we should replace the specific software name with a fixed origin. Premising that I agree that the current user agent may be re-discussed, to include more information, like, "Phorge", and the version of Phorge.

Actually no, user agents for automated tooling often includes some sort of contact email or website or link to a page describing the bot/tool. I think this should be treated like that.

Also, in case, maybe probably nice to have a getUserAgent(__CLASS__) method in parent class PhutilAuthAdapter()

@valerio.bozzolan: thats not what Andre says. He says another area of the code uses a terrible user agent and we should change that to a better one at the same time.

Maybe better to discuss a little bit in T15848: newOAuth1Future doesn't set a User-Agent since I'm curious but also confused

I would like to better understand the root problem. I see that Cloudflare was showing a captcha to Phorge (what Phorge?). It seems it was because an user agent was missing. But aklapper said that we are already setting an user agent.

I just discovered that sometime we can spawn warnings above the comment box.

güncelleme