Correct an issue on the obversers when we use URL direct... When using URL direct, the register can occur before we have the javascript loaded

Author: Jérémy JANISZEWSKI

src/Community.Components/Components/Observers/FluentCxObserverProvider.razor.cs

@@ -27,7 +27,7 @@ public sealed partial class FluentCxObserverProvider
     /// </summary>
     /// <remarks>This field is used to track the rendering state of the object.  It is intended for internal
     /// use and should not be accessed directly.</remarks>
-    private bool _isRendered;
+    private bool _hasModule;
 
     /// <summary>
     /// A thread-safe queue used to store pending observer items for processing.
@@ -108,7 +108,7 @@ public FluentCxObserverProvider()
     /// <returns>A <see cref="ValueTask"/> that represents the asynchronous operation.</returns>
     internal async ValueTask NotifyAsync(GroupObserverItem group)
     {
-        if (_isRendered)
+        if (_hasModule)
         {
             foreach (var item in group.Items)
             {
@@ -144,7 +144,7 @@ internal async ValueTask NotifyAsync(GroupObserverItem group)
     /// <returns>A <see cref="ValueTask"/> that represents the asynchronous operation.</returns>
     internal async ValueTask NotifyAsync(ObserverItem item)
     {
-        if (_isRendered)
+        if (_hasModule)
         {
             if (item.ObserveIntersection)
             {
@@ -172,7 +172,7 @@ internal async ValueTask NotifyAsync(ObserverItem item)
     /// <returns></returns>
     internal async ValueTask UnnotifyGroupAsync(string groupId)
     {
-        if (_isRendered)
+        if (_hasModule)
         {
             var items = ObserverState.GetItems(groupId);
 
@@ -192,7 +192,7 @@ internal async ValueTask UnnotifyGroupAsync(string groupId)
     /// <returns>A <see cref="ValueTask"/> that represents the asynchronous operation.</returns>
     internal async ValueTask UnnotifyItemAsync(string id)
     {
-        if (_isRendered)
+        if (_hasModule)
         {
             await UnregisterIntersectAsync(id, id);
             await UnregisterResizeAsync(id, id);
@@ -202,9 +202,9 @@ internal async ValueTask UnnotifyItemAsync(string id)
     /// <inheritdoc />
     public async Task RegisterIntersectAsync(string groupId, string elementId, ElementReference elementReference)
     {
-        if (_jsModule is not null)
+        if (_hasModule)
         {
-            await _jsModule.InvokeVoidAsync("fluentCxObserverProvider.registerIntersect", groupId, elementId, elementReference, _dotNetRef, new { debounce = (int)DebounceIntersect.TotalMilliseconds, threshold = Threshold });
+            await _jsModule!.InvokeVoidAsync("fluentCxObserverProvider.registerIntersect", groupId, elementId, elementReference, _dotNetRef, new { debounce = (int)DebounceIntersect.TotalMilliseconds, threshold = Threshold });
         }
     }
 
@@ -217,9 +217,9 @@ public Task RegisterIntersectAsync(string elementId, ElementReference elementRef
     /// <inheritdoc />
     public async Task RegisterResizeAsync(string groupId, string elementId, ElementReference elementReference)
     {
-        if (_jsModule is not null)
+        if (_hasModule)
         {
-            await _jsModule.InvokeVoidAsync("fluentCxObserverProvider.registerResize", groupId, elementId, elementReference, _dotNetRef, new { debounce = (int)DebounceResize.TotalMilliseconds });
+            await _jsModule!.InvokeVoidAsync("fluentCxObserverProvider.registerResize", groupId, elementId, elementReference, _dotNetRef, new { debounce = (int)DebounceResize.TotalMilliseconds });
         }
     }
 
@@ -238,9 +238,9 @@ public Task UnregisterIntersectAsync(string elementId)
     /// <inheritdoc />
     public async Task UnregisterIntersectAsync(string groupId, string elementId)
     {
-        if (_jsModule is not null)
+        if (_hasModule)
         {
-            await _jsModule.InvokeVoidAsync("fluentCxObserverProvider.unregisterIntersect", groupId, elementId);
+            await _jsModule!.InvokeVoidAsync("fluentCxObserverProvider.unregisterIntersect", groupId, elementId);
         }
     }
 
@@ -253,27 +253,27 @@ public Task UnregisterResizeAsync(string elementId)
     /// <inheritdoc />
     public async Task UnregisterResizeAsync(string groupId, string elementId)
     {
-        if (_jsModule is not null)
+        if (_hasModule)
         {
-            await _jsModule.InvokeVoidAsync("fluentCxObserverProvider.unregisterResize", groupId, elementId);
+            await _jsModule!.InvokeVoidAsync("fluentCxObserverProvider.unregisterResize", groupId, elementId);
         }
     }
 
     /// <inheritdoc />
     public async Task RegisterMutationAsync(string groupId, string elementId, ElementReference elementReference, MutationObserverOptions options)
     {
-        if (_jsModule is not null)
+        if (_hasModule)
         {
-            await _jsModule.InvokeVoidAsync("fluentCxObserverProvider.registerMutation", groupId, elementId, elementReference, options);
+            await _jsModule!.InvokeVoidAsync("fluentCxObserverProvider.registerMutation", groupId, elementId, elementReference, options);
         }
     }
 
     /// <inheritdoc />
     public async Task UnregisterMutationAsync(string groupId, string elementId)
     {
-        if (_jsModule is not null)
+        if (_hasModule)
         {
-            await _jsModule.InvokeVoidAsync("fluentCxObserverProvider.unregisterMutation", groupId, elementId);
+            await _jsModule!.InvokeVoidAsync("fluentCxObserverProvider.unregisterMutation", groupId, elementId);
         }
     }
 
@@ -291,8 +291,8 @@ protected override async Task OnAfterRenderAsync(bool firstRender)
 
         if (firstRender)
         {
-            _isRendered = firstRender;
             _jsModule ??= await JSRuntime.InvokeAsync<IJSObjectReference>("import", JsModulePath);
+            _hasModule = _jsModule is not null;
 
             while (_pendingItems.TryDequeue(out var item))
             {