Merge pull request #157 from Flowpack/dfeyer-patch-1

dfeyer · web-flow · commit 96b6b2e880b0 · 2016-11-04T21:09:17.000+01:00
BUGIFX: Remove useless variable in traverseNodes callback

This change solve issue with PHP 7.1 that don't allow it.
diff --git a/Classes/Flowpack/ElasticSearch/ContentRepositoryAdaptor/Service/IndexWorkspaceTrait.php b/Classes/Flowpack/ElasticSearch/ContentRepositoryAdaptor/Service/IndexWorkspaceTrait.php
@@ -71,7 +71,7 @@ protected function indexWorkspaceWithDimensions($workspaceName, array $dimension
         $rootNode = $context->getRootNode();
         $indexedNodes = 0;
 
-        $traverseNodes = function (NodeInterface $currentNode, &$indexedNodes) use ($limit, &$indexedNodes, &$traverseNodes) {
+        $traverseNodes = function (NodeInterface $currentNode, &$indexedNodes) use ($limit, &$traverseNodes) {
             if ($limit !== null && $indexedNodes > $limit) {
                 return;
             }

Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,7 @@ protected function indexWorkspaceWithDimensions($workspaceName, array $dimension`
`71`	`71`	`$rootNode = $context->getRootNode();`
`72`	`72`	`$indexedNodes = 0;`
`73`	`73`
`74`		`- $traverseNodes = function (NodeInterface $currentNode, &$indexedNodes) use ($limit, &$indexedNodes, &$traverseNodes) {`
	`74`	`+ $traverseNodes = function (NodeInterface $currentNode, &$indexedNodes) use ($limit, &$traverseNodes) {`
`75`	`75`	`if ($limit !== null && $indexedNodes > $limit) {`
`76`	`76`	`return;`
`77`	`77`	`}`