Closure: chops 'uses' when are longer than WRAP_LENGTH

Author: dg

src/PhpGenerator/Closure.php

@@ -42,10 +42,14 @@ public function __toString()
 		foreach ($this->uses as $param) {
 			$uses[] = ($param->isReference() ? '&' : '') . '$' . $param->getName();
 		}
+		$useStr = strlen($tmp = implode(', ', $uses)) > Helpers::WRAP_LENGTH && count($uses) > 1
+			? "\n\t" . implode(",\n\t", $uses) . "\n"
+			: $tmp;
+
 		return 'function '
 			. ($this->returnReference ? '&' : '')
 			. $this->parametersToString()
-			. ($this->uses ? ' use (' . implode(', ', $uses) . ')' : '')
+			. ($this->uses ? " use ($useStr)" : '')
 			. $this->returnTypeToString()
 			. " {\n" . Nette\Utils\Strings::indent(ltrim(rtrim($this->body) . "\n"), 1) . '}';
 	}

tests/PhpGenerator/Closure.long.phpt

@@ -0,0 +1,49 @@
+<?php
+
+use Nette\PhpGenerator\Closure;
+use Tester\Assert;
+
+
+require __DIR__ . '/../bootstrap.php';
+
+
+$function = new Closure;
+$function->setBody('return null;');
+
+for ($name = 'abcde'; $name < 'abcdr'; $name++) {
+	$function->addParameter($name);
+	$function->addUse($name);
+}
+
+Assert::match(
+'function (
+	$abcde,
+	$abcdf,
+	$abcdg,
+	$abcdh,
+	$abcdi,
+	$abcdj,
+	$abcdk,
+	$abcdl,
+	$abcdm,
+	$abcdn,
+	$abcdo,
+	$abcdp,
+	$abcdq
+) use (
+	$abcde,
+	$abcdf,
+	$abcdg,
+	$abcdh,
+	$abcdi,
+	$abcdj,
+	$abcdk,
+	$abcdl,
+	$abcdm,
+	$abcdn,
+	$abcdo,
+	$abcdp,
+	$abcdq
+) {
+	return null;
+}', (string) $function);