In Differential, filter repository operations to just "Land" operations again
Summary: Reverts D18276. See PHI18 for discussion. The additional rules here (roughly, "only show the first successful operation") didn't actually work out for the other types of operations. This is all just figuring out a stopgap, T12935 and other changes should eventually provide real pathways here. Test Plan: Straight revert. Reviewers: chad Reviewed By: chad Differential Revision: https://secure.phabricator.com/D18281
This commit is contained in:
@@ -1051,14 +1051,14 @@ final class DifferentialRevisionViewController extends DifferentialController {
|
||||
return null;
|
||||
}
|
||||
|
||||
// NOTE: The upstream currently supports only "Land" operations, but
|
||||
// third parties have other operations (see PHI18). For now, we show any
|
||||
// operation that exists. We'll refine this in the future.
|
||||
|
||||
$operations = id(new DrydockRepositoryOperationQuery())
|
||||
->setViewer($viewer)
|
||||
->withObjectPHIDs(array($revision->getPHID()))
|
||||
->withIsDismissed(false)
|
||||
->withOperationTypes(
|
||||
array(
|
||||
DrydockLandRepositoryOperation::OPCONST,
|
||||
))
|
||||
->execute();
|
||||
if (!$operations) {
|
||||
return null;
|
||||
|
||||
Reference in New Issue
Block a user