From 03a7643b75265c5f0b03831dd58d228fde29590c Mon Sep 17 00:00:00 2001
From: Mark McDowall <markus.mcd5@gmail.com>
Date: Mon, 27 Feb 2012 21:52:03 -0800
Subject: [PATCH] SignalR will now update quality in UI as well (when
 applicable)

---
 NzbDrone.Core/Providers/DiskScanProvider.cs |   2 +-
 NzbDrone.Core/Providers/DownloadProvider.cs |   2 +-
 NzbDrone.Core/Providers/SignalRProvider.cs  |  21 +++++++++++++++++---
 NzbDrone.Web/Content/Images/Missing.png     | Bin 5089 -> 760 bytes
 NzbDrone.Web/Scripts/NzbDrone/grid.js       |  19 ++++++++++++------
 NzbDrone.Web/Views/Series/Episode.cshtml    |   2 +-
 6 files changed, 34 insertions(+), 12 deletions(-)

diff --git a/NzbDrone.Core/Providers/DiskScanProvider.cs b/NzbDrone.Core/Providers/DiskScanProvider.cs
index 9720171c8..fc024f8ea 100644
--- a/NzbDrone.Core/Providers/DiskScanProvider.cs
+++ b/NzbDrone.Core/Providers/DiskScanProvider.cs
@@ -202,7 +202,7 @@ namespace NzbDrone.Core.Providers
                 _externalNotificationProvider.OnDownload(message, series);
                 
                 foreach(var episode in episodes)
-                    _signalRProvider.UpdateEpisodeStatus(episode.EpisodeId, EpisodeStatusType.Ready);
+                    _signalRProvider.UpdateEpisodeStatus(episode.EpisodeId, EpisodeStatusType.Ready, parseResult.Quality);
             }
             else
             {
diff --git a/NzbDrone.Core/Providers/DownloadProvider.cs b/NzbDrone.Core/Providers/DownloadProvider.cs
index 426288867..f9473c0bc 100644
--- a/NzbDrone.Core/Providers/DownloadProvider.cs
+++ b/NzbDrone.Core/Providers/DownloadProvider.cs
@@ -67,7 +67,7 @@ namespace NzbDrone.Core.Providers
                     _historyProvider.Add(history);
                     _episodeProvider.MarkEpisodeAsFetched(episode.EpisodeId);
 
-                    _signalRProvider.UpdateEpisodeStatus(episode.EpisodeId, EpisodeStatusType.Downloading);
+                    _signalRProvider.UpdateEpisodeStatus(episode.EpisodeId, EpisodeStatusType.Downloading, null);
                 }
 
                 _externalNotificationProvider.OnGrab(downloadTitle);
diff --git a/NzbDrone.Core/Providers/SignalRProvider.cs b/NzbDrone.Core/Providers/SignalRProvider.cs
index aac95512a..b815d1466 100644
--- a/NzbDrone.Core/Providers/SignalRProvider.cs
+++ b/NzbDrone.Core/Providers/SignalRProvider.cs
@@ -4,6 +4,7 @@ using System.Linq;
 using System.Text;
 using NLog;
 using NzbDrone.Core.Model;
+using NzbDrone.Core.Repository.Quality;
 using SignalR;
 using SignalR.Hosting.AspNet;
 using SignalR.Hubs;
@@ -15,11 +16,25 @@ namespace NzbDrone.Core.Providers
     {
         private static readonly Logger logger = LogManager.GetCurrentClassLogger();
 
-        public virtual void UpdateEpisodeStatus(int episodeId, EpisodeStatusType episodeStatus)
+        public virtual void UpdateEpisodeStatus(int episodeId, EpisodeStatusType episodeStatus, Quality quality)
         {
-            logger.Trace("Sending Status update to client. EpisodeId: {0}, Status: {1}", episodeId, episodeStatus);
+            try
+            {
+                logger.Trace("Sending Status update to client. EpisodeId: {0}, Status: {1}", episodeId, episodeStatus);
 
-            GetClients().updatedStatus(episodeId, episodeStatus.ToString());
+                GetClients().updatedStatus(new
+                                               {
+                                                       EpisodeId = episodeId,
+                                                       EpisodeStatus = episodeStatus.ToString(),
+                                                       Quality = (quality == null ? String.Empty : quality.QualityType.ToString())
+                                               });
+                var test = 0;
+            }
+            catch (Exception ex)
+            {
+                logger.Trace("Error");
+                throw;
+            }
         }
 
         private dynamic GetClients()
diff --git a/NzbDrone.Web/Content/Images/Missing.png b/NzbDrone.Web/Content/Images/Missing.png
index 15c3101d0692eb5f47367fc48c247ff7a6cee1d9..2bbc78831b77dbb4442a30d0d0ed1bc0e3746810 100644
GIT binary patch
delta 660
zcmaE;{)2Ub#pH=xHuX$-o-U3d6}OW9{Qqx%IDn0T@yHQ_Ba99Vyp{Y6EEgtxXFy?Y
z3^=Rec1%MdlkbR^gAVf%AqO3fskQt^^c#L~3fyOYG@0lBfBnU6TY<)S1R9<?v4Txy
za^nrA4&BBXQ$6PHZ~S2)FrVSkPKWb%l&*6E71c9*Vu+m4xKOa8pYdZo*GIz*ktw^F
z6&g+y?$8xj&+;ffq`-#ff77f??^V?dOdTP|-I+exX<5n0zGuGe*fbj~B)@dw<HR2t
zn)4Vq6oMw5wX?F2vsCNw@?&6hS;HQ7>EcI&J(DE1GqMP5TzYmr<A($KNnYv=3`t6h
z=9H_}FRWRvmcV+Ow~*P7LBOG9me9!yTv-bmRIl(_00TkrQ^Qe(jCT)8nH?A&djAe&
zp7L$u(^id`cPtzVHHHax4w>@NaR$d#IRzYk*eo#Vh&pR=mkGpHFz&EvxU9>-*m9m(
zsQCmlOxOY{Z1QCH6p&;+^LzG*)`h!R6&gNNJ~*oIiNS9>1B-wizm24e+<}*34Gf2#
zH&!DIC=jx$GPv--{EBV_$a$~0Wn@@#7dHR6!vS=B*G1uG1-1{n7!?{gjy6b5Jyxe7
zX&iL;xf{s053DbA+<eD)ihKXTGGU3W%p408@2_O9=TJzREfm~PVK3`kZ-2$Fi9w;E
z?au)*pxO3&{{8=7#OCHE@LFBK`jWwu^yarpLZCDue}@&Q+kOw9fP-^g6EL<UR+?S3
zbqMwWMySP=^jC~b87Im$L1Z<^xIOF)$V?uF-M_n-59RkV00Ae1r>mdKI;Vst0FrtB
Ah5!Hn

delta 5040
zcmV;h6Hn~;1>q-<IDZOFX+uL$P-t&-Z*ypGa3D!TLm+T+Z)Rz1WdHzp+MQE(Sd;e_
zKHv9c4^~3h@UfR{fdC>StO&>uS)ve<0AYj>5<ta;hzeE|6%m|BRm6hT3P_!Z;-DzF
zXF-aHT8mX&IPM?Zr?q{0@9pjT*ZaKZoag+`dCw1k5fUbm=6|AU069{HJUZAPk55R%
z$-RIA6-eL&AQ0xu!e<4=008g<d3b(wus{3(uWtYX0C3eVBofEr|AV?vCRYF;kpSQ#
z66Xs6kWv81E>y@A0LT~suv4>S3ILP<0Bm`DLLvaF4FK%)Nj?Pt*r}7;7Xa9z9H|HZ
zjR63eC`Tj$K!3XcU{!%qECRs70HCZuA}$2Lt^t5qwlYTofV~9(c8*w(4?ti5fSE!p
z%m5%b0suoE6U_r4Oaq`W(!b!TUvP!ENC5!A%azTSOVTqGxRuZvck=My;vwR~Y_URN
z7by^C3FIQ2mzyIKNaq7g&I|wm8u`(|{y0C7=jP<$=6`1AZ142NqW){}Zz4V+@!$Tu
zi~<Az5KhnU+F#}6)&p?I0kCzyYpGiR+ExH)-}zlL+X<jr3ebA_i+MQwKpvi<P{=$T
z9105y?ImKNy)fg8KL0b}Z{Ul5_WVE|{@Z%+0CAc?nXSO1gZ*(~Zg#Fxj_1n+LNRXl
z=Og~dg@1qjjw#XNG_hPP6^ij#iMUWAO~-k;QjtU<$(7;~>3=fuAC~<IlXndOtcFj(
zPPPZN%WS}Y?g0Xe4p`E506>28EsPoqkpK{9G%|Vj005J}`Hw&=0RYXHq~ibpyyzHQ
zsFW8>#s~laM4*8xut5h5!4#~(4xGUqyucR%VSffhKnx_nJP<+#<bWIsp%|7y8C1YJ
z*aWq(0~(+an&A+%!7(@u=im}tf$MM=24EPT!Wg`U2?RmN2oqr;I*1Wsj@Tm32p5@-
z1R`NbG?IX%AnAw{Q6k02a-;&OLTZs+NF(wsauhj@TtNDe+sGg?iu{VaM=_LvvQY!n
z0)KTtxu_o+ipHRG(R4HqU4WLNYtb6C9zB4zqNmYI=yh}eeTt4_fYC7yW{lZkT#ScB
zV2M~7CdU?I<ybXVk2PZ*ST}YR8^E4n?+7FUi+~gC2wsE`!fb+&kVjZdSVO2K>?5=i
zx(HVZgM=}{CnA%mPqZa^68Xe<Vmh&qSbs*WChj2~A)X~(Ck_)|lSm{E$&%zw3Lzzs
zGD!<bYe?HjEu>SVKGG0roJ=O`kZsA{<Os5WtRR<>w~!BzPm=q|!{oOVI>m_MObMbS
zQlyj;N;PFa<pialGD4Z4s!`3Up414ch&rEINo}BZP%l%5sS`AHnkCJf7DLOVEq|lc
z(3)vyY4>O^>2$gY-Gd%Qm(Z8eYv>2*=jns=cMJ`N4THx>VkjAF8G9M07`GWOnM|ey
z)0dgZR4~^v8<}UA514ONSSt1^d=-((5|uiYR+WC0=c-gyb5%dpd8!Lkt5pxHURHgk
zMpd&=<EaVM7OT~%9aX!jHm=T6cYjikP|s0cqrPALg8E|(vWA65poU1JRAaZs8I2(p
z#xiB`SVGovRs-uSYnV-9TeA7=Om+qP8+I>yOjAR1s%ETak!GFdam@h^#<Ae=IoX^_
z&LPeX&U-BbEk7->)@rS0t$wXH+Irf)+G6c;?H29p+V6F6oj{!|o%K3xI)C?d3A#4A
zQM!e?+jY<CKGS3CdFcuD%JmNE-O)$&ZS<q{7wYfU@6jJOFf<4@kQr<-IAie4kYng;
zm}$7t@Py&05zA=0k;G`D(Mh8xxF+t0XX7<^7d~dJZyaK*G~Q+0Ydm3MX)@cS#H7XK
zzA4R=Yno=d(X`Wa%*@Cv+<&attjX+-In|tNo?*V(yxaVZg{4KD#Y&4di^rDQmVC<v
zmQ9xTte93lR(V$SR@baa)?Di>>uuIoY)~6ln+%&eo6EMSt(&dHcAIVA6yg+*DbgwR
zQ*PQZ?ELHs?3(Nb?K$>g_9gah_Rk&691<PYJ9ImIa&&dfc5HCG>wl!-G{dRHsl(}4
zXB%gsbDi@w7p6;)%MzD%mlsoQr;4X;pL)xc<r?m~(zVm|qno>%+^yMd)ZNTI#eJ*$
zO)i@o$z8)e??LqN_gLa_%;TM>o2SC_kmoO6c3xRt`@J4dvz#WL)-Y|z+r(Soy~}%G
zIzByR`p)SCKE^%*pMQFvhrXu1BHul}BYxI?nSKZSp8Grc%l(h|zu|fE7V%C6U;)7a
z<pI5c8iBI|YXctynFOT=H3f|Yy9O@|J{3X?2@P2va+7bs7xEkVV>8@mESk|3$_Skm
zS{wQ>%qC18))9_|&j{ZTes8AvOzF(F2#DZEY>2oYX&IRp`G0*BDJn9mF6vRVQ*?23
z_bk?|<XQV?y^isZsf@Wh+iJFQc4w?=Y*K7v?3=iNxT?5;c!&5!@s|>G6C?@kiR8rC
z#65}Qa{}jVnlqf_npBo_W3J`gqPZ95>CVfZcRX1&S&)1<g_shTvOnd6AVN?t7*FM=
zZcQB%@`Rg(Pk%+eqK%?avA4KNJd);}R+aWR-8+3l`e=qvMs>zB2~Schd65~Cxg+yU
zRz%j`tk2nT*)2JgoRplSQVnUAv@6#zw<xzyW-BX`4di*|ZOR*yhsyWlBl&al+ZAku
zOwpsXQm#-A7ECXwD|lNNTX?ugy+~TrGv8+Z>iHuJf`1l#y^yd_xUjR>xOiFd;3B_8
zyA~shQx|tGF<nx+WO!-t(tXQl%d(dJP~uopRq|?i-13f6gVLp?Lo0$;G?l58$;)o8
z^jf)N6=9WRRqtw-)mv77Di@VstZ=NTuK2J<xaQ(lPG8l0^?7aD+TKdH%I)h&>!j;$
ztoK>JuYXFtYC+Y|hVTuo8}W_h8((co-gKdQYW0rIw9U%R12tha?OV*YtlRRTHly}>
zoqt`+R{gE3x4zjX+Sb3_cYE^=gB=w+-tUy`ytONMS8KgRef4hA?t<Nq8e$u|zvh13
zxJP$S#h#CQrF#eVMeplsbZ>0jufM;t32jm~jej0UI&kHifN$EH?V5MBa9S!3!a?Q1
zC*P)gd*e{(q0YnH!_D8Bf4B7r>qvPk(mKC&tSzH$pgp0z@92!9ogH2sN4~fJe(y2k
zV|B+hk5`_cohUu=`Q(C=<ELb&o}ErVJ=B@pdG}2GnQL89UA<>R&z?UQbnZ;IU-!xL
z-hbzhUGThc^dk3S+a<S4tv#+iM}Bbq;YhDrZ|i0E%k5V@u5|QG>pRi!(|`JEz}0it
z_}4C7pLxCS#_SunZYJFvxFx#v_;&W~7k3KoOx#_1k9e>AzS{lj2l@}{f3*IwWx#FV
z_+Y?b&%;>{?+yuvp8k~o(}&^GN6bgnBY#FCjgOrl9~%uCz4Bzvli{bbrxVZ0epdf^
z>vOB;-~HnIOV3#R*zgPai_gEVd8zYq@2jb=I>#f&AH2?aJ@Kaet<Kwh@3`-}-^aZl
znJD}~{!sJL`eVnZ&`<Y1=Y0PBFZYD=`z6w!GXMYp1ZP1_K>z@;j(q!3lK=n!AdyNc
ze+d3bL_t(|+U%NHY+ThD$A9O}=9$<Y$99y&iL;Op$UqVZ327mvEU8*mh(1)J;sL2I
z^r>n^+Xp%i?L#Z~p;A?KQ6G?yDj}pwMMXe&P^*Zg2(=+<7?y0rA;$K2yW_EEyXSm;
zxcAQ7J9B4Zk^n;Lk?iBS%bEZG`~Kf|f4*~$m5A_ZZWf>N_NgZjzuEyNa@qDvyMaNk
z&jNMezg~Z;{o#DU{$vRN+kmeEUj&W~Zx|w#?o&gfSxNP!(2izqdR}SmFneiEfd2r`
z0e^@h^kJL2(jcD5Wgh|_%?=M9+_V3B#&&H}qvKmqRt%H^D+VYKL5u@N5TmhYf9K@V
zne&`2oK?lCxpTnJfhY0>yYjIRn8;=C0)BPVZ8zp_xNRTnw~i?*o<hYFsCW`nD<P(i
z)Jlj`LzIQpHw-F<m0FK64pOfp^(v+LdCs0VC2zcXavFFnU$CG2$Oug2vN7PN+pi!0
z_TIz$*m_eAE15=g6~}%Lr}Vz)f5l79=iOn&?FG*hU=55C&yK73&`zw>Ai7#%e)58x
ze&bEvKJnIHfrs-2d*PZAn8;;^fu{~0z4h3w$L><Gb%W^A1?sb>L@&>EJTD?3K8Y51
z5nL$=h!|9AfYg?oReX34rdH+6<1fp<kH3BfxIbU8&s-A%6S?ee;Q7ydf8ogPefQm?
zOua<y;!9#ymLi`Ife!)ph=@US;Ijdh>?3vE5tSI+!Rc3D=9Ry_v;=%DU$9@eDgqO^
z>=xjayT9`Jt$XgcliKVF8XvqF{epnBNe0jAA|452n%7#y1kqqbNQ{h=*?pIqdgmQp
z{QaL+fqU}>`}Hd#Fp<lqe}R{d-v7B<_8z^1>c!(Bjfbrst#^EY#+8g17r5^s(e69O
zzorG18D!wVm(-=gyZrOnzt00l@&)_+mCVFHyZzon2ln1^h|1|d_69Fqz?)9+K47B-
zBjSQL;*v3fUbb0!<!RZtV>3tYJ30b9HId6EKNNw9T=vfKU0c7gfB)FsYW4JA#8gT>
z^d11OTfr*E*So8Ed@^ml3qE{Z*BaQb{aLo(d;|Lq?LQ1Wx@H1iy8O+7dk#^%@S;d<
zC0g(8!K!_o2f>Gf)z^CytRoohdBpgh0gOSkL+M}7%XJ6$E8zQHJg&(D5AHvb%Z&`I
zie8@Y8LTaEfq*oEe;6OUv@GujtN*!4IM8e3f;XO=s#R+5ye7AQ{<8zXch^ebdwUM=
zp)vDzq~3+K>0JWAjdnHe25U?2p&xBp;|Oky7--DTuwf*Pf=9ghs5^m)T=u~D&P})U
z)y{R-y9t92Sy2Ld_pTdHGNxVc!TZ`J?Muiw;rLj7>xAsuf4e8`6^%VT@Zi|^D7C2~
zx}p(Bq9-Tq1M$HIf>i=R8UPyv-=X*R=e=>Y7AmU_l~R?#m;rd`as=)jjF&~Upeve&
z-Jrf$$0<ALnn4^Bo`6I!CJ?OcdY9;6^}z=99+Z`K4AqKerR?B1?ltB=066T;Po3+5
ziCnfHI5?Q7e}*R~4LE{`Zq_y`rcjmw3%8fvlT%Y>KKVAiix!L{)Efq;E@*8+(sWA_
zhvD@@nT5rr!@%*V1<kI}4FhqdT*V-Q1X89ykQn#NTIGSb^*>&(TF)s<g$Q847=w1)
zF?GYWw)V#}gnK_Nd-0c$h5$EoBd~j2pOW@UQj9c9e;Zdmf#40630|!u63#%)Ok~<p
zX0N`DgfH7zqYkjE+mf;$(*Q}gRgW#OM1l|ZW-TBCqzLpKBz(_#7%VY(>+DDZ>6jI+
z9%%y_{=7!eUG5pvO)62}>Hcl`hVDY3ZcL}%+pXbyCTKkgrp_MLHzwL^K@vzHzS>k^
zTv=DTe-S7<n%0Y5^tcusn<$W%(RtJ2{(Ue0S`{xCWdXX}jlg8Jp~6<^rS}Lu_w+<B
zBSh%EDIZ#1Xov-Hwi|(WSE{C2kTnNtvzS?nT%x4Yme->^(E&cVPy#BA2EdufnK)mt
z7b+{&LTN<@^cuFHF%jUSelGKl>E@X>@Dc#8e-bImj%eWZUZuqGQY{s-oH4DHQ#9bV
z`o^>>Nz*kBe6L6#X-FgyzcmsmfRpW8F6{)Ko?dVeu{!k|(FZ|Gq<11FH4k`eg%*@k
zV6hYgnM|>?QUQ48a>d;7srQSMjbt|T<xPu5ftF5E((Qi7WEGH1AbGU6M0S|z<%$NL
zf9y#hU$8as%Zsyhi1#(YMh0nz?prc5&Ch}N10LWqdJnc_c9{868Q__G!Jb^pitO>X
z3zNlaY6P%)2dRX;5+#+ECxaSq(>r0k`z@C0!|F@(UU5+aKU&k0GGDOEz+;o|6-kbc
zBdE?G4FqeX&f6xZ*8MJej{tAQ6J)mRf8bJa9^mnO!9KO-ZIyh%{_VN*)BiYsagpTc
zR)kp3VD1|1p#^_fKHc;VU?oyyw(OudTjYbK@+9zsE7{n3xKOxIoSv(Y+PDK1ON3+;
zOs}R}nb1Y=&Ap_k6PHXs>&ADgvc1HasTqB_-RiCc@&$Vuc;JnbXDbVfWm21Ve<~}r
z?wU?c9RWKuLS}5cTCo>pYI+9XoB4wM^40E6e;Ih{rW?2QkBx0qjpZe&&dp$)M%R^7
zR3#ae--CHRy<v=4W}RA?nU<;P8A^_PCtt9Cajnh=ZU&y+ecjl$?c25}(VBW`N$QJ5
zbiL8Ba*_bpUf^S?K9cK4Nem1re?*v`nv!$V#WL{qe8GO^TA$Et0)Dq`^XQ$MH*X-5
z$tV%fwW>7AB^t{%x?1g7jQJXmXZlI>526xD1yC>9a;ex1*ay00oU1+~^LB|J0e-k+
z%f{icO{2t<DHYt6(6y?FG3Z(i5#yeXB@!qrhLudI;CQ;SB2%+-T%231e*nJ*ev&WP
zAL`uoicTpfa@nl6SNW}xfpq_d;UTiwK@zF7YHkKJe}$W~T9&2KG7C#KORH5J;R)c!
zQM2qvddwOiatwG7IF^p7?7B=J7K%)|52Y-`Xe!kjMue40l~pZu;05oj_&G1zef;Od
z5hwLA-~ez-;P04s?=5dFCR+eb0_AIUQm_91-zuMQ``-XsEk2Bq*Rovz0000<MNUMn
GLSTZBiQNAH

diff --git a/NzbDrone.Web/Scripts/NzbDrone/grid.js b/NzbDrone.Web/Scripts/NzbDrone/grid.js
index af1219627..e2fc1f040 100644
--- a/NzbDrone.Web/Scripts/NzbDrone/grid.js
+++ b/NzbDrone.Web/Scripts/NzbDrone/grid.js
@@ -63,9 +63,9 @@ $(function () {
     var signalRProvider = $.connection.signalRProvider;
 
     // Declare a function on the chat hub so the server can invoke it
-    signalRProvider.updatedStatus = function (episodeId, episodeStatus) {
-        var imageSrc = '../../Content/Images/' + episodeStatus + '.png';
-        var row = $('tr.episodeId_' + episodeId);
+    signalRProvider.updatedStatus = function (data) {
+        var imageSrc = '../../Content/Images/' + data.EpisodeStatus + '.png';
+        var row = $('tr.episodeId_' + data.EpisodeId);
 
         if (row.length == 0)
             return;
@@ -75,13 +75,20 @@ $(function () {
         if (statusImage.length == 0)
             return;
 
-        statusImage.attr('alt', episodeStatus);
-        statusImage.attr('title', episodeStatus);
+        statusImage.attr('alt', data.EpisodeStatus);
+        statusImage.attr('title', data.EpisodeStatus);
         statusImage.attr('src', imageSrc);
 
-        if (episodeStatus != "Missing") {
+        if (data.EpisodeStatus != "Missing") {
             statusImage.parent('td').removeClass('episodeMissing');
         }
+
+        if (data.Quality != null) {
+            var qualityColumn = $(row).find('.episodeQuality');
+            
+            if (qualityColumn)
+                qualityColumn.text(data.Quality);
+        }
     };
 
     // Start the connection
diff --git a/NzbDrone.Web/Views/Series/Episode.cshtml b/NzbDrone.Web/Views/Series/Episode.cshtml
index e9b5c9e32..56c4575ac 100644
--- a/NzbDrone.Web/Views/Series/Episode.cshtml
+++ b/NzbDrone.Web/Views/Series/Episode.cshtml
@@ -6,7 +6,7 @@
     <td>@Model.EpisodeNumber</td>
     <td>@Model.Title</td>
     <td>@Model.AirDate</td>
-    <td>@Model.Quality</td>
+    <td class="episodeQuality">@Model.Quality</td>
     
     @{
         string cellColourClass = String.Empty;