Fix how we use EE::ProtectedRefAccess

This is a mess... Using prepend will give a different ancestors
chain we're not expecting. To fix this we'll need to know what
exactly methods we want to use in each classes using this module.

app/models/concerns/protected_branch_access.rb

@@ -2,19 +2,18 @@
 
 module ProtectedBranchAccess
   extend ActiveSupport::Concern
+  include ProtectedRefAccess
+  include EE::ProtectedRefAccess # Can't use prepend. It'll override wrongly
 
   included do
-    include ProtectedRefAccess
-    prepend EE::ProtectedRefAccess
-
     belongs_to :protected_branch
 
     delegate :project, to: :protected_branch
+  end
 
-    def check_access(user)
-      return false if access_level == Gitlab::Access::NO_ACCESS
+  def check_access(user)
+    return false if access_level == Gitlab::Access::NO_ACCESS
 
-      super
-    end
+    super
   end
 end

app/models/concerns/protected_tag_access.rb

@@ -2,11 +2,10 @@
 
 module ProtectedTagAccess
   extend ActiveSupport::Concern
+  include ProtectedRefAccess
+  include EE::ProtectedRefAccess # Can't use prepend. It'll override wrongly
 
   included do
-    include ProtectedRefAccess
-    prepend EE::ProtectedRefAccess
-
     belongs_to :protected_tag
 
     delegate :project, to: :protected_tag

ee/app/models/concerns/ee/protected_ref_access.rb

@@ -10,7 +10,7 @@ module EE
     extend ActiveSupport::Concern
     extend ::Gitlab::Utils::Override
 
-    prepended do
+    included do
       belongs_to :user
       belongs_to :group