From 91aea923d29c7fd97436f7cc9b9559cf07ca46c3 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Gr=C3=A9goire=20Delattre?= <gregoire.delattre@gmail.com>
Date: Fri, 2 Jun 2017 15:02:47 +0200
Subject: [PATCH] Use the immutable subtitles in the player

---
 src/public/js/components/buttons/download.js |  8 ++++----
 src/public/js/components/list/details.js     |  3 ++-
 src/public/js/components/shows/details.js    | 12 +++---------
 3 files changed, 9 insertions(+), 14 deletions(-)

diff --git a/src/public/js/components/buttons/download.js b/src/public/js/components/buttons/download.js
index 5f7a58b..3729b88 100644
--- a/src/public/js/components/buttons/download.js
+++ b/src/public/js/components/buttons/download.js
@@ -76,14 +76,14 @@ class Player extends React.Component {
       <div className="embed-responsive embed-responsive-16by9">
         <video controls>
           <source src={this.props.url} type="video/mp4"/>
-          {this.props.subtitles !== undefined && subtitles.toKeyedSeq().map(function(el, index) {
+          {this.props.subtitles !== undefined && this.props.subtitles.toIndexedSeq().map(function(el, index) {
             return (
               <track
                 key={index}
                 kind="subtitles"
-                label={el.language}
-                src={el.vvt_file}
-                srcLang={el.language}
+                label={el.get('language')}
+                src={el.get('vvt_file')}
+                srcLang={el.get('language')}
               />
             );
           })}
diff --git a/src/public/js/components/list/details.js b/src/public/js/components/list/details.js
index 4388296..6286ece 100644
--- a/src/public/js/components/list/details.js
+++ b/src/public/js/components/list/details.js
@@ -16,7 +16,8 @@ export default function ListDetails(props) {
 
   const trackedSeason = props.data.get('tracked_season');
   const trackedEpisode = props.data.get('tracked_episode');
-  if (trackedEpisode !== null && trackedSeason !== null) {
+  if (trackedEpisode !== null && trackedSeason !== null
+    && trackedEpisode !== undefined && trackedSeason !== undefined) {
     if ((trackedSeason === 0) && (trackedEpisode === 0)) {
       wishlistStr = "Whole show tracked";
     } else {
diff --git a/src/public/js/components/shows/details.js b/src/public/js/components/shows/details.js
index 46931a3..0580985 100644
--- a/src/public/js/components/shows/details.js
+++ b/src/public/js/components/shows/details.js
@@ -1,7 +1,7 @@
 import React from 'react'
 import { connect } from 'react-redux'
 import { bindActionCreators } from 'redux'
-import { toJS } from 'immutable'
+import { List } from 'immutable'
 import { addTorrent } from '../../actions/torrents'
 import { refreshSubtitles } from '../../actions/subtitles'
 import { addShowToWishlist, deleteShowFromWishlist, getEpisodeDetails, updateShowDetails } from '../../actions/shows'
@@ -177,12 +177,6 @@ class Season extends React.Component {
 }
 
 function Episode(props) {
-  // TODO: remove this when everything uses immutable
-  let subtitles;
-  if (props.data.has('subtitles') && props.data.get('subtitles')) {
-    subtitles = props.data.get('subtitles').toJS();
-  }
-
   return (
     <tr>
       <th scope="row" className="col-xs-2">
@@ -198,7 +192,7 @@ function Episode(props) {
         <span className="pull-right episode-buttons">
           <SubtitlesButton
             url={props.data.get('polochon_url')}
-            subtitles={subtitles}
+            subtitles={props.data.get('subtitles')}
             refreshSubtitles={props.refreshSubtitles}
             resourceID={props.data.get('show_imdb_id')}
             season={props.data.get('season')}
@@ -218,7 +212,7 @@ function Episode(props) {
           })}
           <DownloadButton
             url={props.data.get('polochon_url')}
-            subtitles={subtitles}
+            subtitles={props.data.get('subtitles')}
             xs
           />
           <GetDetailsButton