Skip to content

Commit 0976f79

Browse files
committed
Revert "Dumper: dumps Closure as PHP code" (BC break)
Because there is no option to use your own printer This reverts commit 59bb35e.
1 parent b05a3ae commit 0976f79

File tree

2 files changed

+3
-8
lines changed

2 files changed

+3
-8
lines changed

src/PhpGenerator/Dumper.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,10 +46,10 @@ private function dumpVar(&$var, array $parents = [], int $level = 0, int $column
4646
} elseif (is_array($var)) {
4747
return $this->dumpArray($var, $parents, $level, $column);
4848

49+
} elseif ($var instanceof Literal) {
50+
return ltrim(Nette\Utils\Strings::indent(trim((string) $var), $level), "\t");
51+
4952
} elseif (is_object($var)) {
50-
if ($var instanceof Literal || $var instanceof Closure) {
51-
return ltrim(Nette\Utils\Strings::indent(trim((string) $var), $level), "\t");
52-
}
5353
return $this->dumpObject($var, $parents, $level);
5454

5555
} elseif (is_resource($var)) {

tests/PhpGenerator/Dumper.dump().phpt

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,6 @@ Assert::same('\'He\ll\\\\\o \\\'wor\\\\\\\'ld\\\\\'', $dumper->dump('He\ll\\\o \
4545
// literal
4646
Assert::same('[$s]', $dumper->dump([new Literal('$s')]));
4747
Assert::same("[strlen('hello')]", $dumper->dump([new Literal('strlen(?)', ['hello'])]));
48-
same('[
49-
function () {
50-
return 1;
51-
},
52-
]', $dumper->dump([(new Nette\PhpGenerator\Closure)->setBody('return 1;')]));
5348

5449

5550
// arrays

0 commit comments

Comments
 (0)