Needs work
Project:
Smart Date
Version:
4.2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Aug 2021 at 17:23 UTC
Updated:
24 Feb 2026 at 16:47 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
bbombachiniComment #3
bbombachiniComment #4
mandclu commentedThanks for identifying this, and for providing a patch.
I confess that as someone deeply familiar with the inner workings of this module, I am aware that it is possible to expose the end date field by setting the time close enough to the end of the day that the end time will be on the next day, after which the end date can be changed as desired. I recognize that it isn't reasonable to expect a user to understand that process, however.
The only potential issue I have with your patch is setting the date to a null value, which seems like it could create some unexpected behaviour when validating or saving the values.
That said, I don't have the time available to test it right now, so marking this as "needs review" for now.
Comment #5
hip commentedNot sure if talking about same issue. I use 'custom' and '0' as the only allowed duration increments, being 0 the value by default. Once in the 'create a new node' form I can't access to a second date form field (only second time one). I choose 'custom' from the 'Duration' drop down but still only get the field for Date1, Time1 and Time2. No Date2 is shown.
Comment #7
damienmckennaComment #10
damienmckennaI created a MR that works against the 4.2.x branch.
FYI this does not resolve the problem with the overlay option.
Comment #11
damienmckennaIt does work with the overlay, but I think there's a bug in the logic - if I set the start time to 9pm and select a duration of 4 hours it does show the end date field, but if I manually edit the end time to 1am it does not show the date field.