Change function name to indicate it relies on d3

mysociety · Oct 19, 2016 · db2dd72 · db2dd72
1 parent a28bd0e
commit db2dd72
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/assets/javascripts/time_series.js b/app/assets/javascripts/time_series.js
@@ -1,4 +1,4 @@
-function timeSeries(selector, data, title, metric_name) {
+function d3TimeSeries(selector, data, title, metric_name) {
 
   // Add the title
   var wrapper_element = document.querySelectorAll(selector)[0];

diff --git a/app/views/statistics/index.html.erb b/app/views/statistics/index.html.erb
@@ -146,9 +146,9 @@
     <%= javascript_include_tag "https://cdnjs.cloudflare.com/ajax/libs/d3/3.5.12/d3.min.js" %>
     <%= javascript_include_tag "time_series" %>
     <script type="text/javascript">
-      if (typeof timeSeries == 'function') {
+      if (typeof d3TimeSeries == 'function') {
         d3.json("<%= statistics_url(:format => 'json') %>", function(data) {
-          timeSeries(
+          d3TimeSeries(
             "#request-hide-events-data",
             data["requests"]["hides_by_week"],
             "<%= _("Number of request hide events over time") %>",