silverstripe / deploynaut

code/backends/DeploymentStrategy.php

Doc Comments +2 added lines

@@ -75,6 +75,7 @@ 
 	}
 
 	/**
+	 * @param string $code
 	 */
 	public function setActionCode($code) {
 		$this->actionCode = $code;
@@ -89,6 +90,7 @@ 
 
 	/**
 	 * @param int
+	 * @param string $seconds
 	 */
 	public function setEstimatedTime($seconds) {
 		$this->estimatedTime = $seconds;

code/control/DNRoot.php

Doc Comments +1 added lines, -1 removed lines

@@ -961,7 +961,7 @@
 	 * @param array $data
 	 * @param Form $form
 	 *
-	 * @return bool|HTMLText|SS_HTTPResponse
+	 * @return SS_HTTPResponse|null
 	 */
 	public function doCreateEnvironment($data, Form $form) {
 		$this->setCurrentActionType(self::ACTION_ENVIRONMENTS);

code/control/DeployDispatcher.php

Doc Comments +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 	 *
 	 * @param \SS_HTTPRequest $request
 	 *
-	 * @return \HTMLText|\SS_HTTPResponse
+	 * @return SS_HTTPResponse|null
 	 */
 	public function index(\SS_HTTPRequest $request) {
 		return $this->redirect(\Controller::join_links($this->Link(), 'history'), 302);

code/control/GitDispatcher.php

Doc Comments +5 added lines, -5 removed lines

@@ -47,7 +47,7 @@ 
 	 *
 	 * @param \SS_HTTPRequest $request
 	 *
-	 * @return \HTMLText|\SS_HTTPResponse
+	 * @return SS_HTTPResponse|null
 	 */
 	public function index(\SS_HTTPRequest $request) {
 		return $this->redirect(\Controller::join_links($this->Link(), 'show'), 302);
@@ -72,7 +72,7 @@ 
 	/**
 	 * @param SS_HTTPRequest $request
 	 *
-	 * @return string
+	 * @return SS_HTTPResponse
 	 */
 	public function show(\SS_HTTPRequest $request) {
 
@@ -166,7 +166,7 @@ 
 	}
 
 	/**
-	 * @param $project
+	 * @param DNProject $project
 	 *
 	 * @return array
 	 */
@@ -182,7 +182,7 @@ 
 	}
 
 	/**
-	 * @param $project
+	 * @param DNProject $project
 	 *
 	 * @return array
 	 */
@@ -198,7 +198,7 @@ 
 	}
 
 	/**
-	 * @param $project
+	 * @param DNProject $project
 	 *
 	 * @return array
 	 */

code/model/jobs/DNDeployment.php

Doc Comments +3 added lines

@@ -82,6 +82,9 @@
 		'Deployer.Name' => 'Deployer'
 	);
 
+	/**
+	 * @param false|string $token
+	 */
 	public function setResqueToken($token) {
 		$this->ResqueToken = $token;
 	}

code/SS_Datetimezone.php

Indentation +2 added lines, -2 removed lines

@@ -6,8 +6,8 @@
  */
 class SS_Datetimezone extends SS_Datetime {
 	/**
-	* Returns the date in the raw SQL-format specific to a given timezone passed from the Member class, e.g. “2006-01-18 16:32:04”
-	*/
+	 * Returns the date in the raw SQL-format specific to a given timezone passed from the Member class, e.g. “2006-01-18 16:32:04”
+	 */
 	public function Nice() {
 		//instantiate new DateTime object based off received timestamp in the default timezone
 		$timestamp = new DateTime($this->value, new DateTimeZone(date_default_timezone_get()));

code/model/DNEnvironment.php

Doc Comments +1 added lines, -1 removed lines

@@ -670,7 +670,7 @@
 	/**
 	 * A list of past deployments.
 	 * @param string $orderBy - the name of a DB column to sort in descending order
-	 * @return \ArrayList
+	 * @return DataList
 	 */
 	public function DeployHistory($orderBy = '') {
 		$sort = [];

code/api/DeploynautAPIFormatter.php

Indentation -1 removed lines

@@ -117,7 +117,6 @@
 
 	/**
 	 * Return data about a particular {@link Member} of the stack for use in API response.
-
 	 * Notes:
 	 * 1) This method returns null instead of an array if the member doesn't exists anymore
 	 * 2) 'role' can be null in the response. This is the case of an admin, or an operations