joomla / joomla-cms

Home of the Joomla! Content Management System
https://www.joomla.org
GNU General Public License v2.0
4.76k stars 3.64k forks source link

issues when entering dates in persian calendar #28702

Closed rasouldanesh90 closed 4 years ago

rasouldanesh90 commented 4 years ago

Hello We choose a solar date On some dates, a unit is added to the month or day, and we reopen the calendar or save the content. The Joomla version I am using is 3.9.12.

@farhadst @dGrammatiko

infograf768 commented 4 years ago

Is this the solution ? https://github.com/joomla/joomla-cms/pull/28700 Please test.

rasouldanesh90 commented 4 years ago

I tested your solution, no change I think this problem should be in the file media / system / js / fields / calendar.js Lines 1014 to 1042 that change the value "data-alt-value" when the calendar opens

infograf768 commented 4 years ago

This file has not been touched for years and is a dynarch project. I would not touch it even with twisters...

richard67 commented 4 years ago

@rasouldanesh90 3.9.12 is a bit old, current version is 3.9.16, and 3.9.17 will be released soon. You should keep your Joomla always up to date because of security patches. Beside this, could you check if the corrections from pull requests #28596 and #17859 solve that issue? You should apply both changes together.

infograf768 commented 4 years ago

Also, please specify some dates concerned (in gregorian format)

balvard commented 4 years ago

This error appeared when you want to declare personal function to control date in persian and we need return true date but returned null In gregorian is true But joomla team forget to return true data for those function

infograf768 commented 4 years ago

@balvard It is impossible to do anything if you do not explain exactly (with code) what you want to do and where precisely, in your opinion, is the error in the js files. fa-IR is using a custom calendar.js file which is installed in media/fa-IR/js/ Here is its uncompressed version calendar-uncompressed.js.zip

Quy commented 4 years ago

Closing due to lack of response. You can comment here and we can re-open the issue.