jemproject / JEM-Project

JEM - an Event Manager for Joomla
https://www.joomlaeventmanager.net
GNU General Public License v3.0
25 stars 36 forks source link

(Short) Dateformat settings for backend can't be localized #1707

Closed jojo12 closed 3 months ago

jojo12 commented 3 months ago

@mckillo: There's a post in the forum The (short) dateformat settings for backend can't be changed to m-d-y, which is used in US. This should be adapted. the author [dlborgman] says in the JEM version for Joomla 3.3 this was possible.

jojo12 commented 3 months ago

My solution is: administrator/components/com_jem/views/events/tmpl/default.php line 158: change from: $displaydate = JemOutput::formatDateTime($row->dates, null, $row->enddates, null,'d.m.Y'); to: $displaydate = JemOutput::formatShortDateTime($row->dates, null, $row->enddates, null,'d.m.Y'); is this right?

mckillo commented 3 months ago

@jojo12: Now, in the events view, the Start Date column is format as JEM Setting defined in Basic Settings - Display - Format Date (short). If the Format Date (short) is empty, then the 'd.m.Y' is the default date format. See commit above. Is it right? image

jojo12 commented 3 months ago

I think it would be better, if this is set as short format, because this is a table which needs short dates. The details view can be the long date format!

jojo12 commented 3 months ago

when I use your change, and the JEM setting for short date format is empty, there is a very strange thing instead of the date Anyway, if the JEM setting is empty, I get:

C+0000Apr._JEApr._April+0000RApr.AMUTC_th00+0000RUTC_DiAMUTCE

mckillo commented 3 months ago

Check now the commit above

jojo12 commented 3 months ago

with that, when I empty the JEM setting, it's the same strange result! But why try to let it empty?

mckillo commented 3 months ago

I agree with you, I get the same strange string. I have reviewed the code and there is missing string into INI to define the date format. See the commit above, and now It works.

mckillo commented 3 months ago

@jojo12 I have updated the commit of this case. My events view was also affected by the error. Do you agree with the solution? It it's so, close the issue.

jojo12 commented 3 months ago

I agree. thanks