simple_drilldown 0.6.3 → 0.6.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 32063b0fd8dec6ee846e38eb37753d795acb2c6f3cca5d98a45d6c6a9c9bd868
-  data.tar.gz: 4c95e97c4780f82a6227a0f5e2048c7501d29e5534e7c74953746be915f42b1d
+  metadata.gz: 42c85e43c9d07d94ed0d7c502afa8b4ee638f86ca41bf6b742f7f34cfd47a355
+  data.tar.gz: 506ac06824fd78a878302544967b7f997bfae64c6e4e26da9b5955603595ffc2
 SHA512:
-  metadata.gz: b78736b952b5c7727ea627d83a4c47a72cf0665fded104fd6f65bb0f0adfc8b09a6899c4b832ce3a05991e05be2be8496ba6935ac24bc618af4492f7b997833b
-  data.tar.gz: e4f976920fcd12bd3820c390b22dd0d36af9a13f6d3f01517acb82574879d0196f4fc6fe7417a91a5a916a064d3187bc115119bfa1f16ea8954d53e9ee61b159
+  metadata.gz: 4dbf710685173878b38ce96238eb9c1e558bd4b9472d0f928146920f21b2de3a4eb3c64195ad87da9940383f7190c6d89404419390bce6f4fcceb134a959f634
+  data.tar.gz: aa1f9ba981439836636dcab9a0adcd61b834f7c5e2667b86dd43980d7b3a3072f2df2b4ab6e7d6f7161fa7ed449c9473fd1a796692680dbffeb6a25e9977f0ee

data/app/views/drilldown/_chart.html.erb

@@ -20,7 +20,7 @@
   case @search.display_type
   when SimpleDrilldown::Search::DisplayType::PIE
 %>
-  <%= pie_chart data, height: '24rem' %>
+  <%= pie_chart data, height: '24rem', events: ['click'] %>
 <% when SimpleDrilldown::Search::DisplayType::BAR %>
   <%= column_chart data, height: '24rem' %>
 <% when SimpleDrilldown::Search::DisplayType::LINE %>
@@ -40,7 +40,7 @@
     options += @remaining_dimensions.keys.map { |name| [controller.c_dimension_defs[name][:pretty_name], name] }  %>
     <%= t(i == 0 ? :group_by : :then_by) %>:
     <%= form.select 'dimensions', options, { :selected => @search.dimensions && @search.dimensions[i] },
-        { :onChange => 'form.submit()', :name => 'search[dimensions][]', :id => "search_dimensions_#{i}" } %>
+        { onChange: 'form.submit()', name: 'search[dimensions][]', id: "search_dimensions_#{i}" } %>
   <% end %>
 
   <br/>
@@ -60,3 +60,23 @@
   <%= form.check_box :list, { :onChange => 'form.submit()' } %>
   <%= form.label :list, t(:list) %>
 </div>
+
+<% if @dimensions.size == 1 &&
+    [SimpleDrilldown::Search::DisplayType::BAR, SimpleDrilldown::Search::DisplayType::PIE].include?(@search.display_type) %>
+  <%= javascript_tag do %>
+    $('#chart-1').on('click', function(e){
+      chart = Chartkick.charts['chart-1'].getChartObject();
+      firstPoint = chart.getElementsAtEvent(e)[0]
+      if (firstPoint) {
+        label = chart.data.labels[firstPoint._index];
+        value = chart.data.datasets[firstPoint._datasetIndex].data[firstPoint._index];
+        console.log("Label: " + label + ", Value: " + value);
+        new_location = window.location.toString();
+        new_location = new_location.replace("&search[dimensions][]=<%= @dimensions[0][:url_param_name] %>", '');
+        new_location = new_location + '&search[filter][<%= @dimensions[0][:url_param_name] %>][]=' + label;
+        console.log(new_location);
+        window.location = new_location;
+      }
+    });
+  <% end %>
+<% end %>

data/app/views/drilldown/_excel_record_list.builder

@@ -2,6 +2,9 @@
 
 xml << render(partial: '/drilldown/excel_row_header')
 
-result[:transactions].each do |t|
-  xml << render(partial: '/drilldown/excel_row', locals: { transaction: t, previous_transaction: nil, errors: [], error_row: false, meter1_errors: false })
+result[:records].each do |t|
+  xml << render(
+    partial: '/drilldown/excel_row',
+    locals: { transaction: t, previous_transaction: nil, errors: [], error_row: false, meter1_errors: false }
+  )
 end

data/app/views/drilldown/_excel_row.builder

@@ -6,7 +6,12 @@ xml.Row do
 
   @search.fields.each_with_index do |field, i|
     if field == 'time'
-      value = (transaction.respond_to?(:completed_at) ? transaction.completed_at : transaction.created_at).localtime.strftime('%Y-%m-%d %H:%M')
+      value = ((
+      if transaction.respond_to?(:completed_at)
+        transaction.completed_at
+      else
+        transaction.created_at
+      end)).localtime.strftime('%Y-%m-%d %H:%M')
     else
       value = if @transaction_fields_map[field.to_sym][:attr_method]
                 @transaction_fields_map[field.to_sym][:attr_method].call(transaction)

data/app/views/drilldown/_excel_summary_row.builder

@@ -9,7 +9,11 @@ xml.Row do
     end
   end
   if dimension > 0
-    xml.Cell('ss:StyleID' => 'Outer', 'ss:Index' => dimension.to_s) { xml.Data value_label(dimension - 1, result[:value]), 'ss:Type' => 'String' }
+    xml.Cell('ss:StyleID' => 'Outer',
+             'ss:Index' => dimension.to_s) do
+      xml.Data value_label(dimension - 1, result[:value]),
+               'ss:Type' => 'String'
+    end
   end
 
   xml.Cell('ss:StyleID' => 'Outer') { xml.Data result[:count].inspect, 'ss:Type' => 'Number' }

data/app/views/drilldown/_export_links.html.erb

@@ -0,0 +1,2 @@
+<%= link_to 'Excel', @search.url_options.merge(action: :excel_export) %>&nbsp;|
+<%= link_to 'HTML', @search.url_options.merge(action: :html_export), data_popup: ['Elections', 'toolbar=no,location=no,directories=no,status=no,menubar=no,scrollbars=yes,resizable=yes, width=1024px'] %>

data/app/views/drilldown/_field.html.erb

@@ -22,7 +22,7 @@
   </tr>
 <% else %>
   <div class="form-group">
-    <%= form.label "filter[#{dimension_name}]", t(dimension_name) %>
+    <%= form.label "filter[#{dimension_name}]", t(dimension_name, default: :"activerecord.models.#{dimension_name}") %>
     <%= select :search, :filter,
         choices,
         { :selected => @search.filter[dimension_name] },

data/app/views/drilldown/_record_list.html.erb

@@ -1,9 +1,9 @@
-<% unless result[:transactions].empty? %>
+<% unless result[:records].empty? %>
   <tr>
     <td colspan="<%= controller.c_summary_fields.size + 1 %>">
       <table class="table table-condensed table-bordered" style="padding-bottom: 10px;">
         <%= render :partial => '/drilldown/row_header' %>
-        <% result[:transactions].each do |t| %>
+        <% result[:records].each do |t| %>
           <%= render :partial => '/drilldown/row', :locals => { :transaction => t, :previous_transaction => nil, :errors => [], :error_row => false, :meter1_errors => false } %>
         <% end %>
       </table>

data/app/views/drilldown/data_1.builder

@@ -4,8 +4,12 @@ xml.chart(xAxisName: (@dimensions[0][:pretty_name] || 'Elections').gsub("'", '')
           showValues: '1', caption: caption, subcaption: subcaption,
           yAxisName: "Election #{t(@search.select_value.downcase)}", numberSuffix: '') do
   @result[:rows].each do |res|
-    xml.set(name: @dimensions[0][:label_method] ? @dimensions[0][:label_method].call(res[:value]) : res[:value],
+    xml.set name: @dimensions[0][:label_method] ? @dimensions[0][:label_method].call(res[:value]) : res[:value],
             value: res[@search.select_value.downcase.to_sym],
-            link: @dimensions[0][:url_param_name] ? CGI.escape(url_for(@search.drill_down(@dimensions, res[:value]).url_options)) : '')
+            link: if @dimensions[0][:url_param_name]
+                    CGI.escape(url_for(@search.drill_down(@dimensions, res[:value]).url_options))
+                  else
+                    ''
+                  end
   end
 end

data/app/views/drilldown/data_2.builder

@@ -1,23 +1,35 @@
 # frozen_string_literal: true
 
-xml.chart xAxisName: (@dimensions[0][:pretty_name] || 'Elections').gsub("'", ''), palette: '2',
-          caption: caption, subcaption: subcaption,
-          showNames: '1', showValues: @result[:rows].size > 15 || @result[:rows][0] && @result[:rows][0][:rows].size > 4 ? 0 : 1, decimals: '0',
-          numberPrefix: '', clustered: '0', exeTime: '1.5', showPlotBorder: '0', zGapPlot: '30',
-          zDepth: '90', divLineEffect: 'emboss', startAngX: '10', endAngX: '18', startAngY: '-10',
-          numberSuffix: '', zAxisName: 'Z Axis', yAxisName: "Transaction #{t(@search.select_value.downcase)}", endAngY: '-40' do
+xml.chart(
+  xAxisName: (@dimensions[0][:pretty_name] || 'Elections').gsub("'", ''), palette: '2',
+  caption: caption, subcaption: subcaption,
+  showNames: '1',
+  showValues: @result[:rows].size > 15 || @result[:rows][0] && @result[:rows][0][:rows].size > 4 ? 0 : 1,
+  decimals: '0',
+  numberPrefix: '', clustered: '0', exeTime: '1.5', showPlotBorder: '0', zGapPlot: '30',
+  zDepth: '90', divLineEffect: 'emboss', startAngX: '10', endAngX: '18', startAngY: '-10',
+  numberSuffix: '', zAxisName: 'Z Axis',
+  yAxisName: "Transaction #{t(@search.select_value.downcase)}", endAngY: '-40'
+) do
   unless @result[:rows].empty?
     xml.categories do
       @result[:rows].each do |result|
-        xml.category label: @dimensions[0][:label_method] ? @dimensions[0][:label_method].call(result[:value]) : result[:value]
+        label_method = @dimensions[0][:label_method]
+        xml.category label: label_method ? label_method.call(result[:value]) : result[:value]
       end
     end
 
     @result[:rows][0][:rows].reverse.each_with_index do |result, i|
-      xml.dataset seriesName: @dimensions[1][:label_method] ? @dimensions[1][:label_method].call(result[:value]) : result[:value] do
+      name = if @dimensions[1][:label_method]
+               @dimensions[1][:label_method].call(result[:value])
+             else
+               result[:value]
+             end
+      xml.dataset seriesName: name do
         @result[:rows].each do |res|
           value = res[:rows].reverse[i][:value]
-          xml.set(label: @dimensions[0][:label_method] ? @dimensions[0][:label_method].call(res[:value]) : "#{res[:value]}, #{value}",
+          label_method = @dimensions[0][:label_method]
+          xml.set(label: (label_method ? label_method.call(res[:value]) : "#{res[:value]}, #{value}"),
                   value: res[:rows].reverse[i][@search.select_value.downcase.to_sym],
                   link: CGI.escape(url_for(@search.drill_down(@dimensions, res[:value], value).url_options)))
         end

data/app/views/drilldown/data_3.builder

@@ -1,25 +1,48 @@
 # frozen_string_literal: true
 
-xml.chart xAxisName: (@dimensions[0][:pretty_name] || 'Elections').gsub("'", ''), palette: '2',
-          caption: caption, subcaption: subcaption,
-          showNames: '1', showValues: @result[:rows].size > 15 || @result[:rows][0] && @result[:rows][0][:rows].size > 4 ? 0 : 1, decimals: '0',
-          numberPrefix: '', clustered: '0', exeTime: '1.5', showPlotBorder: '0', zGapPlot: '30',
-          zDepth: '90', divLineEffect: 'emboss', startAngX: '10', endAngX: '18', startAngY: '-10',
-          numberSuffix: '', zAxisName: 'Z Axis', yAxisName: "Transaction #{t(@search.select_value.downcase)}", endAngY: '-40' do
+xml.chart(
+  xAxisName: (@dimensions[0][:pretty_name] || 'Elections').gsub("'", ''), palette: '2',
+  caption: caption, subcaption: subcaption,
+  showNames: '1',
+  showValues:
+      @result[:rows].size > 15 || @result[:rows][0] && @result[:rows][0][:rows].size > 4 ? 0 : 1,
+  decimals: '0',
+  numberPrefix: '', clustered: '0', exeTime: '1.5', showPlotBorder: '0', zGapPlot: '30',
+  zDepth: '90', divLineEffect: 'emboss', startAngX: '10', endAngX: '18', startAngY: '-10',
+  numberSuffix: '', zAxisName: 'Z Axis',
+  yAxisName: "Transaction #{t(@search.select_value.downcase)}", endAngY: '-40'
+) do
   unless @result[:rows].empty?
     xml.categories do
       @result[:rows].each do |result|
-        xml.category label: @dimensions[0][:label_method] ? @dimensions[0][:label_method].call(result[:value]) : result[:value]
+        xml.category label: if @dimensions[0][:label_method]
+                              @dimensions[0][:label_method].call(result[:value])
+                            else
+                              result[:value]
+                            end
       end
     end
 
     @result[:rows][0][:rows].reverse.each_with_index do |result, i|
-      xml.dataset seriesName: @dimensions[1][:label_method] ? @dimensions[1][:label_method].call(result[:value]) : result[:value] do
+      series_name =
+        if @dimensions[1][:label_method]
+          @dimensions[1][:label_method].call(result[:value])
+        else
+          result[:value]
+        end
+      xml.dataset seriesName: series_name do
         @result[:rows].each do |res|
           value = res[:rows].reverse[i][:value]
-          xml.set(label: @dimensions[0][:label_method] ? @dimensions[0][:label_method].call(res[:value]) : "#{res[:value]}, #{value}",
-                  value: res[:rows].reverse[i][@search.select_value.downcase.to_sym],
-                  link: CGI.escape(url_for(@search.drill_down(@dimensions, res[:value], value).url_options)))
+          xml.set(
+            label:
+                if @dimensions[0][:label_method]
+                  @dimensions[0][:label_method].call(res[:value])
+                else
+                  "#{res[:value]}, #{value}"
+                end,
+            value: res[:rows].reverse[i][@search.select_value.downcase.to_sym],
+            link: CGI.escape(url_for(@search.drill_down(@dimensions, res[:value], value).url_options))
+          )
         end
       end
     end

data/app/views/drilldown/excel_export.builder

@@ -15,12 +15,14 @@ xml.Workbook(
   xml.Worksheet 'ss:Name' => 'Transaction Summary' do
     xml.Table do
       xml.Row 'ss:Height' => '18.75' do
-        xml.Cell 'ss:MergeAcross' => @search.list ? @search.fields.size - 1 : @dimensions.size + 2, 'ss:StyleID' => 'MainTitle' do
+        xml.Cell 'ss:MergeAcross' => @search.list ? @search.fields.size - 1 : @dimensions.size + 2,
+                 'ss:StyleID' => 'MainTitle' do
           xml.Data caption, 'ss:Type' => 'String'
         end
       end
       xml.Row 'ss:Height' => '15.75' do
-        xml.Cell 'ss:MergeAcross' => @search.list ? @search.fields.size - 1 : @dimensions.size + 2, 'ss:StyleID' => 'SubTitle' do
+        xml.Cell 'ss:MergeAcross' => @search.list ? @search.fields.size - 1 : @dimensions.size + 2,
+                 'ss:StyleID' => 'SubTitle' do
           xml.Data subcaption, 'ss:Type' => 'String'
         end
       end

data/app/views/drilldown/excel_export_transactions.builder

@@ -12,7 +12,7 @@ xml.Workbook(
 ) do
   xml << render(partial: '/layouts/excel_styles')
 
-  xml.Worksheet 'ss:Name' => 'Transactions' do
+  xml.Worksheet 'ss:Name' => 'Drilldown' do
     xml.Table do
       xml.Row 'ss:Height' => '18.75' do
         xml.Cell 'ss:MergeAcross' => '35', 'ss:StyleID' => 'MainTitle' do
@@ -42,7 +42,7 @@ xml.Workbook(
         end
       end
 
-      @transactions.each do |transaction|
+      @records.each do |transaction|
         xml.Row do
           @transaction_fields.each do |field|
             field_map = controller.c_fields[field.to_sym]

data/app/views/drilldown/index.html.erb

@@ -13,25 +13,25 @@
     }
 </style>
 
-<%= form_for @search, :html => {:id => 'edit_search_SEARCH', :method => :get, :class => :search, :style => 'background: inherit', :onsubmit => 'return this.submit()'}, :url => {} do |form| %>
+<%= form_for @search, html: { id: 'edit_search_SEARCH', method: :get, class: :search, style: 'background: inherit', onsubmit: 'return this.submit()' }, url: {} do |form| %>
     <div class="row">
       <div class="col-md-3" valign="top">
         <ul class="nav nav-tabs">
-          <li class="active">
-            <a href="#filter" data-toggle="tab"><em><%= t :filter %></em></a></li>
-          <li><a href="#fields" data-toggle="tab"><em><%= t :fields %></em></a>
+          <li class="nav-item">
+            <a id="filter-tab" class="nav-link active" data-target="#filter" data-toggle="tab"><%= t :filter %></a></li>
+          <li><a id="fields-tab" class="nav-link" data-target="#fields" data-toggle="tab"><%= t :fields %></a>
           </li>
         </ul>
 
         <!-- Tab panes -->
         <div class="tab-content">
           <div class="tab-pane active" id="filter">
-            <%= render :partial => '/drilldown/tab_buttons', :locals => {:form => form} %>
-            <%= render :partial => '/drilldown/filter', :locals => {:form => form} %>
+            <%= render '/drilldown/tab_buttons', form: form %>
+            <%= render '/drilldown/filter', form: form  %>
           </div>
           <div class="tab-pane" id="fields">
-            <%= render :partial => '/drilldown/tab_buttons', :locals => {:form => form} %>
-            <%= render :partial => '/drilldown/fields', :locals => {:form => form} %>
+            <%= render '/drilldown/tab_buttons', form: form %>
+            <%= render '/drilldown/fields', form: form %>
           </div>
         </div>
 
@@ -39,20 +39,16 @@
       <div class="col-md-9">
         <div class="row" valign="top">
           <div class="col-md-12">
-            <%= render :partial => '/drilldown/chart', :locals => {:form => form} %>
-            <br/>
+            <%= render '/drilldown/chart', form: form %>
           </div>
         </div>
 
         <div class="row">
           <div class="col-md-12" valign="top">
             <div style="float: right; clear: both;">
-              <%= link_to 'Excel', @search.url_options.merge(action: :excel_export) %>&nbsp;|
-              <%= link_to 'HTML', @search.url_options.merge(action: :html_export), :data_popup => ['Elections', 'toolbar=no,location=no,directories=no,status=no,menubar=no,scrollbars=yes,resizable=yes, width=1024px'] %>
+              <%= render '/drilldown/export_links' %>
             </div>
-            <br/>
-
-            <%= render :partial => '/drilldown/summary_table' %>
+            <%= render '/drilldown/summary_table' %>
           </div>
         </div>
       </div>
@@ -60,16 +56,16 @@
 
 <% end %> <!-- END  FORM -->
 
-<%=javascript_tag do %>
-form = document.getElementById("edit_search_SEARCH");
-form.submit = function() {
-    elements = form.elements;
-    params = new Array();
-    for (var el, i = 0; (el = elements[i] ); i++) {
-      if (el.type == "text" && el.value == "") {
+<%= javascript_tag do %>
+  form = document.getElementById("edit_search_SEARCH");
+  form.submit = function() {
+    let elements = form.elements;
+    let params = [];
+    for (let el, i = 0; (el = elements[i] ); i++) {
+      if (el.name === '' || (el.type === "text" && el.value === "")) {
         continue;
       }
-      if (el.type == "select-multiple") {
+      if (el.type === "select-multiple") {
         if (el.selectedIndex > 0) {
           for (var j = 0; j < el.options.length; j++) {
             // Ignore "All" and unselected options
@@ -80,28 +76,24 @@ form.submit = function() {
         }
         continue;
       }
-      if (el.type == "select-one" && el.selectedIndex == 0) {
+      if (el.type === "select-one" && el.selectedIndex === 0) {
         continue;
       }
-      if (el.type == "checkbox" && el.checked != true) {
+      if (el.type === "checkbox" && el.checked !== true) {
         continue;
       }
-      if (el.type == "radio" && el.checked != true) {
+      if (el.type === "radio" && (el.checked !== true || el.value === 'NONE')) {
         continue;
       }
-      if (el.type == "hidden" && (el.value == "0" || el.value == "")) {
+      if (el.type === "hidden" && (el.value === "0" || el.value === "")) {
         continue;
       }
-      if (el.type == "submit") {
+      if (el.type === "submit") {
         continue;
       }
       params.push("" + el.name + "=" + encodeURIComponent(el.value));
     }
     location = form.action + "?" + params.join('&');
     return false;
-};
-
-$(document).ready(function () {
-  $('[data-behaviour~=datepicker]').datepicker({format: "yyyy-mm-dd"});
-})
+  };
 <% end %>

data/lib/simple_drilldown/controller.rb

@@ -33,6 +33,7 @@ module SimpleDrilldown
           begin
             base.c_target_class = base.name.chomp('Controller').constantize
           rescue NameError
+            # No default target class found
           end
         end
       end
@@ -101,7 +102,9 @@ module SimpleDrilldown
         raise "Unexpected options: #{options.inspect}" if options.present?
 
         queries.each do |query_opts|
-          raise "Unknown options: #{query_opts.keys.inspect}" unless (query_opts.keys - %i[select includes where]).empty?
+          unless (query_opts.keys - %i[select includes where]).empty?
+            raise "Unknown options: #{query_opts.keys.inspect}"
+          end
         end
 
         c_dimension_defs[name.to_s] = {
@@ -122,10 +125,10 @@ module SimpleDrilldown
           interval: interval,
           label_method: label_method,
           legal_values: legal_values,
-          pretty_name: I18n.t(name),
+          pretty_name: I18n.t(name, default: :"activerecord.models.#{name}"),
           queries: queries,
           reverse: reverse,
-          select_expression: queries.size > 1 ? "COALESCE(#{queries.map { |q| q[:select] }.join(',')})" : queries[0][:select],
+          select_expression: "COALESCE(#{queries.map { |q| q[:select] }.join(',')})",
           row_class: row_class,
           url_param_name: name.to_s
         }
@@ -182,7 +185,7 @@ module SimpleDrilldown
             dimension_def = c_dimension_defs[field]
             raise "Unknown filter field: #{field.inspect}" if dimension_def.nil?
 
-            values = [*values]
+            values = Array(values)
             if dimension_def[:interval]
               values *= 2 if values.size == 1
               raise "Need 2 values for interval filter: #{values.inspect}" if values.size != 2
@@ -237,9 +240,9 @@ module SimpleDrilldown
         when Hash
           sql = +''
           include.each do |parent, child|
-            sql << ' ' + make_join(joins, model, parent)
+            sql << " #{make_join(joins, model, parent)}"
             ass = model.to_s.camelize.constantize.reflect_on_association parent
-            sql << ' ' + make_join(joins, parent, child, ass.class_name.constantize)
+            sql << " #{make_join(joins, parent, child, ass.class_name.constantize)}"
           end
           sql
         when Symbol
@@ -373,8 +376,7 @@ module SimpleDrilldown
       selected = @search.filter[dimension_name] || []
       raise "Unknown dimension #{dimension_name.inspect}: #{c_dimension_defs.keys.inspect}" unless dimension
 
-      choices = [[t(:all), nil]] +
-                (dimension[:legal_values]&.call(@search)&.map { |o| o.is_a?(Array) ? o[0..1].map(&:to_s) : o.to_s } || [])
+      choices = [[t(:all), nil]] + (legal_values_for_dimension(dimension) || [])
       choices_html = choices.map do |c|
         %(<option value="#{c[1]}"#{' SELECTED' if selected.include?(c[1])}>#{c[0]}</option>)
       end.join("\n")
@@ -388,31 +390,29 @@ module SimpleDrilldown
 
     def excel_export
       index(false)
-      headers['Content-Type'] = 'application/vnd.ms-excel'
-      headers['Content-Disposition'] = 'attachment; filename="transactions.xml"'
-      headers['Cache-Control'] = ''
+      set_excel_headers
       render template: '/drilldown/excel_export', layout: false
     end
 
-    def excel_export_transactions
+    def excel_export_records
       params[:search][:list] = '1'
       index(false)
-      @transactions = get_transactions(@result)
-      headers['Content-Type'] = 'application/vnd.ms-excel'
-      headers['Content-Disposition'] = 'attachment; filename="transactions.xml"'
-      render template: '/drilldown/excel_export_transactions', layout: false
+      @records = get_records(@result)
+      set_excel_headers
+      render template: '/drilldown/excel_export_records', layout: false
     end
 
-    def xml_export
-      params[:search][:list] = '1'
-      index(false)
-      @transactions = get_transactions(@result)
-      headers['Content-Type'] = 'text/xml'
-      headers['Content-Disposition'] = 'attachment; filename="transactions.xml"'
-      render template: '/drilldown/xml_export', layout: false
+    private
+
+    def set_excel_headers
+      headers['Content-Type'] = 'application/vnd.ms-excel'
+      headers['Content-Disposition'] = %{attachment; filename="#{c_target_class.table_name}.xml"}
+      headers['Cache-Control'] = ''
     end
 
-    private
+    def legal_values_for_dimension(dimension)
+      dimension[:legal_values]&.call(@search)&.map { |o| o.is_a?(Array) ? o[0..1].map(&:to_s) : o.to_s }
+    end
 
     def new_search_object
       SimpleDrilldown::Search.new(params[:search]&.to_unsafe_h, c_default_fields, c_default_select_value)
@@ -495,7 +495,8 @@ module SimpleDrilldown
 
       result_rows = []
       loop do
-        sub_result = result_from_rows(rows, row_index, dimension + 1, values + [rows[row_index]["value#{dimension + 1}"]])
+        sub_result = result_from_rows(rows, row_index, dimension + 1,
+                                      values + [rows[row_index]["value#{dimension + 1}"]])
         break if sub_result.nil?
 
         result_rows << sub_result
@@ -517,23 +518,20 @@ module SimpleDrilldown
     end
 
     def populate_list(conditions, includes, result, values)
-      if result[:rows]
-        return result[:rows].each { |r| populate_list(conditions, includes, r, values + [r[:value]]) }
-      end
+      return result[:rows].each { |r| populate_list(conditions, includes, r, values + [r[:value]]) } if result[:rows]
+
       list_includes = merge_includes(includes, c_list_includes)
       @search.fields.each do |field|
         field_def = c_fields[field.to_sym]
         raise "Field definition missing for: #{field.inspect}" unless field_def
 
         field_includes = field_def[:include]
-        if field_includes
-          list_includes = merge_includes(list_includes , field_includes)
-        end
+        list_includes = merge_includes(list_includes, field_includes) if field_includes
       end
       if @search.list_change_times
         @history_fields.each do |f|
           if @search.fields.include? f
-            list_includes = merge_includes(list_includes, assignment: { order: :"#{f}_changes" } )
+            list_includes = merge_includes(list_includes, assignment: { order: :"#{f}_changes" })
           end
         end
       end
@@ -541,17 +539,17 @@ module SimpleDrilldown
       list_conditions = list_conditions(conditions, values)
       base_query = c_target_class.unscoped.where(c_base_condition).joins(joins).order(c_list_order)
       base_query = base_query.where(list_conditions) if list_conditions
-      result[:transactions] = base_query.to_a
+      result[:records] = base_query.to_a
     end
 
     def merge_includes(*args)
       hash = hash_includes(*args)
-      result = hash.dup.map { |k, v|
+      result = hash.dup.map do |k, v|
         if v.blank?
           hash.delete(k)
           k
         end
-      }.compact
+      end.compact
       result << hash unless hash.blank?
       case result.size
       when 0
@@ -567,14 +565,15 @@ module SimpleDrilldown
       args.inject({}) do |h, inc|
         case inc
         when Array
-          inc.each { |v|
+          inc.each do |v|
             h = hash_includes(h, v)
-          }
+          end
         when Hash
-          inc.each { |k, v|
+          inc.each do |k, v|
             h[k] = merge_includes(h[k], v)
-          }
+          end
         when NilClass, FalseClass
+          # Leave as it is
         when String, Symbol
           h[inc] ||= []
         else
@@ -594,10 +593,10 @@ module SimpleDrilldown
       [list_conditions_string, *(conditions[1..-1] + values)]
     end
 
-    def get_transactions(tree)
-      return tree[:transactions] if tree[:transactions]
+    def get_records(tree)
+      return tree[:records] if tree[:records]
 
-      tree[:rows].map { |r| get_transactions(r) }.flatten
+      tree[:rows].map { |r| get_records(r) }.flatten
     end
 
     class ScopeHolder

data/lib/simple_drilldown/helper.rb

@@ -11,9 +11,8 @@ module SimpleDrilldown
     end
 
     def caption
-      result = @search.title || "#{controller.c_target_class} #{t(@search.select_value.downcase)}" +
-                                (@dimensions && @dimensions.any? ? ' by ' + @dimensions.map { |d| d[:pretty_name] }.join(' and ') : '')
-      result.gsub('$date', [*@search.filter[:calendar_date]].uniq.join(' - '))
+      result = @search.title || caption_txt
+      result.gsub('$date', Array(@search.filter[:calendar_date]).uniq.join(' - '))
     end
 
     def subcaption
@@ -21,9 +20,15 @@ module SimpleDrilldown
     end
 
     def summary_row(result, parent_result = nil, dimension = 0, headers = [], new_row = true)
-      html = render(partial: '/drilldown/summary_row', locals: { result: result, parent_result: parent_result, new_row: new_row, dimension: dimension, headers: headers, with_results: !result[:rows] })
+      html = render(partial: '/drilldown/summary_row', locals: {
+                      result: result, parent_result: parent_result, new_row: new_row, dimension: dimension,
+                      headers: headers, with_results: !result[:rows]
+                    })
       if result[:rows]
-        sub_headers = headers + [{ value: result[:value], display_row_count: result[:nodes] + result[:row_count] * (@search.list ? 1 : 0) }]
+        sub_headers = headers + [{
+          value: result[:value],
+          display_row_count: result[:nodes] + result[:row_count] * (@search.list ? 1 : 0)
+        }]
         significant_rows = result[:rows].reject { |r| r[:row_count].zero? }
         significant_rows.each_with_index do |r, i|
           html << summary_row(r, result, dimension + 1, sub_headers, i.positive?)
@@ -31,7 +36,12 @@ module SimpleDrilldown
       elsif @search.list
         html << render(partial: '/drilldown/record_list', locals: { result: result })
       end
-      html << render(partial: '/drilldown/summary_total_row', locals: { result: result, parent_result: parent_result, headers: headers.dup, dimension: dimension }) if dimension < @dimensions.size
+      if dimension < @dimensions.size
+        html << render(partial: '/drilldown/summary_total_row',
+                       locals: {
+                         result: result, parent_result: parent_result, headers: headers.dup, dimension: dimension
+                       })
+      end
 
       html
     end
@@ -41,25 +51,42 @@ module SimpleDrilldown
       if result[:rows]
         significant_rows = result[:rows].reject { |r| r[:row_count].zero? }
         significant_rows.each_with_index do |r, i|
-          sub_headers = if i.zero?
-                          if dimension.zero?
-                            headers
-                          else
-                            headers + [{ value: result[:value], display_row_count: result[:nodes] + result[:row_count] * (@search.list ? 1 : 0) }]
-                          end
-                        else
-                          [] # [{:value => result[:value], :row_count => result[:row_count]}]
-                        end
+          sub_headers =
+            if i.zero?
+              if dimension.zero?
+                headers
+              else
+                headers + [{
+                  value: result[:value],
+                  display_row_count: result[:nodes] + result[:row_count] * (@search.list ? 1 : 0)
+                }]
+              end
+            else
+              [] # [{:value => result[:value], :row_count => result[:row_count]}]
+            end
           xml << excel_summary_row(r, result, dimension + 1, sub_headers)
         end
       else
-        xml << render(partial: '/drilldown/excel_summary_row', locals: { result: result, parent_result: parent_result, headers: headers.dup, dimension: dimension })
+        xml << render(partial: '/drilldown/excel_summary_row',
+                      locals: { result: result, parent_result: parent_result, headers: headers.dup,
+                                dimension: dimension })
 
         xml << render(partial: '/drilldown/excel_record_list', locals: { result: result }) if @search.list
       end
 
-      xml << render(partial: '/drilldown/excel_summary_total_row', locals: { result: result, headers: headers.dup, dimension: dimension }) if dimension < @dimensions.size
+      if dimension < @dimensions.size
+        xml << render(partial: '/drilldown/excel_summary_total_row', locals: {
+                        result: result, headers: headers.dup, dimension: dimension
+                      })
+      end
       xml
     end
+
+    private
+
+    def caption_txt
+      "#{controller.c_target_class} #{t(@search.select_value.downcase)}" +
+        (@dimensions && @dimensions.any? ? " by #{@dimensions.map { |d| d[:pretty_name] }.join(' and ')}" : '')
+    end
   end
 end

data/lib/simple_drilldown/search.rb

@@ -19,17 +19,9 @@ module SimpleDrilldown
       VOLUME_COMPENSATED = 'VOLUME_COMPENSATED'
     end
 
-    attr_reader :dimensions
-    attr_reader :display_type
-    attr_reader :fields
-    attr_reader :filter
-    attr_accessor :list
-    attr_accessor :percent
-    attr_reader :list_change_times
-    attr_reader :order_by_value
-    attr_reader :select_value
-    attr_reader :title
-    attr_reader :default_fields
+    attr_reader :dimensions, :display_type, :fields, :filter, :list_change_times, :order_by_value,
+                :select_value, :title, :default_fields
+    attr_accessor :list, :percent
 
     def self.validators_on(_attribute)
       []
@@ -60,7 +52,7 @@ module SimpleDrilldown
         @dimensions = attributes && attributes[:dimensions] || []
         @dimensions.delete_if(&:empty?)
         @filter = attributes && attributes[:filter] ? attributes[:filter] : {}
-        @filter.keys.dup.each { |k| @filter[k] = [*@filter[k]] }
+        @filter.keys.dup.each { |k| @filter[k] = Array(@filter[k]) }
         @filter.each do |_k, v|
           v.delete('')
           v.delete('Select Some Options')
@@ -108,6 +100,10 @@ module SimpleDrilldown
       'SEARCH'
     end
 
+    def list?
+      list
+    end
+
     def drill_down(dimensions, *values)
       raise 'Too many values' if values.size > self.dimensions.size
 

data/lib/simple_drilldown/version.rb

@@ -1,3 +1,3 @@
 module SimpleDrilldown
-  VERSION = '0.6.3'
+  VERSION = '0.6.8'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: simple_drilldown
 version: !ruby/object:Gem::Version
-  version: 0.6.3
+  version: 0.6.8
 platform: ruby
 authors:
 - Uwe Kubosch
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-09-11 00:00:00.000000000 Z
+date: 2020-09-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: chartkick
@@ -97,6 +97,7 @@ files:
 - app/views/drilldown/_excel_styles.builder
 - app/views/drilldown/_excel_summary_row.builder
 - app/views/drilldown/_excel_summary_total_row.builder
+- app/views/drilldown/_export_links.html.erb
 - app/views/drilldown/_field.html.erb
 - app/views/drilldown/_fields.html.erb
 - app/views/drilldown/_filter.html.erb
@@ -145,7 +146,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="