Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#3050] - Date calendar format fix

Success

User tests: Successful: Unsuccessful:

avatar fastslack
fastslack
4 Feb 2014

Do not send unused % to JDate format

avatar fastslack fastslack - open - 4 Feb 2014
avatar Bakual
Bakual - comment - 4 Feb 2014

At least two issues with that:

  • $format is used within the calendar formfield on line 170. If you change it to the date format, you need a change there as well otherwise strftime() will fail.
        // Handle the special case for "now".
        if (strtoupper($this->value) == 'NOW')
        {
            $this->value = strftime($format);
        }
  • The difference between the two formats (data or strftime) are not only a leading %. Especially if it comes to the time part it will fail, but there are other places as well. For the time, strftime uses %H:%M:%S while date uses H:i:s to get the regular time format 21:15:00.

edited because I read code wrong, sorry

avatar fastslack fastslack - close - 4 Feb 2014
avatar fastslack fastslack - change - 4 Feb 2014
Status New ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2014-02-04 20:19:08
avatar fastslack fastslack - close - 4 Feb 2014
avatar fastslack fastslack - head_ref_deleted - 4 Feb 2014

Add a Comment

Login with GitHub to post a comment