Fix stop() & remove overkill resetSurvey()

Wootric · Dec 16, 2020 · a9394d3 · a9394d3
1 parent 28f3f18
commit a9394d3
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/androidsdk/src/main/java/com/wootric/androidsdk/SurveyManager.java b/androidsdk/src/main/java/com/wootric/androidsdk/SurveyManager.java
@@ -93,15 +93,12 @@ public static SurveyManager getSharedInstance() {
     synchronized void stop() {
         handler.removeCallbacks(surveyRunner);
 
-        resetSurvey();
-
         if (surveySupportFragment != null) {
             surveySupportFragment.dismiss();
-            surveySupportFragment = null;
         } else if (surveyFragment != null) {
             surveyFragment.dismiss();
-            surveyFragment = null;
         }
+        resetSurvey();
         Log.d(Constants.TAG, "Survey stopped");
     }
 
@@ -371,7 +368,6 @@ private String getOriginUrl(Activity activity) {
                 originUrl = pm.getApplicationLabel(appInfo).toString();
             } catch (Exception e) {
                 Log.e(Constants.TAG, "getOriginUrl: " + e.toString());
-                resetSurvey();
                 e.printStackTrace();
             }
         }