[OpenLayers-Dev] Very small issue

Frédéric Junod frederic.junod at camptocamp.com
Fri Jun 29 02:14:07 EDT 2007


Hello Glen,

I've filled a ticket for that:
http://trac.openlayers.org/ticket/788

Thanks for the report !

fréd

Le Fri, 29 Jun 2007 14:39:59 +1000,
"Glen Stampoultzis" <gstamp at gmail.com> a écrit :

> In Control.js:
> 
>     /**
>      * @constructor
>      *
>      * @param {Object} options
>      */
>     initialize: function (options) {
>         // We do this before the extend so that instances can override
>         // className in options.
>         this.displayClass = this.CLASS_NAME.replace("OpenLayers.",
> "ol").replace(".","");
> 
>         OpenLayers.Util.extend(this, options);
> 
>         this.id = OpenLayers.Util.createUniqueID(this.CLASS_NAME +
> "_"); },
> 
> 
> the replace only replaces the first period.  Better would be
> something like this:
> 
> 
> this.displayClass = this.CLASS_NAME.replace("OpenLayers.",
> "ol").replace(/\./g,"");
> 
> 
> 
> Regards,
> 
> Glen


-- 
Frédéric Junod
Camptocamp SA



More information about the Dev mailing list