Skip to content

Commit

Permalink
fix[Tinymce]: fixed bug when init multiple tinymces at the same time …
Browse files Browse the repository at this point in the history
…(#2152)
  • Loading branch information
PanJiaChen authored May 27, 2019
1 parent e28701c commit f891202
Showing 1 changed file with 27 additions and 7 deletions.
34 changes: 27 additions & 7 deletions src/components/Tinymce/dynamicLoadScript.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,11 @@
let callbacks = []

function loadedTinymce() {
// to fixed https://github.com/PanJiaChen/vue-element-admin/issues/2144
// check is successfully downloaded script
return window.tinymce
}

const dynamicLoadScript = (src, callback) => {
const existingScript = document.getElementById(src)
const cb = callback || function() {}
Expand All @@ -7,31 +15,43 @@ const dynamicLoadScript = (src, callback) => {
script.src = src // src url for the third-party library being loaded.
script.id = src
document.body.appendChild(script)

callbacks.push(cb)
const onEnd = 'onload' in script ? stdOnEnd : ieOnEnd
onEnd(script, cb)
onEnd(script)
}

if (existingScript && cb) cb(null, existingScript)
if (existingScript && cb) {
if (loadedTinymce()) {
cb(null, existingScript)
} else {
callbacks.push(cb)
}
}

function stdOnEnd(script, cb) {
function stdOnEnd(script) {
script.onload = function() {
// this.onload = null here is necessary
// because even IE9 works not like others
this.onerror = this.onload = null
cb(null, script)
for (const cb of callbacks) {
cb(null, script)
}
callbacks = null
}
script.onerror = function() {
this.onerror = this.onload = null
cb(new Error('Failed to load ' + src), script)
}
}

function ieOnEnd(script, cb) {
function ieOnEnd(script) {
script.onreadystatechange = function() {
if (this.readyState !== 'complete' && this.readyState !== 'loaded') return
this.onreadystatechange = null
cb(null, script) // there is no way to catch loading errors in IE8
for (const cb of callbacks) {
cb(null, script) // there is no way to catch loading errors in IE8
}
callbacks = null
}
}
}
Expand Down

0 comments on commit f891202

Please sign in to comment.