Added message length validity (1..8000) and rewrote 'target' method

app/mailers/redstoner_mailer.rb

@@ -48,6 +48,6 @@ class RedstonerMailer < ActionMailer::Base
   def new_message_mail(user, message)
     @user = user
     @message = message
-    mail(to: @user.email, subject: "#{message.user_sender.name} sent you a new message.")
+    mail(to: @user.email, subject: "#{message.user_sender.name} sent you a new message")
   end
 end

app/models/message.rb

@@ -7,6 +7,8 @@ class Message < ActiveRecord::Base
 
   validates_presence_of :user_sender, :user_target, :text, on: :create
 
+  validates_length_of :text, in: 1..8000
+
   def sender
     @sender ||= if self.user_sender.present?
       user_sender
@@ -16,8 +18,11 @@ class Message < ActiveRecord::Base
   end
 
   def target
-    # can be nil
+    @target ||= if self.user_target.present?
-    @target ||= user_target
+      user_target
+    else
+      User.first
+    end
   end
 
   def send_new_message_mail