gitlab-org--gitlab-foss/db/migrate/20160307221555_disallow_bla...

# rubocop:disable all
class DisallowBlankLineCodeOnNote < ActiveRecord::Migration
  def up
    execute("UPDATE notes SET line_code = NULL WHERE line_code = ''")
  end

  def down
    # noop
  end
end
Enable RuboCop for migrations Migrations shouldn't fail RuboCop checks - especially lint checks, such as the nested method check. To avoid changing code in existing migrations, add the magic comment to the top of each of them to skip that file. 2016-06-09 12:39:16 +00:00			`# rubocop:disable all`
Disallow blank (non-null) values for a Note's `line_code` attribute It's unclear how these blank values got added, but GitLab.com had a few: ``` irb(main):002:0> Note.where("line_code IS NOT NULL AND line_code = ''").count => 439 ``` We've added a migration to convert any existing records to use a NULL value when blank, and updated Note to set blank values to nil before validation. 2016-03-11 22:46:50 +00:00			`class DisallowBlankLineCodeOnNote < ActiveRecord::Migration`
			`def up`
			`execute("UPDATE notes SET line_code = NULL WHERE line_code = ''")`
			`end`

			`def down`
			`# noop`
			`end`
			`end`