Node: getIterator() must exists (BC break)

Author: dg

src/Latte/Compiler/Nodes/AuxiliaryNode.php

@@ -24,4 +24,10 @@ public function print(PrintContext $context): string
 	{
 		return ($this->callable)($context);
 	}
+
+
+	public function &getIterator(): \Generator
+	{
+		false && yield;
+	}
 }

src/Latte/Compiler/Nodes/NopNode.php

@@ -18,4 +18,10 @@ public function print(PrintContext $context): string
 	{
 		return '';
 	}
+
+
+	public function &getIterator(): \Generator
+	{
+		false && yield;
+	}
 }

src/Latte/Compiler/Nodes/Php/IdentifierNode.php

@@ -33,4 +33,10 @@ public function print(PrintContext $context): string
 	{
 		return $this->name;
 	}
+
+
+	public function &getIterator(): \Generator
+	{
+		false && yield;
+	}
 }

src/Latte/Compiler/Nodes/Php/NameNode.php

@@ -86,4 +86,10 @@ public function toCodeString(): string
 		};
 		return $prefix . implode('\\', $this->parts);
 	}
+
+
+	public function &getIterator(): \Generator
+	{
+		false && yield;
+	}
 }

src/Latte/Compiler/Nodes/Php/ScalarNode.php

@@ -12,4 +12,8 @@
 
 abstract class ScalarNode extends ExpressionNode
 {
+	public function &getIterator(): \Generator
+	{
+		false && yield;
+	}
 }

src/Latte/Compiler/Nodes/Php/SuperiorTypeNode.php

@@ -26,4 +26,10 @@ public function print(PrintContext $context): string
 	{
 		throw new \LogicException('Cannot directly print SuperiorTypeNode');
 	}
+
+
+	public function &getIterator(): \Generator
+	{
+		false && yield;
+	}
 }

src/Latte/Compiler/Nodes/TextNode.php

@@ -34,4 +34,10 @@ public function isWhitespace(): bool
 	{
 		return trim($this->content) === '';
 	}
+
+
+	public function &getIterator(): \Generator
+	{
+		false && yield;
+	}
 }

src/Latte/Compiler/TemplateParser.php

@@ -238,6 +238,8 @@ public function parseLatteStatement(): ?Node
 			throw new \LogicException("Incorrect behavior of {{$startTag->name}} parser, unexpected returned value (on line {$startTag->position->line})");
 		}
 
+		$this->checkNodeCompatibility($node);
+
 		if ($this->location === self::LocationHead && $startTag->outputMode !== $startTag::OutputNone) {
 			$this->location = self::LocationText;
 		}
@@ -445,4 +447,17 @@ public function isTagAllowed(string $name): bool
 	{
 		return !$this->policy || $this->policy->isTagAllowed($name);
 	}
+
+
+	public function checkNodeCompatibility(Node $node): void
+	{
+		static $prev = ['Nette\Bridges\ApplicationLatte\Nodes\NNonceNode' => true];
+		if (
+			!isset($prev[$node::class])
+			&& (new \ReflectionMethod($node, 'getIterator'))->getDeclaringClass()->getName() === Node::class
+		) {
+			trigger_error('Class ' . $node::class . ' should contain method getIterator(), see https://bit.ly/latte-666');
+		}
+		$prev[$node::class] = true;
+	}
 }

src/Latte/Compiler/TemplateParserHtml.php

@@ -454,6 +454,7 @@ private function openNAttrNodes(array $toOpen): array
 
 			} elseif ($res instanceof Node) {
 				$this->parser->ensureIsConsumed($tag);
+				$this->parser->checkNodeCompatibility($res);
 				$res->position = $tag->position;
 				$tag->replaceNAttribute($res);
 				$this->parser->popTag();
@@ -477,6 +478,7 @@ private function finishNAttrNodes(AreaNode $node, array $toClose): AreaNode
 		while ([$gen, $tag] = array_pop($toClose)) {
 			$gen->send([$node, null]);
 			$node = $gen->getReturn();
+			$this->parser->checkNodeCompatibility($node);
 			$node->position = $tag->position;
 			$this->parser->popTag();
 			$this->parser->ensureIsConsumed($tag);

src/Latte/Essential/Nodes/ContentTypeNode.php

@@ -71,4 +71,10 @@ public function print(PrintContext $context): string
 			)
 			: '';
 	}
+
+
+	public function &getIterator(): \Generator
+	{
+		false && yield;
+	}
 }