openlp-core team mailing list archive

[Tim Bentley <1618489@xxxxxxxxxxxxxxxxxx>]

** Changed in: openlp
    Milestone: None => 2.9.1

-- 
You received this bug notification because you are a member of OpenLP
Core, which is subscribed to OpenLP.
https://bugs.launchpad.net/bugs/1618489

Title:
  Improper characters in Mediashout import cause traceback

Status in OpenLP:
  Fix Committed

Bug description:
  If Mediashout import has improper characters, the importer causes a
  traceback:

  --- Exception Traceback --- Traceback (most recent call last): 
  File "openlp\core\ui\wizard.py", line 216, in on_current_id_changed 
  File "openlp\plugins\songs\forms\songimportform.py", line 351, in perform_wizard 
  File "openlp\plugins\songs\lib\importers\mediashout.py", line 77, in do_import 
  File "openlp\plugins\songs\lib\importers\mediashout.py", line 100, in process_song 
  File "openlp\plugins\songs\lib\importers\songimport.py", line 353, in finish 
  File "openlp\plugins\songs\lib\openlyricsxml.py", line 102, in add_verse_to_lyrics 
  File "lxml.etree.pyx", line 2972, in lxml.etree.CDATA.__cinit__ (src\lxml\lxml.etree.c:69195) 
  File "apihelpers.pxi", line 1393, in lxml.etree._utf8 (src\lxml\lxml.etree.c:27125) 
  ValueError: All strings must be XML compatible: Unicode or ASCII, no NULL bytes or control characters

  Such a file can be found in the db provided in here:
  https://support.openlp.org/scp/tickets.php?id=3690

  Such characters can be found in the database on:
  Verses > record > 2856.

  Here’s a screen capture of the characters,
  I’m unable to copy them.
  https://drive.google.com/file/d/0B9y8rZiYItltcHBpUUt0OWR4aGc/view?usp=sharing

  Copy-paste sample:
  “That covers me and
  Raised this dead man's life
  It's all because of
  Jesus I'm alive”

  Win 10 | 2.4.2

To manage notifications about this bug go to:
https://bugs.launchpad.net/openlp/+bug/1618489/+subscriptions