From aa0bc52a87e98eed2590ab3a5f95f55eb3968bc0 Mon Sep 17 00:00:00 2001 From: Toon Claes Date: Wed, 25 Jan 2017 11:12:09 +0100 Subject: `can?` already includes the `feature_available?` check `can?` already includes the `feature_available?` check, so no need to check this again. --- app/helpers/compare_helper.rb | 1 - 1 file changed, 1 deletion(-) diff --git a/app/helpers/compare_helper.rb b/app/helpers/compare_helper.rb index 8fcb82f17d5..2aa0449c46e 100644 --- a/app/helpers/compare_helper.rb +++ b/app/helpers/compare_helper.rb @@ -3,7 +3,6 @@ module CompareHelper from.present? && to.present? && from != to && - project.feature_available?(:merge_requests, current_user) && can?(current_user, :create_merge_request, project) && project.repository.branch_names.include?(from) && project.repository.branch_names.include?(to) -- cgit v1.2.1