0.6c: bunch of minor improvements and bugfixes

diff --git a/app/controllers/amf_controller.rb b/app/controllers/amf_controller.rb
index c0b642edaeb4faf313b473c53ea8b30230d50f2a..6c07d24ad7c108ef63a69078eadf1fac58338fa9 100644 (file)
--- a/app/controllers/amf_controller.rb
+++ b/app/controllers/amf_controller.rb
@@ -11,7 +11,7 @@ class AmfController < ApplicationController
   # encoded in the Actionscript Message Format (AMF).
   #
   # Public domain. Set your tab width to 4 to read this document. :)
-  # editions Systeme D / Richard Fairhurst 2004-2007
+  # editions Systeme D / Richard Fairhurst 2004-2008
   
   # to trap errors (getway_old,putway,putpoi,deleteway only):
   #   return(-1,"message")             <-- just puts up a dialogue
@@ -225,7 +225,7 @@ class AmfController < ApplicationController
 
     RAILS_DEFAULT_LOGGER.info("  Message: getway, id=#{wayid}")
 
-    readwayquery(wayid).each {|row|
+    readwayquery(wayid,true).each {|row|
       points<<[long2coord(row['longitude'].to_f,baselong,masterscale),lat2coord(row['latitude'].to_f,basey,masterscale),row['id'].to_i,nil,tag2array(row['tags'])]
       xmin = [xmin,row['longitude'].to_f].min
       xmax = [xmax,row['longitude'].to_f].max
@@ -329,25 +329,17 @@ class AmfController < ApplicationController
     xc={}; yc={}; tagc={}; vc={}
     if originalway>0
       way=originalway
-         if oldversion==0
-           readwayquery(way).each { |row|
-                 id=row['id'].to_i
+         if oldversion==0 then r=readwayquery(way,false)
+                                          else r=readwayquery_old(way,oldversion,true) end
+         r.each { |row|
+               id=row['id'].to_i
+               if (id>0) then
                  xc[id]=row['longitude'].to_f
                  yc[id]=row['latitude' ].to_f
                  tagc[id]=row['tags']
-                 vc[id]=1
-               }
-         else
-           readwayquery_old(way,oldversion,true).each { |row|
-                 id=row['id'].to_i
-                 if (id>0) then
-                       xc[id]=row['longitude'].to_f
-                       yc[id]=row['latitude' ].to_f
-                       tagc[id]=row['tags']
-                       vc[id]=row['visible'].to_i
-                 end
-               }
-         end
+                 vc[id]=row['visible'].to_i
+           end
+         }
       ActiveRecord::Base.connection.update("UPDATE current_ways SET timestamp=#{db_now},user_id=#{uid},visible=1 WHERE id=#{way}")
     else
       way=ActiveRecord::Base.connection.insert("INSERT INTO current_ways (user_id,timestamp,visible) VALUES (#{uid},#{db_now},1)")
@@ -392,7 +384,7 @@ class AmfController < ApplicationController
       elsif xc.has_key?(node)
                nodelist.push(node)
         # old node from original way - update
-        if (xs!=xc[node] or (ys/0.0000001).round!=(yc[node]/0.0000001).round or tagstr!=tagc[node] or vc[node]==0)
+        if ((xs/0.0000001).round!=(xc[node]/0.0000001).round or (ys/0.0000001).round!=(yc[node]/0.0000001).round or tagstr!=tagc[node] or vc[node]==0)
           ActiveRecord::Base.connection.insert("INSERT INTO nodes (id,latitude,longitude,timestamp,user_id,visible,tags,tile) VALUES (#{node},#{lat},#{long},#{db_now},#{uid},1,#{tagsql},#{tile})")
           ActiveRecord::Base.connection.update("UPDATE current_nodes SET latitude=#{lat},longitude=#{long},timestamp=#{db_now},user_id=#{uid},tags=#{tagsql},visible=1,tile=#{tile} WHERE id=#{node}")
         end
@@ -571,13 +563,16 @@ end
 # ====================================================================
 # Support functions for remote calls
 
-def readwayquery(id)
-  ActiveRecord::Base.connection.select_all "SELECT latitude*0.0000001 AS latitude,longitude*0.0000001 AS longitude,current_nodes.id,tags "+
-      "    FROM current_way_nodes,current_nodes "+
-      "   WHERE current_way_nodes.id=#{id} "+
-      "     AND current_way_nodes.node_id=current_nodes.id "+
-      "     AND current_nodes.visible=1 "+
-      "   ORDER BY sequence_id"
+def readwayquery(id,insistonvisible)
+  sql=<<-EOF
+    SELECT latitude*0.0000001 AS latitude,longitude*0.0000001 AS longitude,current_nodes.id,tags,visible 
+      FROM current_way_nodes,current_nodes 
+     WHERE current_way_nodes.id=#{id} 
+       AND current_way_nodes.node_id=current_nodes.id 
+  EOF
+  if insistonvisible then sql+=" AND current_nodes.visible=1 " end
+  sql+=" ORDER BY sequence_id"
+  ActiveRecord::Base.connection.select_all(sql)
 end
 
 def getlastversion(id,version)

diff --git a/app/views/site/edit.rhtml b/app/views/site/edit.rhtml
index 5b1b8110818c75f17cb8f1aacb77fd7bfc605338..c5d40d9a8ee987ae30c3605a4b4880ccf561b3f5 100644 (file)
--- a/app/views/site/edit.rhtml
+++ b/app/views/site/edit.rhtml
@@ -47,6 +47,13 @@
 <script type="text/javascript">
   var brokenContentSize = $("content").offsetWidth == 0;
   var fo = new SWFObject("/potlatch/potlatch.swf?d="+Math.round(Math.random()*1000), "potlatch", "700", "600", "6", "#FFFFFF");
+  var changesaved = true;
+
+  window.onbeforeunload=function() {
+       if (!changesaved) {
+         return "You have unsaved changes. (To save in Potlatch, you should deselect the current way or point.)";
+       }
+  }
 
   function doSWF(lat,lon,sc) {
     if (sc < 11) sc = 11;

diff --git a/public/potlatch/potlatch.swf b/public/potlatch/potlatch.swf
index 49a8c68759759ff8d943ea1857cc40e721442c0a..c1b86227bb54f175e8979b9b35ccab1b5df78215 100755 (executable)
Binary files a/public/potlatch/potlatch.swf and b/public/potlatch/potlatch.swf differ