Home › Forums › Calendar Products › Events Calendar PRO › $this->daysOfWeekShort in Main.php still not fixed
- This topic has 6 replies, 2 voices, and was last updated 7 years ago by Mister.
-
AuthorPosts
-
March 13, 2017 at 3:20 am #1253216MisterParticipant
Hi,
I reported this bug already in January: https://theeventscalendar.com/support/forums/topic/deprecated-4-4-daysofweekshort-shows-months-instead-of-days/
In the-events-calendar/src/Tribe/Main.php on line 2196 there is an annoying copy-paste bug.
Current wrong code:
$this->daysOfWeekShort = Tribe__Date_Utils::get_localized_months_short();
Correct code would be
$this->daysOfWeekShort = Tribe__Date_Utils::get_localized_weekdays_short();
I wait for this fix now since more than 2 month, and after every release I need to manually fix by correcting the core files.
Please address asap, this is such an easy fix.
March 14, 2017 at 3:53 pm #1254120NicoMemberHey Martin,
Thanks for reaching out once again on this! Although the properties are deprecated (and will be removed in the future), you are right about the confusion in the code.
I couldn’t find this logged, but will do now and include the fix in the next maintenance release, coming out shortly.
Thanks once again for insisting on this, and sorry we lost track of your report,
Best,
NicoMarch 14, 2017 at 3:57 pm #1254123MisterParticipantHi Nico,
thanks for taking care of that.
One more question: As those properties are deprecated – how is the functionality then covered? Are there new properties for the day names?
March 17, 2017 at 8:57 am #1255849NicoMemberNo problem Mister (sorry I called you Martin before, my bad)!
I would use Tribe__Date_Utils::get_localized_weekdays_short(); in your custom code. Although it’s deprecated we are still using it internally. Some more changes will be done after the propriety is removed to account for this.
I’ve set the thread status to Pending Fix so it doesn’t get autoclosed + you’ll get a heads up when the fix is released 🙂
Best,
NicoMarch 23, 2017 at 7:36 pm #1259260NicoMemberHi there Mister!
Just wanted to share with you that maintenance release 4.4.5 of our plugins is out, which includes a fix for this issue 🙂
Find out more about the release → https://theeventscalendar.com/maintenance-release-events-calendar-4-4-5-event-tickets-4-4-5-premium-add-ons/
Best,
NicoApril 14, 2017 at 9:35 am #1270098Support DroidKeymasterHey there! This thread has been pretty quiet for the last three weeks, so we’re going to go ahead and close it to avoid confusion with other topics. If you’re still looking for help with this, please do open a new thread, reference this one and we’d be more than happy to continue the conversation over there.
Thanks so much!
The Events Calendar Support Team -
AuthorPosts
- The topic ‘$this->daysOfWeekShort in Main.php still not fixed’ is closed to new replies.