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