# See http://doc.gitlab.com/ce/development/migration_style_guide.html
# for more information on how to write migrations for GitLab.

class ChangePersonalAccessTokensDefaultBackToEmptyArray < ActiveRecord::Migration
  include Gitlab::Database::MigrationHelpers

  # Set this constant to true if this migration requires downtime.
  DOWNTIME = false

  # When a migration requires downtime you **must** uncomment the following
  # constant and define a short and easy to understand explanation as to why the
  # migration requires downtime.
  # DOWNTIME_REASON = ''

  # When using the methods "add_concurrent_index" or "add_column_with_default"
  # you must disable the use of transactions as these methods can not run in an
  # existing transaction. When using "add_concurrent_index" make sure that this
  # method is the _only_ method called in the migration, any other changes
  # should go in a separate migration. This ensures that upon failure _only_ the
  # index creation fails and can be retried or reverted easily.
  #
  # To disable transactions uncomment the following line and remove these
  # comments:
  # disable_ddl_transaction!

  def up
    # The default needs to be `[]`, but all existing access tokens need to have `scopes` set to `['api']`.
    # It's easier to achieve this by adding the column with the `['api']` default, and then changing the default to
    # `[]`.
    change_column_default :personal_access_tokens, :scopes, [].to_yaml
  end

  def down
    # The default needs to be `[]`, but all existing access tokens need to have `scopes` set to `['api']`.
    # It's easier to achieve this by adding the column with the `['api']` default, and then changing the default to
    # `[]`.
    change_column_default :personal_access_tokens, :scopes, ['api'].to_yaml
  end
end