Page MenuHomePhorge

Fix RuntimeException: min(): Array must contain at least one element
ClosedPublic

Authored by aklapper on Aug 22 2023, 09:09.
Tags
None
Referenced Files
F3318127: D25417.1743274863.diff
Fri, Mar 28, 19:01
F3317757: D25417.1743267025.diff
Fri, Mar 28, 16:50
F3312520: D25417.1743211715.diff
Fri, Mar 28, 01:28
F3311685: D25417.1743207044.diff
Fri, Mar 28, 00:10
F3308821: D25417.1743177579.diff
Thu, Mar 27, 15:59
F3308694: D25417.1743174269.diff
Thu, Mar 27, 15:04
F3308438: D25417.1743167152.diff
Thu, Mar 27, 13:05
F3308058: D25417.1743158931.diff
Thu, Mar 27, 10:48

Details

Summary

Make sure that $epochs[] is never empty but contains 0 so min($epochs) will not complain.

Closes T15623

Test Plan

Carefully read the code (as I am unaware of reproduction steps).

Diff Detail

Repository
rP Phorge
Branch
T15623min (branched from master)
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 797
Build 797: arc lint + arc unit

Unit TestsFailed

TimeTest
229 msPhabricatorLibraryTestCase::testLibraryMap
Assertion failed, expected 'true' (at PhutilLibraryTestCase.php:45): The library map is out of date. Rebuild it with `arc liberate`. These entries differ: class.PhabricatorProjectDescriptionTransaction, xmap.PhabricatorProjectDescriptionTransaction.
76 msDiffusionServeControllerTestCase::testHandleRequest
2 assertions passed.
204 msPhabricatorCelerityTestCase::testCelerityMaps
1 assertion passed.
12 msPhabricatorConduitTestCase::testConduitMethods
1 assertion passed.
0 msPhabricatorInfrastructureTestCase::testApplicationsInstalled
1 assertion passed.
View Full Test Results (1 Failed · 7 Passed)

Event Timeline

Obviously not a clean tree, sorry. Thus ran arc liberate, now arc asked whether to include unstaged changes in src/__phutil_library_map__.php (yes)

and that's now wrong because src/__phutil_library_map__.php includes a non-existing file though I had ran git stash. I love git...

Try again, after an rm and arc liberate on master, and then a git rebase master on this branch

This revision is now accepted and ready to land.Aug 22 2023, 09:27