include view, conform to new tracker interface

Author: stevecrozz

lib/errbit_github_plugin.rb

@@ -3,7 +3,10 @@
 require 'errbit_github_plugin/issue_tracker'
 require 'errbit_github_plugin/rails'
 
-ErrbitPlugin::Register.add_issue_tracker(
-  'IssueTrackers::GithubIssuesTracker',
-  ErrbitGithubPlugin::IssueTracker
-)
+module ErrbitGithubPlugin
+  def self.root
+    File.expand_path '../..', __FILE__
+  end
+end
+
+ErrbitPlugin::Registry.add_issue_tracker(ErrbitGithubPlugin::IssueTracker)

lib/errbit_github_plugin/issue_tracker.rb

@@ -53,26 +53,30 @@ def project_id
       app.github_repo
     end
 
-    def check_params
+    def errors
       errors = []
       if self.class.fields.detect {|f| params[f[0]].blank? }
         errors << [:base, 'You must specify your GitHub username and password']
       end
       errors
     end
 
-    def create_issue(problem, reported_by = nil)
+    def github_client
       # Login using OAuth token, if given.
       if oauth_token
-        client = Octokit::Client.new(:login => username, :oauth_token => oauth_token)
+        Octokit::Client.new(
+          :login => params['username'], :oauth_token => oauth_token)
       else
-        client = Octokit::Client.new(:login => username, :password => password)
+        Octokit::Client.new(
+          :login => params['username'], :password => params['password'])
       end
+    end
 
+    def create_issue(problem, reported_by = nil)
       begin
-        issue = client.create_issue(
+        issue = github_client.create_issue(
           project_id,
-          issue_title(problem),
+          "[#{ problem.environment }][#{ problem.where }] #{problem.message.to_s.truncate(100)}",
           self.class.body_template.result(binding).unpack('C*').pack('U*')
         )
         @url = issue.html_url

views/github_issues_body.txt.erb

@@ -0,0 +1,45 @@
+[See this exception on Errbit](<%= problem.url %> "See this exception on Errbit")
+<% if notice = problem.notices.first %>
+# <%= notice.message %> #
+## Summary ##
+<% if notice.request['url'].present? %>
+  ### URL ###
+  [<%= notice.request['url'] %>](<%= notice.request['url'] %>)"
+<% end %>
+### Where ###
+<%= notice.where %>
+
+### Occured ###
+<%= notice.created_at.to_s(:micro) %>
+
+### Similar ###
+<%= (notice.problem.notices_count - 1).to_s %>
+
+## Params ##
+```
+<%= pretty_hash(notice.params) %>
+```
+
+## Session ##
+```
+<%= pretty_hash(notice.session) %>
+```
+
+## Backtrace ##
+```
+<% notice.backtrace_lines.each do |line| %><%= line.number %>:  <%= line.file_relative %> -> **<%= line.method %>**
+<% end %>
+```
+
+## Environment ##
+
+<table>
+<% for key, val in notice.env_vars %>
+  <tr>
+    <td><%= key %>:</td>
+    <td><%= val %></td>
+  </tr>
+<% end %>
+</table>
+<% end %>
+