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