calendar - wrong repeating count when using sliceMultiDayEvents

sdetweil

@MarcLandis ok. system timezone.

sdetweil

change calendar.js 3 places
give it a try

change (line 633)

const maxCount = Math.ceil((event.endDate - 1 - moment(event.startDate, "x").endOf("day").format("x")) / ONE_DAY) + 1;

to

const maxCount = Math.round((event.endDate - 1 - moment(event.startDate, "x").endOf("day").format("x")) / ONE_DAY) + 1;

ceil takes 1.04 to 2, round to 1

					let midnight
						= moment(event.startDate, "x")
							.clone()
							.startOf("day")
							.add(1, "day")
							.endOf('day')  // add this (else its start of day, not end) line 641
							.format("x");

and recalc of midmnight (line split like above for clarity) line 654

	midnight = moment(midnight, "x")
                                   .add(1, "day")
                                   .endOf('day')  // add this 
                                   .format("x"); // next day

works in Europe/Berlin and America/Chicago 
2 bugs
1 count incorrect
2 loop control

MarcLandis

@sdetweil this fixes it. The test calendar looks good and my real one with the birthdays (mostly affected one) is perfect. All other events are good too.

Thx for your help.

sdetweil

@MarcLandis i’ll submit an issue and a pr…

and a testcase

sdetweil

@MarcLandis
submitted PR https://github.com/MagicMirrorOrg/MagicMirror/pull/3543/
with new testcase to validate

hopefully we will get it into next release Oct 1.

MarcLandis

@sdetweil Thank you - even it doesn’t make it into the next release I have a way to manually fix it now.

sdetweil

@MarcLandis the fix was accepted and merged

sdetweil

@MarcLandis you could test it now by
https://forum.magicmirror.builders/topic/14327/testing-new-fixes-or-solving-current

remember to do (to get release level dependencies)

npm run install-mm

assuming the config in this MM is set correctly for sliceMultiDayEvents

MarcLandis

@sdetweil I was on a business trip the last days.

I noticed that multi fullday events will be shown one day in the past:

This is from today (20. Sep)

I don’t have the config to show past days enables.

sdetweil

@MarcLandis hm, same calendar entries ?

is this on the edited calendar.js or the develop branch?

im on a trip til tuesday