[Differential] [Request, 35 lines] D4023: kformat: Make it possible to properly translate relative days of the week

mpyne (Michael Pyne) noreply at phabricator.kde.org
Sun Jan 8 17:09:30 UTC 2017


mpyne created this revision.
mpyne added reviewers: Frameworks, kde-i18n-doc, aspotashev.
mpyne set the repository for this revision to R244 KCoreAddons.
Restricted Application added a project: Frameworks.

REVISION SUMMARY
  KFormat::formatRelativeDate <https://api.kde.org/frameworks/kcoreaddons/html/classKFormat.html#a6adf12070942eb8e6c9efd862edb8b65> allows one to obtain a human-readable date like "Last Wednesday" or "Next Thursday" for dates within a week of the current date.
  
  The returned string cannot be properly translated in all languages since it's generated using a placeholder of the form "Next %1" -- the word Next (or Last) may itself need to be modified to be correct which can't be done with the current translation system.
  
  The fix is just to return "Next $dayName" directly, based on the day name as appropriate, and allow the translators to translate the entire phrase.
  
  The auto tests are adjusted as well.  They were actually incorrect before this change, since the API documentation for `formatRelativeDate` makes clear that the date format is only applied if the date is *not* within a week of the current date.
  
  See also Bug 335106 <https://bugs.kde.org/show_bug.cgi?id=335106>.

TEST PLAN
  Change builds and installs properly, and all kcoreaddons tests continue to pass (with the `kformattest` autotest revised per this change).

REPOSITORY
  R244 KCoreAddons

REVISION DETAIL
  https://phabricator.kde.org/D4023

AFFECTED FILES
  autotests/kformattest.cpp
  src/lib/util/kformatprivate.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mpyne, #frameworks, kde-i18n-doc, aspotashev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170108/df45a529/attachment.html>


More information about the Kde-frameworks-devel mailing list