CreateEnvJob::perform() - Code Metrics - Inspection of "Remove unncessary tear downs/failure setting in jo..." - silverstripe/deploynaut - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#520)

by Sean

created 2016-02-29 20:55 UTC

CreateEnvJob::perform() A

↳ Parent: CreateEnvJob

Complexity

Conditions	4
Paths	5

Size

Total Lines	19
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
c	3
b	0
f	0
dl	0
loc	19
rs	9.2
cc	4
eloc	12
nc	5
nop	0

<?php
class CreateEnvJob extends DeploynautJob {

	/**
	 *
	 * @var array
	 */
	public $args;

	/**
	 *
	 */
	public function setUp() {
		$this->updateStatus('Started');
		chdir(BASE_PATH);
	}

	/**
	 *
	 */
	public function perform() {
		echo "[-] CreateEnvJob starting" . PHP_EOL;
		// This is a bit icky, but there is no easy way of capturing a failed deploy by using the PHP Resque
		try {
			$envCreate = DNCreateEnvironment::get()->byId($this->args['createID']);
			if(!($envCreate && $envCreate->exists())) {
				throw new RuntimeException(sprintf('Could not find create environment record %s', $args['createID']));
			}

			// This will throw and exception if it fails.
			$envCreate->createEnvironment();

		} catch(Exception $e) {
			echo "[-] CreateEnvJob failed" . PHP_EOL;
			throw $e;
		}
		$this->updateStatus('Finished');
		echo "[-] CreateEnvJob finished" . PHP_EOL;
	}

	/**
	 *
	 * @param string $status
	 * @global array $databaseConfig
	 */
	protected function updateStatus($status) {
		global $databaseConfig;
		DB::connect($databaseConfig);

		$record = DNCreateEnvironment::get()->byID($this->args['createID']);
		$record->Status = $status;
		$record->write();
	}

	/**
	 *
	 * @return DNData
	 */
	protected function DNData() {
		return DNData::inst();
	}
}



1			<?php
2			class CreateEnvJob extends DeploynautJob {
3
4			/**
5			*
6			* @var array
7			*/
8			public $args;
9
10			/**
11			*
12			*/
13			public function setUp() {
14			$this->updateStatus('Started');
15			chdir(BASE_PATH);
16			}
17
18			/**
19			*
20			*/
21			public function perform() {
22			echo "[-] CreateEnvJob starting" . PHP_EOL;
23			// This is a bit icky, but there is no easy way of capturing a failed deploy by using the PHP Resque
24			try {
25			$envCreate = DNCreateEnvironment::get()->byId($this->args['createID']);
26			if(!($envCreate && $envCreate->exists())) {
27			throw new RuntimeException(sprintf('Could not find create environment record %s', $args['createID']));
28			}
29
30			// This will throw and exception if it fails.
31			$envCreate->createEnvironment();
32
33			} catch(Exception $e) {
34			echo "[-] CreateEnvJob failed" . PHP_EOL;
35			throw $e;
36			}
37			$this->updateStatus('Finished');
38			echo "[-] CreateEnvJob finished" . PHP_EOL;
39			}
40
41			/**
42			*
43			* @param string $status
44			* @global array $databaseConfig
45			*/
46			protected function updateStatus($status) {
47			global $databaseConfig;
48			DB::connect($databaseConfig);
49
50			$record = DNCreateEnvironment::get()->byID($this->args['createID']);
51			$record->Status = $status;
52			$record->write();
53			}
54
55			/**
56			*
57			* @return DNData
58			*/
59			protected function DNData() {
60			return DNData::inst();
61			}
62			}
63
64

silverstripe / deploynaut

Pull Request — master (#520)

CreateEnvJob::perform() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like