No translation for timestamps with ISO 8601

[herrvigg]

Issue by felixwerner
Wednesday Mar 11, 2015 at 22:01 GMT
Originally opened as qTranslate-Team/qtranslate-x#76

---

timestamps are translated if time format convert function is turned on.
a exception in the qtranxf_convertFormat() function would be great for iso timestamps if using get_the_date('c')

[herrvigg]

Comment by fburatti
Friday Mar 13, 2015 at 11:13 GMT

---

When I used Mqtranslate, the ideal setting for the dates in my themes was "Use strftime instead of date formats and replace with the predefined formats for each language."

Now with Qtranslate-x seems more compatible "Use emulated date function".

get_the_date ('c') does not work, then I use this snippet to get a valid date format according to [RFC 3339]:

 <?php echo date('c',strtotime(qtranxf_formatPostDateTime('%D %T','',''))); ?>

[herrvigg]

Comment by johnclause
Tuesday Mar 17, 2015 at 06:57 GMT

---

Hi guys, could you test the latest version from GitHub? It has some changes in date/time functions, but I am not sure it they would address this problem.

[herrvigg]

Comment by fburatti
Tuesday Mar 17, 2015 at 09:48 GMT

---

Hi John, in a fresh install, I can confirm that get_the_date ('c') work correctly, both with version 3.2.7 that with the 3.2.8 pre-release (with "use emulated date function" enabled).

[herrvigg]

Comment by johnclause
Tuesday Mar 17, 2015 at 16:10 GMT

---

Thank you, @filippo, I guess we can close this one then? You can still write to closed issue and we can re-open it again if needed.

[herrvigg]

Comment by felixwerner
Tuesday Mar 17, 2015 at 16:28 GMT

---

on both versions i still have the same problem when using get_the_date( 'c' ).
but maybe i wasnt clear enough the last time.
it works well if i use the emulated date option but if i use the the emulated date with language translation i still see: March 11th, 2015 instead of: 2015-03-11T17:20:39+00:00

crawling through the plugin files i found this function qtranxf_convertFormat() which has a exception for
get_the_date( 'U' ) to not convert this date format. would it be possible to do this for c as well?

this would be especially useful for other plugins like yoast seo which are using for meta fields ISO timestamps with get_the_date( 'c' )

[herrvigg]

Comment by johnclause
Tuesday Mar 17, 2015 at 17:51 GMT

---

@felixwerner: the date/time code of former qTranslate almost never got updated and apparently quite obsolete by now. There must be a better way of doing the whole thing by now in a more native way. Do you understand PHP code? Would you mind to take a lead to find out the best way and to implement it?

[herrvigg]

Comment by johnclause
Wednesday Mar 18, 2015 at 06:51 GMT

---

@felixwerner: Please, help me to understand how it is supposed to work, I never paid close attention to date/time conversions so far. Do you mean by "emulated date with language translation" the choice "Use emulated date function and replace formats with the predefined formats for each language."? If this is so, then 'c' will be overridden with what you have in the field "Date Format" of language configuration, and then it worked correctly in my test. Is this how it should be?

[herrvigg]

Comment by felixwerner
Wednesday Mar 18, 2015 at 20:34 GMT

---

yes if using the 'Use emulated date function and replace formats with the predefined formats for each language.' the 'c' will be overridden with general language date settings
there is a exception for 'U' to not overwrite the format if you choose this option.
i now added to the qtranxf_convertFormat() function in qtranslate_utils.php following line:

if('c'==$format) return qtranxf_convertDateFormatToStrftimeFormat($format);

it seems to work now if this line is added the 'c' date will show correctly.

[herrvigg]

Comment by johnclause
Wednesday Mar 18, 2015 at 20:43 GMT

---

I believe, exception for 'U' is there because 'U' basically means unformatted, raw format, so it stays raw in any case. But 'c' would not fit this logic. Why do you need exception, could not you use "Use emulated date function" option without override, if you provide your own format?

[herrvigg]

Comment by johnclause
Thursday Mar 19, 2015 at 06:13 GMT

---

In fact it seems that there are 4 special formats, which may not need to be converted: 'Z', 'c', 'r' and 'U'.
Maybe all 4 need to be excluded? I am afraid to break people sites by changing this behavior.

I do not really know how it should be, I did not designed this thing, and I am pretty sure there is a better WP standardized way by now. Please, help me to design it, if you have knowledge on how WP now expects to do date time localization.

[herrvigg]

Comment by johnclause
Thursday Mar 19, 2015 at 20:06 GMT

---

I added exceptions language-neutral formats exceptions ( 'Z', 'c', 'r', 'U') to the qtranxf_convertFormat on the latest on GitHub development version (3.2.8): https://github.com/qTranslate-Team/qtranslate-x. Please, @felixwerner, test that it does what you need, and @fburatti test that it does not break anything for you. Thanks.

[herrvigg]

Comment by fburatti
Thursday Mar 19, 2015 at 23:11 GMT

---

I was trying the beta version 3.8 but then you have already published the 3.9!
It all seems to go well, including seo Yoast meta tags (article: published_time, etc ...)
I tried all options but "Use emulated date function" remains the most accurate.

[herrvigg]

Comment by johnclause
Friday Mar 20, 2015 at 00:14 GMT

---

Thank you, @fburatti, 3.2.9 is the same as 3.2.8, I just increased the number, so that when people update the release, it would not mix with development version. Thanks a lot for your testing, if you found something I would do another release right away ;)

I guess, we can close this issue. We can still write to closed issue and may reopen it later if needed.