?
avatar balvard
balvard
3 Aug 2017

Steps to reproduce the issue

In persian language if choose calendar to select a date you see an error.
Date is currct but day name incorrect.
Before change

Expected result

if change this code in file date-helper.min.js the problem will be ok.
After change

Actual result

At end of this file (media/system/js/fields/calendar-locales/date/jalali/date-helper.min.js) change this code :

from : return t=(t+1)%7};
to : return t=(t)%7};

System information (as much as possible)

Additional comments

solved url
Solved file
Best Reg.
Alireza Balvardi

avatar balvard balvard - open - 3 Aug 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 3 Aug 2017
avatar balvard balvard - change - 3 Aug 2017
The description was changed
avatar balvard balvard - edited - 3 Aug 2017
avatar balvard balvard - change - 3 Aug 2017
The description was changed
avatar balvard balvard - edited - 3 Aug 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 3 Aug 2017
Priority Critical Medium
avatar franz-wohlkoenig franz-wohlkoenig - change - 3 Aug 2017
Category Language & Strings Multilanguage JavaScript Language & Strings Multilanguage
avatar tonypartridge
tonypartridge - comment - 3 Aug 2017

@dgt41 can you review/make a pull?

avatar franz-wohlkoenig franz-wohlkoenig - change - 3 Aug 2017
Status New Discussion
avatar balvard
balvard - comment - 3 Aug 2017

I solved this problem.
I wait to update joomla and release this problem.
please see images.

avatar balvard balvard - change - 3 Aug 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-08-03 07:39:37
Closed_By balvard
avatar balvard balvard - close - 3 Aug 2017
avatar tonypartridge
tonypartridge - comment - 3 Aug 2017

@balvard I saw that's why I asked dimitirs to review since he built the calendar into Joomla! You could create a pull request for your change @balvard

avatar infograf768
infograf768 - comment - 3 Aug 2017

@farhadst
Please have a look

avatar dgt41
dgt41 - comment - 3 Aug 2017

@balvard please make a PR with your changes so we can test it! Thanks

avatar balvard
balvard - comment - 5 Aug 2017

Hi to everyone
This problem just see in persian language and refer to jalali script in joomla 3.7.x
for test you can download latest joomla version (3.7.4) and install persian language.
so login with persian language and add new article , then in publishing tab choose calendar and make print screen.
then download this file :
https://www.alirezabalvardi.ir/media/system/js/fields/calendar-locales/date/jalali/date-helper.min.js
and replace it on your joomla .
and open new article in new tab and select calendar.
Best Reg.
Alireza Balvardi

avatar farhadst
farhadst - comment - 6 Aug 2017

@infograf768 Yes, this problem was to persian calendar
with file replacement problem fixed

avatar infograf768
infograf768 - comment - 6 Aug 2017

@dgt41 not sure @balvard will do pr, can you or shall I ?

avatar dgt41
dgt41 - comment - 6 Aug 2017

@infograf768 please do so if you can!

avatar infograf768
infograf768 - comment - 7 Aug 2017

On it.

avatar infograf768
infograf768 - comment - 7 Aug 2017

Add a Comment

Login with GitHub to post a comment