From eb753bb30eaea823b564904b294b9709f6b2938f Mon Sep 17 00:00:00 2001
From: Herculino Trotta <netotrotta@gmail.com>
Date: Sat, 4 Jan 2025 01:32:09 -0300
Subject: [PATCH] feat(transactions): properly sum income and expense when
 selected

also added a flatTotal (old behavior) for future use
---
 app/templates/cotton/transaction/item.html     |  2 +-
 .../cotton/ui/transactions_action_bar.html     | 18 +++++++++++++-----
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/app/templates/cotton/transaction/item.html b/app/templates/cotton/transaction/item.html
index ed0adc9..8007599 100644
--- a/app/templates/cotton/transaction/item.html
+++ b/app/templates/cotton/transaction/item.html
@@ -1,5 +1,5 @@
 {% load i18n %}
-<div class="transaction d-flex my-1">
+<div class="transaction d-flex my-1 {% if transaction.type == "EX" %}expense{% else %}income{% endif %}">
   {% if not disable_selection %}
   <label class="px-3 d-flex align-items-center justify-content-center">
     <input class="form-check-input" type="checkbox" name="transactions" value="{{ transaction.id }}" id="check-{{ transaction.id }}" aria-label="{% translate 'Select' %}" hx-preserve>
diff --git a/app/templates/cotton/ui/transactions_action_bar.html b/app/templates/cotton/ui/transactions_action_bar.html
index bc73ccc..ec98c03 100644
--- a/app/templates/cotton/ui/transactions_action_bar.html
+++ b/app/templates/cotton/ui/transactions_action_bar.html
@@ -57,15 +57,23 @@
       </button>
       <div class="vr mx-3 tw-align-middle"></div>
       <span _="on selected_transactions_updated from #actions-bar
-                set total to 0.0
-                for amt in <.transaction:has(input[name='transactions']:checked) .main-amount .amount/>
+                set realTotal to 0.0
+                set flatTotal to 0.0
+                for transaction in <.transaction:has(input[name='transactions']:checked)/>
+                  set amt to first <.main-amount .amount/> in transaction
                   set amountValue to parseFloat(amt.getAttribute('data-amount'))
                   if not isNaN(amountValue)
-                    set total to total + (amountValue * 100)
+                    set flatTotal to flatTotal + (amountValue * 100)
+
+                    if transaction match .income
+                      set realTotal to realTotal + (amountValue * 100)
+                    else
+                      set realTotal to realTotal - (amountValue * 100)
+                    end
                   end
                 end
-                set total to total / 100
-                put total.toLocaleString(undefined, {minimumFractionDigits: 0, maximumFractionDigits: 40}) into me
+                set realTotal to realTotal / 100
+                put realTotal.toLocaleString(undefined, {minimumFractionDigits: 0, maximumFractionDigits: 40}) into me
               end
               on click
                 set original_value to my innerText