Differential D25005 Diff 32 src/applications/harbormaster/integration/HarbormasterHookHandler.php

Changeset View

Standalone View

				<?php

				abstract class HarbormasterHookHandler
				extends Phobject {

				public static function getHandlers() {
				return id(new PhutilClassMapQuery())
				->setAncestorClass(__CLASS__)
				->setUniqueMethod('getName')
				->execute();
				}

				public static function getHandler($handler) {
				$base = idx(self::getHandlers(), $handler);

				if ($base) {
				return (clone $base);
				}

				return null;
				}

				abstract public function getName();

				abstract public function handleRequest(AphrontRequest $request);
				speckUnsubmitted Not Done Inline Actions Could we add documentation to these for future implementations know what's expected? I think in particular `getName()` actually affects the expected URL to be hit from an external CI, correct? Would a UT be appropriate to ensure that implementations of `HarbormasterHookHandler` don't overlap their names? speck: Could we add documentation to these for future implementations know what's expected? I think in…

				}