Download as
Browse Code »

Commit 617f3a2f942fdd157ed1f05642bbcd8042dd6340

Authored by André Guedes
Committed by Rodrigo Souto
1 parent a710bcc7
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

Changed migration to use find_each batch update

Showing 1 changed file with 7 additions and 4 deletions   Show diff stats
db/migrate/20150319114233_change_default_content_privacy.rb
  Diff comments   View file @617f3a2
@@ -3,10 +3,13 @@ class ChangeDefaultContentPrivacy < ActiveRecord::Migration @@ -3,10 +3,13 @@ class ChangeDefaultContentPrivacy < ActiveRecord::Migration
3 ids = execute("SELECT id from profiles WHERE NOT public_profile") 3 ids = execute("SELECT id from profiles WHERE NOT public_profile")
4 profiles_ids = ids.map { |p| p["id"] } 4 profiles_ids = ids.map { |p| p["id"] }
5 unless ids.num_tuples.zero? 5 unless ids.num_tuples.zero?
6 - execute("UPDATE articles SET published = (1>2), show_to_followers = (1=1)  
7 - WHERE profile_id IN (#{profiles_ids.join(",")}) AND published")  
8 - blocks = Block.includes(:box).where(:boxes => {:owner_type => "Profile", :owner_id => profiles_ids}).where('type != ?', "MainBlock")  
9 - blocks.each do |block| 6 + execute('UPDATE articles SET published = (1>2), show_to_followers = (1=1)
  7 + FROM articles AS a INNER JOIN profiles ON a.profile_id = profiles.id
  8 + WHERE NOT profiles.public_profile AND articles.id = a.id AND a.published = (1=1)')
  9 + Block.includes(:box).where(
  10 + :boxes => {:owner_type => "Profile",
  11 + :owner_id => profiles_ids}).where(
  12 + 'type != ?', "MainBlock").find_each do |block|
10 block.display_user = 'followers' 13 block.display_user = 'followers'
11 block.save 14 block.save
12 end 15 end