Page MenuHomePhorge

Fix information page about Read-only status
AbandonedPublic

Authored by valerio.bozzolan on Mar 24 2023, 08:41.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 10, 23:53
Unknown Object (File)
Mon, Apr 8, 23:24
Unknown Object (File)
Mon, Apr 8, 23:17
Unknown Object (File)
Mon, Apr 1, 01:00
Unknown Object (File)
Mar 21 2024, 15:49
Unknown Object (File)
Mar 20 2024, 22:00
Unknown Object (File)
Mar 16 2024, 21:48
Unknown Object (File)
Mar 9 2024, 19:42

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)