[Openlp-core] [Merge] lp:~trb143/openlp/more_media into lp:openlp

2019-06-14 Thread noreply
The proposal to merge lp:~trb143/openlp/more_media into lp:openlp has been 
updated.

Status: Needs review => Merged

For more details, see:
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368843
-- 
Your team OpenLP Core is subscribed to branch lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] [Bug 1832923] [NEW] SongSelect error handling

2019-06-14 Thread Johnthan
Public bug reported:

A user on the forum had an error when trying to import from the
SongSelect websiteOpenLP locked up and did NOT handle the error the
website returned

Please see https://forums.openlp.org/discussion/4346/openlp-crashes-
when-trying-to-view-import-a-song-from-songselect#latest

This was reported on the current released version of OpenLP, not the
development version.

** Affects: openlp
 Importance: Undecided
 Status: New

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

Title:
  SongSelect error handling

Status in OpenLP:
  New

Bug description:
  A user on the forum had an error when trying to import from the
  SongSelect websiteOpenLP locked up and did NOT handle the error
  the website returned

  Please see https://forums.openlp.org/discussion/4346/openlp-crashes-
  when-trying-to-view-import-a-song-from-songselect#latest

  This was reported on the current released version of OpenLP, not the
  development version.

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

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


Re: [Openlp-core] [Merge] lp:~trb143/openlp/more_media into lp:openlp

2019-06-14 Thread Phill
Review: Approve


-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368843
Your team OpenLP Core is subscribed to branch lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] macOS Test Results: Passed

2019-06-14 Thread Raoul Snyman
macOS tests passed!
-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368843
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] Linting: Passed

2019-06-14 Thread Raoul Snyman
Linting passed!
-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368843
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] Linux Test Results: Passed

2019-06-14 Thread Raoul Snyman
Linux tests passed!
-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368843
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] [Merge] lp:~trb143/openlp/more_media into lp:openlp

2019-06-14 Thread Tim Bentley
Tim Bentley has proposed merging lp:~trb143/openlp/more_media into lp:openlp.

Commit message:
Adds basic streaming service item
UI to set up streaming 
Fix issues with suffix refactor

Requested reviews:
  OpenLP Core (openlp-core)

For more details, see:
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368842
-- 
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.
=== modified file 'openlp/core/common/settings.py'
--- openlp/core/common/settings.py	2019-06-10 20:22:22 +
+++ openlp/core/common/settings.py	2019-06-14 19:30:51 +
@@ -210,6 +210,8 @@
 'media/media auto start': QtCore.Qt.Unchecked,
 'media/stream command': '',
 'media/vlc arguments': '',
+'media/video': '',
+'media/audio': '',
 'remotes/download version': '0.0',
 'players/background color': '#00',
 'servicemanager/last directory': None,

=== modified file 'openlp/core/lib/__init__.py'
--- openlp/core/lib/__init__.py	2019-05-22 06:47:00 +
+++ openlp/core/lib/__init__.py	2019-06-14 19:30:51 +
@@ -173,6 +173,7 @@
 HasNotes = 20
 HasThumbnails = 21
 HasMetaData = 22
+CanStream = 23
 
 
 def get_text_file_string(text_file_path):

=== modified file 'openlp/core/ui/media/__init__.py'
--- openlp/core/ui/media/__init__.py	2019-04-13 13:00:22 +
+++ openlp/core/ui/media/__init__.py	2019-06-14 19:30:51 +
@@ -26,6 +26,19 @@
 
 log = logging.getLogger(__name__ + '.__init__')
 
+# Audio and video extensions copied from 'include/vlc_interface.h' from vlc 2.2.0 source
+AUDIO_EXT = ['*.3ga', '*.669', '*.a52', '*.aac', '*.ac3', '*.adt', '*.adts', '*.aif', '*.aifc', '*.aiff', '*.amr',
+ '*.aob', '*.ape', '*.awb', '*.caf', '*.dts', '*.flac', '*.it', '*.kar', '*.m4a', '*.m4b', '*.m4p', '*.m5p',
+ '*.mid', '*.mka', '*.mlp', '*.mod', '*.mpa', '*.mp1', '*.mp2', '*.mp3', '*.mpc', '*.mpga', '*.mus',
+ '*.oga', '*.ogg', '*.oma', '*.opus', '*.qcp', '*.ra', '*.rmi', '*.s3m', '*.sid', '*.spx', '*.thd', '*.tta',
+ '*.voc', '*.vqf', '*.w64', '*.wav', '*.wma', '*.wv', '*.xa', '*.xm']
+VIDEO_EXT = ['*.3g2', '*.3gp', '*.3gp2', '*.3gpp', '*.amv', '*.asf', '*.avi', '*.bik', '*.divx', '*.drc', '*.dv',
+ '*.f4v', '*.flv', '*.gvi', '*.gxf', '*.iso', '*.m1v', '*.m2v', '*.m2t', '*.m2ts', '*.m4v', '*.mkv',
+ '*.mov', '*.mp2', '*.mp2v', '*.mp4', '*.mp4v', '*.mpe', '*.mpeg', '*.mpeg1', '*.mpeg2', '*.mpeg4', '*.mpg',
+ '*.mpv2', '*.mts', '*.mtv', '*.mxf', '*.mxg', '*.nsv', '*.nuv', '*.ogg', '*.ogm', '*.ogv', '*.ogx', '*.ps',
+ '*.rec', '*.rm', '*.rmvb', '*.rpl', '*.thp', '*.tod', '*.ts', '*.tts', '*.txd', '*.vob', '*.vro', '*.webm',
+ '*.wm', '*.wmv', '*.wtv', '*.xesc', '*.nut', '*.rv', '*.xvid']
+
 
 class MediaState(object):
 """

=== modified file 'openlp/core/ui/media/mediacontroller.py'
--- openlp/core/ui/media/mediacontroller.py	2019-06-05 04:53:18 +
+++ openlp/core/ui/media/mediacontroller.py	2019-06-14 19:30:51 +
@@ -42,9 +42,9 @@
 from openlp.core.lib.serviceitem import ItemCapabilities
 from openlp.core.lib.ui import critical_error_message_box
 from openlp.core.ui import DisplayControllerType
-from openlp.core.ui.media import MediaState, ItemMediaInfo, MediaType, parse_optical_path
+from openlp.core.ui.media import MediaState, ItemMediaInfo, MediaType, parse_optical_path, VIDEO_EXT, AUDIO_EXT
 from openlp.core.ui.media.endpoint import media_endpoint
-from openlp.core.ui.media.vlcplayer import AUDIO_EXT, VIDEO_EXT, VlcPlayer, get_vlc
+from openlp.core.ui.media.vlcplayer import VlcPlayer, get_vlc
 
 
 log = logging.getLogger(__name__)
@@ -184,7 +184,8 @@
 display.has_audio = False
 self.vlc_player.setup(display, preview)
 
-def set_controls_visible(self, controller, value):
+@staticmethod
+def set_controls_visible(controller, value):
 """
 After a new display is configured, all media related widget will be created too
 
@@ -229,7 +230,10 @@
 display = self._define_display(controller)
 if controller.is_live:
 # if this is an optical device use special handling
-if service_item.is_capable(ItemCapabilities.IsOptical):
+if service_item.is_capable(ItemCapabilities.CanStream):
+is_valid = self._check_file_type(controller, display, True)
+controller.media_info.media_type = MediaType.Stream
+elif 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)
@@ -249,7 +253,10 @@
 controller.media_info.start_time = service_item.start_time
 controller.media_info.end_time = service_item.end_time
 elif 

[Openlp-core] [Merge] lp:~trb143/openlp/more_media into lp:openlp

2019-06-14 Thread Tim Bentley
Tim Bentley has proposed merging lp:~trb143/openlp/more_media into lp:openlp.

Commit message:
Adds basic streaming service item
UI to set up streaming 
Fix issues with suffix refactor

Requested reviews:
  OpenLP Core (openlp-core)

For more details, see:
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368843
-- 
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.
=== modified file 'openlp/core/common/settings.py'
--- openlp/core/common/settings.py	2019-06-10 20:22:22 +
+++ openlp/core/common/settings.py	2019-06-14 19:28:31 +
@@ -210,6 +210,8 @@
 'media/media auto start': QtCore.Qt.Unchecked,
 'media/stream command': '',
 'media/vlc arguments': '',
+'media/video': '',
+'media/audio': '',
 'remotes/download version': '0.0',
 'players/background color': '#00',
 'servicemanager/last directory': None,

=== modified file 'openlp/core/lib/__init__.py'
--- openlp/core/lib/__init__.py	2019-05-22 06:47:00 +
+++ openlp/core/lib/__init__.py	2019-06-14 19:28:31 +
@@ -173,6 +173,7 @@
 HasNotes = 20
 HasThumbnails = 21
 HasMetaData = 22
+CanStream = 23
 
 
 def get_text_file_string(text_file_path):

=== modified file 'openlp/core/ui/media/__init__.py'
--- openlp/core/ui/media/__init__.py	2019-04-13 13:00:22 +
+++ openlp/core/ui/media/__init__.py	2019-06-14 19:28:31 +
@@ -26,6 +26,19 @@
 
 log = logging.getLogger(__name__ + '.__init__')
 
+# Audio and video extensions copied from 'include/vlc_interface.h' from vlc 2.2.0 source
+AUDIO_EXT = ['*.3ga', '*.669', '*.a52', '*.aac', '*.ac3', '*.adt', '*.adts', '*.aif', '*.aifc', '*.aiff', '*.amr',
+ '*.aob', '*.ape', '*.awb', '*.caf', '*.dts', '*.flac', '*.it', '*.kar', '*.m4a', '*.m4b', '*.m4p', '*.m5p',
+ '*.mid', '*.mka', '*.mlp', '*.mod', '*.mpa', '*.mp1', '*.mp2', '*.mp3', '*.mpc', '*.mpga', '*.mus',
+ '*.oga', '*.ogg', '*.oma', '*.opus', '*.qcp', '*.ra', '*.rmi', '*.s3m', '*.sid', '*.spx', '*.thd', '*.tta',
+ '*.voc', '*.vqf', '*.w64', '*.wav', '*.wma', '*.wv', '*.xa', '*.xm']
+VIDEO_EXT = ['*.3g2', '*.3gp', '*.3gp2', '*.3gpp', '*.amv', '*.asf', '*.avi', '*.bik', '*.divx', '*.drc', '*.dv',
+ '*.f4v', '*.flv', '*.gvi', '*.gxf', '*.iso', '*.m1v', '*.m2v', '*.m2t', '*.m2ts', '*.m4v', '*.mkv',
+ '*.mov', '*.mp2', '*.mp2v', '*.mp4', '*.mp4v', '*.mpe', '*.mpeg', '*.mpeg1', '*.mpeg2', '*.mpeg4', '*.mpg',
+ '*.mpv2', '*.mts', '*.mtv', '*.mxf', '*.mxg', '*.nsv', '*.nuv', '*.ogg', '*.ogm', '*.ogv', '*.ogx', '*.ps',
+ '*.rec', '*.rm', '*.rmvb', '*.rpl', '*.thp', '*.tod', '*.ts', '*.tts', '*.txd', '*.vob', '*.vro', '*.webm',
+ '*.wm', '*.wmv', '*.wtv', '*.xesc', '*.nut', '*.rv', '*.xvid']
+
 
 class MediaState(object):
 """

=== modified file 'openlp/core/ui/media/mediacontroller.py'
--- openlp/core/ui/media/mediacontroller.py	2019-06-05 04:53:18 +
+++ openlp/core/ui/media/mediacontroller.py	2019-06-14 19:28:31 +
@@ -42,9 +42,9 @@
 from openlp.core.lib.serviceitem import ItemCapabilities
 from openlp.core.lib.ui import critical_error_message_box
 from openlp.core.ui import DisplayControllerType
-from openlp.core.ui.media import MediaState, ItemMediaInfo, MediaType, parse_optical_path
+from openlp.core.ui.media import MediaState, ItemMediaInfo, MediaType, parse_optical_path, VIDEO_EXT, AUDIO_EXT
 from openlp.core.ui.media.endpoint import media_endpoint
-from openlp.core.ui.media.vlcplayer import AUDIO_EXT, VIDEO_EXT, VlcPlayer, get_vlc
+from openlp.core.ui.media.vlcplayer import VlcPlayer, get_vlc
 
 
 log = logging.getLogger(__name__)
@@ -184,7 +184,8 @@
 display.has_audio = False
 self.vlc_player.setup(display, preview)
 
-def set_controls_visible(self, controller, value):
+@staticmethod
+def set_controls_visible(controller, value):
 """
 After a new display is configured, all media related widget will be created too
 
@@ -229,7 +230,10 @@
 display = self._define_display(controller)
 if controller.is_live:
 # if this is an optical device use special handling
-if service_item.is_capable(ItemCapabilities.IsOptical):
+if service_item.is_capable(ItemCapabilities.CanStream):
+is_valid = self._check_file_type(controller, display, True)
+controller.media_info.media_type = MediaType.Stream
+elif 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)
@@ -249,7 +253,10 @@
 controller.media_info.start_time = service_item.start_time
 controller.media_info.end_time = service_item.end_time
 elif 

[Openlp-core] [Merge] lp:~trb143/openlp/more_media into lp:openlp

2019-06-14 Thread Tim Bentley
The proposal to merge lp:~trb143/openlp/more_media into lp:openlp has been 
updated.

Status: Needs review => Superseded

For more details, see:
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368842
-- 
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


Re: [Openlp-core] [Merge] lp:~trb143/openlp/more_media into lp:openlp

2019-06-14 Thread Phill
Review: Approve


-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368841
Your team OpenLP Core is subscribed to branch lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] Linting: Passed

2019-06-14 Thread Raoul Snyman
Linting passed!
-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368842
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


Re: [Openlp-core] [Merge] lp:~trb143/openlp/more_media into lp:openlp

2019-06-14 Thread Phill
Review: Needs Fixing

Just one inconsistency with text labels (see inline) and the failed linting, 
but other than that looks ok!

Diff comments:

> 
> === modified file 'openlp/core/ui/media/mediatab.py'
> --- openlp/core/ui/media/mediatab.py  2019-05-04 19:47:06 +
> +++ openlp/core/ui/media/mediatab.py  2019-06-14 18:00:17 +
> @@ -68,11 +68,15 @@
>  self.left_layout.addWidget(self.live_media_group_box)
>  self.stream_media_group_box = QtWidgets.QGroupBox(self.left_column)
>  self.stream_media_group_box.setObjectName('stream_media_group_box')
> -self.stream_media_layout = 
> QtWidgets.QHBoxLayout(self.stream_media_group_box)
> +self.stream_media_layout = 
> QtWidgets.QFormLayout(self.stream_media_group_box)
>  self.stream_media_layout.setObjectName('stream_media_layout')
>  self.stream_media_layout.setContentsMargins(0, 0, 0, 0)
> -self.stream_edit = QtWidgets.QLabel(self)
> -self.stream_media_layout.addWidget(self.stream_edit)
> +self.video_edit = QtWidgets.QLineEdit(self)
> +self.stream_media_layout.addRow(translate('MediaPlugin.MediaTab', 
> 'Video:'), self.video_edit)

colon here

> +self.audio_edit = QtWidgets.QLineEdit(self)
> +self.stream_media_layout.addRow(translate('MediaPlugin.MediaTab', 
> 'Audio'), self.audio_edit)

but not here

> +self.stream_cmd = QtWidgets.QLabel(self)
> +self.stream_media_layout.addWidget(self.stream_cmd)
>  self.left_layout.addWidget(self.stream_media_group_box)
>  self.vlc_arguments_group_box = QtWidgets.QGroupBox(self.left_column)
>  self.vlc_arguments_group_box.setObjectName('vlc_arguments_group_box')


-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368841
Your team OpenLP Core is subscribed to branch lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] Linux Test Results: Passed

2019-06-14 Thread Raoul Snyman
Linux tests passed!
-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368842
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] [Merge] lp:~trb143/openlp/more_media into lp:openlp

2019-06-14 Thread Tim Bentley
The proposal to merge lp:~trb143/openlp/more_media into lp:openlp has been 
updated.

Status: Needs review => Superseded

For more details, see:
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368841
-- 
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] Linux Test Results: Passed

2019-06-14 Thread Raoul Snyman
Linux tests passed!
-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368841
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] Linting: Failed

2019-06-14 Thread Raoul Snyman
Linting failed, please see https://ci.openlp.io/job/MP-03-Linting/119/ for more 
details
-- 
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368841
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] [Merge] lp:~trb143/openlp/more_media into lp:openlp

2019-06-14 Thread Tim Bentley
Tim Bentley has proposed merging lp:~trb143/openlp/more_media into lp:openlp.

Commit message:
Adds basic streaming service item
UI to set up streaming 
Fix issues with suffix refactor

Requested reviews:
  OpenLP Core (openlp-core)

For more details, see:
https://code.launchpad.net/~trb143/openlp/more_media/+merge/368841
-- 
Your team OpenLP Core is requested to review the proposed merge of 
lp:~trb143/openlp/more_media into lp:openlp.
=== modified file 'openlp/core/common/settings.py'
--- openlp/core/common/settings.py	2019-06-10 20:22:22 +
+++ openlp/core/common/settings.py	2019-06-14 18:00:17 +
@@ -210,6 +210,8 @@
 'media/media auto start': QtCore.Qt.Unchecked,
 'media/stream command': '',
 'media/vlc arguments': '',
+'media/video': '',
+'media/audio': '',
 'remotes/download version': '0.0',
 'players/background color': '#00',
 'servicemanager/last directory': None,

=== modified file 'openlp/core/lib/__init__.py'
--- openlp/core/lib/__init__.py	2019-05-22 06:47:00 +
+++ openlp/core/lib/__init__.py	2019-06-14 18:00:17 +
@@ -173,6 +173,7 @@
 HasNotes = 20
 HasThumbnails = 21
 HasMetaData = 22
+CanStream = 23
 
 
 def get_text_file_string(text_file_path):

=== modified file 'openlp/core/ui/media/__init__.py'
--- openlp/core/ui/media/__init__.py	2019-04-13 13:00:22 +
+++ openlp/core/ui/media/__init__.py	2019-06-14 18:00:17 +
@@ -26,6 +26,19 @@
 
 log = logging.getLogger(__name__ + '.__init__')
 
+# Audio and video extensions copied from 'include/vlc_interface.h' from vlc 2.2.0 source
+AUDIO_EXT = ['*.3ga', '*.669', '*.a52', '*.aac', '*.ac3', '*.adt', '*.adts', '*.aif', '*.aifc', '*.aiff', '*.amr',
+ '*.aob', '*.ape', '*.awb', '*.caf', '*.dts', '*.flac', '*.it', '*.kar', '*.m4a', '*.m4b', '*.m4p', '*.m5p',
+ '*.mid', '*.mka', '*.mlp', '*.mod', '*.mpa', '*.mp1', '*.mp2', '*.mp3', '*.mpc', '*.mpga', '*.mus',
+ '*.oga', '*.ogg', '*.oma', '*.opus', '*.qcp', '*.ra', '*.rmi', '*.s3m', '*.sid', '*.spx', '*.thd', '*.tta',
+ '*.voc', '*.vqf', '*.w64', '*.wav', '*.wma', '*.wv', '*.xa', '*.xm']
+VIDEO_EXT = ['*.3g2', '*.3gp', '*.3gp2', '*.3gpp', '*.amv', '*.asf', '*.avi', '*.bik', '*.divx', '*.drc', '*.dv',
+ '*.f4v', '*.flv', '*.gvi', '*.gxf', '*.iso', '*.m1v', '*.m2v', '*.m2t', '*.m2ts', '*.m4v', '*.mkv',
+ '*.mov', '*.mp2', '*.mp2v', '*.mp4', '*.mp4v', '*.mpe', '*.mpeg', '*.mpeg1', '*.mpeg2', '*.mpeg4', '*.mpg',
+ '*.mpv2', '*.mts', '*.mtv', '*.mxf', '*.mxg', '*.nsv', '*.nuv', '*.ogg', '*.ogm', '*.ogv', '*.ogx', '*.ps',
+ '*.rec', '*.rm', '*.rmvb', '*.rpl', '*.thp', '*.tod', '*.ts', '*.tts', '*.txd', '*.vob', '*.vro', '*.webm',
+ '*.wm', '*.wmv', '*.wtv', '*.xesc', '*.nut', '*.rv', '*.xvid']
+
 
 class MediaState(object):
 """

=== modified file 'openlp/core/ui/media/mediacontroller.py'
--- openlp/core/ui/media/mediacontroller.py	2019-06-05 04:53:18 +
+++ openlp/core/ui/media/mediacontroller.py	2019-06-14 18:00:17 +
@@ -42,9 +42,9 @@
 from openlp.core.lib.serviceitem import ItemCapabilities
 from openlp.core.lib.ui import critical_error_message_box
 from openlp.core.ui import DisplayControllerType
-from openlp.core.ui.media import MediaState, ItemMediaInfo, MediaType, parse_optical_path
+from openlp.core.ui.media import MediaState, ItemMediaInfo, MediaType, parse_optical_path, VIDEO_EXT, AUDIO_EXT
 from openlp.core.ui.media.endpoint import media_endpoint
-from openlp.core.ui.media.vlcplayer import AUDIO_EXT, VIDEO_EXT, VlcPlayer, get_vlc
+from openlp.core.ui.media.vlcplayer import VlcPlayer, get_vlc
 
 
 log = logging.getLogger(__name__)
@@ -184,7 +184,8 @@
 display.has_audio = False
 self.vlc_player.setup(display, preview)
 
-def set_controls_visible(self, controller, value):
+@staticmethod
+def set_controls_visible(controller, value):
 """
 After a new display is configured, all media related widget will be created too
 
@@ -229,7 +230,10 @@
 display = self._define_display(controller)
 if controller.is_live:
 # if this is an optical device use special handling
-if service_item.is_capable(ItemCapabilities.IsOptical):
+if service_item.is_capable(ItemCapabilities.CanStream):
+is_valid = self._check_file_type(controller, display, True)
+controller.media_info.media_type = MediaType.Stream
+elif 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)
@@ -249,7 +253,10 @@
 controller.media_info.start_time = service_item.start_time
 controller.media_info.end_time = service_item.end_time
 elif 

[Openlp-core] [Bug 1832874] Re: Temporary songs appear in the 'Delete Duplicate songs' wizard

2019-06-14 Thread Tim Bentley
Code to fix this was removed from the Songs plugin in 2.2 but was there
in 2.0

** Changed in: openlp
   Status: New => Confirmed

** Changed in: openlp
   Importance: Undecided => High

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

Title:
  Temporary songs appear in the 'Delete Duplicate songs' wizard

Status in OpenLP:
  Confirmed

Bug description:
  Windows 10
  OpenLP 2.4.6

  When you run the Delete Duplicate songs wizard most songs marked in
  the db as Temporary will show up and the user has no idea that they
  are temporary

  It would be best if the songs were removed from the db when they are
  no longer needed, but at the very least they should not show up in the
  wizard.

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

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] [Bug 1832876] [NEW] Please add songbook to the 'Delete Duplicate Songs' Wizard

2019-06-14 Thread Johnthan
Public bug reported:

Windows 10
OpenLP 2.4.6

When trying to delete duplicate songs using the wizard, please add the
songbook and theme to the information displayed.  It helps to know that
there are three duplicates of a song, but each is in a different
songbook or that they have different themes.

Most musicians add bits to the song that make them special for just
them!

** Affects: openlp
 Importance: Undecided
 Status: New

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

Title:
  Please add songbook to the 'Delete Duplicate Songs' Wizard

Status in OpenLP:
  New

Bug description:
  Windows 10
  OpenLP 2.4.6

  When trying to delete duplicate songs using the wizard, please add the
  songbook and theme to the information displayed.  It helps to know
  that there are three duplicates of a song, but each is in a different
  songbook or that they have different themes.

  Most musicians add bits to the song that make them special for just
  them!

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

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp


[Openlp-core] [Bug 1832874] [NEW] Temporary songs appear in the 'Delete Duplicate songs' wizard

2019-06-14 Thread Johnthan
Public bug reported:

Windows 10
OpenLP 2.4.6

When you run the Delete Duplicate songs wizard most songs marked in the
db as Temporary will show up and the user has no idea that they are
temporary

It would be best if the songs were removed from the db when they are no
longer needed, but at the very least they should not show up in the
wizard.

** Affects: openlp
 Importance: Undecided
 Status: New

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

Title:
  Temporary songs appear in the 'Delete Duplicate songs' wizard

Status in OpenLP:
  New

Bug description:
  Windows 10
  OpenLP 2.4.6

  When you run the Delete Duplicate songs wizard most songs marked in
  the db as Temporary will show up and the user has no idea that they
  are temporary

  It would be best if the songs were removed from the db when they are
  no longer needed, but at the very least they should not show up in the
  wizard.

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

___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp