From 78c2d58586990ee4259d7ff9ba84ca904ac55be1 Mon Sep 17 00:00:00 2001 From: Aviv Eyal Date: Sun, 22 Jun 2014 12:30:42 -0700 Subject: [PATCH] make "Land to..." button call diffusion.looksoon Summary: add looksoon call after every attempt at landing. This includes failed attempts, to elevate "not a fast-forward" issues, although there are probably smarter things to be done about that. Test Plan: Land, look at logs. Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: epriestley, Korvin Differential Revision: https://secure.phabricator.com/D9518 --- .../controller/DifferentialRevisionLandController.php | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/src/applications/differential/controller/DifferentialRevisionLandController.php b/src/applications/differential/controller/DifferentialRevisionLandController.php index 2f07b9ee60..2d0619b1a4 100644 --- a/src/applications/differential/controller/DifferentialRevisionLandController.php +++ b/src/applications/differential/controller/DifferentialRevisionLandController.php @@ -141,6 +141,15 @@ final class DifferentialRevisionLandController extends DifferentialController { } $lock->unlock(); + + $looksoon = new ConduitCall( + 'diffusion.looksoon', + array( + 'callsigns' => array($repository->getCallsign()) + )); + $looksoon->setUser($request->getUser()); + $looksoon->execute(); + return $response; }