From 4f7e7845b863a283ea474cb514a7b9538f9b3f13 Mon Sep 17 00:00:00 2001
From: Art Manalili <artmanalili@gmail.com>
Date: Mon, 8 Apr 2019 20:24:39 +0800
Subject: [PATCH] Make send confirmation fee static

---
 .../sendConfirmation.test.js.snap             |  8 +++----
 .../scenes/SendConfirmationScene.js           | 23 +++++++++++++------
 2 files changed, 20 insertions(+), 11 deletions(-)

diff --git a/src/__tests__/__snapshots__/sendConfirmation.test.js.snap b/src/__tests__/__snapshots__/sendConfirmation.test.js.snap
index 9be3d0d1937..02227e28006 100644
--- a/src/__tests__/__snapshots__/sendConfirmation.test.js.snap
+++ b/src/__tests__/__snapshots__/sendConfirmation.test.js.snap
@@ -196,7 +196,7 @@ exports[`SendConfirmation should render with destination 1`] = `
                   ]
                 }
               >
-                
+                Fee: + 0 (0)
               </FormattedText>
             </Row>
             <Row
@@ -460,7 +460,7 @@ exports[`SendConfirmation should render with standard props 1`] = `
                   ]
                 }
               >
-                
+                Fee: + 0 (0)
               </FormattedText>
             </Row>
             <AddressTextWithBlockExplorerModal
@@ -709,7 +709,7 @@ exports[`SendConfirmation should render with unique identifier button with "ADD"
                   ]
                 }
               >
-                
+                Fee: + 0 (0)
               </FormattedText>
             </Row>
             <Row
@@ -974,7 +974,7 @@ exports[`SendConfirmation should render with unique identifier button with "ADD"
                   ]
                 }
               >
-                
+                Fee: + 0 (0)
               </FormattedText>
             </Row>
             <Row
diff --git a/src/components/scenes/SendConfirmationScene.js b/src/components/scenes/SendConfirmationScene.js
index cd3bd7fb06b..63ea10856d4 100644
--- a/src/components/scenes/SendConfirmationScene.js
+++ b/src/components/scenes/SendConfirmationScene.js
@@ -246,11 +246,9 @@ export class SendConfirmation extends Component<Props, State> {
 
               <Scene.Padding style={{ paddingHorizontal: 54 }}>
                 <Scene.Item style={{ alignItems: 'center', flex: -1 }}>
-                  {feeCalculated && (
-                    <Scene.Row style={{ paddingVertical: 4 }}>
-                      <Text style={[styles.feeAreaText]}>{this.networkFeeSyntax()}</Text>
-                    </Scene.Row>
-                  )}
+                  <Scene.Row style={{ paddingVertical: 4 }}>
+                    <Text style={[styles.feeAreaText]}>{this.networkFeeSyntax()}</Text>
+                  </Scene.Row>
 
                   {!!destination && (
                     <Scene.Row style={{ paddingVertical: 10 }}>
@@ -365,7 +363,6 @@ export class SendConfirmation extends Component<Props, State> {
 
   networkFeeSyntax = () => {
     const { networkFee, parentNetworkFee, parentDisplayDenomination, exchangeRates } = this.props
-    if (!networkFee && !parentNetworkFee) return ''
 
     const primaryInfo: GuiCurrencyInfo = {
       displayCurrencyCode: this.props.currencyCode,
@@ -389,6 +386,18 @@ export class SendConfirmation extends Component<Props, State> {
       exchangeDenomination: this.state.secondaryDisplayDenomination
     }
 
+    if (!networkFee && !parentNetworkFee) {
+      const cryptoFeeSymbolParent = parentDisplayDenomination.symbol ? parentDisplayDenomination.symbol : null
+      const cryptoFeeSymbolPrimary = primaryInfo.displayDenomination.symbol ? primaryInfo.displayDenomination.symbol : null
+      const cryptoFeeSymbol = () => {
+        if (cryptoFeeSymbolParent) return cryptoFeeSymbolParent
+        if (cryptoFeeSymbolPrimary) return cryptoFeeSymbolPrimary
+        return ''
+      }
+      const fiatFeeSymbol = secondaryInfo.displayDenomination.symbol ? secondaryInfo.displayDenomination.symbol : ''
+      return sprintf(s.strings.send_confirmation_fee_line, `${cryptoFeeSymbol()} 0`, `${fiatFeeSymbol} 0`)
+    }
+
     if (parentNetworkFee && bns.gt(parentNetworkFee, '0')) {
       const cryptoFeeSymbol = parentDisplayDenomination.symbol ? parentDisplayDenomination.symbol : ''
       // multiplier for display denomination
@@ -441,7 +450,7 @@ export class SendConfirmation extends Component<Props, State> {
       const fiatFeeString = `${fiatFeeSymbol} ${fiatFeeAmountPretty}`
       return sprintf(s.strings.send_confirmation_fee_line, cryptoFeeString, fiatFeeString)
     }
-    return ''
+    return sprintf(s.strings.send_confirmation_fee_line, '0', '0')
   }
 }