simple_drilldown 0.9.3 → 0.9.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 787eb533d80540491493c1fb623d8358b72974059b007a838fe3f41c39f2314c
-  data.tar.gz: 284a49c988fe530ac7e64840ba2162c5afb4424217570905618f7595d2328d2b
+  metadata.gz: fb72497b4b3d1137bcba13a86c51f4547aa83200a6ef6cfd4940cc26da5dd431
+  data.tar.gz: e487608abefd17185ef9319e304a2489439a4053121f9c540639317e981141a1
 SHA512:
-  metadata.gz: b381290a734b9b701d4d33c86fb21e9ac7a6e9ccf5cae8b4c0263bccc5d80e8e155262cd916f6373d40b53732a1272c2d1c3b7bf8733b3d1d1744bbe7e797850
-  data.tar.gz: cc8569627e64cc18618d26f8e0036d22cfd79b31c6e5ad4752db8fe663c0f9053fb6fff3376e526e20e6d562379156c57d604992b85e076486cee9e7fd5342e2
+  metadata.gz: 23afcd4a1e3498d316ae6e8d48788fb93e982bbb5982a0e2c5b33277a30c4f14cf23f04c42bcda6fee0d8aec4242644f1e248340088f9cd473bf95f90582687a
+  data.tar.gz: a841a7dbfffbd22401acddea431d934cbd72d5ed82f8eca35816210230df1becb8d0bfd5ec60dd97f9af7c0f3db904eb4cd8342e82e731339730ff3cfb389e90

data/app/assets/stylesheets/simple_drilldown/application.css

@@ -19,7 +19,6 @@ th.header {
 td {
   border: 1px solid #dfded2;
   padding: 2px;
-  font-size: 10px;
 }
 
 td.numeric {

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

@@ -2,14 +2,14 @@
   data =
     case @dimensions.size
     when 0
-      { @result[:value] => @result[:count] }
+      { @result[:value] => @result[@search.select_value] }
     when 1
-      @result[:rows].map { |r| [@dimensions[0][:label_method] ? @dimensions[0][:label_method].call(r[:value]) : r[:value], r[:count]] }
+      @result[:rows].map { |r| [@dimensions[0][:label_method] ? @dimensions[0][:label_method].call(r[:value]) : r[:value], r[@search.select_value]] }
     when 2
       @result[:rows].map do |r|
         {
           name: r[:value],
-          data: r[:rows].map { |r2| [r2[:value], r2[:count]] }
+          data: r[:rows].map { |r2| [r2[:value], r2[@search.select_value]] }
         }
       end
     when 3
@@ -56,6 +56,22 @@
   <%= form.radio_button 'display_type', SimpleDrilldown::Search::DisplayType::NONE, { :onChange => 'form.submit()' } %>
   <%= form.label :display_type_none, t(:none) %>
 
+  <% if controller.c_summary_fields.present? && @search.display_type != SimpleDrilldown::Search::DisplayType::NONE %>
+    <div class="text-center">
+      <label style="margin-bottom:0;margin-right:1em" for="search_select_value">Value: </label>
+      <div class="form-check form-check-inline">
+        <%= form.radio_button :select_value, SimpleDrilldown::Search::SelectValue::COUNT, class: "form-check-input", onChange: 'form.submit()' %>
+        <label class="form-check-label mr-3" for="search_select_value_count"><%= t(:count) %></label>
+      </div>
+      <% controller.c_summary_fields.each do |field| %>
+        <div class="form-check form-check-inline">
+          <%= form.radio_button :select_value, field, class: "form-check-input", onChange: 'form.submit()' %>
+          <label class="form-check-label mr-3" for="search_select_value_<%= field %>"><%= t(:field, default: field.to_s.titleize) %></label>
+        </div>
+      <% end %>
+    </div>
+<% end %>
+
   <%= form.check_box :order_by_value, { :onChange => 'form.submit()' } %>
   <%= form.label :order_by_value, t(:order_by_value) %>
 
@@ -68,10 +84,10 @@
   <%= javascript_tag do %>
     $('#chart-1').on('click', function(e){
     chart = Chartkick.charts['chart-1'].getChartObject();
-    firstPoint = chart.getElementsAtEvent(e)[0]
+    firstPoint = chart.getElementsAtEventForMode(e, 'nearest', { intersect: true }, false)[0]
     if (firstPoint) {
-    label = chart.data.labels[firstPoint._index];
-    value = chart.data.datasets[firstPoint._datasetIndex].data[firstPoint._index];
+    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(/#.*$/, '');

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

@@ -1,3 +1,4 @@
+<% row_class = dimension > 0 && @dimensions[dimension - 1]&.[](:row_class)&.call(result[:value]) %>
 <% if new_row -%>
   <tr class="<%= cycle("odd", "even", :name => "dim#{dimension}") %>">
     <% ((dimension + 1)..(@dimensions.size)).each { |i| cycle("odd", "even", :name => "dim#{i}") if current_cycle("dim#{i}") != current_cycle("dim#{dimension}") } -%>
@@ -14,5 +15,13 @@
       (<%= 100 * result[:count] / parent_result[:count] %>%)
     <% end %>
   </td>
+  <% controller.c_summary_fields.each do |summary_field| %>
+    <td class="<%=row_class%>" align="right">
+      <%= result[summary_field] %>
+      <% if @search.percent && parent_result && parent_result[summary_field] > 0 %>
+        (<%= 100 * result[summary_field] / parent_result[summary_field] %>%)
+      <% end %>
+    </td>
+  <% end %>
   </tr>
 <% end %>

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

@@ -1,14 +1,16 @@
-<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>
+<div class="table-responsive">
+  <table id="drilldown-summary-table" class="table table-sm 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>
+</div>

data/lib/generators/drilldown_controller/templates/drilldown_controller.rb.erb

@@ -12,6 +12,15 @@ class <%= class_name %>DrilldownController < SimpleDrilldown::Controller
   # How should we count the reords?
   # select 'count(*) as count'
 
+  # Multiple select values:
+  # select <<~SQL
+  #   COUNT(*) as count,
+  #   sum(hours) as hours,
+  #   sum(billable_hours) as billable_hours
+  # SQL
+  # summary_fields :hours, :billable_hours
+  # default_select_value :billable_hours
+
   # When selecting records, what relations should be included for optimization?
   # Other relations can be included for specific dimensions and fields.
   # base_includes :user, :comments # TODO(uwe): Read relations from schema?

data/lib/simple_drilldown/controller.rb

@@ -58,7 +58,7 @@ module SimpleDrilldown
       end
 
       def default_select_value(default_select_value)
-        self.c_default_select_value = default_select_value
+        self.c_default_select_value = default_select_value.to_sym
       end
 
       def target_class(target_class)
@@ -85,7 +85,15 @@ module SimpleDrilldown
         self.c_summary_fields = summary_fields.flatten
       end
 
-      def dimension(name, select_expression = name, options = {})
+      def dimension(name, select_expression = name.to_s, options = {})
+        if select_expression.is_a?(Hash)
+          unless options.blank?
+            raise "select_expression should be a string: #{select_expression.class}, options: #{options.inspect}"
+          end
+
+          options = select_expression
+          select_expression = name.to_s
+        end
         interval = options.delete(:interval)
         label_method = options.delete(:label_method)
         legal_values = options.delete(:legal_values) || legal_values_for(name)
@@ -327,16 +335,7 @@ module SimpleDrilldown
       includes.flatten!
       includes.keep_if(&:present?).uniq!
       if @search.order_by_value && @dimensions.size <= 1
-        order = case @search.select_value
-                when Search::SelectValue::VOLUME
-                  'volume DESC'
-                when Search::SelectValue::VOLUME_COMPENSATED
-                  'volume_compensated DESC'
-                when Search::SelectValue::COUNT
-                  'count DESC'
-                else
-                  'count DESC'
-                end
+        order = "#{c_summary_fields.find { |f| f == @search.select_value } || 'count'} DESC"
       else
         order = (1..@dimensions.size).map { |i| "value#{i}" }.join(',')
         order = nil if order.empty?

data/lib/simple_drilldown/helper.rb

@@ -89,7 +89,8 @@ module SimpleDrilldown
     private
 
     def caption_txt
-      "#{controller.c_target_class} #{I18n.t(@search.select_value.downcase)}" +
+      "#{controller.c_target_class} #{I18n.t(@search.select_value.downcase,
+                                             default: @search.select_value.to_s.titleize)}" +
         (@dimensions && @dimensions.any? ? " by #{@dimensions.map { |d| d[:pretty_name] }.join(' and ')}" : '')
     end
   end

data/lib/simple_drilldown/search.rb

@@ -14,9 +14,7 @@ module SimpleDrilldown
     end
 
     module SelectValue
-      COUNT = 'COUNT'
-      VOLUME = 'VOLUME'
-      VOLUME_COMPENSATED = 'VOLUME_COMPENSATED'
+      COUNT = :count
     end
 
     attr_reader :dimensions, :display_type, :fields, :filter, :list_change_times, :order_by_value,
@@ -62,7 +60,7 @@ module SimpleDrilldown
         @display_type = DisplayType::BAR if @dimensions.size >= 2 && @display_type == DisplayType::PIE
 
         @order_by_value = attributes && (attributes[:order_by_value] == '1')
-        @select_value = attributes&.dig(:select_value).present? ? attributes[:select_value] : @default_select_value
+        @select_value = attributes&.dig(:select_value).presence&.to_sym || @default_select_value
         @list = attributes&.[](:list) == '1'
         @percent = attributes&.[](:percent) == '1'
         @list_change_times = attributes&.[](:list_change_times) == '1'

data/lib/simple_drilldown/version.rb

@@ -1,3 +1,3 @@
 module SimpleDrilldown
-  VERSION = '0.9.3'
+  VERSION = '0.9.8'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: simple_drilldown
 version: !ruby/object:Gem::Version
-  version: 0.9.3
+  version: 0.9.8
 platform: ruby
 authors:
 - Uwe Kubosch
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-20 00:00:00.000000000 Z
+date: 2021-05-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: caxlsx_rails