simple_drilldown 0.7.4 → 0.9.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1a36aab8c580f75685e00d36a6aea2bbb1b58b43c153db650ec9298cf0b5fff6
-  data.tar.gz: 6c1a7cda78d09fdff2519c1db52dd4c5aa7f2c7c273bc7b9f6bc730cf02b6d80
+  metadata.gz: f1987db21273c12d879bae55633fd337003d7ced07c152a09e9269f88d8928af
+  data.tar.gz: b6e9149cbcc597a3e407aac96936c77956c9489d87a4ec14beb225d6e993cbc4
 SHA512:
-  metadata.gz: 587b979884e3cdb8a85fee6438358bf5ba3878ff55b1355b52e987b1b896d686558fbcde181d9c1bc1a62b0ec7a6fbe591fb36c621d59d5688e0655c31170dff
-  data.tar.gz: a4e90f4858363cea570a5544efb0ed45bb4ae541a4a2238511bf726755f6ced093e5f9dde0574f21f6e1b7621b9564788c9795f2ebd1e8c721014472b2f50d44
+  metadata.gz: b7325de41bc98a3b712d8a9cdf4c52099208d35e1026cff81a4c75be4bd2e33933000bc6f0df9aceb36ce0979826cc3819fac444f0f1ec1626c964aa31aca167
+  data.tar.gz: a2185eeda0623ece2aca690e92419cdc7821f18321e4c4cd6a7d02b4159ba0e970739f5df1928661d244d84b9eaacd1371974dbaeb84d618718fb4b4a2b7e71d

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

@@ -1,19 +1,19 @@
 <%
   data =
-      case @dimensions.size
-      when 0
-        { @result[:value] => @result[:count] }
-      when 1
-        @result[:rows].map { |r| [@dimensions[0][:label_method] ? @dimensions[0][:label_method].call(r[:value]) : r[:value], r[:count]] }
-      when 2
-        @result[:rows].map do |r|
-          {
-                  name: r[:value],
-                  data: r[:rows].map { |r2| [r2[:value], r2[:count]] }
-          }
-        end
-      when 3
+    case @dimensions.size
+    when 0
+      { @result[:value] => @result[:count] }
+    when 1
+      @result[:rows].map { |r| [@dimensions[0][:label_method] ? @dimensions[0][:label_method].call(r[:value]) : r[:value], r[:count]] }
+    when 2
+      @result[:rows].map do |r|
+        {
+          name: r[:value],
+          data: r[:rows].map { |r2| [r2[:value], r2[:count]] }
+        }
       end
+    when 3
+    end
 %>
 
 <%
@@ -37,10 +37,10 @@
   <% (0..2).each do |i|
     options = [['', '']]
     options << [@dimensions[i][:pretty_name], @dimensions[i][:url_param_name]] if @dimensions[i]
-    options += @remaining_dimensions.keys.map { |name| [controller.c_dimension_defs[name][:pretty_name], name] }  %>
+    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/>
@@ -62,21 +62,22 @@
 </div>
 
 <% if @dimensions.size == 1 &&
-    [SimpleDrilldown::Search::DisplayType::BAR, SimpleDrilldown::Search::DisplayType::PIE].include?(@search.display_type) %>
+  [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;
-      }
+    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(/#.*$/, '');
+    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_xlsx.xlsx.axlsx

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+def excel_header_row(sheet)
+  padding_cells = @dimensions.empty? ? 1 : @dimensions.size
+  row = (1...(padding_cells - 1)).map { |_n| nil }
+  @search.fields.each_with_index { |field, _i| row << t(field) }
+  sheet.add_row row
+end
+
+def excel_row(sheet, wrap, transaction:)
+  padding_cells = @dimensions.empty? ? 1 : @dimensions.size
+  row = (1..(padding_cells - 1)).map { |_n| nil }
+
+  @search.fields.each_with_index do |field, _i|
+    value = if field == 'time'
+              (transaction.respond_to?(:completed_at) ? transaction.completed_at : transaction.created_at)
+                .localtime.strftime('%Y-%m-%d %H:%M')
+            elsif controller.c_fields[field.to_sym][:attr_method]
+              controller.c_fields[field.to_sym][:attr_method].call(transaction)
+            else
+              transaction.send(field)
+            end
+
+    field_def = controller.c_fields[field.to_sym]
+    if @search.list_change_times && field_def[:list_change_times] && transaction.assignment.try(:order)
+      changes = transaction.assignment.order.send("#{field}_changes")
+                           .map do |al|
+        [al.created_at.localtime.strftime('%H:%M'),
+         PacMan.yaml_load(al.new_values).symbolize_keys[field.to_sym]]
+      end
+      last_change = changes.last.try(:[], 1)
+      value =
+        "#{changes.map { |al| al.join(' ') }.join("\n")}#{"\nActual: #{value}" if value != last_change}"
+    end
+
+    row << value
+  end
+  sheet.add_row row, style: wrap
+end
+
+sheet.add_row []
+excel_header_row(sheet)
+wrap = sheet.styles.add_style alignment: { wrap_text: true }
+result[:records].each { |t| excel_row(sheet, wrap, transaction: t) }
+sheet.add_row []

data/app/views/drilldown/_excel_summary_row_xlsx.xlsx.axlsx

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+row = [
+  *(1..(dimension - headers.size - 1)).map { |_n| '' },
+  *headers.map.with_index { |h, i| value_label(@dimensions.size - headers.size + i - 1, h[:value]) },
+  *(value_label(dimension - 1, result[:value]) if dimension.positive?),
+  *((result[:count].to_f / parent_result[:count]).inspect if parent_result && @search.percent),
+  result[:count].inspect,
+  *(if parent_result && @search.percent
+      (
+        if parent_result[controller.c_summary_fields[0]].positive?
+          result[controller.c_summary_fields[0]].to_f / parent_result[controller.c_summary_fields[0]]
+        else
+          0
+        end).inspect
+    end),
+  result[controller.c_summary_fields[0]].inspect
+]
+if controller.c_summary_fields.size > 1
+  if parent_result && @search.percent
+    percent =
+      if parent_result[controller.c_summary_fields[1]].positive?
+        result[controller.c_summary_fields[1]].to_f / parent_result[controller.c_summary_fields[1]]
+      else
+        0
+      end
+    row << percent.inspect
+  end
+  row << result[controller.c_summary_fields[1]].inspect
+end
+sheet.add_row row

data/app/views/drilldown/_excel_summary_total_row_xlsx.xlsx.axlsx

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+row = (1..dimension).map { |_n| nil }
+
+(@dimensions.size - dimension).times { |_i| row << nil }
+row << result[:count]
+sheet.add_row row

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

@@ -1,2 +1,3 @@
 <%= link_to 'Excel', @search.url_options.merge(action: :excel_export) %>&nbsp;|
+<%= link_to 'Excel (XML)', @search.url_options.merge(action: :excel_export, format: :xml) %>&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

@@ -1,25 +1,17 @@
 <% if dimension_name == 'calendar_date' %>
   <% dates = [*@search.filter[dimension_name]] %>
-  <tr>
-    <td align="right" valign="top">
-      <%= form.label "filter[calendar_date]", t(:from_date) %>
-    </td>
-    <td>
-      <input type="text" data-behaviour="datepicker" id="search_filter[from_<%= dimension_name %>]"
-             name="search[filter][<%= dimension_name %>][]" class="form-control" value="<%= dates[0] %>"
-             placeholder="yyyy-mm-dd"/>
-    </td>
-  </tr>
-  <tr>
-    <td align="right" valign="top">
-      <%= form.label "filter[calendar_date]", t(:to_date) %>
-    </td>
-    <td>
-      <input type="text" data-behaviour="datepicker" id="search_filter[to_<%= dimension_name %>]"
-             name="search[filter][<%= dimension_name %>][]" class="form-control" value="<%= dates[1] || dates[0] %>"
-             placeholder="yyyy-mm-dd"/>
-    </td>
-  </tr>
+  <div class="form-group">
+    <%= form.label "filter[calendar_date]", t(:from_date) %>
+    <input type="text" data-behaviour="datepicker" id="search_filter[from_<%= dimension_name %>]"
+           name="search[filter][<%= dimension_name %>][]" class="form-control date" value="<%= dates[0] %>"
+           placeholder="yyyy-mm-dd"/>
+  </div>
+  <div class="form-group">
+    <%= form.label "filter[calendar_date]", t(:to_date) %>
+    <input type="text" data-behaviour="datepicker" id="search_filter[to_<%= dimension_name %>]"
+           name="search[filter][<%= dimension_name %>][]" class="form-control date" value="<%= dates[1] %>"
+           placeholder="yyyy-mm-dd"/>
+  </div>
 <% else %>
   <div class="form-group">
     <%= form.label "filter[#{dimension_name}]", t(dimension_name, default: :"activerecord.models.#{dimension_name}") %>

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

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

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

@@ -1,12 +1,14 @@
-<table class="table table-condensed table-bordered">
-<tr>
-  <% @dimensions.each do |d| %>
-    <th><%=h d[:pretty_name]%></th>
-  <% end %>
-  <th><%= t controller.c_target_class.table_name.capitalize %></th>
-  <%= controller.c_summary_fields.map{|l| "<th>#{t(l)}</th>"}.join("\n").html_safe %>
-</tr>
-
-<%=summary_row(@result) %>
-
+<table id="drilldown-summary-table" class="table table-condensed table-bordered">
+  <thead>
+    <tr>
+      <% @dimensions.each do |d| %>
+        <th><%= h d[:pretty_name] %></th>
+      <% end %>
+      <th><%= t controller.c_target_class.table_name.capitalize %></th>
+      <%= controller.c_summary_fields.map { |l| "<th>#{t(l)}</th>" }.join("\n").html_safe %>
+    </tr>
+  </thead>
+  <tbody>
+    <%= summary_row(@result) %>
+  </tbody>
 </table>

data/app/views/drilldown/excel_export_xlsx.xlsx.axlsx

@@ -0,0 +1,86 @@
+# frozen_string_literal: true
+
+def excel_summary_row_xlsx(sheet, result, parent_result = nil, dimension = 0, headers = [])
+  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
+      excel_summary_row_xlsx(sheet, r, result, dimension + 1, sub_headers)
+    end
+  else
+    render(partial: '/drilldown/excel_summary_row_xlsx',
+           locals: { sheet: sheet, result: result, parent_result: parent_result, headers: headers.dup,
+                     dimension: dimension })
+
+    if @search.list
+      render(partial: '/drilldown/excel_record_list_xlsx',
+             locals: { sheet: sheet, result: result })
+    end
+  end
+
+  return if dimension >= @dimensions.size
+
+  render(partial: '/drilldown/excel_summary_total_row_xlsx', locals: {
+           sheet: sheet, result: result, headers: headers.dup, dimension: dimension
+         })
+end
+
+xlsx_package.use_shared_strings = true
+wb = xlsx_package.workbook
+wb.add_worksheet(name: 'Transaction Summary') do |sheet|
+  # header_colspan =
+  #   if @search.list
+  #     [@dimensions.size - 1, 0].max + @search.fields.size - 1
+  #   else
+  #     @dimensions.size - 1 + (controller.c_summary_fields.size + 1) * (@search.percent ? 2 : 1)
+  #   end
+
+  # xml.Row 'ss:Height' => '18.75' do
+  #   xml.Cell 'ss:MergeAcross' => header_colspan, 'ss:StyleID' => 'MainTitle' do
+  #     xml.Data caption, 'ss:Type' => 'String'
+  #   end
+  # end
+  sheet.add_row [caption]
+
+  # xml.Row 'ss:Height' => '15.75' do
+  #   xml.Cell 'ss:MergeAcross' => header_colspan, 'ss:StyleID' => 'SubTitle' do
+  #     xml.Data subcaption, 'ss:Type' => 'String'
+  #   end
+  # end
+  sheet.add_row [subcaption]
+
+  # xml.Row do
+  #   @dimensions.each do |d|
+  #     xml.Cell 'ss:StyleID' => 'DimensionHeading' do
+  #       xml.Data (h d[:pretty_name]).to_s, 'ss:Type' => 'String'
+  #     end
+  #   end
+  #   xml.Cell 'ss:StyleID' => 'Heading', 'ss:MergeAcross' => @search.percent ? 1 : 0 do
+  #     xml.Data t(controller.c_target_class.table_name), 'ss:Type' => 'String'
+  #   end
+  #   controller.c_summary_fields.each do |f|
+  #     xml.Cell 'ss:StyleID' => 'Heading', 'ss:MergeAcross' => @search.percent ? 1 : 0 do
+  #       xml.Data f, 'ss:Type' => 'String'
+  #     end
+  #   end
+  # end
+  sheet.add_row [
+    *@dimensions.map { |d| (h d[:pretty_name]).to_s },
+    t(controller.c_target_class.table_name),
+    *controller.c_summary_fields.map { |f| f }
+  ]
+
+  excel_summary_row_xlsx(sheet, @result)
+end

data/app/views/layouts/simple_drilldown/application.html.erb

@@ -1,15 +1,12 @@
 <!DOCTYPE html>
 <html>
-<head>
-  <title>Simple drilldown</title>
-  <%= csrf_meta_tags %>
-  <%= csp_meta_tag %>
-
-  <%= stylesheet_link_tag    "simple_drilldown/application", media: "all" %>
-</head>
-<body>
-
-<%= yield %>
-
-</body>
+  <head>
+    <title>Simple drilldown</title>
+    <%= csrf_meta_tags %>
+    <%= csp_meta_tag %>
+    <%= stylesheet_link_tag "simple_drilldown/application", media: "all" %>
+  </head>
+  <body>
+    <%= yield %>
+  </body>
 </html>

data/lib/simple_drilldown.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require 'caxlsx_rails'
 require 'simple_drilldown/engine'
 
 module SimpleDrilldown

data/lib/simple_drilldown/changes.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+module SimpleDrilldown
+  # Allow tracking changes for a field
+  module Changes
+    def self.included(clazz)
+      clazz.extend ClassMethods
+    end
+
+    # Class methods for Changes
+    module ClassMethods
+      def changes_for(*fields)
+        fields.each do |field|
+          condition_proc = lambda do
+            in_join = is_a?(ActiveRecord::Associations::JoinDependency::JoinAssociation)
+            table_alias = in_join ? aliased_table_name : AuditLog.table_name
+            "#{table_alias}.new_values LIKE '%#{field}%'"
+          end
+          has_many :"#{field}_changes", -> { where(condition_proc.call).order(:created_at) },
+                   class_name: :AuditLog, foreign_key: :record_id
+        end
+      end
+    end
+  end
+end

data/lib/simple_drilldown/controller.rb

@@ -15,7 +15,7 @@ module SimpleDrilldown
     class_attribute :c_default_fields, default: []
     class_attribute :c_default_select_value, default: SimpleDrilldown::Search::SelectValue::COUNT
     class_attribute :c_dimension_defs
-    class_attribute :c_fields
+    class_attribute :c_fields, default: []
     class_attribute :c_list_includes, default: []
     class_attribute :c_list_order
     class_attribute :c_select, default: 'count(*) as count'
@@ -45,8 +45,8 @@ module SimpleDrilldown
         self.c_base_condition = base_condition
       end
 
-      def base_includes(base_includes)
-        self.c_base_includes = base_includes
+      def base_includes(*base_includes)
+        self.c_base_includes = base_includes.flatten
       end
 
       def base_group(base_group)
@@ -69,7 +69,7 @@ module SimpleDrilldown
         self.c_select = select
       end
 
-      def list_includes(list_includes)
+      def list_includes(*list_includes)
         self.c_list_includes = list_includes.flatten
       end
 
@@ -128,7 +128,7 @@ module SimpleDrilldown
           interval: interval,
           label_method: label_method,
           legal_values: legal_values,
-          pretty_name: I18n.t(name, default: :"activerecord.models.#{name}"),
+          pretty_name: I18n.t(name, default: [:"activerecord.models.#{name}", name.to_s.titleize]),
           queries: queries,
           reverse: reverse,
           select_expression:
@@ -153,14 +153,15 @@ module SimpleDrilldown
               end
               includes.uniq!
             end
-            rows = c_target_class.unscoped.where(c_base_condition)
-                                 .select("#{query[:select]} AS value")
-                                 .where(filter_conditions || '1=1')
-                                 .where(query[:where] || '1=1')
-                                 .joins(make_join([], c_target_class.name.underscore.to_sym, includes))
-                                 .order('value')
-                                 .group(:value)
-                                 .to_a
+            rows_query = c_target_class.unscoped.where(c_base_condition)
+                                       .select("#{query[:select]} AS value")
+                                       .joins(make_join([], c_target_class.name.underscore.to_sym, includes))
+                                       .order('value')
+                                       .group(:value)
+            # rows_query = rows_query.without_deleted if c_target_class.try :paranoid?
+            rows_query = rows_query.where(filter_conditions) if filter_conditions
+            rows_query = rows_query.where(query[:where]) if query[:where]
+            rows = rows_query.to_a
             filter_fields = search.filter[field.to_s]
             filter_fields&.each do |selected_value|
               next if rows.find { |r| r[:value].to_s == selected_value }
@@ -191,9 +192,6 @@ module SimpleDrilldown
 
             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
-
               if values[0].present? && values[1].present?
                 condition_strings << "#{dimension_def[:select_expression]} BETWEEN ? AND ?"
                 condition_values += values
@@ -262,7 +260,9 @@ module SimpleDrilldown
           include_alias = include.to_s.pluralize
           case ass.macro
           when :belongs_to
-            "LEFT JOIN #{include_table} #{include_alias} ON #{include_alias}.id = #{model_table}.#{include}_id"
+            pk_col = ass.association_primary_key
+            fk_col = ass.options[:foreign_key] || "#{include}_id"
+            "LEFT JOIN #{include_table} #{include_alias} ON #{include_alias}.#{pk_col} = #{model_table}.#{fk_col}"
           when :has_one, :has_many
             fk_col = ass.options[:foreign_key] || "#{model}_id"
             sql = +"LEFT JOIN #{include_table} #{include_alias} ON #{include_alias}.#{fk_col} = #{model_table}.id"
@@ -274,7 +274,8 @@ module SimpleDrilldown
                 ass_order_prefixed.gsub!(/\b#{cname}\b/, "#{include_alias}.#{cname}")
               end
               paranoid_clause = 'AND t2.deleted_at IS NULL' if ass.klass.paranoid?
-              # FIXME(uwe):  Should we add "where" from the ScopeHolder here as well?  Ref: DrilldownChanges#changes_for
+              # FIXME(uwe):  Should we add "where" from the ScopeHolder here as well?
+              #              Ref: SimpleDrilldown::Changes#changes_for
               min_query = <<~SQL
                 SELECT MIN(#{ass_order}) FROM #{include_table} t2 WHERE t2.#{fk_col} = #{model_table}.id #{paranoid_clause}
               SQL
@@ -347,8 +348,8 @@ module SimpleDrilldown
       rows = c_target_class.unscoped.where(c_base_condition).select(select).where(conditions)
                            .joins(joins)
                            .group(group)
-                           .order(order).to_a
-
+                           .order(order)
+                           .to_a
       if rows.empty?
         @result = { value: 'All', count: 0, row_count: 0, nodes: 0, rows: [] }
         c_summary_fields.each { |f| @result[f] = 0 }
@@ -395,12 +396,19 @@ module SimpleDrilldown
 
     def excel_export
       index(false)
-      set_excel_headers
-      if params.dig(:search, :list) == '1'
-        @records = get_records(@result)
-        render template: '/drilldown/excel_export_records', layout: false
-      else
-        render template: '/drilldown/excel_export', layout: false
+      respond_to do |format|
+        format.xlsx do
+          render xlsx: c_target_class.table_name, template: 'drilldown/excel_export_xlsx'
+        end
+        format.xml do
+          set_excel_headers
+          if params.dig(:search, :list) == '1'
+            @records = get_records(@result)
+            render template: 'drilldown/excel_export_records', layout: false
+          else
+            render template: 'drilldown/excel_export', layout: false
+          end
+        end
       end
     end
 

data/lib/simple_drilldown/engine.rb

@@ -9,7 +9,7 @@ module SimpleDrilldown
     config.autoload_paths << File.dirname(__dir__)
 
     initializer 'simple_drilldown.assets.precompile' do |app|
-      app.config.assets.precompile += %w[chartkick.js]
+      app.config.assets.precompile += %w[simple_drilldown/application.css chartkick.js]
     end
 
     ActionDispatch::Routing::Mapper.include SimpleDrilldown::Routing

data/lib/simple_drilldown/helper.rb

@@ -38,7 +38,7 @@ module SimpleDrilldown
           html << summary_row(r, result, dimension + 1, sub_headers, i.positive?)
         end
       elsif @search.list
-        html << render(partial: '/drilldown/record_list', locals: { result: result })
+        html << render(partial: '/drilldown/record_list', locals: { result: result, dimension: dimension })
       end
       if dimension < @dimensions.size
         html << render(partial: '/drilldown/summary_total_row',
@@ -89,7 +89,7 @@ module SimpleDrilldown
     private
 
     def caption_txt
-      "#{controller.c_target_class} #{t(@search.select_value.downcase)}" +
+      "#{controller.c_target_class} #{I18n.t(@search.select_value.downcase)}" +
         (@dimensions && @dimensions.any? ? " by #{@dimensions.map { |d| d[:pretty_name] }.join(' and ')}" : '')
     end
   end

data/lib/simple_drilldown/routing.rb

@@ -8,8 +8,11 @@ module SimpleDrilldown
       controller ||= path
       get "#{path}(.:format)" => "#{controller}#index", as: path
       scope path, controller: controller, as: path do
-        %i[excel_export excel_export_records html_export].each { |action| get action }
+        { excel_export: :xlsx, excel_export_records: :xlsx, html_export: :html }.each do |action, format|
+          get action, defaults: { format: format }
+        end
         get 'choices/:dimension_name', action: :choices, as: :choices
+        yield if block_given?
       end
     end
   end

data/lib/simple_drilldown/search.rb

@@ -81,15 +81,15 @@ module SimpleDrilldown
 
     def url_options
       o = {
-          search: {
-              title: title,
-              list: list ? '1' : '0',
-              percent: percent ? '1' : '0',
-              list_change_times: list_change_times ? '1' : '0',
-              filter: filter,
-              dimensions: dimensions,
-              display_type: display_type,
-          },
+        search: {
+          title: title,
+          list: list ? '1' : '0',
+          percent: percent ? '1' : '0',
+          list_change_times: list_change_times ? '1' : '0',
+          filter: filter,
+          dimensions: dimensions,
+          display_type: display_type,
+        },
       }
       o[:search][:fields] = fields unless fields == @default_fields
       o

data/lib/simple_drilldown/version.rb

@@ -1,3 +1,3 @@
 module SimpleDrilldown
-  VERSION = '0.7.4'
+  VERSION = '0.9.0'
 end

metadata

@@ -1,29 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: simple_drilldown
 version: !ruby/object:Gem::Version
-  version: 0.7.4
+  version: 0.9.0
 platform: ruby
 authors:
 - Uwe Kubosch
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-02 00:00:00.000000000 Z
+date: 2021-05-20 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: caxlsx_rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.6'
 - !ruby/object:Gem::Dependency
   name: chartkick
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.3'
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.3'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
@@ -91,12 +105,13 @@ files:
 - app/mailers/simple_drilldown/application_mailer.rb
 - app/models/simple_drilldown/application_record.rb
 - app/views/drilldown/_chart.html.erb
-- app/views/drilldown/_excel_record_list.builder
-- app/views/drilldown/_excel_row.builder
+- app/views/drilldown/_excel_record_list_xlsx.xlsx.axlsx
 - app/views/drilldown/_excel_row_header.builder
 - app/views/drilldown/_excel_styles.builder
 - app/views/drilldown/_excel_summary_row.builder
+- app/views/drilldown/_excel_summary_row_xlsx.xlsx.axlsx
 - app/views/drilldown/_excel_summary_total_row.builder
+- app/views/drilldown/_excel_summary_total_row_xlsx.xlsx.axlsx
 - app/views/drilldown/_export_links.html.erb
 - app/views/drilldown/_field.html.erb
 - app/views/drilldown/_fields.html.erb
@@ -114,6 +129,7 @@ files:
 - app/views/drilldown/data_3.builder
 - app/views/drilldown/excel_export.builder
 - app/views/drilldown/excel_export_records.builder
+- app/views/drilldown/excel_export_xlsx.xlsx.axlsx
 - app/views/drilldown/html_export.html.erb
 - app/views/drilldown/index.html.erb
 - app/views/drilldown/print.html.erb
@@ -126,6 +142,7 @@ files:
 - lib/generators/drilldown_controller/templates/drilldown_controller.rb.erb
 - lib/generators/drilldown_controller/templates/drilldown_controller_test.rb.erb
 - lib/simple_drilldown.rb
+- lib/simple_drilldown/changes.rb
 - lib/simple_drilldown/controller.rb
 - lib/simple_drilldown/engine.rb
 - lib/simple_drilldown/helper.rb
@@ -153,7 +170,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.3
+rubygems_version: 3.2.15
 signing_key:
 specification_version: 4
 summary: Simple data warehouse and drilldown.

data/app/views/drilldown/_excel_record_list.builder

@@ -1,10 +0,0 @@
-# frozen_string_literal: true
-
-xml << render(partial: '/drilldown/excel_row_header')
-
-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

@@ -1,29 +0,0 @@
-# frozen_string_literal: true
-
-xml.Row do
-  padding_cells = @dimensions.empty? ? 1 : @dimensions.size
-  1.upto(padding_cells - 1) { |_n| xml.Cell('ss:StyleID' => 'Outer') }
-
-  @search.fields.each_with_index do |field, i|
-    if field == 'time'
-      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)
-              else
-                transaction.send(field)
-              end
-    end
-
-    field_def = @transaction_fields_map[field.to_sym]
-
-    xml.Cell('ss:Index' => (padding_cells + i).to_s) do
-      xml.Data value, 'ss:Type' => 'String'
-    end
-  end
-end