From 8711eae4d8e8864b4abe0a1d8e0f156a1e6bfd84 Mon Sep 17 00:00:00 2001 From: Hans Kokx Date: Tue, 29 Apr 2025 15:00:48 +0200 Subject: [PATCH] Fix updateShouldNotify on ArcaneServiceProvider Signed-off-by: Hans Kokx --- lib/src/providers/service_provider.dart | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/lib/src/providers/service_provider.dart b/lib/src/providers/service_provider.dart index 68856ec..fb2fdf9 100644 --- a/lib/src/providers/service_provider.dart +++ b/lib/src/providers/service_provider.dart @@ -33,12 +33,13 @@ class ArcaneServiceProvider extends InheritedNotifier { super.key, }); - /// Determines whether the widget should notify its dependents. - /// - /// This always returns `true`, meaning dependents will always be notified - /// when this widget is rebuilt. @override - bool updateShouldNotify(_) => true; + bool updateShouldNotify(covariant ArcaneServiceProvider oldWidget) { + return !const DeepCollectionEquality().equals( + serviceInstances, + oldWidget.serviceInstances, + ); + } /// Retrieves the nearest `ArcaneServiceProvider` in the widget tree. ///