Page MenuHomePhorge

Fix doc link to Restarting Phorge
ClosedPublic

Authored by l2dy on Nov 8 2023, 14:15.
Tags
None
Referenced Files
F3319034: D25459.1743296897.diff
Sat, Mar 29, 01:08
F3301120: D25459.1743051241.diff
Wed, Mar 26, 04:54
F3300606: D25459.1743043325.diff
Wed, Mar 26, 02:42
F3296366: D25459.1742972069.diff
Tue, Mar 25, 06:54
F3290026: D25459.1742858066.diff
Sun, Mar 23, 23:14
F3249289: D25459.1742285525.diff
Mon, Mar 17, 08:12
F3223584: D25459.1741978675.diff
Thu, Mar 13, 18:57
F3220148: D25459.1741800422.diff
Tue, Mar 11, 17:27

Details

Summary

See Q81. Link to /diviner/find/ could not find the documentation because it was renamed to "Restarting Phorge".

Test Plan
  1. Uninstall optional PHP extension zip and restart Phorge
  2. Vist /config/issue/ and click on a missing extension issue
  3. Click on the documentation link to Restarting

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

l2dy requested review of this revision.Nov 8 2023, 14:15

I've tested this locally with:

$ php -a
require 'scripts/init/init-script.php';
echo PhabricatorEnv::getDoclink('Restarting Phabricator');
echo PhabricatorEnv::getDoclink('Restarting Phorge');

Resulting in:

https://we.phorge.it/diviner/find/?name=Restarting%20Phabricator&type=article&jump=1

https://we.phorge.it/diviner/find/?name=Restarting%20Phorge&type=article&jump=1

So indeed the second works :)

Thanks for this change!

This revision is now accepted and ready to land.Nov 8 2023, 17:51
speck subscribed.

Would there be a bunch of these lying around?

In D25459#13138, @speck wrote:

Would there be a bunch of these lying around?

Probably but I cannot catch them. No other results with:

$ grep -R " Phabricator'" --include="*.php" .
This revision was automatically updated to reflect the committed changes.