cross-solution / YAWIK

module/Core/src/Core/Controller/Plugin/Mailer.php

Indentation +5 added lines, -5 removed lines

@@ -87,11 +87,11 @@
         return $sendMail ? $this->send($mail) : $mail;
     }
 	
-	/**
-	 * @param ContainerInterface $container
-	 *
-	 * @return Mailer
-	 */
+    /**
+     * @param ContainerInterface $container
+     *
+     * @return Mailer
+     */
     public static function factory(ContainerInterface $container)
     {
         return new static($container->get('Core/MailService'));

module/Core/src/Core/Entity/AbstractIdentifiableEntity.php

Indentation +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
  *
  * @author Anthonius Munthi <me@itstoni.com>
  * @ODM\MappedSuperclass
-  */
+ */
 abstract class AbstractIdentifiableEntity extends AbstractEntity implements IdentifiableEntityInterface
 {
     use IdentifiableEntityTrait;

module/Behat/src/MailContext.php

Doc Comments +3 added lines

@@ -150,6 +150,9 @@
         );
     }
 
+    /**
+     * @param string $contents
+     */
     private function parseEmail($contents)
     {
         $addresses = $this->parseEmailAddress($contents);

Spacing +26 added lines, -26 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,33 +153,33 @@ 
     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);
-        $toAddress1 = $hasMatch ? $matches[1]:null;
+        $regex = sprintf('/^To\:\s+%s/im', $radd);
+        $hasMatch = preg_match($regex, $contents, $matches);
+        $toAddress1 = $hasMatch ? $matches[1] : null;
 
         // get email to address
-        $regex = sprintf('/^To\:.*%s/im',$radd);
-        $hasMatch = preg_match($regex,$contents,$matches);
-        $toAddress2 = $hasMatch ? $matches[1]:null;
+        $regex = sprintf('/^To\:.*%s/im', $radd);
+        $hasMatch = preg_match($regex, $contents, $matches);
+        $toAddress2 = $hasMatch ? $matches[1] : null;
 
         $this->fromMails[] = $fromAddress;
         $this->toMails[] = $toAddress1;
@@ -187,15 +187,15 @@ 
 
         return [
             'from' => $fromAddress,
-            'to' => [$toAddress1,$toAddress2],
+            'to' => [$toAddress1, $toAddress2],
         ];
     }
 
     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

Indentation +28 added lines, -28 removed lines

@@ -26,41 +26,41 @@
  */
 class MailServiceFactory implements FactoryInterface
 {
-	public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
-	{
-		$config = $container->get('Config');
-		$mails = isset($config['mails']) ? $config['mails'] : [];
+    public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
+    {
+        $config = $container->get('Config');
+        $mails = isset($config['mails']) ? $config['mails'] : [];
 		
-		/* @var \Auth\Options\ModuleOptions $authOptions */
-		$authOptions = $container->get('Auth/Options');
+        /* @var \Auth\Options\ModuleOptions $authOptions */
+        $authOptions = $container->get('Auth/Options');
 		
-		/* @var \Core\Options\MailServiceOptions $mailServiceOptions */
-		$mailServiceOptions = $container->get('Core/MailServiceOptions');
+        /* @var \Core\Options\MailServiceOptions $mailServiceOptions */
+        $mailServiceOptions = $container->get('Core/MailServiceOptions');
 		
-		$configArray = [
-			'from' => [
-				'name' => $authOptions->getFromName(),
-				'email' => $authOptions->getFromEmail()
-			],
-		];
+        $configArray = [
+            'from' => [
+                'name' => $authOptions->getFromName(),
+                'email' => $authOptions->getFromEmail()
+            ],
+        ];
 
-		$configArray['transport'] = $this->getTransport($mailServiceOptions);
-		$configArray = array_merge($configArray, $mails);
+        $configArray['transport'] = $this->getTransport($mailServiceOptions);
+        $configArray = array_merge($configArray, $mails);
 		
-		$config = new MailServiceConfig($configArray);
-		$service   = new MailService($container,$config->toArray());
-		$config->configureServiceManager($service);
-		foreach($config->toArray() as $name=>$value){
-			$method = 'set'.$name;
-			if(method_exists($service,$method)){
-				call_user_func([$service,$method],$value);
-			}
-		}
+        $config = new MailServiceConfig($configArray);
+        $service   = new MailService($container,$config->toArray());
+        $config->configureServiceManager($service);
+        foreach($config->toArray() as $name=>$value){
+            $method = 'set'.$name;
+            if(method_exists($service,$method)){
+                call_user_func([$service,$method],$value);
+            }
+        }
 		
-		return $service;
-	}
+        return $service;
+    }
 
-	public function getTransport(MailServiceOptions $mailServiceOptions)
+    public function getTransport(MailServiceOptions $mailServiceOptions)
     {
         $type = $mailServiceOptions->getTransportClass();
         if (MailService::TRANSPORT_SMTP == $type) {

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/JobContext.php

Doc Comments +3 added lines

@@ -215,6 +215,9 @@
 		$this->buildJob($status,$fields->getRowsHash());
 	}
 
+	/**
+	 * @param \Organizations\Entity\Organization $organization
+	 */
 	public function buildJob($status, $definitions,$organization = null)
     {
         $normalizedField = [

Braces +3 added lines, -3 removed lines

@@ -184,7 +184,7 @@ 
 		$element = $page->find('css',$mapMultiple[$field]);
 		if(!is_null($element) && $element->getAttribute('multiple')=='multiple'){
 			$this->minkContext->selectOption($value,$multipleField);
-		}else{
+		} else{
 			$locator = $mapSelect2[$field];
 			$this->select2Context->iFillInSelect2Field($locator,$value);
 		}
@@ -247,14 +247,14 @@ 
             if($user instanceof User){
                 $job->setUser($user);
                 $job->setOrganization($organization);
-            }else{
+            } else{
                 throw new \Exception('There is no user with this login:"'.$normalizedField['user'.'"']);
             }
         }
 
         if($status == 'draft'){
             $job->setIsDraft(true);
-        }elseif($status == 'published'){
+        } elseif($status == 'published'){
             $job->setIsDraft(false);
             $job->setDatePublishStart(new \DateTime());
         }

Indentation +230 added lines, -230 removed lines

@@ -34,188 +34,188 @@ 
  */
 class JobContext implements Context
 {
-	use CommonContextTrait;
+    use CommonContextTrait;
 	
-	/**
-	 * @var Select2Context
-	 */
-	private $select2Context;
+    /**
+     * @var Select2Context
+     */
+    private $select2Context;
 	
-	/**
-	 * @var Job
-	 */
-	private $currentJob;
+    /**
+     * @var Job
+     */
+    private $currentJob;
 	
-	/**
-	 * @var JobRepository
-	 */
-	static private $jobRepo;
+    /**
+     * @var JobRepository
+     */
+    static private $jobRepo;
 	
-	/**
-	 * @param User $user
-	 */
-	static public function removeJobByUser(User $user)
-	{
-		$repo = static::$jobRepo;
-		$results = $repo->findBy(['user' => $user]);
-		foreach($results as $result){
-			$repo->remove($result,true);
-		}
-	}
+    /**
+     * @param User $user
+     */
+    static public function removeJobByUser(User $user)
+    {
+        $repo = static::$jobRepo;
+        $results = $repo->findBy(['user' => $user]);
+        foreach($results as $result){
+            $repo->remove($result,true);
+        }
+    }
 	
-	/**
-	 * @BeforeScenario
-	 *
-	 * @param BeforeScenarioScope $scope
-	 */
-	public function beforeScenario(BeforeScenarioScope $scope)
-	{
-		$this->select2Context = $scope->getEnvironment()->getContext(Select2Context::class);
-		if(is_null(static::$jobRepo)){
-			$this->gatherContexts($scope);
-			static::$jobRepo = $this->getJobRepository();
-		}
-	}
+    /**
+     * @BeforeScenario
+     *
+     * @param BeforeScenarioScope $scope
+     */
+    public function beforeScenario(BeforeScenarioScope $scope)
+    {
+        $this->select2Context = $scope->getEnvironment()->getContext(Select2Context::class);
+        if(is_null(static::$jobRepo)){
+            $this->gatherContexts($scope);
+            static::$jobRepo = $this->getJobRepository();
+        }
+    }
 	
-	/**
-	 * @Given I go to job board page
-	 */
-	public function iGoToJobBoardPage()
-	{
-		$this->visit('/jobboard');
-	}
+    /**
+     * @Given I go to job board page
+     */
+    public function iGoToJobBoardPage()
+    {
+        $this->visit('/jobboard');
+    }
 	
-	/**
-	 * @Given I go to create job page
-	 */
-	public function iGoToCreateJob()
-	{
-	    $url = $this->buildUrl('lang/jobs/manage',['action' => 'edit']);
-		$this->visit($url);
-	}
+    /**
+     * @Given I go to create job page
+     */
+    public function iGoToCreateJob()
+    {
+        $url = $this->buildUrl('lang/jobs/manage',['action' => 'edit']);
+        $this->visit($url);
+    }
 	
-	/**
-	 * @Given I go to job overview page
-	 */
-	public function iGoToJobOverviewPage()
-	{
-		$this->visit('/jobs');
-	}
+    /**
+     * @Given I go to job overview page
+     */
+    public function iGoToJobOverviewPage()
+    {
+        $this->visit('/jobs');
+    }
 	
-	/**
-	 * @Given I go to edit job draft with title :jobTitle
-	 * @param $jobTitle
-	 * @throws \Exception when job is not found
-	 */
-	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));
-		}
-		$this->currentJob = $job;
-		$url = $this->buildUrl('lang/jobs/manage',[
-		    'id' => $job->getId()
+    /**
+     * @Given I go to edit job draft with title :jobTitle
+     * @param $jobTitle
+     * @throws \Exception when job is not found
+     */
+    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));
+        }
+        $this->currentJob = $job;
+        $url = $this->buildUrl('lang/jobs/manage',[
+            'id' => $job->getId()
         ]);
-		$this->visit($url);
-	}
+        $this->visit($url);
+    }
 	
-	/**
-	 * @Given I don't have any classification data
-	 */
-	public function iDonTHaveAnyClassificationData()
-	{
-		$this->currentJob->setClassifications(new Classifications());
-		$this->getJobRepository()->store($this->currentJob);
-	}
+    /**
+     * @Given I don't have any classification data
+     */
+    public function iDonTHaveAnyClassificationData()
+    {
+        $this->currentJob->setClassifications(new Classifications());
+        $this->getJobRepository()->store($this->currentJob);
+    }
 	
-	/**
-	 * @When I don't have any posted job
-	 */
-	public function iDonTHaveAnyPostedJob()
-	{
-		/* @var $jobRepository JobRepository */
-		/* @var $job Job */
-		$user = $this->getUserContext()->getCurrentUser();
+    /**
+     * @When I don't have any posted job
+     */
+    public function iDonTHaveAnyPostedJob()
+    {
+        /* @var $jobRepository JobRepository */
+        /* @var $job Job */
+        $user = $this->getUserContext()->getCurrentUser();
 
-		$jobRepository = $this->getJobRepository();
-		$results = $jobRepository->getUserJobs($user->getId());
-		foreach($results as $job){
-			$jobRepository->remove($job,true);
-		}
-		$this->currentJob = null;
-	}
+        $jobRepository = $this->getJobRepository();
+        $results = $jobRepository->getUserJobs($user->getId());
+        foreach($results as $job){
+            $jobRepository->remove($job,true);
+        }
+        $this->currentJob = null;
+    }
 	
-	/**
-	 * @When I fill job location search with :search and choose :choice
-	 *
-	 */
-	public function iFillJobLocationAndChoose($search,$choice)
-	{
-		$select2 = $this->select2Context;
-		$select2->iFillInSelect2FieldWith('jobBase[geoLocation]',$search,$choice);
-	}
+    /**
+     * @When I fill job location search with :search and choose :choice
+     *
+     */
+    public function iFillJobLocationAndChoose($search,$choice)
+    {
+        $select2 = $this->select2Context;
+        $select2->iFillInSelect2FieldWith('jobBase[geoLocation]',$search,$choice);
+    }
 	
-	/**
-	 * @When I choose :value from :field
-	 */
-	public function iJobClassificationSelect($value,$field)
-	{
-		$field = Inflector::camelize($field);
+    /**
+     * @When I choose :value from :field
+     */
+    public function iJobClassificationSelect($value,$field)
+    {
+        $field = Inflector::camelize($field);
 		
-		$mapSelect2 = [
-			'professions' => '#classifications-professions-span .select2-container',
-			'industries'  => '#classifications-industries-span .select2-container',
-			'employmentTypes' => '#classifications-employmentTypes-span .select2-container',
-		];
+        $mapSelect2 = [
+            'professions' => '#classifications-professions-span .select2-container',
+            'industries'  => '#classifications-industries-span .select2-container',
+            'employmentTypes' => '#classifications-employmentTypes-span .select2-container',
+        ];
 		
-		$mapMultiple = [
-			'professions'       => "select#classifications-professions",
-			'industries'        => "select#classifications-industries",
-			'employmentTypes'    => "select#classifications-employmentTypes",
-		];
+        $mapMultiple = [
+            'professions'       => "select#classifications-professions",
+            'industries'        => "select#classifications-industries",
+            'employmentTypes'    => "select#classifications-employmentTypes",
+        ];
 		
-		if(!isset($mapSelect2[$field])){
-			throw new \Exception('Undefined field selection value "'.$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{
-			$locator = $mapSelect2[$field];
-			$this->select2Context->iFillInSelect2Field($locator,$value);
-		}
-	}
+        $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{
+            $locator = $mapSelect2[$field];
+            $this->select2Context->iFillInSelect2Field($locator,$value);
+        }
+    }
 	
-	/**
-	 * @return JobRepository
-	 */
-	public function getJobRepository()
-	{
-		return $this->getRepository('Jobs/Job');
-	}
+    /**
+     * @return JobRepository
+     */
+    public function getJobRepository()
+    {
+        return $this->getRepository('Jobs/Job');
+    }
 	
-	/**
-	 * @return CategoriesRepo
-	 */
-	public function getCategoriesRepository()
-	{
-		return $this->getRepository('Jobs/Category');
-	}
+    /**
+     * @return CategoriesRepo
+     */
+    public function getCategoriesRepository()
+    {
+        return $this->getRepository('Jobs/Category');
+    }
 	
-	/**
-	 * @When I have a :status job with the following:
-	 * @param TableNode $fields
-	 */
-	public function iHaveAJobWithTheFollowing($status,TableNode $fields)
-	{
-		$this->buildJob($status,$fields->getRowsHash());
-	}
+    /**
+     * @When I have a :status job with the following:
+     * @param TableNode $fields
+     */
+    public function iHaveAJobWithTheFollowing($status,TableNode $fields)
+    {
+        $this->buildJob($status,$fields->getRowsHash());
+    }
 
-	public function buildJob($status, $definitions,$organization = null)
+    public function buildJob($status, $definitions,$organization = null)
     {
         $normalizedField = [
             'template' => 'modern',
@@ -284,84 +284,84 @@ 
     }
 
 	
-	private function setLocation(Job $job, $term)
-	{
-		/* @var $client Photon */
-		$client = $this->coreContext->getServiceManager()->get('Geo/Client');
-		$result = $client->queryOne($term);
-		$location = new Location();
-		$serialized = Json::encode($result);
-		$location->fromString($serialized);
+    private function setLocation(Job $job, $term)
+    {
+        /* @var $client Photon */
+        $client = $this->coreContext->getServiceManager()->get('Geo/Client');
+        $result = $client->queryOne($term);
+        $location = new Location();
+        $serialized = Json::encode($result);
+        $location->fromString($serialized);
 		
-		$locations = $job->getLocations();
-		if(count($locations)){
-			$locations->clear();
-		}
-		$job->getLocations()->add($location);
-	}
+        $locations = $job->getLocations();
+        if(count($locations)){
+            $locations->clear();
+        }
+        $job->getLocations()->add($location);
+    }
 	
-	private function addProfessions(Job &$job,$terms)
-	{
-		$professions = $this->getCategories($terms);
-		foreach($professions as $profession){
-			$values = $job->getClassifications()->getProfessions()->getValues();
-			if(!is_array($values) || !in_array($profession,$values)){
-				$job->getClassifications()->getProfessions()->getItems()->add($profession);
-			}
-		}
-	}
+    private function addProfessions(Job &$job,$terms)
+    {
+        $professions = $this->getCategories($terms);
+        foreach($professions as $profession){
+            $values = $job->getClassifications()->getProfessions()->getValues();
+            if(!is_array($values) || !in_array($profession,$values)){
+                $job->getClassifications()->getProfessions()->getItems()->add($profession);
+            }
+        }
+    }
 	
-	private function addIndustries(Job &$job, $terms)
-	{
-		$industries = $this->getCategories($terms);
-		foreach($industries as $industry){
-			$values = $job->getClassifications()->getIndustries()->getValues();
-			if(!is_array($values) || !in_array($industry,$values)){
-				$job->getClassifications()->getIndustries()->getItems()->add($industry);
-			}
-		}
-	}
+    private function addIndustries(Job &$job, $terms)
+    {
+        $industries = $this->getCategories($terms);
+        foreach($industries as $industry){
+            $values = $job->getClassifications()->getIndustries()->getValues();
+            if(!is_array($values) || !in_array($industry,$values)){
+                $job->getClassifications()->getIndustries()->getItems()->add($industry);
+            }
+        }
+    }
 	
-	/**
-	 * @param array $categories
-	 *
-	 * @return mixed
-	 */
-	private function getCategories(array $categories)
-	{
-		$catRepo = $this->getCategoriesRepository();
+    /**
+     * @param array $categories
+     *
+     * @return mixed
+     */
+    private function getCategories(array $categories)
+    {
+        $catRepo = $this->getCategoriesRepository();
 		
-		// get a professions
-		$qb = $catRepo->createQueryBuilder()
-		              ->field('name')->in($categories)
-		              ->getQuery()
-		;
-		$results = $qb->execute();
-		return $results->toArray();
-	}
+        // get a professions
+        $qb = $catRepo->createQueryBuilder()
+                        ->field('name')->in($categories)
+                        ->getQuery()
+        ;
+        $results = $qb->execute();
+        return $results->toArray();
+    }
 	
 	
-	/**
-	 * @return Job
-	 */
-	private function getCurrentUserJobDraft($jobTitle)
-	{
-		$repo = $this->getJobRepository();
-		$user = $this->getCurrentUser();
+    /**
+     * @return Job
+     */
+    private function getCurrentUserJobDraft($jobTitle)
+    {
+        $repo = $this->getJobRepository();
+        $user = $this->getCurrentUser();
 		
-		$job = $repo->findDraft($user);
+        $job = $repo->findDraft($user);
 		
-		if(is_null($job)){
-			$job = new Job();
-			$job
-				->setUser($user)
-				->setOrganization($user->getOrganization()->getOrganization())
-				->setStatus(StatusInterface::CREATED)
-			;
-			$job->setIsDraft(true);
-		}
-		$job->setTitle($jobTitle);
-		$repo->store($job);
-		return $job;
-	}
+        if(is_null($job)){
+            $job = new Job();
+            $job
+                ->setUser($user)
+                ->setOrganization($user->getOrganization()->getOrganization())
+                ->setStatus(StatusInterface::CREATED)
+            ;
+            $job->setIsDraft(true);
+        }
+        $job->setTitle($jobTitle);
+        $repo->store($job);
+        return $job;
+    }
 }

Spacing +46 added lines, -46 removed lines

@@ -58,8 +58,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);
 		}
 	}
 	
@@ -71,7 +71,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();
 		}
@@ -90,7 +90,7 @@ 
 	 */
 	public function iGoToCreateJob()
 	{
-	    $url = $this->buildUrl('lang/jobs/manage',['action' => 'edit']);
+	    $url = $this->buildUrl('lang/jobs/manage', ['action' => 'edit']);
 		$this->visit($url);
 	}
 	
@@ -110,11 +110,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->buildUrl('lang/jobs/manage',[
+		$url = $this->buildUrl('lang/jobs/manage', [
 		    'id' => $job->getId()
         ]);
 		$this->visit($url);
@@ -140,8 +140,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;
 	}
@@ -150,16 +150,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);
 		
@@ -175,18 +175,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);
 		}
 	}
 	
@@ -210,20 +210,20 @@ 
 	 * @When I have a :status job with the following:
 	 * @param TableNode $fields
 	 */
-	public function iHaveAJobWithTheFollowing($status,TableNode $fields)
+	public function iHaveAJobWithTheFollowing($status, TableNode $fields)
 	{
-		$this->buildJob($status,$fields->getRowsHash());
+		$this->buildJob($status, $fields->getRowsHash());
 	}
 
-	public function buildJob($status, $definitions,$organization = null)
+	public function buildJob($status, $definitions, $organization = null)
     {
         $normalizedField = [
             'template' => 'modern',
         ];
-        foreach($definitions as $field => $value){
+        foreach ($definitions 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;
         }
@@ -232,49 +232,49 @@ 
         $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(is_null($organization)){
+            if (is_null($organization)) {
                 $organization = $user->getOrganization()->getOrganization();
             }
-            if($user instanceof User){
+            if ($user instanceof User) {
                 $job->setUser($user);
                 $job->setOrganization($organization);
-            }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['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);
             }
         }
@@ -294,18 +294,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);
 			}
 		}
@@ -314,9 +314,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);
 			}
 		}
@@ -351,7 +351,7 @@ 
 		
 		$job = $repo->findDraft($user);
 		
-		if(is_null($job)){
+		if (is_null($job)) {
 			$job = new Job();
 			$job
 				->setUser($user)

module/Organizations/src/Organizations/Entity/Organization.php

Spacing +5 added lines, -5 removed lines

@@ -47,17 +47,17 @@ 
     /**
      * Always enabled even if there are no active jobs
      */
-    const PROFILE_ALWAYS_ENABLE     = 'always';
+    const PROFILE_ALWAYS_ENABLE = 'always';
 
     /**
      * Hide if there are no jobs available
      */
-    const PROFILE_ACTIVE_JOBS       = 'active-jobs';
+    const PROFILE_ACTIVE_JOBS = 'active-jobs';
 
     /**
      * Always disabled profile
      */
-    const PROFILE_DISABLED          = 'disabled';
+    const PROFILE_DISABLED = 'disabled';
 
     /**
      * Event name of post construct event.
@@ -285,7 +285,7 @@ 
      */
     public function getParent($returnSelf = false)
     {
-        return $this->parent ? : ($returnSelf ? $this : null);
+        return $this->parent ?: ($returnSelf ? $this : null);
     }
 
     /**
@@ -516,7 +516,7 @@ 
      */
     public function getPermissionsResourceId()
     {
-        return 'organization:' . $this->getId();
+        return 'organization:'.$this->getId();
     }
 
     /**

module/Organizations/src/Organizations/Form/OrganizationsProfileFieldset.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     public function getHydrator()
     {
         if (!$this->hydrator) {
-            $hydrator           = new EntityHydrator();
+            $hydrator = new EntityHydrator();
             $this->setHydrator($hydrator);
         }
         return $this->hydrator;