From bb29548d193b56c73e84ff0c7888d721aa3e2902 Mon Sep 17 00:00:00 2001
From: webchick <webchick@24967.no-reply.drupal.org>
Date: Sun, 22 Sep 2013 01:39:42 +0200
Subject: [PATCH] Issue #1793648 by nod_, kmox83, droplet, rteijeiro:
Follow-up: replace all occurence of Drupal.settings with drupalSettings.
---
core/includes/ajax.inc | 8 ++++----
core/includes/common.inc | 8 ++++----
core/includes/form.inc | 2 +-
core/lib/Drupal/Core/Ajax/AjaxResponse.php | 2 +-
core/lib/Drupal/Core/Ajax/SettingsCommand.php | 8 ++++----
core/misc/ajax.js | 20 +++++++++----------
core/misc/drupal.js | 7 ++-----
core/misc/tabledrag.js | 8 ++++----
core/modules/locale/locale.datepicker.js | 8 ++++----
core/modules/node/node.js | 6 +++---
core/modules/overlay/overlay-parent.js | 18 ++++++++---------
.../lib/Drupal/simpletest/WebTestBase.php | 12 +++++------
core/modules/statistics/statistics.js | 8 ++++----
.../system/Tests/Ajax/FrameworkTest.php | 2 +-
.../system/Tests/Common/JavaScriptTest.php | 6 +++---
core/modules/views/js/ajax_view.js | 10 +++++-----
core/modules/views/js/base.js | 6 +++---
.../views/Controller/ViewAjaxController.php | 2 +-
core/modules/views/views.theme.inc | 2 +-
core/modules/views_ui/js/ajax.js | 14 ++++++-------
core/modules/views_ui/js/views-admin.js | 2 +-
core/themes/bartik/color/preview.js | 8 ++++----
22 files changed, 82 insertions(+), 85 deletions(-)
diff --git a/core/includes/ajax.inc b/core/includes/ajax.inc
index bb777988df67..8157f079093f 100644
--- a/core/includes/ajax.inc
+++ b/core/includes/ajax.inc
@@ -289,7 +289,7 @@ function ajax_render($commands = array()) {
$commands = array_merge($extra_commands, $commands);
}
- // Now add a command to merge changes and additions to Drupal.settings.
+ // Now add a command to merge changes and additions to drupalSettings.
$scripts = drupal_add_js();
if (!empty($scripts['settings'])) {
$settings = drupal_merge_js_settings($scripts['settings']['data']);
@@ -808,7 +808,7 @@ function ajax_command_css($selector, $argument) {
* Creates a Drupal Ajax 'settings' command.
*
* The 'settings' command instructs the client either to use the given array as
- * the settings for ajax-loaded content or to extend Drupal.settings with the
+ * the settings for ajax-loaded content or to extend drupalSettings with the
* given array, depending on the value of the $merge parameter.
*
* This command is implemented by Drupal.AjaxCommands.prototype.settings()
@@ -820,9 +820,9 @@ function ajax_command_css($selector, $argument) {
* array.
* @param $merge
* Whether or not the passed settings in $argument should be merged into the
- * global Drupal.settings on the page. By default (FALSE), the settings that
+ * global drupalSettings on the page. By default (FALSE), the settings that
* are passed to Drupal.attachBehaviors will not include the global
- * Drupal.settings.
+ * drupalSettings.
*
* @return
* An array suitable for use with the ajax_render() function.
diff --git a/core/includes/common.inc b/core/includes/common.inc
index be97724479fe..1c5dd17092b9 100644
--- a/core/includes/common.inc
+++ b/core/includes/common.inc
@@ -2063,7 +2063,7 @@ function drupal_html_id($id) {
* on.
* - Add settings ('setting'): Adds settings to Drupal's global storage of
* JavaScript settings. Per-page settings are required by some modules to
- * function properly. All settings will be accessible at Drupal.settings.
+ * function properly. All settings will be accessible at drupalSettings.
*
* Examples:
* @code
@@ -2108,9 +2108,9 @@ function drupal_html_id($id) {
* hosted on the local server. These files will not be aggregated if
* JavaScript aggregation is enabled.
* - 'setting': An associative array with configuration options. The array is
- * merged directly into Drupal.settings. All modules should wrap their
+ * merged directly into drupalSettings. All modules should wrap their
* actual configuration settings in another variable to prevent conflicts in
- * the Drupal.settings namespace. Items added with a string key will replace
+ * the drupalSettings namespace. Items added with a string key will replace
* existing settings with that key; items with numeric array keys will be
* added to the existing settings array.
* @param $options
@@ -2371,7 +2371,7 @@ function drupal_get_js($scope = 'header', $javascript = NULL, $skip_alter = FALS
// considered "used" and prevented from being added in a later AJAX
// request.
// Skip if no files were added to the page otherwise jQuery.extend() will
- // overwrite the Drupal.settings.ajaxPageState.css object with an empty
+ // overwrite the drupalSettings.ajaxPageState.css object with an empty
// array.
$css = drupal_add_css();
if (!empty($css)) {
diff --git a/core/includes/form.inc b/core/includes/form.inc
index 306fcbf3839f..05b6d29d6b89 100644
--- a/core/includes/form.inc
+++ b/core/includes/form.inc
@@ -4118,7 +4118,7 @@ function form_process_autocomplete($element, &$form_state) {
$element['#attached']['library'][] = array('system', 'drupal.autocomplete');
// Provide a hidden element for the JavaScript behavior to bind to. Since
// this element is for client-side functionality only, do not process input.
- // @todo Refactor autocomplete.js to accept Drupal.settings instead of
+ // @todo Refactor autocomplete.js to accept drupalSettings instead of
// requiring extraneous markup.
$element['autocomplete'] = array(
'#type' => 'hidden',
diff --git a/core/lib/Drupal/Core/Ajax/AjaxResponse.php b/core/lib/Drupal/Core/Ajax/AjaxResponse.php
index 7660f77876b8..3e4d8a30c998 100644
--- a/core/lib/Drupal/Core/Ajax/AjaxResponse.php
+++ b/core/lib/Drupal/Core/Ajax/AjaxResponse.php
@@ -129,7 +129,7 @@ protected function ajaxRender(Request $request) {
$this->addCommand($resource_command, TRUE);
}
- // Prepend a command to merge changes and additions to Drupal.settings.
+ // Prepend a command to merge changes and additions to drupalSettings.
$scripts = drupal_add_js();
if (!empty($scripts['settings'])) {
$settings = drupal_merge_js_settings($scripts['settings']['data']);
diff --git a/core/lib/Drupal/Core/Ajax/SettingsCommand.php b/core/lib/Drupal/Core/Ajax/SettingsCommand.php
index d31fdbf3d47a..723f05b12e11 100644
--- a/core/lib/Drupal/Core/Ajax/SettingsCommand.php
+++ b/core/lib/Drupal/Core/Ajax/SettingsCommand.php
@@ -13,7 +13,7 @@
* AJAX command for adjusting Drupal's JavaScript settings.
*
* The 'settings' command instructs the client either to use the given array as
- * the settings for ajax-loaded content or to extend Drupal.settings with the
+ * the settings for ajax-loaded content or to extend drupalSettings with the
* given array, depending on the value of the $merge parameter.
*
* This command is implemented by Drupal.AjaxCommands.prototype.settings()
@@ -32,10 +32,10 @@ class SettingsCommand implements CommandInterface {
protected $settings;
/**
- * Whether the settings should be merged into the global Drupal.settings.
+ * Whether the settings should be merged into the global drupalSettings.
*
* By default (FALSE), the settings that are passed to Drupal.attachBehaviors
- * will not include the global Drupal.settings.
+ * will not include the global drupalSettings.
*
* @var boolean
*/
@@ -47,7 +47,7 @@ class SettingsCommand implements CommandInterface {
* @param array $settings
* An array of key/value pairs of JavaScript settings.
* @param boolean $merge
- * Whether the settings should be merged into the global Drupal.settings.
+ * Whether the settings should be merged into the global drupalSettings.
*/
public function __construct(array $settings, $merge = FALSE) {
$this->settings = $settings;
diff --git a/core/misc/ajax.js b/core/misc/ajax.js
index bdff14be61de..a586945fddea 100644
--- a/core/misc/ajax.js
+++ b/core/misc/ajax.js
@@ -1,4 +1,4 @@
-(function ($, window) {
+(function ($, window, Drupal, drupalSettings) {
"use strict";
@@ -366,7 +366,7 @@ Drupal.ajax.prototype.beforeSerialize = function (element, options) {
// $.ajax(). When there is no form and $.ajax() is used, beforeSerialize()
// isn't called, but don't rely on that: explicitly check this.form.
if (this.form) {
- var settings = this.settings || Drupal.settings;
+ var settings = this.settings || drupalSettings;
Drupal.detachBehaviors(this.form, settings, 'serialize');
}
@@ -385,7 +385,7 @@ Drupal.ajax.prototype.beforeSerialize = function (element, options) {
// @see ajax_base_page_theme()
// @see drupal_get_css()
// @see drupal_get_js()
- var pageState = Drupal.settings.ajaxPageState;
+ var pageState = drupalSettings.ajaxPageState;
options.data['ajax_page_state[theme]'] = pageState.theme;
options.data['ajax_page_state[theme_token]'] = pageState.theme_token;
for (var cssFile in pageState.css) {
@@ -492,7 +492,7 @@ Drupal.ajax.prototype.success = function (response, status) {
// commands is not sufficient, because behaviors from the entire form need
// to be reattached.
if (this.form) {
- var settings = this.settings || Drupal.settings;
+ var settings = this.settings || drupalSettings;
Drupal.attachBehaviors(this.form, settings);
}
@@ -545,7 +545,7 @@ Drupal.ajax.prototype.error = function (response, uri) {
$(this.element).removeClass('progress-disabled').prop('disabled', false);
// Reattach behaviors, if they were detached in beforeSerialize().
if (this.form) {
- var settings = response.settings || this.settings || Drupal.settings;
+ var settings = response.settings || this.settings || drupalSettings;
Drupal.attachBehaviors(this.form, settings);
}
throw new Drupal.AjaxError(response, uri);
@@ -596,7 +596,7 @@ Drupal.AjaxCommands.prototype = {
case 'replaceAll':
case 'empty':
case 'remove':
- settings = response.settings || ajax.settings || Drupal.settings;
+ settings = response.settings || ajax.settings || drupalSettings;
Drupal.detachBehaviors(wrapper, settings);
}
@@ -624,7 +624,7 @@ Drupal.AjaxCommands.prototype = {
// optional.
if (new_content.parents('html').length > 0) {
// Apply any settings from the returned JSON if available.
- settings = response.settings || ajax.settings || Drupal.settings;
+ settings = response.settings || ajax.settings || drupalSettings;
Drupal.attachBehaviors(new_content, settings);
}
},
@@ -633,7 +633,7 @@ Drupal.AjaxCommands.prototype = {
* Command to remove a chunk from the page.
*/
remove: function (ajax, response, status) {
- var settings = response.settings || ajax.settings || Drupal.settings;
+ var settings = response.settings || ajax.settings || drupalSettings;
Drupal.detachBehaviors($(response.selector), settings);
$(response.selector).remove();
},
@@ -676,7 +676,7 @@ Drupal.AjaxCommands.prototype = {
*/
settings: function (ajax, response, status) {
if (response.merge) {
- $.extend(true, Drupal.settings, response.settings);
+ $.extend(true, drupalSettings, response.settings);
}
else {
ajax.settings = response.settings;
@@ -733,4 +733,4 @@ Drupal.AjaxCommands.prototype = {
}
};
-})(jQuery, this);
+})(jQuery, this, Drupal, drupalSettings);
diff --git a/core/misc/drupal.js b/core/misc/drupal.js
index 9c86a0a33470..6c04130376db 100644
--- a/core/misc/drupal.js
+++ b/core/misc/drupal.js
@@ -14,9 +14,6 @@ if (window.jQuery) {
"use strict";
-// Populate Drupal.settings with the drupalSettings variable.
-Drupal.settings = drupalSettings;
-
/**
* Custom error type thrown after attach/detach if one or more behaviors failed.
*
@@ -75,7 +72,7 @@ DrupalBehaviorError.prototype = new Error();
* is used.
* @param settings
* An object containing settings for the current context. If none is given,
- * the global Drupal.settings object is used.
+ * the global drupalSettings object is used.
*/
Drupal.attachBehaviors = function (context, settings) {
context = context || document;
@@ -120,7 +117,7 @@ domready(function () { Drupal.attachBehaviors(document, drupalSettings); });
* is used.
* @param settings
* An object containing settings for the current context. If none given, the
- * global Drupal.settings object is used.
+ * global drupalSettings object is used.
* @param trigger
* A string containing what's causing the behaviors to be detached. The
* possible triggers are:
diff --git a/core/misc/tabledrag.js b/core/misc/tabledrag.js
index 525422443d3a..57e48a202e49 100644
--- a/core/misc/tabledrag.js
+++ b/core/misc/tabledrag.js
@@ -1,4 +1,4 @@
-(function ($) {
+(function ($, Drupal, drupalSettings) {
"use strict";
@@ -1069,9 +1069,9 @@ Drupal.tableDrag.prototype.row.prototype.isValidSwap = function (row) {
* DOM element what will be swapped with the row group.
*/
Drupal.tableDrag.prototype.row.prototype.swap = function (position, row) {
- Drupal.detachBehaviors(this.group, Drupal.settings, 'move');
+ Drupal.detachBehaviors(this.group, drupalSettings, 'move');
$(row)[position](this.group);
- Drupal.attachBehaviors(this.group, Drupal.settings);
+ Drupal.attachBehaviors(this.group, drupalSettings);
this.changed = true;
this.onSwap(row);
};
@@ -1257,4 +1257,4 @@ $.extend(Drupal.theme, {
}
});
-})(jQuery);
+})(jQuery, Drupal, drupalSettings);
diff --git a/core/modules/locale/locale.datepicker.js b/core/modules/locale/locale.datepicker.js
index 55edf37d05e7..b23e57da4816 100644
--- a/core/modules/locale/locale.datepicker.js
+++ b/core/modules/locale/locale.datepicker.js
@@ -3,7 +3,7 @@
* Datepicker JavaScript for the Locale module.
*/
-(function ($) {
+(function ($, Drupal, drupalSettings) {
"use strict";
@@ -12,7 +12,7 @@
*/
Drupal.behaviors.localeDatepicker = {
attach: function(context, settings) {
- // This code accesses Drupal.settings and localized strings via Drupal.t().
+ // This code accesses drupalSettings and localized strings via Drupal.t().
// So this code should run after these are initialized. By placing it in an
// attach behavior this is assured.
$.datepicker.regional['drupal-locale'] = $.extend({
@@ -78,9 +78,9 @@ Drupal.behaviors.localeDatepicker = {
dateFormat: Drupal.t('mm/dd/yy'),
firstDay: 0,
isRTL: 0
- }, Drupal.settings.jquery.ui.datepicker);
+ }, drupalSettings.jquery.ui.datepicker);
$.datepicker.setDefaults($.datepicker.regional['drupal-locale']);
}
};
-})(jQuery);
+})(jQuery, Drupal, drupalSettings);
diff --git a/core/modules/node/node.js b/core/modules/node/node.js
index b61c1175f9da..f0b5a38f37e1 100644
--- a/core/modules/node/node.js
+++ b/core/modules/node/node.js
@@ -3,7 +3,7 @@
* Defines Javascript behaviors for the node module.
*/
-(function ($) {
+(function ($, Drupal, drupalSettings) {
"use strict";
@@ -27,7 +27,7 @@ Drupal.behaviors.nodeDetailsSummaries = {
$context.find('.node-form-author').drupalSetSummary(function (context) {
var $context = $(context);
- var name = $context.find('.form-item-name input').val() || Drupal.settings.anonymous,
+ var name = $context.find('.form-item-name input').val() || drupalSettings.anonymous,
date = $context.find('.form-item-date input').val();
return date ?
Drupal.t('By @name on @date', { '@name': name, '@date': date }) :
@@ -67,4 +67,4 @@ Drupal.behaviors.nodeDetailsSummaries = {
}
};
-})(jQuery);
+})(jQuery, Drupal, drupalSettings);
diff --git a/core/modules/overlay/overlay-parent.js b/core/modules/overlay/overlay-parent.js
index 4f452992655d..ae1d5b8f468d 100644
--- a/core/modules/overlay/overlay-parent.js
+++ b/core/modules/overlay/overlay-parent.js
@@ -3,7 +3,7 @@
* Attaches the behaviors for the Overlay parent pages.
*/
-(function ($, Drupal, displace) {
+(function ($, Drupal, displace, drupalSettings) {
"use strict";
@@ -372,13 +372,13 @@ Drupal.overlay.isAdminLink = function (url) {
// Turn the list of administrative paths into a regular expression.
if (!this.adminPathRegExp) {
var prefix = '';
- if (Drupal.settings.overlay.pathPrefixes.length) {
+ if (drupalSettings.overlay.pathPrefixes.length) {
// Allow path prefixes used for language negatiation followed by slash,
// and the empty string.
- prefix = '(' + Drupal.settings.overlay.pathPrefixes.join('/|') + '/|)';
+ prefix = '(' + drupalSettings.overlay.pathPrefixes.join('/|') + '/|)';
}
- var adminPaths = '^' + prefix + '(' + Drupal.settings.overlay.paths.admin.replace(/\s+/g, '|') + ')$';
- var nonAdminPaths = '^' + prefix + '(' + Drupal.settings.overlay.paths.non_admin.replace(/\s+/g, '|') + ')$';
+ var adminPaths = '^' + prefix + '(' + drupalSettings.overlay.paths.admin.replace(/\s+/g, '|') + ')$';
+ var nonAdminPaths = '^' + prefix + '(' + drupalSettings.overlay.paths.non_admin.replace(/\s+/g, '|') + ')$';
adminPaths = adminPaths.replace(/\*/g, '.*');
nonAdminPaths = nonAdminPaths.replace(/\*/g, '.*');
this.adminPathRegExp = new RegExp(adminPaths);
@@ -750,9 +750,9 @@ Drupal.overlay.fragmentizeLink = function (link, parentLocation) {
function refreshRegion(regionName, regionSelector) {
var $region = $(regionSelector);
Drupal.detachBehaviors($region);
- $.get(Drupal.url(Drupal.settings.overlay.ajaxCallback + '/' + regionName), function (newElement) {
+ $.get(Drupal.url(drupalSettings.overlay.ajaxCallback + '/' + regionName), function (newElement) {
$region.replaceWith($(newElement));
- Drupal.attachBehaviors($region, Drupal.settings);
+ Drupal.attachBehaviors($region, drupalSettings);
});
}
@@ -827,7 +827,7 @@ Drupal.overlay.getPath = function (link) {
if (path.charAt(0) !== '/') {
path = '/' + path;
}
- path = path.replace(new RegExp(Drupal.settings.basePath + Drupal.settings.scriptPath), '');
+ path = path.replace(new RegExp(drupalSettings.basePath + drupalSettings.scriptPath), '');
return path;
};
@@ -875,4 +875,4 @@ $.extend(Drupal.theme, {
}
});
-})(jQuery, Drupal, Drupal.displace);
+})(jQuery, Drupal, Drupal.displace, drupalSettings);
diff --git a/core/modules/simpletest/lib/Drupal/simpletest/WebTestBase.php b/core/modules/simpletest/lib/Drupal/simpletest/WebTestBase.php
index 24a010a415f1..42f217dec731 100644
--- a/core/modules/simpletest/lib/Drupal/simpletest/WebTestBase.php
+++ b/core/modules/simpletest/lib/Drupal/simpletest/WebTestBase.php
@@ -78,9 +78,9 @@ abstract class WebTestBase extends TestBase {
protected $plainTextContent;
/**
- * The value of Drupal.settings for the currently-loaded page.
+ * The value of drupalSettings for the currently-loaded page.
*
- * Drupal.settings refers to the Drupal.settings JavaScript variable.
+ * drupalSettings refers to the drupalSettings JavaScript variable.
*
* @var Array
*/
@@ -2348,9 +2348,9 @@ protected function drupalGetContent() {
}
/**
- * Gets the value of Drupal.settings for the currently-loaded page.
+ * Gets the value of drupalSettings for the currently-loaded page.
*
- * Drupal.settings refers to the Drupal.settings JavaScript variable.
+ * drupalSettings refers to the drupalSettings JavaScript variable.
*/
protected function drupalGetSettings() {
return $this->drupalSettings;
@@ -2404,9 +2404,9 @@ protected function drupalSetContent($content, $url = 'internal:') {
}
/**
- * Sets the value of Drupal.settings for the currently-loaded page.
+ * Sets the value of drupalSettings for the currently-loaded page.
*
- * Drupal.settings refers to the Drupal.settings JavaScript variable.
+ * drupalSettings refers to the drupalSettings JavaScript variable.
*/
protected function drupalSetSettings($settings) {
$this->drupalSettings = $settings;
diff --git a/core/modules/statistics/statistics.js b/core/modules/statistics/statistics.js
index f869ed7c297d..5d66655a466c 100644
--- a/core/modules/statistics/statistics.js
+++ b/core/modules/statistics/statistics.js
@@ -1,4 +1,4 @@
-(function ($) {
+(function ($, Drupal, drupalSettings) {
"use strict";
@@ -6,8 +6,8 @@
$.ajax({
type: "POST",
cache: false,
- url: Drupal.settings.statistics.url,
- data: Drupal.settings.statistics.data
+ url: drupalSettings.statistics.url,
+ data: drupalSettings.statistics.data
});
});
-})(jQuery);
+})(jQuery, Drupal, drupalSettings);
diff --git a/core/modules/system/lib/Drupal/system/Tests/Ajax/FrameworkTest.php b/core/modules/system/lib/Drupal/system/Tests/Ajax/FrameworkTest.php
index eb570ee1a250..e84a7d96d4f1 100644
--- a/core/modules/system/lib/Drupal/system/Tests/Ajax/FrameworkTest.php
+++ b/core/modules/system/lib/Drupal/system/Tests/Ajax/FrameworkTest.php
@@ -210,7 +210,7 @@ public function testLazyLoadOverriddenCSS() {
$this->drupalPostAjaxForm('ajax_forms_test_lazy_load_form', array('add_files' => TRUE), array('op' => t('Submit')));
// Verify that the resulting HTML does not load the overridden CSS file.
- // We add a "?" to the assertion, because Drupal.settings may include
+ // We add a "?" to the assertion, because drupalSettings may include
// information about the file; we only really care about whether it appears
// in a LINK or STYLE tag, for which Drupal always adds a query string for
// cache control.
diff --git a/core/modules/system/lib/Drupal/system/Tests/Common/JavaScriptTest.php b/core/modules/system/lib/Drupal/system/Tests/Common/JavaScriptTest.php
index 96168fea8447..7c4704669231 100644
--- a/core/modules/system/lib/Drupal/system/Tests/Common/JavaScriptTest.php
+++ b/core/modules/system/lib/Drupal/system/Tests/Common/JavaScriptTest.php
@@ -148,13 +148,13 @@ function testHeaderSetting() {
$this->assertTrue(strpos($javascript, 'pathPrefix') > 0, 'Rendered JavaScript header returns pathPrefix setting.');
$this->assertTrue(strpos($javascript, 'currentPath') > 0, 'Rendered JavaScript header returns currentPath setting.');
- // Only the second of these two entries should appear in Drupal.settings.
+ // Only the second of these two entries should appear in drupalSettings.
drupal_add_js(array('commonTest' => 'commonTestShouldNotAppear'), 'setting');
drupal_add_js(array('commonTest' => 'commonTestShouldAppear'), 'setting');
- // Only the second of these entries should appear in Drupal.settings.
+ // Only the second of these entries should appear in drupalSettings.
drupal_add_js(array('commonTestJsArrayLiteral' => array('commonTestJsArrayLiteralOldValue')), 'setting');
drupal_add_js(array('commonTestJsArrayLiteral' => array('commonTestJsArrayLiteralNewValue')), 'setting');
- // Only the second of these two entries should appear in Drupal.settings.
+ // Only the second of these two entries should appear in drupalSettings.
drupal_add_js(array('commonTestJsObjectLiteral' => array('key' => 'commonTestJsObjectLiteralOldValue')), 'setting');
drupal_add_js(array('commonTestJsObjectLiteral' => array('key' => 'commonTestJsObjectLiteralNewValue')), 'setting');
// Real world test case: multiple elements in a render array are adding the
diff --git a/core/modules/views/js/ajax_view.js b/core/modules/views/js/ajax_view.js
index 50d3750806aa..643b8b8309d3 100644
--- a/core/modules/views/js/ajax_view.js
+++ b/core/modules/views/js/ajax_view.js
@@ -2,7 +2,7 @@
* @file
* Handles AJAX fetching of views, including filter submission and response.
*/
-(function ($) {
+(function ($, Drupal, drupalSettings) {
"use strict";
@@ -11,8 +11,8 @@
*/
Drupal.behaviors.ViewsAjaxView = {};
Drupal.behaviors.ViewsAjaxView.attach = function() {
- if (Drupal.settings && Drupal.settings.views && Drupal.settings.views.ajaxViews) {
- $.each(Drupal.settings.views.ajaxViews, function(i, settings) {
+ if (drupalSettings && drupalSettings.views && drupalSettings.views.ajaxViews) {
+ $.each(drupalSettings.views.ajaxViews, function(i, settings) {
Drupal.views.instances[i] = new Drupal.views.ajaxView(settings);
});
}
@@ -29,7 +29,7 @@ Drupal.views.ajaxView = function(settings) {
this.$view = $(selector);
// Retrieve the path to use for views' ajax.
- var ajax_path = Drupal.settings.views.ajax_path;
+ var ajax_path = drupalSettings.views.ajax_path;
// If there are multiple views this might've ended up showing up multiple times.
if (ajax_path.constructor.toString().indexOf("Array") !== -1) {
@@ -145,4 +145,4 @@ Drupal.AjaxCommands.prototype.viewsScrollTop = function (ajax, response) {
}
};
-})(jQuery);
+})(jQuery, Drupal, drupalSettings);
diff --git a/core/modules/views/js/base.js b/core/modules/views/js/base.js
index 5cb7e5ed7742..f929e4314c54 100644
--- a/core/modules/views/js/base.js
+++ b/core/modules/views/js/base.js
@@ -2,7 +2,7 @@
* @file
* Some basic behaviors and utility functions for Views.
*/
-(function ($) {
+(function ($, Drupal, drupalSettings) {
"use strict";
@@ -63,7 +63,7 @@
*/
Drupal.Views.getPath = function (href) {
href = Drupal.Views.pathPortion(href);
- href = href.substring(Drupal.settings.basePath.length, href.length);
+ href = href.substring(drupalSettings.basePath.length, href.length);
// 3 is the length of the '?q=' added to the url without clean urls.
if (href.substring(0, 3) === '?q=') {
href = href.substring(3, href.length);
@@ -77,4 +77,4 @@
return href;
};
-})(jQuery);
+})(jQuery, Drupal, drupalSettings);
diff --git a/core/modules/views/lib/Drupal/views/Controller/ViewAjaxController.php b/core/modules/views/lib/Drupal/views/Controller/ViewAjaxController.php
index f8deb3d596e1..e1aae782f553 100644
--- a/core/modules/views/lib/Drupal/views/Controller/ViewAjaxController.php
+++ b/core/modules/views/lib/Drupal/views/Controller/ViewAjaxController.php
@@ -127,7 +127,7 @@ public function ajaxView(Request $request) {
$response->addCommand(new ScrollTopCommand(".view-dom-id-$dom_id"));
$view->displayHandlers->get($display_id)->setOption('pager_element', $pager_element);
}
- // Reuse the same DOM id so it matches that in Drupal.settings.
+ // Reuse the same DOM id so it matches that in drupalSettings.
$view->dom_id = $dom_id;
$preview = $view->preview($display_id, $args);
diff --git a/core/modules/views/views.theme.inc b/core/modules/views/views.theme.inc
index 6c0af6efaabd..d383da663e28 100644
--- a/core/modules/views/views.theme.inc
+++ b/core/modules/views/views.theme.inc
@@ -99,7 +99,7 @@ function template_preprocess_views_view(&$variables) {
// with its HTML, because one view may appear several times on the page. So
// we set up a hash with the current time, $dom_id, to issue a "unique"
// identifier for each view. This identifier is written to both
- // Drupal.settings and the DIV wrapper.
+ // drupalSettings and the DIV wrapper.
$variables['dom_id'] = $view->dom_id;
$variables['attributes']['class'][] = 'view-dom-id-' . $variables['dom_id'];
}
diff --git a/core/modules/views_ui/js/ajax.js b/core/modules/views_ui/js/ajax.js
index 570ca5980d5c..dc327ee4e9fe 100644
--- a/core/modules/views_ui/js/ajax.js
+++ b/core/modules/views_ui/js/ajax.js
@@ -2,14 +2,14 @@
* @file
* Handles AJAX submission and response in Views UI.
*/
-(function ($) {
+(function ($, Drupal, drupalSettings) {
"use strict";
Drupal.AjaxCommands.prototype.viewsSetForm = function (ajax, response, status) {
- var ajax_title = Drupal.settings.views.ajax.title;
- var ajax_body = Drupal.settings.views.ajax.id;
- var ajax_popup = Drupal.settings.views.ajax.popup;
+ var ajax_title = drupalSettings.views.ajax.title;
+ var ajax_body = drupalSettings.views.ajax.id;
+ var ajax_popup = drupalSettings.views.ajax.popup;
$(ajax_title).html('<h2>' + response.title + '</h2>');
$(ajax_body).html(response.output);
$(ajax_popup).dialog('open');
@@ -38,8 +38,8 @@
};
Drupal.AjaxCommands.prototype.viewsDismissForm = function (ajax, response, status) {
- Drupal.AjaxCommands.prototype.viewsSetForm({}, {'title': '', 'output': Drupal.settings.views.ajax.defaultForm});
- $(Drupal.settings.views.ajax.popup).dialog('close');
+ Drupal.AjaxCommands.prototype.viewsSetForm({}, {'title': '', 'output': drupalSettings.views.ajax.defaultForm});
+ $(drupalSettings.views.ajax.popup).dialog('close');
};
Drupal.AjaxCommands.prototype.viewsHighlight = function (ajax, response, status) {
@@ -220,4 +220,4 @@
}
};
-})(jQuery);
+})(jQuery, Drupal, drupalSettings);
diff --git a/core/modules/views_ui/js/views-admin.js b/core/modules/views_ui/js/views-admin.js
index d20b0af32ba0..b09cbeb767c6 100644
--- a/core/modules/views_ui/js/views-admin.js
+++ b/core/modules/views_ui/js/views-admin.js
@@ -978,7 +978,7 @@ Drupal.viewsUi.resizeModal = function (e, no_shrink) {
});
// Ensure inner popup height matches.
- $(Drupal.settings.views.ajax.popup).css('height', height + 'px');
+ $(drupalSettings.views.ajax.popup).css('height', height + 'px');
$scroll.css({
'height': scrollHeight + 'px',
diff --git a/core/themes/bartik/color/preview.js b/core/themes/bartik/color/preview.js
index 551c0e04bf1a..cb57121169c2 100644
--- a/core/themes/bartik/color/preview.js
+++ b/core/themes/bartik/color/preview.js
@@ -2,7 +2,7 @@
* @file
* Preview for the Bartik theme.
*/
-(function ($) {
+(function ($, Drupal, drupalSettings) {
"use strict";
@@ -11,11 +11,11 @@
callback: function(context, settings, form, farb, height, width) {
// Change the logo to be the real one.
if (!this.logoChanged) {
- $('#preview #preview-logo img').attr('src', Drupal.settings.color.logo);
+ $('#preview #preview-logo img').attr('src', drupalSettings.color.logo);
this.logoChanged = true;
}
// Remove the logo if the setting is toggled off.
- if (Drupal.settings.color.logo === null) {
+ if (drupalSettings.color.logo === null) {
$('div').remove('#preview-logo');
}
@@ -42,4 +42,4 @@
form.find('#preview #preview-site-name').css('color', form.find('#palette input[name="palette[titleslogan]"]').val());
}
};
-})(jQuery);
+})(jQuery, Drupal, drupalSettings);
--
GitLab