Fix title.

app/controllers/admin/coloncancerpredict2s_controller.rb

@@ -321,29 +321,23 @@ class Admin::Coloncancerpredict2sController < OrbitAdminController
 			@form_to_show.auto_write_predict_js
 		end
 	end
-	@index = 0
+	@site_locales = Site.last.in_use_locales.each do |locale|
-	Dir.chdir("public") do
+		I18n.with_locale(locale) do
-		while File.exist?('cancerfield_back'+@index.to_s+'.txt')
+			File.open(Coloncancerpredictfields2s::ToolTableMap[I18n.locale], 'w') do |f|
-			@index += 1
-		end
-	end
-	Dir.chdir("public") do
-		@site_locales = Site.last.in_use_locales.each do |locale|
-			I18n.with_locale(locale) do
-				@file_tmp = File.new(Coloncancerpredictfields2s::ToolTableMap[I18n.locale], 'w')
 				tmp_table_texts = create_table(locale)
-				@file_tmp.write(tmp_table_texts)
+				f.write(tmp_table_texts)
-				@file_tmp.close
 			end
 		end
 	end
-	Dir.chdir("public") do
+	idx = 0
-		@file_back = File.open('colon_cancerfield_back'+@index.to_s+'2.txt', 'w')
+	while File.exist?("public/colon_cancerfield2_back#{idx}.txt")
-		@file_back.write(@form_to_show.attributes)
+		idx += 1
-		@file_back.close
+	end
-		@file_org = File.open('colon_cancerfield_org'+@index.to_s+'2.txt', 'w')
+	File.open("public/colon_cancerfield2_back#{idx}.txt", 'w') do |f|
-		@file_org.write(Coloncancerpredictfields2s.where("title"=>@app_title+'_back').first.attributes)
+		f.write(@form_to_show.attributes)
-		@file_org.close
+	end
+	File.open("public/colon_cancerfield2_org#{idx}.txt", 'w') do |f|
+		f.write(Coloncancerpredictfields1s.where("title"=>@app_title+'_back').first.attributes)
 	end
 	redirect_to admin_coloncancerpredict2s_path
   end
@@ -631,12 +625,10 @@ class Admin::Coloncancerpredict2sController < OrbitAdminController
 		#FileUtils.rm_r(dir_path, :force => true) if Dir.exist?(dir_path)
 		FileUtils.mkdir dir_path if !Dir.exist?(dir_path) #create dir for storing tmp_file if dir doesn't exist
 		@filename = ("#{Time.now.year}_%02s_%02s_export_cancer_tool_record.xlsx"%[Time.now.month,Time.now.day]).gsub(" ","0")
-		Dir.chdir(dir_path) do 
+		File.open("#{dir_path}#{@filename}", 'w') do |f|
-			File.open(@filename, 'w') do |f|
+			f.write render_to_string( :handlers=> [:axlsx], :formats=> [:xlsx] ,:partial=> 'export_cancer_tool_record.xlsx',:locals=> {:results=>cancer_records} )
-				f.write render_to_string( :handlers=> [:axlsx], :formats=> [:xlsx] ,:partial=> 'export_cancer_tool_record.xlsx',:locals=> {:results=>cancer_records} )
-			end
 		end
-		tmp_filename_data = File.read(dir_path +@filename)
+		tmp_filename_data = File.read("#{dir_path}#{@filename}")
 		send_data(tmp_filename_data, type: 'application/xlsx', disposition: 'attachment', filename: @filename)
   end
   def create_first_field
@@ -651,13 +643,11 @@ class Admin::Coloncancerpredict2sController < OrbitAdminController
 		@form_to_show.title=@app_title
 		@form_to_show.save
 		@form_to_show = Coloncancerpredictfields2s.where("title"=>@app_title).first
-		Dir.chdir("public") do
+		@site_locales = Site.last.in_use_locales.each do |locale|
-			@site_locales = Site.last.in_use_locales.each do |locale|
+			I18n.with_locale(locale) do
-				I18n.with_locale(locale) do
+				File.open(Coloncancerpredictfields2s::ToolTableMap[I18n.locale], 'w') do |f|
-					@file_tmp = File.new(Coloncancerpredictfields2s::ToolTableMap[I18n.locale], 'w')
 					tmp_table_texts = create_table(locale)
-					@file_tmp.write(tmp_table_texts)
+					f.write(tmp_table_texts)
-					@file_tmp.close
 				end
 			end
 		end

app/controllers/coloncancerpredict2s_controller.rb

@@ -148,10 +148,10 @@ class Coloncancerpredict2sController < ApplicationController
 	tags = OrbitHelper.widget_tags
 	categories = OrbitHelper.widget_categories || []
 	@table_str = File.read(Coloncancerpredictfields2s::ToolTableMap[I18n.locale])
-	preidct_js_url = '/assets/colon_cancer_predict2.js'
+	preidct_js_url = "/assets/#{Coloncancerpredictfields2s::JS}"
 	if File.exist?(Coloncancerpredictfields2s::JSFileName)
 		js_filename = File.read(Coloncancerpredictfields2s::JSFileName)
-		if js_filename.include?('colon_cancer_predict2.js')
+		if js_filename.include?(Coloncancerpredictfields2s::JS)
 			asset = Rails.application.assets[js_filename]
 			preidct_js_url = "#{Rails.application.config.assets.prefix}/#{asset.digest_path}"
 		else
@@ -168,10 +168,10 @@ class Coloncancerpredict2sController < ApplicationController
 	tags = OrbitHelper.widget_tags
 	categories = OrbitHelper.widget_categories || []
 	@table_str = File.read(Coloncancerpredictfields2s::ToolTableMap[I18n.locale])
-	preidct_js_url = '/assets/colon_cancer_predict2.js'
+	preidct_js_url = "/assets/#{Coloncancerpredictfields2s::JS}"
 	if File.exist?(Coloncancerpredictfields2s::JSFileName)
 		js_filename = File.read(Coloncancerpredictfields2s::JSFileName)
-		if js_filename.include?('colon_cancer_predict2.js')
+		if js_filename.include?(Coloncancerpredictfields2s::JS)
 			asset = Rails.application.assets[js_filename]
 			preidct_js_url = "#{Rails.application.config.assets.prefix}/#{asset.digest_path}"
 		else

app/models/coloncancerpredictfields2s.rb

@@ -9,6 +9,8 @@ class Coloncancerpredictfields2s
   AdvanceFields = ["revert_value","map_values","coloncancer_predict_mapping_file2"]
   TherapyFields = ["variable","name","hint","comment_text","choice_fields","lpv_impact","active_choice","disable_condition"]
   TherapyOnly = ["lpv_impact","active_choice","disable_condition"]
+  ModuleAppPath = Pathname.new(File.expand_path(__dir__)).dirname.dirname.to_s.freeze
+  JS = "colon_cancer_predict2.js"
   JSFileName = "public/cc2_tool_js_filename.txt".freeze
   ToolTableMap = I18n.available_locales.map do |locale|
     [locale, "public/colon_cancer_tool_table_tmp_#{locale}2.txt".freeze]
@@ -393,8 +395,7 @@ class Coloncancerpredictfields2s
   end
   def auto_write_predict_js(force_reload=true)
     js_codes = generate_jscode
-    module_app_path = Pathname.new(File.expand_path(__dir__)).dirname.dirname.to_s
+    save_path = "#{ModuleAppPath}/app/assets/javascripts/#{JS}"
-    save_path = module_app_path + '/app/assets/javascripts/colon_cancer_predict2.js'
     file_texts = File.read(save_path)
     need_write = false
     str1 = "/* auto add start */"