List is a model now

Author: amatsuda

app/controllers/messages_controller.rb

@@ -6,7 +6,7 @@ def index
     if (list_name = params[:list_name])
       @list = List.find_by_name list_name
 
-      messages = Message.with_recursive(parent_and_children: [Message.where(list_id: @list.id, parent_id: nil).order(:id).limit(100), Message.joins('inner join parent_and_children on messages.parent_id = parent_and_children.id')])
+      messages = Message.with_recursive(parent_and_children: [Message.where(list_id: @list, parent_id: nil).order(:id).limit(100), Message.joins('inner join parent_and_children on messages.parent_id = parent_and_children.id')])
         .joins('inner join parent_and_children on parent_and_children.id = messages.id')
       @messages = compose_tree(messages)
     elsif (query = params[:q])
@@ -23,7 +23,7 @@ def index
   # GET /messages/ruby-dev/1
   def show
     @list = List.find_by_name(params[:list_name])
-    @message = Message.find_by(list_id: @list.id, list_seq: params[:list_seq])
+    @message = Message.find_by(list_id: @list, list_seq: params[:list_seq])
   end
 
   private

app/models/message.rb

@@ -48,13 +48,13 @@ def from_mail(mail, list, list_seq)
 
     # mail.in_reply_to returns strange Array object in some cases (?), so let's use the raw value
     parent_message_id_header = extract_message_id_from_in_reply_to(mail.header[:in_reply_to]&.value)
-    self.parent_id = Message.where(list_id: list.id, message_id_header: parent_message_id_header).pick(:id) if parent_message_id_header
+    self.parent_id = Message.where(list_id: list, message_id_header: parent_message_id_header).pick(:id) if parent_message_id_header
     if !self.parent_id && (String === mail.references)
-      self.parent_id = Message.where(list_id: list.id, message_id_header: mail.references).pick(:id)
+      self.parent_id = Message.where(list_id: list, message_id_header: mail.references).pick(:id)
     end
     if !self.parent_id && (Array === mail.references)
       mail.references.compact.each do |ref|
-        break if (self.parent_id = Message.where(list_id: list.id, message_id_header: ref).pick(:id))
+        break if (self.parent_id = Message.where(list_id: list, message_id_header: ref).pick(:id))
       end
     end