cross-solution / YAWIK

module/Behat/src/MailContext.php

Spacing +22 added lines, -22 removed lines

@@ -49,7 +49,7 @@ 
         $core = $scope->getEnvironment()->getContext(CoreContext::class);
         /* @var FileTransport $transport */
         $transport = $core->getServiceManager()->get('Core/MailService')->getTransport();
-        $path = $transport->getOptions()->getPath() . '/*.eml';
+        $path = $transport->getOptions()->getPath().'/*.eml';
         foreach (glob($path) as $filename) {
             unlink($filename);
         }
@@ -105,25 +105,25 @@ 
         $regex = '/.*('.preg_quote($text).').*/im';
         $matches = [];
         $multiMessages = false;
-        if(count($this->messages) > 1){
+        if (count($this->messages) > 1) {
             $multiMessages = true;
         }
         $content = "";
-        foreach($this->messages as $key=>$definition){
+        foreach ($this->messages as $key=>$definition) {
             $content = $definition['contents'];
-            if(preg_match($regex,$content,$match)){
+            if (preg_match($regex, $content, $match)) {
                 $matches[] = $match;
             }
         }
-        $failMessage = sprintf('Can not find text "%s" in any email sent',$text);
-        if(!$multiMessages){
+        $failMessage = sprintf('Can not find text "%s" in any email sent', $text);
+        if (!$multiMessages) {
             $failMessage = sprintf(
                 'Can not find text "%s" in sent email. Here\'s the email content: %s',
                 $text,
                 PHP_EOL.PHP_EOL.$content
             );
         }
-        Assert::true(count($matches)>0,$failMessage);
+        Assert::true(count($matches) > 0, $failMessage);
     }
 
     /**
@@ -136,16 +136,16 @@ 
 
         $path = $transport->getOptions()->getPath().'/*.eml';
 
-        foreach(glob($path) as $filename){
+        foreach (glob($path) as $filename) {
             $id = md5($filename);
-            if(!isset($this->messages[$id])){
+            if (!isset($this->messages[$id])) {
                 $contents = file_get_contents($filename);
-                $this->messages[$id]  = $this->parseEmail($contents);
+                $this->messages[$id] = $this->parseEmail($contents);
             }
         }
 
         Assert::true(
-            count($this->messages)>0,
+            count($this->messages) > 0,
             'No email have been sent'
         );
     }
@@ -153,28 +153,28 @@ 
     private function parseEmail($contents)
     {
         $addresses = $this->parseEmailAddress($contents);
-        $subject =$this->parseSubject($contents);
+        $subject = $this->parseSubject($contents);
 
         $contents = strip_tags($contents);
         $contents = preg_replace("/(^[\r\n]*|[\r\n]+)[\s\t]*[\r\n]+/", "\n", $contents);
 
-        return array_merge($addresses,$subject,['contents' => $contents]);
+        return array_merge($addresses, $subject, ['contents' => $contents]);
     }
 
     private function parseEmailAddress($contents)
     {
         // pattern to get email address
-        $radd =  '(\b[A-Z0-9._%+-]+@(?:[A-Z0-9-]+\.)+[A-Z]{2,6}\b)';
+        $radd = '(\b[A-Z0-9._%+-]+@(?:[A-Z0-9-]+\.)+[A-Z]{2,6}\b)';
 
         // get email from address
-        $regex = sprintf('/^From\:.*%s/im',$radd);
-        $hasMatch = preg_match($regex,$contents,$matches);
-        $fromAddress = $hasMatch ? $matches[1]:null;
+        $regex = sprintf('/^From\:.*%s/im', $radd);
+        $hasMatch = preg_match($regex, $contents, $matches);
+        $fromAddress = $hasMatch ? $matches[1] : null;
 
         // get email to address
-        $regex = sprintf('/^To\:\s+%s/im',$radd);
-        $hasMatch = preg_match($regex,$contents,$matches);
-        $toAddress = $hasMatch ? $matches[1]:null;
+        $regex = sprintf('/^To\:\s+%s/im', $radd);
+        $hasMatch = preg_match($regex, $contents, $matches);
+        $toAddress = $hasMatch ? $matches[1] : null;
 
         $this->fromMails[] = $fromAddress;
         $this->toMails[] = $toAddress;
@@ -188,8 +188,8 @@ 
     private function parseSubject($contents)
     {
         $pattern = '/Subject\:(.*)/i';
-        preg_match($pattern,$contents,$matches);
-        $subject = isset($matches[1]) ? $matches[1]:null;
+        preg_match($pattern, $contents, $matches);
+        $subject = isset($matches[1]) ? $matches[1] : null;
         $this->subjects[] = $subject;
         return [
             'subject' => trim($subject)

module/Core/src/Core/Mail/FileTransport.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     {
         $options  = $this->options;
         $filename = call_user_func($options->getCallback(), $this);
-        $file     = $options->getPath() . DIRECTORY_SEPARATOR . $filename;
+        $file     = $options->getPath().DIRECTORY_SEPARATOR.$filename;
 
 
         $contents = $message->toString();

module/Core/src/Core/Mail/MailServiceFactory.php

Spacing +7 added lines, -7 removed lines

@@ -26,7 +26,7 @@ 
  */
 class MailServiceFactory implements FactoryInterface
 {
-	public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
+	public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
 	{
 		$config = $container->get('Config');
 		$mails = isset($config['mails']) ? $config['mails'] : [];
@@ -48,12 +48,12 @@ 
 		$configArray = array_merge($configArray, $mails);
 		
 		$config = new MailServiceConfig($configArray);
-		$service   = new MailService($container,$config->toArray());
+		$service = new MailService($container, $config->toArray());
 		$config->configureServiceManager($service);
-		foreach($config->toArray() as $name=>$value){
+		foreach ($config->toArray() as $name=>$value) {
 			$method = 'set'.$name;
-			if(method_exists($service,$method)){
-				call_user_func([$service,$method],$value);
+			if (method_exists($service, $method)) {
+				call_user_func([$service, $method], $value);
 			}
 		}
 		
@@ -65,11 +65,11 @@ 
         $type = $mailServiceOptions->getTransportClass();
         if (MailService::TRANSPORT_SMTP == $type) {
             return new Smtp($mailServiceOptions);
-        }elseif(MailService::TRANSPORT_FILE == $type){
+        }elseif (MailService::TRANSPORT_FILE == $type) {
             $fileOptions = new FileOptions();
             $fileOptions->setPath($mailServiceOptions->getPath());
             return new FileTransport($fileOptions);
-        }elseif(MailService::TRANSPORT_SENDMAIL == $type){
+        }elseif (MailService::TRANSPORT_SENDMAIL == $type) {
             return new Sendmail();
         }
 

Braces +2 added lines, -2 removed lines

@@ -65,11 +65,11 @@
         $type = $mailServiceOptions->getTransportClass();
         if (MailService::TRANSPORT_SMTP == $type) {
             return new Smtp($mailServiceOptions);
-        }elseif(MailService::TRANSPORT_FILE == $type){
+        } elseif(MailService::TRANSPORT_FILE == $type){
             $fileOptions = new FileOptions();
             $fileOptions->setPath($mailServiceOptions->getPath());
             return new FileTransport($fileOptions);
-        }elseif(MailService::TRANSPORT_SENDMAIL == $type){
+        } elseif(MailService::TRANSPORT_SENDMAIL == $type){
             return new Sendmail();
         }
 

module/Core/src/Core/Options/MailServiceOptions.php

Spacing +8 added lines, -8 removed lines

@@ -99,7 +99,7 @@ 
         return $this;
     }
 
-    public function getUsername(){
+    public function getUsername() {
         return $this->connectionConfig['username'];
     }
 
@@ -110,7 +110,7 @@ 
         return $this;
     }
 
-    public function getPassword(){
+    public function getPassword() {
         return $this->connectionConfig['password'];
     }
 
@@ -121,7 +121,7 @@ 
         return $this;
     }
 
-    public function getSsl(){
+    public function getSsl() {
         return $this->connectionConfig['ssl'];
     }
 
@@ -130,7 +130,7 @@ 
         return $this;
     }
 
-    public function getTransportClass(){
+    public function getTransportClass() {
         return $this->transportClass;
     }
 
@@ -139,7 +139,7 @@ 
      */
     public function getPath()
     {
-        if(is_null($this->path)){
+        if (is_null($this->path)) {
             $this->setPath(sys_get_temp_dir().'/yawik/mails');
         }
         return $this->path;
@@ -151,9 +151,9 @@ 
      */
     public function setPath($path)
     {
-        if(!is_dir($path)){
-            mkdir($path,0777,true);
-            chmod($path,0777);
+        if (!is_dir($path)) {
+            mkdir($path, 0777, true);
+            chmod($path, 0777);
         }
         $this->path = $path;
         return $this;

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,13 +87,13 @@ 
 	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);
+					$repo->remove($user, true);
 					$repo->getDocumentManager()->refresh($user);
-				}catch (\Exception $e){
+				} catch (\Exception $e) {
 				
 				}
 			}
@@ -118,8 +118,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);
 		}
 	}
 	
@@ -127,15 +127,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');
 	}
 	
 	/**
@@ -145,9 +145,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);
 		}
 	}
 	
@@ -156,7 +156,7 @@ 
 	 * @param $role
 	 * @param TableNode $fields
 	 */
-	public function iHaveUserWithTheFollowing($role,TableNode $fields)
+	public function iHaveUserWithTheFollowing($role, TableNode $fields)
 	{
 		$normalizedFields = [
 			'login' => 'test@login.com',
@@ -165,7 +165,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,16 +211,16 @@ 
 	/**
 	 * @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;
@@ -235,7 +235,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 */
@@ -246,10 +246,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);
@@ -269,19 +269,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);
@@ -303,7 +303,7 @@ 
 	 */
 	public function iSpecifyTheUsernameAs($username)
 	{
-		$this->minkContext->fillField('Login name',$username);
+		$this->minkContext->fillField('Login name', $username);
 	}
 	
 	/**
@@ -311,7 +311,7 @@ 
 	 */
 	public function iSpecifyThePasswordAs($password)
 	{
-		$this->minkContext->fillField('Password',$password);
+		$this->minkContext->fillField('Password', $password);
 	}
 	
 	/**
@@ -322,8 +322,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);
@@ -379,13 +379,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);
@@ -393,7 +393,7 @@ 
 	
 	private function addCreatedUser(UserInterface $user)
 	{
-		if(!in_array($user,static::$users)){
+		if (!in_array($user, static::$users)) {
 			static::$users[] = $user;
 		}
 	}

Braces +2 added lines, -2 removed lines

@@ -93,7 +93,7 @@ 
 					JobContext::removeJobByUser($user);
 					$repo->remove($user,true);
 					$repo->getDocumentManager()->refresh($user);
-				}catch (\Exception $e){
+				} catch (\Exception $e){
 				
 				}
 			}
@@ -280,7 +280,7 @@ 
 			$organization->setOrganizationName($organizationName);
 			$permissions = $organization->getPermissions();
 			$permissions->grant($user,Permissions::PERMISSION_ALL);
-		}else {
+		} else {
 			$organization->getPermissions()->grant($user,Permissions::PERMISSION_ALL);
 		}
 		$organization->setUser($user);

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 */
 			$user = $this->getUserContext()->getCurrentUser();
 			$jobRepo->getDocumentManager()->refresh($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)

module/Organizations/config/module.config.php

Spacing +18 added lines, -18 removed lines

@@ -13,7 +13,7 @@ 
                 ],
             ],
             'annotation' => [
-                'paths' => [ __DIR__ . '/../src/Organizations/Entity']
+                'paths' => [__DIR__.'/../src/Organizations/Entity']
             ],
         ],
         'eventmanager' => [
@@ -40,7 +40,7 @@ 
         'translation_file_patterns' => [
             [
                 'type' => 'gettext',
-                'base_dir' => __DIR__ . '/../language',
+                'base_dir' => __DIR__.'/../language',
                 'pattern' => '%s.mo',
             ],
         ],
@@ -49,9 +49,9 @@ 
 
     'controllers' => [
         'factories' => [
-	        'Organizations/InviteEmployee' => [\Organizations\Controller\InviteEmployeeController::class,'factory'],
+	        'Organizations/InviteEmployee' => [\Organizations\Controller\InviteEmployeeController::class, 'factory'],
             'Organizations/Index' => 'Organizations\Factory\Controller\IndexControllerFactory',
-            'Organizations/Profile' => [\Organizations\Controller\ProfileController::class,'factory']
+            'Organizations/Profile' => [\Organizations\Controller\ProfileController::class, 'factory']
         ]
     ],
 
@@ -66,18 +66,18 @@ 
     'view_manager' => [
         // Map template to files. Speeds up the lookup through the template stack.
         'template_map' => [
-             'organizations/index/edit' => __DIR__ . '/../view/organizations/index/form.phtml',
-             'organizations/form/employees-fieldset' => __DIR__ . '/../view/form/employees-fieldset.phtml',
-             'organizations/form/employee-fieldset' => __DIR__ .'/../view/form/employee-fieldset.phtml',
-             'organizations/form/invite-employee-bar' => __DIR__ . '/../view/form/invite-employee-bar.phtml',
-             'organizations/error/no-parent' => __DIR__ . '/../view/error/no-parent.phtml',
-             'organizations/error/invite' => __DIR__ . '/../view/error/invite.phtml',
-             'organizations/mail/invite-employee' => __DIR__ . '/../view/mail/invite-employee.phtml',
-             'organizations/form/workflow-fieldset' => __DIR__ . '/../view/form/workflow-fieldset.phtml',
+             'organizations/index/edit' => __DIR__.'/../view/organizations/index/form.phtml',
+             'organizations/form/employees-fieldset' => __DIR__.'/../view/form/employees-fieldset.phtml',
+             'organizations/form/employee-fieldset' => __DIR__.'/../view/form/employee-fieldset.phtml',
+             'organizations/form/invite-employee-bar' => __DIR__.'/../view/form/invite-employee-bar.phtml',
+             'organizations/error/no-parent' => __DIR__.'/../view/error/no-parent.phtml',
+             'organizations/error/invite' => __DIR__.'/../view/error/invite.phtml',
+             'organizations/mail/invite-employee' => __DIR__.'/../view/mail/invite-employee.phtml',
+             'organizations/form/workflow-fieldset' => __DIR__.'/../view/form/workflow-fieldset.phtml',
         ],
         // Where to look for view templates not mapped above
         'template_path_stack' => [
-            __DIR__ . '/../view',
+            __DIR__.'/../view',
         ],
     ],
     'form_elements' => [
@@ -147,11 +147,11 @@ 
                 'allow' => [
                     'Entity/OrganizationImage',
                     'route/lang/organizations/invite',
-                    'Organizations/InviteEmployee' => [ 'accept' ],
+                    'Organizations/InviteEmployee' => ['accept'],
                 ],
                 'deny' => [
                     'route/lang/organizations',
-                    'Organizations/InviteEmployee' => [ 'invite' ],
+                    'Organizations/InviteEmployee' => ['invite'],
                 ],
             ],
             // recruiters are allowed to view their companies
@@ -159,7 +159,7 @@ 
                 'allow' => [
                     'route/lang/organizations',
                     'Organizations/InviteEmployee',
-                    'Entity/Organization' => [ 'edit' => 'Organizations/Write' ],
+                    'Entity/Organization' => ['edit' => 'Organizations/Write'],
                 ],
             ],
         ],
@@ -175,8 +175,8 @@ 
             'organizations' => [
                 'label' => 'Organizations',
                 'route' => 'lang/organizations',
-                'order' => 65,                             // allows to order the menu items
-                'resource' => 'route/lang/organizations',  // if a resource is defined, the acl will be applied.
+                'order' => 65, // allows to order the menu items
+                'resource' => 'route/lang/organizations', // if a resource is defined, the acl will be applied.
 
                 'pages' => [
                     'list' => [