Ticket #1965 (closed bug: wontfix)

Opened 4 years ago

Last modified 4 years ago

MapGuide layer caches dynamic overlay

Reported by: ksgeograf Owned by:
Priority: minor Milestone: 2.8 Release
Component: Layer.MapGuide Version: 2.7
Keywords: Cc:
State:

Description

A dynamic overlay in MapGuide is, as the name states, dynamic. The image is requested with a HTTP "GET" call, and is thus subjected to the browser cache. Since the layer can be changed by server code, the layer should refresh, when the map is refreshed.

OpenLayers use the "viewRequestID" to do this for other purposes. The attached path applies the "viewRequestID" to each request, so the layer is refreshed, whenever required by the map. The sequence number is assigned to the same queryString key that the official viewer uses, "SEQ", to prevent name clashes.

Attachments

Disable cache of dynamic overlays.patch Download (0.7 KB) - added by ksgeograf 4 years ago.

Change History

Changed 4 years ago by ksgeograf

Changed 4 years ago by crschmidt

  • milestone changed from 2.9 Release to 2.8 Release

Changed 4 years ago by crschmidt

  • status changed from new to closed
  • state Review deleted
  • resolution set to wontfix

I don't think we should do this. In all other code, which can also change on the server side, we explicitly use the 'redraw(true)' function to redraw; this adds a salt to the URL. (i've just tested this with the mapguide code, and it does work.) I don't see any reason for MapGuide to be special in this way; using redraw(true) has the client able to control the caching of the layer.

Unless there's some reason that MapGuide is special here, I don't see the reason for it to be added here. If the standard "allow the client to determine when to avoid the cache' is insufficient, then I believe this patch is actually wider than just MapGuide: a 'never hit cache' property on the layer would be reasonable and allow for other layers to take advantage of it as well.

So, if you can provide an explanation of what makes MapGuide special here, this might be a valid ticket. However, I think it's better addressed as a lower level (Layer.HTTPRequest) feature request, or a documentation issue.

Changed 4 years ago by ksgeograf

I was not aware of the redraw(true) method.

redraw(true) does not work well for my usage, but I can see the rationale for not wanting to add my patch.

Note: See TracTickets for help on using tickets.