deblan/FOSElasticaBundle
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Somewhat clearer exception when Elastica results do not match Doctrine objects. Consistent behavior for both transform() and hybrdidTrasnform().

Browse source
This commit is contained in:
nurikabe 2012-04-19 15:05:18 +01:00 committed by Richard Miller
parent 68e73366d9
commit 16f621b78f
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
Doctrine/AbstractElasticaToModelTransformer.php
View file

@ -5,7 +5,6 @@ namespace FOQ\ElasticaBundle\Doctrine;
use FOQ\ElasticaBundle\HybridResult;
use FOQ\ElasticaBundle\Transformer\ElasticaToModelTransformerInterface;
use Elastica_Document;
use RuntimeException;
/**
* Maps Elastica documents with Doctrine objects
@ -74,6 +73,9 @@ abstract class AbstractElasticaToModelTransformer implements ElasticaToModelTran
}, $elasticaObjects);
$objects = $this->findByIdentifiers($this->objectClass, $this->options['identifier'], $ids, $this->options['hydrate']);
if (count($objects) < count($elasticaObjects)) {
throw new \RuntimeException('Cannot find corresponding Doctrine objects for all Elastica results.');
};
$identifierGetter = 'get'.ucfirst($this->options['identifier']);
@ -90,9 +92,6 @@ abstract class AbstractElasticaToModelTransformer implements ElasticaToModelTran
public function hybridTransform(array $elasticaObjects)
{
$objects = $this->transform($elasticaObjects);
if (count($objects) < count($elasticaObjects)) {
throw new RuntimeException('Cannot transform all Elastica results into objects.');
};
$result = array();
for ($i = 0; $i < count($elasticaObjects); $i++) {