Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

weird display of date #699

Open
papillon-unix opened this issue May 4, 2024 · 6 comments
Open

weird display of date #699

papillon-unix opened this issue May 4, 2024 · 6 comments

Comments

@papillon-unix
Copy link

papillon-unix commented May 4, 2024

Hello

I'm new to Pico, working on a website with it and I got to love this ultra-light cms very quickly.

Since I need PHP 8 for other needs I run the 3.0.0-alpha.2

Everything works fine except one very annoying thing :

When a YAML file contains a date, for example :

date: 2024-05-05 18:30:00

And when I tried to display it in a template, whether with a date filter or not :

{{ post.date|date("m/d/Y H:i") }} or {{ post.date }}

it returns an UTC time instead of the right time of my timezone. In this case it will display:

2024-05-05 20:30:00 instead of 2024-05-05 18:30:00

I checked that my config is set to Europe/Paris and when I tried to display {{ "now"|date("m/d/Y H:i") }} it returns the correct time.

I searched for hours a solution to this problem and couldn't find it.

So I had to patch my template with something like {% set adjustedDate = post.date|date_modify('-2 hours') %} to get the correct time.

Any idea where does it come from?

Besides, so sad that PicoCMS is abandonned. I tried many flat file cms and this one is really the lightest and the best for my needs.

Thanks a lot if anyone reads me.

@PhrozenByte
Copy link
Collaborator

Check the timezone config in Pico's config.yml. Also note that page.date already contains a date string; use page.time to access the timestamp, or page.date_formatted to print the formatted date according to the date_format config in Pico's config.yml.

@papillon-unix
Copy link
Author

Thanks a lot for the answer.

I did put the timezone in config.yml

I can't really use date_formatted because I want to format differently under different situations.

I doesn't work any better with page.time - I still get a date convert to UTC instead of the actual value of date written in the file..

@PhrozenByte
Copy link
Collaborator

PhrozenByte commented May 6, 2024

The timezone config in config.yml tells Pico to call PHP's date_default_timezone_set(), which in turn tells PHP the default timezone to use with date(). So it can't be an issue with date() (i.e. the output), but with the input.

Pico doesn't parse the date header on its own, but simply uses the value provided by Symfony YAML, the YAML parser Pico uses. So I just checked and the YAML specs apparently specify that a date string must be interpreted as UTC if no timezone is given. I assume that Symfony YAML just follows the specs here and therefore interprets your date string as UTC.

Thus, please try adding a timezone to your YAML headers, like:

date: 2024-05-05 18:30:00+02:00

@PhrozenByte
Copy link
Collaborator

I assume this to be pretty unexpected. We should probably overrule the YAML specs here and use the timezone config in config.yml instead - if this is even possible (see below). Thus changing this to a upstream bug. However, please don't expect a fix in the near future (if ever, as you know, Pico's development has stopped).

A few notes for a possible future fix: The current Symfony YAML version returns just a timestamp (timezone-independent by design), but newer Symfony YAML releases allow returning a DateTime instance using some parser flag. However, it might still be impossible to safely distinguish between a DateTime instance that used UTC as fallback, or a user specifically telling the timezone to be UTC (e.g. 2024-05-05 18:30:00Z). There's currently no option in Symfony YAML to change the default timezone; we could propose an upstream patch, but since the YAML specs seem to be pretty clear about the matter, I'm not so sure whether they would accept such patch. So, in the end, I'm not sure whether we can fix it. If not, we should at least document it...

@papillon-unix
Copy link
Author

Thank you so much. Following your suggestion, I added the timezone directly to the YAML headers, and it works perfectly. Your explanations are very clear. I was initially confused because I had previously used the CMS Stacey, where such specification wasn't necessary. I switched to Pico due to the PHP 8 compatibility of its fork. Once again, thank you. I'll stick with my template-side fix.

I've heard that Pico has been sold and that the new owner isn't responding to commits. Is that correct? Is it possible that they acquired it with the intention to shut it down?

@PhrozenByte
Copy link
Collaborator

Let's keep this open since it's a unfixed bug 👍

I've heard that Pico has been sold and that the new owner isn't responding to commits. Is that correct? Is it possible that they acquired it with the intention to shut it down?

Gilbert, the original author, sold the project a very long time ago to Greg, the current owner. AFAIK Greg is no PHP developer, he just owns the project. I took over development some years ago and basically rewrote it from scratch while keeping Gilbert's base concept, mostly because I was using it myself. However, I no longer have the time (nor will tbh...) to work on Pico. I still provide some basic support for the remaining users though...

Since nobody else wanted to take over responsibility yet, Pico is basically abandoned from a development perspective. However, I believe that if someone wants to take responsibility, one could still get write access, just as I did years ago. I'm not aware of any plans to shut it down.

@PhrozenByte PhrozenByte reopened this May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants