Ticket #800 (closed feature: fixed)
Simplify mergeNewParams() Functionality
| Reported by: | euzuro | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 2.5 Release |
| Component: | Layer.HTTPRequest | Version: | 2.4 |
| Keywords: | Cc: | ||
| State: |
Description
Right now everyone has to take care of redrawing themselves on mergeNewParams. This is silly. It should happen automatically. Enter redraw() from #795. Once that patch is approved then all we need to do is call redraw() on the layer after merging in the new params, and each layer will redraw itself accordingly.
Attachments
-
mergeNewParams.patch
(2.6 KB) - added by euzuro
6 years ago.
- using our favorite new redraw() function, mergeNewParameters() becomes much simpler. Just do the magic at the HTTPRequest level, where it should have been from the start. Lines inserted: 1. Lines deleted: 33. Vive la france!
-
mergeNewParams.2.patch
(3.1 KB) - added by euzuro
6 years ago.
- sorry. make that one insertion *35* deletions. STAT!
-
mergeNewParams.3.patch
(8.6 KB) - added by tschaub
6 years ago.
- with a test - includes patch for #795 - apologies
Change History
Note: See
TracTickets for help on using
tickets.
