From 87cbb792b9d141168ff51fb7de0e71a0672a19b4 Mon Sep 17 00:00:00 2001
From: Hillel Coren <hillelcoren@gmail.com>
Date: Wed, 2 Nov 2022 10:29:15 +0200
Subject: [PATCH] Improve payment term edit screen

---
 .../payment_term/edit/payment_term_edit.dart  | 21 +++++++++++--------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/lib/ui/payment_term/edit/payment_term_edit.dart b/lib/ui/payment_term/edit/payment_term_edit.dart
index 245e865b7..060dc99ad 100644
--- a/lib/ui/payment_term/edit/payment_term_edit.dart
+++ b/lib/ui/payment_term/edit/payment_term_edit.dart
@@ -70,6 +70,16 @@ class _PaymentTermEditState extends State<PaymentTermEdit> {
     }
   }
 
+  void _onSavePressed() {
+    final bool isValid = _formKey.currentState.validate();
+
+    if (!isValid) {
+      return;
+    }
+
+    widget.viewModel.onSavePressed(context);
+  }
+
   @override
   Widget build(BuildContext context) {
     final viewModel = widget.viewModel;
@@ -81,15 +91,7 @@ class _PaymentTermEditState extends State<PaymentTermEdit> {
           ? localization.newPaymentTerm
           : localization.editPaymentTerm,
       onCancelPressed: (context) => viewModel.onCancelPressed(context),
-      onSavePressed: (context) {
-        final bool isValid = _formKey.currentState.validate();
-
-        if (!isValid) {
-          return;
-        }
-
-        viewModel.onSavePressed(context);
-      },
+      onSavePressed: (context) => _onSavePressed(),
       body: Form(
           key: _formKey,
           child: Builder(builder: (BuildContext context) {
@@ -105,6 +107,7 @@ class _PaymentTermEditState extends State<PaymentTermEdit> {
                       validator: (value) => value == null || value.isEmpty
                           ? localization.pleaseEnterAValue
                           : null,
+                      onSavePressed: (context) => _onSavePressed(),
                     ),
                   ],
                 ),