Default Clock wrong after update

[BKeyport]

@drstang At least you can override it for now…

[Richard238]

Just updated my MM, clock is showing 18:13 when it’s 11:54…

[sdetweil]

@drstang I don’t understand this, the code does

		if (this.config.timezone) {
			now.tz(this.config.timezone);		

which means it only uses its own config values, not the global config.

when I set it like this

		{
			module: "clock",
			position: "top_left",
			config:{
				displaySeconds:false,
				timezone:"UTC"
			}
		},

it correctly changed the displayed time from 08 (us central time) to 13

if I set it like this

	timeFormat: 24,
	units: "metric",
	timezone:"UTC",
	modules: [

it did not change the displayed time (as expected)

[BKeyport]

@sdetweil Correction to your statement. timezone is set to null by default. timezone: null, - therefore, that if line would evaluate to false unless you override with timezone config item.

[sdetweil]

@BKeyport correct,

so a ‘forced’ timezone would not be set

this is the same code as before.

[BKeyport]

@sdetweil

@sdetweil said in Default Clock wrong after update:

which means it only uses its own config values, not the global config.

This is what I was correcting. ;)

[sdetweil]

@BKeyport but he said,

changing it in the module config did NOT work
but
changing it in the GENERAL config DID work…

and it was NOT set in the original config he was using prior to the upgrade

[sdetweil]

@BKeyport said in Default Clock wrong after update:

This is what I was correcting.

I don’t understand…

what I said was

this module only adjusts the timezone when

    config :{ 
        timezone: '??????',
    }

in its module definition

NOT when its set in the global configuration, (before modules defined)

timeFormat: 24,
	units: "metric",
	timezone:"UTC",  <---  added this for testing, value NOT set in module config 
	modules: [

so I don’t see what you are ‘correcting’

[drstang]

@sdetweil thats correct - when I changed timezone from ‘null’ to ‘America/New_York,’ the module did not work at all. When I went into the MM config.js and added the timezone line, it worked.

Which now I realize is also a bit frustrating because one of my modules also shows sunrise/sunset times, and thats still off (EST + 4 hrs).

[drstang]

@sdetweil and also, yes. prior to the upgrade, my general config was NOT set to a TZ prior to the upgrade. I