allow getting forms from db without appending locale

allow configurability of different strategies of form identity naming
opensrp · Jul 8, 2021 · 95a6544 · 95a6544
1 parent c9c6c7e
commit 95a6544
Show file tree

Hide file tree

Showing 6 changed files with 47 additions and 19 deletions.
diff --git a/...zard/src/main/java/com/vijay/jsonwizard/activities/FormConfigurationJsonFormActivity.java b/...zard/src/main/java/com/vijay/jsonwizard/activities/FormConfigurationJsonFormActivity.java
@@ -15,6 +15,7 @@
 import com.vijay.jsonwizard.interfaces.OnFormFetchedCallback;
 import com.vijay.jsonwizard.utils.AppExecutors;
 import com.vijay.jsonwizard.utils.FormUtils;
+import com.vijay.jsonwizard.utils.FormUtilsFactory;
 import com.vijay.jsonwizard.utils.NativeFormLangUtils;
 
 import org.json.JSONException;
@@ -38,11 +39,15 @@ public class FormConfigurationJsonFormActivity extends JsonFormActivity {
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
 
-        formUtils = new FormUtils();
+        formUtils = getFormUtils();
         JSONObject jsonObject = getmJSONObject();
         checkIfFormUpdate(jsonObject);
     }
 
+    protected FormUtils getFormUtils(){
+        return FormUtilsFactory.newInstance();
+    }
+
     private void checkIfFormUpdate(@NonNull JSONObject formJsonObject) {
         if (FormUtils.isFormNew(formJsonObject)) {
             showFormVersionUpdateDialog(formJsonObject, getString(R.string.form_update_title), getString(R.string.form_update_message));

diff --git a/...son-form-wizard/src/main/java/com/vijay/jsonwizard/activities/JsonWizardFormActivity.java b/...son-form-wizard/src/main/java/com/vijay/jsonwizard/activities/JsonWizardFormActivity.java
@@ -8,7 +8,7 @@
 /**
  * Created by keyman on 04/12/2018.
  */
-public class JsonWizardFormActivity extends JsonFormActivity {
+public class JsonWizardFormActivity extends FormConfigurationJsonFormActivity {
 
     @Override
     public void initializeFormFragment() {

diff --git a/...-wizard/src/main/java/com/vijay/jsonwizard/activities/NoLocaleJsonWizardFormActivity.java b/...-wizard/src/main/java/com/vijay/jsonwizard/activities/NoLocaleJsonWizardFormActivity.java
@@ -0,0 +1,11 @@
+package com.vijay.jsonwizard.activities;
+
+import com.vijay.jsonwizard.utils.FormUtils;
+import com.vijay.jsonwizard.utils.FormUtilsFactory;
+
+public class NoLocaleJsonWizardFormActivity extends JsonWizardFormActivity {
+    @Override
+    protected FormUtils getFormUtils() {
+        return FormUtilsFactory.noLocaleInstance();
+    }
+}
diff --git a/android-json-form-wizard/src/main/java/com/vijay/jsonwizard/utils/FormUtils.java b/android-json-form-wizard/src/main/java/com/vijay/jsonwizard/utils/FormUtils.java
@@ -1979,13 +1979,17 @@ public JSONObject getFormJson(@NonNull Context context, @NonNull String formIden
         }
     }
 
-    private ClientFormContract.Model getClientFormFromRepository(@NonNull Context context, @NonNull ClientFormContract.Dao clientFormRepository, String formIdentity) {
-        //Check the current locale of the app to load the correct version of the form in the desired language
+    protected String getLocaleFormIdentity(final Context context, final String formIdentity){
         String locale = context.getResources().getConfiguration().locale.getLanguage();
-        String localeFormIdentity = formIdentity;
         if (!Locale.ENGLISH.getLanguage().equals(locale)) {
-            localeFormIdentity = localeFormIdentity + "-" + locale;
+            return formIdentity + "-" + locale;
         }
+        return formIdentity;
+    }
+
+    private ClientFormContract.Model getClientFormFromRepository(@NonNull Context context, @NonNull ClientFormContract.Dao clientFormRepository, String formIdentity) {
+        //Check the current locale of the app to load the correct version of the form in the desired language
+        String localeFormIdentity = getLocaleFormIdentity(context, formIdentity);
 
         ClientFormContract.Model clientForm = clientFormRepository.getActiveClientFormByIdentifier(localeFormIdentity);
 
@@ -2061,13 +2065,8 @@ public void onCancelClicked() {
 
     @Nullable
     public JSONObject getSubFormJsonFromRepository(@NonNull Context context, @NonNull ClientFormContract.Dao clientFormDao, String formIdentity, String subFormsLocation, boolean translateSubForm) throws JSONException {
-        String locale = context.getResources().getConfiguration().locale.getLanguage();
-
         //Check the current locale of the app to load the correct version of the form in the desired language
-        String localeFormIdentity = formIdentity;
-        if (!Locale.ENGLISH.getLanguage().equals(locale)) {
-            localeFormIdentity = localeFormIdentity + "-" + locale;
-        }
+        String localeFormIdentity = getLocaleFormIdentity(context, formIdentity);
 
         String dbFormName = StringUtils.isBlank(subFormsLocation) ? localeFormIdentity : subFormsLocation + "/" + localeFormIdentity;
         ClientFormContract.Model clientForm = clientFormDao.getActiveClientFormByIdentifier(dbFormName);
@@ -2098,13 +2097,8 @@ public JSONObject getSubFormJsonFromRepository(@NonNull Context context, @NonNul
 
     @Nullable
     public BufferedReader getRulesFromRepository(@NonNull Context context, @NonNull ClientFormContract.Dao clientFormDao, @NonNull String fileName) {
-        String locale = context.getResources().getConfiguration().locale.getLanguage();
-
         //Check the current locale of the app to load the correct version of the form in the desired language
-        String localeFormIdentity = fileName;
-        if (!Locale.ENGLISH.getLanguage().equals(locale)) {
-            localeFormIdentity = localeFormIdentity + "-" + locale;
-        }
+        String localeFormIdentity = getLocaleFormIdentity(context, fileName);
 
         ClientFormContract.Model clientForm = clientFormDao.getActiveClientFormByIdentifier(localeFormIdentity);
         if (clientForm == null && StringUtils.isNotBlank(fileName) && fileName.contains("/") && !fileName.endsWith("/")) {

diff --git a/android-json-form-wizard/src/main/java/com/vijay/jsonwizard/utils/FormUtilsFactory.java b/android-json-form-wizard/src/main/java/com/vijay/jsonwizard/utils/FormUtilsFactory.java
@@ -0,0 +1,18 @@
+package com.vijay.jsonwizard.utils;
+
+import android.content.Context;
+
+public class FormUtilsFactory {
+    public static FormUtils newInstance(){
+        return new FormUtils();
+    }
+
+    public static FormUtils noLocaleInstance(){
+        return new FormUtils(){
+            @Override
+            protected String getLocaleFormIdentity(Context context, String formIdentity) {
+                return formIdentity;
+            }
+        };
+    }
+}
diff --git a/gradle.properties b/gradle.properties
@@ -1,4 +1,4 @@
-VERSION_NAME=2.1.0-SNAPSHOT
+VERSION_NAME=2.1.1-SNAPSHOT
 VERSION_CODE=1
 GROUP=org.smartregister
 POM_SETTING_DESCRIPTION=OpenSRP Client Native Form Json Wizard