X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/f435fce007d7817bd35f3beb93f0e3a844446154..2c9c427587a7cb42136315d6c1d533d3eb058545:/app/models/trace.rb?ds=inline diff --git a/app/models/trace.rb b/app/models/trace.rb index 2d055fd28..eed09dbe5 100644 --- a/app/models/trace.rb +++ b/app/models/trace.rb @@ -7,14 +7,14 @@ class Trace < ActiveRecord::Base scope :visible, -> { where(:visible => true) } scope :visible_to, ->(u) { visible.where("visibility IN ('public', 'identifiable') OR user_id = ?", u) } - scope :visible_to_all, -> { where(:visibility => %w(public identifiable)) } + scope :visible_to_all, -> { where(:visibility => %w[public identifiable]) } scope :tagged, ->(t) { joins(:tags).where(:gpx_file_tags => { :tag => t }) } validates :user, :presence => true, :associated => true validates :name, :presence => true, :length => 1..255 validates :description, :presence => { :on => :create }, :length => 1..255 validates :timestamp, :presence => true - validates :visibility, :inclusion => %w(private public trackable identifiable) + validates :visibility, :inclusion => %w[private public trackable identifiable] def destroy super @@ -29,7 +29,7 @@ class Trace < ActiveRecord::Base def tagstring=(s) self.tags = if s.include? "," - s.split(/\s*,\s*/).select { |tag| tag !~ /^\s*$/ }.collect do |tag| + s.split(/\s*,\s*/).reject { |tag| tag =~ /^\s*$/ }.collect do |tag| tt = Tracetag.new tt.tag = tag tt @@ -261,7 +261,7 @@ class Trace < ActiveRecord::Base first = true # If there are any existing points for this trace then delete them - Tracepoint.delete_all(:gpx_id => id) + Tracepoint.where(:gpx_id => id).delete_all gpx.points do |point| if first