Improper characters in Mediashout import cause traceback

Bug #1618489 reported by Azaziah
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenLP
Fix Committed
Undecided
Tomas Groth

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

Related branches

Tomas Groth (tomasgroth)
description: updated
Tomas Groth (tomasgroth)
Changed in openlp:
assignee: nobody → Tomas Groth (tomasgroth)
status: New → Fix Committed
Tim Bentley (trb143)
Changed in openlp:
milestone: none → 2.9.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.