[Kde-pim] Review Request 113252: Extract messages from javascript wizards

Luigi Toscano luigi.toscano at tiscali.it
Wed Nov 13 23:38:42 GMT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113252/
-----------------------------------------------------------

(Updated Nov. 13, 2013, 11:38 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDEPIM.


Repository: kdepim-runtime


Description
-------

Translate the javascript files used in the wizards that were not translated before.
I was going to commit this directly (as it does not impact the code, and it fixes a long-standing issue), but this change depends on a kdelibs patch which has been committed today (https://git.reviewboard.kde.org/r/113218/), which means kdelibs 4.11.3. The code in this review adds some calls to qsTr(), and when that kdelibs patch is not applied the wizard scripts exit with a run time error (and a blank window). So the code in this review adds an implicit dependency on kdelibs 4.11.3; the cmake code says "kdelibs 4.10, kdepimlibs 4.11". Would it be acceptable?


Diffs
-----

  accountwizard/wizards/tine20/Messages.sh 30a1fec 
  accountwizard/wizards/tine20/tine20wizard.es 6970b1d 
  resources/imap/wizard/Messages.sh 8fbc16c 
  resources/imap/wizard/imapwizard.es dcab06f 
  resources/kolabproxy/wizard/Messages.sh 5654648 
  resources/kolabproxy/wizard/kolabwizard.es 842e757 
  resources/maildir/wizard/Messages.sh 69bb8e9 
  resources/maildir/wizard/maildirwizard.es af61f8a 
  resources/mbox/wizard/Messages.sh d720fde 
  resources/mbox/wizard/mailboxwizard.es 286d4c0 
  resources/pop3/wizard/Messages.sh fee8e77 
  resources/pop3/wizard/pop3wizard.js 47262e4 

Diff: http://git.reviewboard.kde.org/r/113252/diff/


Testing
-------


Thanks,

Luigi Toscano

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list