silverstripe / deploynaut

code/api/DeploynautAPI.php

Doc Comments +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 	/**
 	 *
 	 * @param SS_HTTPRequest $request
-	 * @return string
+	 * @return SS_HTTPResponse
 	 */
 	public function listProjects(SS_HTTPRequest $request) {
 		$response = array(

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 			'projects' => array(),
 		);
 
-		if($request->httpMethod() != 'GET') return $this->message('API not found', 404);;
+		if($request->httpMethod() != 'GET') return $this->message('API not found', 404); ;
 
 		foreach(DNProject::get() as $item) {
 			if($item->canView($this->getMember())) {

code/backends/CapistranoDeploymentBackend.php

Doc Comments +1 added lines

@@ -300,6 +300,7 @@
 	/**
 	 * Utility function for triggering the db rebuild and flush.
 	 * Also cleans up and generates new error pages.
+	 * @param DeploynautLogFile $log
 	 */
 	public function rebuild(DNEnvironment $environment, $log) {
 		$name = $environment->getFullName();

Spacing +3 added lines, -3 removed lines

@@ -177,7 +177,7 @@ 
 		$env = $environment->Project()->getProcessEnv();
 
 		if(!$args) $args = array();
-		$args['history_path'] = realpath(DEPLOYNAUT_LOG_PATH.'/');
+		$args['history_path'] = realpath(DEPLOYNAUT_LOG_PATH . '/');
 
 		// Inject env string directly into the command.
 		// Capistrano doesn't like the $process->setEnv($env) we'd normally do below.
@@ -191,7 +191,7 @@ 
 
 		$data = DNData::inst();
 		// Generate a capfile from a template
-		$capTemplate = file_get_contents(BASE_PATH.'/deploynaut/Capfile.template');
+		$capTemplate = file_get_contents(BASE_PATH . '/deploynaut/Capfile.template');
 		$cap = str_replace(
 			array('<config root>', '<ssh key>', '<base path>'),
 			array($data->getEnvironmentDir(), DEPLOYNAUT_SSH_KEY, BASE_PATH),
@@ -201,7 +201,7 @@ 
 		if(defined('DEPLOYNAUT_CAPFILE')) {
 			$capFile = DEPLOYNAUT_CAPFILE;
 		} else {
-			$capFile = ASSETS_PATH.'/Capfile';
+			$capFile = ASSETS_PATH . '/Capfile';
 		}
 		file_put_contents($capFile, $cap);
 

code/backends/DeploymentBackend.php

Doc Comments +5 added lines

@@ -10,6 +10,7 @@ 
 	 * @param DeploynautLogFile $log
 	 * @param DNProject $project
 	 * @param type $leaveMaintenancePage
+	 * @return void
 	 */
 	public function deploy(DNEnvironment $environment, $sha, DeploynautLogFile $log, DNProject $project, $leaveMaintenancePage = false);
 
@@ -19,6 +20,7 @@ 
 	 *
 	 * @param DNDataTransfer $dataTransfer
 	 * @param DeploynautLogFile $log
+	 * @return void
 	 */
 	public function dataTransfer(DNDataTransfer $dataTransfer, DeploynautLogFile $log);
 
@@ -28,6 +30,7 @@ 
 	 * @param DNEnvironment $environment
 	 * @param DeploynautLogFile $log
 	 * @param DNProject $project
+	 * @return void
 	 */
 	public function enableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project);
 
@@ -37,6 +40,7 @@ 
 	 * @param DNEnvironment $environment
 	 * @param DeploynautLogFile $log
 	 * @param DNProject $project
+	 * @return void
 	 */
 	public function disableMaintenance(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project);
 
@@ -46,6 +50,7 @@ 
 	 * @param DNEnvironment $environment
 	 * @param DeploynautLogFile $log
 	 * @param DNProject $project
+	 * @return void
 	 */
 	public function ping(DNEnvironment $environment, DeploynautLogFile $log, DNProject $project);
 

code/backends/PackageGenerator.php

Doc Comments +2 added lines, -1 removed lines

@@ -13,8 +13,9 @@
 	/**
 	 * Generate the package file, saving to the given location
 	 *
-	 * @param $baseDir string The base directory of the project, checked out from git.
+	 * @param string $baseDir string The base directory of the project, checked out from git.
 	 * @param $outputFilename string The filename to write to.
+	 * @param string $sha
 	 *
 	 * @return boolean True on success
 	 */

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 
  		// Default, cacheless implementation
  		} else {
- 			$filename = TEMP_FOLDER .'/' . $sha . '.tar.gz';
+ 			$filename = TEMP_FOLDER . '/' . $sha . '.tar.gz';
  			if($this->generatePackage($sha, $repositoryDir, $outputFilename, $log)) {
  				return $filename;
  			}

code/backends/SizeRestrictedPackageCache.php

Doc Comments +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
 	/**
 	 * Take the identifier an make it safe to use as a directory name.
 	 *
-	 * @param string $identfiier The unsanitised directory name.
+	 * @param string $identifier The unsanitised directory name.
 	 */
 	protected function sanitiseDirName($identifier) {
 		$safe = preg_replace('/[^A-Za-z0-9_-]/', '', $identifier);

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
 			return $filename;
 
 		} else {
-			if($this->cacheSize) $this->reduceDirSizeTo($buildPath, $this->cacheSize-1, $log);
+			if($this->cacheSize) $this->reduceDirSizeTo($buildPath, $this->cacheSize - 1, $log);
 
 			if($generator->generatePackage($sha, $repositoryDir, $filename, $log)) {
 				return $filename;
@@ -94,7 +94,7 @@ 
 			    return filemtime($a) > filemtime($b);
 			});
 
-			for($i=0;$i<sizeof($files)-$count;$i++) {
+			for($i = 0; $i < sizeof($files) - $count; $i++) {
 				$log->write("Removing " . $files[$i] . " from package cache");
 				unlink($files[$i]);
 			}

code/control/DeployForm.php

Doc Comments +5 added lines, -1 removed lines

@@ -18,6 +18,7 @@ 
 	 * Validate a commit sha
 	 *
 	 * @param string $sha
+	 * @param string $field
 	 * @return boolean
 	 */
 	protected function validateCommit($sha, $field) {
@@ -100,6 +101,10 @@ 
  */
 class DeployForm extends Form {
 
+	/**
+	 * @param DNRoot $controller
+	 * @param string $name
+	 */
 	public function __construct($controller, $name, DNEnvironment $environment, DNProject $project) {
 		if($environment->HasPipelineSupport()) {
 			// Determine if commits are filtered
@@ -261,7 +266,6 @@ 
 	/**
 	 * Generate fields necessary to select from a filtered commit list
 	 *
-	 * @param DNEnvironment $environment
 	 * @param DataList $commits List of commits
 	 * @return FormField
 	 */

Spacing +8 added lines, -8 removed lines

@@ -32,7 +32,7 @@ 
 		}
 
 		// Check validity of commit
-		if (!preg_match('/^[a-f0-9]{40}$/', $sha)) {
+		if(!preg_match('/^[a-f0-9]{40}$/', $sha)) {
 			$this->validationError(
 				$field,
 				"Invalid release SHA: " . Convert::raw2xml($sha),
@@ -143,7 +143,7 @@ 
 						->setAttribute('onclick',
 							"return confirm('This will begin a release pipeline, but with the following exclusions:\\n" .
 							" - No messages will be sent\\n" .
-							" - No capistrano actions will be invoked\\n".
+							" - No capistrano actions will be invoked\\n" .
 							" - No deployments or snapshots will be created.');"
 						)
 				);
@@ -153,7 +153,7 @@ 
 					FormAction::create('doDeploy', "Direct deployment (bypass pipeline)")
 						->addExtraClass('btn btn-warning')
 						->setAttribute('onclick',
-							"return confirm('This will start a direct deployment, bypassing the pipeline ".
+							"return confirm('This will start a direct deployment, bypassing the pipeline " .
 							"process in place.\\n\\nAre you sure this is necessary?');"
 						)
 				);
@@ -188,7 +188,7 @@ 
 		foreach($project->DNBranchList() as $branch) {
 			$sha = $branch->SHA();
 			$name = $branch->Name();
-			$branches[$sha . '-' . $name] = $name . ' (' . substr($sha,0,8) . ', ' . $branch->LastUpdated()->TimeDiff() . ' old)';
+			$branches[$sha . '-' . $name] = $name . ' (' . substr($sha, 0, 8) . ', ' . $branch->LastUpdated()->TimeDiff() . ' old)';
 		}
 
 		// Tags
@@ -196,7 +196,7 @@ 
 		foreach($project->DNTagList()->setLimit(null) as $tag) {
 			$sha = $tag->SHA();
 			$name = $tag->Name();
-			$tags[$sha . '-' . $tag] = $name . ' (' . substr($sha,0,8) . ', ' . $tag->Created()->TimeDiff() . ' old)';
+			$tags[$sha . '-' . $tag] = $name . ' (' . substr($sha, 0, 8) . ', ' . $tag->Created()->TimeDiff() . ' old)';
 		}
 		$tags = array_reverse($tags);
 
@@ -210,7 +210,7 @@ 
 					$redeploy[$envName] = array();
 				}
 				if(!isset($redeploy[$envName][$sha])) {
-					$redeploy[$envName][$sha] = substr($sha,0,8) . ' (deployed ' . $deploy->obj('LastEdited')->Ago() . ')';
+					$redeploy[$envName][$sha] = substr($sha, 0, 8) . ' (deployed ' . $deploy->obj('LastEdited')->Ago() . ')';
 				}
 			}
 		}
@@ -268,11 +268,11 @@ 
 	protected function buildPipelineField($commits) {
 		// Get filtered commits
 		$filteredCommits = array();
-		foreach ($commits as $commit) {
+		foreach($commits as $commit) {
 			$title = sprintf(
 				"%s (%s, %s old)",
 				$commit->Message,
-				substr($commit->SHA,0,8),
+				substr($commit->SHA, 0, 8),
 				$commit->dbObject('Created')->TimeDiff()
 			);
 			$filteredCommits[$commit->SHA] = $title;

code/control/DNRoot.php

Doc Comments +7 added lines, -4 removed lines

@@ -161,7 +161,7 @@ 
 	/**
 	 * Action
 	 *
-	 * @return string - HTML
+	 * @return HTMLText - HTML
 	 */
 	public function projects(SS_HTTPRequest $request) {
 		// Performs canView permission check by limiting visible projects in DNProjectsList() call.
@@ -854,6 +854,7 @@ 
 	}
 
 	/**
+	 * @param SS_HTTPRequest $request
 	 * @return Form
 	 */
 	public function getDataTransferForm($request) {
@@ -1189,6 +1190,8 @@ 
 
 	/**
 	 * Build snapshot move form.
+	 * @param SS_HTTPRequest $request
+	 * @param DataObject $dataArchive
 	 */
 	public function getMoveForm($request, $dataArchive = null) {
 		$dataArchive = $dataArchive ? $dataArchive : DNDataArchive::get()->byId($request->requestVar('DataArchiveID'));
@@ -1330,7 +1333,7 @@ 
 	 * TODO To be replaced with a method that just returns the list of archives this {@link Member} has access to.
 	 *
 	 * @param Member $member The {@link Member} to check (or null to check the currently logged in Member)
-	 * @return boolean true if $member has access to upload or download to at least one {@link DNEnvironment}.
+	 * @return boolean|null true if $member has access to upload or download to at least one {@link DNEnvironment}.
 	 */
 	public function CanViewArchives(Member $member = null) {
 		if(!$member) $member = Member::currentUser();
@@ -1357,7 +1360,7 @@ 
 	}
 
 	/**
-	 * @return ArrayList The list of all archive files that can be accessed by the currently logged-in {@link Member}
+	 * @return PaginatedList The list of all archive files that can be accessed by the currently logged-in {@link Member}
 	 */
 	public function CompleteDataArchives() {
 		$project = $this->getCurrentProject();
@@ -1375,7 +1378,7 @@ 
 	}
 
 	/**
-	 * @return ArrayList The list of "pending" data archives which are waiting for a file
+	 * @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() {

Spacing +18 added lines, -18 removed lines

@@ -107,11 +107,11 @@ 
 	 * @return boolean
 	 */
 	public static function FlagSnapshotsEnabled() {
-		if (defined('FLAG_SNAPSHOTS_ENABLED') && FLAG_SNAPSHOTS_ENABLED) return true;
-		if (defined('FLAG_SNAPSHOTS_ENABLED_FOR_MEMBERS') && FLAG_SNAPSHOTS_ENABLED_FOR_MEMBERS) {
+		if(defined('FLAG_SNAPSHOTS_ENABLED') && FLAG_SNAPSHOTS_ENABLED) return true;
+		if(defined('FLAG_SNAPSHOTS_ENABLED_FOR_MEMBERS') && FLAG_SNAPSHOTS_ENABLED_FOR_MEMBERS) {
 			$allowedMembers = explode(';', FLAG_SNAPSHOTS_ENABLED_FOR_MEMBERS);
 			$member = Member::currentUser();
-			if ($allowedMembers && $member && in_array($member->Email, $allowedMembers)) return true;
+			if($allowedMembers && $member && in_array($member->Email, $allowedMembers)) return true;
 		}
 		return false;
 	}
@@ -138,7 +138,7 @@ 
 			)
 		);
 
-		Requirements::css(FRAMEWORK_ADMIN_DIR .'/thirdparty/chosen/chosen/chosen.css');
+		Requirements::css(FRAMEWORK_ADMIN_DIR . '/thirdparty/chosen/chosen/chosen.css');
 	}
 
 	/**
@@ -265,7 +265,7 @@ 
 		// Framing an environment as a "group of people with download access"
 		// makes more sense to the user here, while still allowing us to enforce
 		// environment specific restrictions on downloading the file later on.
-		$envs = $project->DNEnvironmentList()->filterByCallback(function($item) {return $item->canUploadArchive();});
+		$envs = $project->DNEnvironmentList()->filterByCallback(function($item) {return $item->canUploadArchive(); });
 		$envsMap = array();
 		foreach($envs as $env) {
 			$envsMap[$env->ID] = $env->Name;
@@ -293,7 +293,7 @@ 
 		$form->disableSecurityToken();
 		$form->addExtraClass('fields-wide');
 		// Tweak the action so it plays well with our fake URL structure.
-		$form->setFormAction($project->Link().'/UploadSnapshotForm');
+		$form->setFormAction($project->Link() . '/UploadSnapshotForm');
 
 		return $form;
 	}
@@ -412,7 +412,7 @@ 
 		// Framing an environment as a "group of people with download access"
 		// makes more sense to the user here, while still allowing us to enforce
 		// environment specific restrictions on downloading the file later on.
-		$envs = $project->DNEnvironmentList()->filterByCallback(function($item) {return $item->canUploadArchive();});
+		$envs = $project->DNEnvironmentList()->filterByCallback(function($item) {return $item->canUploadArchive(); });
 		$envsMap = array();
 		foreach($envs as $env) {
 			$envsMap[$env->ID] = $env->Name;
@@ -434,7 +434,7 @@ 
 		$form->disableSecurityToken();
 		$form->addExtraClass('fields-wide');
 		// Tweak the action so it plays well with our fake URL structure.
-		$form->setFormAction($project->Link().'/PostSnapshotForm');
+		$form->setFormAction($project->Link() . '/PostSnapshotForm');
 
 		return $form;
 	}
@@ -446,7 +446,7 @@ 
 		}
 
 		$validEnvs = $project->DNEnvironmentList()
-			->filterByCallback(function($item) {return $item->canUploadArchive();});
+			->filterByCallback(function($item) {return $item->canUploadArchive(); });
 
 		// Validate $data['EnvironmentID'] by checking against $validEnvs.
 		$environment = $validEnvs->find('ID', $data['EnvironmentID']);
@@ -745,7 +745,7 @@ 
 		$form = new DeployForm($this, 'DeployForm', $environment, $project);
 
 		// Tweak the action so it plays well with our fake URL structure.
-		$form->setFormAction($request->getURL().'/DeployForm');
+		$form->setFormAction($request->getURL() . '/DeployForm');
 		return $form;
 	}
 
@@ -859,7 +859,7 @@ 
 	public function getDataTransferForm($request) {
 		// Performs canView permission check by limiting visible projects
 		$envs = $this->getCurrentProject()->DNEnvironmentList()
-			->filterByCallback(function($item) {return $item->canBackup();});
+			->filterByCallback(function($item) {return $item->canBackup(); });
 		if(!$envs) {
 			return new SS_HTTPResponse("Environment '" . Convert::raw2xml($request->latestParam('Environment')) . "' not found.", 404);
 		}
@@ -876,7 +876,7 @@ 
 				FormAction::create('doDataTransfer', 'Create')->addExtraClass('btn')
 			)
 		);
-		$form->setFormAction($request->getURL().'/DataTransferForm');
+		$form->setFormAction($request->getURL() . '/DataTransferForm');
 
 		return $form;
 	}
@@ -892,12 +892,12 @@ 
 		$dataArchive = null;
 
 		// Validate direction.
-		if ($data['Direction']=='get') {
+		if($data['Direction'] == 'get') {
 			$validEnvs = $this->getCurrentProject()->DNEnvironmentList()
-				->filterByCallback(function($item) {return $item->canBackup();});
-		} else if ($data['Direction']=='push') {
+				->filterByCallback(function($item) {return $item->canBackup(); });
+		} else if($data['Direction'] == 'push') {
 			$validEnvs = $this->getCurrentProject()->DNEnvironmentList()
-				->filterByCallback(function($item) {return $item->canRestore();});
+				->filterByCallback(function($item) {return $item->canRestore(); });
 		} else {
 			throw new LogicException('Invalid direction');
 		}
@@ -909,7 +909,7 @@ 
 		}
 
 		// Validate mode.
-		if (!in_array($data['Mode'], array('all', 'assets', 'db'))) {
+		if(!in_array($data['Mode'], array('all', 'assets', 'db'))) {
 			throw new LogicException('Invalid mode');
 		}
 
@@ -1022,7 +1022,7 @@ 
 		// Performs canView permission check by limiting visible projects
 		$project = $this->getCurrentProject();
 		$envs = $project->DNEnvironmentList()
-			->filterByCallback(function($item) {return $item->canRestore();});
+			->filterByCallback(function($item) {return $item->canRestore(); });
 
 		if(!$envs) {
 			return new SS_HTTPResponse("Environment '" . Convert::raw2xml($request->latestParam('Environment')) . "' not found.", 404);

code/control/EmailMessagingService.php

Doc Comments +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 	 *
 	 * @param PipelineStep $source Client step
 	 * @param string $from
-	 * @param string|Member $to
+	 * @param string $to
 	 * @param string $subject
 	 * @param string $body
 	 */

code/model/DNEnvironment.php

Doc Comments +3 added lines

@@ -919,6 +919,9 @@
 		$fields->insertAfter($createConfigField, 'noDeployConfig');
 	}
 
+	/**
+	 * @param FieldList $fields
+	 */
 	protected function setPipelineConfigurationFields($fields) {
 		if(!$this->config()->get('allow_web_editing')) {
 			return;

Spacing +17 added lines, -17 removed lines

@@ -107,9 +107,9 @@ 
 		"CanBackupGroups"   => "Group",
 		"ArchiveUploaders"   => "Member", // Who can upload archive files linked to this environment
 		"ArchiveUploaderGroups" => "Group",
-		"ArchiveDownloaders" => "Member",  // Who can download archive files from this environment
+		"ArchiveDownloaders" => "Member", // Who can download archive files from this environment
 		"ArchiveDownloaderGroups" => "Group",
-		"ArchiveDeleters"    => "Member",  // Who can delete archive files from this environment,
+		"ArchiveDeleters"    => "Member", // Who can delete archive files from this environment,
 		"ArchiveDeleterGroups" => "Group",
 		"PipelineApprovers"  => "Member", // Who can approve / reject pipelines from this environment
 		"PipelineApproverGroups" => "Group",
@@ -291,7 +291,7 @@ 
 	 */
 	public function CanCancelPipeline() {
 		// do we have a current pipeline
-		if ($this->HasCurrentPipeline()) {
+		if($this->HasCurrentPipeline()) {
 			return $this->CurrentPipeline()->canAbort();
 		}
 		return false;
@@ -599,22 +599,22 @@ 
 		// The DeployHistory function is far too slow to use for this
 		$deploy = DNDeployment::get()->filter(array('EnvironmentID' => $this->ID, 'Status' => 'Finished'))->sort('LastEdited DESC')->first();
 
-		if (!$deploy || (!$deploy->SHA)) {
+		if(!$deploy || (!$deploy->SHA)) {
 			return false;
 		}
 
 		$repo = $this->Project()->getRepository();
-		if (!$repo) {
+		if(!$repo) {
 			return $deploy;
 		}
 
 		try {
 			$commit = $repo->getCommit($deploy->SHA);
-			if ($commit) {
+			if($commit) {
 				$deploy->Message = Convert::raw2xml($commit->getMessage());
 			}
 			// We can't find this SHA, so we ignore adding a commit message to the deployment
-		} catch (Exception $ex) { }
+		} catch(Exception $ex) { }
 
 		return $deploy;
 	}
@@ -644,7 +644,7 @@ 
 					$deploy->Message = Convert::raw2xml($commit->getMessage());
 				}
 				// We can't find this SHA, so we ignore adding a commit message to the deployment
-			} catch (Exception $ex) { }
+			} catch(Exception $ex) { }
 			$ammendedHistory->push($deploy);
 		}
 
@@ -682,7 +682,7 @@ 
 	 * @return string
 	 */
 	public function Link() {
-		return $this->Project()->Link()."/environment/" . $this->Name;
+		return $this->Project()->Link() . "/environment/" . $this->Name;
 	}
 
 
@@ -887,7 +887,7 @@ 
 		// Add actions
 		$action = new FormAction('check', 'Check Connection');
 		$action->setUseButtonTag(true);
-		$action->setAttribute('data-url', Director::absoluteBaseURL().'naut/api/'.$this->Project()->Name.'/'.$this->Name.'/ping');
+		$action->setAttribute('data-url', Director::absoluteBaseURL() . 'naut/api/' . $this->Project()->Name . '/' . $this->Name . '/ping');
 		$fields->insertBefore($action, 'Name');
 
 		// Allow extensions
@@ -950,8 +950,8 @@ 
 	 */
 	public function onBeforeWrite() {
 		parent::onBeforeWrite();
-		if($this->Name && $this->Name.'.rb' != $this->Filename) {
-			$this->Filename = $this->Name.'.rb';
+		if($this->Name && $this->Name . '.rb' != $this->Filename) {
+			$this->Filename = $this->Name . '.rb';
 		}
 		$this->checkEnvironmentPath();
 		$this->writeConfigFile();
@@ -976,11 +976,11 @@ 
 		if(!$this->config()->get('allow_web_editing')) return;
 
 		// Create a basic new environment config from a template
-		if( !$this->envFileExists()
+		if(!$this->envFileExists()
 			&& $this->Filename
 			&& $this->CreateEnvConfig
 		) {
-			$templateFile = $this->config()->template_file ?: BASE_PATH.'/deploynaut/environment.template';
+			$templateFile = $this->config()->template_file ?: BASE_PATH . '/deploynaut/environment.template';
 			file_put_contents($this->getConfigFilename(), file_get_contents($templateFile));
 		} else if($this->envFileExists() && $this->DeployConfig) {
 			file_put_contents($this->getConfigFilename(), $this->DeployConfig);
@@ -1047,7 +1047,7 @@ 
 		if(!$this->Filename) {
 			return '';
 		}
-		return $this->DNData()->getEnvironmentDir().'/'.$this->Project()->Name.'/'.$this->Filename;
+		return $this->DNData()->getEnvironmentDir() . '/' . $this->Project()->Name . '/' . $this->Filename;
 	}
 
 	/**
@@ -1126,10 +1126,10 @@ 
 		$filter = isset($config->PipelineConfig->FilteredCommits)
 			? $config->PipelineConfig->FilteredCommits
 			: null;
-		if (empty($filter)) return null;
+		if(empty($filter)) return null;
 
 		// Create and execute filter
-		if (!class_exists($filter)) throw new Exception(sprintf("Class %s does not exist", $filter));
+		if(!class_exists($filter)) throw new Exception(sprintf("Class %s does not exist", $filter));
 		$commitClass = $filter::create();
 		// setup the environment to check for commits
 		$commitClass->env = $dependsOnEnv;