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

Request to update America/Mexico_City Timezone info #645

Open
rloview opened this issue Aug 4, 2023 · 2 comments
Open

Request to update America/Mexico_City Timezone info #645

rloview opened this issue Aug 4, 2023 · 2 comments
Assignees

Comments

@rloview
Copy link

rloview commented Aug 4, 2023

Describe the bug
I use version 1.0.7 or 3.2.12
I think that need to update the timezone info of America/Mexico_City.
The America/Mexico_City time zone no longer applies DST.
However, if I create a DtStart object with the Mexico_City timezone, the time sets with DST applied.

To Reproduce

import net.fortuna.ical4j.model.DateTime;
import net.fortuna.ical4j.model.TimeZone;
import net.fortuna.ical4j.model.TimeZoneRegistry;
import net.fortuna.ical4j.model.TimeZoneRegistryFactory;
import net.fortuna.ical4j.model.property.DtStart;

public class Test {
	public static void main(String[] args) throws Exception {
		long current = System.currentTimeMillis();
		
		DtStart dtStart = new DtStart(new DateTime(current));
		TimeZoneRegistry registry = TimeZoneRegistryFactory.getInstance().createRegistry();
		TimeZone dtStartTimezone = registry.getTimeZone("America/Mexico_City");
		dtStart.setTimeZone(dtStartTimezone);
		System.out.println("America/Mexico_City DtStart " + dtStart.toString());
	}
}

Expected behavior
The output information should be DTSTART;TZID=America/Mexico_City:20230804T043327 same as GMT-6.
However, the result of executing that code is DTSTART;TZID=America/Mexico_City:20230804T053327

Screenshots
image

@rloview
Copy link
Author

rloview commented Aug 17, 2023

@benfortuna Hello. Could I know the schedule that the timezone information will be updated to 2023c version?
For example, I checked in the file below that the latest version is using 2022g.
https://github.com/ical4j/ical4j/blob/ical4j-3.2.12/src/main/resources/zoneinfo-global/America/Mexico_City.ics

I'm sorry to interrupt but please reply.

@benfortuna benfortuna self-assigned this Aug 17, 2023
@benfortuna
Copy link
Member

Hi, ical4j has the option of updating itself at runtime with the latest timezone definitions from tzurl.org. So whilst the definitions in git may be a bit out of date, it is best to check the tzurl.org site to see if that definition is correct.

We also maintain that site, so if you find that definition is incorrect let me know: https://www.tzurl.org/zoneinfo/America/Mexico_City

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants