fix: Pressing home/up button or back key on a New Config entry should…

… display the Discard dialog
kaushiknsanji · Nov 11, 2019 · e0eef93 · e0eef93
1 parent 1f9526e
commit e0eef93
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/...in/java/com/example/kaushiknsanji/storeapp/ui/products/config/ProductConfigPresenter.java b/...in/java/com/example/kaushiknsanji/storeapp/ui/products/config/ProductConfigPresenter.java
@@ -848,6 +848,11 @@ public void onUpOrBackAction() {
             //to see if we have some unsaved changes
             if (mProductId > ProductConfigContract.NEW_PRODUCT_INT) {
                 mProductConfigView.readExistingProductDetails();
+            } else {
+                //If this is for a New Product entry and the details are not saved,
+                //then display the discard dialog to see if the user wants to keep editing/discard
+                //the changes
+                mProductConfigView.showDiscardDialog();
             }
 
         } else {

diff --git a/.../java/com/example/kaushiknsanji/storeapp/ui/suppliers/config/SupplierConfigPresenter.java b/.../java/com/example/kaushiknsanji/storeapp/ui/suppliers/config/SupplierConfigPresenter.java
@@ -964,6 +964,11 @@ public void onUpOrBackAction() {
             //to see if we have some unsaved changes
             if (mSupplierId > SupplierConfigContract.NEW_SUPPLIER_INT) {
                 mSupplierConfigView.readExistingSupplierDetails();
+            } else {
+                //If this for a New Supplier entry and the details are not saved,
+                //then display the discard dialog to see if the user wants to keep editing/discard
+                //the changes
+                mSupplierConfigView.showDiscardDialog();
             }
 
         } else {