discourse/db/migrate/20170831180419_remove_whisper_topic_links.rb

# frozen_string_literal: true

class RemoveWhisperTopicLinks < ActiveRecord::Migration[4.2]
  def change
    execute <<-SQL
      DELETE FROM topic_links
       USING topic_links tl
   LEFT JOIN posts p ON p.id = tl.post_id
       WHERE p.post_type = 4
         AND topic_links.id = tl.id
    SQL
  end
end
DEV: enable frozen string literal on all files This reduces chances of errors where consumers of strings mutate inputs and reduces memory usage of the app. Test suite passes now, but there may be some stuff left, so we will run a few sites on a branch prior to merging 2019-05-03 08:17:27 +10:00			`# frozen_string_literal: true`

Fix all the errors to get our tests green on Rails 5.1. 2017-08-31 12:06:56 +08:00			`class RemoveWhisperTopicLinks < ActiveRecord::Migration[4.2]`
SECURITY: do not include links from whispers in topic summary map https://meta.discourse.org/t/staff-whispers-links-in-whispers-showing-up-publicly-in-topics-summary/69134?u=techapj 2017-08-31 23:44:54 +05:30			`def change`
			`execute <<-SQL`
			`DELETE FROM topic_links`
			`USING topic_links tl`
			`LEFT JOIN posts p ON p.id = tl.post_id`
			`WHERE p.post_type = 4`
			`AND topic_links.id = tl.id`
			`SQL`
			`end`
			`end`