Dropping duplicate GPS points is very expensive and rarely finds

[rails.git] / app / controllers / swf_controller.rb

diff --git a/app/controllers/swf_controller.rb b/app/controllers/swf_controller.rb
index ce185b59a2de08c1778975831fb53883ed2bc9a2..b576aae5c7c3242faba92f2fb7b5aa9d962a1be0 100644 (file)
--- a/app/controllers/swf_controller.rb
+++ b/app/controllers/swf_controller.rb
@@ -1,4 +1,6 @@
 class SwfController < ApplicationController
+       session :off
+       before_filter :check_availability
 
 # to log:
 # RAILS_DEFAULT_LOGGER.error("Args: #{args[0]}, #{args[1]}, #{args[2]}, #{args[3]}")
@@ -18,10 +20,10 @@ class SwfController < ApplicationController
                basey           =params['basey'].to_f
                masterscale     =params['masterscale'].to_f
        
-               xmin=params['xmin'].to_f; xminr=xmin/0.000001
-               xmax=params['xmax'].to_f; xmaxr=xmax/0.000001
-               ymin=params['ymin'].to_f; yminr=ymin/0.000001
-               ymax=params['ymax'].to_f; ymaxr=ymax/0.000001
+               xmin=params['xmin'].to_f;
+               xmax=params['xmax'].to_f;
+               ymin=params['ymin'].to_f;
+               ymax=params['ymax'].to_f;
        
                # -     Begin movie
        
@@ -44,22 +46,19 @@ class SwfController < ApplicationController
                lastfile='-1'
        
                if params['token']
-                       token=sqlescape(params['token'])
+                        user=User.authenticate(:token => params[:token])
                        sql="SELECT gps_points.latitude*0.000001 AS lat,gps_points.longitude*0.000001 AS lon,gpx_files.id AS fileid,UNIX_TIMESTAMP(gps_points.timestamp) AS ts "+
-                                " FROM gpx_files,gps_points,users "+
+                                " FROM gpx_files,gps_points "+
                                 "WHERE gpx_files.id=gpx_id "+
-                                "  AND gpx_files.user_id=users.id "+
-                                "  AND token='#{token}' "+
-                                "  AND (gps_points.longitude BETWEEN #{xminr} AND #{xmaxr}) "+
-                                "  AND (gps_points.latitude BETWEEN #{yminr} AND #{ymaxr}) "+
+                                "  AND gpx_files.user_id=#{user.id} "+
+                                "  AND "+OSM.sql_for_area(ymin,xmin,ymax,xmax)+
                                 "  AND (gps_points.timestamp IS NOT NULL) "+
                                 "ORDER BY fileid DESC,ts "+
                                 "LIMIT 10000"
                else
                        sql="SELECT latitude*0.000001 AS lat,longitude*0.000001 AS lon,gpx_id AS fileid,UNIX_TIMESTAMP(timestamp) AS ts "+
                                 " FROM gps_points "+
-                                "WHERE (longitude BETWEEN #{xminr} AND #{xmaxr}) "+
-                                "  AND (latitude  BETWEEN #{yminr} AND #{ymaxr}) "+
+                                "WHERE "+OSM.sql_for_area(ymin,xmin,ymax,xmax)+
                                 "  AND (gps_points.timestamp IS NOT NULL) "+
                                 "ORDER BY fileid DESC,ts "+
                                 "LIMIT 10000"
@@ -90,8 +89,6 @@ class SwfController < ApplicationController
                # - Draw unwayed segments
                
                if params['unwayed']=='true'
-                       lastx=lasty=-999999
-       
                        sql="SELECT cn1.latitude AS lat1,cn1.longitude AS lon1,"+
                                "               cn2.latitude AS lat2,cn2.longitude AS lon2 "+
                                "  FROM current_segments "+
@@ -101,25 +98,25 @@ class SwfController < ApplicationController
                                " WHERE (cn1.longitude BETWEEN #{xmin} AND #{xmax})"+
                                "   AND (cn1.latitude  BETWEEN #{ymin} AND #{ymax})"+
                                "   AND segment_id IS NULL"+
+                               "   AND current_segments.visible=1"+
                                "   AND cn1.id=node_a AND cn1.visible=1"+
                                "   AND cn2.id=node_b AND cn2.visible=1"
                        seglist=ActiveRecord::Base.connection.select_all sql
                        
                        seglist.each do |row|
-                               xs1=long2coord(row['lon1'].to_f); ys1=lat2coord(row['lat1'].to_f)
-                               xs2=long2coord(row['lon2'].to_f); ys2=lat2coord(row['lat2'].to_f)
-                               if (xs1==lastx and ys1==lasty)
-                                       b+=drawTo(xs2*20, ys2*20)
-                                       lastx=xs2; lasty=ys2
-                               elsif (xs2==lastx and ys2==lasty)
-                                       b+=drawTo(xs1*20, ys1*20)
-                                       lastx=xs1; lasty=ys1
+                               xs1=(long2coord(row['lon1'].to_f,baselong,masterscale)*20).floor; ys1=(lat2coord(row['lat1'].to_f,basey,masterscale)*20).floor
+                               xs2=(long2coord(row['lon2'].to_f,baselong,masterscale)*20).floor; ys2=(lat2coord(row['lat2'].to_f,basey,masterscale)*20).floor
+                               if (xs1==absx and ys1==absy)
+                                       b+=drawTo(absx,absy,xs2,ys2)
+                                       absx=xs2; absy=ys2
+                               elsif (xs2==absx and ys2==absy)
+                                       b+=drawTo(absx,absy,xs1,ys1)
+                                       absx=xs1; absy=ys1
                                else
-                                       b+=startAndMove(xs1*20,ys1*20,'10')
-                                       b+=drawTo(xs2*20, ys2*20)
-                                       lastx=xs2; lasty=ys2
+                                       b+=startAndMove(xs1,ys1,'10')
+                                       b+=drawTo(xs1,ys1,xs2,ys2)
+                                       absx=xs2; absy=ys2
                                end
-                               absx=lastx.floor; absy=lasty.floor
                                while b.length>80 do
                                        r+=[b.slice!(0...80)].pack("B*")
                                end
@@ -141,8 +138,7 @@ class SwfController < ApplicationController
                m=packRect(bounds_left,bounds_right,bounds_bottom,bounds_top) + 0.chr + 12.chr + packUI16(1) + m
                m='FWS' + 6.chr + packUI32(m.length+8) + m
        
-               response.headers["Content-Type"]="application/x-shockwave-flash"
-               render :text=>m
+               render :text => m, :content_type => "application/x-shockwave-flash"
        end
 
        private