Removed user checks from to_xml_node, as these should already be

[rails.git] / app / models / old_relation.rb

diff --git a/app/models/old_relation.rb b/app/models/old_relation.rb
index 34d76ff9c664fb122c955c749871501a1e755247..390c142dbc1b940773b8185b8e352f94f6b2dfe0 100644 (file)
--- a/app/models/old_relation.rb
+++ b/app/models/old_relation.rb
@@ -105,25 +105,21 @@ class OldRelation < ActiveRecord::Base
     el1['version'] = self.version.to_s
     el1['changeset'] = self.changeset_id.to_s
     
-    if self.redacted?
-      el1['redacted'] = self.redaction.id.to_s
+    el1['redacted'] = self.redaction.id.to_s if self.redacted?
+    
+    self.old_members.each do |member|
+      e = XML::Node.new 'member'
+      e['type'] = member.member_type.to_s.downcase
+      e['ref'] = member.member_id.to_s # "id" is considered uncool here as it should be unique in XML
+      e['role'] = member.member_role.to_s
+      el1 << e
     end
     
-    unless self.redacted? and (@user.nil? or not @user.moderator?)
-      self.old_members.each do |member|
-        e = XML::Node.new 'member'
-        e['type'] = member.member_type.to_s.downcase
-        e['ref'] = member.member_id.to_s # "id" is considered uncool here as it should be unique in XML
-        e['role'] = member.member_role.to_s
-        el1 << e
-      end
- 
-      self.old_tags.each do |tag|
-        e = XML::Node.new 'tag'
-        e['k'] = tag.k
-        e['v'] = tag.v
-        el1 << e
-      end
+    self.old_tags.each do |tag|
+      e = XML::Node.new 'tag'
+      e['k'] = tag.k
+      e['v'] = tag.v
+      el1 << e
     end
 
     return el1