GitHub PullRequestReviewCommentEvent at 2014-05-10 19:23:24
Repository: website
Actor: moonglum
Payload: {"comment":{"url":"https://api.github.com/repos/hacken-in/website/pulls/comments/12505767","id":12505767,"diff_hunk":"@@ -275,14 +275,14 @@ def attended_by_external_user?(session_uuid)\n   end\n \n   def self.this_week_by_day\n-    week_stats = self.this_week.group_by_day.count\n+    week_stats = self.reorder(nil).this_week.group_by_day.count\n     Hash[(Date.today.beginning_of_week .. Date.today.end_of_week).map do |day|\n-      [day.strftime(\"%a\"), week_stats[day.day] || 0]\n+      [day.strftime(\"%a\"), week_stats[day] || 0]\n     end]\n   end\n \n   def self.this_week_by_category\n-    Hash[self.this_week.group_by_category.count.map { |category_id, count| [ Category.title_for(category_id), count ] }]\n+    Hash[self.reorder(nil).this_week.group_by_category.count.map { |category_id, count| [ Category.title_for(category_id), count ] }]","path":"app/models/single_event.rb","position":50,"original_position":50,"commit_id":"67877dc64de32708a4e85ca740600dd72672e43f","original_commit_id":"1be6f4538dcf0fb74969637ab9a1502cd88fcd66","user":{"login":"moonglum","id":95788,"avatar_url":"https://avatars.githubusercontent.com/u/95788?","gravatar_id":"84ac75cc9db741ff266392c83de9aa15","url":"https://api.github.com/users/moonglum","html_url":"https://github.com/moonglum","followers_url":"https://api.github.com/users/moonglum/followers","following_url":"https://api.github.com/users/moonglum/following{/other_user}","gists_url":"https://api.github.com/users/moonglum/gists{/gist_id}","starred_url":"https://api.github.com/users/moonglum/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/moonglum/subscriptions","organizations_url":"https://api.github.com/users/moonglum/orgs","repos_url":"https://api.github.com/users/moonglum/repos","events_url":"https://api.github.com/users/moonglum/events{/privacy}","received_events_url":"https://api.github.com/users/moonglum/received_events","type":"User","site_admin":false},"body":"Ok, thanks for the explanation! `.reorder(nil)` looks weird.","created_at":"2014-05-10T19:23:24Z","updated_at":"2014-05-10T19:23:24Z","html_url":"https://github.com/hacken-in/website/pull/372#discussion_r12505767","pull_request_url":"https://api.github.com/repos/hacken-in/website/pulls/372","_links":{"self":{"href":"https://api.github.com/repos/hacken-in/website/pulls/comments/12505767"},"html":{"href":"https://github.com/hacken-in/website/pull/372#discussion_r12505767"},"pull_request":{"href":"https://api.github.com/repos/hacken-in/website/pulls/372"}}}}