fix: maker checker condition handling (openMF#1216)

Added a test to check if the client created is actually active or not. The app then shows the message accordingly.
saurabh28march · Dec 27, 2019 · 326cfd0 · 326cfd0
1 parent 4490f18
commit 326cfd0
Show file tree

Hide file tree

Showing 7 changed files with 32 additions and 3 deletions.
diff --git a/...d/src/main/java/com/mifos/mifosxdroid/online/createnewclient/CreateNewClientFragment.java b/...d/src/main/java/com/mifos/mifosxdroid/online/createnewclient/CreateNewClientFragment.java
@@ -355,6 +355,12 @@ public void showClientCreatedSuccessfully(int message) {
         getActivity().getSupportFragmentManager().popBackStack();
     }
 
+    @Override
+    public void showWaitingForCheckerApproval(int message) {
+        Toaster.show(rootView, message);
+        getActivity().getSupportFragmentManager().popBackStack();
+    }
+
     @Override
     public void showMessage(int message) {
         Toaster.show(rootView, message);

diff --git a/...id/src/main/java/com/mifos/mifosxdroid/online/createnewclient/CreateNewClientMvpView.java b/...id/src/main/java/com/mifos/mifosxdroid/online/createnewclient/CreateNewClientMvpView.java
@@ -22,6 +22,8 @@ public interface CreateNewClientMvpView extends MvpView {
 
     void showClientCreatedSuccessfully(int s);
 
+    void showWaitingForCheckerApproval(int s);
+
     void showMessage(int message);
 
     void showMessage(String message);

diff --git a/.../src/main/java/com/mifos/mifosxdroid/online/createnewclient/CreateNewClientPresenter.java b/.../src/main/java/com/mifos/mifosxdroid/online/createnewclient/CreateNewClientPresenter.java
@@ -164,8 +164,14 @@ public void onError(Throwable e) {
                     @Override
                     public void onNext(Client client) {
                         getMvpView().showProgressbar(false);
-                        getMvpView().showClientCreatedSuccessfully(
-                                R.string.client_created_successfully);
+                        if (client.getClientId() != null) {
+                            getMvpView().showClientCreatedSuccessfully(
+                                    R.string.client_created_successfully);
+                        } else {
+                            getMvpView().showWaitingForCheckerApproval(
+                                    R.string.waiting_for_checker_approval
+                            );
+                        }
                     }
                 }));
     }

diff --git a/...c/main/java/com/mifos/mifosxdroid/online/datatablelistfragment/DataTableListFragment.java b/...c/main/java/com/mifos/mifosxdroid/online/datatablelistfragment/DataTableListFragment.java
@@ -322,6 +322,12 @@ public void showClientCreatedSuccessfully(Client client) {
         }
     }
 
+    @Override
+    public void showWaitingForCheckerApproval(int message) {
+        getActivity().getSupportFragmentManager().popBackStack();
+        Toaster.show(rootView, message, Toast.LENGTH_SHORT);
+    }
+
     @Override
     public void showProgressbar(boolean b) {
         if (b) {

diff --git a/...rc/main/java/com/mifos/mifosxdroid/online/datatablelistfragment/DataTableListMvpView.java b/...rc/main/java/com/mifos/mifosxdroid/online/datatablelistfragment/DataTableListMvpView.java
@@ -10,4 +10,6 @@ public interface DataTableListMvpView extends MvpView {
     void showMessage(String message);
 
     void showClientCreatedSuccessfully(Client client);
+
+    void showWaitingForCheckerApproval(int message);
 }
diff --git a/.../main/java/com/mifos/mifosxdroid/online/datatablelistfragment/DataTableListPresenter.java b/.../main/java/com/mifos/mifosxdroid/online/datatablelistfragment/DataTableListPresenter.java
@@ -121,7 +121,13 @@ public void onError(Throwable e) {
                     @Override
                     public void onNext(Client client) {
                         getMvpView().showProgressbar(false);
-                        getMvpView().showClientCreatedSuccessfully(client);
+                        if (client.getClientId() != null) {
+                            getMvpView().showClientCreatedSuccessfully(
+                                    client);
+                        } else {
+                            getMvpView().showWaitingForCheckerApproval(
+                                    R.string.waiting_for_checker_approval);
+                        }
                     }
                 }));
     }

diff --git a/mifosng-android/src/main/res/values/strings.xml b/mifosng-android/src/main/res/values/strings.xml
@@ -164,6 +164,7 @@
     <string name="no_search_result_found">No results found for entered query</string>
     <string name="no_search_query_entered">No Search Query Entered!</string>
     <string name="client_created_successfully">Client Created Successfully</string>
+    <string name="waiting_for_checker_approval">Waiting for Checker\'s Approval</string>
     <string name="no_staff_associated_with_office">No Staff associated with this Office</string>
     <string name="adding_client_address">Adding Client Address</string>
     <string name="updating_client_address">Updating Client Address</string>