I have an event content type that includes a date field using the "repeating date" widget. When I click on "Next" on the "Set parameters for 'Modify node fields'" page I get a message "The UNTIL value is required for repeating dates." in red at the top of the page. But I don't want to check the box next to the date field, since I don't want to change any dates (if this box is checked then the error message goes away, even if I leave all the date info blank).

Although this may ultimately need fixing in Date, I wonder what is different about the form submission when the repeating date field's checkbox is or isn't checked, that causes Date (or probably Date Repeat API to be precise) to throw a wobbly?

CommentFileSizeAuthor
#2 580090_repeat_validate.patch1.07 KBavivalma

Comments

patchak’s picture

Hey there I can confirm and reproduce this. I have the exact same issue.

Using all fresh copies from two days ago of date API, CCK, Views, VBO ...

It seems that it refuses to work when there is no value in the until... But we can't add anything without enabling the field, which is not desired either, of course.

avivalma’s picture

Title: Modifiy node fields - conflict with date module's repeating date field widget » Date_repeat - do not require UNTIL on empty date field
Project: Views Bulk Operations (VBO) » Date
Version: 6.x-1.8 » 6.x-2.x-dev
Status: Active » Needs review
StatusFileSize
new1.07 KB

I fixed this issue.
However i'm not entirely sure that empty($form_values[$field_name]['value']) this part of IF condition is important anymore.

cybermache’s picture

#2 patch worked for me. I was trying to change the Node's Comment access depending on if another CCK field had a certain value. Even though I wasn't selecting the date fields in the 'Modify node fields' action in Rules this error kept appear which restricted me from completing my RULE.

Now is works like a charm.

Thanks

damienmckenna’s picture

Issue summary: View changes
Status: Needs review » Closed (won't fix)

Unfortunately the D6 version of this module is no longer supported, but we appreciate the time you put into this. If this problem is relevant for D7 too, please reopen the issue. Thanks.