cross-solution / YAWIK

public/index.php

Spacing +10 added lines, -10 removed lines

@@ -11,23 +11,23 @@ 
 
 date_default_timezone_set('Europe/Berlin');
 
-if(!version_compare(PHP_VERSION, '5.6.0', 'ge')){
-    echo sprintf('<p>Sorry, YAWIK requires at least PHP 5.6.0 to run, but this server currently provides PHP %s</p>',PHP_VERSION);
+if (!version_compare(PHP_VERSION, '5.6.0', 'ge')) {
+    echo sprintf('<p>Sorry, YAWIK requires at least PHP 5.6.0 to run, but this server currently provides PHP %s</p>', PHP_VERSION);
     echo '<p>Please ask your servers\' administrator to install the proper PHP version.</p>';
     exit;
 }
 
 if (php_sapi_name() == 'cli-server') {
     $parseUrl = parse_url(substr($_SERVER["REQUEST_URI"], 1));
-    $route = isset($parseUrl['path']) ? $parseUrl['path']:null;
-    if (is_file(__DIR__ . '/' . $route)) {
-        if(substr($route, -4) == ".php"){
-            require __DIR__ . '/' . $route;     // Include requested script files
+    $route = isset($parseUrl['path']) ? $parseUrl['path'] : null;
+    if (is_file(__DIR__.'/'.$route)) {
+        if (substr($route, -4) == ".php") {
+            require __DIR__.'/'.$route; // Include requested script files
             exit;
         }
-        return false;           // Serve file as is
+        return false; // Serve file as is
     } else {                    // Fallback to index.php
-        $_GET["q"] = $route;    // Try to emulate the behaviour of a .htaccess here.
+        $_GET["q"] = $route; // Try to emulate the behaviour of a .htaccess here.
     }
 }
 
@@ -47,8 +47,8 @@ 
 } else {
     echo '<p>Could not initialize autoloading. This happens, if the dependencies are not installed yet.</p>';
     echo '<p>Please try to install the dependencies via: </p>';
-    echo '<code>cd '. realpath('.') .'<br>./install.sh</code>';
-    echo '<p>exit at ' . __FILE__ . ' in line ' . __LINE__ .'</p>';
+    echo '<code>cd '.realpath('.').'<br>./install.sh</code>';
+    echo '<p>exit at '.__FILE__.' in line '.__LINE__.'</p>';
     exit;
 }
 

module/Behat/src/ApplicationContext.php

Spacing +2 added lines, -2 removed lines

@@ -35,13 +35,13 @@
 		/* @var $repo JobRepository */
 		$repo = $this->getRepository('Jobs/Job');
 		$job = $repo->findOneBy(['title' => $title]);
-		if(!$job instanceof Job){
+		if (!$job instanceof Job) {
 			throw new \Exception('There is no job titled: "'.$title.'"');
 		}
 		$job->setApplyId($job->getId());
 		$repo->store($job);
 
-		$url = $this->generateUrl('lang/apply',[
+		$url = $this->generateUrl('lang/apply', [
 		    'applyId' => $job->getApplyId()
         ]);
 		$this->visit($url);

module/Behat/src/InstallContext.php

Spacing +11 added lines, -11 removed lines

@@ -31,8 +31,8 @@ 
 
     public function __construct()
     {
-        static::$configFile = getcwd() . '/config/autoload/install.module.php';
-        static::$yawikGlobalConfig = getcwd() . '/config/autoload/yawik.config.global.php';
+        static::$configFile = getcwd().'/config/autoload/install.module.php';
+        static::$yawikGlobalConfig = getcwd().'/config/autoload/yawik.config.global.php';
         static::$yawikBackupConfig = str_replace('yawik.config.global.php', 'yawik.backup', static::$yawikGlobalConfig);
     }
 
@@ -42,18 +42,18 @@ 
     public function iHaveInstallModuleActivated()
     {
         $target = static::$configFile;
-        if(!file_exists($target)){
+        if (!file_exists($target)) {
             $source = __DIR__.'/../resources/install.module.php';
-            copy($source,$target);
-            chmod($target,0777);
+            copy($source, $target);
+            chmod($target, 0777);
         }
 
         // backup existing file
         $yawikBackupConfig = static::$yawikBackupConfig;
         $yawikGlobalConfig = static::$yawikGlobalConfig;
 
-        if(is_file($yawikGlobalConfig)){
-            rename($yawikGlobalConfig,$yawikBackupConfig);
+        if (is_file($yawikGlobalConfig)) {
+            rename($yawikGlobalConfig, $yawikBackupConfig);
         }
     }
 
@@ -92,7 +92,7 @@ 
 
     static public function restoreConfig()
     {
-        if(is_file($file = static::$configFile)){
+        if (is_file($file = static::$configFile)) {
             unlink($file);
         }
 
@@ -100,8 +100,8 @@ 
         $yawikBackupConfig = static::$yawikBackupConfig;
         $yawikGlobalConfig = static::$yawikGlobalConfig;
 
-        if(is_file($yawikBackupConfig)){
-            rename($yawikBackupConfig,$yawikGlobalConfig);
+        if (is_file($yawikBackupConfig)) {
+            rename($yawikBackupConfig, $yawikGlobalConfig);
         }
     }
 
@@ -112,6 +112,6 @@ 
     {
         $config = $this->getService('config');
         $connection = $config['doctrine']['connection']['odm_default']['connectionString'];
-        $this->minkContext->fillField('db_conn',$connection);
+        $this->minkContext->fillField('db_conn', $connection);
     }
 }

module/Behat/src/CommonContextTrait.php

Spacing +3 added lines, -3 removed lines

@@ -51,16 +51,16 @@
 		$this->summaryFormContext = $scope->getEnvironment()->getContext(SummaryFormContext::class);
 	}
 	
-	public function generateUrl($name,array $params=array(),array $options=array())
+	public function generateUrl($name, array $params = array(), array $options = array())
 	{
 	    $defaults = ['lang'=>'en'];
-	    $params = array_merge($defaults,$params);
+	    $params = array_merge($defaults, $params);
         /* @var Url $urlHelper */
         $urlHelper = $this
             ->getService('ViewHelperManager')
             ->get('url')
         ;
-        $url = $urlHelper($name,$params,$options);
+        $url = $urlHelper($name, $params, $options);
 
         return $this->coreContext->generateUrl($url);
 	}

module/Behat/src/UserContext.php

Spacing +46 added lines, -46 removed lines

@@ -64,7 +64,7 @@ 
      */
     protected $currentUser;
 	
-	public function __construct($parameters=[])
+	public function __construct($parameters = [])
 	{
 		$defaultLoginInfo = [
 			'facebook' => [
@@ -76,8 +76,8 @@ 
 				'session_password-login' => getenv('LINKEDIN_USER_PASSWORD')
 			],
 		];
-		$socialLoginConfig = isset($parameters['social_login_info']) ? $parameters['social_login_info']:[];
-		$this->socialLoginInfo = array_merge($defaultLoginInfo,$socialLoginConfig);
+		$socialLoginConfig = isset($parameters['social_login_info']) ? $parameters['social_login_info'] : [];
+		$this->socialLoginInfo = array_merge($defaultLoginInfo, $socialLoginConfig);
 	}
 
 	/**
@@ -87,12 +87,12 @@ 
 	static public function afterSuite(AfterSuiteScope $scope)
 	{
 		$repo = static::$userRepo;
-		foreach(static::$users as $user){
-			if($repo->findByLogin($user->getLogin())){
-				try{
+		foreach (static::$users as $user) {
+			if ($repo->findByLogin($user->getLogin())) {
+				try {
 					JobContext::removeJobByUser($user);
-					$repo->remove($user,true);
-				}catch (\Exception $e){
+					$repo->remove($user, true);
+				} catch (\Exception $e) {
 				
 				}
 			}
@@ -117,8 +117,8 @@ 
 	{
 		$provider = strtolower($provider);
 		$mink = $this->minkContext;
-		foreach($this->socialLoginInfo[$provider] as $field=>$value){
-			$mink->fillField($field,$value);
+		foreach ($this->socialLoginInfo[$provider] as $field=>$value) {
+			$mink->fillField($field, $value);
 		}
 	}
 	
@@ -126,15 +126,15 @@ 
 	 * @Given I am logged in as a recruiter
 	 * @Given I am logged in as a recruiter with :organization as organization
 	 */
-	public function iAmLoggedInAsARecruiter($organization=null)
+	public function iAmLoggedInAsARecruiter($organization = null)
 	{
 		$user = $this->thereIsAUserIdentifiedBy(
 			'test@recruiter.com',
-			'test',User::ROLE_RECRUITER,
+			'test', User::ROLE_RECRUITER,
 			'Test Recruiter',
 			$organization
 		);
-		$this->startLogin($user,'test');
+		$this->startLogin($user, 'test');
 	}
 	
 	/**
@@ -144,9 +144,9 @@ 
 	public function iDonTHaveUser($login)
 	{
 		$repo = $this->getUserRepository();
-		$user=$repo->findByLogin($login);
-		if($user instanceof UserInterface){
-			$repo->remove($user,true);
+		$user = $repo->findByLogin($login);
+		if ($user instanceof UserInterface) {
+			$repo->remove($user, true);
 		}
 	}
 	
@@ -155,7 +155,7 @@ 
 	 * @param $role
 	 * @param TableNode $fields
 	 */
-	public function iHaveUserWithTheFollowing($role,TableNode $fields)
+	public function iHaveUserWithTheFollowing($role, TableNode $fields)
 	{
 		$normalizedFields = [
 			'login' => 'test@login.com',
@@ -164,7 +164,7 @@ 
 			'password' => 'test',
 			'organization' => 'Cross Solution'
 		];
-		foreach($fields->getRowsHash() as $field=>$value){
+		foreach ($fields->getRowsHash() as $field=>$value) {
 			$field = Inflector::camelize($field);
 			$normalizedFields[$field] = $value;
 		}
@@ -184,14 +184,14 @@ 
 	 */
 	public function iAmLoggedInAsAnAdmin()
 	{
-		$user = $this->thereIsAUserIdentifiedBy('test@admin.com','test',User::ROLE_ADMIN);
-		$this->startLogin($user,'test');
+		$user = $this->thereIsAUserIdentifiedBy('test@admin.com', 'test', User::ROLE_ADMIN);
+		$this->startLogin($user, 'test');
 	}
 	
 	private function startLogin(UserInterface $user, $password)
 	{
 		$currentUser = $this->currentUser;
-		if(!is_object($currentUser) || $user->getId()!=$currentUser->getId()){
+		if (!is_object($currentUser) || $user->getId() != $currentUser->getId()) {
 			$this->iWantToLogIn();
 			$this->iSpecifyTheUsernameAs($user->getLogin());
 			$this->iSpecifyThePasswordAs($password);
@@ -211,15 +211,15 @@ 
 	/**
 	 * @Given there is a user :email identified by :password
 	 */
-	public function thereIsAUserIdentifiedBy($email, $password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter",$organization=null)
+	public function thereIsAUserIdentifiedBy($email, $password, $role = User::ROLE_RECRUITER, $fullname = "Test Recruiter", $organization = null)
 	{
 		$repo = $this->getUserRepository();
-		if(!is_object($user=$repo->findByEmail($email))){
-			$user = $this->createUser($email,$password,$role,$fullname,$organization);
+		if (!is_object($user = $repo->findByEmail($email))) {
+			$user = $this->createUser($email, $password, $role, $fullname, $organization);
 		}
 		
-		if(!is_null($organization)){
-			$this->iHaveMainOrganization($user,$organization);
+		if (!is_null($organization)) {
+			$this->iHaveMainOrganization($user, $organization);
 		}
 		$this->addCreatedUser($user);
 		return $user;
@@ -234,7 +234,7 @@ 
 	 *
 	 * @return \Auth\Entity\UserInterface
 	 */
-	public function createUser($email,$password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter")
+	public function createUser($email, $password, $role = User::ROLE_RECRUITER, $fullname = "Test Recruiter")
 	{
 		/* @var Register $service */
 		/* @var User $user */
@@ -245,10 +245,10 @@ 
 		$user->setRole($role);
 		$settings = $user->getSettings('Applications');
 		
-		$expFullName = explode(' ',$fullname);
+		$expFullName = explode(' ', $fullname);
 		$info = $user->getInfo();
 		$info->setFirstName(array_shift($expFullName));
-		$info->setLastName(count($expFullName)>0 ? implode(' ',$expFullName):'');
+		$info->setLastName(count($expFullName) > 0 ? implode(' ', $expFullName) : '');
 		$info->setEmail($email);
 		$info->setEmailVerified(true);
 		$repo->store($user);
@@ -273,19 +273,19 @@ 
 	 * @When I have :organization as my main organization
 	 * @param $orgName
 	 */
-	public function iHaveMainOrganization(UserInterface $user,$orgName)
+	public function iHaveMainOrganization(UserInterface $user, $orgName)
 	{
 		/* @var $repoOrganization OrganizationRepository */
 		$repoOrganization = $this->coreContext->getRepositories()->get('Organizations/Organization');
-		$organization=$repoOrganization->findByName($orgName);
-		if(!$organization instanceof Organization){
+		$organization = $repoOrganization->findByName($orgName);
+		if (!$organization instanceof Organization) {
 			$organization = new Organization();
 			$organizationName = new OrganizationName($orgName);
 			$organization->setOrganizationName($organizationName);
 			$permissions = $organization->getPermissions();
-			$permissions->grant($user,Permissions::PERMISSION_ALL);
-		}else {
-			$organization->getPermissions()->grant($user,Permissions::PERMISSION_ALL);
+			$permissions->grant($user, Permissions::PERMISSION_ALL);
+		} else {
+			$organization->getPermissions()->grant($user, Permissions::PERMISSION_ALL);
 		}
 		$organization->setUser($user);
 		$repoOrganization->store($organization);
@@ -307,7 +307,7 @@ 
 	 */
 	public function iSpecifyTheUsernameAs($username)
 	{
-		$this->minkContext->fillField('Login name',$username);
+		$this->minkContext->fillField('Login name', $username);
 	}
 	
 	/**
@@ -315,7 +315,7 @@ 
 	 */
 	public function iSpecifyThePasswordAs($password)
 	{
-		$this->minkContext->fillField('Password',$password);
+		$this->minkContext->fillField('Password', $password);
 	}
 	
 	/**
@@ -326,8 +326,8 @@ 
 		$repo = $this->getUserRepository();
 		$user = $repo->findByLogin($username);
 		
-		if(!$user instanceof User){
-			throw new \Exception(sprintf('There is no user with this login: "%s"',$username));
+		if (!$user instanceof User) {
+			throw new \Exception(sprintf('There is no user with this login: "%s"', $username));
 		}
 		$this->iWantToLogIn();
 		$this->iSpecifyTheUsernameAs($username);
@@ -383,13 +383,13 @@ 
 	{
 		$repo = $this->getUserRepository();
 		$data = $table->getRowsHash();
-		$email = isset($data['email']) ? $data['email']:'test@example.com';
-		$password = isset($data['password']) ? $data['password']:'test';
-		$fullname = isset($data['fullname']) ? $data['fullname']:'Test User';
-		$role = isset($data['role']) ? $data['role']:User::ROLE_RECRUITER;
+		$email = isset($data['email']) ? $data['email'] : 'test@example.com';
+		$password = isset($data['password']) ? $data['password'] : 'test';
+		$fullname = isset($data['fullname']) ? $data['fullname'] : 'Test User';
+		$role = isset($data['role']) ? $data['role'] : User::ROLE_RECRUITER;
 		
-		if(!is_object($user=$repo->findByLogin($email))){
-			$user = $this->createUser($email,$password,$role,$fullname);
+		if (!is_object($user = $repo->findByLogin($email))) {
+			$user = $this->createUser($email, $password, $role, $fullname);
 		}
 		$this->currentUser = $user;
 		$this->addCreatedUser($user);
@@ -397,7 +397,7 @@ 
 	
 	private function addCreatedUser(UserInterface $user)
 	{
-		if(!in_array($user,static::$users)){
+		if (!in_array($user, static::$users)) {
 			static::$users[] = $user;
 		}
 	}

module/Behat/src/SettingsContext.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 	 */
 	public function iGoToEmailTemplatePage()
 	{
-	    $url = $this->generateUrl('lang/settings',[
+	    $url = $this->generateUrl('lang/settings', [
 	        'module' => 'Applications'
         ]);
 		$this->visit($url);

module/Behat/src/JobContext.php

Spacing +44 added lines, -44 removed lines

@@ -57,8 +57,8 @@ 
 	{
 		$repo = static::$jobRepo;
 		$results = $repo->findBy(['user' => $user]);
-		foreach($results as $result){
-			$repo->remove($result,true);
+		foreach ($results as $result) {
+			$repo->remove($result, true);
 		}
 	}
 	
@@ -70,7 +70,7 @@ 
 	public function beforeScenario(BeforeScenarioScope $scope)
 	{
 		$this->select2Context = $scope->getEnvironment()->getContext(Select2Context::class);
-		if(is_null(static::$jobRepo)){
+		if (is_null(static::$jobRepo)) {
 			$this->gatherContexts($scope);
 			static::$jobRepo = $this->getJobRepository();
 		}
@@ -89,7 +89,7 @@ 
 	 */
 	public function iGoToCreateJob()
 	{
-	    $url = $this->generateUrl('lang/jobs/manage',['action' => 'edit']);
+	    $url = $this->generateUrl('lang/jobs/manage', ['action' => 'edit']);
 		$this->visit($url);
 	}
 	
@@ -109,11 +109,11 @@ 
 	public function iGoToEditJobWithTitle($jobTitle)
 	{
 		$job = $this->getJobRepository()->findOneBy(['title' => $jobTitle]);
-		if(!$job instanceof Job){
-			throw new \Exception(sprintf('Job with title "%s" is not found',$jobTitle));
+		if (!$job instanceof Job) {
+			throw new \Exception(sprintf('Job with title "%s" is not found', $jobTitle));
 		}
 		$this->currentJob = $job;
-		$url = $this->generateUrl('lang/jobs/manage',[
+		$url = $this->generateUrl('lang/jobs/manage', [
 		    'id' => $job->getId()
         ]);
 		$this->visit($url);
@@ -139,8 +139,8 @@ 
 
 		$jobRepository = $this->getJobRepository();
 		$results = $jobRepository->getUserJobs($user->getId());
-		foreach($results as $job){
-			$jobRepository->remove($job,true);
+		foreach ($results as $job) {
+			$jobRepository->remove($job, true);
 		}
 		$this->currentJob = null;
 	}
@@ -149,16 +149,16 @@ 
 	 * @When I fill job location search with :search and choose :choice
 	 *
 	 */
-	public function iFillJobLocationAndChoose($search,$choice)
+	public function iFillJobLocationAndChoose($search, $choice)
 	{
 		$select2 = $this->select2Context;
-		$select2->iFillInSelect2FieldWith('jobBase[geoLocation]',$search,$choice);
+		$select2->iFillInSelect2FieldWith('jobBase[geoLocation]', $search, $choice);
 	}
 	
 	/**
 	 * @When I choose :value from :field
 	 */
-	public function iJobClassificationSelect($value,$field)
+	public function iJobClassificationSelect($value, $field)
 	{
 		$field = Inflector::camelize($field);
 		
@@ -174,18 +174,18 @@ 
 			'employmentTypes'    => "select#classifications-employmentTypes",
 		];
 		
-		if(!isset($mapSelect2[$field])){
+		if (!isset($mapSelect2[$field])) {
 			throw new \Exception('Undefined field selection value "'.$field.'"');
 		}
 		
 		$multipleField = $mapMultiple[$field];
 		$page = $this->minkContext->getSession()->getPage();
-		$element = $page->find('css',$mapMultiple[$field]);
-		if(!is_null($element) && $element->getAttribute('multiple')=='multiple'){
-			$this->minkContext->selectOption($value,$multipleField);
-		}else{
+		$element = $page->find('css', $mapMultiple[$field]);
+		if (!is_null($element) && $element->getAttribute('multiple') == 'multiple') {
+			$this->minkContext->selectOption($value, $multipleField);
+		} else {
 			$locator = $mapSelect2[$field];
-			$this->select2Context->iFillInSelect2Field($locator,$value);
+			$this->select2Context->iFillInSelect2Field($locator, $value);
 		}
 	}
 	
@@ -209,62 +209,62 @@ 
 	 * @When I have a :status job with the following:
 	 * @param TableNode $fields
 	 */
-	public function iHaveAJobWithTheFollowing($status,TableNode $fields)
+	public function iHaveAJobWithTheFollowing($status, TableNode $fields)
 	{
 		$normalizedField = [
 			'template' => 'modern',
 		];
-		foreach($fields->getRowsHash() as $field => $value){
+		foreach ($fields->getRowsHash() as $field => $value) {
 			$field = Inflector::camelize($field);
-			if($field == 'professions' || $field == 'industries'){
-				$value = explode(',',$value);
+			if ($field == 'professions' || $field == 'industries') {
+				$value = explode(',', $value);
 			}
 			$normalizedField[$field] = $value;
 		}
 		$jobRepo = $this->getJobRepository();
 		$job = $jobRepo->findOneBy(['title' => $normalizedField['title']]);
-		if(!$job instanceof Job){
+		if (!$job instanceof Job) {
 			$job = new Job();
 			$job->setTitle($normalizedField['title']);
 		}
-		if(isset($normalizedField['user'])){
+		if (isset($normalizedField['user'])) {
 			/* @var $userRepo UserRepository */
 			$userRepo = $this->getRepository('Auth\Entity\User');
 			$user = $userRepo->findOneBy(['login' => $normalizedField['user']]);
-			if($user instanceof User){
+			if ($user instanceof User) {
 				$job->setUser($user);
 				$job->setOrganization($user->getOrganization()->getOrganization());
-			}else{
+			} else {
 				throw new \Exception('There is no user with this login:"'.$normalizedField['user'.'"']);
 			}
 		}
 		
-		if($status == 'draft'){
+		if ($status == 'draft') {
 			$job->setIsDraft(true);
-		}elseif($status == 'published'){
+		}elseif ($status == 'published') {
 			$job->setIsDraft(false);
 			$job->setDatePublishStart(new \DateTime());
 		}
 		$job->setStatus(Status::ACTIVE);
 		
-		if(isset($normalizedField['location'])){
-			$this->setLocation($job,$normalizedField['location']);
+		if (isset($normalizedField['location'])) {
+			$this->setLocation($job, $normalizedField['location']);
 		}
-		if(isset($normalizedField['companyName'])){
+		if (isset($normalizedField['companyName'])) {
 			//$job->setCompany($normalizedField['companyName']);
 		}
-		if(isset($normalizedField['professions'])){
-			$this->addProfessions($job,$normalizedField['professions']);
+		if (isset($normalizedField['professions'])) {
+			$this->addProfessions($job, $normalizedField['professions']);
 		}
 		
-		if(isset($normalizedField['industries'])){
-			$this->addIndustries($job,$normalizedField['industries']);
+		if (isset($normalizedField['industries'])) {
+			$this->addIndustries($job, $normalizedField['industries']);
 		}
-		if(isset($normalizedField['employmentTypes'])){
+		if (isset($normalizedField['employmentTypes'])) {
 			$types = $this->getCategories([$normalizedField['employmentTypes']]);
 			$type = array_shift($types);
 			$values = $job->getClassifications()->getEmploymentTypes()->getValues();
-			if(!is_array($values) || !in_array($type,$values)){
+			if (!is_array($values) || !in_array($type, $values)) {
 				$job->getClassifications()->getEmploymentTypes()->getItems()->add($type);
 			}
 		}
@@ -283,18 +283,18 @@ 
 		$location->fromString($serialized);
 		
 		$locations = $job->getLocations();
-		if(count($locations)){
+		if (count($locations)) {
 			$locations->clear();
 		}
 		$job->getLocations()->add($location);
 	}
 	
-	private function addProfessions(Job &$job,$terms)
+	private function addProfessions(Job &$job, $terms)
 	{
 		$professions = $this->getCategories($terms);
-		foreach($professions as $profession){
+		foreach ($professions as $profession) {
 			$values = $job->getClassifications()->getProfessions()->getValues();
-			if(!is_array($values) || !in_array($profession,$values)){
+			if (!is_array($values) || !in_array($profession, $values)) {
 				$job->getClassifications()->getProfessions()->getItems()->add($profession);
 			}
 		}
@@ -303,9 +303,9 @@ 
 	private function addIndustries(Job &$job, $terms)
 	{
 		$industries = $this->getCategories($terms);
-		foreach($industries as $industry){
+		foreach ($industries as $industry) {
 			$values = $job->getClassifications()->getIndustries()->getValues();
-			if(!is_array($values) || !in_array($industry,$values)){
+			if (!is_array($values) || !in_array($industry, $values)) {
 				$job->getClassifications()->getIndustries()->getItems()->add($industry);
 			}
 		}
@@ -340,7 +340,7 @@ 
 		
 		$job = $repo->findDraft($user);
 		
-		if(is_null($job)){
+		if (is_null($job)) {
 			$job = new Job();
 			$job
 				->setUser($user)