From 8b72d1f8e9bab8bf232f9cfb45488645d45de4e2 Mon Sep 17 00:00:00 2001
From: Aron Carroll <self@aroncarroll.com>
Date: Mon, 6 Aug 2012 16:56:17 +0100
Subject: [PATCH] [#2802] Update i18n calls in modules

Updated to work with lazy loaded translations
---
 ckan/public/base/javascript/modules/api-info.js           | 4 ++--
 ckan/public/base/javascript/modules/confirm-delete.js     | 8 ++++----
 ckan/public/base/javascript/modules/slug-preview.js       | 2 ++
 .../public/base/javascript/plugins/jquery.slug-preview.js | 4 +++-
 4 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/ckan/public/base/javascript/modules/api-info.js b/ckan/public/base/javascript/modules/api-info.js
index 7a64e56fb16..939ed6bdf5e 100644
--- a/ckan/public/base/javascript/modules/api-info.js
+++ b/ckan/public/base/javascript/modules/api-info.js
@@ -17,8 +17,8 @@ this.ckan.module('api-info', function (jQuery, _) {
     options: {
       template: null,
       i18n: {
-        noTemplate: _('There is no API data to load for this resource').fetch(),
-        loadError: _('Failed to load data API information').fetch()
+        noTemplate: _('There is no API data to load for this resource'),
+        loadError: _('Failed to load data API information')
       }
     },
 
diff --git a/ckan/public/base/javascript/modules/confirm-delete.js b/ckan/public/base/javascript/modules/confirm-delete.js
index e233a0a109a..eb25a90400c 100644
--- a/ckan/public/base/javascript/modules/confirm-delete.js
+++ b/ckan/public/base/javascript/modules/confirm-delete.js
@@ -4,10 +4,10 @@ this.ckan.module('confirm-delete', function (jQuery, _) {
     options: {
       /* Locale options can be overidden with data-module-i18n attribute */
       i18n: {
-        heading: _('Please Confirm Action').fetch(),
-        content: _('Are you sure you want to delete this item?').fetch(),
-        confirm: _('Confirm').fetch(),
-        cancel: _('Cancel').fetch()
+        heading: _('Please Confirm Action'),
+        content: _('Are you sure you want to delete this item?'),
+        confirm: _('Confirm'),
+        cancel: _('Cancel')
       },
       template: [
         '<div class="modal">',
diff --git a/ckan/public/base/javascript/modules/slug-preview.js b/ckan/public/base/javascript/modules/slug-preview.js
index 8573016913d..9152d7ef5b2 100644
--- a/ckan/public/base/javascript/modules/slug-preview.js
+++ b/ckan/public/base/javascript/modules/slug-preview.js
@@ -29,6 +29,7 @@ this.ckan.module('slug-preview-slug', function (jQuery, _) {
       prefix: '',
       placeholder: '<slug>',
       i18n: {
+        url:  _('URL'),
         edit: _('Edit')
       }
     },
@@ -53,6 +54,7 @@ this.ckan.module('slug-preview-slug', function (jQuery, _) {
           prefix: options.prefix,
           placeholder: options.placeholder,
           i18n: {
+            'URL': this.i18n('url'),
             'Edit': this.i18n('edit')
           }
         });
diff --git a/ckan/public/base/javascript/plugins/jquery.slug-preview.js b/ckan/public/base/javascript/plugins/jquery.slug-preview.js
index 58975dc442c..9850cc1a98f 100644
--- a/ckan/public/base/javascript/plugins/jquery.slug-preview.js
+++ b/ckan/public/base/javascript/plugins/jquery.slug-preview.js
@@ -37,6 +37,7 @@
         value.text(val);
       }
 
+      preview.find('strong').text(options.i18n['URL'] + ':');
       preview.find('.slug-preview-prefix').text(options.prefix);
       preview.find('button').text(options.i18n['Edit']).click(function (event) {
         event.preventDefault();
@@ -61,11 +62,12 @@
     prefix: '',
     placeholder: '',
     i18n: {
+      'URL': 'URL',
       'Edit': 'Edit'
     },
     template: [
       '<div class="slug-preview">',
-      '<strong>URL:</strong>',
+      '<strong></strong>',
       '<span class="slug-preview-prefix"></span><span class="slug-preview-value"></span>',
       '<button class="btn btn-small"></button>',
       '</div>'