bug #23 Fix detection of QueryPlatform (sstok)

This PR was merged into the 2.0-dev branch.

Discussion
----------

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | Closes rollerworks/search#178
| License       | MIT
| Doc PR        | 

Commits
-------

7847fbb Fix detection of QueryPlatform
dd3ac69 Run travis with all groups

.travis.yml

@@ -46,7 +46,6 @@ before_script:
 script:
     - export SYMFONY_DEPRECATIONS_HELPER=strict
     - vendor/bin/phpunit --verbose
-    - if [ "$DB" != "" ] && [ "$DB" != "sqlite" ]; then vendor/bin/simple-phpunit --configuration tests/travis/$DB.travis.xml --exclude-group non-functional; fi;
-    - if [ "$DB" = "sqlite" ]; then vendor/bin/simple-phpunit --configuration tests/travis/sqlite.travis.xml; fi;
+    - if [ "$DB" != "" ]; then vendor/bin/simple-phpunit --configuration tests/travis/$DB.travis.xml; fi;
     - if [[ $lint = 1 ]]; then php php-cs-fixer.phar fix --dry-run --diff --no-ansi; fi
     - if [[ $lint = 1 ]]; then phpstan analyse -c phpstan.neon -l5 --ansi src tests; fi

src/CachedDqlConditionGenerator.php

@@ -86,10 +86,7 @@ public function getWhereClause(string $prependQuery = ''): string
         if (null === $this->whereClause) {
             $cacheKey = $this->getCacheKey();
 
-            $this->nativePlatform = $this->getQueryPlatform(
-                $this->conditionGenerator->getEntityManager()->getConnection(),
-                $this->conditionGenerator->getFieldsConfig()->getFields()
-            );
+            $this->nativePlatform = $this->getQueryPlatform($this->conditionGenerator->getEntityManager()->getConnection());
 
             if (null !== $cacheItem = $this->cacheDriver->get($cacheKey)) {
                 list($this->whereClause, $this->parameters) = $cacheItem;

src/DqlConditionGenerator.php

@@ -104,7 +104,7 @@ public function getWhereClause(string $prependQuery = ''): string
             $connection = $this->entityManager->getConnection();
             $queryGenerator = new QueryGenerator($connection, $platform, $fields);
 
-            $this->nativePlatform = $this->getQueryPlatform($connection, $fields);
+            $this->nativePlatform = $this->getQueryPlatform($connection);
             $this->whereClause = $queryGenerator->getGroupQuery($this->searchCondition->getValuesGroup());
             $this->parameters = $platform->getEmbeddedValues();
         }

src/NativeQueryConditionGenerator.php

@@ -66,7 +66,7 @@ public function getWhereClause(string $prependQuery = ''): string
             $fields = $this->fieldsConfig->getFields();
             $connection = $this->entityManager->getConnection();
 
-            $queryGenerator = new QueryGenerator($connection, $this->getQueryPlatform($connection, $fields), $fields);
+            $queryGenerator = new QueryGenerator($connection, $this->getQueryPlatform($connection), $fields);
             $this->whereClause = $queryGenerator->getGroupQuery($this->searchCondition->getValuesGroup());
         }
 

src/QueryPlatformTrait.php

@@ -15,29 +15,22 @@
 
 use Doctrine\DBAL\Connection;
 use Rollerworks\Component\Search\Doctrine\Dbal\QueryPlatform;
+use Rollerworks\Component\Search\Doctrine\Dbal\QueryPlatform\SqlQueryPlatform;
 
 /**
  * @internal
  */
 trait QueryPlatformTrait
 {
-    /**
-     * Gets the QueryPlatform based on the connection.
-     *
-     * @param Connection $connection
-     * @param array      $fields
-     *
-     * @return QueryPlatform
-     */
-    protected function getQueryPlatform(Connection $connection, array $fields): QueryPlatform
+    protected function getQueryPlatform(Connection $connection): QueryPlatform
     {
         $dbPlatform = ucfirst($connection->getDatabasePlatform()->getName());
         $platformClass = 'Rollerworks\\Component\\Search\\Doctrine\\Dbal\\QueryPlatform\\'.$dbPlatform.'QueryPlatform';
 
         if (class_exists($platformClass)) {
-            return new $platformClass($connection, $fields);
+            return new $platformClass($connection);
         }
 
-        throw new \RuntimeException(sprintf('No supported class found for database-platform "%s".', $dbPlatform));
+        return new SqlQueryPlatform($connection);
     }
 }