Permalink
Browse files

Fixed MiroBridge abort when Miro metadata has no video filename

Sometimes Miro metadata has no video filename. Skip and remove these
invalid Miro items.
  • Loading branch information...
1 parent afbbb6e commit 57d0e8b5ff02691a8765ab25c5a8aacc004b67d5 @rdv rdv committed with wagnerrp Oct 2, 2011
Showing with 16 additions and 0 deletions.
  1. +16 −0 mythtv/contrib/imports/mirobridge/mirobridge/mirobridge_interpreter_4_0_2.py
@@ -278,6 +278,14 @@ def do_mythtv_getunwatched(self, line):
if hasattr(it.get_parent(), u'url'):
if filetypes.is_torrent_filename(it.get_parent().url):
continue
+
+ # Any item without a proper file name needs to be removed as Miro metadata is corrupt
+ if it.get_filename() == None:
+ it.expire()
+ self.statistics[u'Miro_videos_deleted']+=1
+ logging.info(u'Unwatched video (%s) has been removed from Miro as item had no valid file name' % it.get_title())
+ continue
+
self.printItems(it)
self.videofiles.append(self._get_item_dict(it))
if self.verbose:
@@ -304,6 +312,14 @@ def do_mythtv_getwatched(self, line):
if hasattr(it.get_parent(), u'url'):
if filetypes.is_torrent_filename(it.get_parent().url):
continue
+
+ # Any item without a proper file name needs to be removed as Miro metadata is corrupt
+ if it.get_filename() == None:
+ it.expire()
+ self.statistics[u'Miro_videos_deleted']+=1
+ logging.info(u'Watched video (%s) has been removed from Miro as item had no valid file name' % it.get_title())
+ continue
+
self.printItems(it)
self.videofiles.append(self._get_item_dict(it))
if self.verbose:

0 comments on commit 57d0e8b

Please sign in to comment.