Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.I only want to set the scheduling date, not the time (to simplify things for users).
On a site I upgraded where the date format was d/m/Y everything worked fine - date popup, but no time entry box or time picker.
But as soon as I tried to submit the Scheduler module settings screen I get the message "The Date Popup module only accepts the following formats: H:i:s, h:i:sA". The Reset to defaults button doesn't quite work either - I get the same error message (this is with an initial scheduler_date_format variable value of d/m/Y.
So there seems to be something up with the validation... i.e. 1) the validation shouldn't run when you Reset to defaults, and 2) it would be great if the empty string were permitted as the time format. Oh, and 3) the error message would make more sense if it included the word "time" before formats.
In the meantime, manually setting the scheduler_time_format variable to something like s:5:"Y-m-d"; or s:5:"d/m/Y"; seems to do the trick.
Thanks for all the work getting the date popup integrated (you can probably tell how long it is since I last upgraded scheduler module!!!!)... much nicer to use :-)
PS I'm sure I saw some discussion about this in another issue but can't find it now.
Comments
Comment #1
jonathan1055 CreditAttribution: jonathan1055 commentedHello gpk,
Yes this does seem to be a problem, I've just tested it in D6. I agree with your (1), (2) and (3), and also that editing the variable seems to work and you can select just a date via the calendar. So I'm not sure of the reasons behind that specific validation when selecting Date Pop-up as the entry method.
Are you still on Drupal 6 or have you moved to D7 now? We would need to check what happens in D7.
Jonathan
Comment #2
jonathan1055 CreditAttribution: jonathan1055 commentedIn D7 we do not have a 'reset' button so point (1) is only in relation to D6 and therefore probably is not critical enough to warrant the change.
For point (2) we have an issue #1720338: Allow for no time field on date popup which will do exactly as you require, and in conjunction with #1069668: Default time with user override you have the ability to set the time which will be used behind the scenes. Please follow those issues and when there are patches to test, you are welcome to do so, and give us your feedback.
For point (3) yes the wording of that error message should be improved. I will do that in conjunction with another patch for better text description of the values allowed in the format.
Thanks
Jonathan
Comment #3
jonathan1055 CreditAttribution: jonathan1055 commentedSorry, I should not have set this to fixed until those other issues are implemented.
Comment #4
jonathan1055 CreditAttribution: jonathan1055 commentedThe functionality requested is now covered by other issues - see the related box. Hence closing this issue now.