azerothcore-wotlk icon indicating copy to clipboard operation
azerothcore-wotlk copied to clipboard

(Holidays): Holiday dates must be filled for the next however many years.

Open heyitsbench opened this issue 1 year ago • 15 comments

Current Behaviour

We have run out of Darkmoon Faires.

Expected Blizzlike Behaviour

We need more Darkmoon Faires.

Source

Assumption based on the established eternal presence of the Darkmoon Faire.

Steps to reproduce the problem

  1. Open the calendar.
  2. Observe the Darkmoon Faire is no more.

Extra Notes

We just need to add more dates for events in holiday_dates. Shouldn't be too difficult.

AC rev. hash/commit

be48e3c2493b420c62c293897e321c0bf7c0483f

Operating system

Windows 10 x64

Custom changes or Modules

mod-ptr-template

heyitsbench avatar May 19 '23 15:05 heyitsbench

Is it just out of the calender? Because it shouldnt stop as the end time is 2030-01-01 06:00:00

Kitzunu avatar May 21 '23 20:05 Kitzunu

Is it just out of the calender? Because it shouldnt stop as the end time is 2030-01-01 06:00:00

Yes, it only affects the in-game calendar, the events still play out in the world as expected.

heyitsbench avatar May 21 '23 20:05 heyitsbench

confirm

Darkmoon is displayed on the calendar

lemondsMs avatar May 22 '23 09:05 lemondsMs

Good day, players at CC discovered this bug in game and started to talk about it in sever discord (because now it's time for DMF).

kerricsson avatar Jun 02 '23 17:06 kerricsson

We should never have to manually feed by hand the upcoming recurrent world events such as Darkmoon Faire. It should have been automated in the first place. Otherwise, we'll just push the very same problem later.

Synful-Syn avatar Jun 04 '23 00:06 Synful-Syn

Whichever solution comes first should be acceptable.

heyitsbench avatar Jun 04 '23 01:06 heyitsbench

Could anyone see if this works? https://gist.github.com/Gultask/c06f6dbb543594618164ffc2241c31ec

Uses the Classic schedule convention

Gultask avatar Jun 04 '23 18:06 Gultask

image

Doesn't seem to do anything

AFAIK that's one of AC's tables created with the idea of moving client data to the database, I am not sure if it's even implemented tbh

Nyeriah avatar Jun 04 '23 20:06 Nyeriah

I definitely managed to switch dates around for existing entries, but new ones don't appear for me

Gultask avatar Jun 04 '23 20:06 Gultask

AFAIK that's one of AC's tables created with the idea of moving client data to the database, I am not sure if it's even implemented tbh

Doubt that, considering the table exists (and seems to work) on TC.

heyitsbench avatar Jun 04 '23 20:06 heyitsbench

Alright I figured it out after much fiddling... It only accepts 25 entries. (???)

So we have to delete old ones

Gultask avatar Jun 04 '23 20:06 Gultask

I did, however, not notice that the start date mentioned is for the building event and not DMF itself so it's starting every Tuesday 😩

Gultask avatar Jun 04 '23 20:06 Gultask

#define MAX_HOLIDAY_DATES 26 ? why?

Gultask avatar Jun 04 '23 20:06 Gultask

why?

Fun.

heyitsbench avatar Jun 04 '23 20:06 heyitsbench

It is due to client limitations 25

Kitzunu avatar Jul 30 '23 10:07 Kitzunu