Ticket #866 (closed feature: fixed)

Opened 6 years ago

Last modified 6 years ago

patch to deprecate MapServer.Untiled

Reported by: pspencer Owned by: euzuro
Priority: minor Milestone: 2.5 Release
Component: Layer.MapServer Version: 2.4
Keywords: Cc:
State:

Description

With the introduction of Grid.singleTile, Untiled layers are deprecated. The attached patch does the following:

* modifies MapServer.Untiled to inherit directly from MapServer with a singleTile attribute set to true, and emit a console warning about deprecation (copied from WMS.Untiled directly) * modifies examples/mapserver_untiled.html to use a MapServer layer with a singleTile: true option * modifies tests/Layer/test_MapServer.html to add a useful singleTile test

tests/Layer/test_MapServer_Untiled.html should be removed from trunk as well.

Attachments

untiled.patch Download (11.8 KB) - added by pspencer 6 years ago.
patch file to update r3815 with changes to support untiled mapserver layers through use of Grid.singleLayer
mapserver.untiled.patch Download (23.8 KB) - added by crschmidt 6 years ago.
merge in tests
mapserver.patch Download (23.8 KB) - added by crschmidt 6 years ago.
update to trunk
mapserver.2.patch Download (24.2 KB) - added by tschaub 6 years ago.
delete test page and modify test list

Change History

Changed 6 years ago by pspencer

patch file to update r3815 with changes to support untiled mapserver layers through use of Grid.singleLayer

Changed 6 years ago by crschmidt

  • keywords review added

Changed 6 years ago by euzuro

  • owner changed from crschmidt to euzuro
  • status changed from new to assigned

Changed 6 years ago by tschaub

Updated the patch to include changes in r3928 (#file2).

Three tests fail in Layer/test_MapServer_Untiled.html. Is this expected? If so, this test page should be removed with this patch.

Changed 6 years ago by crschmidt

Note that the ticket description does say "tests/Layer/test_MapServer_Untiled.html should be removed from trunk as well."

The tests which are failing are:

  • a copy paste error that should never have been tested in the first place (MapServer CGI doesn't use SRS, and the URL is explicitly checking for SRS)
  • A change in an internal variable name (but it's not an APIProperty, so we don't care)
  • A change to the default ratio value (1 -> 1.5)

So, these tests should be failing -- I guess my only question is whether we should ifx them and merge them into the MapServer.html test file. I think the answer is yes.

For the record, having misbehavior which is tested makes me a sad panda -- it's embarrassing to remove it. I wonder how we can avoid that in the future.

Changed 6 years ago by crschmidt

merge in tests

Changed 6 years ago by crschmidt

Move the tests into Layer/test_MapServer.html, and fix the ones that the api change breaks.

Tim -- you said "uploaded a new patch", but I don't see a new patch. Am I missing something? (Also, r3928 only seems slightly related.)

Changed 6 years ago by euzuro

I apologize for not chiming in on this earlier. I started attacking this ticket but then when I saw that tests were failing I dropped it and got sidetracked. sorry. should have at least added a comment to that effect.

I'll wait on tim putting up the patch he mentions here, and will/can take it over from there if people want. sorry for my negligence.

Changed 6 years ago by tschaub

Sorry, "Updated the patch" only meant updated on my end, not uploaded.

Changed 6 years ago by crschmidt

update to trunk

Changed 6 years ago by tschaub

delete test page and modify test list

Changed 6 years ago by tschaub

  • keywords commit added; review removed

Looks good to me. The mapserver.2.patch modifies list-tests.html so all pass in IE/FF. Please commit.

Changed 6 years ago by crschmidt

  • keywords commit removed
  • status changed from assigned to closed
  • resolution set to fixed

(In [4068]) deprecate MapServer.Untiled in favor of MapServer with singleTile: true. (Closes #866) Thanks for the help to all.

Note: See TracTickets for help on using tickets.