Feature #8852

Add method overrideArgument() to AbstractViewHelper

Added by Bastian Waidelich about 5 years ago. Updated almost 5 years ago.

Status:Resolved Start date:2010-07-14
Priority:Should have Due date:
Assigned To:Bastian Waidelich % Done:

100%

Category:Core
Target version:-
Has patch:

Description

When registering an argument twice, you'll get an exception

#1253036401: Argument "xyz" has already been defined, thus it should not be defined again.

That's intended, but we need the possibility to override argument registrations (e.g. for the form.checkbox vh that sets the "value" argument required)

Associated revisions

Revision 8cb5a00c
Added by Bastian Waidelich about 5 years ago

[!!!][+TASK] Fluid (ViewHelpers): The value argument is required again in form.checkbox and form.radio ViewHelpers. This is a breaking change, as the value argument has not been mandatory for a while. But it probably won't break existing templates as omitting the value makes no sense at all. Relates to #8852
[+FEATURE] Fluid (Core): implemented overrideArgument() method in AbstractViewHelper to be able to override previously registered arguments in subclasses. This resolves #8852
[+BUGFIX] Fluid (Core): Negative numeric values are properly converted to FALSE when used in boolean ViewHelper arguments. This resolves #8893

History

#1 Updated by Bastian Waidelich about 5 years ago

  • Status changed from Accepted to Resolved
  • % Done changed from 0 to 100

Applied in changeset r4864.

Also available in: Atom PDF