Opened 7 years ago

Last modified 6 years ago

#314 new enhancement

WMS capabilities loader should use store

Reported by: bartvde Owned by:
Priority: major Milestone: 1.2
Component: GeoExt.tree.WMSCapabilitiesLoader Version: trunk
Keywords: Cc:
State: Needs more work

Description

Once ticket:287 gets in, we will be able to use a store in the WMS Capabilities Loader (which does not have a Trac component as yet). This idea was originally discussed in ticket:272

Attachments (1)

ticket314.patch (9.6 KB) - added by bartvde 7 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 7 years ago by bartvde

In order to get support back for layerParams, we need ticket:315 as well.

Changed 7 years ago by bartvde

comment:2 Changed 7 years ago by bartvde

I did not see an non-hacky way to preserve the load test, so I removed it instead.

I'll only set this ticket to review once the dependent tickets (ticket:287 and ticket:315) get committed.

comment:3 Changed 7 years ago by elemoine

  • Component changed from GeoExt.tree to GeoExt.tree.WMSCapabilitiesLoader
  • Owner ahocevar deleted

comment:4 Changed 7 years ago by ahocevar

  • State changed from None to Review

Setting to review. Ticket depends on #287.

comment:5 Changed 7 years ago by ahocevar

  • Keywords api change added
  • Priority changed from major to blocker

Bart, would there be any API changes resulting from this change? If not, please remove the api change keywords and lower the priority of this ticket.

comment:6 Changed 7 years ago by ahocevar

  • Keywords api change removed
  • Priority changed from blocker to major
  • State changed from Review to Needs more work

Bart: Looking at #287 and the getParentNode method here, I wonder if it would make things simpler to really just store the parent on the WMSCapabilitiesRecord (like Pierre suggested in #287). Another thing: you removed the requestMethod override. Was this on purpose?

Also, it seems that this ticket does not propose an API change. So this is not a blocker any more.

comment:7 Changed 7 years ago by bartvde

  • Milestone changed from 1.0 to 1.1

comment:8 Changed 6 years ago by ahocevar

  • Milestone changed from 1.1 to 1.2

Batch move of tickets to finish the 1.1 milestone.

Note: See TracTickets for help on using tickets.