Page MenuHomePhorge

Fix doc link to Restarting Phorge
ClosedPublic

Authored by l2dy on Nov 8 2023, 14:15.

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
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 896
Build 896: arc lint + arc unit

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.