simple_drilldown 0.9.4 → 0.9.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a9258ecdeb91deb6d4a17a1a23e28f1e0e9cbf243588d9f40e057f93f16c0892
-  data.tar.gz: 5a6e71f4400c9259fe90a04420372adfd29982bede1aec6968d06050c44de9cf
+  metadata.gz: 8e0ff048ca6aef6cfe9f4e3ccc63eb22f7a303c53612bff80afffff62bc4fbc1
+  data.tar.gz: fa462127353656e4f3ace83e97ac3c91065a3f0aa1b8a1a32c9429ab755c27ea
 SHA512:
-  metadata.gz: c84bcd765bdb3a7fd29a67a2401849ca065505ec888b0691f6a3b3b79cd94ad5db7a8a28a12b575f6ab6675e328209e7aaa0c13b43c52f65fe227312efaf2e41
-  data.tar.gz: b87a85a90e3843d4deed1aa9725e914622073e77cfdf02db5e771912bd457812a846298f171d4673ee9d6cdcf9154bd661c6a2cd651231503de227fef1422f52
+  metadata.gz: 8c4fa801f0dadea11dc6821a750be32af77574f89d28500f867ac5224a45cf0533392630781ebf502e8a9d7726b0f463d3fe178ebab6193275e6024b27042a6f
+  data.tar.gz: a820957b4d377ede56478ed6d46bc4622cf43738f5f14601fedff63af6eb4db54fe228e68b35036341f5332c4acd53e591175f436e7267cda980d5e832c225df

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) %>
 

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/app/views/drilldown/_summary_total_row.html.erb

@@ -1,4 +1,12 @@
 <tr class="summary">
   <td colspan="<%= @dimensions.size - dimension %>"></td>
   <td align="right"><%= result[:count] %></td>
+  <% controller.c_summary_fields.each do |summary_field| %>
+    <td 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>

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.4'
+  VERSION = '0.9.9'
 end

metadata

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