Add a message when unable to save an object through api.

lib/api/commits.rb

@@ -108,7 +108,7 @@ module API
         if note.save
           present note, with: Entities::CommitNote
         else
-          render_api_error!("Failed to save note #{note.errors.messages}", 422)
+          render_api_error!("Failed to save note #{note.errors.messages}", 400)
         end
       end
     end

lib/api/deploy_keys.rb

@@ -58,7 +58,7 @@ module API
         if key.valid? && user_project.deploy_keys << key
           present key, with: Entities::SSHKey
         else
-          render_validation_error!(key)
+          render_api_error!("Failed to add key #{key.errors.messages}", 400)
         end
       end
 

lib/api/groups.rb

@@ -54,7 +54,7 @@ module API
         if @group.save
           present @group, with: Entities::Group
         else
-          render_api_error!("Failed to save group #{@group.errors.messages}", 422)
+          render_api_error!("Failed to save group #{@group.errors.messages}", 400)
         end
       end
 
@@ -97,7 +97,7 @@ module API
         if result
           present group
         else
-          render_api_error!("Failed to transfer project #{project.errors.messages}", 422)
+          render_api_error!("Failed to transfer project #{project.errors.messages}", 400)
         end
       end
     end

lib/api/issues.rb

@@ -104,7 +104,7 @@ module API
 
         # Validate label names in advance
         if (errors = validate_label_params(params)).any?
-          render_api_error!({ labels: errors }, 400)
+          render_api_error!("Unable to validate label: #{errors}"}, 400)
         end
 
         issue = ::Issues::CreateService.new(user_project, current_user, attrs).execute
@@ -118,7 +118,7 @@ module API
 
           present issue, with: Entities::Issue
         else
-          render_validation_error!(issue)
+          render_api_error!("Unable to create issue #{issue.errors.messages}", 400)
         end
       end
 
@@ -142,7 +142,7 @@ module API
 
         # Validate label names in advance
         if (errors = validate_label_params(params)).any?
-          render_api_error!({ labels: errors }, 400)
+          render_api_error!("Unable to validate label: #{errors}"}, 400)
         end
 
         issue = ::Issues::UpdateService.new(user_project, current_user, attrs).execute(issue)
@@ -158,7 +158,7 @@ module API
 
           present issue, with: Entities::Issue
         else
-          render_validation_error!(issue)
+          render_api_error!("Unable to update issue #{issue.errors.messages}", 400)
         end
       end
 

lib/api/labels.rb

@@ -37,7 +37,7 @@ module API
         if label.valid?
           present label, with: Entities::Label
         else
-          render_validation_error!(label)
+          render_api_error!("Unable to create label #{label.errors.messages}", 400)
         end
       end
 
@@ -90,7 +90,7 @@ module API
         if label.update(attrs)
           present label, with: Entities::Label
         else
-          render_validation_error!(label)
+          render_api_error!("Unable to create label #{label.errors.messages}", 400)
         end
       end
     end

lib/api/merge_requests.rb

@@ -137,7 +137,7 @@ module API
 
         # Validate label names in advance
         if (errors = validate_label_params(params)).any?
-          render_api_error!({ labels: errors }, 400)
+          render_api_error!("Unable to validate label: #{errors}"}, 400)
         end
 
         merge_request = ::MergeRequests::UpdateService.new(user_project, current_user, attrs).execute(merge_request)
@@ -233,7 +233,7 @@ module API
         if note.save
           present note, with: Entities::MRNote
         else
-          render_validation_error!(note)
+          render_api_error!("Failed to save note #{note.errors.messages}", 400)
         end
       end
     end

lib/api/milestones.rb

@@ -48,7 +48,7 @@ module API
         if milestone.valid?
           present milestone, with: Entities::Milestone
         else
-          not_found!("Milestone #{milestone.errors.messages}")
+          render_api_error!("Failed to create milestone #{milestone.errors.messages}", 400)
         end
       end
 
@@ -72,7 +72,7 @@ module API
         if milestone.valid?
           present milestone, with: Entities::Milestone
         else
-          not_found!("Milestone #{milestone.errors.messages}")
+          render_api_error!("Failed to update milestone #{milestone.errors.messages}", 400)
         end
       end
     end

lib/api/notes.rb

@@ -93,7 +93,7 @@ module API
           if @note.valid?
             present @note, with: Entities::Note
           else
-            bad_request!('Invalid note')
+            render_api_error!("Failed to save note #{note.errors.messages}", 400)
           end
         end