Bridge Rails values into JavaScript through a JSON element

4 files changed, 36 insertions, 24 deletions

diff --git a/app/helpers/admin_helper.rb b/app/helpers/admin_helper.rb
index 5a1a42a3..be20eb8b 100644
--- a/app/helpers/admin_helper.rb
+++ b/app/helpers/admin_helper.rb
@@ -14,4 +14,28 @@ module AdminHelper
                 :role => "img", :class => classes,
                 :title => hint, "aria-label" => hint)
   end
+
+  def admin_bridge
+    {
+      :urls => {
+        :search           => admin_search_path,
+        :menu_search      => admin_menu_search_path,
+        :dashboard_search => admin_dashboard_search_path,
+        :parameterize_preview => parameterize_preview_nodes_path
+      },
+      :strings => {
+        :lock_lost_prefix => t("layouts.admin.lock_lost_prefix"),
+        :lock_lost_link   => t("layouts.admin.lock_lost_link"),
+        :insert_image     => t("layouts.admin.insert_image"),
+        :insert_image_tooltip => t("layouts.admin.insert_image_tooltip"),
+        :copied           => t("layouts.admin.copied"),
+        :needs_redaktion  => t("layouts.admin.needs_redaktion"),
+        :import_markdown  => t("layouts.admin.import_markdown"),
+        :import_markdown_tooltip => t("layouts.admin.import_markdown_tooltip"),
+        :import_markdown_placeholder => t("layouts.admin.import_markdown_placeholder"),
+        :cancel           => t("admin.common.cancel"),
+        :insert           => t("layouts.admin.insert")
+      }
+    }.to_json
+  end
 end
diff --git a/app/views/layouts/admin.html.erb b/app/views/layouts/admin.html.erb
index 082f172c..3f6667a8 100644
--- a/app/views/layouts/admin.html.erb
+++ b/app/views/layouts/admin.html.erb
@@ -15,27 +15,8 @@
     <%= javascript_include_tag 'admin_bundle' %>
     <%= tinymce_assets %>
     <link rel="stylesheet" href="<%= mtime_busted_path('/stylesheets/admin.css') %>">
-    <%= javascript_tag nonce: true do %>
-      var ADMIN_SEARCH_URL = "<%= admin_search_path %>";
-      var ADMIN_MENU_SEARCH_URL = "<%= admin_menu_search_path %>";
-      var ADMIN_STRINGS = <%= raw({
-        :lock_lost_prefix => t(".lock_lost_prefix"),
-        :lock_lost_link => t(".lock_lost_link"),
-        :insert_image => t(".insert_image"),
-        :insert_image_tooltip => t(".insert_image_tooltip"),
-        :copied => t(".copied"),
-        :needs_redaktion => t(".needs_redaktion"),
-        :import_markdown => t(".import_markdown"),
-        :import_markdown_tooltip => t(".import_markdown_tooltip"),
-        :import_markdown_placeholder => t(".import_markdown_placeholder"),
-        :cancel => t("admin.common.cancel"),
-        :insert => t(".insert")
-      }.to_json) %>;
-      var PARAMETERIZE_PREVIEW_URL = "<%= parameterize_preview_nodes_path %>";
-      var DASHBOARD_SEARCH_URL = "<%= admin_dashboard_search_path %>";
-    <% end %>
-    <script src="<%= mtime_busted_path('/javascripts/admin_search.js') %>"></script>
     <script src="<%= mtime_busted_path('/javascripts/admin_interface.js') %>"></script>
+    <script src="<%= mtime_busted_path('/javascripts/admin_search.js') %>"></script>
     <script src="<%= mtime_busted_path('/javascripts/related_assets.js') %>"></script>
     <%= yield :head %>
   </head>
@@ -62,5 +43,6 @@
         <%= yield %>
       </div>
     </div>
+    <script type="application/json" id="cccms_config"><%= raw admin_bridge %></script>
   </body>
 </html>
diff --git a/public/javascripts/admin_interface.js b/public/javascripts/admin_interface.js
index 9b28eb06..abe434bf 100644
--- a/public/javascripts/admin_interface.js
+++ b/public/javascripts/admin_interface.js
@@ -1,4 +1,10 @@
+var CCCMS_CONFIG;
+var ADMIN_STRINGS;
+
 $(document).ready(function () {
+  CCCMS_CONFIG = JSON.parse(document.getElementById('cccms_config').textContent);
+  ADMIN_STRINGS = CCCMS_CONFIG.strings;
+
   admin_search.initialize();
 
   function effective_scheme() {
diff --git a/public/javascripts/admin_search.js b/public/javascripts/admin_search.js
index 852f5339..8c86a74f 100644
--- a/public/javascripts/admin_search.js
+++ b/public/javascripts/admin_search.js
@@ -22,7 +22,7 @@ admin_search = {
     initSearchPicker({
       inputSelector: "#search_term",
       resultsSelector: "#menu_search_results",
-      url: ADMIN_SEARCH_URL,
+      url: CCCMS_CONFIG.urls.search,
       isActive: function() { return $('#search_widget').is(':visible'); },
       resultsHeaderHtml: "<p class='search_more'>Press Enter to see all results ⏎</p>"
     });
@@ -41,7 +41,7 @@ admin_search = {
 function initSearchPicker(options) {
   var inputSelector = options.inputSelector;
   var resultsSelector = options.resultsSelector;
-  var url = options.url || ADMIN_MENU_SEARCH_URL;
+  var url = options.url || CCCMS_CONFIG.urls.menu_search;
   var onSelect = options.onSelect;
   var isActive = options.isActive;
   var resultsHeaderHtml = options.resultsHeaderHtml;
@@ -138,7 +138,7 @@ dashboard_search = {
     initSearchPicker({
       inputSelector: "#dashboard_search_term",
       resultsSelector: "#dashboard_search_results",
-      url: DASHBOARD_SEARCH_URL,
+      url: CCCMS_CONFIG.urls.dashboard_search,
       resultsHeaderHtml: "<p class='search_more'>Press Enter to see all results ⏎</p>",
       renderResults: function(data, results, resultsHeaderHtml) {
         var found = false;
@@ -225,7 +225,7 @@ parent_search = {
 
     clearTimeout(parent_search.path_timeout);
     parent_search.path_timeout = setTimeout(function() {
-        $.get(PARAMETERIZE_PREVIEW_URL, { title: title }, function(slug) {
+        $.get(CCCMS_CONFIG.urls.parameterize_preview, { title: title }, function(slug) {
         $("#resulting_path").text(window.location.origin + "/" + (prefix ? prefix + "/" : "") + slug);
       });
     }, 300);