Merge pull request #606 from mlibrary/fixup/ga-limits

We're hitting GA quota limits on monograph_catalog pages.

app/presenters/concerns/analytics_presenter.rb

@@ -5,8 +5,12 @@ def pageviews_by_path(path)
     count = 0
     profile = google_analytics_profile
     if profile.present?
-      Pageview.results(profile).each do |entry|
-        count += entry[:pageviews].to_i if entry[:pagePath] == path
+      begin
+        Pageview.results(profile).each do |entry|
+          count += entry[:pageviews].to_i if entry[:pagePath] == path
+        end
+      rescue OAuth2::Error => e
+        Rails.logger.error(e.code["message"])
       end
     end
     return count
@@ -16,10 +20,17 @@ def pageviews_by_ids(ids)
     count = 0
     profile = google_analytics_profile
     if profile.present?
-      Pageview.results(profile).each do |entry|
-        ids.each do |id|
-          count += entry[:pageviews].to_i if entry[:pagePath].include? id
+      begin
+        Pageview.results(profile).each do |entry|
+          ids.each do |id|
+            count += entry[:pageviews].to_i if entry[:pagePath].include? id
+          end
         end
+      rescue OAuth2::Error => e
+        # TODO: we're hitting GA quotas for monograph_catalog pages in production.
+        # Need to figure out a better way to do this...
+        Rails.logger.error(e.code["message"])
+        return nil
       end
     end
     return count

app/views/monograph_catalog/index.html.erb

@@ -29,9 +29,12 @@
       <div class="description">
         <%= render_markdown @monograph_presenter.description.first || '' %>
       </div>
+      <% pageviews = @monograph_presenter.pageviews %>
+      <% unless pageviews.nil? %>
       <div>
-        <p>This item has been viewed <b><%= @monograph_presenter.pageviews %></b> times.</p>
+        <p>This item has been viewed <b><%= pageviews %></b> times.</p>
       </div>
+      <% end %>
       <div><p></p></div>
       <div class="isbn">
         <% if defined?(@monograph_presenter.isbn) %>