`can?` already includes the `feature_available?` check
`can?` already includes the `feature_available?` check, so no need to check this again.
This commit is contained in:
parent
eaffe8cd72
commit
aa0bc52a87
|
@ -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)
|
||||
|
|
Loading…
Reference in New Issue