discourse/db/migrate/20170602132735_fix_group_allow_membership_requests.rb

# frozen_string_literal: true

class FixGroupAllowMembershipRequests < ActiveRecord::Migration[4.2]
  def up
    execute <<~SQL
    UPDATE groups g
    SET allow_membership_requests = 'f'
    WHERE NOT EXISTS (SELECT 1 FROM group_users gu WHERE gu.owner = 't' AND gu.group_id = g.id LIMIT 1)
    SQL
  end

  def down
    raise ActiveRecord::IrreversibleMigration
  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 06:17:27 +08: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 FixGroupAllowMembershipRequests < ActiveRecord::Migration[4.2]`
FIX: Groups that do not have any owners should not allow membership requests. 2017-06-02 22:15:57 +08:00			`def up`
			`execute <<~SQL`
			`UPDATE groups g`
			`SET allow_membership_requests = 'f'`
			`WHERE NOT EXISTS (SELECT 1 FROM group_users gu WHERE gu.owner = 't' AND gu.group_id = g.id LIMIT 1)`
			`SQL`
			`end`

			`def down`
			`raise ActiveRecord::IrreversibleMigration`
			`end`
			`end`