modified date format

[vanshikaarora]

Earler the date wasn't correctly localized Changing format of display date in SimpleDateFormat function brings it to the format as mentioned

Fixes #2336 Image details: date not correctly localized in Japanese

What changes did you make and why?
Modified media detail fragment

[codecov-io]

Codecov Report

Merging #2341 into master will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff           @@
##           master   #2341   +/-   ##
======================================
  Coverage    5.94%   5.94%           
======================================
  Files         237     237           
  Lines       11348   11348           
  Branches     1022    1022           
======================================
  Hits          675     675           
  Misses      10620   10620           
  Partials       53      53

Impacted Files	Coverage Δ
...fr/free/nrw/commons/media/MediaDetailFragment.java	0% <0%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ddc83f1...7d20bb1. Read the comment docs.

[maskaravivek]

Found this in Wikipedia's code base:

private static String getDateStringWithSkeletonPattern(@NonNull Date date, @NonNull String pattern) {
        return new SimpleDateFormat(android.text.format.DateFormat.getBestDateTimePattern(Locale.getDefault(), pattern), Locale.getDefault()).format(date);
    }

calling it like:

getDateStringWithSkeletonPattern(date, "dd MMM yyyy")

[vanshikaarora]

Found this in Wikipedia's code base:

private static String getDateStringWithSkeletonPattern(@NonNull Date date, @NonNull String pattern) {
        return new SimpleDateFormat(android.text.format.DateFormat.getBestDateTimePattern(Locale.getDefault(), pattern), Locale.getDefault()).format(date);
    }

calling it like:

getDateStringWithSkeletonPattern(date, "dd MMM yyyy")

getBestDateTimePattern

@maskaravivek does this getBestDateTimePattern supplies approiate ormat for different languages?

[maskaravivek]

Looks like! You will have to play around a bit. :)

[vanshikaarora]

@maskaravivek I have added the function in DateUtil.
Kindly review and share your feedback :)

[nicolas-raoul]

Could you please post a screenshot in Japanese? An important difference of Japanese is that it places the year first.
Thanks! :-)

[vanshikaarora]

@nicolas-raoul Can you suggest me any other such language. Actually my phone doesn't supports japanese

[nicolas-raoul]

I suggest Chinese or Persian (Iran).
The best would be to try a language of each color zone:
https://en.wikipedia.org/wiki/Date_format_by_country#Usage_map

[vanshikaarora]

OK I'll check that out 👍

[vanshikaarora]

I suggest Chinese or Persian (Iran).
The best would be to try a language of each color zone:
https://en.wikipedia.org/wiki/Date_format_by_country#Usage_map

@nicolas-raoul I have tried for each color zone

For Chinese

For Hindi

For Philippines

For canada

For German

[nicolas-raoul]

The screenshots look perfect :-)

[vanshikaarora]

The screenshots look perfect :-)

Good to see this :)

[vanshikaarora]

Thanks for approving the Pull Request :)