<div dir="ltr">Hi Simon,<br><div><div class="gmail_extra"><br></div><div class="gmail_extra">Thank you for diving into this!<br></div><div class="gmail_extra"><br></div><div class="gmail_extra">I left a few comments below.<br><br></div><div class="gmail_extra"><div class="gmail_quote">2017-03-18 14:38 GMT+01:00 Simon Hanna <span dir="ltr"><<a href="mailto:simon.hanna@serve-me.info" target="_blank">simon.hanna@serve-me.info</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi there,<br>
<br>
Since my church will only switch to OpenLP if it supports presenting two<br>
languages, I decided to work on it.<br>
<br>
I'm happy to announce that I finished the heavy lifting on the song<br>
editor. So I would appreciate your feedback on it. The code lives here<br>
<a href="https://code.launchpad.net/~thelinuxguy/openlp/multi-language" rel="noreferrer" target="_blank">https://code.launchpad.net/~<wbr>thelinuxguy/openlp/multi-<wbr>language</a><br>
<br>
No database migrations need to be done, and all the changes to specify<br>
the languages are reset once an older version of openlp(which silently<br>
ignores what language a verse is in) overwrites the language<br>
information, so yet get a "clean" database.<br>
<br>
Since I'm not doing any database migrations the "primary language" is<br>
just the first language encountered and changing it doesn't actually get<br>
saved. (This will change in the future)<br>
<br>
There's still a few things concerning the editor that need to be addressed:<br>
<br>
- Not everything is translatable yet and some of the strings I used are<br>
subject to change<br>
<br>
- How to handle verse order (This currently only picks up the "primary<br>
language").<br>
<br></blockquote><div> </div><div>I would suggest just to have 1 verse order list. Otherwise it will be very difficult to handle presentation sync'ed between multiple languages.<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
- What language information should be stored in the database.<br>
<br>
    - The primary language?<br></blockquote><div> </div><div>Yes.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
    - All languages so you can search by language?<br></blockquote><div> </div><div>I would store everything as searchable, but maybe make it depend on a setting.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
    - How about Titles for each language? (Other information as well?)<br></blockquote><div> </div><div>I would votes for multiple titles, one per languages.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
- How to handle invalid (or unknown) language codes that are encountered<br>
<br>
<br>
Since the points I mentioned above need discussion, and I don't have a<br>
very strong opinion on them, I'm leaving them for now.<br>
<br>
My next milestone would be the ability to choose one language that is<br>
going to be presented and after that add support for two languages, ...<br><br></blockquote><div><br></div><div> IMHO the multi language editor-buttons should only appear after being enabled per song.<br><br></div><div>Keep up the good work :)<br><br></div><div>/Tomas<br></div></div></div></div></div>