Bug #81714
closedNo valid HTML output Form->DatePicker
100%
Description
Hello Together,
the SysExt Form in Typo 3 8.7.2 renders wrong html for DatePicker.
This is wrong:
<input class="small form-control" id="kontakt-datepicker-1" type="input" name="tx_form_formframework[kontakt][datepicker-1][date]" placeholder="d.m.Y" readonly="1" />
This is right
<input class="small form-control" id="kontakt-datepicker-1" type="text" name="tx_form_formframework[kontakt][datepicker-1][date]" placeholder="d.m.Y" readonly="readonly" />
It is wrong in Class typo3/sysext/form/Classes/ViewHelpers/Form/DatePickerViewHelper.php:
Line 90:
Wrong:
$this->tag->addAttribute('type', 'input');
Right:
$this->tag->addAttribute('type', 'text');
Line 104:
Wrong:
$this->tag->addAttribute('readonly', true);
Right:
$this->tag->addAttribute('readonly', 'readonly');
Thank you for change.
Updated by Gerrit Code Review over 7 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53361
Updated by Gerrit Code Review over 7 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53361
Updated by Gerrit Code Review over 7 years ago
Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/53420
Updated by Ralf Zimmermann over 7 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 6ed16e54a0dfccb959eabbd8468c6674a7cb8891.
Updated by Anonymous about 7 years ago
Not sure why readonly is used here at all.