Problem/Motivation

Smart Date Recurr format display time even if field is set to ALL Day

Screen Shot of smart date field

Screen Shot of Recurring format display showing time

Steps to reproduce

Add Smart Date Field to Content Type - Set field to Recurr Format in Manage Display

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#8 smart_date-all_day_timezone-3170136-8.patch769 bytesmandclu
#2 Screen Shot 2020-09-10 at 3.37.30 AM.png71.09 KBmandclu

Comments

thronedigital created an issue. See original summary.

mandclu’s picture

mandclu commented
StatusFileSize
new Screen Shot 2020-09-10 at 3.37.30 AM.png71.09 KB

I can't reproduce what you're getting. I used the same data as you, and here's what I got.

It looks like time is off by 5 hours. Have you done anything with timezones?

thronedigital’s picture

thronedigital commented

I did change my timezone from the install UTC to Chicago in hopes of fixing the time off by 5 hours, to no avail.

I'm not sure what is going on here or where to go from here.

mandclu’s picture

mandclu commented

If you edit the node, does it still show as all day?

If you create a new node does it have the same problem?

thronedigital’s picture

thronedigital commented

When i edit a node I have the all day checkbox checked

When a create a new node with the same setup - same issue.

thronedigital’s picture

thronedigital commented

Ok I rebuilt the site with the exact config

It works now.

Maybe i'm starting to understand. I'm thinking if the Drupal regional timezone gets changed it causes this issue to happen, I need to do more testing.

trishcoddington’s picture

trishcoddington commented

We are experiencing the same issue. When the Node is viewed by an anonymous user or, by a user whose Time Zone is the same as the site, an "All day" date is displayed like this:

Date: Mar 6, 2021 | All day (All day)

However, if the logged-in user is set with a Time Zone that is different from that of the site, the "All day" date is displayed like this:

Date: Mar 6, 2021 | All day (Mar 5, 2021 | 10:00pm - Mar 6, 2021 | 9:59pm PST)

We'd like it to display as shown in the first example, regardless of the user's time zone setting.

mandclu’s picture

mandclu commented
Status: Active » Needs review
StatusFileSize
new smart_date-all_day_timezone-3170136-8.patch769 bytes

Fair, there's really no point in showing a local time display for an event that's meant to be all day. The attached patch should cause the local display to not be shown for all day events

mandclu’s picture

mandclu commented
Status: Needs review » Fixed

I'm not 100% sure the solution implemented in the patch aligns with what you wanted, but merging this in for now. Please feel free to open a child issue if you need some additional refinement.

  • mandclu committed cc2f8c6 on 3.1.x 
    Issue #3170136 by mandclu: Smart Date Recurr Format showing time even if...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.