scinote-eln/scinote-web
1
1
Fork 0
mirror of https://github.com/scinote-eln/scinote-web.git synced 2025-09-16 01:54:34 +08:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #5215 from artoscinote/ma_SCI_8212

Browse source 
Properly handle repository snapshots in reports [SCI-8212]
This commit is contained in:
Alex Kriuchykhin 2023-03-28 10:39:46 +02:00 • committed by GitHub
commit e3a0f3c55f
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/services/report_actions/report_content.rb
View file

@ -82,7 +82,7 @@ module ReportActions
my_module_element = save_element!({ 'my_module_id' => my_module.id }, :my_module, experiment_element)
my_module.live_and_snapshot_repositories_list.each do |repository|
next unless @repositories.include?(repository.id)
next unless @repositories.include?(repository.parent_id || repository.id)
save_element!(
{ 'my_module_id' => my_module.id, 'repository_id' => repository.id },

2
app/views/reports/wizard/_third_step.html.erb
View file

@ -71,7 +71,7 @@
<span class="sci-checkbox-container">
<input type="checkbox"
class="sci-checkbox repositories-setting"
value="<%= repository.id %>"
value="<%= repository.is_a?(RepositorySnapshot) ? repository.parent_id : repository.id %>"
<%= 'checked' if report.new_record? ||
@project_contents[:repositories].include?(repository.id) ||
(repository.is_a?(Repository) && repository.repository_snapshots.exists?(id: @project_contents[:repositories])) %> />