redmine_plugins/redmine_dmsf
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

#1413 Debug messages removed

Browse Source
This commit is contained in:
Karel Pičman 2022-12-16 10:17:34 +01:00
parent 7e62385ac3
commit a7375b8d1a
1 changed files with 1 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
lib/redmine_dmsf/webdav/dmsf_resource.rb
View File

@ -194,7 +194,6 @@ module RedmineDmsf
# <instance> should be of entity to be deleted, we simply follow the Dmsf entity method # <instance> should be of entity to be deleted, we simply follow the Dmsf entity method
# for deletion and return of appropriate status based on outcome. # for deletion and return of appropriate status based on outcome.
def delete def delete
Rails.logger.info ">>> DELETE #{@path}"
if file if file
raise Forbidden unless User.current.admin? || User.current.allowed_to?(:file_delete, project) raise Forbidden unless User.current.admin? || User.current.allowed_to?(:file_delete, project)
raise Forbidden unless (!parent.exist? || !parent.folder || DmsfFolder.permissions?(parent.folder, false)) raise Forbidden unless (!parent.exist? || !parent.folder || DmsfFolder.permissions?(parent.folder, false))
@ -217,7 +216,7 @@ module RedmineDmsf
end end
elsif folder elsif folder
raise Locked if folder.locked? raise Locked if folder.locked?
# To fullfil Litmus requirements to not delete folder if fragments are in the URL # To fulfill Litmus requirements to not delete folder if fragments are in the URL
uri = URI(uri_encode(request.get_header('REQUEST_URI'))) uri = URI(uri_encode(request.get_header('REQUEST_URI')))
raise BadRequest if uri.fragment.present? raise BadRequest if uri.fragment.present?
raise Forbidden unless User.current.admin? || User.current.allowed_to?(:folder_manipulation, project) raise Forbidden unless User.current.admin? || User.current.allowed_to?(:folder_manipulation, project)
@ -231,13 +230,6 @@ module RedmineDmsf
# Process incoming MOVE request # Process incoming MOVE request
# Behavioural differences between collection and single entity # Behavioural differences between collection and single entity
def move(dest_path, overwrite) def move(dest_path, overwrite)
# Don't allow editors like vi to create a temporary file during saving as they deletes it later including the
# document's history and saves a completely new document
Rails.logger.info ">>> MOVE #{@path} to #{dest_path}"
# if("#{@path}~" == dest_path)
# Rails.logger.info ">>> MethodNotAllowed"
# return MethodNotAllowed
# end
dest = ResourceProxy.new(dest_path, @request, @response, @options.merge(user: @user)) dest = ResourceProxy.new(dest_path, @request, @response, @options.merge(user: @user))
return PreconditionFailed if !dest.resource.is_a?(DmsfResource) || dest.resource.project.nil? return PreconditionFailed if !dest.resource.is_a?(DmsfResource) || dest.resource.project.nil?
parent = dest.resource.parent parent = dest.resource.parent
@ -436,7 +428,6 @@ module RedmineDmsf
# Lock # Lock
def lock(args) def lock(args)
Rails.logger.info ">>> LOCK #{@path}"
unless parent&.exist? unless parent&.exist?
e = Dav4rack::LockFailure.new e = Dav4rack::LockFailure.new
e.add_failure @path, Conflict e.add_failure @path, Conflict
@ -509,7 +500,6 @@ module RedmineDmsf
# Token based unlock (authenticated) will ensure that a correct token is sent, further ensuring # Token based unlock (authenticated) will ensure that a correct token is sent, further ensuring
# ownership of token before permitting unlock # ownership of token before permitting unlock
def unlock(token) def unlock(token)
Rails.logger.info ">>> UNLOCK #{@path}"
unless exist? unless exist?
return super(token) return super(token)
end end
@ -547,7 +537,6 @@ module RedmineDmsf
# HTTP PUT request. # HTTP PUT request.
def put(request, response) def put(request, response)
Rails.logger.info ">>> PUT #{@path}"
raise BadRequest if collection? raise BadRequest if collection?
raise Forbidden unless User.current.admin? || User.current.allowed_to?(:file_manipulation, project) raise Forbidden unless User.current.admin? || User.current.allowed_to?(:file_manipulation, project)
raise Forbidden unless (!parent.exist? || !parent.folder || DmsfFolder.permissions?(parent.folder, false)) raise Forbidden unless (!parent.exist? || !parent.folder || DmsfFolder.permissions?(parent.folder, false))