openlp-core team mailing list archive

[Phill <1732348@xxxxxxxxxxxxxxxxxx>]

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)

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

Title:
  MediaInfo's XML output has changed

Status in OpenLP:
  New

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://mediaarea.net/MediaInfo/ChangeLog

  Test output:

  ======================================================================
  ERROR: test_media_length (tests.interfaces.openlp_core.ui.media.vendor.test_mediainfoWrapper.TestMediainfoWrapper)
  ----------------------------------------------------------------------
  Traceback (most recent call last):
    File "OpenLP/tests/interfaces/openlp_core/ui/media/vendor/test_mediainfoWrapper.py", line 48, in test_media_length
      self.assertEqual(results.tracks[0].duration, test_data[1], 'The correct duration is returned for ' +
    File "OpenLP/openlp/core/ui/media/vendor/mediainfoWrapper.py", line 130, in tracks
      self._populate_tracks()
    File "OpenLP/openlp/core/ui/media/vendor/mediainfoWrapper.py", line 122, in _populate_tracks
      for xml_track in self.xml_dom.Mediainfo.File.find_all("track"):
  AttributeError: 'NoneType' object has no attribute 'File'

  ----------------------------------------------------------------------

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