# HG changeset patch # User Franck Deroche # Date 1206968841 -7200 # Node ID 49b0a2f3a1185678d6818b485ce0dfba2d378ea2 # Parent d9490757a1116bc9ff67ffd6d81356a296d7d5ce Fix IE: Rend l'administration utilisable avec Internet Explorer 6 En réalité, seulement trois réels changements sont introduits ici: - il existait quelques problèmes dans le code supposé être executés lorsque le navigateur ne connait pas les treewalkers, le code fait désormais ce qu'il faut (la mauvaise fonction était associé à l'édition des brouillons, la fonction de test ne renvoyait pas true / false) - Internet Explorer 6 ne connait pas textContent. Une fonction essayant diverses possibilités de remplacement, et faisant un parcours récursif de tous les noeuds fils dans le pire des cas a été mise en place - le code cherchant un remplacement à DOMParser supposait que l'objet ActiveX XMLDOM fournissait exactement la même API, ce qui aurait été trop beau. Un wrapper a été développé autour de XMLDOM qui fournit la seule fonction de DOMParser utilisé dans le script. Evidemment ce n'est pas une solution totalement générique... Avec ces modifications, il est possible de poster depuis Internet Explorer (testé sur la version 6), Safari (testé avec la version 3.1) et Firefox (testé avec les versions 2 et 3) diff -r d9490757a111 -r 49b0a2f3a118 admin.js --- a/admin.js Mon Mar 31 14:53:49 2008 +0200 +++ b/admin.js Mon Mar 31 15:07:21 2008 +0200 @@ -37,36 +37,66 @@ elt.addEventListener(evt, func, useCpt); else if(elt.attachEvent) elt.attachEvent("on" + evt, func); + else if(elt["on" + evt]) + elt["on" + evt] = function() {elt["on" + evt](); func();} else elt["on" + evt] = func; + }, + getTextContent: function(node) { + if(node.textContent) + return node.textContent; + if(node.innerText) + return node.innerText; + // We tried hard to use something fast, it didn't work, so we'll recurse + if(node.nodeType == 3 || node.nodeType == 4) + return node.data; + var textContent = '', i; + for(i = 0; i < node.childNodes.length; i++) + textContent += t.getTextContent(node.childNodes[i]); + return textContent; + }, + empty: function(node) { + while(node.firstChild) + node.removeChild(node.firstChild); } } t.evtListener(window, "load", xmlRequest.init, false); t.evtListener(window, "load", hideForms, false); +t.evtListener(window, "load", init, false); var editFilter = { + accept: function(node) { + if(node.nodeName.toLowerCase() == 'a' && + node.firstChild && + node.firstChild.nodeValue == 'Editer') + return true; + return false; + }, + acceptNode: function(node) { - if(node.nodeName == 'a' && node.firstChild && node.firstChild.nodeValue == 'Editer') - return NodeFilter.FILTER_ACCEPT; - return NodeFilter.FILTER_SKIP; + return this.accept(node) ? NodeFilter.FILTER_ACCEPT : NodeFilter.FILTER_SKIP; } } var brouillonsFilter = { - acceptNode:function(node) { - if(node.nodeName == 'a' && node.firstChild && ( + accept: function(node) { + if(node.nodeName.toLowerCase() == 'a' && node.firstChild && ( node.firstChild.nodeValue == 'Editer' || node.firstChild.nodeValue == 'Publier' || node.firstChild.nodeValue == 'Supprimer')) - return NodeFilter.FILTER_ACCEPT; - return NodeFilter.FILTER_SKIP; + return true; + return false; + }, + + acceptNode: function(node) { + return this.accept(node) ? NodeFilter.FILTER_ACCEPT : NodeFilter.FILTER_SKIP; } } -window.onload=function() { +function init() { var ar=document.getElementsByTagName('li'); for(var i=0; i'); + errorNode = this.obj.createElement('parsererror'); + errorNode.appendChild(this.obj.createTextNode(error)); + this.obj.documentElement.appendChild(errorNode); + } + return this.obj; + } + } + } t.evtListener(document.getElementById("editPost").Save, "click", saveFormContent, false); // TreeWalkers in all their glory try { - wEdit = document.createTreeWalker(document.getElementById("GestPosts"), NodeFilter.SHOW_ALL, editFilter, true); + wEdit = document.createTreeWalker(document.getElementById("GestPosts"), NodeFilter.SHOW_ELEMENT, editFilter, true); while(wEdit.nextNode()) { t.evtListener(wEdit.currentNode, "click", editPubl, false); } - wBr = document.createTreeWalker(document.getElementById("GestBrouillons"), NodeFilter.SHOW_ALL, brouillonsFilter, true); + wBr = document.createTreeWalker(document.getElementById("GestBrouillons"), NodeFilter.SHOW_ELEMENT, brouillonsFilter, true); while(wBr.nextNode()) { if(wBr.currentNode.firstChild.nodeValue == 'Editer') t.evtListener(wBr.currentNode,"click", editBrouillon, false); else if(wBr.currentNode.firstChild.nodeValue == 'Publier') t.evtListener(wBr.currentNode, "click", pubBrouillon, false); @@ -100,7 +146,7 @@ var tmpLinks = document.getElementById("GestPosts") .getElementsByTagName("a"); for(var i = 0; i < tmpLinks.length; i++) { - if(editFilter.acceptNode(tmpLinks[i])) { + if(editFilter.accept(tmpLinks[i])) { t.evtListener(tmpLinks[i], "click", editPubl, false) } } @@ -108,8 +154,11 @@ tmpLinks = document.getElementById("GestBrouillons") .getElementsByTagName("a"); for(var i = 0; i < tmpLinks.length; i++) { - if(brouillonsFilter.acceptNode(tmpLinks[i])) { - t.evtListener(tmpLinks[i], "click", editPubl, false) + if(brouillonsFilter.accept(tmpLinks[i])) { + if(tmpLinks[i].firstChild.nodeValue == 'Editer') + t.evtListener(tmpLinks[i], "click", editBrouillon, false); + else if(tmpLinks[i].firstChild.nodeValue == 'Publier') + t.evtListener(tmpLinks[i], "click", pubBrouillon, false); } } } @@ -146,14 +195,15 @@ whichVersion.style.display='block'; el.pubIsShown = true; var data = xmlRequest.get("admin_xml.php?type=brouillon&id=" + whichVersion.id.value); - var doc = dp.parseFromString("" + data.getElementsByTagName("contenu")[0].firstChild.data + "", "text/xml"); - var parserErrors = doc.getElementsByTagName('parsererror'); - if(parserErrors.length > 0) { - pubComment.textContent = ""; - pubComment.appendChild(doc.firstChild); + var doc = dp.parseFromString("" + t.getTextContent(data.getElementsByTagName("contenu")[0]) + "", "text/xml"); + t.empty(pubComment); + if(doc.getElementsByTagName('parsererror').length > 0) { + var textNode = document.createTextNode(t.getTextContent(doc.documentElement) + "\n"); + pubComment.appendChild(textNode); } else { - pubComment.textContent = "Le post est fait de XML valide, bon pour être poster"; + var textNode = document.createTextNode("Le post est fait de XML valide, bon pour être poster"); + pubComment.appendChild(textNode); whichVersion.pubButton.disabled = false; } } else { @@ -184,21 +234,22 @@ el.editPostIsShown = 1; el.parentNode.parentNode.className = 'Pinned'; editPost.style.display = 'block'; - editPost.Titre.value = data.getElementsByTagName('titre')[0].textContent; - editPost.Contenu.value = data.getElementsByTagName('contenu')[0].firstChild.data; + editPost.Titre.value = t.getTextContent(data.getElementsByTagName('titre')[0]); + editPost.Contenu.value = t.getTextContent(data.getElementsByTagName('contenu')[0]); if(type == 'post') { editPost.mood.style.display = ''; - editPost.mood.value = data.getElementsByTagName('mood')[0].textContent; + editPost.mood.value = t.getTextContent(data.getElementsByTagName('mood')[0]); editPost.Tags.style.display = ''; editPost.Tags.value = ''; - var tags = data.getElementsByTagName('tag'); + var tags = data.getElementsByTagName('tag'), tag; for(var i = 0; i < tags.length; i++) { - if(tags[i].textContent.indexOf(' ') > -1) + tag = t.getTextContent(tags[i]); + if(tag.indexOf(' ') > -1) { - editPost.Tags.value += '"' + tags[i].textContent + '"'; + editPost.Tags.value += '"' + tag + '"'; } else { - editPost.Tags.value += tags[i].textContent; + editPost.Tags.value += tag; } editPost.Tags.value += ' '; } @@ -221,7 +272,7 @@ } function saveFormContent(e) { - var el = e.target.form; // Should be the form... + var el = bpEvt(e).rTarget.form; // Should be the form... var toggleLink = el.parentNode.firstChild; var id=toggleLink.href.substring(toggleLink.href.indexOf("#") + 4, toggleLink.href.length); if(xmlRequest.send('admin_xml.php?id=' + id + '&mode=edit&type=' + el.type,