silverstripe / deploynaut

code/backends/DeploymentStrategy.php

Doc Comments +2 added lines

@@ -79,6 +79,7 @@ 
     }
 
     /**
+     * @param string $code
      */
     public function setActionCode($code)
     {
@@ -95,6 +96,7 @@ 
 
     /**
      * @param int
+     * @param string $seconds
      */
     public function setEstimatedTime($seconds)
     {

Braces +25 added lines, -50 removed lines

@@ -1,7 +1,6 @@ 
 <?php
 
-class DeploymentStrategy extends ViewableData
-{
+class DeploymentStrategy extends ViewableData {
 
     const SUCCESS_CODE = 'success';
 
@@ -56,8 +55,7 @@ 
      * @param DNEnvironment $environment
      * @param array $options
      */
-    public function __construct(DNEnvironment $environment, $options = array())
-    {
+    public function __construct(DNEnvironment $environment, $options = array()) {
         $this->environment = $environment;
         $this->options = $options;
     }
@@ -65,47 +63,41 @@ 
     /**
      * @param string $title
      */
-    public function setActionTitle($title)
-    {
+    public function setActionTitle($title) {
         $this->actionTitle = $title;
     }
 
     /**
      * @return string
      */
-    public function getActionTitle()
-    {
+    public function getActionTitle() {
         return $this->actionTitle;
     }
 
     /**
      */
-    public function setActionCode($code)
-    {
+    public function setActionCode($code) {
         $this->actionCode = $code;
     }
 
     /**
      * @return string
      */
-    public function getActionCode()
-    {
+    public function getActionCode() {
         return $this->actionCode;
     }
 
     /**
      * @param int
      */
-    public function setEstimatedTime($seconds)
-    {
+    public function setEstimatedTime($seconds) {
         $this->estimatedTime = $seconds;
     }
 
     /**
      * @return int Time in minutes
      */
-    public function getEstimatedTime()
-    {
+    public function getEstimatedTime() {
         return $this->estimatedTime;
     }
 
@@ -114,8 +106,7 @@ 
      * @param string $from
      * @param string $to
      */
-    public function setChange($title, $from, $to)
-    {
+    public function setChange($title, $from, $to) {
         return $this->changes[$title] = array(
             'from' => $from,
             'to' => $to
@@ -126,8 +117,7 @@ 
      * @param string $title
      * @param string $desc
      */
-    public function setChangeDescriptionOnly($title, $desc)
-    {
+    public function setChangeDescriptionOnly($title, $desc) {
         return $this->changes[$title] = array(
             'description' => $desc
         );
@@ -138,8 +128,7 @@ 
      *
      * @return array
      */
-    public function getChangesModificationNeeded()
-    {
+    public function getChangesModificationNeeded() {
         $filtered = [];
         foreach ($this->changes as $change => $details) {
             if (array_key_exists('description', $details)) {
@@ -164,8 +153,7 @@ 
      *		)
      *	)
      */
-    public function getChanges()
-    {
+    public function getChanges() {
         return $this->changes;
     }
 
@@ -174,8 +162,7 @@ 
      *
      * @return ArrayData|null
      */
-    public function getChange($key)
-    {
+    public function getChange($key) {
         $changes = $this->getChanges();
         if (array_key_exists($key, $changes)) {
             return new ArrayData($changes[$key]);
@@ -187,8 +174,7 @@ 
      * @param string $option
      * @param string $value
      */
-    public function setOption($option, $value)
-    {
+    public function setOption($option, $value) {
         $this->options[$option] = $value;
     }
 
@@ -196,8 +182,7 @@ 
      * @param string $option
      * @return string|null
      */
-    public function getOption($option)
-    {
+    public function getOption($option) {
         if (!empty($this->options[$option])) {
             return $this->options[$option];
         }
@@ -206,32 +191,28 @@ 
     /**
      * @return string
      */
-    public function getOptions()
-    {
+    public function getOptions() {
         return $this->options;
     }
 
     /**
      * @param string $code
      */
-    public function setValidationCode($code)
-    {
+    public function setValidationCode($code) {
         $this->validationCode = $code;
     }
 
     /**
      * @return string
      */
-    public function getValidationCode()
-    {
+    public function getValidationCode() {
         return $this->validationCode;
     }
 
     /**
      * @param string $msg
      */
-    public function setMessage($msg, $code = self::ERROR_CODE)
-    {
+    public function setMessage($msg, $code = self::ERROR_CODE) {
         $this->messages[] = [
             'text' => $msg,
             'code' => $code
@@ -251,8 +232,7 @@ 
     /**
      * @return array
      */
-    public function getMessages()
-    {
+    public function getMessages() {
         return $this->messages;
     }
 
@@ -261,8 +241,7 @@ 
      *
      * @return array
      */
-    public function toArray()
-    {
+    public function toArray() {
         $fields = array(
             'actionTitle',
             'actionCode',
@@ -283,8 +262,7 @@ 
     /**
      * @return string
      */
-    public function toJSON()
-    {
+    public function toJSON() {
         return json_encode($this->toArray(), JSON_PRETTY_PRINT);
     }
 
@@ -294,8 +272,7 @@ 
      *
      * @param string $json
      */
-    public function fromJSON($json)
-    {
+    public function fromJSON($json) {
         $decoded = json_decode($json, true);
         return $this->fromArray($decoded);
     }
@@ -306,8 +283,7 @@ 
      *
      * @param string $data
      */
-    public function fromArray($data)
-    {
+    public function fromArray($data) {
         $fields = array(
             'actionTitle',
             'actionCode',
@@ -328,8 +304,7 @@ 
     /**
      * @return DNDeployment
      */
-    public function createDeployment()
-    {
+    public function createDeployment() {
         $deployment = DNDeployment::create();
         $deployment->EnvironmentID = $this->environment->ID;
         // Pull out the SHA from the options so we can make it queryable.

code/control/DNRoot.php

Doc Comments +2 added lines, -2 removed lines

@@ -997,7 +997,7 @@ 
      * @param array $data
      * @param Form $form
      *
-     * @return bool|HTMLText|SS_HTTPResponse
+     * @return SS_HTTPResponse|null
      */
     public function doCreateEnvironment($data, Form $form)
     {
@@ -2430,7 +2430,7 @@ 
     /**
      * Create project action.
      *
-     * @return SS_HTTPResponse
+     * @return HTMLText
      */
     public function createproject(SS_HTTPRequest $request)
     {

Braces +84 added lines, -168 removed lines

@@ -170,8 +170,7 @@ 
     /**
      * Include requirements that deploynaut needs, such as javascript.
      */
-    public static function include_requirements()
-    {
+    public static function include_requirements() {
 
         // JS should always go to the bottom, otherwise there's the risk that Requirements
         // puts them halfway through the page to the nearest <script> tag. We don't want that.
@@ -209,8 +208,7 @@ 
      *
      * @return boolean
      */
-    public static function FlagSnapshotsEnabled()
-    {
+    public static function FlagSnapshotsEnabled() {
         if (defined('FLAG_SNAPSHOTS_ENABLED') && FLAG_SNAPSHOTS_ENABLED) {
             return true;
         }
@@ -227,8 +225,7 @@ 
     /**
      * @return ArrayList
      */
-    public static function get_support_links()
-    {
+    public static function get_support_links() {
         $supportLinks = self::config()->support_links;
         if ($supportLinks) {
             return new ArrayList($supportLinks);
@@ -238,8 +235,7 @@ 
     /**
      * @return array
      */
-    public static function get_template_global_variables()
-    {
+    public static function get_template_global_variables() {
         return array(
             'RedisUnavailable' => 'RedisUnavailable',
             'RedisWorkersCount' => 'RedisWorkersCount',
@@ -250,8 +246,7 @@ 
 
     /**
      */
-    public function init()
-    {
+    public function init() {
         parent::init();
 
         if (!Member::currentUser() && !Session::get('AutoLoginHash')) {
@@ -267,8 +262,7 @@ 
     /**
      * @return string
      */
-    public function Link()
-    {
+    public function Link() {
         return "naut/";
     }
 
@@ -278,8 +272,7 @@ 
      * @param SS_HTTPRequest $request
      * @return \SS_HTTPResponse
      */
-    public function index(SS_HTTPRequest $request)
-    {
+    public function index(SS_HTTPRequest $request) {
         return $this->redirect($this->Link() . 'projects/');
     }
 
@@ -289,8 +282,7 @@ 
      * @param SS_HTTPRequest $request
      * @return string - HTML
      */
-    public function projects(SS_HTTPRequest $request)
-    {
+    public function projects(SS_HTTPRequest $request) {
         // Performs canView permission check by limiting visible projects in DNProjectsList() call.
         return $this->customise(array(
             'Title' => 'Projects',
@@ -301,8 +293,7 @@ 
      * @param SS_HTTPRequest $request
      * @return HTMLText
      */
-    public function nav(SS_HTTPRequest $request)
-    {
+    public function nav(SS_HTTPRequest $request) {
         return $this->renderWith('Nav');
     }
 
@@ -310,8 +301,7 @@ 
      * Return a link to the navigation template used for AJAX requests.
      * @return string
      */
-    public function NavLink()
-    {
+    public function NavLink() {
         $currentProject = $this->getCurrentProject();
         $projectName = $currentProject ? $currentProject->Name : null;
         return Controller::join_links(Director::absoluteBaseURL(), 'naut', 'nav', $projectName);
@@ -323,8 +313,7 @@ 
      * @param SS_HTTPRequest $request
      * @return SS_HTTPResponse - HTML
      */
-    public function snapshots(SS_HTTPRequest $request)
-    {
+    public function snapshots(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
         return $this->getCustomisedViewSection('SnapshotsSection', 'Data Snapshots');
     }
@@ -335,8 +324,7 @@ 
      * @param SS_HTTPRequest $request
      * @return string - HTML
      */
-    public function createsnapshot(SS_HTTPRequest $request)
-    {
+    public function createsnapshot(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         // Performs canView permission check by limiting visible projects
@@ -362,8 +350,7 @@ 
      * @param SS_HTTPRequest $request
      * @return string - HTML
      */
-    public function uploadsnapshot(SS_HTTPRequest $request)
-    {
+    public function uploadsnapshot(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         // Performs canView permission check by limiting visible projects
@@ -387,8 +374,7 @@ 
      * Return the upload limit for snapshot uploads
      * @return string
      */
-    public function UploadLimit()
-    {
+    public function UploadLimit() {
         return File::format_size(min(
             File::ini2bytes(ini_get('upload_max_filesize')),
             File::ini2bytes(ini_get('post_max_size'))
@@ -401,8 +387,7 @@ 
      * @param SS_HTTPRequest $request
      * @return Form
      */
-    public function getUploadSnapshotForm(SS_HTTPRequest $request)
-    {
+    public function getUploadSnapshotForm(SS_HTTPRequest $request) {
         // Performs canView permission check by limiting visible projects
         $project = $this->getCurrentProject();
         if (!$project) {
@@ -459,8 +444,7 @@ 
      *
      * @return bool|HTMLText|SS_HTTPResponse
      */
-    public function doUploadSnapshot($data, Form $form)
-    {
+    public function doUploadSnapshot($data, Form $form) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         // Performs canView permission check by limiting visible projects
@@ -559,8 +543,7 @@ 
      * @param SS_HTTPRequest $request
      * @return Form
      */
-    public function getPostSnapshotForm(SS_HTTPRequest $request)
-    {
+    public function getPostSnapshotForm(SS_HTTPRequest $request) {
         // Performs canView permission check by limiting visible projects
         $project = $this->getCurrentProject();
         if (!$project) {
@@ -611,8 +594,7 @@ 
      *
      * @return SS_HTTPResponse
      */
-    public function doPostSnapshot($data, $form)
-    {
+    public function doPostSnapshot($data, $form) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         $project = $this->getCurrentProject();
@@ -649,8 +631,7 @@ 
      * @param SS_HTTPRequest $request
      * @return SS_HTTPResponse - HTML
      */
-    public function snapshotslog(SS_HTTPRequest $request)
-    {
+    public function snapshotslog(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
         return $this->getCustomisedViewSection('SnapshotsSection', 'Data Snapshots Log');
     }
@@ -660,8 +641,7 @@ 
      * @return SS_HTTPResponse|string
      * @throws SS_HTTPResponse_Exception
      */
-    public function postsnapshotsuccess(SS_HTTPRequest $request)
-    {
+    public function postsnapshotsuccess(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         // Performs canView permission check by limiting visible projects
@@ -695,8 +675,7 @@ 
      * @param SS_HTTPRequest $request
      * @return \SS_HTTPResponse
      */
-    public function project(SS_HTTPRequest $request)
-    {
+    public function project(SS_HTTPRequest $request) {
         return $this->getCustomisedViewSection('ProjectOverview', '', array('IsAdmin' => Permission::check('ADMIN')));
     }
 
@@ -707,8 +686,7 @@ 
      *
      * @return SS_HTTPResponse
      */
-    public function toggleprojectstar(SS_HTTPRequest $request)
-    {
+    public function toggleprojectstar(SS_HTTPRequest $request) {
         $project = $this->getCurrentProject();
         if (!$project) {
             return $this->project404Response();
@@ -734,8 +712,7 @@ 
      * @param SS_HTTPRequest $request
      * @return \SS_HTTPResponse
      */
-    public function branch(SS_HTTPRequest $request)
-    {
+    public function branch(SS_HTTPRequest $request) {
         $project = $this->getCurrentProject();
         if (!$project) {
             return $this->project404Response();
@@ -756,8 +733,7 @@ 
      * @param SS_HTTPRequest $request
      * @return \SS_HTTPResponse
      */
-    public function environment(SS_HTTPRequest $request)
-    {
+    public function environment(SS_HTTPRequest $request) {
         // Performs canView permission check by limiting visible projects
         $project = $this->getCurrentProject();
         if (!$project) {
@@ -785,8 +761,7 @@ 
      *
      * @return SS_HTTPResponse
      */
-    public function doDryRun($data, DeployForm $form)
-    {
+    public function doDryRun($data, DeployForm $form) {
         return $this->beginPipeline($data, $form, true);
     }
 
@@ -797,8 +772,7 @@ 
      * @param DeployForm $form
      * @return \SS_HTTPResponse
      */
-    public function startPipeline($data, $form)
-    {
+    public function startPipeline($data, $form) {
         return $this->beginPipeline($data, $form);
     }
 
@@ -810,8 +784,7 @@ 
      * @param bool $isDryRun
      * @return \SS_HTTPResponse
      */
-    protected function beginPipeline($data, DeployForm $form, $isDryRun = false)
-    {
+    protected function beginPipeline($data, DeployForm $form, $isDryRun = false) {
         $buildName = $form->getSelectedBuild($data);
 
         // Performs canView permission check by limiting visible projects
@@ -851,8 +824,7 @@ 
      * @return SS_HTTPResponse
      * @throws SS_HTTPResponse_Exception
      */
-    public function pipeline(SS_HTTPRequest $request)
-    {
+    public function pipeline(SS_HTTPRequest $request) {
         $params = $request->params();
         $pipeline = Pipeline::get()->byID($params['Identifier']);
 
@@ -883,8 +855,7 @@ 
      * @param SS_HTTPRequest $request
      * @return string
      */
-    public function createenv(SS_HTTPRequest $request)
-    {
+    public function createenv(SS_HTTPRequest $request) {
         $params = $request->params();
         if ($params['Identifier']) {
             $record = DNCreateEnvironment::get()->byId($params['Identifier']);
@@ -913,8 +884,7 @@ 
     }
 
 
-    public function createenvlog(SS_HTTPRequest $request)
-    {
+    public function createenvlog(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         $params = $request->params();
@@ -947,8 +917,7 @@ 
      * @param SS_HTTPRequest $request
      * @return Form
      */
-    public function getCreateEnvironmentForm(SS_HTTPRequest $request)
-    {
+    public function getCreateEnvironmentForm(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_ENVIRONMENTS);
 
         $project = $this->getCurrentProject();
@@ -999,8 +968,7 @@ 
      *
      * @return bool|HTMLText|SS_HTTPResponse
      */
-    public function doCreateEnvironment($data, Form $form)
-    {
+    public function doCreateEnvironment($data, Form $form) {
         $this->setCurrentActionType(self::ACTION_ENVIRONMENTS);
 
         $project = $this->getCurrentProject();
@@ -1030,8 +998,7 @@ 
      * @param SS_HTTPRequest $request
      * @return \SS_HTTPResponse
      */
-    public function metrics(SS_HTTPRequest $request)
-    {
+    public function metrics(SS_HTTPRequest $request) {
         // Performs canView permission check by limiting visible projects
         $project = $this->getCurrentProject();
         if (!$project) {
@@ -1052,8 +1019,7 @@ 
      *
      * @return DNData
      */
-    public function DNData()
-    {
+    public function DNData() {
         return DNData::inst();
     }
 
@@ -1062,8 +1028,7 @@ 
      *
      * @return SS_List
      */
-    public function DNProjectList()
-    {
+    public function DNProjectList() {
         $memberId = Member::currentUserID();
         if (!$memberId) {
             return new ArrayList();
@@ -1081,8 +1046,7 @@ 
     /**
      * @return ArrayList
      */
-    public function getPlatformSpecificStrings()
-    {
+    public function getPlatformSpecificStrings() {
         $strings = $this->config()->platform_specific_strings;
         if ($strings) {
             return new ArrayList($strings);
@@ -1094,8 +1058,7 @@ 
      *
      * @return SS_List
      */
-    public function getStarredProjects()
-    {
+    public function getStarredProjects() {
         $member = Member::currentUser();
         if ($member === null) {
             return new ArrayList();
@@ -1119,8 +1082,7 @@ 
      *
      * @return ArrayList
      */
-    public function Navigation($limit = 5)
-    {
+    public function Navigation($limit = 5) {
         $navigation = new ArrayList();
 
         $currentProject = $this->getCurrentProject();
@@ -1173,8 +1135,7 @@ 
      *
      * @return Form
      */
-    public function getDeployForm($request = null)
-    {
+    public function getDeployForm($request = null) {
 
         // Performs canView permission check by limiting visible projects
         $project = $this->getCurrentProject();
@@ -1220,8 +1181,7 @@ 
      *
      * @return SS_HTTPResponse|string
      */
-    public function gitRevisions(SS_HTTPRequest $request)
-    {
+    public function gitRevisions(SS_HTTPRequest $request) {
 
         // Performs canView permission check by limiting visible projects
         $project = $this->getCurrentProject();
@@ -1372,8 +1332,7 @@ 
      *
      * @return bool
      */
-    protected function checkCsrfToken(SS_HTTPRequest $request, $resetToken = true)
-    {
+    protected function checkCsrfToken(SS_HTTPRequest $request, $resetToken = true) {
         $token = SecurityToken::inst();
 
         // Ensure the submitted token has a value
@@ -1399,8 +1358,7 @@ 
      *
      * @return string
      */
-    public function deploySummary(SS_HTTPRequest $request)
-    {
+    public function deploySummary(SS_HTTPRequest $request) {
 
         // Performs canView permission check by limiting visible projects
         $project = $this->getCurrentProject();
@@ -1458,8 +1416,7 @@ 
      * @throws ValidationException
      * @throws null
      */
-    public function startDeploy(SS_HTTPRequest $request)
-    {
+    public function startDeploy(SS_HTTPRequest $request) {
 
         // Ensure the CSRF Token is correct
         if (!$this->checkCsrfToken($request)) {
@@ -1502,8 +1459,7 @@ 
      * @return SS_HTTPResponse|string
      * @throws SS_HTTPResponse_Exception
      */
-    public function deploy(SS_HTTPRequest $request)
-    {
+    public function deploy(SS_HTTPRequest $request) {
         $params = $request->params();
         $deployment = DNDeployment::get()->byId($params['Identifier']);
 
@@ -1538,8 +1494,7 @@ 
      * @return string
      * @throws SS_HTTPResponse_Exception
      */
-    public function deploylog(SS_HTTPRequest $request)
-    {
+    public function deploylog(SS_HTTPRequest $request) {
         $params = $request->params();
         $deployment = DNDeployment::get()->byId($params['Identifier']);
 
@@ -1575,8 +1530,7 @@ 
      *
      * @return Form
      */
-    public function getDataTransferForm(SS_HTTPRequest $request = null)
-    {
+    public function getDataTransferForm(SS_HTTPRequest $request = null) {
         // Performs canView permission check by limiting visible projects
         $envs = $this->getCurrentProject()->DNEnvironmentList()->filterByCallback(function ($item) {
             return $item->canBackup();
@@ -1612,8 +1566,7 @@ 
      * @return SS_HTTPResponse
      * @throws SS_HTTPResponse_Exception
      */
-    public function doDataTransfer($data, Form $form)
-    {
+    public function doDataTransfer($data, Form $form) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         // Performs canView permission check by limiting visible projects
@@ -1686,8 +1639,7 @@ 
      * @return SS_HTTPResponse|string
      * @throws SS_HTTPResponse_Exception
      */
-    public function transfer(SS_HTTPRequest $request)
-    {
+    public function transfer(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         $params = $request->params();
@@ -1721,8 +1673,7 @@ 
      * @return string
      * @throws SS_HTTPResponse_Exception
      */
-    public function transferlog(SS_HTTPRequest $request)
-    {
+    public function transferlog(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         $params = $request->params();
@@ -1761,8 +1712,7 @@ 
      *                            otherwise the state is inferred from the request data.
      * @return Form
      */
-    public function getDataTransferRestoreForm(SS_HTTPRequest $request, DNDataArchive $dataArchive = null)
-    {
+    public function getDataTransferRestoreForm(SS_HTTPRequest $request, DNDataArchive $dataArchive = null) {
         $dataArchive = $dataArchive ? $dataArchive : DNDataArchive::get()->byId($request->requestVar('DataArchiveID'));
 
         // Performs canView permission check by limiting visible projects
@@ -1821,8 +1771,7 @@ 
      * @return HTMLText
      * @throws SS_HTTPResponse_Exception
      */
-    public function restoresnapshot(SS_HTTPRequest $request)
-    {
+    public function restoresnapshot(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         /** @var DNDataArchive $dataArchive */
@@ -1854,8 +1803,7 @@ 
      * @return HTMLText
      * @throws SS_HTTPResponse_Exception
      */
-    public function deletesnapshot(SS_HTTPRequest $request)
-    {
+    public function deletesnapshot(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         /** @var DNDataArchive $dataArchive */
@@ -1881,8 +1829,7 @@ 
      *        from the request data.
      * @return Form
      */
-    public function getDeleteForm(SS_HTTPRequest $request, DNDataArchive $dataArchive = null)
-    {
+    public function getDeleteForm(SS_HTTPRequest $request, DNDataArchive $dataArchive = null) {
         $dataArchive = $dataArchive ? $dataArchive : DNDataArchive::get()->byId($request->requestVar('DataArchiveID'));
 
         // Performs canView permission check by limiting visible projects
@@ -1919,8 +1866,7 @@ 
      * @return bool|SS_HTTPResponse
      * @throws SS_HTTPResponse_Exception
      */
-    public function doDelete($data, Form $form)
-    {
+    public function doDelete($data, Form $form) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         // Performs canView permission check by limiting visible projects
@@ -1959,8 +1905,7 @@ 
      * @return HTMLText
      * @throws SS_HTTPResponse_Exception
      */
-    public function movesnapshot(SS_HTTPRequest $request)
-    {
+    public function movesnapshot(SS_HTTPRequest $request) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         /** @var DNDataArchive $dataArchive */
@@ -1989,8 +1934,7 @@ 
      *
      * @return Form|SS_HTTPResponse
      */
-    public function getMoveForm(SS_HTTPRequest $request, DNDataArchive $dataArchive = null)
-    {
+    public function getMoveForm(SS_HTTPRequest $request, DNDataArchive $dataArchive = null) {
         $dataArchive = $dataArchive ? $dataArchive : DNDataArchive::get()->byId($request->requestVar('DataArchiveID'));
 
         $envs = $dataArchive->validTargetEnvironments();
@@ -2030,8 +1974,7 @@ 
      * @throws ValidationException
      * @throws null
      */
-    public function doMove($data, Form $form)
-    {
+    public function doMove($data, Form $form) {
         $this->setCurrentActionType(self::ACTION_SNAPSHOT);
 
         // Performs canView permission check by limiting visible projects
@@ -2069,8 +2012,7 @@ 
      *
      * @return string
      */
-    public static function RedisUnavailable()
-    {
+    public static function RedisUnavailable() {
         try {
             Resque::queues();
         } catch (Exception $e) {
@@ -2084,16 +2026,14 @@ 
      *
      * @return int
      */
-    public static function RedisWorkersCount()
-    {
+    public static function RedisWorkersCount() {
         return count(Resque_Worker::all());
     }
 
     /**
      * @return array
      */
-    public function providePermissions()
-    {
+    public function providePermissions() {
         return array(
             self::DEPLOYNAUT_BYPASS_PIPELINE => array(
                 'name' => "Bypass Pipeline",
@@ -2137,8 +2077,7 @@ 
     /**
      * @return DNProject|null
      */
-    public function getCurrentProject()
-    {
+    public function getCurrentProject() {
         $projectName = trim($this->getRequest()->param('Project'));
         if (!$projectName) {
             return null;
@@ -2153,8 +2092,7 @@ 
      * @param DNProject|null $project
      * @return DNEnvironment|null
      */
-    public function getCurrentEnvironment(DNProject $project = null)
-    {
+    public function getCurrentEnvironment(DNProject $project = null) {
         if ($this->getRequest()->param('Environment') === null) {
             return null;
         }
@@ -2177,8 +2115,7 @@ 
      *
      * @return string - one of the consts from self::$action_types
      */
-    public function getCurrentActionType()
-    {
+    public function getCurrentActionType() {
         return $this->actionType;
     }
 
@@ -2187,8 +2124,7 @@ 
      *
      * @param string $actionType string - one of the consts from self::$action_types
      */
-    public function setCurrentActionType($actionType)
-    {
+    public function setCurrentActionType($actionType) {
         $this->actionType = $actionType;
     }
 
@@ -2202,8 +2138,7 @@ 
      * @param Member|null $member The {@link Member} to check (or null to check the currently logged in Member)
      * @return boolean|null true if $member has access to upload or download to at least one {@link DNEnvironment}.
      */
-    public function CanViewArchives(Member $member = null)
-    {
+    public function CanViewArchives(Member $member = null) {
         if ($member === null) {
             $member = Member::currentUser();
         }
@@ -2239,8 +2174,7 @@ 
      *
      * @return PaginatedList
      */
-    public function CreateEnvironmentList()
-    {
+    public function CreateEnvironmentList() {
         $project = $this->getCurrentProject();
         if ($project) {
             return new PaginatedList($project->CreateEnvironments()->sort("Created DESC"), $this->request);
@@ -2253,8 +2187,7 @@ 
      *
      * @return PaginatedList
      */
-    public function CompleteDataArchives()
-    {
+    public function CompleteDataArchives() {
         $project = $this->getCurrentProject();
         $archives = new ArrayList();
 
@@ -2273,8 +2206,7 @@ 
      * @return PaginatedList The list of "pending" data archives which are waiting for a file
      * to be delivered offline by post, and manually uploaded into the system.
      */
-    public function PendingDataArchives()
-    {
+    public function PendingDataArchives() {
         $project = $this->getCurrentProject();
         $archives = new ArrayList();
         foreach ($project->DNEnvironmentList() as $env) {
@@ -2290,8 +2222,7 @@ 
     /**
      * @return PaginatedList
      */
-    public function DataTransferLogs()
-    {
+    public function DataTransferLogs() {
         $project = $this->getCurrentProject();
 
         $transfers = DNDataTransfer::get()->filterByCallback(function ($record) use ($project) {
@@ -2311,8 +2242,7 @@ 
     /**
      * @return null|PaginatedList
      */
-    public function DeployHistory()
-    {
+    public function DeployHistory() {
         if ($env = $this->getCurrentEnvironment()) {
             $history = $env->DeployHistory();
             if ($history->count() > 0) {
@@ -2327,8 +2257,7 @@ 
     /**
      * @return SS_HTTPResponse
      */
-    protected function project404Response()
-    {
+    protected function project404Response() {
         return new SS_HTTPResponse(
             "Project '" . Convert::raw2xml($this->getRequest()->param('Project')) . "' not found.",
             404
@@ -2338,8 +2267,7 @@ 
     /**
      * @return SS_HTTPResponse
      */
-    protected function environment404Response()
-    {
+    protected function environment404Response() {
         $envName = Convert::raw2xml($this->getRequest()->param('Environment'));
         return new SS_HTTPResponse("Environment '" . $envName . "' not found.", 404);
     }
@@ -2350,8 +2278,7 @@ 
      *
      * @return string
      */
-    protected function sendResponse($status, $content)
-    {
+    protected function sendResponse($status, $content) {
         // strip excessive newlines
         $content = preg_replace('/(?:(?:\r\n|\r|\n)\s*){2}/s', "\n", $content);
 
@@ -2374,8 +2301,7 @@ 
      *
      * @param string $mode
      */
-    protected function validateSnapshotMode($mode)
-    {
+    protected function validateSnapshotMode($mode) {
         if (!in_array($mode, array('all', 'assets', 'db'))) {
             throw new LogicException('Invalid mode');
         }
@@ -2387,8 +2313,7 @@ 
      *
      * @return SS_HTTPResponse
      */
-    protected function getCustomisedViewSection($sectionName, $title = '', $data = array())
-    {
+    protected function getCustomisedViewSection($sectionName, $title = '', $data = array()) {
         // Performs canView permission check by limiting visible projects
         $project = $this->getCurrentProject();
         if (!$project) {
@@ -2408,8 +2333,7 @@ 
      *
      * @return ArrayList
      */
-    public function AmbientMenu()
-    {
+    public function AmbientMenu() {
         $list = new ArrayList();
 
         if (Member::currentUserID()) {
@@ -2432,8 +2356,7 @@ 
      *
      * @return SS_HTTPResponse
      */
-    public function createproject(SS_HTTPRequest $request)
-    {
+    public function createproject(SS_HTTPRequest $request) {
         if ($this->canCreateProjects()) {
             return $this->render(['CurrentTitle' => 'Create Stack']);
         }
@@ -2445,8 +2368,7 @@ 
      *
      * @return bool
      */
-    public function canCreateProjects($member = null)
-    {
+    public function canCreateProjects($member = null) {
         if (!$member) {
             $member = Member::currentUser();
         }
@@ -2460,8 +2382,7 @@ 
     /**
      * @return Form
      */
-    public function CreateProjectForm()
-    {
+    public function CreateProjectForm() {
         $form = Form::create(
             $this,
             __FUNCTION__,
@@ -2476,8 +2397,7 @@ 
     /**
      * @return FieldList
      */
-    protected function getCreateProjectFormFields()
-    {
+    protected function getCreateProjectFormFields() {
         $fields = FieldList::create();
         $fields->merge([
             TextField::create('Name', 'Title')->setDescription('Limited to alphanumeric characters, underscores and hyphens.'),
@@ -2490,8 +2410,7 @@ 
     /**
      * @return FieldList
      */
-    protected function getCreateProjectFormActions()
-    {
+    protected function getCreateProjectFormActions() {
         $fields = FieldList::create(
             FormAction::create('doCreateProject', 'Create Stack')
         );
@@ -2507,8 +2426,7 @@ 
      *
      * @return SS_HTTPResponse
      */
-    public function doCreateProject($data, $form)
-    {
+    public function doCreateProject($data, $form) {
         $form->loadDataFrom($data);
         $project = DNProject::create();
 
@@ -2540,8 +2458,7 @@ 
      *
      * @return SS_HTTPResponse
      */
-    public function createprojectprogress(SS_HTTPRequest $request)
-    {
+    public function createprojectprogress(SS_HTTPRequest $request) {
         $project = $this->getCurrentProject();
         if (!$project) {
             return $this->httpError(404);
@@ -2591,8 +2508,7 @@ 
         return $response;
     }
 
-    public function checkrepoaccess(SS_HTTPRequest $request)
-    {
+    public function checkrepoaccess(SS_HTTPRequest $request) {
         $project = $this->getCurrentProject();
         if (!$project) {
             return $this->httpError(404);

code/api/DeploynautAPI.php

Braces +9 added lines, -18 removed lines

@@ -13,8 +13,7 @@ 
  * DeploynautAPI > APIProject > APIEnvironment
  *
  */
-class DeploynautAPI extends APINoun
-{
+class DeploynautAPI extends APINoun {
 
     /**
      * Default URL handlers - (Action)/(ID)//(OtherID)
@@ -46,8 +45,7 @@ 
      * @param SS_HTTPRequest $request
      * @return SS_HTTPResponse
      */
-    public function listProjects(SS_HTTPRequest $request)
-    {
+    public function listProjects(SS_HTTPRequest $request) {
         $response = array(
             'href' => Director::absoluteURL($this->Link()),
             'projects' => array(),
@@ -75,8 +73,7 @@ 
      * @param SS_HTTPRequest $request
      * @return APIProject
      */
-    public function project(SS_HTTPRequest $request)
-    {
+    public function project(SS_HTTPRequest $request) {
         $project = $this->getProject();
         if (!$project) {
             return $this->project404Response();
@@ -90,8 +87,7 @@ 
      * @param SS_HTTPRequest $request
      * @return APIEnvironment
      */
-    public function environment(SS_HTTPRequest $request)
-    {
+    public function environment(SS_HTTPRequest $request) {
         $project = $this->getProject();
         if (!$project) {
             return $this->project404Response();
@@ -107,8 +103,7 @@ 
     /**
      * @return SS_HTTPResponse
      */
-    protected function project404Response()
-    {
+    protected function project404Response() {
         $projectName = Convert::raw2xml($this->getRequest()->latestParam('Project'));
         return new SS_HTTPResponse('Project "' . $projectName . '" not found.', 404);
     }
@@ -116,8 +111,7 @@ 
     /**
      * @return SS_HTTPResponse
      */
-    protected function environment404Response()
-    {
+    protected function environment404Response() {
         $envName = Convert::raw2xml($this->getRequest()->latestParam('Environment'));
         return new SS_HTTPResponse('Environment "' . $envName . '" not found.', 404);
     }
@@ -127,8 +121,7 @@ 
      *
      * @return DNProject
      */
-    protected function getProject()
-    {
+    protected function getProject() {
         $projectName = $this->getRequest()->param('Project');
         return DNProject::get()->filter('Name', $projectName)->first();
     }
@@ -138,8 +131,7 @@ 
      *
      * @return DNEnvironment
      */
-    protected function getEnvironment()
-    {
+    protected function getEnvironment() {
         $projectName = $this->getRequest()->param('Project');
         $project = DNProject::get()->filter('Name', $projectName)->first();
         $environmentName = $this->getRequest()->param('Environment');
@@ -149,8 +141,7 @@ 
     /**
      * @return string
      */
-    public function Link()
-    {
+    public function Link() {
         return 'naut/api';
     }
 }

code/api/nouns/APIEnvironment.php

Braces +10 added lines, -20 removed lines

@@ -1,7 +1,6 @@ 
 <?php
 
-class APIEnvironment extends APINoun
-{
+class APIEnvironment extends APINoun {
 
     /**
      * @var array
@@ -15,8 +14,7 @@ 
      * @param SS_HTTPRequest $request
      * @return SS_HTTPResponse
      */
-    public function index(SS_HTTPRequest $request)
-    {
+    public function index(SS_HTTPRequest $request) {
         if (!$this->record->canView($this->getMember())) {
             return $this->message('You are not authorized to view this environment', 403);
         }
@@ -52,8 +50,7 @@ 
      * @param SS_HTTPRequest $request
      * @return SS_HTTPResponse
      */
-    public function ping(SS_HTTPRequest $request)
-    {
+    public function ping(SS_HTTPRequest $request) {
         if (!$this->record->canView($this->getMember())) {
             return $this->message('You are not authorized to do that on this environment', 403);
         }
@@ -71,8 +68,7 @@ 
      * @param SS_HTTPRequest $request
      * @return SS_HTTPResponse
      */
-    public function deploy(SS_HTTPRequest $request)
-    {
+    public function deploy(SS_HTTPRequest $request) {
         if (!$this->record->canView($this->getMember())) {
             return $this->message('You are not authorized to do that on this environment', 403);
         }
@@ -89,8 +85,7 @@ 
     /**
      * @return string
      */
-    public function Link()
-    {
+    public function Link() {
         return Controller::join_links(
             $this->parent->Link(),
             $this->record->Project()->Name,
@@ -101,16 +96,14 @@ 
     /**
      * @return SS_HTTPResponse
      */
-    protected function showRecord()
-    {
+    protected function showRecord() {
         return $this->getAPIResponse($this->record->toMap());
     }
 
     /**
      * @return SS_HTTPResponse
      */
-    protected function createPing()
-    {
+    protected function createPing() {
         if (!$this->record->canDeploy($this->getMember())) {
             return $this->message('You are not authorized to do that on this environment', 403);
         }
@@ -135,8 +128,7 @@ 
      * @param int $ID
      * @return SS_HTTPResponse
      */
-    protected function getPing($ID)
-    {
+    protected function getPing($ID) {
         $ping = DNPing::get()->byID($ID);
         if (!$ping) {
             return $this->message('Ping not found', 404);
@@ -152,8 +144,7 @@ 
     /**
      * @return SS_HTTPResponse
      */
-    protected function createDeploy()
-    {
+    protected function createDeploy() {
         if (!$this->record->canDeploy($this->getMember())) {
             return $this->message('You are not authorized to do that on this environment', 403);
         }
@@ -188,8 +179,7 @@ 
      * @param int $id
      * @return SS_HTTPResponse
      */
-    protected function getDeploy($id)
-    {
+    protected function getDeploy($id) {
         $deploy = DNDeployment::get()->byID($id);
         if (!$deploy) {
             return $this->message('Deploy not found', 404);

code/api/nouns/APINoun.php

Braces +10 added lines, -20 removed lines

@@ -2,8 +2,7 @@ 
 /**
  * Base class for the APINouns
  */
-class APINoun extends Controller
-{
+class APINoun extends Controller {
 
     /**
      * @var controller
@@ -26,8 +25,7 @@ 
      * @param Controller|null $parent
      * @param DataObject|null $record
      */
-    public function __construct(\Controller $parent = null, DataObject $record = null)
-    {
+    public function __construct(\Controller $parent = null, DataObject $record = null) {
         $this->record = $record;
         $this->parent = $parent;
         parent::__construct();
@@ -36,8 +34,7 @@ 
     /**
      * Enable basic auth on the API
      */
-    public function init()
-    {
+    public function init() {
         $this->member = BasicAuth::requireLogin('Deploynaut API');
         parent::init();
     }
@@ -45,16 +42,14 @@ 
     /**
      * @return DataObject
      */
-    public function getRecord()
-    {
+    public function getRecord() {
         return $this->record;
     }
 
     /**
      * @return Member
      */
-    public function getMember()
-    {
+    public function getMember() {
         return $this->member;
     }
 
@@ -62,8 +57,7 @@ 
      * @param array $output
      * @return SS_HTTPResponse
      */
-    protected function getAPIResponse($output)
-    {
+    protected function getAPIResponse($output) {
         $response = $this->getResponse();
         if ($this->respondWithText()) {
             $body = print_r($output, true);
@@ -79,8 +73,7 @@ 
     /**
      * @return boolean
      */
-    protected function respondWithJSON()
-    {
+    protected function respondWithJSON() {
         if ($this->getRequest()->getExtension() == 'json') {
             return true;
         }
@@ -93,8 +86,7 @@ 
     /**
      * @return boolean
      */
-    protected function respondWithText()
-    {
+    protected function respondWithText() {
         if ($this->getRequest()->getExtension() == 'txt') {
             return true;
         }
@@ -107,8 +99,7 @@ 
     /**
      * @return array|null
      */
-    protected function getRequestBody()
-    {
+    protected function getRequestBody() {
         return Convert::json2array($this->getRequest()->getBody());
     }
 
@@ -119,8 +110,7 @@ 
      * @param int $statusCode
      * @return SS_HTTPResponse
      */
-    protected function message($message, $statusCode)
-    {
+    protected function message($message, $statusCode) {
         $response = $this->getAPIResponse(array(
             'message' => $message,
             'statusCode' => $statusCode

code/api/nouns/APIProject.php

Braces +6 added lines, -12 removed lines

@@ -1,7 +1,6 @@ 
 <?php
 
-class APIProject extends APINoun
-{
+class APIProject extends APINoun {
 
     /**
      * @var array
@@ -15,8 +14,7 @@ 
      * @param SS_HTTPRequest $request
      * @return SS_HTTPResponse
      */
-    public function index(SS_HTTPRequest $request)
-    {
+    public function index(SS_HTTPRequest $request) {
         if (!$this->record->canView($this->getMember())) {
             return $this->message('You are not authorized to this environment', 403);
         }
@@ -48,8 +46,7 @@ 
      * @param SS_HTTPRequest $request
      * @return SS_HTTPResponse
      */
-    public function fetch(SS_HTTPRequest $request)
-    {
+    public function fetch(SS_HTTPRequest $request) {
         if (!$this->record->canView($this->getMember())) {
             return $this->message('You are not authorized to do that on this environment', 403);
         }
@@ -67,8 +64,7 @@ 
      * @param int $ID
      * @return SS_HTTPResponse
      */
-    protected function getFetch($ID)
-    {
+    protected function getFetch($ID) {
         $ping = DNGitFetch::get()->byID($ID);
         if (!$ping) {
             return $this->message('Fetch not found', 404);
@@ -84,8 +80,7 @@ 
     /**
      * @return SS_HTTPResponse
      */
-    protected function createFetch()
-    {
+    protected function createFetch() {
         /** @var DNGitFetch $fetch */
         $fetch = DNGitFetch::create();
         $fetch->ProjectID = $this->record->ID;
@@ -107,8 +102,7 @@ 
     /**
      * @return string
      */
-    public function Link()
-    {
+    public function Link() {
         return Controller::join_links(
             $this->parent->Link(),
             $this->record->Name

code/backends/CapistranoDeploymentBackend.php

Braces +12 added lines, -24 removed lines

@@ -1,18 +1,15 @@ 
 <?php
 use \Symfony\Component\Process\Process;
 
-class CapistranoDeploymentBackend extends Object implements DeploymentBackend
-{
+class CapistranoDeploymentBackend extends Object implements DeploymentBackend {
 
     protected $packageGenerator;
 
-    public function getPackageGenerator()
-    {
+    public function getPackageGenerator() {
         return $this->packageGenerator;
     }
 
-    public function setPackageGenerator(PackageGenerator $packageGenerator)
-    {
+    public function setPackageGenerator(PackageGenerator $packageGenerator) {
         $this->packageGenerator = $packageGenerator;
     }
 
@@ -24,8 +21,7 @@ 
      *
      * @return DeploymentStrategy
      */
-    public function planDeploy(DNEnvironment $environment, $options)
-    {
+    public function planDeploy(DNEnvironment $environment, $options) {
         $strategy = new DeploymentStrategy($environment, $options);
 
         $currentBuild = $environment->CurrentBuild();
@@ -131,8 +127,7 @@ 
     /**
      * Enable a maintenance page for the given environment using the maintenance:enable Capistrano task.
      */
-    public function enableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project)
-    {
+    public function enableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project) {
         $name = $environment->getFullName();
         $command = $this->getCommand('maintenance:enable', 'web', $environment, null, $log);
         $command->run(function ($type, $buffer) use ($log) {
@@ -148,8 +143,7 @@ 
     /**
      * Disable the maintenance page for the given environment using the maintenance:disable Capistrano task.
      */
-    public function disableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project)
-    {
+    public function disableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project) {
         $name = $environment->getFullName();
         $command = $this->getCommand('maintenance:disable', 'web', $environment, null, $log);
         $command->run(function ($type, $buffer) use ($log) {
@@ -165,8 +159,7 @@ 
     /**
      * Check the status using the deploy:check capistrano method
      */
-    public function ping(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project)
-    {
+    public function ping(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project) {
         $command = $this->getCommand('deploy:check', 'web', $environment, null, $log);
         $command->run(function ($type, $buffer) use ($log) {
             $log->write($buffer);
@@ -177,8 +170,7 @@ 
     /**
      * @inheritdoc
      */
-    public function dataTransfer(DNDataTransfer $dataTransfer, DeploynautLogFile $log)
-    {
+    public function dataTransfer(DNDataTransfer $dataTransfer, DeploynautLogFile $log) {
         if ($dataTransfer->Direction == 'get') {
             $this->dataTransferBackup($dataTransfer, $log);
         } else {
@@ -222,8 +214,7 @@ 
      * @param DeploynautLogFile $log
      * @return \Symfony\Component\Process\Process
      */
-    public function getCommand($action, $roles, DNEnvironment $environment, $args = null, DeploynautLogFile $log)
-    {
+    public function getCommand($action, $roles, DNEnvironment $environment, $args = null, DeploynautLogFile $log) {
         $name = $environment->getFullName();
         $env = $environment->Project()->getProcessEnv();
 
@@ -277,8 +268,7 @@ 
      * @param DNDataTransfer    $dataTransfer
      * @param DeploynautLogFile $log
      */
-    protected function dataTransferBackup(DNDataTransfer $dataTransfer, DeploynautLogFile $log)
-    {
+    protected function dataTransferBackup(DNDataTransfer $dataTransfer, DeploynautLogFile $log) {
         $environment = $dataTransfer->Environment();
         $name = $environment->getFullName();
 
@@ -354,8 +344,7 @@ 
      * Also cleans up and generates new error pages.
      * @param DeploynautLogFile $log
      */
-    public function rebuild(DNEnvironment $environment, $log)
-    {
+    public function rebuild(DNEnvironment $environment, $log) {
         $name = $environment->getFullName();
         $command = $this->getCommand('deploy:migrate', 'web', $environment, null, $log);
         $command->run(function ($type, $buffer) use ($log) {
@@ -376,8 +365,7 @@ 
      * @param DNDataTransfer $dataTransfer
      * @param DeploynautLogFile $log
      */
-    protected function dataTransferRestore($workingDir, DNDataTransfer $dataTransfer, DeploynautLogFile $log)
-    {
+    protected function dataTransferRestore($workingDir, DNDataTransfer $dataTransfer, DeploynautLogFile $log) {
         $environment = $dataTransfer->Environment();
         $name = $environment->getFullName();
 

code/backends/DemoDeploymentBackend.php

Braces +6 added lines, -12 removed lines

@@ -7,8 +7,7 @@ 
  *
  * It's useful for demonstrating how the system works, and how you can write deployment back-ends
  */
-class DemoDeploymentBackend extends Object implements DeploymentBackend
-{
+class DemoDeploymentBackend extends Object implements DeploymentBackend {
 
     /**
      * Create a deployment strategy.
@@ -18,8 +17,7 @@ 
      *
      * @return DeploymentStrategy
      */
-    public function planDeploy(DNEnvironment $environment, $options = array())
-    {
+    public function planDeploy(DNEnvironment $environment, $options = array()) {
         return new DeploymentStrategy($environment, $options);
     }
 
@@ -69,23 +67,19 @@ 
     /**
      * @inheritdoc
      */
-    public function dataTransfer(DNDataTransfer $dataTransfer, DeploynautLogFile $log)
-    {
+    public function dataTransfer(DNDataTransfer $dataTransfer, DeploynautLogFile $log) {
         die('Not implemented');
     }
 
-    public function enableMaintenance(DNEnvironment $environment, \DeploynautLogFile $log, DNProject $project)
-    {
+    public function enableMaintenance(DNEnvironment $environment, \DeploynautLogFile $log, DNProject $project) {
         $log->write(sprintf('Maintenance page enabled on "%s"', $environment->getFullName()));
     }
 
-    public function disableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project)
-    {
+    public function disableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project) {
         $log->write(sprintf('Maintenance page disabled on "%s"', $environment->getFullName()));
     }
 
-    public function ping(\DNEnvironment $environment, DeploynautLogFile $log, DNProject $project)
-    {
+    public function ping(\DNEnvironment $environment, DeploynautLogFile $log, DNProject $project) {
         $log->write(sprintf('Ping "%s"', $environment->getFullName()));
     }
 }

code/backends/DeploymentBackend.php

Braces +1 added lines, -2 removed lines

@@ -1,7 +1,6 @@
 <?php
 
-interface DeploymentBackend
-{
+interface DeploymentBackend {
 
     /**
      * Creates a deployment strategy that can be used to do some pre-flight checks.