senaduka/old-holivud2
1
0
Fork 0
Code Issues Pull Requests 6 Actions Packages Projects Releases Wiki Activity

make question answers required #41

Closed
bilal.catic wants to merge 3 commits from make-questions-required-in-medical-release into master
pull from: make-questions-required-in-medical-release
merge into: senaduka:master
Conversation 2 Commits 3 Files Changed 7 +1 -1
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 4dad4adde2 - Show all commits

2
app/models/medical_release.rb
View File

@@ -82,7 +82,7 @@ class MedicalRelease < ApplicationRecord
def valid_answers
(1..ContractTemplate::NUMBER_OF_CUSTOM_FIELDS).each do |index|
if contract_template["question_#{index}_text"].present? &&
if contract_template && contract_template["question_#{index}_text"].present? &&
public_send("question_#{index}_answer").blank?
errors.add("question_#{index}", I18n.t('medical_releases.custom_validation_errors.question_answer_is_required'))
end