Ticket #2297 (closed feature: fixed)

Opened 4 years ago

Last modified 4 years ago

Create an urlAppend function that appends strings to urls and handles ? and & appropriately

Reported by: ahocevar Owned by:
Priority: minor Milestone: 2.9 Release
Component: Util Version: SVN
Keywords: Cc:
State: Complete

Description

The attached patch introduces OpenLayers.Util.urlAppend, and changes Layer.HTTPRequest.getFullRequestString to use it.

This was inspired by the need to append to an url for #2224.

The attached patch comes with tests, tests pass in FF3 and IE8.

Attachments

openlayers-2297.patch Download (4.3 KB) - added by ahocevar 4 years ago.

Change History

Changed 4 years ago by ahocevar

Changed 4 years ago by crschmidt

  • state changed from Review to Commit

APplies, tests pass, simplifies code. Win.

Changed 4 years ago by ahocevar

  • status changed from new to closed
  • state changed from Commit to Complete
  • resolution set to fixed

(In [9730]) "Create an urlAppend function that appends strings to urls and handles ? and & appropriately". r=crschmidt (closes #2297)

Note: See TracTickets for help on using tickets.