Merge pull request #711 from rubyforgood/710-items-value

display an estimated value of contents on the Distribution manifests

Author: seanmarcia

app/controllers/distributions_controller.rb

@@ -33,6 +33,7 @@ def index
                      .distributions
                      .includes(:partner, :storage_location, :line_items, :items)
                      .order(created_at: :desc)
+    @total_value_all_distributions = total_value(@distributions)
   end
 
   def create
@@ -135,4 +136,12 @@ def send_notification(org, dist)
   def distribution_params
     params.require(:distribution).permit(:comment, :agency_rep, :issued_at, :partner_id, :storage_location_id, line_items_attributes: %i(item_id quantity _destroy))
   end
+
+  def total_value(distributions)
+    total_value_all_distributions = 0
+    distributions.each do |distribution|
+      total_value_all_distributions += distribution.value_per_itemizable
+    end
+    total_value_all_distributions
+  end
 end

app/controllers/items_controller.rb

@@ -53,7 +53,7 @@ def destroy
   private
 
   def item_params
-    params.require(:item).permit(:name, :category, :partner_key)
+    params.require(:item).permit(:name, :category, :partner_key, :value)
   end
 
   def filter_params(parameters = nil)

app/helpers/items_helper.rb

@@ -0,0 +1,9 @@
+module ItemsHelper
+  def item_value(value, addition = '')
+    if value.zero?
+      ''
+    else
+      addition + ActionController::Base.helpers.number_to_currency(value, precision: value.round == value ? 0 : 2)
+    end
+  end
+end

app/models/concerns/itemizable.rb

@@ -72,6 +72,10 @@ def line_items_quantities
     end
   end
 
+  def value_per_itemizable
+    line_items.sum(&:value_per_line_item)
+  end
+
   private
 
   def line_item_items_quantity_is_positive

app/models/item.rb

@@ -2,15 +2,16 @@
 #
 # Table name: items
 #
-#  id              :bigint(8)        not null, primary key
+#  id              :integer          not null, primary key
 #  name            :string
 #  category        :string
-#  created_at      :datetime
-#  updated_at      :datetime
+#  created_at      :datetime         not null
+#  updated_at      :datetime         not null
 #  barcode_count   :integer
 #  organization_id :integer
 #  active          :boolean          default(TRUE)
 #  partner_key     :string
+#  value           :decimal(5, 2)    default(0.0)
 #
 
 class Item < ApplicationRecord
@@ -19,6 +20,7 @@ class Item < ApplicationRecord
   validates :name, uniqueness: { scope: :organization }
   validates :name, presence: true
   validates :organization, presence: true
+  validates :value, numericality: { greater_than_or_equal_to: 0 }
 
   has_many :line_items, dependent: :destroy
   has_many :inventory_items, dependent: :destroy

app/models/line_item.rb

@@ -17,4 +17,8 @@ class LineItem < ApplicationRecord
 
   validates :item_id, presence: true
   validates :quantity, numericality: { other_than: 0, only_integer: true }
+
+  def value_per_line_item
+    item.value * quantity
+  end
 end

app/pdfs/distribution_pdf.rb

@@ -1,5 +1,6 @@
 class DistributionPdf
   include Prawn::View
+  include ItemsHelper
 
   def initialize(organization, distribution)
     @distribution = distribution
@@ -9,11 +10,11 @@ def initialize(organization, distribution)
       text organization.address, align: :right
       text organization.email, align: :right
     end
-    data = [["Items Received", "Quantity"]]
+    data = [["Items Received", "Value/item", "Total value", "Quantity"]]
     data += @distribution.line_items.sorted.map do |c|
-      [c.item.name, c.quantity]
+      [c.item.name, item_value(c.item.value), item_value(c.value_per_line_item), c.quantity]
     end
-    data += [["", ""], ["Total Items Received", @distribution.line_items.total]]
+    data += [["", "", "", ""], ["Total Items Received", "", item_value(@distribution.value_per_itemizable), @distribution.line_items.total]]
 
     move_down 55
 
@@ -45,22 +46,23 @@ def initialize(organization, distribution)
       row(0).borders = [:bottom]
       row(0).border_width = 2
       row(0).font_style = :bold
-      row(0).column(-1).borders = %i(bottom left)
+      row(0).column(1..-1).borders = %i(bottom left)
 
       # Total Items footer row
       row(-1).borders = [:top]
       row(-1).font_style = :bold
-      row(-1).column(-1).borders = %i(top left)
+      row(-1).column(2..-1).borders = %i(top left)
+      row(-1).column(2..-1).border_left_color = "aaaaaa"
 
       # Footer spacing row
       row(-2).borders = [:top]
       row(-2).padding = [2, 0, 2, 0]
 
-      column(0).width = 400
+      column(0).width = 250
 
       # Quantity column
-      column(1).row(1..-3).borders = [:left]
-      column(1).row(1..-3).border_left_color = "aaaaaa"
+      column(1..3).row(1..-3).borders = [:left]
+      column(1..3).row(1..-3).border_left_color = "aaaaaa"
       column(1).style align: :right
     end
 

app/queries/items_by_storage_collection_and_quantity_query.rb

@@ -14,6 +14,7 @@ def call
         @items_by_storage_collection_and_quantity[row.id] = {
           item_name: row.name,
           item_category: row.category,
+          item_value: row.value,
           item_barcode_count: row.barcode_count
         }
       end

app/queries/items_by_storage_collection_query.rb

@@ -19,6 +19,7 @@ def call
                         items.category,
                         items.barcode_count,
                         items.partner_key,
+                        items.value,
                         storage_locations.name as storage_name,
                         storage_locations.id as storage_id,
                         sum(inventory_items.quantity) as quantity

app/views/distributions/_distribution_item_row.html.erb

@@ -1,4 +1,6 @@
 <tr>
   <td><%= distribution_item_row.quantity %></td>
+  <td><%= item_value(distribution_item_row.item.value) %></td>
+  <td><%= item_value(distribution_item_row.value_per_line_item) %></td>
   <td><%= distribution_item_row.item.name %></td>
 </tr>