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/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

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);
             }

Spacing +8 added lines, -8 removed lines

@@ -70,7 +70,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;
@@ -85,7 +85,7 @@ 
     {
         return $this->resultConverter->convert(
             $this->filter,
-            $this->getResponse($offset,$itemCountPerPage)
+            $this->getResponse($offset, $itemCountPerPage)
         );
     }
 
@@ -108,19 +108,19 @@ 
      * @return  \SolrQueryResponse
      * @throws  \Exception
      */
-    protected function getResponse($offset=0,$itemCountPerPage=0)
+    protected function getResponse($offset = 0, $itemCountPerPage = 0)
     {
         $id = md5($offset.$itemCountPerPage);
-        if(!isset($this->responses[$id])){
+        if (!isset($this->responses[$id])) {
             $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->responses[$id] = $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);
             }
         }
 

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

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);
         }
 

Indentation +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
                     )); // join
 
                 $query->addField('locations:[subquery]')
-                      ->addField('distance:min(geodist(points,'.$coordinate.'))');
+                        ->addField('distance:min(geodist(points,'.$coordinate.'))');
 
             }
 

Spacing +5 added lines, -5 removed lines

@@ -40,11 +40,11 @@ 
      */
     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 = \SolrUtils::escapeQueryChars($search);
-        }else{
+        } else {
             $q = '*:*';
         }
 
@@ -53,10 +53,10 @@ 
         $query->addFilterQuery('isActive:1');
         $query->addField('*');
         
-        if(isset($params['location'])){
+        if (isset($params['location'])) {
             /* @var Location $location */
             $location = $params['location'];
-            if(is_object($location->getCoordinates())){
+            if (is_object($location->getCoordinates())) {
                 $coordinate = Util::convertLocationCoordinates($location);
 
                 $query->addFilterQuery(

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 +2 added lines, -2 removed lines

@@ -55,8 +55,8 @@
     /**
      * Filter query based on given value
      *
-     * @param mixed $value
-     * @return \SolrDisMaxQuery
+     * @param \Zend\Stdlib\Parameters $value
+     * @return \SolrParams
      */
     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)
     {

Braces +2 added lines, -2 removed lines

@@ -97,7 +97,7 @@ 
         $solrDoc = $this->generateInputDocument($document, new \SolrInputDocument());
         try{
             $this->solrManager->addDocument($solrDoc,$this->solrManager->getOptions()->getJobsPath());
-        }catch (\Exception $e){
+        } catch (\Exception $e){
             // @TODO: What to do when the process failed?
         }
     }
@@ -140,7 +140,7 @@ 
         if(!is_null($job->getOrganization())){
             try {
                 $this->processOrganization($job,$document);
-            }catch (\Exception $e){
+            } catch (\Exception $e){
                 // @TODO: What to do when the process failed?
             }
         }

Spacing +37 added lines, -37 removed lines

@@ -91,13 +91,13 @@ 
      */
     protected function updateIndex($document)
     {
-        if(!$document instanceof Job){
+        if (!$document instanceof Job) {
             return;
         }
         $solrDoc = $this->generateInputDocument($document, new \SolrInputDocument());
-        try{
-            $this->solrManager->addDocument($solrDoc,$this->solrManager->getOptions()->getJobsPath());
-        }catch (\Exception $e){
+        try {
+            $this->solrManager->addDocument($solrDoc, $this->solrManager->getOptions()->getJobsPath());
+        } catch (\Exception $e) {
             // @TODO: What to do when the process failed?
         }
     }
@@ -110,41 +110,41 @@ 
      */
     public function generateInputDocument(Job $job, $document)
     {
-        $document->addField('id',$job->getId());
-        $document->addField('applyId',$job->getApplyId());
-        $document->addField('entityName','job');
-        $document->addField('title',$job->getTitle());
-        $document->addField('applicationEmail',$job->getContactEmail());
+        $document->addField('id', $job->getId());
+        $document->addField('applyId', $job->getApplyId());
+        $document->addField('entityName', 'job');
+        $document->addField('title', $job->getTitle());
+        $document->addField('applicationEmail', $job->getContactEmail());
         if ($job->getLink()) {
             $document->addField('link', $job->getLink());
-            $oldErrorReporting=error_reporting(0);
+            $oldErrorReporting = error_reporting(0);
             $dom = new \DOMDocument();
             // @TODO: 
             // we temporary put the html into the index. This will be removed later. For now it's the easiest way to get
             // some searchable content
             $dom->loadHTMLFile($job->getLink());
             error_reporting($oldErrorReporting);
-            $document->addField('html',$dom->saveHTML());
+            $document->addField('html', $dom->saveHTML());
         }
-        if($job->getDateCreated()){
-            $document->addField('dateCreated',Util::convertDateTime($job->getDateCreated()));
+        if ($job->getDateCreated()) {
+            $document->addField('dateCreated', Util::convertDateTime($job->getDateCreated()));
         }
-        if($job->getDateModified()){
-            $document->addField('dateModified',Util::convertDateTime($job->getDateModified()));
+        if ($job->getDateModified()) {
+            $document->addField('dateModified', Util::convertDateTime($job->getDateModified()));
         }
-        if($job->getDatePublishStart()){
-            $document->addField('datePublishStart',Util::convertDateTime($job->getDatePublishStart()));
+        if ($job->getDatePublishStart()) {
+            $document->addField('datePublishStart', Util::convertDateTime($job->getDatePublishStart()));
         }
-        if($job->getDatePublishEnd()){
-            $document->addField('datePublishEnd',Util::convertDateTime($job->getDatePublishEnd()));
+        if ($job->getDatePublishEnd()) {
+            $document->addField('datePublishEnd', Util::convertDateTime($job->getDatePublishEnd()));
         }
-        $document->addField('isActive',$job->isActive());
-        $document->addField('lang',$job->getLanguage());
-        $this->processLocation($job,$document);
-        if(!is_null($job->getOrganization())){
+        $document->addField('isActive', $job->isActive());
+        $document->addField('lang', $job->getLanguage());
+        $this->processLocation($job, $document);
+        if (!is_null($job->getOrganization())) {
             try {
-                $this->processOrganization($job,$document);
-            }catch (\Exception $e){
+                $this->processOrganization($job, $document);
+            } catch (\Exception $e) {
                 // @TODO: What to do when the process failed?
             }
         }
@@ -157,14 +157,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());
     }
 
     /**
@@ -172,30 +172,30 @@ 
      * @param Job                $job
      * @param \SolrInputDocument $document
      */
-    public function processLocation(Job $job,$document)
+    public function processLocation(Job $job, $document)
     {
         /* @var \Jobs\Entity\Location $location */
-        $locations=$job->getLocations();
-        foreach($locations as $location){
+        $locations = $job->getLocations();
+        foreach ($locations as $location) {
 
             $loc = new \SolrInputDocument();
             $loc->addField('entityName', 'location');
-            if(is_object($location->getCoordinates())){
+            if (is_object($location->getCoordinates())) {
                 $coordinate = Util::convertLocationCoordinates($location);
                 $loc->addField('point', $coordinate);
                 $loc->addField('latLon', $coordinate);
                 $document->addField('locations', $coordinate);
                 $document->addField('points', $coordinate);
-                $loc->addField('id', $job->getId() . '-' . $coordinate);
+                $loc->addField('id', $job->getId().'-'.$coordinate);
                 $loc->addField('city', $location->getCity());
                 $loc->addField('country', $location->getCountry());
                 $loc->addField('region', $location->getRegion());
                 $loc->addField('postalCode', $location->getPostalCode());
-                $document->addField('regionList',$location->getRegion());
+                $document->addField('regionList', $location->getRegion());
                 $document->addChildDocument($loc);
             }
             unset($loc);
         }
-        $document->addField('location',$job->getLocation());
+        $document->addField('location', $job->getLocation());
     }
 }
\ No newline at end of file

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);
         }
     }

module/Solr/config/module.config.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
         'Core/CreatePaginator/Events' => [
             'listeners' => [
                 'Solr/Listener/CreatePaginator' => [
-                    \Core\Listener\Events\CreatePaginatorEvent::EVENT_CREATE_PAGINATOR,'onCreatePaginator',1
+                    \Core\Listener\Events\CreatePaginatorEvent::EVENT_CREATE_PAGINATOR, 'onCreatePaginator', 1
                 ]
             ]
         ]