From b2a5e361e8a163552bed60b5bf6e3813268b5eac Mon Sep 17 00:00:00 2001 From: Harry Bomrah Date: Fri, 10 Apr 2015 16:33:47 +0800 Subject: [PATCH 1/5] fix for cache clear on page part submit --- app/models/sub_part.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/models/sub_part.rb b/app/models/sub_part.rb index c2dd336..ff9f9dd 100644 --- a/app/models/sub_part.rb +++ b/app/models/sub_part.rb @@ -17,8 +17,8 @@ class SubPart belongs_to :page_part belongs_to :mobile_page_part - after_save :clear_cache - after_destroy :clear_cache + # after_save :clear_cache + # after_destroy :clear_cache def clear_cache Rails.cache.delete_matched( /#{self.id.to_s}/ ) From 4ecfb9bf80a2c596ff036a7b91ae5dfa822d2d07 Mon Sep 17 00:00:00 2001 From: Harry Bomrah Date: Fri, 10 Apr 2015 18:38:25 +0800 Subject: [PATCH 2/5] fix error for page submit --- app/models/page.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/models/page.rb b/app/models/page.rb index aeea7fe..0f3d801 100644 --- a/app/models/page.rb +++ b/app/models/page.rb @@ -32,8 +32,8 @@ class Page before_create :assign_page_number - after_save :clear_cache - after_destroy :clear_cache + # after_save :clear_cache + # after_destroy :clear_cache def clear_cache I18n.available_locales.each do |locale| From 72c120c0ae39b78cc28f4d2bc21091b678c68353 Mon Sep 17 00:00:00 2001 From: Harry Bomrah Date: Tue, 14 Apr 2015 16:57:44 +0800 Subject: [PATCH 3/5] rails menu cache removed --- app/helpers/application_helper.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index 8764b2f..768f399 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -76,7 +76,7 @@ module ApplicationHelper end def render_menu - menu_html = Rails.cache.fetch(['main_menu',request.original_fullpath, I18n.locale], race_condition_ttl: 2.seconds) do + # menu_html = Rails.cache.fetch(['main_menu',request.original_fullpath, I18n.locale], race_condition_ttl: 2.seconds) do # json_file = File.read(File.join(Rails.root, 'public', "menu.json")) # @items = JSON.parse(json_file) if $mobile.blank? @@ -165,9 +165,9 @@ module ApplicationHelper end h = create_menu(@items,0) h.html_safe - end + # end - menu_html + # menu_html end def render_view From 449d809b35b7eb5ac4e18bfc479e36f0af8037aa Mon Sep 17 00:00:00 2001 From: Harry Bomrah Date: Wed, 15 Apr 2015 13:39:46 +0800 Subject: [PATCH 4/5] debug added for modules and widgets frontend --- app/helpers/application_helper.rb | 51 +++++++++++++++++++++++++++++-- app/helpers/pages_helper.rb | 46 +++++++++++++++++++++++++++- app/models/site.rb | 2 ++ 3 files changed, 96 insertions(+), 3 deletions(-) diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index 768f399..3a64c46 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -229,7 +229,11 @@ module ApplicationHelper doc = Nokogiri::HTML(file, nil, "UTF-8") file.close controller = "#{params[:target_controller].capitalize}_controller".classify.constantize.new - data = controller.send("#{params[:target_action]}") rescue nil + begin + data = controller.send("#{params[:target_action]}")# rescue nil + rescue Exception => e + write_debug_file(e,params[:target_controller],params[:target_action]) if Site::DEBUG + end if !data.nil? wrap_elements = doc.css("*[data-list][data-level='0']") htmls = parsing_repeats_again(wrap_elements,data,1) @@ -261,7 +265,11 @@ module ApplicationHelper doc = Nokogiri::HTML(file, nil, "UTF-8") file.close controller = "#{params[:target_controller].capitalize}_controller".classify.constantize.new - data = controller.send("#{params[:target_action]}") rescue nil + begin + data = controller.send("#{params[:target_action]}")# rescue nil + rescue Exception => e + write_debug_file(e,params[:target_controller],params[:target_action]) if Site::DEBUG + end if data.nil? return "
No content to show.
".html_safe end @@ -496,4 +504,43 @@ module ApplicationHelper target end + def write_debug_file(e,controller_name,action_name) + directory_name = "tmp/debug" + Dir.mkdir(directory_name) unless File.exists?(directory_name) + fn = "#{directory_name}/#{controller_name}_#{action_name}.html" + error_trace_spans = "" + e.backtrace.each do |bt| + error_trace_spans = error_trace_spans + "#{bt}
" + end + con = "#{controller_name.capitalize}_controller".classify.constantize + File.open(fn, "w"){ |file| + file.puts " + + Debug result + + +

Error Message

+
+

#{e.message}

+
+

Request Details

+
+ Url : #{request.url}
+ Controller : #{con.to_s}
+ Action : #{action_name.capitalize} +
+

Error Trace

+
+ #{error_trace_spans} +
+

Params

+
+ #{OrbitHelper.params} +
+ + " + + } + end + end diff --git a/app/helpers/pages_helper.rb b/app/helpers/pages_helper.rb index 4d451ae..f7376fc 100644 --- a/app/helpers/pages_helper.rb +++ b/app/helpers/pages_helper.rb @@ -71,7 +71,11 @@ module PagesHelper wrap_elements = doc.css("*[data-repeat]") controller = "#{controller_name.capitalize.pluralize}_controller".classify.constantize.new OrbitHelper.set_current_widget_module controller_name - data = controller.send("#{widget_method}") rescue nil + begin + data = controller.send("#{widget_method}") #rescue nil + rescue Exception => e + write_widget_debug_file(e,controller_name,widget_method,subpart_id) + end if !data.nil? wrap_elements = doc.css("*[data-list][data-level='0']") htmls = widget_parsing_repeats_again(wrap_elements,data,1) @@ -91,5 +95,45 @@ module PagesHelper end end + def write_widget_debug_file(e,controller_name,action_name,sub_part) + directory_name = "tmp/debug" + Dir.mkdir(directory_name) unless File.exists?(directory_name) + fn = "#{directory_name}/#{controller_name}_#{action_name}_#{(sub_part if !sub_part.nil?)}.html" + error_trace_spans = "" + e.backtrace.each do |bt| + error_trace_spans = error_trace_spans + "#{bt}
" + end + con = "#{controller_name.capitalize.pluralize}_controller".classify.constantize + File.open(fn, "w"){ |file| + file.puts " + + Debug result + + +

Error Message

+
+

#{e.message}

+
+

Request Details

+
+ Url : #{request.url}
+ Controller : #{con.to_s}
+ Action : #{action_name.capitalize}
+ SubPart Id : #{sub_part} +
+

Error Trace

+
+ #{error_trace_spans} +
+

Params

+
+ #{OrbitHelper.params} +
+ + " + + } + end + end diff --git a/app/models/site.rb b/app/models/site.rb index eda2909..ed4f3c7 100644 --- a/app/models/site.rb +++ b/app/models/site.rb @@ -3,6 +3,8 @@ class Site include Mongoid::Timestamps include Slug + DEBUG = false + field :title, as: :slug_title, type: String, localize: true field :school, type: String field :department, type: String From ef8728c6fa890b0274fa5f4b43d9f88f0f466511 Mon Sep 17 00:00:00 2001 From: Harry Bomrah Date: Fri, 17 Apr 2015 15:54:32 +0800 Subject: [PATCH 5/5] modified debugging for frontend --- app/helpers/application_helper.rb | 11 ++++++++--- app/helpers/pages_helper.rb | 11 ++++++++--- 2 files changed, 16 insertions(+), 6 deletions(-) diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb index 3a64c46..966f092 100644 --- a/app/helpers/application_helper.rb +++ b/app/helpers/application_helper.rb @@ -505,8 +505,11 @@ module ApplicationHelper end def write_debug_file(e,controller_name,action_name) - directory_name = "tmp/debug" - Dir.mkdir(directory_name) unless File.exists?(directory_name) + url_dir_name = request.fullpath.split("?")[0] + url_dir_name = URI.decode(url_dir_name) + url_dir_name = (url_dir_name == "/" ? "home" : url_dir_name.sub("/","").gsub("/","_").gsub("-","_").gsub(" ","_")) + directory_name = "tmp/debug/#{url_dir_name}" + FileUtils.mkdir_p(directory_name) unless File.exists?(directory_name) fn = "#{directory_name}/#{controller_name}_#{action_name}.html" error_trace_spans = "" e.backtrace.each do |bt| @@ -516,6 +519,8 @@ module ApplicationHelper File.open(fn, "w"){ |file| file.puts " + + Debug result @@ -525,7 +530,7 @@ module ApplicationHelper

Request Details

- Url : #{request.url}
+ Url : #{URI.decode(request.url)}
Controller : #{con.to_s}
Action : #{action_name.capitalize}
diff --git a/app/helpers/pages_helper.rb b/app/helpers/pages_helper.rb index f7376fc..a6f2780 100644 --- a/app/helpers/pages_helper.rb +++ b/app/helpers/pages_helper.rb @@ -96,8 +96,11 @@ module PagesHelper end def write_widget_debug_file(e,controller_name,action_name,sub_part) - directory_name = "tmp/debug" - Dir.mkdir(directory_name) unless File.exists?(directory_name) + url_dir_name = request.fullpath.split("?")[0] + url_dir_name = URI.decode(url_dir_name) + url_dir_name = (url_dir_name == "/" ? "home" : url_dir_name.sub("/","").gsub("/","_").gsub("-","_").gsub(" ","_")) + directory_name = "tmp/debug/#{url_dir_name}" + FileUtils.mkdir_p(directory_name) unless File.exists?(directory_name) fn = "#{directory_name}/#{controller_name}_#{action_name}_#{(sub_part if !sub_part.nil?)}.html" error_trace_spans = "" e.backtrace.each do |bt| @@ -107,6 +110,8 @@ module PagesHelper File.open(fn, "w"){ |file| file.puts " + + Debug result @@ -116,7 +121,7 @@ module PagesHelper

Request Details

- Url : #{request.url}
+ Url : #{URI.decode(request.url)}
Controller : #{con.to_s}
Action : #{action_name.capitalize}
SubPart Id : #{sub_part}