Bug #34186

Setting a value always resets the field on mapping errors

Added by Kira Backes almost 10 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Should have
Category:
ViewHelpers
Target version:
-
Start date:
2012-02-22
Due date:
% Done:

0%

Estimated time:
Has patch:
No

Description

If I set a value on any field of the form it gets reset every time there
is a mapping error, so the user has to reenter the field every time.

Could easily be fixed by changing the order of the if-clause in
\TYPO3\Fluid\ViewHelpers\Form\AbstractFormFieldViewHelper::getValue


Related issues

Has duplicate TYPO3.Fluid - Bug #47363: Error Classes are only set for form fields in object contextClosed2013-04-18

Actions
#1

Updated by Kira Backes almost 10 years ago

Okay, it’s not that easy to just change the if-else order, then submit buttons disappear.. I guess we need another attribute for a start value like that or we only use the submitted form data if it is not NULL.

#2

Updated by Bastian Waidelich almost 10 years ago

Hi Kira,

Please provide more information. Did you use one of the form viewhelpers (which one?) and bind it to a property of the form object? Can you provide some (fluid) code snippets?

#3

Updated by Bastian Waidelich almost 10 years ago

  • Status changed from New to Needs Feedback
#4

Updated by Sebastian Kurfuerst over 9 years ago

  • Status changed from Needs Feedback to Accepted
  • Assignee set to Sebastian Kurfuerst

OK; the automatic fill-in does is partly broken when not using object accessor mode.

Fix is underway.

#5

Updated by Gerrit Code Review over 9 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10732

#6

Updated by Gerrit Code Review over 9 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/10733

#7

Updated by Bastian Waidelich about 8 years ago

  • Category set to ViewHelpers
  • Assignee changed from Sebastian Kurfuerst to Bastian Waidelich

The mentioned review (http://review.typo3.org/10733) does not fix this. I'll take care asap

#8

Updated by Gerrit Code Review about 8 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/24794

#9

Updated by Bastian Waidelich almost 7 years ago

  • Status changed from Under Review to Closed

Truly sorry for the huge delay, this was a tricky one (because it is a breaking behavior).. But I just moved this to https://jira.typo3.org/browse/FLOW-213 and take care of it right now

Also available in: Atom PDF