Pull-through wasn't working because appendNode doesn't force a redraw - fix that
[potlatch2.git] / REFACTOR.txt
1 == Potlatch 2 refactoring ==
2
3 Aims:
4 - to enable multiple Maps to be shown on-screen (e.g. for junction editor)
5 - to bring Connection and VectorLayer code together
6
7 Current known problems:
8 - AttentionEvents don't work
9
10 To check:
11 - MapDust
12 - VectorLayers from GPX
13 - VectorLayers from shapefiles
14 - Save/OAuth not tested
15 - removeLayer in MapPaint.as should blank - does it?
16
17 == Changes overall ==
18
19 Map.as becomes a collection of MapPaint objects, all with the same lat/long/scale
20 (plus a backdrop sprite and a background imagery sprite)
21
22 Map.editableLayer returns a reference to the single MapPaint object where isBackground is false
23
24 MapPaint.as gains a reference to a Connection - i.e. the source data for this layer
25
26 == Code tidying ==
27
28 There are lots more of these throughout the code, marked with fixmes of the form '>>>> REFACTOR'
29
30 - remove all back-references to Map except where necessary for lat/long/scale
31 - getParam is messy - stuff sent via flashvars needs somewhere to live: currently in Globals which is ugly
32 - tileset/setbackground stuff is screwed up again and needs refactoring a little