deprecate OpenLayers.loadURL

[fredj]

Is there any reasons that we want to keep OpenLayers.loadURL ? OpenLayers.Request.GET (#1565) offers the same functionality but with more flexibility.

[fredj]

mark function as deprecated, update examples

[erilem]

It looks good. I don't see any reason for this not to go in 2.11 or 2.12.

[rdewit]

Shouldn't the loadURL method give a deprecation warning like this?

OpenLayers.Console.warn(OpenLayers.i18n("methodDeprecated",
                {'newMethod':'OpenLayers.Request.GET'}));

That's what other deprecated methods seem to do.

[fredj]

add a warning in OpenLayers.loadURL

[erilem]

I would personally prefer 3443.0.patch , because I'm against clobbering and bloating the OpenLayers code with "deprecated" messages. I understand that we want to clearly warn the developer about deprecated usages, but I'd favor relying on the Google Closure compiler (or similar) for that ( http://code.google.com/p/closure-compiler/wiki/Warnings). So -1 on 3443.1.patch and +1 on 3443.0.patch . What do others think?

[ahocevar]

I agree with erilem here.

[erilem]

Replying to erilem:

I would personally prefer 3443.0.patch , because I'm against clobbering and bloating the OpenLayers code with "deprecated" messages. I understand that we want to clearly warn the developer about deprecated usages, but I'd favor relying on the Google Closure compiler (or similar) for that ( http://code.google.com/p/closure-compiler/wiki/Warnings). So -1 on 3443.1.patch and +1 on 3443.0.patch . What do others think?

#3320 expresses what I think in more detail.

[fredj]

+1 for 3443.0.patch instead of 3443.1.patch

[fredj]

(In [12206]) mark the OpenLayers.loadURL method as deprecated (replaced by OpenLayers.Request.GET). r=erilem,rdewit,ahocevar (closes #3443)

[fredj]

 http://trac.osgeo.org/openlayers/wiki/Release/2.11/Notes updated

[ahocevar]

in 2.11-rc2