Added logic in to protect from nil attributes

Author: bad6e

app/presenters/user_presenter.rb

@@ -23,7 +23,7 @@ def starred_repos
   end
 
   def commit_mess
-    if user.commit_messages == "[]"
+    if user.commit_messages == "[\"User has no commits\"]"
       ["You have no commit messages"]
     else
       user.commit_messages.gsub("\"","").split(", ")[2..-2]
@@ -67,7 +67,7 @@ def longest_streaks
   end
 
   def followers_activities
-    if user.follower_messages == "[]"
+    if user.follower_messages == "{}"
       ["None of your friends are coding"]
     else
       user.follower_messages.gsub("\"","").gsub("\\","").gsub("{","").gsub("}","").split(", ")

app/services/github_service.rb

@@ -42,25 +42,59 @@ def total_starred_repos(user)
   end
 
   def commit_message(user)
-    list      = parse(connection.get("/users/#{user.nickname}/events"))
-    events    = list.select { |item| item[:type] == "PushEvent" }
-    commits   = events.map{ |event| event[:payload][:commits] }.flatten!
-    messages  = commits.collect{ |commit|  commit[:message] }
-    messages[0..9]
+    list = parse(connection.get("/users/#{user.nickname}/events"))
+    if list == "[]"
+      ["User has no events"]
+    else
+      find_events_from_list(list)
+    end
   end
 
   def followers_commit_message(name)
     list      = parse(connection.get("/users/#{name}/events"))
     if list == "[]"
       ["User has no events"]
     else
-      events    = list.select { |item| item[:type] == "PushEvent" }
-      commits   = events.map{ |event| event[:payload][:commits] }.flatten!
-      messages  = commits.collect{ |commit|  commit[:message] }
-      messages[0]
+      find_events_from_list(list)
     end
   end
 
+  def find_events_from_list(list)
+    events = list.select { |item| item[:type] == "PushEvent" }
+    if events == "[]"
+      ["User has no Push Events"]
+    else
+      find_commits_from_events(events)
+    end
+  end
+
+  def find_commits_from_events(events)
+    commits = events.map{ |event| event[:payload][:commits] }.flatten!
+    if commits.nil?
+      ["User has no commits"]
+    else
+      find_messages_from_commits_user(commits)
+    end
+  end
+
+  def find_messages_from_commits_user(commits)
+    messages  = commits.collect{ |commit| commit[:message] }
+    if messages == "[]"
+      ["User has no commit messages"]
+    else
+      messages[0..9]
+    end
+  end
+
+  # def find_messages_from_commits(commits)
+  #   messages  = commits.collect{ |commit| commit[:message] }
+  #   if messages == "[]"
+  #     ["User has no commit messages"]
+  #   else
+  #     messages[0]
+  #   end
+  # end
+
   def followers_activity(user)
     names = find_user_follows(user).collect {|item| item[:login] }
     messages = names.map do |name|