scinote-eln/scinote-web
1
1
Fork 0
mirror of https://github.com/scinote-eln/scinote-web.git synced 2024-12-25 01:03:18 +08:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #5248 from artoscinote/ma_SCI_8259

Browse source 
Properly copy visibility and public assignments when cloning protocol [SCI-8259]
This commit is contained in:
Alex Kriuchykhin 2023-04-04 11:31:21 +02:00 committed by GitHub
commit ed9f3d49d4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
app/models/protocol.rb
View file

@ -594,7 +594,9 @@ class Protocol < ApplicationRecord
last_modified_by: current_user,
team: team,
protocol_type: :in_repository_draft,
skip_user_assignments: true
skip_user_assignments: true,
visibility: visibility,
default_public_user_role_id: default_public_user_role_id
)
cloned = deep_clone(clone, current_user)
@ -693,6 +695,8 @@ class Protocol < ApplicationRecord
end
def update_automatic_user_assignments
return if skip_user_assignments
case visibility
when 'visible'
create_or_update_public_user_assignments!(added_by)