From 350127fdf1a5c03ac269853be63ef75d2377c0e8 Mon Sep 17 00:00:00 2001 From: Bohung Date: Tue, 14 Jun 2022 22:21:43 +0800 Subject: [PATCH] Fix bug. --- app/controllers/application_forms_controller.rb | 9 +++------ app/views/application_forms/agree_link.html.erb | 12 +++++++++--- config/locales/en.yml | 1 + config/locales/zh_tw.yml | 1 + 4 files changed, 14 insertions(+), 9 deletions(-) diff --git a/app/controllers/application_forms_controller.rb b/app/controllers/application_forms_controller.rb index e3cddf6..7c67ec9 100644 --- a/app/controllers/application_forms_controller.rb +++ b/app/controllers/application_forms_controller.rb @@ -395,8 +395,9 @@ class ApplicationFormsController < ApplicationController def agree_link signup = ApplicationFormSignup.find(params[:id]) rescue nil stage_num = signup ? signup.get_stage_num(params[:cipher]) : nil + @error_message = nil if stage_num.nil? - render_404 + @error_message = I18n.t('application_form.this_link_has_expired') else @type = params[:type] application_form = signup.application_form_main @@ -444,7 +445,7 @@ class ApplicationFormsController < ApplicationController form_params = params[:application_form_signup] form_params_email = form_params[:email] form_params_main_id = form_params[:application_form_main_id] - @signup = ApplicationFormSignup.where(email: form_params_email, application_form_main_id: form_params_main_id ).first + @signup = nil #ApplicationFormSignup.where(email: form_params_email, application_form_main_id: form_params_main_id ).first @application_form = ApplicationFormMain.where(id: form_params_main_id).first @application_form_signup = ApplicationFormSignup.new(application_form_signup_params) @@ -453,10 +454,6 @@ class ApplicationFormsController < ApplicationController else flag = false end - if @signup.present? - @signup.destroy - @signup = nil - end begin if gotcha_valid? || flag if @signup.blank? && @application_form_signup.save diff --git a/app/views/application_forms/agree_link.html.erb b/app/views/application_forms/agree_link.html.erb index 3050952..f367cca 100644 --- a/app/views/application_forms/agree_link.html.erb +++ b/app/views/application_forms/agree_link.html.erb @@ -3,7 +3,7 @@ background: #00ff7f; color: #000000; } - .disagree_text{ + .disagree_text, .error_text{ background: #ff004a; color: #ffffff; } @@ -16,11 +16,17 @@ position: absolute; } -

<% if @type == 'agree' %> +<% @type = 'error' if @error_message %> +

+ +<% if @error_message %> +<%= @error_message.html_safe %> +<%elsif @type == 'agree' %> <%=t('application_form.send_email_to_next_stage_reviewer')%> <% elsif @type == 'disagree' %> <%=t('application_form.email_add_file_success')%> <% elsif @type == 'finish' %> <%=t('application_form.email_edit_file_success')%> -<% end %>

+<% end %> +

\ No newline at end of file diff --git a/config/locales/en.yml b/config/locales/en.yml index 3f37e62..bdb9310 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -27,6 +27,7 @@ en: abstract_number: Abstract number presentation_type: Presentation application_form: + this_link_has_expired: "This link has expired!" close: "Close" send_email_to_next_stage_reviewer: "Email has been sent to reviewer of next stage!" denied_stage: "Denied Stage" diff --git a/config/locales/zh_tw.yml b/config/locales/zh_tw.yml index 9a69392..e41c327 100644 --- a/config/locales/zh_tw.yml +++ b/config/locales/zh_tw.yml @@ -28,6 +28,7 @@ zh_tw: abstract_number: 摘要編號 presentation_type: 發表方式 application_form: + this_link_has_expired: "此連結已失效!" close: "關閉" send_email_to_next_stage_reviewer: "已寄送email給下個階段審核者!" denied_stage: "未通過階段"