recaptime.dev / phorge
@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.) hq.recaptime.dev/wiki/Phorge
phorge phabricator
1
fork

Configure Feed

Select the types of activity you want to include in your feed.

Add missing return to switch/case statement

Summary: `case` clauses in `switch` statements are expected to end with a `return` or `break` statement.

Test Plan: Run static code analysis.

Reviewers: O1 Blessed Committers, avivey

Reviewed By: O1 Blessed Committers, avivey

Subscribers: avivey, tobiaswiese, valerio.bozzolan, Matthew, Cigaryno

Differential Revision: https://we.phorge.it/D26478

Andre Klapper e7a5dd63 816865e1

+2
+2
src/applications/differential/storage/DifferentialTransaction.php
··· 422 422 return 'fa-flag'; 423 423 case DifferentialAction::ACTION_CLAIM: 424 424 return 'fa-flag'; 425 + default: 426 + break; 425 427 } 426 428 case PhabricatorTransactions::TYPE_EDGE: 427 429 switch ($this->getMetadataValue('edge:type')) {