Memoize feature available

app/models/ee/namespace.rb

@@ -33,13 +33,19 @@ module EE
     # for a given Namespace plan. This method should consider ancestor groups
     # being licensed.
     def feature_available?(feature)
-      globally_available = License.feature_available?(feature)
+      @feature_available ||= Hash.new do |h, feature|
+        h[feature] = load_feature_available(feature)
+      end
 
-      if current_application_settings.should_check_namespace_plan?
-        globally_available && feature_available_in_plan?(feature)
-      else
-        globally_available
+      @feature_available[feature]
     end
+
+    def feature_available_in_plan?(feature)
+      @features_available_in_plan ||= Hash.new do |h, feature|
+        h[feature] = plans.any? { |plan| License.plan_includes_feature?(EE_PLANS[plan], feature) }
+      end
+
+      @features_available_in_plan[feature]
     end
 
     def actual_shared_runners_minutes_limit
@@ -59,6 +65,16 @@ module EE
 
     private
 
+    def load_feature_available(feature)
+      globally_available = License.feature_available?(feature)
+
+      if current_application_settings.should_check_namespace_plan?
+        globally_available && feature_available_in_plan?(feature)
+      else
+        globally_available
+      end
+    end
+
     def plans
       @ancestors_plans ||=
         if parent_id
@@ -67,13 +83,5 @@ module EE
           [plan]
         end
     end
-
-    def feature_available_in_plan?(feature)
-      @features_available_in_plan ||= Hash.new do |h, feature|
-        h[feature] = plans.any? { |plan| License.plan_includes_feature?(EE_PLANS[plan], feature) }
-      end
-
-      @features_available_in_plan[feature]
-    end
   end
 end

app/models/ee/project.rb

@@ -344,11 +344,19 @@ module EE
     private
 
     def licensed_feature_available?(feature)
+      @licensed_feature_available ||= Hash.new do |h, feature|
+        h[feature] = load_licensed_feature_available(feature)
+      end
+
+      @licensed_feature_available[feature]
+    end
+
+    def load_licensed_feature_available(feature)
       globally_available = License.feature_available?(feature)
 
       if current_application_settings.should_check_namespace_plan?
         globally_available &&
-          (public? && namespace.public? || namespace.feature_available?(feature))
+          (public? && namespace.public? || namespace.feature_available_in_plan?(feature))
       else
         globally_available
       end

spec/models/ee/namespace_spec.rb

@@ -59,6 +59,12 @@ describe Namespace, models: true do
       is_expected.to be_truthy
     end
 
+    it 'only checks the plan once' do
+      expect(group).to receive(:load_feature_available).once.and_call_original
+
+      2.times { group.feature_available?(:service_desk) }
+    end
+
     context 'when checking namespace plan' do
       before do
         stub_application_setting_on_object(group, should_check_namespace_plan: true)

spec/models/ee/project_spec.rb

@@ -99,6 +99,13 @@ describe Project, models: true do
       end
     end
 
+    it 'only loads licensed availability once' do
+      expect(project).to receive(:load_licensed_feature_available)
+                             .once.and_call_original
+
+      2.times { project.feature_available?(:service_desk) }
+    end
+
     context 'when feature symbol is not included on Namespace features code' do
       let(:feature) { :issues }