user_display_name_cache = {} if user_display_name_cache.nil?
- if user_display_name_cache and user_display_name_cache[self.user_id]
+ if user_display_name_cache and user_display_name_cache.key?(self.user_id)
# use the cache if available
elsif self.user.data_public?
user_display_name_cache[self.user_id] = self.user.display_name
WaySegment.delete_all(['id = ?', self.id])
- i = 0
+ i = 1
segs.each do |n|
seg = WaySegment.new
seg.id = self.id
def preconditions_ok?
return false if self.segs.empty?
self.segs.each do |n|
- segment = Segment.find(n)
+ segment = Segment.find(:first, :conditions => ["id = ?", n])
unless segment and segment.visible and segment.preconditions_ok?
return false
end