Code

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


Ignore:
Timestamp:
10/06/11 00:10:56 (3 years ago)
Author:
paulcollins
Comment:

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?