nocebo/diaspora
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

delete bad post visibilities before traversing the table

Browse source
This commit is contained in:
danielgrippi 2011-03-29 17:30:02 -07:00
parent aa4c20e404
commit 6a51c09a16
1 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
db/migrate/20110328202414_post_visibilities_on_contacts.rb
View file

@ -27,8 +27,6 @@ SQL
end end
def self.delete_duplicate_pvs def self.delete_duplicate_pvs
execute('DELETE FROM post_visibilities WHERE post_visibilities.contact_id = 0')
duplicate_rows = execute <<SQL duplicate_rows = execute <<SQL
SELECT COUNT(pv.contact_id), pv.contact_id, pv.post_id from post_visibilities AS pv SELECT COUNT(pv.contact_id), pv.contact_id, pv.post_id from post_visibilities AS pv
GROUP BY pv.contact_id, pv.post_id GROUP BY pv.contact_id, pv.post_id
@ -47,6 +45,15 @@ SQL
end end
end end
def self.delete_disconnected_pvs
execute <<SQL
DELETE post_visibilities FROM post_visibilities
LEFT OUTER JOIN posts ON post_visibilities.post_id = posts.id
LEFT OUTER JOIN aspects ON post_visibilities.aspect_id = aspects.id
WHERE aspects.id IS NULL OR posts.id IS NULL
SQL
end
def self.up def self.up
create_table :aspect_visibilities do |t| create_table :aspect_visibilities do |t|
t.integer :post_id, :null => false t.integer :post_id, :null => false
@ -59,6 +66,8 @@ SQL
add_foreign_key :aspect_visibilities, :aspects, :dependent => :delete add_foreign_key :aspect_visibilities, :aspects, :dependent => :delete
add_foreign_key :aspect_visibilities, :posts, :dependent => :delete add_foreign_key :aspect_visibilities, :posts, :dependent => :delete
delete_disconnected_pvs
add_column :post_visibilities, :contact_id, :integer, :null => false add_column :post_visibilities, :contact_id, :integer, :null => false
move_author_pvs_to_aspect_pvs move_author_pvs_to_aspect_pvs