Mattflaschen has uploaded a new change for review.
https://gerrit.wikimedia.org/r/187859
Change subject: Update HTML deserialization for jQuery 1.9+ requirements
......................................................................
Update HTML deserialization for jQuery 1.9+ requirements
* Use parseHTML when HTML content is not hard-coded.
* When it is hard-coded, make sure the string starts with <.
Bug: T85507
Change-Id: Ied96dcbbd2db012dbd1edc42699c0bc4bce328aa
---
M modules/ext.moodBar.dashboard/ext.moodBar.dashboard.js
M modules/ext.moodBar/ext.moodBar.core.js
2 files changed, 48 insertions(+), 61 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MoodBar
refs/changes/59/187859/1
diff --git a/modules/ext.moodBar.dashboard/ext.moodBar.dashboard.js
b/modules/ext.moodBar.dashboard/ext.moodBar.dashboard.js
index 45d2aeb..1da6da5 100644
--- a/modules/ext.moodBar.dashboard/ext.moodBar.dashboard.js
+++ b/modules/ext.moodBar.dashboard/ext.moodBar.dashboard.js
@@ -242,7 +242,7 @@
if ( data && data.query &&
data.query.moodbarcomments &&
data.query.moodbarcomments.length > 0
) {
- var $content = $(
data.query.moodbarcomments[0].formatted );
+ var $content = $( $.parseHTML(
data.query.moodbarcomments[0].formatted ) );
$item.replaceWith( $content );
} else {
// Failure, just remove the link.
diff --git a/modules/ext.moodBar/ext.moodBar.core.js
b/modules/ext.moodBar/ext.moodBar.core.js
index cb2cb4d..822d73f 100644
--- a/modules/ext.moodBar/ext.moodBar.core.js
+++ b/modules/ext.moodBar/ext.moodBar.core.js
@@ -12,14 +12,12 @@
$.extend( mb, {
tpl: {
- overlayBase: '\
- <div id="mw-moodBar-overlayWrap"><div
id="mw-moodBar-overlay">\
- <div id="mw-moodBar-pokey"></div>\
- <span
class="mw-moodBar-overlayClose"><a href="#"><html:msg key="moodbar-close"
/></a></span>\
- <div
class="mw-moodBar-overlayContent"></div>\
- </div></div>',
- userinput: '\
- <div
class="mw-moodBar-overlayTitle"><span><html:msg key="INTROTITLE"
/></span></div>\
+ overlayBase: '<div id="mw-moodBar-overlayWrap"><div
id="mw-moodBar-overlay">\
+ <div
id="mw-moodBar-pokey"></div>\
+ <span
class="mw-moodBar-overlayClose"><a href="#"><html:msg key="moodbar-close"
/></a></span>\
+ <div
class="mw-moodBar-overlayContent"></div>\
+ </div></div>',
+ userinput: '<div
class="mw-moodBar-overlayTitle"><span><html:msg key="INTROTITLE"
/></span></div>\
<div
class="mw-moodBar-types-container">\
<div
class="mw-moodBar-types"></div>\
</div>\
@@ -41,23 +39,21 @@
</a>\
<div
class="mw-moodBar-overlayWhatContent"></div>\
</span>',
- emailinput: '\
- <div
class="mw-moodBar-overlayTitle"><span><html:msg key="moodbar-email-title"
/></span></div>\
- <div class="mw-moodBar-form">\
- <div
class="mw-moodBar-desc">\
- <html:msg
key="moodbar-email-desc" />\
+ emailinput: '<div
class="mw-moodBar-overlayTitle"><span><html:msg key="moodbar-email-title"
/></span></div>\
+ <div class="mw-moodBar-form">\
+ <div class="mw-moodBar-desc">\
+ <html:msg
key="moodbar-email-desc" />\
+ </div>\
+ <div
class="mw-moodBar-formInputs">\
+ <div>\
+ <html:msg
key="moodbar-email-input" />\
+ <input
type="text" id="mw-moodBar-emailInput" class="mw-moodBar-emailInput" />\
</div>\
- <div
class="mw-moodBar-formInputs">\
- <div>\
-
<html:msg key="moodbar-email-input" />\
- <input
type="text" id="mw-moodBar-emailInput" class="mw-moodBar-emailInput" />\
- </div>\
- <input
type="button" class="mw-moodBar-emailSubmit" disabled="disabled" />\
- <input
type="button" class="mw-moodBar-emailOptOut" />\
- </div>\
- </div>',
- emailconfirmation: '\
- <div
class="mw-moodBar-overlayTitle"><span><html:msg
key="moodbar-email-confirm-title" /></span></div>\
+ <input type="button"
class="mw-moodBar-emailSubmit" disabled="disabled" />\
+ <input type="button"
class="mw-moodBar-emailOptOut" />\
+ </div>\
+ </div>',
+ emailconfirmation: '<div
class="mw-moodBar-overlayTitle"><span><html:msg
key="moodbar-email-confirm-title" /></span></div>\
<div class="mw-moodBar-form">\
<div
class="mw-moodBar-desc">\
<html:msg
key="moodbar-email-confirm-desc" />\
@@ -67,51 +63,42 @@
<input
type="button" class="mw-moodBar-emailOptOut" />\
</div>\
</div>',
- type: '\
- <span class="mw-moodBar-type
mw-moodBar-type-$1" rel="$1">\
+ type: '<span class="mw-moodBar-type
mw-moodBar-type-$1" rel="$1">\
</span>',
- loading: '\
- <div class="mw-moodBar-state
mw-moodBar-state-loading">\
- <div
class="mw-moodBar-state-title"><html:msg key="moodbar-loading-title" /></div>\
- <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-loading-subtitle"
/></div>\
- </div>',
+ loading: '<div class="mw-moodBar-state
mw-moodBar-state-loading">\
+ <div
class="mw-moodBar-state-title"><html:msg key="moodbar-loading-title" /></div>\
+ <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-loading-subtitle"
/></div>\
+ </div>',
disabling: '\
<div class="mw-moodBar-state
mw-moodBar-state-loading">\
<div
class="mw-moodBar-state-title"><html:msg key="moodbar-disabling-title" /></div>\
<div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-disabling-subtitle"
/></div>\
</div>',
- updatingEmail: '\
- <div class="mw-moodBar-state
mw-moodBar-state-loading">\
- <div
class="mw-moodBar-state-title"><html:msg key="moodbar-updating-title" /></div>\
- <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-updating-subtitle"
/></div>\
- </div>',
- success: '\
- <div class="mw-moodBar-state
mw-moodBar-state-success">\
- <div
class="mw-moodBar-state-title"><html:msg key="moodbar-success-title" /></div>\
- <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-success-subtitle"
/></div>\
- </div>',
- error: '\
- <div class="mw-moodBar-state
mw-moodBar-state-error">\
+ updatingEmail: '<div class="mw-moodBar-state
mw-moodBar-state-loading">\
+ <div
class="mw-moodBar-state-title"><html:msg key="moodbar-updating-title" /></div>\
+ <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-updating-subtitle"
/></div>\
+ </div>',
+ success: '<div class="mw-moodBar-state
mw-moodBar-state-success">\
+ <div
class="mw-moodBar-state-title"><html:msg key="moodbar-success-title" /></div>\
+ <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-success-subtitle"
/></div>\
+ </div>',
+ error: '<div class="mw-moodBar-state
mw-moodBar-state-error">\
<div
class="mw-moodBar-state-title"><html:msg key="moodbar-error-title" /></div>\
<div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-error-subtitle"
/></div>\
</div>',
- disablingError: '\
- <div class="mw-moodBar-state
mw-moodBar-state-error">\
- <div
class="mw-moodBar-state-title"><html:msg key="moodbar-error-title" /></div>\
- <div
class="mw-moodBar-state-subtitle"><html:msg
key="moodbar-disable-error-subtitle" /></div>\
- </div>',
- blocked: '\
- <div class="mw-moodBar-state
mw-moodBar-state-error">\
- <div
class="mw-moodBar-state-title"><html:msg key="moodbar-blocked-title" /></div>\
- <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-blocked-subtitle"
/></div>\
- </div>',
- ratelimited: '\
- <div class="mw-moodBar-state
mw-moodBar-state-error">\
- <div
class="mw-moodBar-state-title"><html:msg key="moodbar-ratelimited-title"
/></div>\
- <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-ratelimited-subtitle"
/></div>\
- </div>',
- abuse: '\
- <div class="mw-moodBar-state
mw-moodBar-state-error">\
+ disablingError: '<div class="mw-moodBar-state
mw-moodBar-state-error">\
+ <div
class="mw-moodBar-state-title"><html:msg key="moodbar-error-title" /></div>\
+ <div
class="mw-moodBar-state-subtitle"><html:msg
key="moodbar-disable-error-subtitle" /></div>\
+ </div>',
+ blocked: '<div class="mw-moodBar-state
mw-moodBar-state-error">\
+ <div
class="mw-moodBar-state-title"><html:msg key="moodbar-blocked-title" /></div>\
+ <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-blocked-subtitle"
/></div>\
+ </div>',
+ ratelimited: '<div class="mw-moodBar-state
mw-moodBar-state-error">\
+ <div
class="mw-moodBar-state-title"><html:msg key="moodbar-ratelimited-title"
/></div>\
+ <div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-ratelimited-subtitle"
/></div>\
+ </div>',
+ abuse: '<div class="mw-moodBar-state
mw-moodBar-state-error">\
<div
class="mw-moodBar-state-title"><html:msg key="moodbar-abuse-title" /></div>\
<div
class="mw-moodBar-state-subtitle"><html:msg key="moodbar-abuse-subtitle"
/></div>\
</div>'
--
To view, visit https://gerrit.wikimedia.org/r/187859
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ied96dcbbd2db012dbd1edc42699c0bc4bce328aa
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MoodBar
Gerrit-Branch: master
Gerrit-Owner: Mattflaschen <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits