MMM-compliment showing evening compliment anytime

[sdetweil]

@selyjohns I copied your compliments to my config, and do not see that

the code gets the current system time hour (in 24 hour format)

and only uses the evening list if neither morning nor afternoon ranges include the current hour (all OTHER time)

then adds anytime if specified to that list

then adds weather related if specified

then picks one

[selyjohns]

@sdetweil
Ok, thank you for your test.
So maybe a problem on my system time. However it seem to be correct. Another solution to verify that ? (Other than timedatectl ?)

Is it possible to have verbose log to see the how the module decide to display compliment ?

[sdetweil]

@selyjohns there is no debug on how it decided, but you could walk thru the code.

open the developers window ctrl-shift-i and select the source tab, and navigate the left lane to modules , default, compliments, compliments.js

the code will appear in the right pane

scroll down to

let compliments;

the next line of code is using the hour
to test if in between morning, then afternoon
the the else for evening of some

click the number in the left column, turns blue

hit F5 to reload the page. will stop there

you can hover the mouse over variables to examine their value

blue arrow upper right to run , or circle be t to step one instruction

[selyjohns]

@sdetweil Hi,
Ok great.
I be able to see the value of hour variable.
It seem to be correct because it’s 17.

Condition of morning not verified.
Condition of afternoon verified, so the line is executed (because 17 is between 12 and 18)

compliments = this.config.compliments.afternoon.slice(0);

I don’t know why evening compliments are displayed…it’s very strange

[sdetweil]

@selyjohns there are a few more lines of code to step thru…

when it gets to the return compliments u can go er over compliments and then expand the array to see what is in there

[selyjohns]

@sdetweil Yes ! Problem found…
Step by step, i’ve understand

		if (hour >= this.config.morningStartTime && hour < this.config.morningEndTime && this.config.compliments.hasOwnProperty("morning")) {
			compliments = this.config.compliments.morning.slice(0);
		} else if (hour >= this.config.afternoonStartTime && hour < this.config.afternoonEndTime && this.config.compliments.hasOwnProperty("afternoon")) {
			compliments = this.config.compliments.afternoon.slice(0);
		} else if (this.config.compliments.hasOwnProperty("evening")) {
			compliments = this.config.compliments.evening.slice(0);
		}

The JS need to have a “morning” and “afternoon” compliments define, else it choosing “evening” by default if define.

My problem was i don’t had afternoon compliment define…but i’ve evening one.

Thank you very much for your time and your time :) It was very helpful.

[sdetweil]

@selyjohns awesome…

I assume that if u don’t have any entries, you would still leave the list

afternoon: [],

[selyjohns]

@sdetweil sorry i don’t uderstand what you mean ?
I think it works if afternoon is present but empty

[sdetweil]

@selyjohns said in MMM-compliment showing evening compliment anytime:

I think it works if afternoon is present but empty

thats what I meant

[selyjohns]

@sdetweil good :)
Ok so thank you for your help. Maybe it need to be more explicit on documentation.

And they are confusion between compliments and MMM-compliments

I will now close this case.

Have a good day