BaseControl: added ability to multiple forms with different HTML ID [Closes #188]

Solution: set Nette\Forms\Controls\BaseControl::$idMask = 'frm-%2$s-%1$s';

Author: dg

src/Forms/Controls/BaseControl.php

@@ -336,7 +336,8 @@ public function setHtmlId($id)
 	public function getHtmlId()
 	{
 		if (!isset($this->control->id)) {
-			$this->control->id = sprintf(self::$idMask, $this->lookupPath());
+			$form = $this->getForm(false);
+			$this->control->id = sprintf(self::$idMask, $this->lookupPath(), $form ? $form->getName() : '');
 		}
 		return $this->control->id;
 	}

tests/Forms/Forms.idMask.phpt

@@ -0,0 +1,57 @@
+<?php
+
+declare(strict_types=1);
+
+use Nette\Forms\Controls\TextInput;
+use Nette\Forms\Form;
+use Tester\Assert;
+
+
+require __DIR__ . '/../bootstrap.php';
+
+
+Assert::exception(function () {
+	$input = new TextInput('name');
+	$input->getHtmlId();
+}, Nette\InvalidStateException::class, "Component '' is not attached to ''.");
+
+
+test(function () {
+	$container = new Nette\Forms\Container;
+	$container->setParent(null, 'second');
+	$input = $container->addText('name');
+	Assert::same('frm-name', $input->getHtmlId());
+});
+
+
+test(function () {
+	$form = new Form;
+	$container = $form->addContainer('second');
+	$input = $container->addText('name');
+	Assert::same('frm-second-name', $input->getHtmlId());
+});
+
+
+test(function () {
+	$form = new Form;
+	$input = $form->addText('name');
+	Assert::same('frm-name', $input->getHtmlId());
+});
+
+
+test(function () {
+	Nette\Forms\Controls\BaseControl::$idMask = 'frm-%s-%s';
+
+	$form = new Form;
+	$input = $form->addText('name');
+	Assert::same('frm-name-', $input->getHtmlId());
+});
+
+
+test(function () {
+	Nette\Forms\Controls\BaseControl::$idMask = 'frm-%2$s-%1$s';
+
+	$form = new Form('signForm');
+	$input = $form->addText('name');
+	Assert::same('frm-signForm-name', $input->getHtmlId());
+});