Креши приложения из-за вызов репортов до инициализации метрики

[vovka-s]

Привет!
Столкнулся с крешами приложухи, возникающими из-за вызовова reportEvent до инициализации метрики
Добавил проверок на активность - креши прекратились.
Прошу проверить, всё ли верно, и залить в реп

diff --git a/node_modules/yandex-appmetrica-plugin-cordova/platforms/android/com/yandex/metrica/plugin/cordova/AppMetricaPlugin.java b/node_modules/yandex-appmetrica-plugin-cordova/platforms/android/com/yandex/metrica/plugin/cordova/AppMetricaPlugin.java
index 1512def..ff1300a 100644
--- a/node_modules/yandex-appmetrica-plugin-cordova/platforms/android/com/yandex/metrica/plugin/cordova/AppMetricaPlugin.java
+++ b/node_modules/yandex-appmetrica-plugin-cordova/platforms/android/com/yandex/metrica/plugin/cordova/AppMetricaPlugin.java
@@ -209,10 +209,12 @@ public class AppMetricaPlugin extends CordovaPlugin {
             eventParametersJSONString = eventParametersObj.toString();
         } catch (JSONException ignored) {}
 
-        if (eventParametersJSONString != null) {
-            YandexMetrica.reportEvent(eventName, eventParametersJSONString);
-        } else {
-            YandexMetrica.reportEvent(eventName);
+        if (mAppMetricaActivated) {
+            if (eventParametersJSONString != null) {
+                YandexMetrica.reportEvent(eventName, eventParametersJSONString);
+            } else {
+                YandexMetrica.reportEvent(eventName);
+            }
         }
     }
 
@@ -225,7 +227,9 @@ public class AppMetricaPlugin extends CordovaPlugin {
             errorThrowable = new Throwable(errorReason);
         } catch (JSONException ignored) {}
 
-        YandexMetrica.reportError(errorName, errorThrowable);
+        if (mAppMetricaActivated) {
+            YandexMetrica.reportError(errorName, errorThrowable);
+        }
     }
 
     private void setLocation(final JSONArray args,
@@ -233,13 +237,18 @@ public class AppMetricaPlugin extends CordovaPlugin {
         final JSONObject locationObj = args.getJSONObject(0);
 
         final Location location = toLocation(locationObj);
-        YandexMetrica.setLocation(location);
+
+        if (mAppMetricaActivated) {
+            YandexMetrica.setLocation(location);
+        }
     }
 
     private void setLocationTracking(final JSONArray args,
                                      final CallbackContext callbackContext) throws JSONException {
         final boolean enabled = args.getBoolean(0);
 
-        YandexMetrica.setLocationTracking(enabled);
+        if (mAppMetricaActivated) {
+            YandexMetrica.setLocationTracking(enabled);
+        }
     }
 }

This issue body was partially generated by patch-package.

[ghost]

Добрый день!Ваше письмо получено и зарегистрировано под уникальным номером № 18823391. Ответ обязательно будет предоставлен, пожалуйста, ожидайте.Просим Вас не отправлять повторное письмо по данному вопросу - оно будет считаться новой заявкой и может быть обработано позднее.Помощь сервиса: https://yandex.ru/support/direct/?from=email--С уважением,Отдел клиентского сервисател.: 8 800 234-24-80 (звонок из регионов России бесплатный)тел.: +7 495 739-37-77&nbsp19.08.2022, 19:43, "vovka-s" ***@***.***>: Привет! Столкнулся с крешами приложухи, возникающими из-за вызовова reportEvent до инициализации метрики Добавил проверок на активность - креши прекратились. Прошу проверить, всё ли верно, и залить в реп diff --git a/node_modules/yandex-appmetrica-plugin-cordova/platforms/android/com/yandex/metrica/plugin/cordova/AppMetricaPlugin.java b/node_modules/yandex-appmetrica-plugin-cordova/platforms/android/com/yandex/metrica/plugin/cordova/AppMetricaPlugin.java index 1512def..ff1300a 100644 --- a/node_modules/yandex-appmetrica-plugin-cordova/platforms/android/com/yandex/metrica/plugin/cordova/AppMetricaPlugin.java +++ b/node_modules/yandex-appmetrica-plugin-cordova/platforms/android/com/yandex/metrica/plugin/cordova/AppMetricaPlugin.java @@ -209,10 +209,12 @@ public class AppMetricaPlugin extends CordovaPlugin { eventParametersJSONString = eventParametersObj.toString(); } catch (JSONException ignored) {} - if (eventParametersJSONString != null) { - YandexMetrica.reportEvent(eventName, eventParametersJSONString); - } else { - YandexMetrica.reportEvent(eventName); + if (mAppMetricaActivated) { + if (eventParametersJSONString != null) { + YandexMetrica.reportEvent(eventName, eventParametersJSONString); + } else { + YandexMetrica.reportEvent(eventName); + } } } @@ -225,7 +227,9 @@ public class AppMetricaPlugin extends CordovaPlugin { errorThrowable = new Throwable(errorReason); } catch (JSONException ignored) {} - YandexMetrica.reportError(errorName, errorThrowable); + if (mAppMetricaActivated) { + YandexMetrica.reportError(errorName, errorThrowable); + } } private void setLocation(final JSONArray args, @@ -233,13 +237,18 @@ public class AppMetricaPlugin extends CordovaPlugin { final JSONObject locationObj = args.getJSONObject(0); final Location location = toLocation(locationObj); - YandexMetrica.setLocation(location); + + if (mAppMetricaActivated) { + YandexMetrica.setLocation(location); + } } private void setLocationTracking(final JSONArray args, final CallbackContext callbackContext) throws JSONException { final boolean enabled = args.getBoolean(0); - YandexMetrica.setLocationTracking(enabled); + if (mAppMetricaActivated) { + YandexMetrica.setLocationTracking(enabled); + } } } This issue body was partially generated by patch-package. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>