Page MenuHomePhorge

Fix information page about Read-only status
AbandonedPublic

Authored by valerio.bozzolan on Mar 24 2023, 08:41.
Tags
None
Referenced Files
F3301719: D25090.1743057109.diff
Wed, Mar 26, 06:31
F3299451: D25090.1743022032.diff
Tue, Mar 25, 20:47
F3298127: D25090.1742997202.diff
Tue, Mar 25, 13:53
F3297392: D25090.1742984722.diff
Tue, Mar 25, 10:25
F3224859: D25090.1742089419.diff
Sat, Mar 15, 01:43
F3223381: D25090.1741944269.diff
Thu, Mar 13, 09:24
F3220433: D25090.1741810056.diff
Tue, Mar 11, 20:07
F3220432: D25090.1741810053.diff
Tue, Mar 11, 20:07

Details

Reviewers
None
Group Reviewers
O1: Blessed Committers
Summary

This change fixes the following page on your Phorge/Phabricator:

/readonly/config/

If you visit that page (with or without read-only mode), the
following crash will happen:

Call to undefined method PlatformSymbols::getPlatformServerSymbol()

For more information:

https://we.phorge.it/T15185

Probably the regression was introduced in this point:

https://secure.phabricator.com/rP4d3f0dc7c727e6fdc0e8cba0e8e8f2b8ef1bd16c

https://we.phorge.it/rP4d3f0dc7c727e6fdc0e8cba0e8e8f2b8ef1bd16c

Test Plan
  • Visit the page /readonly/config/
  • It does not crash anymore and you can see that beautiful message that - it seems - no person in the world has ever seen since a whole year. Take a good look at this page: it is a precious treasure.

Diff Detail

Repository
rP Phorge
Branch
config-readonly
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 167
Build 167: arc lint + arc unit

Unit TestsFailed

TimeTest
829 msPhabricatorCelerityTestCase::testCelerityMaps
Assertion failed, expected 'true' (at PhabricatorCelerityTestCase.php:32): When this test fails, it means the Celerity resource map is out of date. Run `bin/celerity map` to rebuild it. ACTUAL VALUE
898 msPhabricatorConduitTestCase::testConduitMethods
1 assertion passed.
226 msPhabricatorInfrastructureTestCase::testApplicationsInstalled
1 assertion passed.
222 msPhabricatorInfrastructureTestCase::testRejectMySQLNonUTF8Queries
1 assertion passed.
1,306 msPhabricatorLibraryTestCase::testEverythingImplemented
1 assertion passed.
View Full Test Results (1 Failed · 6 Passed)