Download as
Browse Code »

Commit a51e24d405a6f65df586e3373f5916e5cad1693f

Authored by Thiago Ribeiro
Committed by Rodrigo Souto
1 parent 1c90626d
Exists in master and in 17 other branches add_member_task_reject_details, admin_visible_profile, community_notifications, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, new_security, organization_ratings_improvements, profile_api_improvements, ratings_minor_fixes, send_email_to_admins, stable-spb-1.4, stable-spb-1.5, web_steps_improvements

Fixing migration 20151203121430 

Checking if comments is empty before execute delete
statement because the clause IN has to have a argument.

Signed-off-by: Gustavo Coelho <gust.rod.coelho@gmail.com>
Showing 1 changed file with 3 additions and 1 deletions   Show diff stats
plugins/organization_ratings/db/migrate/20151203121430_destroy_rejected_reports.rb
  Diff comments   View file @a51e24d
@@ -5,7 +5,9 @@ class DestroyRejectedReports &lt; ActiveRecord::Migration @@ -5,7 +5,9 @@ class DestroyRejectedReports &lt; ActiveRecord::Migration
5 settings = YAML.load(task['data']) 5 settings = YAML.load(task['data'])
6 comments << settings[:organization_rating_comment_id] 6 comments << settings[:organization_rating_comment_id]
7 end 7 end
8 - execute("DELETE FROM comments WHERE id IN (#{comments.join(',')})") if comments.present? 8 + if !comments.empty?
  9 + execute("DELETE FROM comments WHERE id IN (#{comments.join(',')})")
  10 + end
9 end 11 end
10 12
11 def down 13 def down