EditController can now throw AttentionEvents too, so they don't all have to go throug...
[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 To check:
8 - MapDust
9 - VectorLayers from GPX
10 - VectorLayers from shapefiles
11 - Save/OAuth not tested
12 - removeLayer in MapPaint.as should blank - does it?
13
14 == Changes overall ==
15
16 Map.as becomes a collection of MapPaint objects, all with the same lat/long/scale
17 (plus a backdrop sprite and a background imagery sprite)
18
19 Map.editableLayer returns a reference to the single MapPaint object where isBackground is false
20
21 MapPaint.as gains a reference to a Connection - i.e. the source data for this layer
22
23 == Code tidying ==
24
25 There are lots more of these throughout the code, marked with fixmes of the form '>>>> REFACTOR'
26
27 - remove all back-references to Map except where necessary for lat/long/scale
28 - getParam is messy - stuff sent via flashvars needs somewhere to live: currently in Globals which is ugly
29 - tileset/setbackground stuff is screwed up again and needs refactoring a little