- π¦πΊAustralia mingsong π¦πΊ
I close it as 8.x branch is not supported anymore.
Which means the new feature quested won't be consider anymore.
- Status changed to Active
over 1 year ago 12:30pm 3 April 2023 - πΊπΈUnited States lpeabody
Re-opened because the patch in #17 was re-rolled against the 5.x branch and the feature request is still valid (5.x doesn't support all-day functionality for a single day).
- πΊπΈUnited States sacarney
Is this feature request meant to address the undesired display of a time on All Day events? If the event is All Day, I don't want the time (12:00AM) to appear on the event in the calendar.
- π¨π¦Canada chrisck Vancouver, BC π¨π¦
While the original issue summary does not mention using smart_date for all day functionality, smart_date is mentioned quite a few times in the comments. I'm reporting that smart_date 4.0.2 is working out of the box with fullcalendar_view 5.1.12 with all day functionality, without requiring any patches.
Perhaps what some are missing is that the field type has to be "Smart date range" and not, the core "Date range" with Smart date widget.
Download and enable the Smart Date Calendar Kit β on a new Drupal install and look at how it's configured.
- π¨π¦Canada geekygnr Waterloo
This patch causes warnings if the end date is not set.
- π¨π¦Canada geekygnr Waterloo
Here is a new patch. Adds some checks for empty values and addressed some phpcs issues.
- πͺπ¨Ecuador LeonelEnriquez98
I updated the patch that @geekygnr left in comment #25 to be compatible with the latest version
5.1.13
- πΊπΈUnited States ferminaguilar
@LeonelEnriquez98 The patched worked for the most part but its posting two days. So I post an event for 1 day 11/20 but when viewing it's showing all day for 11/19 and all day 11/20 for the same event. I just added the start date only and also did the say day for Start and End date and still the same issue.
- πͺπ¨Ecuador LeonelEnriquez98
@ferminaguilar I could reproduce that error. It was related to the
timestamp
variable. So here I leave the new patch. @ferminaguilar @LeonelEnriquez98 I still had the same issue with all-day events starting on the day before they should after applying patch #28. I believe the issue is because when finding the start date, it applies the difference from UTC to a date which has already had the difference applied to it. So it applies the timezone difference twice. Here's a new patch.
- πͺπ¨Ecuador LeonelEnriquez98
@jennine Your patch seems to be working fine. Thanks!
- πΊπΈUnited States xpersonas
Patch #30 fixed the starting a day early for me... but now they are running a day over. I assume that's because my events are coming in with a start date of 00:00:00 day 1 to 00:00:00 day 2. Trying to figure out if I need to adjust the patch for my situation or if I have something configured wrong.
- Status changed to Needs work
4 months ago 6:00pm 12 July 2024 - π©πͺGermany luenemann SΓΌdbaden, Germany
-
+++ b/src/FullcalendarViewPreprocess.php @@ -160,8 +160,8 @@ class FullcalendarViewPreprocess { - $timezone = !empty($start_field_option['settings']['timezone']) ? - $start_field_option['settings']['timezone'] : date_default_timezone_get(); + $timezone = !empty($start_field_option['settings']['timezone_override']) ? + $start_field_option['settings']['timezone_override'] : date_default_timezone_get(); // Set the first day setting.
This reverts changes from π Entries always rendered with the field selected timezone Fixed . But that removes support for datetime and daterange formatters.
timezone is a setting of timestamp formatter.
This should be fixed in a separat issue to support both formatters. -
+++ b/src/FullcalendarViewPreprocess.php @@ -332,7 +332,7 @@ class FullcalendarViewPreprocess { - elseif (strpos($end_field_option['type'], 'daterange') !== FALSE) { + elseif (strpos($end_field_option['type'], 'daterange_default') !== FALSE) {
This change removes support for daterange_plain and daterange_custom formatters. It should be reverted.
Needs work for 2.
-
- Merge request !70Issue #2976922 "Add functionality for all-day events" β (Open) created by luenemann
- First commit to issue fork.
- Merge request !77fullcalendar_view-functionality_for_all_day_events-2976922-31.patch β (Open) created by mortona2k
- πΊπΈUnited States mortona2k Seattle
Rebased on 5.x and cleaned up a section with different checks on $options.