date and time in the menu should be localized

[Xhacker Liu]

Binary package hint: indicator-datetime

In the datetime-service.c, there are

 /* Note: may require some localization tweaks */
 strftime(longstr, 128, "%A, %e %B %Y", ltime);

Yep it absolutely have to be localized.

[Sebastien Bacher (seb128)]

Thank you for taking the time to report this bug and helping to make Ubuntu better. Please answer these questions:

 * Is this reproducible?
 * If so, what specific steps should we take to recreate this bug?

 This will help us to find and resolve the problem.

[Sebastien Bacher (seb128)]

where does that show non translated in the ui?

[Xhacker Liu (xhacker)]

Click the clock in the panel (I mean indicator-datetime), the first item in the menu shows date and time, and it should be translatable.

And it is in datetime-service.c.

[Sebastien Bacher (seb128)]

it's displayed translated there...

[Xhacker Liu (xhacker)]

You know, translation and localization are two different things.

Do you know something about strftime?
It's translated because each part of strftime is translated.
However the format is wrong.

The format string is now "%A, %d %B %Y", and it's not translatable.
So, it displayed as a translated string with wrong format.

[Xhacker Liu (xhacker)]

Sorry, it is "%A, %e %B %Y" now and it is fixed.

[Xhacker Liu (xhacker)]

Err...
I mean the string is fixed(not translatable)...

[Gabor Kelemen (kelemeng)]

It is not:

https://translations.launchpad.net/ubuntu/natty/+source/indicator-datetime/+pots/indicator-datetime/hu/+translate?batch=10&show=all&search=%25Y

So, no %Y there, but the label clearly contains the year. And there is no other similar, already translated or untranslated occurrence of %Y in the code:

gabor@gabor-laptop:~/src/ubuntu/indicator-datetime$ grep -r %Y *
libmap/data/bg.png bináris fájl illeszkedik
libmap/data/cc.png bináris fájl illeszkedik
libmap/data/timezone_-4.png bináris fájl illeszkedik
src/datetime-service.c: strftime(longstr, 128, "%A, %e %B %Y", ltime);
src/datetime-prefs.c: format = "%Y-%m-%d";

Marking the mentioned string as translatable solved the problem.

[Sebastien Bacher (seb128)]

thanks for the details, next time it might be easier to just give some clear description of the issue to make triaging work easier

[Launchpad Janitor (janitor)]

This bug was fixed in the package indicator-datetime - 0.2.0-0ubuntu1

---------------
indicator-datetime (0.2.0-0ubuntu1) natty; urgency=low

  * New upstream release.
    ∘ Remove unused liboobs
    ∘ Add translator comments and make strings translatable (LP: #737326)
    ∘ Remove password dialog for calendars not the keyring
    ∘ Use default matching when geonames aren't available (LP: #740870)
    ∘ UI Cleanups and naming fixes (LP: #740806, LP: #740819)
    ∘ Free JSON parser data
    ∘ Avoid duplicates in GeoNames data (LP: #740884)
    ∘ Make location completion more robust
  * debian/control: Dropping liboobs
 -- Ted Gould <email address hidden> Wed, 23 Mar 2011 16:22:50 -0500