cross-solution / YAWIK

module/Behat/src/UserContext.php

Doc Comments +4 added lines, -1 removed lines

@@ -198,6 +198,9 @@ 
 		$this->startLogin($user,'test');
 	}
 	
+	/**
+	 * @param string $password
+	 */
 	private function startLogin(UserInterface $user, $password)
 	{
 		$currentUser = $this->currentUser;
@@ -242,7 +245,7 @@ 
 	 * @param string $fullname
 	 * @param string $role
 	 *
-	 * @return \Auth\Entity\UserInterface
+	 * @return \Core\Entity\EntityInterface
 	 */
 	public function createUser($email,$password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter")
 	{

Braces +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
 				try{
 					JobContext::removeJobByUser($user);
 					$repo->remove($user,true);
-				}catch (\Exception $e){
+				} catch (\Exception $e){
 				
 				}
 			}
@@ -297,7 +297,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);

Indentation +332 added lines, -332 removed lines

@@ -37,384 +37,384 @@
 {
     use CommonContextTrait;
 	
-	/**
-	 * @var User[]
-	 */
-	static private $users = [];
+    /**
+     * @var User[]
+     */
+    static private $users = [];
 	
-	/**
-	 * @var UserRepository
-	 */
-	static private $userRepo;
+    /**
+     * @var UserRepository
+     */
+    static private $userRepo;
 	
-	/**
-	 * @var string
-	 */
-	static private $currentSession;
+    /**
+     * @var string
+     */
+    static private $currentSession;
 	
-	private $socialLoginInfo = [];
+    private $socialLoginInfo = [];
 	
-	/**
-	 * @var UserInterface
-	 */
-	private $loggedInUser;
+    /**
+     * @var UserInterface
+     */
+    private $loggedInUser;
 
     /**
      * @var User
      */
     protected $currentUser;
 	
-	public function __construct($parameters=[])
-	{
-		$defaultLoginInfo = [
-			'facebook' => [
-				'email' => getenv('FACEBOOK_USER_EMAIL'),
-				'pass' => getenv('FACEBOOK_USER_PASSWORD')
-			],
-			'linkedin' => [
-				'session_key-login' => getenv('LINKEDIN_USER_EMAIL'),
-				'session_password-login' => getenv('LINKEDIN_USER_PASSWORD')
-			],
-		];
-		$socialLoginConfig = isset($parameters['social_login_info']) ? $parameters['social_login_info']:[];
-		$this->socialLoginInfo = array_merge($defaultLoginInfo,$socialLoginConfig);
-	}
+    public function __construct($parameters=[])
+    {
+        $defaultLoginInfo = [
+            'facebook' => [
+                'email' => getenv('FACEBOOK_USER_EMAIL'),
+                'pass' => getenv('FACEBOOK_USER_PASSWORD')
+            ],
+            'linkedin' => [
+                'session_key-login' => getenv('LINKEDIN_USER_EMAIL'),
+                'session_password-login' => getenv('LINKEDIN_USER_PASSWORD')
+            ],
+        ];
+        $socialLoginConfig = isset($parameters['social_login_info']) ? $parameters['social_login_info']:[];
+        $this->socialLoginInfo = array_merge($defaultLoginInfo,$socialLoginConfig);
+    }
 
-	/**
-	 * @AfterSuite
-	 * @param AfterSuiteScope $scope
-	 */
-	static public function afterSuite(AfterSuiteScope $scope)
-	{
-		$repo = static::$userRepo;
-		foreach(static::$users as $user){
-			if($repo->findByLogin($user->getLogin())){
-				try{
-					JobContext::removeJobByUser($user);
-					$repo->remove($user,true);
-				}catch (\Exception $e){
+    /**
+     * @AfterSuite
+     * @param AfterSuiteScope $scope
+     */
+    static public function afterSuite(AfterSuiteScope $scope)
+    {
+        $repo = static::$userRepo;
+        foreach(static::$users as $user){
+            if($repo->findByLogin($user->getLogin())){
+                try{
+                    JobContext::removeJobByUser($user);
+                    $repo->remove($user,true);
+                }catch (\Exception $e){
 				
-				}
-			}
-		}
-	}
+                }
+            }
+        }
+    }
 	
-	/**
-	 * @BeforeScenario
-	 * @param BeforeScenarioScope $scope
-	 */
-	public function beforeScenario(BeforeScenarioScope $scope)
-	{
-		$this->minkContext = $scope->getEnvironment()->getContext(MinkContext::class);
-		$this->coreContext = $scope->getEnvironment()->getContext(CoreContext::class);
-		static::$userRepo = $this->getUserRepository();
-	}
+    /**
+     * @BeforeScenario
+     * @param BeforeScenarioScope $scope
+     */
+    public function beforeScenario(BeforeScenarioScope $scope)
+    {
+        $this->minkContext = $scope->getEnvironment()->getContext(MinkContext::class);
+        $this->coreContext = $scope->getEnvironment()->getContext(CoreContext::class);
+        static::$userRepo = $this->getUserRepository();
+    }
 	
-	/**
-	 * @When I fill in login form with :provider user
-	 */
-	public function iSignInWithSocialUser($provider)
-	{
-		$provider = strtolower($provider);
-		$mink = $this->minkContext;
-		foreach($this->socialLoginInfo[$provider] as $field=>$value){
-			$mink->fillField($field,$value);
-		}
-	}
+    /**
+     * @When I fill in login form with :provider user
+     */
+    public function iSignInWithSocialUser($provider)
+    {
+        $provider = strtolower($provider);
+        $mink = $this->minkContext;
+        foreach($this->socialLoginInfo[$provider] as $field=>$value){
+            $mink->fillField($field,$value);
+        }
+    }
 	
-	/**
-	 * @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)
-	{
-		$user = $this->thereIsAUserIdentifiedBy(
-			'test@recruiter.com',
-			'test',User::ROLE_RECRUITER,
-			'Test Recruiter',
-			$organization
-		);
-		$this->startLogin($user,'test');
-	}
+    /**
+     * @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)
+    {
+        $user = $this->thereIsAUserIdentifiedBy(
+            'test@recruiter.com',
+            'test',User::ROLE_RECRUITER,
+            'Test Recruiter',
+            $organization
+        );
+        $this->startLogin($user,'test');
+    }
 	
-	/**
-	 * @Given I don't have :login user
-	 * @param string $login
-	 */
-	public function iDonTHaveUser($login)
-	{
-		$repo = $this->getUserRepository();
-		$user=$repo->findByLogin($login);
-		if($user instanceof UserInterface){
-			$repo->remove($user,true);
-		}
-	}
+    /**
+     * @Given I don't have :login user
+     * @param string $login
+     */
+    public function iDonTHaveUser($login)
+    {
+        $repo = $this->getUserRepository();
+        $user=$repo->findByLogin($login);
+        if($user instanceof UserInterface){
+            $repo->remove($user,true);
+        }
+    }
 	
-	/**
-	 * @Given I have a :role with the following:
-	 * @param $role
-	 * @param TableNode $fields
-	 */
-	public function iHaveUserWithTheFollowing($role,TableNode $fields)
-	{
-		$normalizedFields = [
-			'login' => 'test@login.com',
-			'fullname' => 'Test Login',
-			'role' => User::ROLE_USER,
-			'password' => 'test',
-			'organization' => 'Cross Solution'
-		];
-		foreach($fields->getRowsHash() as $field=>$value){
-			$field = Inflector::camelize($field);
-			$normalizedFields[$field] = $value;
-		}
+    /**
+     * @Given I have a :role with the following:
+     * @param $role
+     * @param TableNode $fields
+     */
+    public function iHaveUserWithTheFollowing($role,TableNode $fields)
+    {
+        $normalizedFields = [
+            'login' => 'test@login.com',
+            'fullname' => 'Test Login',
+            'role' => User::ROLE_USER,
+            'password' => 'test',
+            'organization' => 'Cross Solution'
+        ];
+        foreach($fields->getRowsHash() as $field=>$value){
+            $field = Inflector::camelize($field);
+            $normalizedFields[$field] = $value;
+        }
 		
-		$this->thereIsAUserIdentifiedBy(
-			$normalizedFields['login'],
-			$normalizedFields['password'],
-			$role,
-			$normalizedFields['fullname'],
-			$normalizedFields['organization']
-		);
+        $this->thereIsAUserIdentifiedBy(
+            $normalizedFields['login'],
+            $normalizedFields['password'],
+            $role,
+            $normalizedFields['fullname'],
+            $normalizedFields['organization']
+        );
 		
-	}
+    }
 	
-	/**
-	 * @Given I am logged in as an administrator
-	 */
-	public function iAmLoggedInAsAnAdmin()
-	{
-		$user = $this->thereIsAUserIdentifiedBy('test@admin.com','test',User::ROLE_ADMIN);
-		$this->startLogin($user,'test');
-	}
+    /**
+     * @Given I am logged in as an administrator
+     */
+    public function iAmLoggedInAsAnAdmin()
+    {
+        $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()){
-			$this->iWantToLogIn();
-			$this->iSpecifyTheUsernameAs($user->getLogin());
-			$this->iSpecifyThePasswordAs($password);
-			$this->iLogIn();
-			$this->currentUser = $user;
-		}
-	}
+    private function startLogin(UserInterface $user, $password)
+    {
+        $currentUser = $this->currentUser;
+        if(!is_object($currentUser) || $user->getId()!=$currentUser->getId()){
+            $this->iWantToLogIn();
+            $this->iSpecifyTheUsernameAs($user->getLogin());
+            $this->iSpecifyThePasswordAs($password);
+            $this->iLogIn();
+            $this->currentUser = $user;
+        }
+    }
 	
-	/**
-	 * @return UserRepository
-	 */
-	public function getUserRepository()
-	{
-		return $this->coreContext->getRepositories()->get('Auth\Entity\User');
-	}
+    /**
+     * @return UserRepository
+     */
+    public function getUserRepository()
+    {
+        return $this->coreContext->getRepositories()->get('Auth\Entity\User');
+    }
 	
-	/**
-	 * @Given there is a user :email identified by :password
-	 */
-	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);
-		}
+    /**
+     * @Given there is a user :email identified by :password
+     */
+    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_null($organization)){
-			$this->iHaveMainOrganization($user,$organization);
-		}
-		$this->addCreatedUser($user);
-		return $user;
-	}
+        if(!is_null($organization)){
+            $this->iHaveMainOrganization($user,$organization);
+        }
+        $this->addCreatedUser($user);
+        return $user;
+    }
 	
-	/**
-	 * @param $email
-	 * @param $password
-	 * @param $username
-	 * @param string $fullname
-	 * @param string $role
-	 *
-	 * @return \Auth\Entity\UserInterface
-	 */
-	public function createUser($email,$password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter")
-	{
-		/* @var Register $service */
-		/* @var User $user */
-		$repo = $this->getUserRepository();
-		$user = $repo->create([]);
-		$user->setLogin($email);
-		$user->setPassword($password);
-		$user->setRole($role);
-		$settings = $user->getSettings('Applications');
+    /**
+     * @param $email
+     * @param $password
+     * @param $username
+     * @param string $fullname
+     * @param string $role
+     *
+     * @return \Auth\Entity\UserInterface
+     */
+    public function createUser($email,$password,$role=User::ROLE_RECRUITER,$fullname="Test Recruiter")
+    {
+        /* @var Register $service */
+        /* @var User $user */
+        $repo = $this->getUserRepository();
+        $user = $repo->create([]);
+        $user->setLogin($email);
+        $user->setPassword($password);
+        $user->setRole($role);
+        $settings = $user->getSettings('Applications');
 		
-		$expFullName = explode(' ',$fullname);
-		$info = $user->getInfo();
-		$info->setFirstName(array_shift($expFullName));
-		$info->setLastName(count($expFullName)>0 ? implode(' ',$expFullName):'');
-		$info->setEmail($email);
-		$info->setEmailVerified(true);
-		$repo->store($user);
-		$repo->getDocumentManager()->refresh($user);
+        $expFullName = explode(' ',$fullname);
+        $info = $user->getInfo();
+        $info->setFirstName(array_shift($expFullName));
+        $info->setLastName(count($expFullName)>0 ? implode(' ',$expFullName):'');
+        $info->setEmail($email);
+        $info->setEmailVerified(true);
+        $repo->store($user);
+        $repo->getDocumentManager()->refresh($user);
 		
-		$eventArgs = new LifecycleEventArgs($user, $repo->getDocumentManager());
-		$repo->getDocumentManager()->getEventManager()->dispatchEvent(
-			Events::postLoad,
-			$eventArgs
-		);
-		/* @var \Core\EventManager\EventManager $events */
-		/* @var \Auth\Listener\Events\AuthEvent $event */
-		//@TODO: [Behat] event not working in travis
-		//$events = $this->coreContext->getEventManager();
-		//$event  = $events->getEvent(AuthEvent::EVENT_USER_REGISTERED, $this);
-		//$event->setUser($user);
-		//$events->triggerEvent($event);
-		return $user;
-	}
+        $eventArgs = new LifecycleEventArgs($user, $repo->getDocumentManager());
+        $repo->getDocumentManager()->getEventManager()->dispatchEvent(
+            Events::postLoad,
+            $eventArgs
+        );
+        /* @var \Core\EventManager\EventManager $events */
+        /* @var \Auth\Listener\Events\AuthEvent $event */
+        //@TODO: [Behat] event not working in travis
+        //$events = $this->coreContext->getEventManager();
+        //$event  = $events->getEvent(AuthEvent::EVENT_USER_REGISTERED, $this);
+        //$event->setUser($user);
+        //$events->triggerEvent($event);
+        return $user;
+    }
 	
-	/**
-	 * @When I have :organization as my main organization
-	 * @param $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 = 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);
-		}
-		$organization->setUser($user);
-		$repoOrganization->store($organization);
-		$repoOrganization->getDocumentManager()->refresh($organization);
-	}
+    /**
+     * @When I have :organization as my main organization
+     * @param $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 = 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);
+        }
+        $organization->setUser($user);
+        $repoOrganization->store($organization);
+        $repoOrganization->getDocumentManager()->refresh($organization);
+    }
 	
-	/**
-	 * @When I want to log in
-	 */
-	public function iWantToLogIn()
-	{
-		$session = $this->minkContext->getSession();
-		$url = $this->generateUrl('lang/auth');
-		$session->visit($url);
-	}
+    /**
+     * @When I want to log in
+     */
+    public function iWantToLogIn()
+    {
+        $session = $this->minkContext->getSession();
+        $url = $this->generateUrl('lang/auth');
+        $session->visit($url);
+    }
 	
-	/**
-	 * @When I specify the username as :username
-	 */
-	public function iSpecifyTheUsernameAs($username)
-	{
-		$this->minkContext->fillField('Login name',$username);
-	}
+    /**
+     * @When I specify the username as :username
+     */
+    public function iSpecifyTheUsernameAs($username)
+    {
+        $this->minkContext->fillField('Login name',$username);
+    }
 	
-	/**
-	 * @When I specify the password as :password
-	 */
-	public function iSpecifyThePasswordAs($password)
-	{
-		$this->minkContext->fillField('Password',$password);
-	}
+    /**
+     * @When I specify the password as :password
+     */
+    public function iSpecifyThePasswordAs($password)
+    {
+        $this->minkContext->fillField('Password',$password);
+    }
 	
-	/**
-	 * @Given I am logged in as :username identified by :password
-	 */
-	public function iAmLoggedInAsIdentifiedBy($username, $password)
-	{
-		$repo = $this->getUserRepository();
-		$user = $repo->findByLogin($username);
+    /**
+     * @Given I am logged in as :username identified by :password
+     */
+    public function iAmLoggedInAsIdentifiedBy($username, $password)
+    {
+        $repo = $this->getUserRepository();
+        $user = $repo->findByLogin($username);
 		
-		if(!$user instanceof User){
-			throw new \Exception(sprintf('There is no user with this login: "%s"',$username));
-		}
-		$this->iWantToLogIn();
-		$this->iSpecifyTheUsernameAs($username);
-		$this->iSpecifyThePasswordAs($password);
-		$this->iLogIn();
+        if(!$user instanceof User){
+            throw new \Exception(sprintf('There is no user with this login: "%s"',$username));
+        }
+        $this->iWantToLogIn();
+        $this->iSpecifyTheUsernameAs($username);
+        $this->iSpecifyThePasswordAs($password);
+        $this->iLogIn();
         $this->currentUser = $user;
-	}
+    }
 	
-	/**
-	 * @When I log in
-	 */
-	public function iLogIn()
-	{
-		$this->minkContext->pressButton('login');
-	}
+    /**
+     * @When I log in
+     */
+    public function iLogIn()
+    {
+        $this->minkContext->pressButton('login');
+    }
 	
-	/**
-	 * @When I press logout link
-	 */
-	public function iPressLogoutLink()
-	{
-		$url = $this->generateUrl('auth-logout');
-		$this->visit($url);
-	}
+    /**
+     * @When I press logout link
+     */
+    public function iPressLogoutLink()
+    {
+        $url = $this->generateUrl('auth-logout');
+        $this->visit($url);
+    }
 	
-	/**
-	 * @Given I log in with username :username and password :password
-	 */
-	public function iLogInWith($username, $password)
-	{
-		$repo = $this->getUserRepository();
-		$user = $repo->findByLogin($username);
-		$this->iWantToLogIn();
-		$this->iSpecifyTheUsernameAs($username);
-		$this->iSpecifyThePasswordAs($password);
-		$this->iLogIn();
-		$this->loggedInUser = $user;
-	}
+    /**
+     * @Given I log in with username :username and password :password
+     */
+    public function iLogInWith($username, $password)
+    {
+        $repo = $this->getUserRepository();
+        $user = $repo->findByLogin($username);
+        $this->iWantToLogIn();
+        $this->iSpecifyTheUsernameAs($username);
+        $this->iSpecifyThePasswordAs($password);
+        $this->iLogIn();
+        $this->loggedInUser = $user;
+    }
 	
-	/**
-	 * @When I go to profile page
-	 */
-	public function iGoToProfilePage()
-	{
-		$url = $this->generateUrl('lang/my');
-		$this->visit($url);
-	}
+    /**
+     * @When I go to profile page
+     */
+    public function iGoToProfilePage()
+    {
+        $url = $this->generateUrl('lang/my');
+        $this->visit($url);
+    }
 	
-	/**
-	 * @Given there is a user with the following:
-	 */
-	public function thereIsAUserWithTheFollowing(TableNode $table)
-	{
-		$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;
+    /**
+     * @Given there is a user with the following:
+     */
+    public function thereIsAUserWithTheFollowing(TableNode $table)
+    {
+        $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;
 		
-		if(!is_object($user=$repo->findByLogin($email))){
-			$user = $this->createUser($email,$password,$role,$fullname);
-		}
-		$this->currentUser = $user;
-		$this->addCreatedUser($user);
-	}
+        if(!is_object($user=$repo->findByLogin($email))){
+            $user = $this->createUser($email,$password,$role,$fullname);
+        }
+        $this->currentUser = $user;
+        $this->addCreatedUser($user);
+    }
 	
-	private function addCreatedUser(UserInterface $user)
-	{
-		if(!in_array($user,static::$users)){
-			static::$users[] = $user;
-		}
-	}
+    private function addCreatedUser(UserInterface $user)
+    {
+        if(!in_array($user,static::$users)){
+            static::$users[] = $user;
+        }
+    }
 	
-	/**
-	 * @When I want to change my password
-	 */
-	public function iWantToChangeMyPassword()
-	{
-		$url = $this->generateUrl('lang/my-password');
-		$this->visit($url);
-	}
+    /**
+     * @When I want to change my password
+     */
+    public function iWantToChangeMyPassword()
+    {
+        $url = $this->generateUrl('lang/my-password');
+        $this->visit($url);
+    }
 
     /**
      * @return User
      */
-	public function getCurrentUser()
+    public function getCurrentUser()
     {
         return $this->currentUser;
     }

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/Core/src/Core/Controller/Plugin/CreatePaginator.php

Doc Comments -1 removed lines

@@ -128,7 +128,6 @@
     }
     
     /**
-     * @param ControllerManager $controllerManager
      * @return CreatePaginator
      * @codeCoverageIgnore
      */

Indentation +10 added lines, -10 removed lines

@@ -44,12 +44,12 @@ 
      */
     protected $request;
 	
-	/**
-	 * CreatePaginator constructor.
-	 *
-	 * @param ContainerInterface $container
-	 * @param HttpRequest $request
-	 */
+    /**
+     * CreatePaginator constructor.
+     *
+     * @param ContainerInterface $container
+     * @param HttpRequest $request
+     */
     public function __construct(ContainerInterface $container, HttpRequest $request)
     {
         $this->serviceManager = $container->get('ServiceManager');
@@ -105,9 +105,9 @@ 
         /* @var \Core\EventManager\EventManager $events */
         /* @var \Zend\Paginator\Paginator $paginator */
         /* @var CreatePaginatorEvent $event */
-	    $events = $this->serviceManager->get('Core/CreatePaginator/Events');
+        $events = $this->serviceManager->get('Core/CreatePaginator/Events');
 
-	    $event = $events->getEvent(CreatePaginatorEvent::EVENT_CREATE_PAGINATOR,$this,[
+        $event = $events->getEvent(CreatePaginatorEvent::EVENT_CREATE_PAGINATOR,$this,[
             'paginatorParams' => $params,
             'paginators' => $paginators,
             'paginatorName' => $paginatorName
@@ -121,8 +121,8 @@ 
             $paginator = $paginators->get($paginatorName,$params);
         }
         $paginator->setCurrentPageNumber(isset($params['page']) ? $params['page'] : 1)
-                  ->setItemCountPerPage(isset($params['count']) ? $params['count'] : 10)
-                  ->setPageRange(isset($params['range']) ? $params['range'] : 5);
+                    ->setItemCountPerPage(isset($params['count']) ? $params['count'] : 10)
+                    ->setPageRange(isset($params['range']) ? $params['range'] : 5);
 
         return $paginator;
     }

Spacing +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
         /* @var CreatePaginatorEvent $event */
 	    $events = $this->serviceManager->get('Core/CreatePaginator/Events');
 
-	    $event = $events->getEvent(CreatePaginatorEvent::EVENT_CREATE_PAGINATOR,$this,[
+	    $event = $events->getEvent(CreatePaginatorEvent::EVENT_CREATE_PAGINATOR, $this, [
             'paginatorParams' => $params,
             'paginators' => $paginators,
             'paginatorName' => $paginatorName
@@ -116,9 +116,9 @@ 
 
         $paginator = $event->getPaginator();
 
-        if(!$paginator instanceof Paginator){
+        if (!$paginator instanceof Paginator) {
             // no paginator created by listener, so let's create default paginator
-            $paginator = $paginators->get($paginatorName,$params);
+            $paginator = $paginators->get($paginatorName, $params);
         }
         $paginator->setCurrentPageNumber(isset($params['page']) ? $params['page'] : 1)
                   ->setItemCountPerPage(isset($params['count']) ? $params['count'] : 10)

module/Core/src/Core/EventManager/EventManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -60,8 +60,8 @@
      *
      * If no event instance is passed, it creates one prior to triggering.
      *
-     * @param EventInterface|string $eventName
-     * @param object|string|null $target
+     * @param EventInterface $eventName
+     * @param \Core\Controller\AdminController $target
      * @param array $argv
      *
      * @return \Zend\EventManager\ResponseCollection

Indentation +3 added lines, -3 removed lines

@@ -55,8 +55,8 @@ 
         return $event;
     }
 	
-	/**
-	 * Trigger an event.
+    /**
+     * Trigger an event.
      *
      * If no event instance is passed, it creates one prior to triggering.
      *
@@ -65,7 +65,7 @@ 
      * @param array $argv
      *
      * @return \Zend\EventManager\ResponseCollection
-	 */
+     */
     public function trigger($eventName, $target = null, $argv = [])
     {
         $event = $eventName instanceOf EventInterface

module/Core/src/Core/Factory/Service/RestClientFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     }
 
     /**
-     * @return mixed
+     * @return null|string
      */
     abstract protected function getUri();
 

Indentation +7 added lines, -7 removed lines

@@ -38,15 +38,15 @@
      */
     protected $serviceLocator;
 	
-	public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
-	{
-		$this->serviceLocator = $container;
-		$service = new RestClient($this->getUri(), $this->getConfig());
-		return $service;
-	}
+    public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
+    {
+        $this->serviceLocator = $container;
+        $service = new RestClient($this->getUri(), $this->getConfig());
+        return $service;
+    }
 	
 	
-	/**
+    /**
      * Create the settings service
      *
      * @param  ServiceLocatorInterface $serviceLocator

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
      */
     protected $serviceLocator;
 	
-	public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
+	public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
 	{
 		$this->serviceLocator = $container;
 		$service = new RestClient($this->getUri(), $this->getConfig());
@@ -54,7 +54,7 @@ 
      */
     public function createService(ServiceLocatorInterface $serviceLocator)
     {
-        return $this($serviceLocator,RestClient::class);
+        return $this($serviceLocator, RestClient::class);
     }
 
     /**

module/Core/src/Core/I18n/LocaleFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      * @param  string             $requestedName
      * @param  null|array         $options
      *
-     * @return object
+     * @return Locale
      * @throws ServiceNotFoundException if unable to resolve the service.
      * @throws ServiceNotCreatedException if an exception is raised when
      *     creating a service.

module/Core/src/Core/Paginator/PaginatorFactoryAbstract.php

Doc Comments -1 removed lines

@@ -41,7 +41,6 @@
     }
 	
 	/**
-     * @param ContainerInterface $serviceLocator
      * @return mixed|Paginator
      */
     public function createService(ContainerInterface $container)

Indentation +10 added lines, -10 removed lines

@@ -28,19 +28,19 @@
 
     public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
     {
-	    /* @var PaginatorService $paginatorService */
-	    /* @var RepositoryService $repositories */
-	    $repositories   = $container->get('repositories');
-	    $repository     = $repositories->get($this->getRepository());
-	    $queryBuilder   = $repository->createQueryBuilder();
-	    $filter         = $container->get('FilterManager')->get($this->getFilter());
-	    $adapter        = new \Core\Paginator\Adapter\DoctrineMongoLateCursor($queryBuilder, $filter, $options);
-	    $service        = new Paginator($adapter);
+        /* @var PaginatorService $paginatorService */
+        /* @var RepositoryService $repositories */
+        $repositories   = $container->get('repositories');
+        $repository     = $repositories->get($this->getRepository());
+        $queryBuilder   = $repository->createQueryBuilder();
+        $filter         = $container->get('FilterManager')->get($this->getFilter());
+        $adapter        = new \Core\Paginator\Adapter\DoctrineMongoLateCursor($queryBuilder, $filter, $options);
+        $service        = new Paginator($adapter);
 	
-	    return $service;
+        return $service;
     }
 	
-	/**
+    /**
      * @param ContainerInterface $serviceLocator
      * @return mixed|Paginator
      */

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
 abstract class PaginatorFactoryAbstract implements FactoryInterface
 {
 
-    public function __invoke( ContainerInterface $container, $requestedName, array $options = null )
+    public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
     {
 	    /* @var PaginatorService $paginatorService */
 	    /* @var RepositoryService $repositories */

module/Core/src/Core/View/Helper/Services.php

Doc Comments -1 removed lines

@@ -86,7 +86,6 @@
     }
     
     /**
-     * @param HelperPluginManager $helperPluginManager
      * @return Services
      */
     public static function factory($sm)

config/common.modules.php

Indentation +19 added lines, -19 removed lines

@@ -11,23 +11,23 @@
  */
 
 return array(
-	'Zend\ServiceManager\Di',
-	'Zend\Session',
-	'Zend\Router',
-	'Zend\Navigation',
-	'Zend\I18n',
-	'Zend\Filter',
-	'Zend\InputFilter',
-	'Zend\Form',
-	'Zend\Validator',
-	'Zend\Log',
-	'Zend\Mvc\Plugin\Prg',
-	'Zend\Mvc\Plugin\Identity',
-	'Zend\Mvc\Plugin\FlashMessenger',
-	'Zend\Mvc\I18n',
-	'Zend\Mvc\Console',
-	'Zend\Hydrator',
-	'Zend\Serializer',
-	'DoctrineModule',
-	'DoctrineMongoODMModule',
+    'Zend\ServiceManager\Di',
+    'Zend\Session',
+    'Zend\Router',
+    'Zend\Navigation',
+    'Zend\I18n',
+    'Zend\Filter',
+    'Zend\InputFilter',
+    'Zend\Form',
+    'Zend\Validator',
+    'Zend\Log',
+    'Zend\Mvc\Plugin\Prg',
+    'Zend\Mvc\Plugin\Identity',
+    'Zend\Mvc\Plugin\FlashMessenger',
+    'Zend\Mvc\I18n',
+    'Zend\Mvc\Console',
+    'Zend\Hydrator',
+    'Zend\Serializer',
+    'DoctrineModule',
+    'DoctrineMongoODMModule',
 );
\ No newline at end of file

module/Install/src/Tracy.php

Indentation +11 added lines, -11 removed lines

@@ -26,18 +26,18 @@
      */
     public function register(array $config)
     {
-    	try{
-		    // enable logging of all error types globally
-		    Debugger::enable($config['mode'], $config['log'], $config['email']);
-		    Debugger::$strictMode = $config['strict'];
-		    Debugger::$showBar = $config['bar'];
+        try{
+            // enable logging of all error types globally
+            Debugger::enable($config['mode'], $config['log'], $config['email']);
+            Debugger::$strictMode = $config['strict'];
+            Debugger::$showBar = $config['bar'];
 		
-		    /** @var \Tracy\Logger $logger */
-		    $logger = Debugger::getLogger();
-		    $logger->emailSnooze = $config['email_snooze'];
-	    }catch (\Exception $e){
-    		throw $e;
-	    }
+            /** @var \Tracy\Logger $logger */
+            $logger = Debugger::getLogger();
+            $logger->emailSnooze = $config['email_snooze'];
+        }catch (\Exception $e){
+            throw $e;
+        }
      
     }
 }

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function register(array $config)
     {
-    	try{
+    	try {
 		    // enable logging of all error types globally
 		    Debugger::enable($config['mode'], $config['log'], $config['email']);
 		    Debugger::$strictMode = $config['strict'];
@@ -35,7 +35,7 @@ 
 		    /** @var \Tracy\Logger $logger */
 		    $logger = Debugger::getLogger();
 		    $logger->emailSnooze = $config['email_snooze'];
-	    }catch (\Exception $e){
+	    } catch (\Exception $e) {
     		throw $e;
 	    }
      

Braces +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
 		    /** @var \Tracy\Logger $logger */
 		    $logger = Debugger::getLogger();
 		    $logger->emailSnooze = $config['email_snooze'];
-	    }catch (\Exception $e){
+	    } catch (\Exception $e){
     		throw $e;
 	    }