OpenLP-2.2.1 problem with import from OpenSong

Hello, i wanted to import some opensong files into OpenLP-2.2.1 but i got an error. I dont have the stacktrace, but it ended with some "int(3012.09) parsing error"
Song file contains:
  <ccli>3012.09</ccli>

In 2.0.5 import works fine.

Comments

  • Hi,
    While OpenLP probably shouldn't crash because of this, it does have a point - "3012.09" doesn't look like a valid CCLI number to me :)
  • edited November 2015
    Problem is the same on WinXP and Win7.
    Traceback (most recent call last):
    File "D:\OpenLP_Development\OpenLP_Code\2.2\openlp\core\ui\wizard.py", line 216, in on_current_id_changed
    File "D:\OpenLP_Development\OpenLP_Code\2.2\openlp\plugins\songs\forms\songimportform.py", line 348, in perform_wizard
    File "D:\OpenLP_Development\OpenLP_Code\2.2\openlp\plugins\songs\lib\importers\opensong.py", line 123, in do_import
    File "D:\OpenLP_Development\OpenLP_Code\2.2\openlp\plugins\songs\lib\importers\opensong.py", line 160, in do_import_file
    ValueError: invalid literal for int() with base 10: '3012.09'
    I need to say i dont have any "D:\OpenLP_Development\OpenLP_Code\2.2\openlp\" in my system.
    Opensong file im trying to import:
    <?xml version="1.0" encoding="UTF-8"?>
    <song>
    <title>1 Korinským 13</title>
    <author>text: Naďa Kinkorová; hudba: Daniel Baláž</author>
    <copyright></copyright>
    <presentation></presentation>
    <ccli>3012.09</ccli>
    <capo print="false"></capo>
    <key></key>
    <aka></aka>
    <key_line></key_line>
    <user1></user1>
    <user2></user2>
    <user3></user3>
    <theme></theme>
    <tempo></tempo>
    <time_sig></time_sig>
    <lyrics>[V1]
    Kdybych se naučil mluvit jak andělé...

    [C]
    Navždy tu zůstává
    víra i naděje,
    ale největším darem Tvým (největším)
    může být jedině láska.

    [V2]
    Nad hněvem vaším ať slunce nezapadá,...
    </lyrics>
    <hymn_number></hymn_number></song>
  • ako pisal @tgs: mas zle ccli cislo, skus vymazat tu bodku
    (change ccli number, you can try to delete dot)

  • Sure, the ccli number is wrong, no argue. In Openlp 2.0.5 import works fine. In new version it does not work.

Sign In or Register to comment.