• Class name "Missions_user" should match pattern /^[A-Z][a-zA-Z0-9]*$/ » roodi
 
1class Missions_user <ActiveRecord:: Base
  
2
  
3    validates :mission_id, presence: true
  
4    validates :user_id, presence: true
  
5
  
6    validate :mission_exist, on: :create
  
7    validate :user_exist, on: :create
  
8
  
 9    private
  • Complexity 2 » saikuro
  • NilCheck - performs a nil-check » reek
 
10        def mission_exist
  
11
  
12            mission = Mission.find_by(id: mission_id)
  
13            if mission == nil
  
14                errors[:mission] = "mission belong to Missions_user is not exist"
  
15            end
  
16
  
17
  
18        end
  
19
  • Complexity 2 » saikuro
  • NilCheck - performs a nil-check » reek
 
20        def user_exist
  
21
  
22            user =  User.find_by(id: user_id)
  
23            if user == nil
  
24                errors[:user] = "user belong to Missions_user is not exist"
  
25            end
  
26
  
27        end
  
28
  
29
  
30end