Changes between Initial Version and Version 1 of Ticket #14184, comment 3


Ignore:
Timestamp:
Oct 6, 2011, 2:10:56 AM (13 years ago)
Author:
Paul Collins

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #14184, comment 3

    initial v1  
    11Was a bit surprised by this one myself. I'm working on the idea that the MultiValueField is wrong rather than the docs since the validators arg is a core Field arg. Patch includes a regression test for this and a fix which appears to resolve the issue.
     2
     3On a related note MultiValueField.validate is overridden to be a noop, but the parent class Field checks if the field is required. I've left it alone but, should MultiValueField.validate be removed so that the Field.validate gets called instead?
Back to Top