OpenLayers OpenLayers

Ticket #174 (closed bug: fixed)

Opened 2 years ago

Last modified 2 years ago

typo in WMS.js

Reported by: tschaub Assigned to: euzuro
Priority: minor Milestone: 2.0 Release
Component: Layer Version:
Keywords: Cc:
State:

Description

The constructor in WMS.js checks this.params.TRANSPARENT instead of params.TRANSPARENT. I think this is wrong. I could be mistaken though.

Attachments

WMS.js.patch (353 bytes) - added by tschaub on 08/15/06 11:20:36.
params.TRANSPARENT instead of this.params.TRANSPARENT
WMS.js.2.patch (356 bytes) - added by tschaub on 08/15/06 11:31:37.
test for true instead of "true"

Change History

08/15/06 11:20:36 changed by tschaub

  • attachment WMS.js.patch added.

params.TRANSPARENT instead of this.params.TRANSPARENT

08/15/06 11:24:47 changed by tschaub

Apologies for the hasty ticket - that wasn't the problem...

08/15/06 11:31:37 changed by tschaub

  • attachment WMS.js.2.patch added.

test for true instead of "true"

08/15/06 11:32:29 changed by tschaub

Ok, I think the second patch solves the problem. Looks like the test should be for true instead of "true".

08/15/06 23:47:11 changed by euzuro

  • status changed from new to closed.
  • resolution set to fixed.

in the HTTPRequest constructor, we make a copy of params into this.params, so the first patch is not quite necessary.

The later patch, regarding true and "true", has been incorporated. Thanks for spotting that!

08/16/06 00:54:41 changed by euzuro

  • milestone set to 2.0 Release.