Page MenuHomePhabricator

Add PhutilInvalidStateException class
ClosedPublic

Authored by joshuaspence on May 12 2015, 11:19 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jul 4, 4:12 AM
Unknown Object (File)
Jun 30 2025, 11:18 PM
Unknown Object (File)
Jun 29 2025, 9:04 PM
Unknown Object (File)
Jun 29 2025, 5:28 PM
Unknown Object (File)
Jun 28 2025, 10:22 PM
Unknown Object (File)
May 29 2025, 8:09 PM
Unknown Object (File)
May 29 2025, 5:09 PM
Unknown Object (File)
May 8 2025, 12:26 AM
Subscribers

Details

Summary

There's a bunch of Call %s before calling %s! exceptions that can be simplified with a custom script.

Test Plan

Wrote unit tests.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 5913
Build 5933: [Placeholder Plan] Wait for 30 Seconds

Unit TestsFailed

TimeTest
8,586 msPhabricatorCelerityTestCase::testCelerityMaps
789 msCelerityResourceTransformerTestCase::testTransformation
10,558 msDifferentialParseRenderTestCase::testParseRender
1,790 msDiffusionGitFileContentQueryTestCase::testAuthorName
1,557 msPhabricatorAphrontViewTestCase::testHasChildren
View Full Test Results (1 Failed · 16 Passed)

Event Timeline

joshuaspence retitled this revision from to Add PhabricatorInvalidStateException.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.

What do you think about putting this in libphutil? We probably have very few arcanist/libphutil callsites, but this doens't feel Phabricator-specific to me.

Yeah, I was considering that. Will do.

Use PhutilInvalidStateException

joshuaspence retitled this revision from Add PhabricatorInvalidStateException to Add PhutilInvalidStateException class.May 13 2015, 9:36 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 13 2015, 9:51 PM
This revision was automatically updated to reflect the committed changes.