Retrait des éléments notés deprecated depuis mathusalem. Jslint.

diff --git a/skins/ajax_scripts/fragment_importer.js b/skins/ajax_scripts/fragment_importer.js
index 9830ef51c5c4cfc2993767e6cc603b3fa9078029..6839a166af47f195b71b006a028c61465bb0006c 100644 (file)
--- a/skins/ajax_scripts/fragment_importer.js
+++ b/skins/ajax_scripts/fragment_importer.js
@@ -10,13 +10,10 @@ var FragmentImporter;
 
 var isTextMime = /^text\/.+/i;
 
 
 var isTextMime = /^text\/.+/i;
 

-FragmentImporter = function(url, onAfterPopulate, baseElement) {
+FragmentImporter = function(url, onAfterPopulate) {

        var thisImporter = this;
        this.url = url;
        this.onAfterPopulate = (!onAfterPopulate) ? function(){return;} : onAfterPopulate;
        var thisImporter = this;
        this.url = url;
        this.onAfterPopulate = (!onAfterPopulate) ? function(){return;} : onAfterPopulate;

-       this.baseElement = baseElement;
-       if (baseElement && window.console)
-               console.warn('Deprecation warning : usage of baseElement will be removed. Use an xml based response.');

 };
 
 FragmentImporter.prototype._load = function(url) {
 };
 
 FragmentImporter.prototype._load = function(url) {


@@ -40,12 +37,12 @@ FragmentImporter.prototype._load = function(url) {
                                break;
                        case 4 :
                                hideProgressImage();
                                break;
                        case 4 :
                                hideProgressImage();

-                               if (req.status == '200')
-                                       thisImporter.populateBaseElement(req);
-                               else
-                                       alert('Error: ' + req.status);
-                               
-               };
+                               if (req.status === 200) {
+                                       thisImporter.populateBaseElement(req); }
+                               else {
+                                       alert('Error: ' + req.status); }
+                               break;
+               }

        };
 
        req.open("GET", url, true);
        };
 
        req.open("GET", url, true);


@@ -53,17 +50,17 @@ FragmentImporter.prototype._load = function(url) {
 };
 
 FragmentImporter.prototype.load = function(fallBackUrl) {
 };
 
 FragmentImporter.prototype.load = function(fallBackUrl) {

-       if (fallBackUrl)
-               this._fallBackUrl = fallBackUrl;
-       else
-               this._fallBackUrl = this.url;
+       if (fallBackUrl) {
+               this._fallBackUrl = fallBackUrl; }
+       else {
+               this._fallBackUrl = this.url; }

        this._load(this.url);
 };
 
 FragmentImporter.prototype.useMacro = function(template, macro, fragmentId, queryString) {
        var url = this.url + "/use_macro?template=" + template + "&macro=" + macro + "&fragmentId=" + fragmentId;
        this._load(this.url);
 };
 
 FragmentImporter.prototype.useMacro = function(template, macro, fragmentId, queryString) {
        var url = this.url + "/use_macro?template=" + template + "&macro=" + macro + "&fragmentId=" + fragmentId;

-       if (queryString)
-               url += '&' + queryString;
+       if (queryString) {
+               url += '&' + queryString; }

        this._load(url);
 };
 
        this._load(url);
 };
 


@@ -76,38 +73,23 @@ FragmentImporter.prototype.populateBaseElement = function(req) {
                window.location.href = this._fallBackUrl;
                return;
        }
                window.location.href = this._fallBackUrl;
                return;
        }

-       if (contentType.indexOf('text/xml') != -1) {
+       if (contentType.indexOf('text/xml') !== -1) {

                var fragments = req.responseXML.documentElement.childNodes;
                var fragments = req.responseXML.documentElement.childNodes;

-               var fragment, dest, scripts;
-               for (var i=0 ; i < fragments.length ; i++) {
+               var fragment, dest, scripts, i, j;
+               for (i=0 ; i < fragments.length ; i++) {

                        fragment = fragments[i];
                        fragment = fragments[i];

-                       if (fragment.nodeName == 'fragment') {
+                       if (fragment.nodeName === 'fragment') {

                                dest = document.getElementById(fragment.getAttribute('id'));
                                dest = document.getElementById(fragment.getAttribute('id'));

-                               if(!dest)
-                                       continue;
-                               dest.innerHTML = fragment.firstChild.nodeValue;
-                               
-                               scripts = dest.getElementsByTagName('script');
-                               for (var j=0 ; j < scripts.length ; j++)
-                                       globalScriptRegistry.loadScript(scripts[j]);
+                               if(dest) {
+                                       dest.innerHTML = fragment.firstChild.nodeValue;
+                                       scripts = dest.getElementsByTagName('script');
+                                       for (j=0 ; j < scripts.length ; j++) {
+                                               globalScriptRegistry.loadScript(scripts[j]); }
+                               }

                        }
                }
        }
                        }
                }
        }

-       else {
-               this.baseElement.innerHTML = req.responseText;
-       
-               var scripts = this.baseElement.getElementsByTagName('script');
-               for (var i=0 ; i < scripts.length ; i++)
-                       globalScriptRegistry.loadScript(scripts[i]);
-       }
-
-       var onAfterPopulate = this.onAfterPopulate;
-       if (typeof(onAfterPopulate) == "string") {
-               if (console)
-                       console.warn('Deprecation warning : onAfterPopulate may not be a string (' + onAfterPopulate + ')');
-               onAfterPopulate = eval(onAfterPopulate);
-       }
-       onAfterPopulate();
+       this.onAfterPopulate();

 };
 
 };
 

-})();
\ No newline at end of file
+}());
\ No newline at end of file