el1['visible'] = self.visible.to_s
el1['timestamp'] = self.timestamp.xmlschema
+ user_display_name_cache = {} if user_display_name_cache.nil?
+
if user_display_name_cache and user_display_name_cache[self.user_id]
# use the cache if available
- else
+ elsif self.user.data_public?
user_display_name_cache[self.user_id] = self.user.display_name
+ else
+ user_display_name_cache[self.user_id] = nil
end
-
- #el1['user'] = self.user.display_name if self.user.data_public?
- el1['user'] = user_display_name_cache[self.user_id]
+
+ el1['user'] = user_display_name_cache[self.user_id] unless user_display_name_cache[self.user_id].nil?
# make sure segments are output in sequence_id order
ordered_segments = []
end
ordered_segments.each do |seg_id|
- if seg_id
+ if seg_id and seg_id != '0'
e = XML::Node.new 'seg'
e['id'] = seg_id
el1 << e