openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #23780
Re: [Merge] lp:~tomasgroth/openlp/dvd into lp:openlp
Review: Needs Fixing
Comments in line.
Have you regenerated the resources!
Have you tests a service build on linux and run on windows and the other way round.
Looks good for a start but I need to test.
Diff comments:
> === modified file 'openlp/core/lib/serviceitem.py'
> --- openlp/core/lib/serviceitem.py 2014-04-14 18:28:04 +0000
> +++ openlp/core/lib/serviceitem.py 2014-06-16 07:07:43 +0000
> @@ -111,6 +111,9 @@
> ``CanEditTitle``
> The capability to edit the title of the item
>
> + ``IsOptical``
> + .Determines is the service_item is based on an optical device
> +
> """
> CanPreview = 1
> CanEdit = 2
> @@ -129,6 +132,7 @@
> HasBackgroundAudio = 15
> CanAutoStartForLive = 16
> CanEditTitle = 17
> + IsOptical = 18
>
>
> class ServiceItem(RegistryProperties):
> @@ -416,7 +420,10 @@
> for text_image in service_item['serviceitem']['data']:
> if not self.title:
> self.title = text_image['title']
> - if path:
> + if self.is_capable(ItemCapabilities.IsOptical):
> + self.has_original_files = False
> + self.add_from_command(text_image['path'], text_image['title'], text_image['image'])
> + elif path:
> self.has_original_files = False
> self.add_from_command(path, text_image['title'], text_image['image'])
> else:
> @@ -427,7 +434,8 @@
> """
> Returns the title of the service item.
> """
> - if self.is_text() or ItemCapabilities.CanEditTitle in self.capabilities:
> + if self.is_text() or self.is_capable(ItemCapabilities.IsOptical) \
> + or self.is_capable(ItemCapabilities.CanEditTitle):
> return self.title
> else:
> if len(self._raw_frames) > 1:
> @@ -495,7 +503,8 @@
> """
> Confirms if the ServiceItem uses a file
> """
> - return self.service_item_type == ServiceItemType.Image or self.service_item_type == ServiceItemType.Command
> + return self.service_item_type == ServiceItemType.Image or \
> + (self.service_item_type == ServiceItemType.Command and not self.is_capable(ItemCapabilities.IsOptical))
>
> def is_text(self):
> """
> @@ -553,7 +562,7 @@
> frame = self._raw_frames[row]
> except IndexError:
> return ''
> - if self.is_image():
> + if self.is_image() or self.is_capable(ItemCapabilities.IsOptical):
> path_from = frame['path']
> else:
> path_from = os.path.join(frame['path'], frame['title'])
> @@ -623,12 +632,17 @@
> self.is_valid = False
> break
> elif self.is_command():
> - file_name = os.path.join(frame['path'], frame['title'])
> - if not os.path.exists(file_name):
> - self.is_valid = False
> - break
> - if suffix_list and not self.is_text():
> - file_suffix = frame['title'].split('.')[-1]
> - if file_suffix.lower() not in suffix_list:
> - self.is_valid = False
> - break
> + if self.is_capable(ItemCapabilities.IsOptical):
> + if not os.path.exists(frame['title']):
> + self.is_valid = False
> + break
> + else:
> + file_name = os.path.join(frame['path'], frame['title'])
> + if not os.path.exists(file_name):
> + self.is_valid = False
> + break
> + if suffix_list and not self.is_text():
> + file_suffix = frame['title'].split('.')[-1]
> + if file_suffix.lower() not in suffix_list:
> + self.is_valid = False
> + break
> \ No newline at end of file
Pep8 failure
>
> === modified file 'openlp/core/ui/media/__init__.py'
> --- openlp/core/ui/media/__init__.py 2014-04-20 22:23:26 +0000
> +++ openlp/core/ui/media/__init__.py 2014-06-16 07:07:43 +0000
> @@ -72,6 +72,9 @@
> length = 0
> start_time = 0
> end_time = 0
> + title_track = 0
> + audio_track = 0
> + subtitle_track = 0
> media_type = MediaType()
>
>
> @@ -107,6 +110,39 @@
> players = players.replace(overridden_player, '[%s]' % overridden_player)
> Settings().setValue('media/players', players)
>
> +
> +def parse_optical_path(input):
> + """
> + Split the optical path info.
> +
> + :param input: The string to parse
> + :return: The elements extracted from the string: filename, title, audio_track, subtitle_track, start, end
> + """
> + clip_info = input.split(sep=':')
> + title = int(clip_info[1])
> + audio_track = int(clip_info[2])
> + subtitle_track = int(clip_info[3])
> + start = float(clip_info[4])
> + end = float(clip_info[5])
> + clip_name = clip_info[6]
> + filename = clip_info[7]
> + # Windows path usually contains a colon after the drive letter
> + if len(clip_info) > 8:
> + filename += clip_info[8]
> + return filename, title, audio_track, subtitle_track, start, end, clip_name
Log debug may be useful incase we have issues and want more info.
Based on word "usually"
> +
> +
> +def format_milliseconds(milliseconds):
> + """
> + Format milliseconds into a human readable time string.
> + :param milliseconds: Milliseconds to format
> + :return: Time string in format: hh.mm.ss,ttt
> + """
> + seconds, millis = divmod(milliseconds, 1000)
> + minutes, seconds = divmod(seconds, 60)
> + hours, minutes = divmod(minutes, 60)
> + return "%02d:%02d:%02d,%03d" % (hours, minutes, seconds, millis)
> +
> from .mediacontroller import MediaController
> from .playertab import PlayerTab
>
>
> === modified file 'openlp/core/ui/media/mediacontroller.py'
> --- openlp/core/ui/media/mediacontroller.py 2014-04-14 19:44:35 +0000
> +++ openlp/core/ui/media/mediacontroller.py 2014-06-16 07:07:43 +0000
> @@ -36,9 +36,10 @@
> from PyQt4 import QtCore, QtGui
>
> from openlp.core.common import OpenLPMixin, Registry, RegistryMixin, RegistryProperties, Settings, UiStrings, translate
> -from openlp.core.lib import OpenLPToolbar
> +from openlp.core.lib import OpenLPToolbar, ItemCapabilities
> from openlp.core.lib.ui import critical_error_message_box
> -from openlp.core.ui.media import MediaState, MediaInfo, MediaType, get_media_players, set_media_players
> +from openlp.core.ui.media import MediaState, MediaInfo, MediaType, get_media_players, set_media_players,\
> + parse_optical_path
> from openlp.core.ui.media.mediaplayer import MediaPlayer
> from openlp.core.common import AppLocation
> from openlp.core.ui import DisplayControllerType
> @@ -368,7 +369,16 @@
> controller.media_info.file_info = QtCore.QFileInfo(service_item.get_frame_path())
> display = self._define_display(controller)
> if controller.is_live:
> - is_valid = self._check_file_type(controller, display, service_item)
> + # if this is an optical device use special handling
> + if service_item.is_capable(ItemCapabilities.IsOptical):
> + log.debug('video is optical and live')
> + path = service_item.get_frame_path()
> + (name, title, audio_track, subtitle_track, start, end, clip_name) = parse_optical_path(path)
> + is_valid = self.media_setup_optical(name, title, audio_track, subtitle_track, start, end, display,
> + controller)
> + else:
> + log.debug('video is not optical and live')
> + is_valid = self._check_file_type(controller, display, service_item)
> display.override['theme'] = ''
> display.override['video'] = True
> if controller.media_info.is_background:
> @@ -379,12 +389,21 @@
> controller.media_info.start_time = service_item.start_time
> controller.media_info.end_time = service_item.end_time
> elif controller.preview_display:
> - is_valid = self._check_file_type(controller, display, service_item)
> + if service_item.is_capable(ItemCapabilities.IsOptical):
> + log.debug('video is optical and preview')
> + path = service_item.get_frame_path()
> + (name, title, audio_track, subtitle_track, start, end, clip_name) = parse_optical_path(path)
> + is_valid = self.media_setup_optical(name, title, audio_track, subtitle_track, start, end, display,
> + controller)
> + else:
> + log.debug('video is not optical and preview')
> + is_valid = self._check_file_type(controller, display, service_item)
> if not is_valid:
> # Media could not be loaded correctly
> critical_error_message_box(translate('MediaPlugin.MediaItem', 'Unsupported File'),
> translate('MediaPlugin.MediaItem', 'Unsupported File'))
> return False
> + log.debug('video mediatype: ' + str(controller.media_info.media_type))
> # dont care about actual theme, set a black background
> if controller.is_live and not controller.media_info.is_background:
> display.frame.evaluateJavaScript('show_video( "setBackBoard", null, null, null,"visible");')
> @@ -436,6 +455,62 @@
> log.debug('use %s controller' % self.current_media_players[controller.controller_type])
> return True
>
> + def media_setup_optical(self, filename, title, audio_track, subtitle_track, start, end, display, controller):
> + """
> + Setup playback of optical media
> +
> + :param filename: Path of the optical device/drive.
> + :param title: The main/title track to play.
> + :param audio_track: The audio track to play.
> + :param subtitle_track: The subtitle track to play.
> + :param start: Start position in miliseconds.
> + :param end: End position in miliseconds.
> + :param display: The display to play the media.
> + :param controller: The media contraoller.
> + :return: True if setup succeded else False.
> + """
> + log.debug('media_setup_optical')
> + if controller is None:
> + controller = self.display_controllers[DisplayControllerType.Plugin]
> + # stop running videos
> + self.media_reset(controller)
> + # Setup media info
> + controller.media_info = MediaInfo()
> + controller.media_info.file_info = QtCore.QFileInfo(filename)
> + if audio_track == -1 and subtitle_track == -1:
> + controller.media_info.media_type = MediaType.CD
> + else:
> + controller.media_info.media_type = MediaType.DVD
> + controller.media_info.start_time = start/1000
> + controller.media_info.end_time = end/1000
> + controller.media_info.length = (end - start)/1000
> + controller.media_info.title_track = title
> + controller.media_info.audio_track = audio_track
> + controller.media_info.subtitle_track = subtitle_track
> + # When called from mediaitem display is None
> + if display is None:
> + display = controller.preview_display
> + # Find vlc player
> + used_players = get_media_players()[0]
> + vlc_player = None
> + for title in used_players:
> + player = self.media_players[title]
> + if player.name == 'vlc':
> + vlc_player = player
> + if vlc_player is None:
> + critical_error_message_box(translate('MediaPlugin.MediaItem', 'VLC player required'),
> + translate('MediaPlugin.MediaItem',
> + 'VLC player required for playback of optical devices'))
> + return False
> + vlc_player.load(display)
> + self.resize(display, vlc_player)
> + self.current_media_players[controller.controller_type] = vlc_player
> + if audio_track == -1 and subtitle_track == -1:
> + controller.media_info.media_type = MediaType.CD
> + else:
> + controller.media_info.media_type = MediaType.DVD
> + return True
> +
> def _check_file_type(self, controller, display, service_item):
> """
> Select the correct media Player type from the prioritized Player list
>
> === modified file 'openlp/core/ui/media/vlcplayer.py'
> --- openlp/core/ui/media/vlcplayer.py 2014-05-07 22:52:06 +0000
> +++ openlp/core/ui/media/vlcplayer.py 2014-06-16 07:07:43 +0000
> @@ -40,7 +40,7 @@
>
> from openlp.core.common import Settings
> from openlp.core.lib import translate
> -from openlp.core.ui.media import MediaState
> +from openlp.core.ui.media import MediaState, MediaType
> from openlp.core.ui.media.mediaplayer import MediaPlayer
>
> log = logging.getLogger(__name__)
> @@ -166,7 +166,19 @@
> file_path = str(controller.media_info.file_info.absoluteFilePath())
> path = os.path.normcase(file_path)
> # create the media
> - display.vlc_media = display.vlc_instance.media_new_path(path)
> + if controller.media_info.media_type == MediaType.CD:
> + display.vlc_media = display.vlc_instance.media_new_location('cdda://' + path)
> + display.vlc_media_player.set_media(display.vlc_media)
> + display.vlc_media_player.play()
> + # Wait for media to start playing. In this case VLC actually returns an error.
> + self.media_state_wait(display, vlc.State.Playing)
> + # If subitems exists, this is a CD
> + audio_cd_tracks = display.vlc_media.subitems()
> + if not audio_cd_tracks or audio_cd_tracks.count() < 1:
> + return False
> + display.vlc_media = audio_cd_tracks.item_at_index(controller.media_info.title_track)
> + else:
> + display.vlc_media = display.vlc_instance.media_new_path(path)
> # put the media in the media player
> display.vlc_media_player.set_media(display.vlc_media)
> # parse the metadata of the file
> @@ -206,14 +218,37 @@
> """
> controller = display.controller
> start_time = 0
> + log.debug('vlc play')
> if self.state != MediaState.Paused and controller.media_info.start_time > 0:
> start_time = controller.media_info.start_time
> threading.Thread(target=display.vlc_media_player.play).start()
> if not self.media_state_wait(display, vlc.State.Playing):
> return False
> + if self.state != MediaState.Paused and controller.media_info.start_time > 0:
> + log.debug('vlc play, starttime set')
> + start_time = controller.media_info.start_time
> + log.debug('mediatype: ' + str(controller.media_info.media_type))
> + # Set tracks for the optical device
> + if controller.media_info.media_type == MediaType.DVD:
> + log.debug('vlc play, playing started')
> + if controller.media_info.title_track > 0:
> + log.debug('vlc play, title_track set: ' + str(controller.media_info.title_track))
> + display.vlc_media_player.set_title(controller.media_info.title_track)
> + display.vlc_media_player.play()
> + if not self.media_state_wait(display, vlc.State.Playing):
> + return False
> + if controller.media_info.audio_track > 0:
> + display.vlc_media_player.audio_set_track(controller.media_info.audio_track)
> + log.debug('vlc play, audio_track set: ' + str(controller.media_info.audio_track))
> + if controller.media_info.subtitle_track > 0:
> + display.vlc_media_player.video_set_spu(controller.media_info.subtitle_track)
> + log.debug('vlc play, subtitle_track set: ' + str(controller.media_info.subtitle_track))
> + if controller.media_info.start_time > 0:
> + log.debug('vlc play, starttime set: ' + str(controller.media_info.start_time))
> + start_time = controller.media_info.start_time
> self.volume(display, controller.media_info.volume)
> if start_time > 0:
> - self.seek(display, controller.media_info.start_time * 1000)
> + self.seek(display, int(start_time * 1000))
> controller.media_info.length = int(display.vlc_media_player.get_media().get_duration() / 1000)
> controller.seek_slider.setMaximum(controller.media_info.length * 1000)
> self.state = MediaState.Playing
>
> === added directory 'openlp/plugins/media/forms'
> === added file 'openlp/plugins/media/forms/__init__.py'
> --- openlp/plugins/media/forms/__init__.py 1970-01-01 00:00:00 +0000
> +++ openlp/plugins/media/forms/__init__.py 2014-06-16 07:07:43 +0000
> @@ -0,0 +1,28 @@
> +# -*- coding: utf-8 -*-
> +# vim: autoindent shiftwidth=4 expandtab textwidth=120 tabstop=4 softtabstop=4
> +
> +###############################################################################
> +# OpenLP - Open Source Lyrics Projection #
> +# --------------------------------------------------------------------------- #
> +# Copyright (c) 2008-2014 Raoul Snyman #
> +# Portions copyright (c) 2008-2014 Tim Bentley, Gerald Britton, Jonathan #
> +# Corwin, Samuel Findlay, Michael Gorven, Scott Guerrieri, Matthias Hub, #
> +# Meinert Jordan, Armin Köhler, Erik Lundin, Edwin Lunando, Brian T. Meyer. #
> +# Joshua Miller, Stevan Pettit, Andreas Preikschat, Mattias Põldaru, #
> +# Christian Richter, Philip Ridout, Simon Scudder, Jeffrey Smith, #
> +# Maikel Stuivenberg, Martin Thompson, Jon Tibble, Dave Warnock, #
> +# Frode Woldsund, Martin Zibricky, Patrick Zimmermann #
> +# --------------------------------------------------------------------------- #
> +# This program is free software; you can redistribute it and/or modify it #
> +# under the terms of the GNU General Public License as published by the Free #
> +# Software Foundation; version 2 of the License. #
> +# #
> +# This program is distributed in the hope that it will be useful, but WITHOUT #
> +# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or #
> +# FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for #
> +# more details. #
> +# #
> +# You should have received a copy of the GNU General Public License along #
> +# with this program; if not, write to the Free Software Foundation, Inc., 59 #
> +# Temple Place, Suite 330, Boston, MA 02111-1307 USA #
> +###############################################################################
>
> === added file 'openlp/plugins/media/forms/mediaclipselectordialog.py'
> --- openlp/plugins/media/forms/mediaclipselectordialog.py 1970-01-01 00:00:00 +0000
> +++ openlp/plugins/media/forms/mediaclipselectordialog.py 2014-06-16 07:07:43 +0000
> @@ -0,0 +1,206 @@
> +# -*- coding: utf-8 -*-
> +# vim: autoindent shiftwidth=4 expandtab textwidth=120 tabstop=4 softtabstop=4
> +
> +###############################################################################
> +# OpenLP - Open Source Lyrics Projection #
> +# --------------------------------------------------------------------------- #
> +# Copyright (c) 2008-2014 Raoul Snyman #
> +# Portions copyright (c) 2008-2014 Tim Bentley, Gerald Britton, Jonathan #
> +# Corwin, Samuel Findlay, Michael Gorven, Scott Guerrieri, Matthias Hub, #
> +# Meinert Jordan, Armin Köhler, Erik Lundin, Edwin Lunando, Brian T. Meyer. #
> +# Joshua Miller, Stevan Pettit, Andreas Preikschat, Mattias Põldaru, #
> +# Christian Richter, Philip Ridout, Simon Scudder, Jeffrey Smith, #
> +# Maikel Stuivenberg, Martin Thompson, Jon Tibble, Dave Warnock, #
> +# Frode Woldsund, Martin Zibricky, Patrick Zimmermann #
> +# --------------------------------------------------------------------------- #
> +# This program is free software; you can redistribute it and/or modify it #
> +# under the terms of the GNU General Public License as published by the Free #
> +# Software Foundation; version 2 of the License. #
> +# #
> +# This program is distributed in the hope that it will be useful, but WITHOUT #
> +# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or #
> +# FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for #
> +# more details. #
> +# #
> +# You should have received a copy of the GNU General Public License along #
> +# with this program; if not, write to the Free Software Foundation, Inc., 59 #
> +# Temple Place, Suite 330, Boston, MA 02111-1307 USA #
> +###############################################################################
> +
> +
> +from PyQt4 import QtCore, QtGui
> +from openlp.core.common import translate
> +
> +class Ui_MediaClipSelector(object):
> + def setupUi(self, MediaClipSelector):
> + MediaClipSelector.setObjectName("MediaClipSelector")
> + MediaClipSelector.resize(683, 739)
> + sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.MinimumExpanding, QtGui.QSizePolicy.MinimumExpanding)
> + sizePolicy.setHorizontalStretch(0)
> + sizePolicy.setVerticalStretch(0)
> + sizePolicy.setHeightForWidth(MediaClipSelector.sizePolicy().hasHeightForWidth())
> + MediaClipSelector.setSizePolicy(sizePolicy)
> + MediaClipSelector.setMinimumSize(QtCore.QSize(683, 686))
> + MediaClipSelector.setFocusPolicy(QtCore.Qt.NoFocus)
> + MediaClipSelector.setAutoFillBackground(False)
> + MediaClipSelector.setInputMethodHints(QtCore.Qt.ImhNone)
> + self.centralwidget = QtGui.QWidget(MediaClipSelector)
> + sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.MinimumExpanding, QtGui.QSizePolicy.MinimumExpanding)
> + sizePolicy.setHorizontalStretch(0)
> + sizePolicy.setVerticalStretch(0)
> + sizePolicy.setHeightForWidth(self.centralwidget.sizePolicy().hasHeightForWidth())
> + self.centralwidget.setSizePolicy(sizePolicy)
> + self.centralwidget.setObjectName("centralwidget")
> + self.gridLayout = QtGui.QGridLayout(self.centralwidget)
> + self.gridLayout.setObjectName("gridLayout")
> + self.media_path_combobox = QtGui.QComboBox(self.centralwidget)
> + self.media_path_combobox.setEnabled(True)
> + sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.Preferred, QtGui.QSizePolicy.Fixed)
> + sizePolicy.setHorizontalStretch(0)
> + sizePolicy.setVerticalStretch(0)
> + sizePolicy.setHeightForWidth(self.media_path_combobox.sizePolicy().hasHeightForWidth())
> + self.media_path_combobox.setSizePolicy(sizePolicy)
> + self.media_path_combobox.setEditable(True)
> + self.media_path_combobox.setObjectName("media_path_combobox")
> + self.gridLayout.addWidget(self.media_path_combobox, 0, 2, 1, 2)
> + self.start_timeedit = QtGui.QTimeEdit(self.centralwidget)
> + self.start_timeedit.setEnabled(True)
> + self.start_timeedit.setObjectName("start_timeedit")
> + self.gridLayout.addWidget(self.start_timeedit, 7, 2, 1, 1)
> + self.end_timeedit = QtGui.QTimeEdit(self.centralwidget)
> + self.end_timeedit.setEnabled(True)
> + self.end_timeedit.setObjectName("end_timeedit")
> + self.gridLayout.addWidget(self.end_timeedit, 8, 2, 1, 1)
> + self.set_start_pushbutton = QtGui.QPushButton(self.centralwidget)
> + self.set_start_pushbutton.setEnabled(True)
> + self.set_start_pushbutton.setObjectName("set_start_pushbutton")
> + self.gridLayout.addWidget(self.set_start_pushbutton, 7, 3, 1, 1)
> + self.load_disc_pushbutton = QtGui.QPushButton(self.centralwidget)
> + self.load_disc_pushbutton.setEnabled(True)
> + self.load_disc_pushbutton.setObjectName("load_disc_pushbutton")
> + self.gridLayout.addWidget(self.load_disc_pushbutton, 0, 4, 1, 1)
> + spacerItem = QtGui.QSpacerItem(20, 40, QtGui.QSizePolicy.Minimum, QtGui.QSizePolicy.Minimum)
> + self.gridLayout.addItem(spacerItem, 9, 3, 1, 1)
> + self.play_pushbutton = QtGui.QPushButton(self.centralwidget)
> + self.play_pushbutton.setEnabled(True)
> + self.play_pushbutton.setText("")
> + icon = QtGui.QIcon()
> + icon.addPixmap(QtGui.QPixmap(":/slides/media_playback_start.png"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
> + self.play_pushbutton.setIcon(icon)
> + self.play_pushbutton.setObjectName("play_pushbutton")
> + self.gridLayout.addWidget(self.play_pushbutton, 6, 0, 1, 1)
> + self.end_point_label = QtGui.QLabel(self.centralwidget)
> + self.end_point_label.setEnabled(True)
> + self.end_point_label.setObjectName("end_point_label")
> + self.gridLayout.addWidget(self.end_point_label, 8, 0, 1, 1)
> + self.subtitle_tracks_combobox = QtGui.QComboBox(self.centralwidget)
> + self.subtitle_tracks_combobox.setEnabled(True)
> + self.subtitle_tracks_combobox.setObjectName("subtitle_tracks_combobox")
> + self.gridLayout.addWidget(self.subtitle_tracks_combobox, 4, 2, 1, 2)
> + self.title_label = QtGui.QLabel(self.centralwidget)
> + self.title_label.setEnabled(True)
> + self.title_label.setObjectName("title_label")
> + self.gridLayout.addWidget(self.title_label, 2, 0, 1, 1)
> + self.audio_tracks_combobox = QtGui.QComboBox(self.centralwidget)
> + self.audio_tracks_combobox.setEnabled(True)
> + self.audio_tracks_combobox.setObjectName("audio_tracks_combobox")
> + self.gridLayout.addWidget(self.audio_tracks_combobox, 3, 2, 1, 2)
> + self.set_end_pushbutton = QtGui.QPushButton(self.centralwidget)
> + self.set_end_pushbutton.setEnabled(True)
> + self.set_end_pushbutton.setObjectName("set_end_pushbutton")
> + self.gridLayout.addWidget(self.set_end_pushbutton, 8, 3, 1, 1)
> + self.save_pushbutton = QtGui.QPushButton(self.centralwidget)
> + self.save_pushbutton.setEnabled(True)
> + self.save_pushbutton.setObjectName("save_pushbutton")
> + self.gridLayout.addWidget(self.save_pushbutton, 10, 3, 1, 1)
> + self.close_pushbutton = QtGui.QPushButton(self.centralwidget)
> + self.close_pushbutton.setEnabled(True)
> + self.close_pushbutton.setObjectName("close_pushbutton")
> + self.gridLayout.addWidget(self.close_pushbutton, 10, 4, 1, 1)
> + self.start_point_label = QtGui.QLabel(self.centralwidget)
> + self.start_point_label.setEnabled(True)
> + self.start_point_label.setObjectName("start_point_label")
> + self.gridLayout.addWidget(self.start_point_label, 7, 0, 1, 2)
> + self.jump_start_pushbutton = QtGui.QPushButton(self.centralwidget)
> + self.jump_start_pushbutton.setEnabled(True)
> + self.jump_start_pushbutton.setObjectName("jump_start_pushbutton")
> + self.gridLayout.addWidget(self.jump_start_pushbutton, 7, 4, 1, 1)
> + self.audio_track_label = QtGui.QLabel(self.centralwidget)
> + self.audio_track_label.setEnabled(True)
> + self.audio_track_label.setObjectName("audio_track_label")
> + self.gridLayout.addWidget(self.audio_track_label, 3, 0, 1, 2)
> + self.media_position_timeedit = QtGui.QTimeEdit(self.centralwidget)
> + self.media_position_timeedit.setEnabled(True)
> + self.media_position_timeedit.setReadOnly(True)
> + self.media_position_timeedit.setObjectName("media_position_timeedit")
> + self.gridLayout.addWidget(self.media_position_timeedit, 6, 4, 1, 1)
> + self.media_view_frame = QtGui.QFrame(self.centralwidget)
> + self.media_view_frame.setMinimumSize(QtCore.QSize(665, 375))
> + self.media_view_frame.setStyleSheet("background-color:black;")
> + self.media_view_frame.setFrameShape(QtGui.QFrame.StyledPanel)
> + self.media_view_frame.setFrameShadow(QtGui.QFrame.Raised)
> + self.media_view_frame.setObjectName("media_view_frame")
> + self.gridLayout.addWidget(self.media_view_frame, 5, 0, 1, 5)
> + self.subtitle_track_label = QtGui.QLabel(self.centralwidget)
> + self.subtitle_track_label.setEnabled(True)
> + self.subtitle_track_label.setObjectName("subtitle_track_label")
> + self.gridLayout.addWidget(self.subtitle_track_label, 4, 0, 1, 2)
> + self.jump_end_pushbutton = QtGui.QPushButton(self.centralwidget)
> + self.jump_end_pushbutton.setEnabled(True)
> + self.jump_end_pushbutton.setObjectName("jump_end_pushbutton")
> + self.gridLayout.addWidget(self.jump_end_pushbutton, 8, 4, 1, 1)
> + self.media_path_label = QtGui.QLabel(self.centralwidget)
> + self.media_path_label.setEnabled(True)
> + self.media_path_label.setObjectName("media_path_label")
> + self.gridLayout.addWidget(self.media_path_label, 0, 0, 1, 2)
> + self.title_combo_box = QtGui.QComboBox(self.centralwidget)
> + self.title_combo_box.setEnabled(True)
> + self.title_combo_box.setProperty("currentText", "")
> + self.title_combo_box.setObjectName("title_combo_box")
> + self.gridLayout.addWidget(self.title_combo_box, 2, 2, 1, 2)
> + self.position_horizontalslider = QtGui.QSlider(self.centralwidget)
> + self.position_horizontalslider.setEnabled(True)
> + self.position_horizontalslider.setTracking(False)
> + self.position_horizontalslider.setOrientation(QtCore.Qt.Horizontal)
> + self.position_horizontalslider.setInvertedAppearance(False)
> + self.position_horizontalslider.setObjectName("position_horizontalslider")
> + self.gridLayout.addWidget(self.position_horizontalslider, 6, 1, 1, 3)
> + #MediaClipSelector.setCentralWidget(self.centralwidget)
Commented out code
> +
> + self.retranslateUi(MediaClipSelector)
> + QtCore.QMetaObject.connectSlotsByName(MediaClipSelector)
> + MediaClipSelector.setTabOrder(self.media_path_combobox, self.load_disc_pushbutton)
> + MediaClipSelector.setTabOrder(self.load_disc_pushbutton, self.title_combo_box)
> + MediaClipSelector.setTabOrder(self.title_combo_box, self.audio_tracks_combobox)
> + MediaClipSelector.setTabOrder(self.audio_tracks_combobox, self.subtitle_tracks_combobox)
> + MediaClipSelector.setTabOrder(self.subtitle_tracks_combobox, self.play_pushbutton)
> + MediaClipSelector.setTabOrder(self.play_pushbutton, self.position_horizontalslider)
> + MediaClipSelector.setTabOrder(self.position_horizontalslider, self.media_position_timeedit)
> + MediaClipSelector.setTabOrder(self.media_position_timeedit, self.start_timeedit)
> + MediaClipSelector.setTabOrder(self.start_timeedit, self.set_start_pushbutton)
> + MediaClipSelector.setTabOrder(self.set_start_pushbutton, self.jump_start_pushbutton)
> + MediaClipSelector.setTabOrder(self.jump_start_pushbutton, self.end_timeedit)
> + MediaClipSelector.setTabOrder(self.end_timeedit, self.set_end_pushbutton)
> + MediaClipSelector.setTabOrder(self.set_end_pushbutton, self.jump_end_pushbutton)
> + MediaClipSelector.setTabOrder(self.jump_end_pushbutton, self.save_pushbutton)
> + MediaClipSelector.setTabOrder(self.save_pushbutton, self.close_pushbutton)
> +
> + def retranslateUi(self, MediaClipSelector):
> + MediaClipSelector.setWindowTitle(translate("MediaPlugin.MediaClipSelector", "Select media clip", None))
> + self.start_timeedit.setDisplayFormat(translate("MediaPlugin.MediaClipSelector", "HH:mm:ss.z", None))
> + self.end_timeedit.setDisplayFormat(translate("MediaPlugin.MediaClipSelector", "HH:mm:ss.z", None))
> + self.set_start_pushbutton.setText(translate("MediaPlugin.MediaClipSelector", "Set current position as start point", None))
> + self.load_disc_pushbutton.setText(translate("MediaPlugin.MediaClipSelector", "Load disc", None))
> + self.end_point_label.setText(translate("MediaPlugin.MediaClipSelector", "End point", None))
> + self.title_label.setText(translate("MediaPlugin.MediaClipSelector", "Title", None))
> + self.set_end_pushbutton.setText(translate("MediaPlugin.MediaClipSelector", "Set current position as end point", None))
> + self.save_pushbutton.setText(translate("MediaPlugin.MediaClipSelector", "Save current clip", None))
> + self.close_pushbutton.setText(translate("MediaPlugin.MediaClipSelector", "Close", None))
> + self.start_point_label.setText(translate("MediaPlugin.MediaClipSelector", "Start point", None))
> + self.jump_start_pushbutton.setText(translate("MediaPlugin.MediaClipSelector", "Jump to start point", None))
> + self.audio_track_label.setText(translate("MediaPlugin.MediaClipSelector", "Audio track", None))
> + self.media_position_timeedit.setDisplayFormat(translate("MediaPlugin.MediaClipSelector", "HH:mm:ss.z", None))
> + self.subtitle_track_label.setText(translate("MediaPlugin.MediaClipSelector", "Subtitle track", None))
> + self.jump_end_pushbutton.setText(translate("MediaPlugin.MediaClipSelector", "Jump to end point", None))
> + self.media_path_label.setText(translate("MediaPlugin.MediaClipSelector", "Media path", None))
> +
Extra blank line?
>
> === added file 'openlp/plugins/media/forms/mediaclipselectorform.py'
> --- openlp/plugins/media/forms/mediaclipselectorform.py 1970-01-01 00:00:00 +0000
> +++ openlp/plugins/media/forms/mediaclipselectorform.py 2014-06-16 07:07:43 +0000
> @@ -0,0 +1,645 @@
> +# -*- coding: utf-8 -*-
> +# vim: autoindent shiftwidth=4 expandtab textwidth=120 tabstop=4 softtabstop=4
> +
> +###############################################################################
> +# OpenLP - Open Source Lyrics Projection #
> +# --------------------------------------------------------------------------- #
> +# Copyright (c) 2008-2014 Raoul Snyman #
> +# Portions copyright (c) 2008-2014 Tim Bentley, Gerald Britton, Jonathan #
> +# Corwin, Samuel Findlay, Michael Gorven, Scott Guerrieri, Matthias Hub, #
> +# Meinert Jordan, Armin Köhler, Erik Lundin, Edwin Lunando, Brian T. Meyer. #
> +# Joshua Miller, Stevan Pettit, Andreas Preikschat, Mattias Põldaru, #
> +# Christian Richter, Philip Ridout, Simon Scudder, Jeffrey Smith, #
> +# Maikel Stuivenberg, Martin Thompson, Jon Tibble, Dave Warnock, #
> +# Frode Woldsund, Martin Zibricky, Patrick Zimmermann #
> +# --------------------------------------------------------------------------- #
> +# This program is free software; you can redistribute it and/or modify it #
> +# under the terms of the GNU General Public License as published by the Free #
> +# Software Foundation; version 2 of the License. #
> +# #
> +# This program is distributed in the hope that it will be useful, but WITHOUT #
> +# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or #
> +# FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for #
> +# more details. #
> +# #
> +# You should have received a copy of the GNU General Public License along #
> +# with this program; if not, write to the Free Software Foundation, Inc., 59 #
> +# Temple Place, Suite 330, Boston, MA 02111-1307 USA #
> +###############################################################################
> +
> +import os
> +if os.name == 'nt':
> + from ctypes import windll
> + import string
> +import sys
> +if sys.platform.startswith('linux'):
> + import dbus
> +import logging
> +from datetime import datetime
> +
> +
> +from PyQt4 import QtCore, QtGui
> +
> +from openlp.core.common import translate
> +from openlp.plugins.media.forms.mediaclipselectordialog import Ui_MediaClipSelector
> +from openlp.core.lib.ui import critical_error_message_box
> +from openlp.core.ui.media import format_milliseconds
> +try:
> + from openlp.core.ui.media.vendor import vlc
> +except (ImportError, NameError, NotImplementedError):
> + pass
> +except OSError as e:
> + if sys.platform.startswith('win'):
> + if not isinstance(e, WindowsError) and e.winerror != 126:
> + raise
> + else:
> + raise
> +
> +log = logging.getLogger(__name__)
> +
> +
> +class MediaClipSelectorForm(QtGui.QDialog, Ui_MediaClipSelector):
> + """
> + Class to manage the clip selection
> + """
> + log.info('%s MediaClipSelectorForm loaded', __name__)
> +
> + def __init__(self, media_item, parent, manager):
> + """
> + Constructor
> + """
> + super(MediaClipSelectorForm, self).__init__(parent)
> + self.media_item = media_item
> + self.setupUi(self)
> + # most actions auto-connect due to the functions name, so only a few left to do
> + self.close_pushbutton.clicked.connect(self.reject)
> + # setup play/pause icon
> + self.play_icon = QtGui.QIcon()
> + self.play_icon.addPixmap(QtGui.QPixmap(":/slides/media_playback_start.png"), QtGui.QIcon.Normal,
> + QtGui.QIcon.Off)
> + self.pause_icon = QtGui.QIcon()
> + self.pause_icon.addPixmap(QtGui.QPixmap(":/slides/media_playback_pause.png"), QtGui.QIcon.Normal,
> + QtGui.QIcon.Off)
> +
> + def reject(self):
> + """
> + Exit Dialog and do not save
> + """
> + log.debug('MediaClipSelectorForm.reject')
> + # Tear down vlc
> + if self.vlc_media_player:
> + self.vlc_media_player.stop()
> + self.vlc_media_player.release()
> + self.vlc_media_player = None
> + if self.vlc_instance:
> + self.vlc_instance.release()
> + self.vlc_instance = None
> + if self.vlc_media:
> + self.vlc_media.release()
> + self.vlc_media = None
> + QtGui.QDialog.reject(self)
> +
> + def exec_(self):
> + """
> + Start dialog
> + """
> + self.reset_ui()
> + self.setup_vlc()
> + return QtGui.QDialog.exec_(self)
> +
> + def reset_ui(self):
> + """
> + Reset the UI to default values
> + """
> + self.playback_length = 0
> + self.position_horizontalslider.setMinimum(0)
> + self.disable_all()
> + self.toggle_disable_load_media(False)
> + self.subtitle_tracks_combobox.clear()
> + self.audio_tracks_combobox.clear()
> + self.title_combo_box.clear()
> + time = QtCore.QTime()
> + self.start_timeedit.setTime(time)
> + self.end_timeedit.setTime(time)
> + self.media_position_timeedit.setTime(time)
> +
> + def setup_vlc(self):
> + """
> + Setup VLC instance and mediaplayer
> + """
> + self.vlc_instance = vlc.Instance()
> + # creating an empty vlc media player
> + self.vlc_media_player = self.vlc_instance.media_player_new()
> + # The media player has to be 'connected' to the QFrame.
> + # (otherwise a video would be displayed in it's own window)
> + # This is platform specific!
> + # You have to give the id of the QFrame (or similar object)
> + # to vlc, different platforms have different functions for this.
> + win_id = int(self.media_view_frame.winId())
> + if sys.platform == "win32":
> + self.vlc_media_player.set_hwnd(win_id)
> + elif sys.platform == "darwin":
> + # We have to use 'set_nsobject' since Qt4 on OSX uses Cocoa
> + # framework and not the old Carbon.
> + self.vlc_media_player.set_nsobject(win_id)
> + else:
> + # for Linux using the X Server
> + self.vlc_media_player.set_xwindow(win_id)
> + self.vlc_media = None
> + # Setup timer every 100 ms to update position
> + self.timer = QtCore.QTimer(self)
> + self.timer.timeout.connect(self.update_position)
> + self.timer.start(100)
> + self.find_optical_devices()
> + self.audio_cd = False
> + self.audio_cd_tracks = None
> +
> + def detect_audio_cd(self, path):
> + """
> + Detects is the given path is an audio CD
> +
> + :param path: Path to the device to be tested.
> + :return: True if it was an audio CD else False.
> + """
> + # Detect by trying to play it as a CD
> + self.vlc_media = self.vlc_instance.media_new_location('cdda://' + path)
> + self.vlc_media_player.set_media(self.vlc_media)
> + self.vlc_media_player.play()
> + # Wait for media to start playing. In this case VLC actually returns an error.
> + self.media_state_wait(vlc.State.Playing)
> + self.vlc_media_player.set_pause(1)
> + # If subitems exists, this is a CD
> + self.audio_cd_tracks = self.vlc_media.subitems()
> + if not self.audio_cd_tracks or self.audio_cd_tracks.count() < 1:
> + return False
> + # Insert into title_combo_box
> + self.title_combo_box.clear()
> + for i in range(self.audio_cd_tracks.count()):
> + item = self.audio_cd_tracks.item_at_index(i)
> + item_title = item.get_meta(vlc.Meta.Title)
> + self.title_combo_box.addItem(item_title, i)
> + self.vlc_media_player.set_media(self.audio_cd_tracks.item_at_index(0))
> + self.audio_cd = True
> + self.title_combo_box.setDisabled(False)
> + self.title_combo_box.setCurrentIndex(0)
> + self.on_title_combo_box_currentIndexChanged(0)
> +
> + return True
> +
> + @QtCore.pyqtSlot(bool)
Need to check witn Raoul as we normally use object.connect for slots
> + def on_load_disc_pushbutton_clicked(self, clicked):
> + """
> + Load the media when the load-button has been clicked
> +
> + :param clicked: Given from signal, not used.
> + """
> + self.disable_all()
> + path = self.media_path_combobox.currentText()
> + # Check if given path is non-empty and exists before starting VLC
> + if path == '':
if not path
> + log.debug('no given path')
> + critical_error_message_box(message=translate('MediaPlugin.MediaClipSelectorForm', 'No path was given'))
> + self.toggle_disable_load_media(False)
> + return
> + if not os.path.exists(path):
> + log.debug('Given path does not exists')
> + critical_error_message_box(message=translate('MediaPlugin.MediaClipSelectorForm',
> + 'Given path does not exists'))
> + self.toggle_disable_load_media(False)
> + return
> + self.vlc_media = self.vlc_instance.media_new_location(path)
> + if not self.vlc_media:
> + log.debug('vlc media player is none')
> + critical_error_message_box(message=translate('MediaPlugin.MediaClipSelectorForm',
> + 'An error happened during initialization of VLC player'))
> + self.toggle_disable_load_media(False)
> + return
> + # put the media in the media player
> + self.vlc_media_player.set_media(self.vlc_media)
> + self.vlc_media_player.audio_set_mute(True)
> + # start playback to get vlc to parse the media
> + if self.vlc_media_player.play() < 0:
> + log.debug('vlc play returned error')
> + critical_error_message_box(message=translate('MediaPlugin.MediaClipSelectorForm',
> + 'VLC player failed playing the media'))
> + self.toggle_disable_load_media(False)
> + return
> + self.vlc_media_player.audio_set_mute(True)
> + if not self.media_state_wait(vlc.State.Playing):
> + # Tests if this is an audio CD
> + if not self.detect_audio_cd(path):
> + critical_error_message_box(message=translate('MediaPlugin.MediaClipSelectorForm',
> + 'VLC player failed playing the media'))
> + self.toggle_disable_load_media(False)
> + return
> + self.vlc_media_player.set_pause(1)
> + self.vlc_media_player.set_time(0)
> + if not self.audio_cd:
> + # Get titles, insert in combobox
> + titles = self.vlc_media_player.video_get_title_description()
> + self.title_combo_box.clear()
> + for title in titles:
> + self.title_combo_box.addItem(title[1].decode(), title[0])
> + # Main title is usually title #1
> + if len(titles) > 1:
> + self.title_combo_box.setCurrentIndex(1)
> + else:
> + self.title_combo_box.setCurrentIndex(0)
> + # Enable audio track combobox if anything is in it
> + if len(titles) > 0:
> + self.title_combo_box.setDisabled(False)
> + self.vlc_media_player.set_pause(1)
> + self.toggle_disable_load_media(False)
> +
> + @QtCore.pyqtSlot(bool)
> + def on_play_pushbutton_clicked(self, clicked):
> + """
> + Toggle the playback
> +
> + :param clicked: Given from signal, not used.
> + """
> + if self.vlc_media_player.get_state() == vlc.State.Playing:
> + self.vlc_media_player.pause()
> + self.play_pushbutton.setIcon(self.play_icon)
> + else:
> + self.vlc_media_player.play()
> + self.media_state_wait(vlc.State.Playing)
> + self.play_pushbutton.setIcon(self.pause_icon)
> +
> + @QtCore.pyqtSlot(bool)
> + def on_set_start_pushbutton_clicked(self, clicked):
> + """
> + Copy the current player position to start_timeedit
> +
> + :param clicked: Given from signal, not used.
> + """
> + vlc_ms_pos = self.vlc_media_player.get_time()
> + time = QtCore.QTime()
> + new_pos_time = time.addMSecs(vlc_ms_pos)
> + self.start_timeedit.setTime(new_pos_time)
> + # If start time is after end time, update end time.
> + end_time = self.end_timeedit.time()
> + if end_time < new_pos_time:
> + self.end_timeedit.setTime(new_pos_time)
> +
> + @QtCore.pyqtSlot(bool)
> + def on_set_end_pushbutton_clicked(self, clicked):
> + """
> + Copy the current player position to end_timeedit
> +
> + :param clicked: Given from signal, not used.
> + """
> + vlc_ms_pos = self.vlc_media_player.get_time()
> + time = QtCore.QTime()
> + new_pos_time = time.addMSecs(vlc_ms_pos)
> + self.end_timeedit.setTime(new_pos_time)
> + # If start time is after end time, update start time.
> + start_time = self.start_timeedit.time()
> + if start_time > new_pos_time:
> + self.start_timeedit.setTime(new_pos_time)
> +
> + @QtCore.pyqtSlot(QtCore.QTime)
> + def on_start_timeedit_timeChanged(self, new_time):
> + """
> + Called when start_timeedit is changed manually
> +
> + :param new_time: The new time
> + """
> + # If start time is after end time, update end time.
> + end_time = self.end_timeedit.time()
> + if end_time < new_time:
> + self.end_timeedit.setTime(new_time)
> +
> + @QtCore.pyqtSlot(QtCore.QTime)
> + def on_end_timeedit_timeChanged(self, new_time):
> + """
> + Called when end_timeedit is changed manually
> +
> + :param new_time: The new time
> + """
> + # If start time is after end time, update start time.
> + start_time = self.start_timeedit.time()
> + if start_time > new_time:
> + self.start_timeedit.setTime(new_time)
> +
> + @QtCore.pyqtSlot(bool)
> + def on_jump_end_pushbutton_clicked(self, clicked):
> + """
> + Set the player position to the position stored in end_timeedit
> +
> + :param clicked: Given from signal, not used.
> + """
> + end_time = self.end_timeedit.time()
> + end_time_ms = end_time.hour() * 60 * 60 * 1000 + \
> + end_time.minute() * 60 * 1000 + \
> + end_time.second() * 1000 + \
> + end_time.msec()
> + self.vlc_media_player.set_time(end_time_ms)
> +
> + @QtCore.pyqtSlot(bool)
> + def on_jump_start_pushbutton_clicked(self, clicked):
> + """
> + Set the player position to the position stored in start_timeedit
> +
> + :param clicked: Given from signal, not used.
> + """
> + start_time = self.start_timeedit.time()
> + start_time_ms = start_time.hour() * 60 * 60 * 1000 + \
> + start_time.minute() * 60 * 1000 + \
> + start_time.second() * 1000 + \
> + start_time.msec()
> + self.vlc_media_player.set_time(start_time_ms)
> +
> + @QtCore.pyqtSlot(int)
> + def on_title_combo_box_currentIndexChanged(self, index):
> + """
> + When a new title is chosen, it is loaded by VLC and info about audio and subtitle tracks is reloaded
> +
> + :param index: The index of the newly chosen title track.
> + """
> + log.debug('in on_title_combo_box_changed, index: %d', index)
> + if not self.vlc_media_player:
> + return
> + if self.audio_cd:
> + self.vlc_media = self.audio_cd_tracks.item_at_index(index)
> + self.vlc_media_player.set_media(self.vlc_media)
> + self.vlc_media_player.set_time(0)
> + self.vlc_media_player.play()
> + self.vlc_media_player.audio_set_mute(True)
> + if not self.media_state_wait(vlc.State.Playing):
> + return
> + # pause
> + self.vlc_media_player.set_pause(1)
> + self.vlc_media_player.set_time(0)
> + self.vlc_media_player.audio_set_mute(False)
> + self.toggle_disable_player(False)
> + else:
> + self.vlc_media_player.set_title(index)
> + self.vlc_media_player.set_time(0)
> + self.vlc_media_player.play()
> + self.vlc_media_player.audio_set_mute(True)
> + if not self.media_state_wait(vlc.State.Playing):
> + return
> + # pause
> + self.vlc_media_player.set_pause(1)
> + self.vlc_media_player.set_time(0)
> + # Get audio tracks, insert in combobox
> + audio_tracks = self.vlc_media_player.audio_get_track_description()
> + self.audio_tracks_combobox.clear()
> + for audio_track in audio_tracks:
> + self.audio_tracks_combobox.addItem(audio_track[1].decode(), audio_track[0])
> + # Enable audio track combobox if anything is in it
> + if len(audio_tracks) > 0:
> + self.audio_tracks_combobox.setDisabled(False)
> + # First track is "deactivated", so set to next if it exists
> + if len(audio_tracks) > 1:
> + self.audio_tracks_combobox.setCurrentIndex(1)
> + # Get subtitle tracks, insert in combobox
> + subtitles_tracks = self.vlc_media_player.video_get_spu_description()
> + self.subtitle_tracks_combobox.clear()
> + for subtitle_track in subtitles_tracks:
> + self.subtitle_tracks_combobox.addItem(subtitle_track[1].decode(), subtitle_track[0])
> + # Enable subtitle track combobox is anything in it
> + if len(subtitles_tracks) > 0:
> + self.subtitle_tracks_combobox.setDisabled(False)
> + self.vlc_media_player.audio_set_mute(False)
> + # If a title or audio track is available the player is enabled
> + if self.title_combo_box.count() > 0 or len(audio_tracks) > 0:
> + self.toggle_disable_player(False)
> + # Set media length info
> + self.playback_length = self.vlc_media_player.get_length()
> + self.position_horizontalslider.setMaximum(self.playback_length)
> + # setup start and end time
> + rounded_vlc_ms_length = int(round(self.playback_length / 100.0) * 100.0)
> + time = QtCore.QTime()
> + playback_length_time = time.addMSecs(rounded_vlc_ms_length)
> + self.start_timeedit.setMaximumTime(playback_length_time)
> + self.end_timeedit.setMaximumTime(playback_length_time)
> + self.end_timeedit.setTime(playback_length_time)
> +
> + @QtCore.pyqtSlot(int)
> + def on_audio_tracks_combobox_currentIndexChanged(self, index):
> + """
> + When a new audio track is chosen update audio track bing played by VLC
> +
> + :param index: The index of the newly chosen audio track.
> + """
> + if not self.vlc_media_player:
> + return
> + audio_track = self.audio_tracks_combobox.itemData(index)
> + log.debug('in on_audio_tracks_combobox_currentIndexChanged, index: %d audio_track: %s' % (index, audio_track))
> + if audio_track and int(audio_track) > 0:
> + self.vlc_media_player.audio_set_track(int(audio_track))
> +
> + @QtCore.pyqtSlot(int)
> + def on_subtitle_tracks_combobox_currentIndexChanged(self, index):
> + """
> + When a new subtitle track is chosen update subtitle track bing played by VLC
> +
> + :param index: The index of the newly chosen subtitle.
> + """
> + if not self.vlc_media_player:
> + return
> + subtitle_track = self.subtitle_tracks_combobox.itemData(index)
> + if subtitle_track:
> + self.vlc_media_player.video_set_spu(int(subtitle_track))
> +
> + def on_position_horizontalslider_sliderMoved(self, position):
> + """
> + Set player position according to new slider position.
> +
> + :param position: Position to seek to.
> + """
> + self.vlc_media_player.set_time(position)
> +
> + def update_position(self):
> + """
> + Update slider position and displayed time according to VLC player position.
> + """
> + if self.vlc_media_player:
> + vlc_ms_pos = self.vlc_media_player.get_time()
> + rounded_vlc_ms_pos = int(round(vlc_ms_pos / 100.0) * 100.0)
> + time = QtCore.QTime()
> + new_pos_time = time.addMSecs(rounded_vlc_ms_pos)
> + self.media_position_timeedit.setTime(new_pos_time)
> + self.position_horizontalslider.setSliderPosition(vlc_ms_pos)
> +
> + def disable_all(self):
> + """
> + Disable all elements in the dialog
> + """
> + self.toggle_disable_load_media(True)
> + self.title_combo_box.setDisabled(True)
> + self.audio_tracks_combobox.setDisabled(True)
> + self.subtitle_tracks_combobox.setDisabled(True)
> + self.toggle_disable_player(True)
> +
> + def toggle_disable_load_media(self, action):
> + """
> + Enable/disable load media combobox and button.
> +
> + :param action: If True elements are disabled, if False they are enabled.
> + """
> + self.media_path_combobox.setDisabled(action)
> + self.load_disc_pushbutton.setDisabled(action)
> +
> + def toggle_disable_player(self, action):
> + """
> + Enable/disable player elements.
> +
> + :param action: If True elements are disabled, if False they are enabled.
> + """
> + self.play_pushbutton.setDisabled(action)
> + self.position_horizontalslider.setDisabled(action)
> + self.media_position_timeedit.setDisabled(action)
> + self.start_timeedit.setDisabled(action)
> + self.set_start_pushbutton.setDisabled(action)
> + self.jump_start_pushbutton.setDisabled(action)
> + self.end_timeedit.setDisabled(action)
> + self.set_end_pushbutton.setDisabled(action)
> + self.jump_end_pushbutton.setDisabled(action)
> + self.save_pushbutton.setDisabled(action)
> +
> + @QtCore.pyqtSlot(bool)
> + def on_save_pushbutton_clicked(self, clicked):
> + """
> + Saves the current media and trackinfo as a clip to the mediamanager
> +
> + :param clicked: Given from signal, not used.
> + """
> + log.debug('in on_save_pushbutton_clicked')
> + start_time = self.start_timeedit.time()
> + start_time_ms = start_time.hour() * 60 * 60 * 1000 + \
> + start_time.minute() * 60 * 1000 + \
> + start_time.second() * 1000 + \
> + start_time.msec()
> + end_time = self.end_timeedit.time()
> + end_time_ms = end_time.hour() * 60 * 60 * 1000 + \
> + end_time.minute() * 60 * 1000 + \
> + end_time.second() * 1000 + \
> + end_time.msec()
> + title = self.title_combo_box.itemData(self.title_combo_box.currentIndex())
> + path = self.media_path_combobox.currentText()
> + optical = ''
> + if self.audio_cd:
> + optical = 'optical:' + str(title) + ':-1:-1:' + str(start_time_ms) + ':' + str(end_time_ms) + ':'
Can this be done with like "optical:%s:-1:-1%s" etc
> + else:
> + audio_track = self.audio_tracks_combobox.itemData(self.audio_tracks_combobox.currentIndex())
> + subtitle_track = self.subtitle_tracks_combobox.itemData(self.subtitle_tracks_combobox.currentIndex())
> + optical = 'optical:' + str(title) + ':' + str(audio_track) + ':' + str(subtitle_track) + ':' + str(
> + start_time_ms) + ':' + str(end_time_ms) + ':'
> + # Ask for an alternative name for the mediaclip
> + while True:
> + new_optical_name, ok = QtGui.QInputDialog.getText(self, translate('MediaPlugin.MediaClipSelectorForm',
> + 'Set name of mediaclip'),
> + translate('MediaPlugin.MediaClipSelectorForm',
> + 'Name of mediaclip:'),
> + QtGui.QLineEdit.Normal)
> + # User pressed cancel, don't save the clip
> + if not ok:
> + return
> + # User pressed ok, but the input text is blank
> + if not new_optical_name:
> + critical_error_message_box(translate('MediaPlugin.MediaClipSelectorForm',
> + 'Enter a valid name or cancel'),
> + translate('MediaPlugin.MediaClipSelectorForm',
> + 'Enter a valid name or cancel'))
> + # The entered new name contains a colon, which we don't allow because colons is used to seperate clip info
> + elif new_optical_name.find(':') >= 0:
> + critical_error_message_box(translate('MediaPlugin.MediaClipSelectorForm', 'Invalid character'),
> + translate('MediaPlugin.MediaClipSelectorForm',
> + 'The name of the mediaclip must not contain the character ":"'))
> + # New name entered and we use it
> + else:
> + break
> + # Append the new name to the optical string and the path
> + optical += new_optical_name + ':' + path
> + self.media_item.add_optical_clip(optical)
> +
> + def media_state_wait(self, media_state):
> + """
> + Wait for the video to change its state
> + Wait no longer than 15 seconds. (loading an optical disc takes some time)
> +
> + :param media_state: VLC media state to wait for.
> + :return: True if state was reached within 15 seconds, False if not or error occurred.
> + """
> + start = datetime.now()
> + while media_state != self.vlc_media_player.get_state():
> + if self.vlc_media_player.get_state() == vlc.State.Error:
> + return False
> + if (datetime.now() - start).seconds > 15:
> + return False
> + return True
> +
> + def find_optical_devices(self):
> + """
> + Attempt to autodetect optical devices on the computer, and add them to the media-dropdown
> + :return:
> + """
> + # Clear list first
> + self.media_path_combobox.clear()
> + # insert empty string as first item
> + self.media_path_combobox.addItem('')
> + if os.name == 'nt':
> + # use win api to fine optical drives
> + bitmask = windll.kernel32.GetLogicalDrives()
> + for letter in string.uppercase:
> + if bitmask & 1:
> + try:
> + type = windll.kernel32.GetDriveTypeW('%s:\\' % letter)
> + # if type is 5, it is a cd-rom drive
> + if type == 5:
> + self.media_path_combobox.addItem('%s:\\' % letter)
> + except Exception as e:
> + log.debug('Exception while looking for optical drives: ', e)
> + bitmask >>= 1
> + elif sys.platform.startswith('linux'):
> + # Get disc devices from dbus and find the ones that are optical
> + bus = dbus.SystemBus()
> + try:
> + udev_manager_obj = bus.get_object('org.freedesktop.UDisks', '/org/freedesktop/UDisks')
> + udev_manager = dbus.Interface(udev_manager_obj, 'org.freedesktop.UDisks')
> + for dev in udev_manager.EnumerateDevices():
> + device_obj = bus.get_object("org.freedesktop.UDisks", dev)
> + device_props = dbus.Interface(device_obj, dbus.PROPERTIES_IFACE)
> + if device_props.Get('org.freedesktop.UDisks.Device', 'DeviceIsDrive'):
> + drive_props = device_props.Get('org.freedesktop.UDisks.Device', 'DriveMediaCompatibility')
> + if any('optical' in prop for prop in drive_props):
> + self.media_path_combobox.addItem(device_props.Get('org.freedesktop.UDisks.Device',
> + 'DeviceFile'))
> + return
> + except dbus.exceptions.DBusException:
> + log.debug('could not use udisks, will try udisks2')
> + udev_manager_obj = bus.get_object('org.freedesktop.UDisks2', '/org/freedesktop/UDisks2')
> + udev_manager = dbus.Interface(udev_manager_obj, 'org.freedesktop.DBus.ObjectManager')
> + for k, v in udev_manager.GetManagedObjects().items():
> + drive_info = v.get('org.freedesktop.UDisks2.Drive', {})
> + drive_props = drive_info.get('MediaCompatibility')
> + if drive_props and any('optical' in prop for prop in drive_props):
> + for device in udev_manager.GetManagedObjects().values():
> + if dbus.String('org.freedesktop.UDisks2.Block') in device:
> + if device[dbus.String('org.freedesktop.UDisks2.Block')][dbus.String('Drive')] == k:
> + block_file = ''
> + for c in device[dbus.String('org.freedesktop.UDisks2.Block')][
> + dbus.String('PreferredDevice')]:
> + if chr(c) != '\x00':
> + block_file += chr(c)
> + self.media_path_combobox.addItem(block_file)
> + elif sys.platform.startswith('darwin'):
> + # Look for DVD folders in devices to find optical devices
> + volumes = os.listdir('/Volumes')
> + candidates = list()
> + for volume in volumes:
> + if volume.startswith('.'):
> + continue
> + dirs = os.listdir('/Volumes/' + volume)
> + # Detect DVD
> + if 'VIDEO_TS' in dirs:
> + self.media_path_combobox.addItem('/Volumes/' + volume)
> + # Detect audio cd
> + files = [f for f in dirs if os.path.isfile(f)]
> + for file in files:
> + if file.endswith('aiff'):
> + self.media_path_combobox.addItem('/Volumes/' + volume)
> + break
>
> === modified file 'openlp/plugins/media/lib/mediaitem.py'
> --- openlp/plugins/media/lib/mediaitem.py 2014-05-03 20:00:17 +0000
> +++ openlp/plugins/media/lib/mediaitem.py 2014-06-16 07:07:43 +0000
> @@ -29,6 +29,7 @@
>
> import logging
> import os
> +from datetime import time
>
> from PyQt4 import QtCore, QtGui
>
> @@ -38,17 +39,21 @@
> build_icon, check_item_selected
> from openlp.core.lib.ui import critical_error_message_box, create_horizontal_adjusting_combo_box
> from openlp.core.ui import DisplayController, Display, DisplayControllerType
> -from openlp.core.ui.media import get_media_players, set_media_players
> +from openlp.core.ui.media import get_media_players, set_media_players, parse_optical_path, format_milliseconds
> from openlp.core.utils import get_locale_key
> +from openlp.core.ui.media.vlcplayer import VLC_AVAILABLE
> +if VLC_AVAILABLE:
> + from openlp.plugins.media.forms.mediaclipselectorform import MediaClipSelectorForm
>
>
> log = logging.getLogger(__name__)
>
>
> CLAPPERBOARD = ':/media/slidecontroller_multimedia.png'
> +OPTICAL = ':/media/media_optical.png'
> VIDEO_ICON = build_icon(':/media/media_video.png')
> AUDIO_ICON = build_icon(':/media/media_audio.png')
> -DVD_ICON = build_icon(':/media/media_video.png')
> +OPTICAL_ICON = build_icon(OPTICAL)
> ERROR_ICON = build_icon(':/general/general_delete.png')
>
>
> @@ -88,6 +93,10 @@
> self.list_view.activateDnD()
>
> def retranslateUi(self):
> + """
> + This method is called automatically to provide OpenLP with the opportunity to translate the ``MediaManagerItem``
> + to another language.
> + """
> self.on_new_prompt = translate('MediaPlugin.MediaItem', 'Select Media')
> self.replace_action.setText(UiStrings().ReplaceBG)
> self.replace_action.setToolTip(UiStrings().ReplaceLiveBG)
> @@ -106,10 +115,36 @@
> self.has_edit_icon = False
>
> def add_list_view_to_toolbar(self):
> + """
> + Creates the main widget for listing items.
> + """
> MediaManagerItem.add_list_view_to_toolbar(self)
> self.list_view.addAction(self.replace_action)
>
> + def add_start_header_bar(self):
> + """
> + Adds buttons to the start of the header bar.
> + """
> + print(get_media_players()[0])
> + if 'vlc' in get_media_players()[0]:
> + diable_optical_button_text = False
> + optical_button_text = translate('MediaPlugin.MediaItem', 'Load CD/DVD')
> + optical_button_tooltip = translate('MediaPlugin.MediaItem', 'Load CD/DVD')
> + else:
> + diable_optical_button_text = True
> + optical_button_text = translate('MediaPlugin.MediaItem', 'Load CD/DVD')
> + optical_button_tooltip = translate('MediaPlugin.MediaItem',
> + 'Load CD/DVD - only supported when VLC is installed and enabled')
> + self.load_optical = self.toolbar.add_toolbar_action('load_optical', icon=OPTICAL_ICON, text=optical_button_text,
> + tooltip=optical_button_tooltip,
> + triggers=self.on_load_optical)
> + if diable_optical_button_text:
> + self.load_optical.setDisabled(True)
> +
> def add_end_header_bar(self):
> + """
> + Adds buttons to the end of the header bar.
> + """
> # Replace backgrounds do not work at present so remove functionality.
> self.replace_action = self.toolbar.add_toolbar_action('replace_action', icon=':/slides/slide_blank.png',
> triggers=self.on_replace_click)
> @@ -198,22 +233,44 @@
> if item is None:
> return False
> filename = item.data(QtCore.Qt.UserRole)
> - if not os.path.exists(filename):
> - if not remote:
> - # File is no longer present
> - critical_error_message_box(
> - translate('MediaPlugin.MediaItem', 'Missing Media File'),
> - translate('MediaPlugin.MediaItem', 'The file %s no longer exists.') % filename)
> - return False
> - (path, name) = os.path.split(filename)
> - service_item.title = name
> - service_item.processor = self.display_type_combo_box.currentText()
> - service_item.add_from_command(path, name, CLAPPERBOARD)
> - # Only get start and end times if going to a service
> - if context == ServiceItemContext.Service:
> - # Start media and obtain the length
> - if not self.media_controller.media_length(service_item):
> - return False
> + # Special handling if the filename is a optical clip
> + if filename.startswith('optical:'):
> + (name, title, audio_track, subtitle_track, start, end, clip_name) = parse_optical_path(filename)
> + if not os.path.exists(name):
> + if not remote:
> + # Optical disc is no longer present
> + critical_error_message_box(
> + translate('MediaPlugin.MediaItem', 'Missing Media File'),
> + translate('MediaPlugin.MediaItem', 'The optical disc %s is no longer available.') % name)
> + return False
> + service_item.processor = self.display_type_combo_box.currentText()
> + service_item.add_from_command(filename, name, CLAPPERBOARD)
> + service_item.title = clip_name
> + # Only set start and end times if going to a service
> + #if context == ServiceItemContext.Service:
commented code
> + # Set the length
> + self.media_controller.media_setup_optical(name, title, audio_track, subtitle_track, start, end, None, None)
> + service_item.set_media_length((end - start) / 1000)
> + service_item.start_time = start / 1000
> + service_item.end_time = end / 1000
> + service_item.add_capability(ItemCapabilities.IsOptical)
> + else:
> + if not os.path.exists(filename):
> + if not remote:
> + # File is no longer present
> + critical_error_message_box(
> + translate('MediaPlugin.MediaItem', 'Missing Media File'),
> + translate('MediaPlugin.MediaItem', 'The file %s no longer exists.') % filename)
> + return False
> + (path, name) = os.path.split(filename)
> + service_item.title = name
> + service_item.processor = self.display_type_combo_box.currentText()
> + service_item.add_from_command(path, name, CLAPPERBOARD)
> + # Only get start and end times if going to a service
> + if context == ServiceItemContext.Service:
> + # Start media and obtain the length
> + if not self.media_controller.media_length(service_item):
> + return False
> service_item.add_capability(ItemCapabilities.CanAutoStartForLive)
> service_item.add_capability(ItemCapabilities.CanEditTitle)
> service_item.add_capability(ItemCapabilities.RequiresMedia)
> @@ -224,12 +281,17 @@
> return True
>
> def initialise(self):
> + """
> + Initialize media item.
> + """
> self.list_view.clear()
> self.list_view.setIconSize(QtCore.QSize(88, 50))
> self.service_path = os.path.join(AppLocation.get_section_data_path(self.settings_section), 'thumbnails')
> check_directory_exists(self.service_path)
> self.load_list(Settings().value(self.settings_section + '/media files'))
> self.rebuild_players()
> + if VLC_AVAILABLE:
> + self.media_clip_selector_form = MediaClipSelectorForm(self, self.main_window, None)
>
> def rebuild_players(self):
> """
> @@ -241,6 +303,9 @@
> ' '.join(self.media_controller.audio_extensions_list), UiStrings().AllFiles)
>
> def display_setup(self):
> + """
> + Setup media controller display.
> + """
> self.media_controller.setup_display(self.display_controller.preview_display, False)
>
> def populate_display_types(self):
> @@ -280,7 +345,6 @@
> Settings().setValue(self.settings_section + '/media files', self.get_file_list())
>
> def load_list(self, media, target_group=None):
> - # Sort the media by its filename considering language specific characters.
> """
> Load the media list
>
> @@ -290,12 +354,22 @@
> media.sort(key=lambda file_name: get_locale_key(os.path.split(str(file_name))[1]))
> for track in media:
> track_info = QtCore.QFileInfo(track)
> - if not os.path.exists(track):
> + if track.startswith('optical:'):
> + # Handle optical based item
> + (file_name, title, audio_track, subtitle_track, start, end, clip_name) = parse_optical_path(track)
> + item_name = QtGui.QListWidgetItem(clip_name)
> + item_name.setIcon(OPTICAL_ICON)
> + item_name.setData(QtCore.Qt.UserRole, track)
> + item_name.setToolTip(file_name + '@' + format_milliseconds(start) + '-' + format_milliseconds(end))
> + elif not os.path.exists(track):
> + # File doesn't exist, mark as error.
> file_name = os.path.split(str(track))[1]
> item_name = QtGui.QListWidgetItem(file_name)
> item_name.setIcon(ERROR_ICON)
> item_name.setData(QtCore.Qt.UserRole, track)
> + item_name.setToolTip(track)
> elif track_info.isFile():
> + # Normal media file handling.
> file_name = os.path.split(str(track))[1]
> item_name = QtGui.QListWidgetItem(file_name)
> if '*.%s' % (file_name.split('.')[-1].lower()) in self.media_controller.audio_extensions_list:
> @@ -303,15 +377,16 @@
> else:
> item_name.setIcon(VIDEO_ICON)
> item_name.setData(QtCore.Qt.UserRole, track)
> - else:
> - file_name = os.path.split(str(track))[1]
> - item_name = QtGui.QListWidgetItem(file_name)
> - item_name.setIcon(build_icon(DVD_ICON))
> - item_name.setData(QtCore.Qt.UserRole, track)
> - item_name.setToolTip(track)
> + item_name.setToolTip(track)
> self.list_view.addItem(item_name)
>
> def get_list(self, type=MediaType.Audio):
> + """
> + Get the list of media, optional select media type.
> +
> + :param type: Type to get, defaults to audio.
> + :return: The media list
> + """
> media = Settings().value(self.settings_section + '/media files')
> media.sort(key=lambda filename: get_locale_key(os.path.split(str(filename))[1]))
> if type == MediaType.Audio:
> @@ -323,6 +398,13 @@
> return media
>
> def search(self, string, show_error):
> + """
> + Performs a search for items containing ``string``
> +
> + :param string: String to be displayed
> + :param show_error: Should the error be shown (True)
> + :return: The search result.
> + """
> files = Settings().value(self.settings_section + '/media files')
> results = []
> string = string.lower()
> @@ -331,3 +413,26 @@
> if filename.lower().find(string) > -1:
> results.append([file, filename])
> return results
> +
> + def on_load_optical(self):
> + """
> + When the load optical button is clicked, open the clip selector window.
> + """
> + self.media_clip_selector_form.exec_()
> +
> + def add_optical_clip(self, optical):
> + """
> + Add a optical based clip to the mediamanager, called from media_clip_selector_form.
> +
> + :param optical: The clip to add.
> + """
> + full_list = self.get_file_list()
> + # If the clip already is in the media list it isn't added and an error message is displayed.
> + if optical in full_list:
> + critical_error_message_box(translate('MediaPlugin.MediaItem', 'Mediaclip already saved'),
> + translate('MediaPlugin.MediaItem', 'This mediaclip has already been saved'))
> + return
> + # Append the optical string to the media list
> + full_list.append(optical)
> + self.load_list([optical])
> + Settings().setValue(self.settings_section + '/media files', self.get_file_list())
>
> === added file 'resources/forms/mediaclipselector.ui'
> --- resources/forms/mediaclipselector.ui 1970-01-01 00:00:00 +0000
> +++ resources/forms/mediaclipselector.ui 2014-06-16 07:07:43 +0000
> @@ -0,0 +1,336 @@
> +<?xml version="1.0" encoding="UTF-8"?>
> +<ui version="4.0">
> + <class>MediaClipSelector</class>
> + <widget class="QMainWindow" name="MediaClipSelector">
> + <property name="geometry">
> + <rect>
> + <x>0</x>
> + <y>0</y>
> + <width>683</width>
> + <height>739</height>
> + </rect>
> + </property>
> + <property name="sizePolicy">
> + <sizepolicy hsizetype="MinimumExpanding" vsizetype="MinimumExpanding">
> + <horstretch>0</horstretch>
> + <verstretch>0</verstretch>
> + </sizepolicy>
> + </property>
> + <property name="minimumSize">
> + <size>
> + <width>683</width>
> + <height>686</height>
> + </size>
> + </property>
> + <property name="focusPolicy">
> + <enum>Qt::NoFocus</enum>
> + </property>
> + <property name="windowTitle">
> + <string>Select media clip</string>
> + </property>
> + <property name="autoFillBackground">
> + <bool>false</bool>
> + </property>
> + <property name="inputMethodHints">
> + <set>Qt::ImhNone</set>
> + </property>
> + <widget class="QWidget" name="centralwidget">
> + <property name="sizePolicy">
> + <sizepolicy hsizetype="MinimumExpanding" vsizetype="MinimumExpanding">
> + <horstretch>0</horstretch>
> + <verstretch>0</verstretch>
> + </sizepolicy>
> + </property>
> + <layout class="QGridLayout" name="gridLayout">
> + <item row="0" column="2" colspan="2">
> + <widget class="QComboBox" name="media_path_combobox">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="sizePolicy">
> + <sizepolicy hsizetype="Preferred" vsizetype="Fixed">
> + <horstretch>0</horstretch>
> + <verstretch>0</verstretch>
> + </sizepolicy>
> + </property>
> + <property name="editable">
> + <bool>true</bool>
> + </property>
> + </widget>
> + </item>
> + <item row="7" column="2">
> + <widget class="QTimeEdit" name="start_timeedit">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="displayFormat">
> + <string>HH:mm:ss.z</string>
> + </property>
> + </widget>
> + </item>
> + <item row="8" column="2">
> + <widget class="QTimeEdit" name="end_timeedit">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="displayFormat">
> + <string>HH:mm:ss.z</string>
> + </property>
> + </widget>
> + </item>
> + <item row="7" column="3">
> + <widget class="QPushButton" name="set_start_pushbutton">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Set current position as start point</string>
> + </property>
> + </widget>
> + </item>
> + <item row="0" column="4">
> + <widget class="QPushButton" name="load_disc_pushbutton">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Load disc</string>
> + </property>
> + </widget>
> + </item>
> + <item row="9" column="3">
> + <spacer name="verticalSpacer">
> + <property name="orientation">
> + <enum>Qt::Vertical</enum>
> + </property>
> + <property name="sizeType">
> + <enum>QSizePolicy::Minimum</enum>
> + </property>
> + <property name="sizeHint" stdset="0">
> + <size>
> + <width>20</width>
> + <height>40</height>
> + </size>
> + </property>
> + </spacer>
> + </item>
> + <item row="6" column="0">
> + <widget class="QPushButton" name="play_pushbutton">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string/>
> + </property>
> + <property name="icon">
> + <iconset>
> + <normaloff>../images/media_playback_start.png</normaloff>../images/media_playback_start.png</iconset>
> + </property>
> + </widget>
> + </item>
> + <item row="8" column="0">
> + <widget class="QLabel" name="end_point_label">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>End point</string>
> + </property>
> + </widget>
> + </item>
> + <item row="4" column="2" colspan="2">
> + <widget class="QComboBox" name="subtitle_tracks_combobox">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + </widget>
> + </item>
> + <item row="2" column="0">
> + <widget class="QLabel" name="title_label">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Title</string>
> + </property>
> + </widget>
> + </item>
> + <item row="3" column="2" colspan="2">
> + <widget class="QComboBox" name="audio_tracks_combobox">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + </widget>
> + </item>
> + <item row="8" column="3">
> + <widget class="QPushButton" name="set_end_pushbutton">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Set current position as end point</string>
> + </property>
> + </widget>
> + </item>
> + <item row="10" column="3">
> + <widget class="QPushButton" name="save_pushbutton">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Save current clip</string>
> + </property>
> + </widget>
> + </item>
> + <item row="10" column="4">
> + <widget class="QPushButton" name="close_pushbutton">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Close</string>
> + </property>
> + </widget>
> + </item>
> + <item row="7" column="0" colspan="2">
> + <widget class="QLabel" name="start_point_label">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Start point</string>
> + </property>
> + </widget>
> + </item>
> + <item row="7" column="4">
> + <widget class="QPushButton" name="jump_start_pushbutton">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Jump to start point</string>
> + </property>
> + </widget>
> + </item>
> + <item row="3" column="0" colspan="2">
> + <widget class="QLabel" name="audio_track_label">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Audio track</string>
> + </property>
> + </widget>
> + </item>
> + <item row="6" column="4">
> + <widget class="QTimeEdit" name="media_position_timeedit">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="readOnly">
> + <bool>true</bool>
> + </property>
> + <property name="displayFormat">
> + <string>HH:mm:ss.z</string>
> + </property>
> + </widget>
> + </item>
> + <item row="5" column="0" colspan="5">
> + <widget class="QFrame" name="media_view_frame">
> + <property name="minimumSize">
> + <size>
> + <width>665</width>
> + <height>375</height>
> + </size>
> + </property>
> + <property name="styleSheet">
> + <string notr="true">background-color:black;</string>
> + </property>
> + <property name="frameShape">
> + <enum>QFrame::StyledPanel</enum>
> + </property>
> + <property name="frameShadow">
> + <enum>QFrame::Raised</enum>
> + </property>
> + </widget>
> + </item>
> + <item row="4" column="0" colspan="2">
> + <widget class="QLabel" name="subtitle_track_label">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Subtitle track</string>
> + </property>
> + </widget>
> + </item>
> + <item row="8" column="4">
> + <widget class="QPushButton" name="jump_end_pushbutton">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Jump to end point</string>
> + </property>
> + </widget>
> + </item>
> + <item row="0" column="0" colspan="2">
> + <widget class="QLabel" name="media_path_label">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="text">
> + <string>Media path</string>
> + </property>
> + </widget>
> + </item>
> + <item row="2" column="2" colspan="2">
> + <widget class="QComboBox" name="title_combo_box">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="currentText" stdset="0">
> + <string/>
> + </property>
> + </widget>
> + </item>
> + <item row="6" column="1" colspan="3">
> + <widget class="QSlider" name="position_horizontalslider">
> + <property name="enabled">
> + <bool>true</bool>
> + </property>
> + <property name="tracking">
> + <bool>false</bool>
> + </property>
> + <property name="orientation">
> + <enum>Qt::Horizontal</enum>
> + </property>
> + <property name="invertedAppearance">
> + <bool>false</bool>
> + </property>
> + </widget>
> + </item>
> + </layout>
> + </widget>
> + </widget>
> + <tabstops>
> + <tabstop>media_path_combobox</tabstop>
> + <tabstop>load_disc_pushbutton</tabstop>
> + <tabstop>title_combo_box</tabstop>
> + <tabstop>audio_tracks_combobox</tabstop>
> + <tabstop>subtitle_tracks_combobox</tabstop>
> + <tabstop>play_pushbutton</tabstop>
> + <tabstop>position_horizontalslider</tabstop>
> + <tabstop>media_position_timeedit</tabstop>
> + <tabstop>start_timeedit</tabstop>
> + <tabstop>set_start_pushbutton</tabstop>
> + <tabstop>jump_start_pushbutton</tabstop>
> + <tabstop>end_timeedit</tabstop>
> + <tabstop>set_end_pushbutton</tabstop>
> + <tabstop>jump_end_pushbutton</tabstop>
> + <tabstop>save_pushbutton</tabstop>
> + <tabstop>close_pushbutton</tabstop>
> + </tabstops>
> + <resources/>
> + <connections/>
> +</ui>
>
> === added file 'resources/images/media_optical.png'
> Binary files resources/images/media_optical.png 1970-01-01 00:00:00 +0000 and resources/images/media_optical.png 2014-06-16 07:07:43 +0000 differ
> === modified file 'resources/images/openlp-2.qrc'
> --- resources/images/openlp-2.qrc 2014-04-14 18:09:47 +0000
> +++ resources/images/openlp-2.qrc 2014-06-16 07:07:43 +0000
> @@ -139,6 +139,7 @@
> <file>media_stop.png</file>
> <file>media_audio.png</file>
> <file>media_video.png</file>
> + <file>media_optical.png</file>
> <file>slidecontroller_multimedia.png</file>
> <file>auto-start_active.png</file>
> <file>auto-start_inactive.png</file>
>
> === modified file 'tests/functional/openlp_core_lib/test_serviceitem.py'
> --- tests/functional/openlp_core_lib/test_serviceitem.py 2014-03-13 20:59:10 +0000
> +++ tests/functional/openlp_core_lib/test_serviceitem.py 2014-06-16 07:07:43 +0000
> @@ -206,3 +206,24 @@
> 'This service item should be able to be run in a can be made to Loop')
> self.assertTrue(service_item.is_capable(ItemCapabilities.CanAppend),
> 'This service item should be able to have new items added to it')
> +
> + def service_item_load_optical_media_from_service_test(self):
> + """
> + Test the Service Item - load an optical media item
> + """
> + # GIVEN: A new service item and a mocked add icon function
> + service_item = ServiceItem(None)
> + service_item.add_icon = MagicMock()
> +
> + # WHEN: We load a serviceitem with optical media
> + line = convert_file_service_item(TEST_PATH, 'serviceitem-dvd.osj')
> + with patch('openlp.core.ui.servicemanager.os.path.exists') as mocked_exists:
> + mocked_exists.return_value = True
> + service_item.set_from_service(line)
> +
> + # THEN: We should get back a valid service item with optical media info
> + self.assertTrue(service_item.is_valid, 'The service item should be valid')
> + self.assertTrue(service_item.is_capable(ItemCapabilities.IsOptical), 'The item should be Optical')
> + self.assertEqual(service_item.start_time, 654.375, 'Start time should be 654.375')
> + self.assertEqual(service_item.end_time, 672.069, 'End time should be 672.069')
> + self.assertEqual(service_item.media_length, 17.694, 'Media length should be 17.694')
>
> === added directory 'tests/interfaces/openlp_plugins/media'
> === added file 'tests/interfaces/openlp_plugins/media/__init__.py'
> === added directory 'tests/interfaces/openlp_plugins/media/forms'
> === added file 'tests/interfaces/openlp_plugins/media/forms/test_mediaclipselectorform.py'
> --- tests/interfaces/openlp_plugins/media/forms/test_mediaclipselectorform.py 1970-01-01 00:00:00 +0000
> +++ tests/interfaces/openlp_plugins/media/forms/test_mediaclipselectorform.py 2014-06-16 07:07:43 +0000
> @@ -0,0 +1,157 @@
> +# -*- coding: utf-8 -*-
> +# vim: autoindent shiftwidth=4 expandtab textwidth=120 tabstop=4 softtabstop=4
> +
> +###############################################################################
> +# OpenLP - Open Source Lyrics Projection #
> +# --------------------------------------------------------------------------- #
> +# Copyright (c) 2008-2014 Raoul Snyman #
> +# Portions copyright (c) 2008-2014 Tim Bentley, Gerald Britton, Jonathan #
> +# Corwin, Samuel Findlay, Michael Gorven, Scott Guerrieri, Matthias Hub, #
> +# Meinert Jordan, Armin Köhler, Erik Lundin, Edwin Lunando, Brian T. Meyer. #
> +# Joshua Miller, Stevan Pettit, Andreas Preikschat, Mattias Põldaru, #
> +# Christian Richter, Philip Ridout, Simon Scudder, Jeffrey Smith, #
> +# Maikel Stuivenberg, Martin Thompson, Jon Tibble, Dave Warnock, #
> +# Frode Woldsund, Martin Zibricky, Patrick Zimmermann #
> +# --------------------------------------------------------------------------- #
> +# This program is free software; you can redistribute it and/or modify it #
> +# under the terms of the GNU General Public License as published by the Free #
> +# Software Foundation; version 2 of the License. #
> +# #
> +# This program is distributed in the hope that it will be useful, but WITHOUT #
> +# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or #
> +# FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for #
> +# more details. #
> +# #
> +# You should have received a copy of the GNU General Public License along #
> +# with this program; if not, write to the Free Software Foundation, Inc., 59 #
> +# Temple Place, Suite 330, Boston, MA 02111-1307 USA #
> +###############################################################################
> +"""
> +Module to test the MediaClipSelectorForm.
> +"""
> +
> +import os
> +from unittest import TestCase, SkipTest
> +from openlp.core.ui.media.vlcplayer import VLC_AVAILABLE
> +
> +if os.name == 'nt' and not VLC_AVAILABLE:
> + raise SkipTest('Windows without VLC, skipping this test since it cannot run without vlc')
> +
> +from PyQt4 import QtGui, QtTest, QtCore
> +
> +from openlp.core.common import Registry
> +from openlp.plugins.media.forms.mediaclipselectorform import MediaClipSelectorForm
> +from tests.interfaces import MagicMock, patch
> +from tests.helpers.testmixin import TestMixin
> +
> +
> +class TestMediaClipSelectorForm(TestCase, TestMixin):
> + """
> + Test the EditCustomSlideForm.
> + """
> + def setUp(self):
> + """
> + Create the UI
> + """
> + Registry.create()
> + self.get_application()
> + self.main_window = QtGui.QMainWindow()
> + Registry().register('main_window', self.main_window)
> + # Mock VLC so we don't actually use it
> + self.vlc_patcher = patch('openlp.plugins.media.forms.mediaclipselectorform.vlc')
> + self.vlc_patcher.start()
> + # Mock the media item
> + self.mock_media_item = MagicMock()
> + # create form to test
> + self.form = MediaClipSelectorForm(self.mock_media_item, self.main_window, None)
> + mock_media_state_wait = MagicMock()
> + mock_media_state_wait.return_value = True
> + self.form.media_state_wait = mock_media_state_wait
> +
> + def tearDown(self):
> + """
> + Delete all the C++ objects at the end so that we don't have a segfault
> + """
> + self.vlc_patcher.stop()
> + del self.form
> + del self.main_window
> +
> + def basic_test(self):
> + """
> + Test if the dialog is correctly set up.
> + """
> + # GIVEN: A mocked QDialog.exec_() method
> + with patch('PyQt4.QtGui.QDialog.exec_') as mocked_exec:
> + # WHEN: Show the dialog.
> + self.form.exec_()
> +
> + # THEN: The media path should be empty.
> + assert self.form.media_path_combobox.currentText() == '', 'There should not be any text in the media path.'
> +
> + def click_load_button_test(self):
> + """
> + Test that the correct function is called when load is clicked, and that it behaves as expected.
> + """
> + # GIVEN: Mocked methods.
> + with patch('openlp.plugins.media.forms.mediaclipselectorform.critical_error_message_box') as \
> + mocked_critical_error_message_box,\
> + patch('openlp.plugins.media.forms.mediaclipselectorform.os.path.exists') as mocked_os_path_exists,\
> + patch('PyQt4.QtGui.QDialog.exec_') as mocked_exec:
> + self.form.exec_()
> +
> + # WHEN: The load button is clicked with no path set
> + QtTest.QTest.mouseClick(self.form.load_disc_pushbutton, QtCore.Qt.LeftButton)
> +
> + # THEN: we should get an error
> + mocked_critical_error_message_box.assert_called_with(message='No path was given')
> +
> + # WHEN: The load button is clicked with a non-existing path
> + mocked_os_path_exists.return_value = False
> + self.form.media_path_combobox.insertItem(0, '/non-existing/test-path.test')
> + self.form.media_path_combobox.setCurrentIndex(0)
> + QtTest.QTest.mouseClick(self.form.load_disc_pushbutton, QtCore.Qt.LeftButton)
> +
> + # THEN: we should get an error
> + assert self.form.media_path_combobox.currentText() == '/non-existing/test-path.test',\
> + 'The media path should be the given one.'
> + mocked_critical_error_message_box.assert_called_with(message='Given path does not exists')
> +
> + # WHEN: The load button is clicked with a mocked existing path
> + mocked_os_path_exists.return_value = True
> + self.form.vlc_media_player = MagicMock()
> + self.form.vlc_media_player.play.return_value = -1
> + self.form.media_path_combobox.insertItem(0, '/existing/test-path.test')
> + self.form.media_path_combobox.setCurrentIndex(0)
> + QtTest.QTest.mouseClick(self.form.load_disc_pushbutton, QtCore.Qt.LeftButton)
> +
> + # THEN: we should get an error
> + assert self.form.media_path_combobox.currentText() == '/existing/test-path.test',\
> + 'The media path should be the given one.'
> + mocked_critical_error_message_box.assert_called_with(message='VLC player failed playing the media')
> +
> + def title_combobox_test(self):
> + """
> + Test the behavior when the title combobox is updated
> + """
> + # GIVEN: Mocked methods and some entries in the title combobox.
> + with patch('PyQt4.QtGui.QDialog.exec_') as mocked_exec:
> + self.form.exec_()
> + self.form.vlc_media_player.get_length.return_value = 1000
> + self.form.audio_tracks_combobox.itemData = MagicMock()
> + self.form.subtitle_tracks_combobox.itemData = MagicMock()
> + self.form.audio_tracks_combobox.itemData.return_value = None
> + self.form.subtitle_tracks_combobox.itemData.return_value = None
> + self.form.title_combo_box.insertItem(0, 'Test Title 0')
> + self.form.title_combo_box.insertItem(1, 'Test Title 1')
> +
> + # WHEN: There exists audio and subtitle tracks and the index is updated.
> + self.form.vlc_media_player.audio_get_track_description.return_value = [(-1, b'Disabled'),
> + (0, b'Audio Track 1')]
> + self.form.vlc_media_player.video_get_spu_description.return_value = [(-1, b'Disabled'),
> + (0, b'Subtitle Track 1')]
> + self.form.title_combo_box.setCurrentIndex(1)
> +
> + # THEN: The subtitle and audio track comboboxes should be updated and get signals and call itemData.
> + self.form.audio_tracks_combobox.itemData.assert_any_call(0)
> + self.form.audio_tracks_combobox.itemData.assert_any_call(1)
> + self.form.subtitle_tracks_combobox.itemData.assert_any_call(0)
>
> === added file 'tests/resources/serviceitem-dvd.osj'
> --- tests/resources/serviceitem-dvd.osj 1970-01-01 00:00:00 +0000
> +++ tests/resources/serviceitem-dvd.osj 2014-06-16 07:07:43 +0000
> @@ -0,0 +1,1 @@
> +[{"serviceitem": {"header": {"auto_play_slides_once": false, "data": "", "processor": "Automatic", "theme": -1, "theme_overwritten": false, "end_time": 672.069, "start_time": 654.375, "capabilities": [12, 18, 16, 4], "media_length": 17.694, "audit": "", "xml_version": null, "title": "First DVD Clip", "auto_play_slides_loop": false, "notes": "", "icon": ":/plugins/plugin_media.png", "type": 3, "background_audio": [], "plugin": "media", "from_plugin": false, "search": "", "will_auto_start": false, "name": "media", "footer": [], "timed_slide_interval": 0}, "data": [{"image": ":/media/slidecontroller_multimedia.png", "path": "optical:1:5:3:654375:672069:First DVD Clip:/dev/sr0", "title": "/dev/sr0"}]}}]
>
--
https://code.launchpad.net/~tomasgroth/openlp/dvd/+merge/223198
Your team OpenLP Core is subscribed to branch lp:openlp.
Follow ups
References