Merge pull request #592 from dod-ccpo/clin-decimals-fixes

CLIN Input field bugfixes

atst/filters.py

@@ -12,14 +12,6 @@ def iconSvg(name):
 
 
 def dollars(value):
-    try:
-        numberValue = float(value)
-    except ValueError:
-        numberValue = 0
-    return "${:,.0f}".format(numberValue)
-
-
-def dollarsWithCents(value):
     try:
         numberValue = float(value)
     except ValueError:
@@ -107,7 +99,6 @@ def normalizeOrder(title):
 def register_filters(app):
     app.jinja_env.filters["iconSvg"] = iconSvg
     app.jinja_env.filters["dollars"] = dollars
-    app.jinja_env.filters["dollarsWithCents"] = dollarsWithCents
     app.jinja_env.filters["usPhone"] = usPhone
     app.jinja_env.filters["readableInteger"] = readableInteger
     app.jinja_env.filters["getOptionLabel"] = getOptionLabel

js/components/text_input.js

@@ -1,5 +1,6 @@
 import MaskedInput, { conformToMask } from 'vue-text-mask'
 import inputValidations from '../lib/input_validations'
+import { formatDollars } from '../lib/dollars'
 
 export default {
   name: 'textinput',
@@ -78,6 +79,9 @@ export default {
     onChange: function(e) {
       // Only invalidate the field when it blurs
       this._checkIfValid({ value: e.target.value, invalidate: true })
+      if (this.validation === 'dollars') {
+        this.value = formatDollars(this._rawValue(e.target.value))
+      }
     },
 
     //

js/lib/dollars.js

@@ -1,8 +1,14 @@
 export const formatDollars = (value, cents = true) => {
   if (typeof value === 'number') {
-    return cents
-      ? `$${value.toFixed(2).replace(/\d(?=(\d{3})+\.)/g, '$&,')}`
-      : `$${value.toFixed(0).replace(/\d(?=(\d{3})+(?!\d))/g, '$&,')}`
+    return value.toLocaleString('us-US', {
+      style: 'currency',
+      currency: 'USD',
+    })
+  } else if (typeof value === 'string') {
+    return parseFloat(value).toLocaleString('us-US', {
+      style: 'currency',
+      currency: 'USD',
+    })
   }
   return ''
 }

templates/task_orders/new/review.html

@@ -154,7 +154,7 @@
           <td><h4>{{ "task_orders.new.review.to_value"| translate }}</h4></td>
           <td class="table-cell--align-right">
             {% if task_order.budget %}
-              {{ task_order.budget | dollarsWithCents }}
+              {{ task_order.budget | dollars }}
             {% endif %}
           </td>
         </tr>
@@ -162,7 +162,7 @@
           <td><h4 class='task-order-form__heading funding-summary__td'>{{ "task_orders.new.review.clin_1"| translate }}</h4></td>
           <td class="table-cell--align-right">
             {% if task_order.clin_01 %}
-              {{ task_order.clin_01 | dollarsWithCents }}
+              {{ task_order.clin_01 | dollars }}
             {% else %}
               {{ RequiredLabel() }}
             {% endif %}
@@ -177,7 +177,7 @@
           </h4></td>
           <td class="table-cell--align-right">
             {% if task_order.clin_02 and config.CLASSIFIED %}
-              {{ task_order.clin_02 | dollarsWithCents or RequiredLabel() }}
+              {{ task_order.clin_02 | dollars or RequiredLabel() }}
             {% endif %}
           </td>
         </tr>
@@ -185,7 +185,7 @@
           <td><h4 class='task-order-form__heading funding-summary__td'>{{ "task_orders.new.review.clin_3"| translate }}</h4></td>
           <td class="table-cell--align-right">
             {% if task_order.clin_03 %}
-              {{ task_order.clin_03 | dollarsWithCents or RequiredLabel() }}
+              {{ task_order.clin_03 | dollars or RequiredLabel() }}
             {% else %}
               {{ RequiredLabel() }}
             {% endif %}
@@ -200,7 +200,7 @@
           </h4></td>
           <td class="table-cell--align-right">
             {% if task_order.clin_04 and config.CLASSIFIED %}
-              {{ task_order.clin_04 | dollarsWithCents or RequiredLabel() }}
+              {{ task_order.clin_04 | dollars or RequiredLabel() }}
             {% endif %}
           </td>
         <tr>

tests/test_filters.py

@@ -7,11 +7,11 @@ from atst.models import AuditEvent
 @pytest.mark.parametrize(
     "input,expected",
     [
-        ("0", "$0"),
-        ("123.00", "$123"),
-        ("1234567", "$1,234,567"),
-        ("-1234", "$-1,234"),
-        ("one", "$0"),
+        ("0", "$0.00"),
+        ("123.00", "$123.00"),
+        ("1234567", "$1,234,567.00"),
+        ("-1234", "$-1,234.00"),
+        ("one", "$0.00"),
     ],
 )
 def test_dollar_fomatter(input, expected):