django.newforms.extras.widgets.SelectDateWidget

if type(value) == type(u''): => isinstance(value, unicode)

Sorry ;-/

Fix + ID's

Here is the patch. I alos added individual id_* to each <select> inspired by CheckboxSelectMultiple?()'s way to do it, it works, i hope it fits Django coding guidelines.

Regards.

Camille.

Fix + ID's 2 (same but uses month_field = '%s_month', etc...)

Fix + ID's 3 (same as Fix + ID's 2 but uses isinstance, this is the good one, sorry for flooding)

Good one, forgget above :-/

Fix + ID's 5

This is IMHO a better way to do that

This is IMHO a better way to do that (Sorry for the other one)

This is IMHO a better way to do that (Err.. Sorry for the other ones)

I suggest using isinstance(value, basestring), or even wrapping the value in unicode().

def render(self, name, value, attrs=None):
    try:
        if isinstance(value, basestring):
            value = datetime.date(*map(int, value.split('-')))
        year_val, month_val, day_val = value.year, value.month, value.day
    except (AttributeError, TypeError, ValueError):
        year_val = month_val = day_val = None

    # -- snip assumed --

def render(self, name, value, attrs=None):
    try:
       
        value = datetime.date(*map(int, unicode(value).split('-')))
        year_val, month_val, day_val = value.year, value.month, value.day
    except (AttributeError, TypeError, ValueError):
        year_val = month_val = day_val = None
 
    # -- snip assumed --

Change to render() for SelectDateWidget? and test to ensure it works

The file I just attached (wasn't logged in, oops) adds tests for SelectDateWidget? rendering for this issue, and uses isinstance instead of type() in * to see whether the value is a string. Hopefully with these, this can be patched in the trunk. (I don't know about the IDs.. should those be patched for this ticket as well, or is that a separate issue?)

Joseph

I think that value_from_datadict should return a date anyway. But that's a side point from this bug, of which the latest patch looks good to go.

Checks isinstance of string in render, and adds a test. Removes TypeError? from exceptions, since isinstance means that can't be raised (as pointed out by SmileyChris?)

Actually, I was wrong. TypeError could be raised still: datetime.date(*map(int, '1-2'.split('-')))

Nevermind removing TypeError?. It could still be raised.. this is the better version of the patch

If "years" is not specified but initial date value is, perhaps self.years should be range(value.year, this_year+10)

Also, if date validation fails (eg. February 31), it would be nice if the widget would repopulate with the failed date, rather than resetting everything back to the first entry in the list.

Otherwise nice... :)

Replying to ryan@pressure.net.nz:

If "years" is not specified but initial date value is, perhaps self.years should be range(value.year, this_year+10)

This is a separate proposal to the fix here, feel free to open a new ticket.

Also, if date validation fails (eg. February 31), it would be nice if the widget would repopulate with the failed date, rather than resetting everything back to the first entry in the list.

While this is a separate issue (bug IMO), I'll fix it here since it's not that difficult.

Rolling back to Accepted and assigning to me.

Allow widget to receive a datetime and also render invalid dates (validation doesn't belong in the widget)

(In [7337]) Fixed #5917 -- More error robustness in date parsing in SelectDateWidget?, plus keep the original date selected on redisplay, even if it was bogus (e.g. 31 Feb). Patch from SmileyChris?.