From cb7439589bbeaba8541e2f76e03b2195605210e0 Mon Sep 17 00:00:00 2001
From: Luke Pulverenti <luke.pulverenti@gmail.com>
Date: Sun, 15 Mar 2015 00:39:29 -0400
Subject: [PATCH] sync fixes

---
 dashboard-ui/scripts/sync.js    | 20 ++++++++++++--------
 dashboard-ui/scripts/syncjob.js |  9 +++++----
 2 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/dashboard-ui/scripts/sync.js b/dashboard-ui/scripts/sync.js
index cad7b3dc71..6af4b3d40e 100644
--- a/dashboard-ui/scripts/sync.js
+++ b/dashboard-ui/scripts/sync.js
@@ -88,17 +88,21 @@
 
         html += '<div>';
         html += '<label for="selectSyncTarget">' + Globalize.translate('LabelSyncTo') + '</label>';
-        html += '<select id="selectSyncTarget" required="required" data-mini="true">';
+        if (options.readOnlySyncTarget) {
+            html += '<input type="text" id="selectSyncTarget" readonly="readonly" />';
+        } else {
+            html += '<select id="selectSyncTarget" required="required" data-mini="true">';
 
-        html += targets.map(function (t) {
+            html += targets.map(function (t) {
 
-            return '<option value="' + t.Id + '">' + t.Name + '</option>';
+                return '<option value="' + t.Id + '">' + t.Name + '</option>';
 
-        }).join('');
-        html += '</select>';
-        if (!targets.length) {
-            html += '<div class="fieldDescription">' + Globalize.translate('LabelSyncNoTargetsHelp') + '</div>';
-            html += '<div class="fieldDescription"><a href="https://github.com/MediaBrowser/Wiki/wiki/Sync" target="_blank">' + Globalize.translate('ButtonLearnMore') + '</a></div>';
+            }).join('');
+            html += '</select>';
+            if (!targets.length) {
+                html += '<div class="fieldDescription">' + Globalize.translate('LabelSyncNoTargetsHelp') + '</div>';
+                html += '<div class="fieldDescription"><a href="https://github.com/MediaBrowser/Wiki/wiki/Sync" target="_blank">' + Globalize.translate('ButtonLearnMore') + '</a></div>';
+            }
         }
         html += '</div>';
 
diff --git a/dashboard-ui/scripts/syncjob.js b/dashboard-ui/scripts/syncjob.js
index bef9113f93..ddcd087b2e 100644
--- a/dashboard-ui/scripts/syncjob.js
+++ b/dashboard-ui/scripts/syncjob.js
@@ -19,7 +19,8 @@
             elem: $('.formFields', page),
             dialogOptions: dialogOptions,
             dialogOptionsFn: getTargetDialogOptionsFn(dialogOptions),
-            showName: true
+            showName: true,
+            readOnlySyncTarget: true
         });
         fillJobValues(page, job, dialogOptions);
     }
@@ -265,7 +266,7 @@
         })[0];
         var targetName = target ? target.Name : '';
 
-        $('#txtTargetName', page).val(targetName);
+        $('#selectSyncTarget', page).val(targetName);
     }
 
     var _jobOptions;
@@ -319,10 +320,10 @@
 
         ApiClient.getJSON(ApiClient.getUrl('Sync/Jobs/' + id)).done(function (job) {
 
-            var quality = $('#selectQuality', form).val();
+            var quality = $('#selectQuality', page).val();
 
             if (quality == 'custom') {
-                quality = $('#txtBitrate', form).val();
+                quality = $('#txtBitrate', page).val();
             }
 
             job.Name = $('#txtSyncJobName', page).val();