openlp-core team mailing list archive

[Tim Bentley <1732348@xxxxxxxxxxxxxxxxxx>]

** Changed in: openlp
     Assignee: (unassigned) => Phill (phill-ridout)

-- 
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:
  Fix Committed

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