actually, it was
http://trac.openlayers.org/changeset/2473#file0
thanks for picking this up. logical bleep on my part. apologies!
erik
On 7/19/07, Christopher Schmidt <[EMAIL PROTECTED]> wrote:
> Erik, can you review this one? 'svn blame' says it belongs to you, and
> it looks like it was part of r2370:
>
> - getBounds: function () {
> - if (!this.bounds) {
> - for (var i = 0; i < this.components.length; i++) {
> + calculateBounds: function() {
> + this.bounds = null;
> + if ( !this.components || (this.components.length > 0)) {
> + this.setBounds(this.components[0].getBounds());
> + for (var i = 1; i < this.components.length; i++) {
>
> It certainly 'feels' wrong to me.
>
>
> -- Chris
>
> On Thu, Jul 19, 2007 at 01:08:56PM -0000, OpenLayers wrote:
> > #850: typo in Collection.calculateBounds()
> > ---------------------+------------------------------------------------------
> > Reporter: pgiraud | Owner:
> > Type: feature | Status: new
> > Priority: minor | Milestone:
> > Component: general | Version: 2.4
> > Keywords: |
> > ---------------------+------------------------------------------------------
> > It is likely that there is a little typo in the calculateBounds in the
> > Geometry.Collection class.
>
> Regards,
> --
> Christopher Schmidt
> MetaCarta
> _______________________________________________
> Dev mailing list
> [email protected]
> http://openlayers.org/mailman/listinfo/dev
>
_______________________________________________
Dev mailing list
[email protected]
http://openlayers.org/mailman/listinfo/dev
