Opened 13 years ago

Closed 13 years ago

#288 closed defect (fixed)

FeaturePanel_Selector - GET request can be too long, causes IE 7 and 8 isues

Reported by: cdeschenes Owned by: adube
Priority: minor Milestone: First milestone
Component: widgets Version: 1.0
Keywords: featurepanel, widget, ajax, post, internet explorer Cc: adube

Description

featurepanel widget ajax request can bug on some configuration using IE 7 and 8 when a request is too long (too many parameter). The bug seem appear on a minority of computer using IE7 and IE8.

To correct the bug, just edit the code to use a post request and not a get request !

Attachments (1)

post_featurepanel.patch (1.0 KB ) - added by cdeschenes 13 years ago.
my fix !

Download all attachments as: .zip

Change History (2)

by cdeschenes, 13 years ago

Attachment: post_featurepanel.patch added

my fix !

comment:1 by adube, 13 years ago

Resolution: fixed
Status: newclosed
Summary: featurepanel widget ajax request can bug on some configuration using IE 7 and 8FeaturePanel_Selector - GET request can be too long, causes IE 7 and 8 isues

Fixed in r1131.

Fix info

When using the featurepanel_selector 'method' option, the parameters were append to the URL directly, thus resulting in very long GET requests. To fix this, the 'params' method used, which automatically convert the method to POST instead. It supports string value, so not much change was needed.

Small change to the patch

I changed the string concatenation method following this good behavior : http://trac.osgeo.org/openlayers/wiki/CodingStandards/StringConcatenation

Note: See TracTickets for help on using tickets.