Handle old reports with no templates properly [SCI-5810]

This commit is contained in:
Martin Artnik 2021-06-14 10:53:46 +02:00
parent dc72bfc527
commit 20484cfe5c
3 changed files with 3 additions and 3 deletions

View file

@ -46,7 +46,7 @@
<%= t("projects.reports.wizard.statuses.step_#{i + 1}") %>
</span>
</div>
<% if @edit %>
<% if @edit && @report.settings["template"].present? %>
<div class="change-step" data-step-id="<%= i + 1 %>">
</div>
<% end %>

View file

@ -1,5 +1,5 @@
<div class="project-selector-container">
<%= render partial: 'reports/wizard/project_template_selector', locals: {report: report} %>
</div>
<div class="report-template-values-container <%= 'hidden' if report.new_record? %>">
<div class="report-template-values-container <%= 'hidden' if report.settings["template"].blank? %>">
</div>

View file

@ -17,7 +17,7 @@
options_for_select(@templates.invert, @active_template),
prompt: t('projects.reports.wizard.first_step.select_template'),
data: {
disable_on_load: report.settings[:template].blank?,
disable_on_load: report.settings[:template].blank? && report.new_record?,
placeholder: t('projects.reports.wizard.first_step.select_template'),
selected_template: report.settings[:template],
values_editor_path: reports_new_template_values_path(report_id: report.id)