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,18 +2,17 @@
 
 module ProtectedBranchAccess
   extend ActiveSupport::Concern
-
-  included do
   include ProtectedRefAccess
 
+  included do
     belongs_to :protected_branch
 
     delegate :project, to: :protected_branch
+  end
 
   def check_access(user)
     return false if access_level == Gitlab::Access::NO_ACCESS
 
     super
   end
-  end
 end

app/models/concerns/protected_tag_access.rb

@@ -2,10 +2,9 @@
 
 module ProtectedTagAccess
   extend ActiveSupport::Concern
-
-  included do
   include ProtectedRefAccess
 
+  included do
     belongs_to :protected_tag
 
     delegate :project, to: :protected_tag