Improper characters in Mediashout import cause traceback

Bug #1618489 reported by Azaziah on 2016-08-30
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenLP
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) on 2016-09-18
description: updated
Tomas Groth (tomasgroth) on 2016-09-25
Changed in openlp:
assignee: nobody → Tomas Groth (tomasgroth)
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers