Ticket #2262 (closed feature: invalid)

Opened 4 years ago

Last modified 4 years ago

getFeature control: more events

Reported by: fvanderbiest Owned by:
Priority: minor Milestone: 2.9 Release
Component: Control Version: SVN
Keywords: getfeature Cc:
State: Needs Discussion

Description

The following patch adds a "beforegetfeature" event, which is triggered before the call to protocol.read(), and a "getfeature" event triggered on success, with a 'features' property.

Attachments

openlayers_ticket2262_r9652_A0.diff Download (3.1 KB) - added by fvanderbiest 4 years ago.

Change History

Changed 4 years ago by fvanderbiest

Changed 4 years ago by bartvde

  • state changed from Review to Needs Discussion

Francois, should this not be the responsibility of the Protocol instead? See also ticket:1767

Changed 4 years ago by fvanderbiest

Bart, you're completely right. And I'd be +1 in introducing the decorator which Eric promotes in #1767.

In fact, we already have one in mapfish: mapfish.Protocol.TriggerEventDecorator which is (sadly) GPL.

Changed 4 years ago by bartvde

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

Hi Francois, thanks for the feedback, I'll close this ticket in favour of ticket:1767

Note: See TracTickets for help on using tickets.