From 5dd947c640d975082c5aa05328fa75bc6ac852fb Mon Sep 17 00:00:00 2001
From: Damyan Ivanov <dam+mobileledger@ktnx.net>
Date: Sun, 15 Mar 2020 07:51:04 +0200
Subject: [PATCH] NT: show currency selection trigger only in a transaction row

not in the header/padding (these would throw on setCurrency(null) call)
---
 .../ui/activity/NewTransactionItemHolder.java    | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/activity/NewTransactionItemHolder.java b/app/src/main/java/net/ktnx/mobileledger/ui/activity/NewTransactionItemHolder.java
index d7017e21..37506c04 100644
--- a/app/src/main/java/net/ktnx/mobileledger/ui/activity/NewTransactionItemHolder.java
+++ b/app/src/main/java/net/ktnx/mobileledger/ui/activity/NewTransactionItemHolder.java
@@ -356,13 +356,15 @@ class NewTransactionItemHolder extends RecyclerView.ViewHolder
         currencyObserver = this::setCurrency;
 
         showCurrencyObserver = showCurrency -> {
-              if (showCurrency) {
-                  tvCurrency.setVisibility(View.VISIBLE);
-              }
-            else {
-                tvCurrency.setVisibility(View.GONE);
-                setCurrencyString(null);
-              }
+            if (item.getType() == ItemType.transactionRow) {
+                if (showCurrency) {
+                    tvCurrency.setVisibility(View.VISIBLE);
+                }
+                else {
+                    tvCurrency.setVisibility(View.GONE);
+                    item.setCurrency(null);
+                }
+            }
         };
     }
     private void updateCurrencyPositionAndPadding(Currency.Position position, boolean hasGap) {
-- 
2.39.2