fixed Npe for obs. levels in v1 get studies call

PhenoApps · Jun 29, 2023 · 3ae0c25 · 3ae0c25
1 parent 3b08443
commit 3ae0c25
Showing 1 changed file with 18 additions and 8 deletions.
diff --git a/app/src/main/java/com/fieldbook/tracker/brapi/service/BrAPIServiceV1.java b/app/src/main/java/com/fieldbook/tracker/brapi/service/BrAPIServiceV1.java
@@ -441,6 +441,23 @@ public void getPlotDetails(final String studyDbId,
                                final Function<BrapiStudyDetails, Void> function,
                                final Function<Integer, Void> failFunction) {
         try {
+
+            //level is not a required field, so passing null here is fine
+            //otherwise this was causing a NPE when observationLevel was null
+            //check if observationLevel is null
+            @Nullable
+            String levelName = null;
+            if (observationLevel != null) {
+
+                if (observationLevel.getObservationLevelName() != null) {
+
+                    levelName = observationLevel.getObservationLevelName();
+
+                }
+            }
+
+            final String level = levelName;
+
             final AtomicInteger currentPage = new AtomicInteger(0);
             final Integer pageSize = Integer.parseInt(context.getSharedPreferences(GeneralKeys.SHARED_PREF_FILE_NAME, 0)
                     .getString(GeneralKeys.BRAPI_PAGE_SIZE, "50"));
@@ -481,13 +498,6 @@ public void onSuccess(ObservationUnitsResponse1 response, int i, Map<String, Lis
                         }else {
                             try {
 
-                                //level is not a required field, so passing null here is fine
-                                //otherwise this was causing a NPE when observationLevel was null
-                                @Nullable String level = null;
-                                if (observationLevel != null) {
-                                    level = observationLevel.getObservationLevelName();
-                                }
-
                                 studiesApi.studiesStudyDbIdObservationunitsGetAsync(
                                         studyDbId, level, currentPage.get(), pageSize,
                                         getBrapiToken(), this);
@@ -508,7 +518,7 @@ public void onFailure(ApiException error, int i, Map<String, List<String>> map)
             };
 
             studiesApi.studiesStudyDbIdObservationunitsGetAsync(
-                    studyDbId, observationLevel.getObservationLevelName(), 0, pageSize,
+                    studyDbId, level, 0, pageSize,
                     getBrapiToken(), callback);
 
         } catch (ApiException e) {