Display full info on Errs#show page

Jared Pace
1 parent 93030e53
Showing 10 changed files with 91 additions and 12 deletions Show diff stats
app/helpers/hash_helper.rb
app/models/err.rb
app/models/notice.rb
app/views/errs/show.html.haml
app/views/notices/_backtrace.html.haml
app/views/notices/_environment.html.haml
app/views/notices/_params.html.haml
app/views/notices/_session.html.haml
app/views/notices/_summary.html.haml
public/stylesheets/application.css
@@ -0,0 +1,20 @@
+module HashHelper
+  
+  def pretty_hash(hash, nesting = 0)
+    tab_size = 2
+    nesting += 1
+    
+    pretty  = "{"
+    sorted_keys = hash.keys.sort
+    sorted_keys.each do |key|
+      val = hash[key].is_a?(Hash) ? pretty_hash(hash[key], nesting) : hash[key].inspect
+      pretty += "\n#{' '*nesting*tab_size}"
+      pretty += "#{key.inspect} => #{val}"
+      pretty += "," unless key == sorted_keys.last
+
+    end
+    nesting -= 1
+    pretty += "\n#{' '*nesting*tab_size}}"
+  end
+  
+end
 \ No newline at end of file
@@ -35,7 +35,7 @@ class Err
   end
   def where
-    where = component
+    where = component.dup
     where << "##{action}" if action.present?
     where
   end
@@ -19,6 +19,9 @@ class Notice
     hoptoad_notice = Hoptoad::V2.parse_xml(hoptoad_xml)
     project = Project.find_by_api_key!(hoptoad_notice['api-key'])
+    hoptoad_notice['request']['component']  = 'unknown' if hoptoad_notice['request']['component'].blank?
+    hoptoad_notice['request']['action']     = nil if hoptoad_notice['request']['action'].blank?
+    
     error = Err.for({
       :project    => project,
       :klass      => hoptoad_notice['error']['class'],
@@ -36,6 +39,22 @@ class Notice
     })
   end
+  def request
+    read_attribute(:request) || {}
+  end
+  
+  def env_vars
+    request['cgi-data'] || {}
+  end
+  
+  def params
+    request['params'] || {}
+  end
+  
+  def session
+    request['session'] || {}
+  end
+  
   def deliver_notification
     Mailer.error_notification(self).deliver
   end
@@ -14,6 +14,18 @@
   = link_to "back to '#{@project.name}'", project_path(@project)
   |
   = link_to 'resolve', '#' if @err.unresolved?
-  
+
+%h3#summary Summary
+= render 'notices/summary', :notice => @notice
+
 %h3#backtrace Backtrace
-= render 'notices/backtrace', :lines => @notice.backtrace
 \ No newline at end of file
+= render 'notices/backtrace', :lines => @notice.backtrace
+
+%h3#environment Environment
+= render 'notices/environment', :notice => @notice
+
+%h3#params Parameters
+= render 'notices/params', :notice => @notice
+
+%h3#session Session
+= render 'notices/session', :notice => @notice
 \ No newline at end of file
-.backtrace
-  %table
+.window
+  %table.backtrace
     %tr
       %th
         %ul.line-numbers
@@ -0,0 +1,6 @@
+.window
+  %table.environment
+    - notice.env_vars.each do |key,val|
+      %tr
+        %th= key
+        %td.main= val
 \ No newline at end of file
@@ -0,0 +1,3 @@
+.window
+  %pre.hash= pretty_hash notice.params
+    
 \ No newline at end of file
@@ -0,0 +1,2 @@
+.window
+  %pre.hash= pretty_hash notice.session
 \ No newline at end of file
@@ -0,0 +1,13 @@
+%table.summary
+  %tr
+    %th URL
+    %td.main= notice.request['url']
+  %tr
+    %th Where
+    %td= notice.err.where
+  %tr
+    %th Occurred
+    %td= notice.created_at.to_s(:micro)
+  %tr
+    %th Similar
+    %td= notice.err.notices.count - 1
 \ No newline at end of file
@@ -203,6 +203,9 @@ td.deploy {
 td.latest {
   white-space: nowrap;
 }
+td.count {
+  text-align: right;
+}
 /* Err Tables */
 table.errs td.message a { 
@@ -214,30 +217,31 @@ table.errs td.message em {
 }
 /* Backtrace */
-.backtrace {
+.window {
   width: 100%;
+  margin-bottom: 1em;
   overflow: auto;
 }
-.backtrace table {
+.window table {
   margin: 0;
 }
-.backtrace td {
+table.backtrace td {
   width: 100%;
   padding: 0;
   margin: 0;
   color: #A2A2A2;
   background-color: #222 !important;
 }
-.backtrace ul.line-numbers li {
+table.backtrace ul.line-numbers li {
   text-align: right;
 }
-.backtrace ul.lines {
+table.backtrace ul.lines {
   padding: 10px 8px;
   overflow: auto;
 }
-.backtrace li {
+table.backtrace li {
   white-space: nowrap;
 }
-.backtrace li.in-project {
+table.backtrace li.in-project {
   color: #2adb2e;
 }
 \ No newline at end of file
	@@ -0,0 +1,20 @@		@@ -0,0 +1,20 @@
		1	+module HashHelper
		2	+
		3	+ def pretty_hash(hash, nesting = 0)
		4	+ tab_size = 2
		5	+ nesting += 1
		6	+
		7	+ pretty = "{"
		8	+ sorted_keys = hash.keys.sort
		9	+ sorted_keys.each do \|key\|
		10	+ val = hash[key].is_a?(Hash) ? pretty_hash(hash[key], nesting) : hash[key].inspect
		11	+ pretty += "\n#{' 'nestingtab_size}"
		12	+ pretty += "#{key.inspect} => #{val}"
		13	+ pretty += "," unless key == sorted_keys.last
		14	+
		15	+ end
		16	+ nesting -= 1
		17	+ pretty += "\n#{' 'nestingtab_size}}"
		18	+ end
		19	+
		20	+end
0	\ No newline at end of file	21	\ No newline at end of file
	@@ -35,7 +35,7 @@ class Err		@@ -35,7 +35,7 @@ class Err
35	end	35	end
36		36
37	def where	37	def where
38	- where = component	38	+ where = component.dup
39	where << "##{action}" if action.present?	39	where << "##{action}" if action.present?
40	where	40	where
41	end	41	end
	@@ -19,6 +19,9 @@ class Notice		@@ -19,6 +19,9 @@ class Notice
19	hoptoad_notice = Hoptoad::V2.parse_xml(hoptoad_xml)	19	hoptoad_notice = Hoptoad::V2.parse_xml(hoptoad_xml)
20	project = Project.find_by_api_key!(hoptoad_notice['api-key'])	20	project = Project.find_by_api_key!(hoptoad_notice['api-key'])
21		21
		22	+ hoptoad_notice['request']['component'] = 'unknown' if hoptoad_notice['request']['component'].blank?
		23	+ hoptoad_notice['request']['action'] = nil if hoptoad_notice['request']['action'].blank?
		24	+
22	error = Err.for({	25	error = Err.for({
23	:project => project,	26	:project => project,
24	:klass => hoptoad_notice['error']['class'],	27	:klass => hoptoad_notice['error']['class'],
	@@ -36,6 +39,22 @@ class Notice		@@ -36,6 +39,22 @@ class Notice
36	})	39	})
37	end	40	end
38		41
		42	+ def request
		43	+ read_attribute(:request) \|\| {}
		44	+ end
		45	+
		46	+ def env_vars
		47	+ request['cgi-data'] \|\| {}
		48	+ end
		49	+
		50	+ def params
		51	+ request['params'] \|\| {}
		52	+ end
		53	+
		54	+ def session
		55	+ request['session'] \|\| {}
		56	+ end
		57	+
39	def deliver_notification	58	def deliver_notification
40	Mailer.error_notification(self).deliver	59	Mailer.error_notification(self).deliver
41	end	60	end
	@@ -14,6 +14,18 @@		@@ -14,6 +14,18 @@
14	= link_to "back to '#{@project.name}'", project_path(@project)	14	= link_to "back to '#{@project.name}'", project_path(@project)
15	\|	15	\|
16	= link_to 'resolve', '#' if @err.unresolved?	16	= link_to 'resolve', '#' if @err.unresolved?
17	-	17	+
		18	+%h3#summary Summary
		19	+= render 'notices/summary', :notice => @notice
		20	+
18	%h3#backtrace Backtrace	21	%h3#backtrace Backtrace
19	-= render 'notices/backtrace', :lines => @notice.backtrace
20	\ No newline at end of file	22	\ No newline at end of file
		23	+= render 'notices/backtrace', :lines => @notice.backtrace
		24	+
		25	+%h3#environment Environment
		26	+= render 'notices/environment', :notice => @notice
		27	+
		28	+%h3#params Parameters
		29	+= render 'notices/params', :notice => @notice
		30	+
		31	+%h3#session Session
		32	+= render 'notices/session', :notice => @notice
21	\ No newline at end of file	33	\ No newline at end of file
1	-.backtrace
2	- %table	1	+.window
		2	+ %table.backtrace
3	%tr	3	%tr
4	%th	4	%th
5	%ul.line-numbers	5	%ul.line-numbers
	@@ -0,0 +1,6 @@		@@ -0,0 +1,6 @@
		1	+.window
		2	+ %table.environment
		3	+ - notice.env_vars.each do \|key,val\|
		4	+ %tr
		5	+ %th= key
		6	+ %td.main= val
0	\ No newline at end of file	7	\ No newline at end of file
	@@ -0,0 +1,3 @@		@@ -0,0 +1,3 @@
		1	+.window
		2	+ %pre.hash= pretty_hash notice.params
		3	+
0	\ No newline at end of file	4	\ No newline at end of file
	@@ -0,0 +1,2 @@		@@ -0,0 +1,2 @@
		1	+.window
		2	+ %pre.hash= pretty_hash notice.session
0	\ No newline at end of file	3	\ No newline at end of file
	@@ -0,0 +1,13 @@		@@ -0,0 +1,13 @@
		1	+%table.summary
		2	+ %tr
		3	+ %th URL
		4	+ %td.main= notice.request['url']
		5	+ %tr
		6	+ %th Where
		7	+ %td= notice.err.where
		8	+ %tr
		9	+ %th Occurred
		10	+ %td= notice.created_at.to_s(:micro)
		11	+ %tr
		12	+ %th Similar
		13	+ %td= notice.err.notices.count - 1
0	\ No newline at end of file	14	\ No newline at end of file