Boddlnagg/WordsLive

OpenLyrics issues

Opened this issue · 2 comments

Some minor issues I noted when testing OpenLyrics import/export:

  • createdIn field is not preserved
  • Themes are put all in one field (Assurance; Trust instead of multiple items)
  • The copyright and author fields get mixed together
  • It's not possible to import multiple songs at once
  • OpenLyrics v0.7 verses have a different syntax and don't get imported. Maybe you can add a check for the OpenLyrics version when importing?

Anyway, thanks for implementing this, I've added you to https://code.google.com/p/openlyrics/wiki/ProjectsUsingOpenLyrics

Thanks for testing this! Most of those issues result from differing data representations when comparing OpenLyrics and WordsLive's "native" PowerPraise format (ppl). It's important to note that my goal was not to allow for an "import-export-cycle" that creates exactly the same file as it was given. I rather tried to provide separate import and export capabilities that work as good as possible on their own.

Your first three points are direct results from this and I hope you'll understand that I won't change much here.

The ability to import multiple songs at once is already on my TODO list (probably I should create some issues here on GitHub so everybody can see what is on that list 😉). It applies to all import formats however, not just to OpenLyrics.

As v0.7 is concerned, I can add support for that, too. It looks like the main difference was the <line> tag, which was present in 0.7 and removed in 0.8 in favor of <br/>. All other differences don't seem to affect the import.

And thanks again for adding me to the list of projects! 😃 Would you mind to add "Chords" to the advanced features (because chords should actually be supported in both import and export)?

Ok, the issues I noted are really minor it's ok to leave them as they are.

I've updated the wiki page with the Chords info.

Of course you are welcome to add support for 0.7. But the only application that actually produces 0.7 files is ExpoSong (which is no longer in development).
So just a warning message would be enough for these files.