Skip to content

Commit

Permalink
Merge pull request #2888 from sparc-request/sry-protocol_merge_to_inc…
Browse files Browse the repository at this point in the history
…lude_non_master_pro_number

SRY - Protocol merge to include non master pro number
  • Loading branch information
Stuart-Johnson committed May 29, 2022
2 parents db90437 + 88b69c9 commit c2abb1b
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions app/controllers/dashboard/protocol_merges_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,7 @@ def perform_protocol_merge
# checking for and assigning research types, impact areas, and affiliations...
if has_research?(@merged_protocol, 'human_subjects') && !has_research?(@master_protocol, 'human_subjects')
@merged_protocol.human_subjects_info.update_attributes(protocol_id: @master_protocol.id)
@master_protocol.research_types_info.update_attributes(human_subjects: true)
elsif has_research?(@merged_protocol, 'vertebrate_animals') && !has_research?(@master_protocol, 'vertebrate_animals')
@merged_protocol.vertebrate_animals_info.update_attributes(protocol_id: @master_protocol.id)
elsif has_research?(@merged_protocol, 'investigational_products') && !has_research?(@master_protocol, 'investigational_products')
Expand Down

0 comments on commit c2abb1b

Please sign in to comment.