The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Approved => Merged
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28484
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mail
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Approved
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28484
Your team OpenLP Core is subscribed to branch lp:openlp.
__
Review: Approve approve
Don't forget aboutdialog.py
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28484
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-
Review: Approve
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28484
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscrib
Frode Woldsund has proposed merging lp:~frodus86/openlp/translate into
lp:openlp.
Requested reviews:
Raoul Snyman (raoul-snyman): approve
Jonathan Corwin (j-corwin)
Fixed the two last translate() functions in the trunk. Now all translate()
functions should be correct.
--
https://code.laun
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28299
Your team OpenLP Core is subscribed to branch lp:openlp.
I'm not shure what you talking about?
Some weeks before I tested it.
translate('AboutForm', 'Copyright \xa9 2004-2010 Raoul Snyman\n')
should work fine at all. Did I missed something in the meantime?
May you have to (re)generate the qm files before restarting.
I tested it here at WinXP a few minu
Would it be worth defining
copyright_symbol=u'\u00a9'.encode('utf8')
in openlp.py (or in some utility file)
Then we can do 'Copyright' + openlp.copyright_symbol + ... etc.
Makes the code a bit more legible?
Cheers,
Martin
--
martinthomp...@openlp.org
The day Microsoft makes something that
Review: Needs Fixing
Didn't work for me. My license tab on the About window shows:
Copyright © 2004-2010 Raoul Snyman
Portions copyright ©2004-2010...
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28299
Your team OpenLP Core is subscribed to branch lp:openlp.
Review: Approve approve
Good stuff.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28299
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launc
Frode Woldsund has proposed merging lp:~frodus86/openlp/translate into
lp:openlp.
Requested reviews:
Raoul Snyman (raoul-snyman): approve
Jonathan Corwin (j-corwin)
Fixed the two last translate() functions in the trunk. Now all translate()
functions should be correct.
--
https://code.laun
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Approved => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28273
Your team OpenLP Core is subscribed to branch lp:openlp.
___
A hack instead of a solution? No thanks.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28273
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.
Of course we could just use '(c)' and translate it to © :)
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28273
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to :
Ah, found it...
u'\u00a9'.encode('utf8')
Frode, can you just fix this please? Fix it in the two places in
openlp/core/ui/aboutdialog.py as well, please. See
http://bazaar.launchpad.net/~openlp-core/openlp/trunk/annotate/694/openlp/core/ui/aboutdialog.py#L157
--
https://code.launchpad.net/~fr
After my print test, I then tried the following, which seemed OK:
>>> from PyQt4 import QtGui
>>> print unicode(QtGui.QApplication.translate('abc', '\xa9'))
©
>>> print unicode(QtGui.QApplication.translate('abc', u'\xa9'))
©
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28273
Yo
This is what I get:
>>> print '\xa9'
�
>>> print u'\xa9'
©
I'm not too sure what we should do. I remember that back in the day we had a
solution to this, but I can't remember what it was. I'll have to go digging
through the revisions.
--
https://code.launchpad.net/~frodus86/openlp/translate/+m
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Approved
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28273
Your team OpenLP Core is subscribed to branch lp:openlp.
__
Review: Approve
Done a few more tests, and am happy it is OK via translate()
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28273
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-c
> ... Should have added that I'm not able to test it within the context of
> OpenLP and translate() though, so if someone confirms this is OK in the app,
> post a comment and I'll switch my Needs Fixing to an Approve.
I have problems with Song import in general, but at editsongdialog.py (line
484
... Should have added that I'm not able to test it within the context of OpenLP
and translate() though, so if someone confirms this is OK in the app, post a
comment and I'll switch my Needs Fixing to an Approve.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28273
Your team Open
Review: Needs Fixing
I get an (R) rather than a (C) at my Windows python prompt if the u is removed.
>>> print '\xa9'
®
>>> print u'\xa9'
©
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28273
Your team OpenLP Core is subscribed to branch lp:openlp.
Review: Approve approve
Awesome.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28273
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpa
Frode Woldsund has proposed merging lp:~frodus86/openlp/translate into
lp:openlp.
Requested reviews:
Raoul Snyman (raoul-snyman): needs fixing
Fixed the two last translate() functions in the trunk. Now all translate()
functions should be correct.
--
https://code.launchpad.net/~frodus86/open
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Rejected => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28272
Your team OpenLP Core is subscribed to branch lp:openlp.
___
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Rejected
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28272
Your team OpenLP Core is subscribed to branch lp:openlp.
__
Review: Needs Fixing needs fixing
Lines 19 and 22 of the diff still contain u's in front of the strings. These
need to be removed.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28272
Your team OpenLP Core is subscribed to branch lp:openlp.
__
Frode Woldsund has proposed merging lp:~frodus86/openlp/translate into
lp:openlp.
Requested reviews:
OpenLP Core (openlp-core)
Fixed the two last translate() functions in the trunk. Now all translate()
functions should be correct.
--
https://code.launchpad.net/~frodus86/openlp/translate/+me
Frode Woldsund has proposed merging lp:~frodus86/openlp/translate into
lp:openlp.
Requested reviews:
OpenLP Core (openlp-core)
Fixed two translate() functions in songplugin
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28267
Your team OpenLP Core is requested to review the
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Rejected
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28250
Your team OpenLP Core is subscribed to branch lp:openlp.
__
Review: Needs Fixing
Remove the u from the strings.
Please don't resubmit already approved and merged proposals. Makes keeping
track of what you're actually proposing harder.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28250
Your team OpenLP Core is subscribed to branch lp:o
Frode Woldsund has proposed merging lp:~frodus86/openlp/translate into
lp:openlp.
Requested reviews:
Raoul Snyman (raoul-snyman): approve
Tim Bentley (trb143)
Jon Tibble (meths)
Replaced the last trUtf8() -> translate()
Removed the unicode strings from the translate() functions, And fixe
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Merged => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28124
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Ma
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Merged
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28124
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Review: Approve approve
Good catch.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28124
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launc
Review: Approve
Approved
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/28124
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Un
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Description changed to:
Replaced the last trUtf8() -> translate()
Removed the unicode strings from the translate() functions, And fixed issues
regarding the [STRING CLEANUP] task on the mailinglist
--
https:/
Frode Woldsund has proposed merging lp:~frodus86/openlp/translate into
lp:openlp.
Requested reviews:
Raoul Snyman (raoul-snyman): approve
Jon Tibble (meths)
Tim Bentley (trb143)
Replaced the last trUtf8() -> translate()
Fixed the syntax on translate() functions that contained (u'%s') and
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Merged => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27825
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Ma
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Merged
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27825
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Review: Approve
Approved
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27825
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Un
Review: Approve
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27825
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscrib
Review: Approve approve
Great!
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27825
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.
Frode Woldsund has proposed merging lp:~frodus86/openlp/translate into
lp:openlp.
Requested reviews:
Jon Tibble (meths)
Tim Bentley (trb143)
Raoul Snyman (raoul-snyman): needs fixing
Replaced the last trUtf8() -> translate()
Fixed the syntax on translate() functions that contained (u'%s'
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Rejected => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27784
Your team OpenLP Core is subscribed to branch lp:openlp.
___
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Rejected
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27784
Your team OpenLP Core is subscribed to branch lp:openlp.
__
Review: Needs Fixing needs fixing
You don't need to write "BiblesPlugin.BiblePlugin" - you can just go with
"BiblePlugin". Please can you just correct all the plugins to work like that.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27784
Your team OpenLP Core is subscribed to br
Review: Approve
Approved
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27784
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Un
Review: Approve
Couple of whitespace nit picks but finishes the trUtf8 -> translate conversion.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/27784
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://laun
Frode Woldsund has proposed merging lp:~frodus86/openlp/translate into
lp:openlp.
Requested reviews:
OpenLP Core (openlp-core)
Replaced the last trUtf8() -> translate()
Fixed the syntax on translate() functions that contained (u'%s') and (u'...'
,u'..' %)
--
https://code.launchpad.net/~fr
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Merged
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26986
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Review: Approve
Approved
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26986
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
U
Review: Approve approve
Looks good!
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26986
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launc
Review: Approve
Approved - last few remaining coding standards issues can be fixed in tree
providing Raoul and Tim approve too.
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26986
Your team OpenLP Core is subscribed to branch lp:openlp.
_
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26985
Your team OpenLP Core is subscribed to branch lp:openlp.
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Description changed to:
Cleaned up the whole tree by replacing all trUtf8() with translate() function
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26985
Your team OpenLP Core is subscribed
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26891
Your team OpenLP Core is subscribed to branch lp:openlp.
Some small issues:
wrong translation context:
openlp/plugins/alerts/alertsplugin.py (forgot u)
openlp/plugins/bibles/lib/mediaitem.py:353 (forgot u)
openlp/plugins/bibles/lib/mediaitem.py:537 (forgot u)
openlp/plugins/songs/forms/editsongdialog.py:453
80 char linebreak:
openlp/plugins/bibles/forms/
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Commit Message changed to:
I'm running ubuntu, and the trunk seems to be broken, I can not run this
branch. And I got the same error when I make a new branch of the trunk 833 and
tryes to run it without any ed
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26889
Your team OpenLP Core is subscribed to branch lp:openlp.
The proposal to merge lp:~frodus86/openlp/translate into lp:openlp has been
updated.
Status: Needs review => Superseded
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26593
Your team OpenLP Core is subscribed to branch lp:openlp.
Review: Needs Fixing
You're missing spaces in various places:
translate('AmendThemeForm','First Color:')
should be:
translate('AmendThemeForm', 'First Color:')
^
space
--
https://code.launchpad.net/~frodus86/openlp/translate/+merge/26593
Your
62 matches
Mail list logo