Merge pull request doctrine#793 from flip111/patch-2

guilhermeblanco · guilhermeblanco · commit d8335eee7b1e · 2013-09-30T11:07:49.000-07:00
Improved error messages in Paginator: LimitSubqueryOutputWalker
diff --git a/lib/Doctrine/ORM/Tools/Pagination/LimitSubqueryOutputWalker.php b/lib/Doctrine/ORM/Tools/Pagination/LimitSubqueryOutputWalker.php
@@ -149,6 +149,10 @@ public function walkSelectStatement(SelectStatement $AST)
             }
         }
 
+        if (count($sqlIdentifier) === 0) {
+            throw new \RuntimeException('The Paginator does not support Queries which only yield ScalarResults.');
+        }
+
         if (count($rootIdentifier) != count($sqlIdentifier)) {
             throw new \RuntimeException(sprintf(
                 'Not all identifier properties can be found in the ResultSetMapping: %s',

Original file line number	Diff line number	Diff line change
`@@ -149,6 +149,10 @@ public function walkSelectStatement(SelectStatement $AST)`
`149`	`149`	`}`
`150`	`150`	`}`
`151`	`151`
	`152`	`+ if (count($sqlIdentifier) === 0) {`
	`153`	`+ throw new \RuntimeException('The Paginator does not support Queries which only yield ScalarResults.');`
	`154`	`+ }`
	`155`	`+`
`152`	`156`	`if (count($rootIdentifier) != count($sqlIdentifier)) {`
`153`	`157`	`throw new \RuntimeException(sprintf(`
`154`	`158`	`'Not all identifier properties can be found in the ResultSetMapping: %s',`