OpenLayers OpenLayers

Ticket #1095: vml.01.patch

File vml.01.patch, 3.3 kB (added by fredj, 1 year ago)

update tschaub's patch to the last svn version, fix a typo in drawLine (join 'parts' array not 'path')

  • lib/OpenLayers/Renderer/VML.js

    old new  
    460460        this.setNodeDimension(node, geometry); 
    461461 
    462462        var resolution = this.getResolution(); 
     463        var numComponents = geometry.components.length; 
     464        var parts = new Array(numComponents); 
    463465 
    464         var path = "m"; 
    465         for (var i = 0; i < geometry.components.length; i++) { 
    466             var x = (geometry.components[i].x/resolution); 
    467             var y = (geometry.components[i].y/resolution); 
    468             path += " " + x.toFixed() + "," + y.toFixed() + " l "; 
     466        var comp, x, y; 
     467        for (var i = 0; i < numComponents; i++) { 
     468            comp = geometry.components[i]; 
     469            x = (comp.x/resolution); 
     470            y = (comp.y/resolution); 
     471            parts[i] = " " + x.toFixed() + "," + y.toFixed() + " l "; 
    469472        } 
    470         if (closeLine) { 
    471             path += " x"; 
    472         } 
    473         path += " e"; 
    474  
    475         node.path = path; 
     473        var end = (closeLine) ? " x e" : " e"; 
     474        node.path = "m" + parts.join("") + end; 
    476475    }, 
    477476 
    478477    /** 
    …… 
    488487 
    489488        var resolution = this.getResolution(); 
    490489     
    491         var path = ""; 
     490        var path = []; 
     491        var linearRing, i, comp, x, y; 
    492492        for (var j = 0; j < geometry.components.length; j++) { 
    493             var linearRing = geometry.components[j]; 
     493            linearRing = geometry.components[j]; 
    494494 
    495             path += "m"; 
    496             for (var i = 0; i < linearRing.components.length; i++) { 
    497                 var x = linearRing.components[i].x / resolution; 
    498                 var y = linearRing.components[i].y / resolution; 
    499                 path += " " + x.toFixed() + "," + y.toFixed(); 
     495            path.push("m"); 
     496            for (i = 0; i < linearRing.components.length; i++) { 
     497                comp = linearRing.components[i]; 
     498                x = comp.x / resolution; 
     499                y = comp.y / resolution; 
     500                path.push(" " + x.toFixed() + "," + y.toFixed()); 
    500501                if (i==0) { 
    501                     path += " l"; 
     502                    path.push(" l"); 
    502503                } 
    503504            } 
    504             path += " x "; 
     505            path.push(" x "); 
    505506        } 
    506         path += "e"; 
    507         node.path = path; 
     507        path.push("e"); 
     508        node.path = path.join(""); 
    508509    }, 
    509510 
    510511    /** 
    …… 
    537538 
    538539        var resolution = this.getResolution(); 
    539540     
    540         var path = ""; 
     541        var path = []; 
     542        var comp, x, y; 
    541543        for (var i = 0; i < geometry.components.length; i++) { 
    542             var x = geometry.components[i].x / resolution; 
    543             var y = geometry.components[i].y / resolution; 
     544            comp = geometry.components[i]; 
     545            x = comp.x / resolution; 
     546            y = comp.y / resolution; 
    544547            if ((i%3)==0 && (i/3)==0) { 
    545                 path += "m"; 
     548                path.push("m"); 
    546549            } else if ((i%3)==1) { 
    547                 path += " c"; 
     550                path.push(" c"); 
    548551            } 
    549             path += " " + x + "," + y; 
     552            path.push(" " + x + "," + y); 
    550553        } 
    551         path += " x e"; 
     554        path.push(" x e"); 
    552555 
    553         node.path = path; 
     556        node.path = path.join(""); 
    554557    }, 
    555558 
    556559    CLASS_NAME: "OpenLayers.Renderer.VML"