Ticket #2833 (closed feature: invalid)

Opened 3 years ago

Last modified 3 years ago

Permalink control: Sets as deprecated the optional parameters in constructor (thinking 3.0)

Reported by: jorix Owned by: crschmidt
Priority: minor Milestone: 2.11 Release
Component: Control.Permalink Version: 2.10
Keywords: Cc:
State:

Description

In the patch is proposed to overload the first parameter of constructor in order to have two constructors: one with "options" as single parameter and another, deprecated, with three parameters.

In the patch proposed changes in two examples (*) to use the new constructor.

The patch includes a specific test.

The tests work well in: FF36, IE8, Chrome6 & Safari5.

(*) The example "getfeatureinfo.html" not working properly because the domain "freemap.in" used is for sale, there are other examples affected by the same problem.

Attachments

ControlPermalink-newCostructor-2833.patch Download (6.4 KB) - added by jorix 3 years ago.

Change History

Changed 3 years ago by jorix

Changed 3 years ago by jorix

  • state set to Review

Please review, thanks.

Changed 3 years ago by ahocevar

  • status changed from new to closed
  • resolution set to invalid

Same situation as #2832

Changed 3 years ago by jorix

  • state Review deleted
Note: See TracTickets for help on using tickets.