Hi,
The following relates to recent changes in v8 nightly builds for the DynamicForm class.
When the new suppressAutoFocus parameter was introduced, the showFieldErrors()
implementation changed to let supressAutoFocus passthru into showErrors().
The change implied having showErrors() called with null+null as 1st and 2nd arguments in order to
supply the new suppressAutoFocus parameter as 3rd argument into showErrors(). Priorly, showErrors()
was being called without any arguments, resulting in undef+undef as 1st and 2nd arguments to showErrors() ...
The problem is that the showErrors() method checks for "undef", not for "null" ...
Was that intended? It does seem to break some existing code ... Any suggestions ?
Kind regards,
The following relates to recent changes in v8 nightly builds for the DynamicForm class.
When the new suppressAutoFocus parameter was introduced, the showFieldErrors()
implementation changed to let supressAutoFocus passthru into showErrors().
The change implied having showErrors() called with null+null as 1st and 2nd arguments in order to
supply the new suppressAutoFocus parameter as 3rd argument into showErrors(). Priorly, showErrors()
was being called without any arguments, resulting in undef+undef as 1st and 2nd arguments to showErrors() ...
The problem is that the showErrors() method checks for "undef", not for "null" ...
Was that intended? It does seem to break some existing code ... Any suggestions ?
Kind regards,
Comment