Skip to content
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.
(cherry picked from commit 57d0e8b)
  • Loading branch information...
1 parent c8c1582 commit 5e3c81f9733c3471f952151da071e0fe9c2ab3ca @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
View
16 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 5e3c81f

Please sign in to comment.
Something went wrong with that request. Please try again.