From 0ae4ba2a1cc3d44f57210215a805694bcc67c481 Mon Sep 17 00:00:00 2001
From: Roger <nevafuse@gmail.com>
Date: Mon, 18 Feb 2019 06:56:57 -0500
Subject: [PATCH] [bravotv] fix drupal settings parser (closes rg3#19214)

---
 youtube_dl/extractor/bravotv.py | 42 ++++++++++++++++++++-------------
 1 file changed, 26 insertions(+), 16 deletions(-)

diff --git a/youtube_dl/extractor/bravotv.py b/youtube_dl/extractor/bravotv.py
index a25d500e4..413fc0ed4 100644
--- a/youtube_dl/extractor/bravotv.py
+++ b/youtube_dl/extractor/bravotv.py
@@ -12,50 +12,62 @@ from ..utils import (
 class BravoTVIE(AdobePassIE):
     _VALID_URL = r'https?://(?:www\.)?bravotv\.com/(?:[^/]+/)+(?P<id>[^/?#]+)'
     _TESTS = [{
-        'url': 'http://www.bravotv.com/last-chance-kitchen/season-5/videos/lck-ep-12-fishy-finale',
-        'md5': '9086d0b7ef0ea2aabc4781d75f4e5863',
+        'url': 'https://www.bravotv.com/the-real-housewives-of-beverly-hills/season-9/episode-1/videos/lisa-rinna-thinks-denise-richards',
+        'md5': 'f8098a7c034fcb1b52ec8b3631803589',
         'info_dict': {
-            'id': 'zHyk1_HU_mPy',
+            'id': 'PPhoit2pxdh0',
             'ext': 'mp4',
-            'title': 'LCK Ep 12: Fishy Finale',
-            'description': 'S13/E12: Two eliminated chefs have just 12 minutes to cook up a delicious fish dish.',
+            'title': 'Lisa Rinna Thinks Denise Richards Will Fit in Well With This Group',
+            'description': 'Denise goes back a long way with Rinna, and now the other ladies have a chance to meet her.',
             'uploader': 'NBCU-BRAV',
-            'upload_date': '20160302',
-            'timestamp': 1456945320,
+            'upload_date': '20190204',
+            'timestamp': 1549302120,
         }
     }, {
         'url': 'http://www.bravotv.com/below-deck/season-3/ep-14-reunion-part-1',
         'only_matching': True,
+    }, {
+        'url': 'https://www.bravotv.com/top-chef/season-16/episode-11/videos/lck-ep-12-the-final-knockout',
+        'only_matching': True,
     }]
 
     def _real_extract(self, url):
         display_id = self._match_id(url)
         webpage = self._download_webpage(url, display_id)
         settings = self._parse_json(self._search_regex(
-            r'jQuery\.extend\(Drupal\.settings\s*,\s*({.+?})\);', webpage, 'drupal settings'),
+            r'<script[^>]+?data-drupal-selector="drupal-settings-json"[^>]*?>({.+?})</script>', webpage, 'drupal settings'),
             display_id)
         info = {}
         query = {
             'mbr': 'true',
         }
         account_pid, release_pid = [None] * 2
-        tve = settings.get('sharedTVE')
+        tve = settings.get('ls_tve')
         if tve:
             query['manifest'] = 'm3u'
             account_pid = 'HNK2IC'
             release_pid = tve['release_pid']
             if tve.get('entitlement') == 'auth':
-                adobe_pass = settings.get('adobePass', {})
+                adobe_pass = settings.get('tve_adobe_auth', {})
                 resource = self._get_mvpd_resource(
                     adobe_pass.get('adobePassResourceId', 'bravo'),
                     tve['title'], release_pid, tve.get('rating'))
                 query['auth'] = self._extract_mvpd_auth(
                     url, release_pid, adobe_pass.get('adobePassRequestorId', 'bravo'), resource)
+            url = smuggle_url(update_url_query(
+                'http://link.theplatform.com/s/%s/media/%s' % (account_pid, release_pid),
+                query), {'force_smil_url': True})
         else:
-            shared_playlist = settings['shared_playlist']
-            account_pid = shared_playlist['account_pid']
-            metadata = shared_playlist['video_metadata'][shared_playlist['default_clip']]
+            ls_playlist = settings['ls_playlist']
+            account_pid = ls_playlist['account_pid']
+            metadata = ls_playlist['video_metadata'][ls_playlist['default_clip']]
             release_pid = metadata['release_pid']
+            if release_pid:
+                url = smuggle_url(update_url_query(
+                    'http://link.theplatform.com/s/%s/media/%s' % (account_pid, release_pid),
+                    query), {'force_smil_url': True})
+            else:
+                url = ls_playlist['player_base_url']
             info.update({
                 'title': metadata['title'],
                 'description': metadata.get('description'),
@@ -66,9 +78,7 @@ class BravoTVIE(AdobePassIE):
         info.update({
             '_type': 'url_transparent',
             'id': release_pid,
-            'url': smuggle_url(update_url_query(
-                'http://link.theplatform.com/s/%s/%s' % (account_pid, release_pid),
-                query), {'force_smil_url': True}),
+            'url': url,
             'ie_key': 'ThePlatform',
         })
         return info