silverstripe / deploynaut

code/model/DNReferenceList.php

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,6 @@ 
 	}
 
 	/**
-	 * @param int $limit - defaults to the ten lates
 	 * @return array()
 	 */
 	protected function getReferences() {
@@ -106,6 +105,7 @@ 
 
 	/**
 	 * Find a build in this set by hash.
+	 * @param string $hash
 	 */
 	public function byName($hash) {
 		if($this->loaded == false) {

code/model/DNTag.php

Doc Comments -1 removed lines

@@ -22,7 +22,6 @@
 	);
 
 	/**
-	 * @param Gitonomy\Git\Commit $commit
 	 * @param DNProject $project
 	 * @param DNData $data
 	 */

code/model/jobs/DNDataTransfer.php

Doc Comments +3 added lines

@@ -76,6 +76,9 @@
 	 */
 	protected $backupBeforePush = true;
 
+	/**
+	 * @param boolean $value
+	 */
 	public function setBackupBeforePush($value) {
 		$this->backupBeforePush = $value;
 	}

code/model/Pipeline.php

Doc Comments +6 added lines, -4 removed lines

@@ -368,7 +368,6 @@ 
 	 * Retrieve the value of a specific config setting
 	 *
 	 * @param string $setting Settings
-	 * @param string $setting,... Sub-settings
 	 * @return mixed Value of setting, or null if not set
 	 */
 	public function getConfigSetting($setting) {
@@ -773,7 +772,7 @@ 
 	 * Sends a specific message to all marked recipients, including the author of this pipeline
 	 *
 	 * @param string $messageID Message ID. One of 'Abort', 'Success', or 'Failure', or some custom message
-	 * @return boolean True if successful
+	 * @return boolean|null True if successful
 	 */
 	public function sendMessage($messageID) {
 		// Check message, subject, and additional arguments to include
@@ -922,7 +921,7 @@ 
 	/**
 	 * Finds the next {@link PipelineStep} that needs to execute. Relies on $this->CurrentStep() being a valid step.
 	 *
-	 * @return PipelineStep|null The next step in the pipeline, or null if none remain.
+	 * @return DataObject|null The next step in the pipeline, or null if none remain.
 	 */
 	protected function findNextStep() {
 		// otherwise get next step in chain
@@ -939,7 +938,7 @@ 
 	/**
 	 * Finds the previous {@link PipelineStep} that executed. Relies on $this->CurrentStep() being a valid step.
 	 *
-	 * @return PipelineStep|null The previous step in the pipeline, or null if this is the first.
+	 * @return DataObject|null The previous step in the pipeline, or null if this is the first.
 	 */
 	public function findPreviousStep() {
 		// otherwise get previous step in chain
@@ -1013,6 +1012,9 @@ 
 		return $this->getField('DryRun');
 	}
 
+	/**
+	 * @param string $action
+	 */
 	public function Link($action = null) {
 		return Controller::join_links($this->Environment()->Link(), 'pipeline', $this->ID, $action);
 	}

code/model/steps/PipelineStep.php

Doc Comments +1 added lines, -2 removed lines

@@ -119,7 +119,6 @@ 
 	 * Retrieve the value of a specific config setting
 	 *
 	 * @param string $setting Settings
-	 * @param string $setting,... Sub-settings
 	 * @return mixed Value of setting, or null if not set
 	 */
 	public function getConfigSetting($setting) {
@@ -233,7 +232,7 @@ 
 	 * Initialise the step unless it's already running (Started state).
 	 * The step will be expected to transition from here to Finished, Failed or Aborted state.
 	 *
-	 * @return boolean True if successfully started, false if error
+	 * @return boolean|null True if successfully started, false if error
 	 */
 	public function start() {
 		$this->Status = 'Started';

code/model/steps/UserConfirmationStep.php

Doc Comments +1 added lines, -1 removed lines

@@ -319,7 +319,7 @@
 	 * @param string $messageID Message ID. One of 'Reject', 'Approve', 'TimeOut' or 'Request'
 	 * @param mixed $recipientGroup Either a numeric index of the next recipient to send to, or "all" for all
 	 * This is used for delayed notification so that failover recipients can be notified.
-	 * @return boolean True if successful
+	 * @return boolean|null True if successful
 	 */
 	protected function sendMessage($messageID, $recipientGroup = 'all') {
 		// Add additionally configured arguments

code/model/DNBranch.php

Doc Comments -1 removed lines

@@ -22,7 +22,6 @@
 	);
 
 	/**
-	 * @param Gitonomy\Git\Commit $commit
 	 * @param DNProject $project
 	 * @param DNData $data
 	 */