cleanup policy (methods) no longer used

pull/10894/head
ulferts 2 years ago committed by Oliver Günther
parent d3e32f8c2c
commit e25d2f14ee
  1. 53
      app/policies/version_policy.rb
  2. 9
      app/policies/work_package_policy.rb

@ -1,53 +0,0 @@
#-- copyright
# OpenProject is an open source project management software.
# Copyright (C) 2012-2022 the OpenProject GmbH
#
# This program is free software; you can redistribute it and/or
# modify it under the terms of the GNU General Public License version 3.
#
# OpenProject is a fork of ChiliProject, which is a fork of Redmine. The copyright follows:
# Copyright (C) 2006-2013 Jean-Philippe Lang
# Copyright (C) 2010-2013 the ChiliProject Team
#
# This program is free software; you can redistribute it and/or
# modify it under the terms of the GNU General Public License
# as published by the Free Software Foundation; either version 2
# of the License, or (at your option) any later version.
#
# This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program; if not, write to the Free Software
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
#
# See COPYRIGHT and LICENSE files for more details.
#++
class VersionPolicy < BasePolicy
private
def cache(version)
@cache ||= Hash.new do |hash, cached_version|
hash[cached_version] = {
show: show_allowed?(cached_version)
}
end
@cache[version]
end
def show_allowed?(version)
@show_cache ||= Hash.new do |hash, queried_version|
permissions = %i[view_work_packages manage_versions]
hash[queried_version] = permissions.any? do |permission|
queried_version.projects.allowed_to(user, permission).exists?
end
end
@show_cache[version]
end
end

@ -45,7 +45,6 @@ class WorkPackagePolicy < BasePolicy
# copying multiple work packages.
{
edit: edit_allowed?(work_package),
log_time: log_time_allowed?(work_package),
move: move_allowed?(work_package),
copy: move_allowed?(work_package),
duplicate: copy_allowed?(work_package), # duplicating is another form of copying
@ -64,14 +63,6 @@ class WorkPackagePolicy < BasePolicy
@edit_cache[work_package.project]
end
def log_time_allowed?(work_package)
@log_time_cache ||= Hash.new do |hash, project|
hash[project] = user.allowed_to?(:log_time, project)
end
@log_time_cache[work_package.project]
end
def move_allowed?(work_package)
@move_cache ||= Hash.new do |hash, project|
hash[project] = user.allowed_to?(:move_work_packages, project)

Loading…
Cancel
Save