Remove deprecated fields (#1227)

CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## Unreleased
+
+### Breaking Changes
+
+- Remove deprecated fields ([#1227](https://github.com/getsentry/sentry-dart/pull/1227))
+  - Remove deprecated fields from the `Scope` class.
+    - `user(SentryUser? user)`, using the `setUser(SentryUser? user)` instead.
+    - `attachements`, using the `attachments` instead.
+  - Remove deprecated field from the `SentryFlutterOptions` class.
+    - `anrTimeoutIntervalMillis`, using the `anrTimeoutInterval` instead.
+    - `autoSessionTrackingIntervalMillis`, using the `autoSessionTrackingInterval` instead.
+
 ## 7.0.0-alpha.3
 
 ### Breaking Changes

dart/lib/src/protocol/sentry_request.dart

@@ -55,7 +55,7 @@ class SentryRequest {
 
   final Map<String, String>? _other;
 
-  @Deprecated('Will be removed in v7')
+  @Deprecated('Will be removed in v8. Use [data] instead')
   Map<String, String> get other => Map.unmodifiable(_other ?? const {});
 
   /// The fragment of the request URL.
@@ -70,7 +70,8 @@ class SentryRequest {
     dynamic data,
     Map<String, String>? headers,
     Map<String, String>? env,
-    @Deprecated('Will be removed in v7.') Map<String, String>? other,
+    @Deprecated('Will be removed in v8. Use [data] instead')
+        Map<String, String>? other,
   })  : _data = data,
         _headers = headers != null ? Map.from(headers) : null,
         // Look for a 'Set-Cookie' header (case insensitive) if not given.
@@ -88,7 +89,8 @@ class SentryRequest {
     dynamic data,
     Map<String, String>? headers,
     Map<String, String>? env,
-    @Deprecated('Will be removed in v7.') Map<String, String>? other,
+    @Deprecated('Will be removed in v8. Use [data] instead')
+        Map<String, String>? other,
   }) {
     // As far as I can tell there's no way to get the uri without the query part
     // so we replace it with an empty string.
@@ -158,7 +160,8 @@ class SentryRequest {
     dynamic data,
     Map<String, String>? headers,
     Map<String, String>? env,
-    @Deprecated('Will be removed in v7.') Map<String, String>? other,
+    @Deprecated('Will be removed in v8. Use [data] instead')
+        Map<String, String>? other,
   }) =>
       SentryRequest(
         url: url ?? this.url,

dart/lib/src/protocol/sentry_user.dart

@@ -42,7 +42,7 @@ class SentryUser {
     this.geo,
     this.name,
     Map<String, dynamic>? data,
-    @Deprecated('Will be removed in v7. Use [data] instead')
+    @Deprecated('Will be removed in v8. Use [data] instead')
         Map<String, dynamic>? extras,
   })  : assert(
           id != null ||
@@ -76,7 +76,7 @@ class SentryUser {
   /// by Sentry.
   final Map<String, dynamic>? data;
 
-  @Deprecated('Will be removed in v7. Use [data] instead')
+  @Deprecated('Will be removed in v8. Use [data] instead')
   final Map<String, dynamic>? extras;
 
   /// Approximate geographical location of the end user or device.
@@ -144,7 +144,7 @@ class SentryUser {
     String? email,
     String? ipAddress,
     String? segment,
-    @Deprecated('Will be removed in v7. Use [data] instead')
+    @Deprecated('Will be removed in v8. Use [data] instead')
         Map<String, dynamic>? extras,
     String? name,
     SentryGeo? geo,

dart/lib/src/scope.dart

@@ -44,16 +44,6 @@ class Scope {
   /// Get the current user.
   SentryUser? get user => _user;
 
-  /// Sets the current user
-  /// This method is deprecated, use the [setUser(user)] instead.
-  /// This method will be removed in the future.
-  /// The breaking change is due to the [enableScopeSync] feature that
-  /// requires returning a [Future].
-  @Deprecated('Use [setUser(user)] instead')
-  set user(SentryUser? user) {
-    _user = user;
-  }
-
   void _setUserSync(SentryUser? user) {
     _user = user;
   }
@@ -151,9 +141,6 @@ class Scope {
 
   List<SentryAttachment> get attachments => List.unmodifiable(_attachments);
 
-  @Deprecated('Use [attachments] instead')
-  List<SentryAttachment> get attachements => attachments;
-
   Scope(this._options);
 
   bool _addBreadCrumbSync(Breadcrumb breadcrumb, {Hint? hint}) {

dart/lib/src/sentry_options.dart

@@ -436,7 +436,7 @@ typedef TracesSamplerCallback = double? Function(
     SentrySamplingContext samplingContext);
 
 /// A NoOp logger that does nothing
-@Deprecated('This will be made private or removed in the future')
+@Deprecated('Will be removed in v8. Disable [debug] instead')
 void noOpLogger(
   SentryLevel level,
   String message, {
@@ -446,7 +446,7 @@ void noOpLogger(
 }) {}
 
 /// A Logger that prints out the level and message
-@Deprecated('This will be made private or removed in the future')
+@Deprecated('Will be removed in v8. Enable [debug] instead')
 void dartLogger(
   SentryLevel level,
   String message, {

flutter/lib/src/sentry_flutter_options.dart

@@ -39,20 +39,6 @@ class SentryFlutterOptions extends SentryOptions {
   /// feature, as this is required to mark Sessions as Crashed.
   bool enableNativeCrashHandling = true;
 
-  /// The session tracking interval in millis. This is the interval to end a
-  /// session if the App goes to the background.
-  /// See: [enableAutoSessionTracking]
-  @Deprecated('Use autoSessionTrackingInterval instead')
-  int get autoSessionTrackingIntervalMillis =>
-      autoSessionTrackingInterval.inMilliseconds;
-
-  @Deprecated('Use autoSessionTrackingInterval instead')
-  set autoSessionTrackingIntervalMillis(int value) {
-    autoSessionTrackingInterval = value >= 0
-        ? Duration(milliseconds: value)
-        : autoSessionTrackingInterval;
-  }
-
   Duration _autoSessionTrackingInterval = Duration(milliseconds: 30000);
 
   /// The session tracking interval. This is the interval to end a
@@ -74,19 +60,6 @@ class SentryFlutterOptions extends SentryOptions {
   /// Java/Kotlin code as well.
   bool anrEnabled = false;
 
-  /// ANR Timeout internal in Millis Default is 5000 = 5s Used by AnrIntegration.
-  /// Available only for Android.
-  /// See: [anrEnabled]
-  @Deprecated('Use anrTimeoutInterval instead')
-  int get anrTimeoutIntervalMillis => anrTimeoutInterval.inMilliseconds;
-
-  @Deprecated('Use anrTimeoutInterval instead')
-  set anrTimeoutIntervalMillis(int value) {
-    assert(value > 0);
-    anrTimeoutInterval =
-        value > 0 ? Duration(milliseconds: value) : anrTimeoutInterval;
-  }
-
   Duration _anrTimeoutInterval = Duration(milliseconds: 5000);
 
   /// ANR Timeout internal. Default is 5000 milliseconds or 5 seconds.