MediaInfo's XML output has changed
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenLP |
Fix Committed
|
High
|
Phill |
Bug Description
As of version 17.10, MediaInfo's XML output has changed, causing tests to fail (and no doubt OpenLP as well). Unfortunately while the changelog says that you can still use the old XML, the command line utility itself does not seem to support that option (I can't figure out how to give it the option).
Changelog: https:/
Test output:
=======
ERROR: test_media_length (tests.
-------
Traceback (most recent call last):
File "OpenLP/
self.
File "OpenLP/
self.
File "OpenLP/
for xml_track in self.xml_
AttributeError: 'NoneType' object has no attribute 'File'
-------
description: | updated |
Changed in openlp: | |
milestone: | none → 2.5.1 |
assignee: | Tim Bentley (trb143) → nobody |
status: | New → Fix Committed |
Changed in openlp: | |
assignee: | nobody → Phill (phill-ridout) |
I haven't looked at the code but the change log states 'Use Option("Inform", "OLDXML") for keeping previous behaviour'
Perhaps we should use this until we can be sure all systems have received the update, then in the future switch to the json interface (introduced in 7.99)