Commit 58bc0be3 authored by Etienne Baqué's avatar Etienne Baqué

Merge branch 'move-membership-state-constants' into 'master'

Move Member State Constants from EE to FOSS

See merge request gitlab-org/gitlab!77132
parents e075c701 532348cb
...@@ -18,6 +18,10 @@ class Member < ApplicationRecord ...@@ -18,6 +18,10 @@ class Member < ApplicationRecord
AVATAR_SIZE = 40 AVATAR_SIZE = 40
ACCESS_REQUEST_APPROVERS_TO_BE_NOTIFIED_LIMIT = 10 ACCESS_REQUEST_APPROVERS_TO_BE_NOTIFIED_LIMIT = 10
STATE_CREATED = 0
STATE_AWAITING = 1
STATE_ACTIVE = 2
attr_accessor :raw_invite_token attr_accessor :raw_invite_token
belongs_to :created_by, class_name: "User" belongs_to :created_by, class_name: "User"
......
...@@ -6,10 +6,6 @@ module EE ...@@ -6,10 +6,6 @@ module EE
extend ::Gitlab::Utils::Override extend ::Gitlab::Utils::Override
prepended do prepended do
STATE_CREATED = 0
STATE_AWAITING = 1
STATE_ACTIVE = 2
state_machine :state, initial: :created do state_machine :state, initial: :created do
event :wait do event :wait do
transition created: :awaiting transition created: :awaiting
...@@ -21,12 +17,12 @@ module EE ...@@ -21,12 +17,12 @@ module EE
transition awaiting: :active transition awaiting: :active
end end
state :created, value: STATE_CREATED state :created, value: ::Member::STATE_CREATED
state :awaiting, value: STATE_AWAITING state :awaiting, value: ::Member::STATE_AWAITING
state :active, value: STATE_ACTIVE state :active, value: ::Member::STATE_ACTIVE
end end
scope :awaiting, -> {where(state: STATE_AWAITING)} scope :awaiting, -> {where(state: ::Member::STATE_AWAITING)}
scope :non_awaiting, -> {where.not(state: STATE_AWAITING)} scope :non_awaiting, -> {where.not(state: ::Member::STATE_AWAITING)}
before_create :set_membership_activation before_create :set_membership_activation
...@@ -154,7 +150,7 @@ module EE ...@@ -154,7 +150,7 @@ module EE
def set_membership_activation def set_membership_activation
return unless group && ::Feature.enabled?(:saas_user_caps, group.root_ancestor, default_enabled: :yaml) return unless group && ::Feature.enabled?(:saas_user_caps, group.root_ancestor, default_enabled: :yaml)
self.state = group.user_cap_reached? ? STATE_AWAITING : STATE_ACTIVE self.state = group.user_cap_reached? ? ::Member::STATE_AWAITING : ::Member::STATE_ACTIVE
end end
end end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment