cross-solution / YAWIK

module/Solr/src/Solr/Paginator/PaginatorFactoryAbstract.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 
     /**
      * @param ServiceLocatorInterface $serviceLocator
-     * @return mixed|Paginator
+     * @return Paginator
      */
     public function createService(ServiceLocatorInterface $serviceLocator)
     {

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
 use Zend\ServiceManager\MutableCreationOptionsInterface;
 use Zend\ServiceManager\ServiceLocatorInterface;
 
-abstract class PaginatorFactoryAbstract implements FactoryInterface,MutableCreationOptionsInterface
+abstract class PaginatorFactoryAbstract implements FactoryInterface, MutableCreationOptionsInterface
 {
     protected $options = [];
 
@@ -47,7 +47,7 @@ 
         $connectPath        = $this->getConnectPath();
         $solrClient         = $serviceLocator->getServiceLocator()->get('Solr/Manager')->getClient($connectPath);
         $resultConverter    = $serviceLocator->getServiceLocator()->get('Solr/ResultConverter');
-        $adapter            = new SolrAdapter($solrClient,$filter,$resultConverter,$this->options);
+        $adapter            = new SolrAdapter($solrClient, $filter, $resultConverter, $this->options);
         $service            = new Paginator($adapter);
 
         $this->setCreationOptions([]);

module/Solr/src/Solr/Bridge/ResultConverter.php

Spacing +13 added lines, -13 removed lines

@@ -42,17 +42,17 @@ 
         $propertiesMap = $filter->getPropertiesMap();
         $class = $filter->getEntityClass();
         $entities = [];
-        foreach($response['response']['docs'] as $doc){
+        foreach ($response['response']['docs'] as $doc) {
             $ob = new $class();
             $properties = $doc->getPropertyNames();
-            foreach($properties as $name){
+            foreach ($properties as $name) {
                 $setter = 'set'.$name;
                 $value = $doc->$name;
                 $value = $this->validateDate($value);
-                if(method_exists($ob,$setter)){
-                    call_user_func(array($ob,$setter),$value);
-                }elseif(isset($propertiesMap[$name])){
-                    $this->handleMappedProperty($propertiesMap[$name],$ob,$value);
+                if (method_exists($ob, $setter)) {
+                    call_user_func(array($ob, $setter), $value);
+                }elseif (isset($propertiesMap[$name])) {
+                    $this->handleMappedProperty($propertiesMap[$name], $ob, $value);
                 }
 
             }
@@ -69,11 +69,11 @@ 
      * @param $object
      * @param $value
      */
-    public function handleMappedProperty($property,$object,$value)
+    public function handleMappedProperty($property, $object, $value)
     {
-        $callback = array($this->filter,$property);
-        if(is_callable($callback)){
-            call_user_func($callback,$object,$value);
+        $callback = array($this->filter, $property);
+        if (is_callable($callback)) {
+            call_user_func($callback, $object, $value);
         }
     }
 
@@ -86,11 +86,11 @@ 
     public function validateDate($value)
     {
         $value = trim($value);
-        $date = \DateTime::createFromFormat(Manager::SOLR_DATE_FORMAT,$value);
+        $date = \DateTime::createFromFormat(Manager::SOLR_DATE_FORMAT, $value);
         $check = $date && ($date->format(Manager::SOLR_DATE_FORMAT) === $value);
-        if($check){
+        if ($check) {
             return $date;
-        }else{
+        } else {
             return $value;
         }
     }

Braces +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
                 $value = $this->validateDate($value);
                 if(method_exists($ob,$setter)){
                     call_user_func(array($ob,$setter),$value);
-                }elseif(isset($propertiesMap[$name])){
+                } elseif(isset($propertiesMap[$name])){
                     $this->handleMappedProperty($propertiesMap[$name],$ob,$value);
                 }
 
@@ -90,7 +90,7 @@ 
         $check = $date && ($date->format(Manager::SOLR_DATE_FORMAT) === $value);
         if($check){
             return $date;
-        }else{
+        } else{
             return $value;
         }
     }

module/Solr/src/Solr/Bridge/Manager.php

Spacing +5 added lines, -5 removed lines

@@ -29,7 +29,7 @@ 
         $this->connectOption = $connectOption;
     }
 
-    public function getClient($path='/solr')
+    public function getClient($path = '/solr')
     {
         $connectOption = $this->connectOption;
         $options = [
@@ -48,15 +48,15 @@ 
      * @param \SolrInputDocument $document
      * @param string $path
      */
-    public function addDocument(\SolrInputDocument $document,$path='/solr')
+    public function addDocument(\SolrInputDocument $document, $path = '/solr')
     {
         $client = $this->getClient($path);
-        try{
+        try {
             $client->addDocument($document);
             $client->commit();
             $client->optimize();
-        }catch (\Exception $e){
-            throw new ServerException('Can not add document to server!',$e->getCode(),$e);
+        } catch (\Exception $e) {
+            throw new ServerException('Can not add document to server!', $e->getCode(), $e);
         }
     }
     

Braces +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
             $client->addDocument($document);
             $client->commit();
             $client->optimize();
-        }catch (\Exception $e){
+        } catch (\Exception $e){
             throw new ServerException('Can not add document to server!',$e->getCode(),$e);
         }
     }

module/Solr/src/Solr/Paginator/Adapter/SolrAdapter.php

Spacing +8 added lines, -8 removed lines

@@ -61,7 +61,7 @@ 
      * @param   ResultConverter             $resultConverter
      * @param   array                       $params
      */
-    public function __construct($client,$filter,$resultConverter,$params=array())
+    public function __construct($client, $filter, $resultConverter, $params = array())
     {
         $this->client           = $client;
         $this->filter           = $filter;
@@ -76,7 +76,7 @@ 
     {
         return $this->resultConverter->convert(
             $this->filter,
-            $this->getResponse($offset,$itemCountPerPage)
+            $this->getResponse($offset, $itemCountPerPage)
         );
     }
 
@@ -99,18 +99,18 @@ 
      * @return  \SolrQueryResponse
      * @throws  \Exception
      */
-    protected function getResponse($offset=0,$itemCountPerPage=5)
+    protected function getResponse($offset = 0, $itemCountPerPage = 5)
     {
-        if(!is_object($this->response)){
+        if (!is_object($this->response)) {
             $query = new \SolrQuery();
-            $query = $this->filter->filter($this->params,$query);
+            $query = $this->filter->filter($this->params, $query);
             $query->setStart($offset);
             $query->setRows($itemCountPerPage);
-            try{
+            try {
                 $this->response = $this->client->query($query);
-            }catch (\Exception $e){
+            } catch (\Exception $e) {
                 $message = 'Failed to process query';
-                throw new ServerException($message,$e->getCode(),$e);
+                throw new ServerException($message, $e->getCode(), $e);
             }
         }
         return $this->response;

Braces +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
             $query->setRows($itemCountPerPage);
             try{
                 $this->response = $this->client->query($query);
-            }catch (\Exception $e){
+            } catch (\Exception $e){
                 $message = 'Failed to process query';
                 throw new ServerException($message,$e->getCode(),$e);
             }

module/Solr/src/Solr/Filter/JobBoardPaginationQuery.php

Spacing +13 added lines, -13 removed lines

@@ -36,27 +36,27 @@ 
      */
     public function createQuery(array $params, $query)
     {
-        $search = isset($params['search']) ? $params['search']:'';
+        $search = isset($params['search']) ? $params['search'] : '';
 
-        if(!empty($search)){
+        if (!empty($search)) {
             $q = 'title:'.$search.' OR organizationName:'.$search;
-        }else{
+        } else {
             $q = '*:*';
         }
 
         $query->setQuery($q);
         
-        if(isset($params['sort'])){
+        if (isset($params['sort'])) {
             $sorts = $this->filterSort($params['sort']);
-            foreach($sorts as $field=>$order){
-                $query->addSortField($field,$order);
+            foreach ($sorts as $field=>$order) {
+                $query->addSortField($field, $order);
             }
         }
 
-        if(isset($params['location'])){
+        if (isset($params['location'])) {
             /* @var Location $location */
             $location = $params['location'];
-            if(!is_null($location->getCoordinates())){
+            if (!is_null($location->getCoordinates())) {
                 $coordinates = $location->getCoordinates()->getCoordinates();
                 $query->addFilterQuery(sprintf(
                     "{!geofilt pt=%s sfield=latLon d=%s}",
@@ -82,9 +82,9 @@ 
      * @param Job       $ob
      * @param string    $value
      */
-    public function convertOrganizationName($ob,$value)
+    public function convertOrganizationName($ob, $value)
     {
-        if(!is_object($ob->getOrganization())){
+        if (!is_object($ob->getOrganization())) {
             $ob->setOrganization(new Organization());
         }
         $orgName = new OrganizationName($value);
@@ -96,12 +96,12 @@ 
      * @param   Job     $ob
      * @param   mixed   $value
      */
-    public function convertCompanyLogo($ob,$value)
+    public function convertCompanyLogo($ob, $value)
     {
-        if(!is_object($ob->getOrganization())){
+        if (!is_object($ob->getOrganization())) {
             $ob->setOrganization(new Organization());
         }
-        $exp    = explode('/',$value);
+        $exp    = explode('/', $value);
         $id     = $exp[3];
         $name   = $exp[4];
         $image = new OrganizationImage();

Braces +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
             if ($this->value['params']['status'] != 'all'){
                 $queryBuilder->field('status.name')->equals($this->value['params']['status']);
             }
-        }else{
+        } else{
             $queryBuilder->field('status.name')->equals(Status::CREATED);
         }
 

module/Solr/src/Solr/Filter/AbstractPaginationQuery.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
     {
         $query = new \SolrQuery();
         
-        return $this->createQuery($value,$query);
+        return $this->createQuery($value, $query);
     }
 
     /**
@@ -101,5 +101,5 @@ 
      * @param   \SolrQuery $query
      * @return  \SolrQuery
      */
-    abstract public function createQuery(array $params,$query);
+    abstract public function createQuery(array $params, $query);
 }
\ No newline at end of file

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     /**
      * Filter query based on given value
      *
-     * @param mixed $value
+     * @param \Zend\Stdlib\Parameters $value
      * @return \SolrQuery
      */
     public function filter($value)

module/Solr/src/Solr/Listener/JobEventSubscriber.php

Doc Comments +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 
     /**
      * @param ServiceLocatorInterface $serviceLocator
-     * @return mixed
+     * @return JobEventSubscriber
      */
     static public function factory(ServiceLocatorInterface $serviceLocator)
     {

Spacing +30 added lines, -30 removed lines

@@ -65,9 +65,9 @@ 
         }
 
         $solrDoc = $this->generateInputDocument($document, new \SolrInputDocument());
-        try{
-            $this->solrManager->addDocument($solrDoc,'/solr/YawikJobs');
-        }catch (\Exception $e){
+        try {
+            $this->solrManager->addDocument($solrDoc, '/solr/YawikJobs');
+        } catch (\Exception $e) {
             // @TODO: What to do when the process failed?
         }
     }
@@ -84,10 +84,10 @@ 
             return;
         }
 
-        $solrDoc = $this->generateInputDocument($document,new \SolrInputDocument());
-        try{
-            $this->solrManager->addDocument($solrDoc,'/solr/YawikJobs');
-        }catch (\Exception $e){
+        $solrDoc = $this->generateInputDocument($document, new \SolrInputDocument());
+        try {
+            $this->solrManager->addDocument($solrDoc, '/solr/YawikJobs');
+        } catch (\Exception $e) {
             // @TODO: What to do when the process failed?
         }
     }
@@ -111,38 +111,38 @@ 
      */
     public function generateInputDocument(Job $job, $document)
     {
-        $document->addField('id',$job->getId());
-        $document->addField('title',$job->getTitle());
-        $document->addField('applicationEmail',$job->getContactEmail());
+        $document->addField('id', $job->getId());
+        $document->addField('title', $job->getTitle());
+        $document->addField('applicationEmail', $job->getContactEmail());
 
-        if($job->getDateCreated()){
+        if ($job->getDateCreated()) {
             $document->addField('dateCreated',
                 $job->getDateCreated()->setTimezone(new \DateTimeZone('UTC'))->format(Manager::SOLR_DATE_FORMAT)
             );
         }
-        if($job->getDateModified()){
+        if ($job->getDateModified()) {
             $document->addField('dateModified',
                 $job->getDateModified()->setTimezone(new \DateTimeZone('UTC'))->format(Manager::SOLR_DATE_FORMAT)
             );
         }
-        if($job->getDatePublishStart()){
+        if ($job->getDatePublishStart()) {
             $document->addField('datePublishStart',
                 $job->getDatePublishStart()->setTimezone(new \DateTimeZone('UTC'))->format(Manager::SOLR_DATE_FORMAT)
             );
         }
 
-        if($job->getDatePublishEnd()){
+        if ($job->getDatePublishEnd()) {
             $document->addField('datePublishEnd',
                 $job->getDatePublishEnd()->setTimezone(new \DateTimeZone('UTC'))->format(Manager::SOLR_DATE_FORMAT)
             );
         }
 
-        $document->addField('isActive',$job->isActive());
-        $document->addField('lang',$job->getLanguage());
+        $document->addField('isActive', $job->isActive());
+        $document->addField('lang', $job->getLanguage());
 
-        $this->processLocation($job,$document);
-        if(!is_null($job->getOrganization())){
-            $this->processOrganization($job,$document);
+        $this->processLocation($job, $document);
+        if (!is_null($job->getOrganization())) {
+            $this->processOrganization($job, $document);
         }
         return $document;
     }
@@ -153,14 +153,14 @@ 
      * @param Job                   $job
      * @param \SolrInputDocument    $document
      */
-    public function processOrganization(Job $job,$document)
+    public function processOrganization(Job $job, $document)
     {
-        if(!is_null($job->getOrganization()->getImage())){
+        if (!is_null($job->getOrganization()->getImage())) {
             $uri = $job->getOrganization()->getImage()->getUri();
-            $document->addField('companyLogo',$uri);
+            $document->addField('companyLogo', $uri);
         }
-        $document->addField('organizationName',$job->getOrganization()->getOrganizationName()->getName());
-        $document->addField('organizationId',$job->getOrganization()->getId());
+        $document->addField('organizationName', $job->getOrganization()->getOrganizationName()->getName());
+        $document->addField('organizationId', $job->getOrganization()->getId());
     }
 
     /**
@@ -168,16 +168,16 @@ 
      * @param Job                $job
      * @param \SolrInputDocument $document
      */
-    public function processLocation(Job $job,$document)
+    public function processLocation(Job $job, $document)
     {
         /* @var \Jobs\Entity\Location $location */
-        foreach($job->getLocations() as $location){
+        foreach ($job->getLocations() as $location) {
             $coord = $location->getCoordinates()->getCoordinates();
-            $document->addField('latLon',doubleval($coord[0]).','.doubleval($coord[1]));
-            $document->addField('postCode',$location->getPostalCode());
-            $document->addField('regionText',$location->getRegion());
+            $document->addField('latLon', doubleval($coord[0]).','.doubleval($coord[1]));
+            $document->addField('postCode', $location->getPostalCode());
+            $document->addField('regionText', $location->getRegion());
         }
 
-        $document->addField('location',$job->getLocation());
+        $document->addField('location', $job->getLocation());
     }
 }
\ No newline at end of file

Braces +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
         $solrDoc = $this->generateInputDocument($document, new \SolrInputDocument());
         try{
             $this->solrManager->addDocument($solrDoc,'/solr/YawikJobs');
-        }catch (\Exception $e){
+        } catch (\Exception $e){
             // @TODO: What to do when the process failed?
         }
     }
@@ -67,7 +67,7 @@ 
         $solrDoc = $this->generateInputDocument($document,new \SolrInputDocument());
         try{
             $this->solrManager->addDocument($solrDoc,'/solr/YawikJobs');
-        }catch (\Exception $e){
+        } catch (\Exception $e){
             // @TODO: What to do when the process failed?
         }
     }

module/Solr/Module.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
         return array(
             'Zend\Loader\StandardAutoloader' => array(
                 'namespaces' => array(
-                    __NAMESPACE__ => __DIR__ . '/src/' . __NAMESPACE__,
+                    __NAMESPACE__ => __DIR__.'/src/'.__NAMESPACE__,
                 ),
             ),
         );

module/Solr/src/Solr/Listener/CreatePaginatorListener.php

Spacing +2 added lines, -2 removed lines

@@ -31,10 +31,10 @@
         $paginatorName = $event->getPaginatorName();
         $paginators = $event->getPaginators();
         $serviceName = 'Solr/'.$paginatorName;
-        if($paginators->has($serviceName)){
+        if ($paginators->has($serviceName)) {
             /* @var \Zend\Paginator\Paginator $paginator */
             // yes, we have that solr paginator to replace $paginatorName
-            $paginator = $paginators->get($serviceName,$params);
+            $paginator = $paginators->get($serviceName, $params);
             $event->setPaginator($paginator);
         }
     }