From 3ff9cc3985d319312dd65a903f1e2a39c2acbf66 Mon Sep 17 00:00:00 2001
From: Michal Korinek <webace@atlas.cz>
Date: Sat, 30 May 2015 09:56:27 +0200
Subject: [PATCH] FormMacros: added support for <label n:name /> Checkbox label
 with caption rendering

Conflicts:
	tests/Forms.Latte/FormMacros.forms.phpt
---
 src/Bridges/FormsLatte/FormMacros.php             |  2 +-
 tests/Forms.Latte/FormMacros.forms.phpt           |  2 +-
 tests/Forms.Latte/expected/FormMacros.forms.html  |  2 ++
 tests/Forms.Latte/expected/FormMacros.forms.phtml | 15 ++++++++++-----
 tests/Forms.Latte/templates/forms.latte           |  2 ++
 5 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/src/Bridges/FormsLatte/FormMacros.php b/src/Bridges/FormsLatte/FormMacros.php
index b0c56834d..0708bf53d 100644
--- a/src/Bridges/FormsLatte/FormMacros.php
+++ b/src/Bridges/FormsLatte/FormMacros.php
@@ -198,7 +198,7 @@ public function macroNameEnd(MacroNode $node, PhpWriter $writer)
 			$node->content = $parts[1] . $parts[2] . '<?php echo Nette\Bridges\FormsLatte\Runtime::renderFormEnd($_form, FALSE) ?>' . $parts[3];
 		} elseif ($tagName === 'label') {
 			if ($node->htmlNode->isEmpty) {
-				$node->content = $parts[1] . '<?php echo $_input->getLabel()->getHtml() ?>' . $parts[3];
+				$node->content = $parts[1] . "<?php echo \$_input->{method_exists(\$_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->getHtml() ?>" . $parts[3];
 			}
 		} elseif ($tagName === 'button') {
 			if ($node->htmlNode->isEmpty) {
diff --git a/tests/Forms.Latte/FormMacros.forms.phpt b/tests/Forms.Latte/FormMacros.forms.phpt
index c05394939..0e2e49d8a 100644
--- a/tests/Forms.Latte/FormMacros.forms.phpt
+++ b/tests/Forms.Latte/FormMacros.forms.phpt
@@ -32,7 +32,7 @@ $form->addText('username', 'Username:'); // must have just one textfield to gene
 $form->addRadioList('sex', 'Sex:', array('m' => 'male', 'f' => 'female'));
 $form->addSelect('select', NULL, array('m' => 'male', 'f' => 'female'));
 $form->addTextArea('area', NULL)->setValue('one<two');
-$form->addCheckbox('checkbox', NULL);
+$form->addCheckbox('checkbox', 'Checkbox');
 $form->addCheckboxList('checklist', 'CheckboxList:', array('m' => 'male', 'f' => 'female'));
 $form->addSubmit('send', 'Sign in');
 $form['my'] = new MyControl;
diff --git a/tests/Forms.Latte/expected/FormMacros.forms.html b/tests/Forms.Latte/expected/FormMacros.forms.html
index 64491ce02..ca9e00a53 100644
--- a/tests/Forms.Latte/expected/FormMacros.forms.html
+++ b/tests/Forms.Latte/expected/FormMacros.forms.html
@@ -77,6 +77,8 @@
 <label for="frm-checkbox"> <input type="checkbox" name="checkbox" id="frm-checkbox"> Label</label>
 <label title=hello for="frm-checkbox"> <input type="checkbox" name="checkbox" id="frm-checkbox"> </label>
 <label title=hello for="frm-checkbox"> <input type="checkbox" name="checkbox" id="frm-checkbox"> </label>
+<label for="frm-checkbox">Checkbox</label>
+<label title=hello for="frm-checkbox">Checkbox</label>
 
 
 	<label for="frm-checklist-m"> <input type="checkbox" name="checklist[]" id="frm-checklist-m" value="m"> male</label>
diff --git a/tests/Forms.Latte/expected/FormMacros.forms.phtml b/tests/Forms.Latte/expected/FormMacros.forms.phtml
index ffa66537b..babe7fe8e 100644
--- a/tests/Forms.Latte/expected/FormMacros.forms.phtml
+++ b/tests/Forms.Latte/expected/FormMacros.forms.phtml
@@ -36,7 +36,7 @@ echo Nette\Bridges\FormsLatte\Runtime::renderFormBegin($form = $_form = $_contro
 
 	<label<?php $_input = is_object($form['username']) ? $form['username'] : $_form[$form['username']]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->attributes() ?>></label>
 	<label<?php $_input = is_object($form['username']) ? $form['username'] : $_form[$form['username']]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->attributes() ?>
-><?php echo $_input->getLabel()->getHtml() ?></label>
+><?php echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->getHtml() ?></label>
 	<input<?php $_input = is_object($form['username']) ? $form['username'] : $_form[$form['username']]; echo $_input->{method_exists($_input, 'getControlPart')?'getControlPart':'getControl'}()->attributes() ?>>
 
 	<?php if ($_label = $_form["my"]->getLabel()) echo $_label ; echo $_form["my"]->getControl() ?>
@@ -59,10 +59,10 @@ echo Nette\Bridges\FormsLatte\Runtime::renderFormBegin($form = $_form = $_contro
 <?php $iterations++; } ?>
 <label<?php $_input = $_form["sex"]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->attributes() ?>></label>
 <label<?php $_input = $_form["sex"]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->attributes() ?>
-><?php echo $_input->getLabel()->getHtml() ?></label>
+><?php echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->getHtml() ?></label>
 <label title="hello"<?php $_input = $_form["sex"]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->addAttributes(array (
   'title' => NULL,
-))->attributes() ?>><?php echo $_input->getLabel()->getHtml() ?></label>
+))->attributes() ?>><?php echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->getHtml() ?></label>
 
 
 <?php if ($_label = $_form["checkbox"]->getLabelPart("")) echo $_label->startTag() ?>
@@ -74,6 +74,11 @@ echo Nette\Bridges\FormsLatte\Runtime::renderFormBegin($form = $_form = $_contro
 <label title=hello<?php $_input = $_form["checkbox"]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->addAttributes(array (
   'title' => NULL,
 ))->attributes() ?>> <input<?php $_input = $_form["checkbox"]; echo $_input->{method_exists($_input, 'getControlPart')?'getControlPart':'getControl'}()->attributes() ?>> </label>
+<label<?php $_input = $_form["checkbox"]; echo $_input->getLabelPart("")->attributes() ?>
+><?php echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->getHtml() ?></label>
+<label title=hello<?php $_input = $_form["checkbox"]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->addAttributes(array (
+  'title' => NULL,
+))->attributes() ?>><?php echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->getHtml() ?></label>
 
 
 <?php $iterations = 0; foreach ($form['checklist']->items as $key => $label) { ?>
@@ -87,10 +92,10 @@ echo Nette\Bridges\FormsLatte\Runtime::renderFormBegin($form = $_form = $_contro
 <?php $iterations++; } ?>
 <label<?php $_input = $_form["checklist"]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->attributes() ?>></label>
 <label<?php $_input = $_form["checklist"]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->attributes() ?>
-><?php echo $_input->getLabel()->getHtml() ?></label>
+><?php echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->getHtml() ?></label>
 <label title="hello"<?php $_input = $_form["checklist"]; echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->addAttributes(array (
   'title' => NULL,
-))->attributes() ?>><?php echo $_input->getLabel()->getHtml() ?></label>
+))->attributes() ?>><?php echo $_input->{method_exists($_input, 'getLabelPart')?'getLabelPart':'getLabel'}()->getHtml() ?></label>
 
 
 <?php if (1) { ?><form id="myForm" class="ajax"<?php echo Nette\Bridges\FormsLatte\Runtime::renderFormBegin($form = $_form = $_control["myForm"], array (
diff --git a/tests/Forms.Latte/templates/forms.latte b/tests/Forms.Latte/templates/forms.latte
index 2f8f5981e..e65eb3c3d 100644
--- a/tests/Forms.Latte/templates/forms.latte
+++ b/tests/Forms.Latte/templates/forms.latte
@@ -41,6 +41,8 @@
 {label checkbox:} {input checkbox:} Label{/label}
 <label n:name="checkbox:" title=hello> <input n:name="checkbox:"> </label>
 <label n:name="checkbox" title=hello> <input n:name="checkbox"> </label>
+<label n:name="checkbox:" />
+<label n:name="checkbox" title=hello />
 
 
 {* partial rendering of CheckboxList *}