Merge pull request #1314 from croneter/fix-streams

Fix PKC not auto-picking audio/subtitle stream when transcoding

resources/language/resource.language.en_gb/strings.po

@@ -630,7 +630,7 @@ msgstr ""
 
 # PKC Settings - Playback
 msgctxt "#30541"
-msgid "Don't ask to pick a certain stream/quality"
+msgid "Transcoding: Auto-pick audio and subtitle stream using Plex defaults"
 msgstr ""
 
 # PKC Settings - Playback

resources/lib/playback_decision.py

@@ -349,8 +349,14 @@ def audio_subtitle_prefs(api, item):
                          action_type='PUT',
                          parameters=args)
         return True
+    return setup_transcoding_audio_subtitle_prefs(mediastreams, part_id)
+
+
+def setup_transcoding_audio_subtitle_prefs(mediastreams, part_id):
     audio_streams_list = []
     audio_streams = []
+    audio_default = None
+    subtitle_default = None
     subtitle_streams_list = []
     # "Don't burn-in any subtitle"
     subtitle_streams = ['1 %s' % utils.lang(39706)]
@@ -379,6 +385,8 @@ def audio_subtitle_prefs(api, item):
                                            utils.lang(39707),  # unknown
                                            codec,
                                            channellayout)
+            if stream.get('default'):
+                audio_default = audio_numb
             audio_streams_list.append(index)
             audio_streams.append(track.encode('utf-8'))
             audio_numb += 1
@@ -391,7 +399,6 @@ def audio_subtitle_prefs(api, item):
                 continue
             # Subtitle is available within the video file
             # Burn in the subtitle, if user chooses to do so
-            default = stream.get('default')
             forced = stream.get('forced')
             try:
                 track = '{} {}'.format(sub_num + 1,
@@ -400,7 +407,8 @@ def audio_subtitle_prefs(api, item):
                 track = '{} {} ({})'.format(sub_num + 1,
                                             utils.lang(39707),  # unknown
                                             stream.get('codec'))
-            if default:
+            if stream.get('default'):
+                subtitle_default = sub_num
                 track = "%s - %s" % (track, utils.lang(39708))  # Default
             if forced:
                 track = "%s - %s" % (track, utils.lang(39709))  # Forced
@@ -410,6 +418,10 @@ def audio_subtitle_prefs(api, item):
             sub_num += 1
 
     if audio_numb > 1:
+        # "Transcoding: Auto-pick audio and subtitle stream using Plex defaults"
+        if utils.settings('bestQuality') == 'true' and audio_default is not None:
+            resp = audio_default
+        else:
             resp = utils.dialog('select', utils.lang(33013), audio_streams)
             if resp == -1:
                 LOG.info('User aborted dialog to select audio stream')
@@ -427,6 +439,10 @@ def audio_subtitle_prefs(api, item):
         # Note: we DO need to tell the PMS that we DONT want any sub
         # Otherwise, the PMS might pick-up the last one
         LOG.info('No subtitles to burn-in')
+    else:
+        # "Transcoding: Auto-pick audio and subtitle stream using Plex defaults"
+        if utils.settings('bestQuality') == 'true' and subtitle_default is not None:
+            resp = subtitle_default
         else:
             resp = utils.dialog('select', utils.lang(33014), subtitle_streams)
             if resp == -1:

resources/settings.xml

@@ -120,7 +120,7 @@
         <setting id="audioBoost" type="slider" label="39001" default="0" range="0,10,100" option="int"/>
         <setting id="subtitleSize" label="39002" type="slider" option="int" range="0,30,300" default="100" />
         <setting id="force_transcode_pix" type="bool" label="30545" default="false" />
-        <setting id="bestQuality" type="bool" label="30541" default="false" />
+        <setting id="bestQuality" type="bool" label="30541" default="false" /><!-- Transcoding: Auto-pick audio and subtitle stream using Plex defaults -->
         <setting id="bestTrailer" type="bool" label="30542" default="true" />
         <setting type="sep" />
         <setting id="offerDelete" type="bool" label="30114" default="false" visible="false"/>