Fix Lint/DuplicateBranch rubocop warnings

[rails.git] / script / statistics
diff --git a/script/statistics b/script/statistics

index ce785bb07741523e483cd782dd7e82930b7fcbd3..05ed637a866eb8a4152bf0657295f92089894d11 100755 (executable)
--- a/script/statistics
+++ b/script/statistics
@@ -1,8 +1,8 @@
  #!/usr/bin/env ruby
  
-require File.dirname(__FILE__) + '/../config/environment'
+require File.join(File.dirname(__FILE__), "..", "config", "environment")
  
-start_time = Time.now
+start_time = Time.now.utc
  
  puts "<html>"
  puts "<head>"
@@ -11,11 +11,12 @@ puts "<title>OpenStreetMap Statistics</title>"
  puts "<style>th { text-align: left }</style>"
  puts "</head>"
  puts "<body>"
-puts "<h2>OpenStreetMap stats report run at #{start_time.to_s}</h2>"
+puts "<h1>OpenStreetMap stats</h1>"
+puts "<h2>Report run at #{start_time}</h2>"
  
  begin
    ActiveRecord::Base.transaction do
-    user_count = User.where(:status => ["active", "confirmed", "suspended"]).count
+    user_count = User.where(:status => %w[active confirmed suspended]).count
      tracepoint_count = Tracepoint.count
      node_count = Node.where(:visible => true).count
      way_count = Way.where(:visible => true).count
@@ -29,18 +30,18 @@ begin
      puts "<tr><td>Number of relations</td><td>#{relation_count}</td></tr>"
      puts "</table>"
  
-    puts "<h2>Top 50 users for uploads of GPS data</h2>"
+    puts '<h2 id="top-traces">Top 50 users for uploads of GPS data</h2>'
      puts "<table>"
      puts "<tr><th>User</th><th>Number of Points</th></tr>"
  
      Trace.where(:inserted => true).group(:user_id).order("sum_size DESC").limit(50).sum(:size).each do |user, count|
-      display_name = User.find(user).display_name.gsub('@', ' at ').gsub('.', ' dot ')
+      display_name = User.find(user).display_name.gsub("@", " at ").gsub(".", " dot ")
        puts "<tr><td><a href=\"/user/#{display_name}\">#{display_name}</a></td><td>#{count}</td></tr>"
      end
  
      puts "</table>"
  
-    puts "<h2>Number of users editing over the past...</h2>"
+    puts '<h2 id="number-of-editors">Number of users editing over the past...</h2>'
      puts "<table>"
      puts "<tr><th>Data Type</th><th>Day</th><th>Week</th><th>Month</th></tr>"
  
@@ -57,8 +58,8 @@ begin
      puts "<tr><th>Nodes</th><td>#{day_count}</td><td>#{week_count}</td><td>#{month_count}</td></tr>"
  
      puts "</table>"
-  
-    puts "<h2>Top users editing over the past...</h2>"
+
+    puts '<h2 id="top-editors">Top users editing over the past...</h2>'
      puts "<table>"
      puts "<tr><th>Day</th><th>Week</th><th>Month</th></tr>"
  
@@ -72,7 +73,7 @@ begin
          if column.nil?
            puts "<td></td>"
          else
-          display_name = User.find(column[0]).display_name.gsub('@', ' at ').gsub('.', ' dot ')
+          display_name = User.find(column[0]).display_name.gsub("@", " at ").gsub(".", " dot ")
            count = column[1]
            puts "<td>#{count} <a href=\"/user/#{display_name}\">#{display_name}</a></td>"
          end
@@ -82,11 +83,11 @@ begin
  
      puts "</table>"
    end
-rescue Exception => e
-  puts "<p><em>Exception: #{e.to_s}</em><br />#{e.backtrace.join('<br />')}</p>"
+rescue StandardError => e
+  puts "<p><em>Exception: #{e}</em><br />#{e.backtrace.join('<br />')}</p>"
  end
  
-puts "<p>Report took #{(Time.new - start_time).to_s} seconds to run</p>"
+puts "<p>Report took #{Time.now.utc - start_time} seconds to run</p>"
  puts "</body>"
  puts "</html>"