Page MenuHomePhorge
Differential D25417

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

Authored by aklapper on Aug 22 2023, 09:09.
Tags
None
Referenced Files
F2992772: D25417.1740249148.diff
Fri, Feb 21, 18:32
F2992211: D25417.1740207953.diff
Fri, Feb 21, 07:05
F2989976: D25417.1740168291.diff
Thu, Feb 20, 20:04
F2989975: D25417.1740168290.diff
Thu, Feb 20, 20:04
F2989974: D25417.1740168288.diff
Thu, Feb 20, 20:04
F2989973: D25417.1740168288.diff
Thu, Feb 20, 20:04
F2989972: D25417.1740168286.diff
Thu, Feb 20, 20:04
F2989925: D25417.1740167801.diff
Thu, Feb 20, 19:56
View All 39 Files
Subscribers
Cigaryno
Matthew
tobiaswiese
valerio.bozzolan

Details

Reviewers
avivey
Group Reviewers
O1: Blessed Committers
Maniphest Tasks
T15623: RuntimeException: min(): Array must contain at least one element at DiffusionBlameController.php:201
Commits
rPbeec08e01913: Fix RuntimeException: min(): Array must contain at least one element
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
Tests Passed
Build Status
Buildable 800
Build 800: arc lint + arc unit

Event Timeline

aklapper created this revision.Aug 22 2023, 09:09
Owners added a reviewer: O1: Blessed Committers.Aug 22 2023, 09:09
Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptAug 22 2023, 09:09
aklapper requested review of this revision.Aug 22 2023, 09:09
Harbormaster completed remote builds in B797: Diff 1320.Aug 22 2023, 09:09
aklapper updated this revision to Diff 1321.Aug 22 2023, 09:12

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)

Harbormaster completed remote builds in B798: Diff 1321.Aug 22 2023, 09:13
aklapper added a comment.Aug 22 2023, 09:15

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...

aklapper updated this revision to Diff 1322.Aug 22 2023, 09:17

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

Harbormaster completed remote builds in B799: Diff 1322.Aug 22 2023, 09:17
aklapper updated this revision to Diff 1323.Aug 22 2023, 09:20

shrug

Harbormaster completed remote builds in B800: Diff 1323.Aug 22 2023, 09:20
avivey accepted this revision.Aug 22 2023, 09:27
This revision is now accepted and ready to land.Aug 22 2023, 09:27
avivey added a comment.Aug 22 2023, 09:27

whatcouldgowrong

Closed by commit rPbeec08e01913: Fix RuntimeException: min(): Array must contain at least one element. · Explain WhyAug 22 2023, 16:31
This revision was automatically updated to reflect the committed changes.
aklapper added a commit: rPbeec08e01913: Fix RuntimeException: min(): Array must contain at least one element.

Revision Contents
Changeset List

PathSizePackages
src/
applications/
diffusion/
controller/
4 linesO1: Blessed Committers

Diff 1323

View Options

src/applications/diffusion/controller/DiffusionBlameController.php

Loading...
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0