From c3d7f7e36e62c51f7c3ac555e4903bde859bdff7 Mon Sep 17 00:00:00 2001
From: Marcel Baumgartner <marcel.kbkm@gmail.com>
Date: Wed, 12 Apr 2023 16:52:09 +0200
Subject: [PATCH] Implemented new soft error boundary crash handler

---
 .../ErrorBoundaries/SoftErrorBoundary.razor   | 29 ++++++-
 Moonlight/Shared/Layouts/MainLayout.razor     | 82 +++++++++----------
 Moonlight/Shared/Views/Test.razor             | 29 +------
 3 files changed, 67 insertions(+), 73 deletions(-)

diff --git a/Moonlight/Shared/Components/ErrorBoundaries/SoftErrorBoundary.razor b/Moonlight/Shared/Components/ErrorBoundaries/SoftErrorBoundary.razor
index 7f56120..3d59dd7 100644
--- a/Moonlight/Shared/Components/ErrorBoundaries/SoftErrorBoundary.razor
+++ b/Moonlight/Shared/Components/ErrorBoundaries/SoftErrorBoundary.razor
@@ -7,13 +7,35 @@
 @inject AlertService AlertService
 @inject SmartTranslateService SmartTranslateService
 
-@ChildContent
+@if (Crashed)
+{
+    <div class="card card-flush h-md-100">
+        <div class="card-body d-flex flex-column justify-content-between mt-9 bgi-no-repeat bgi-size-cover bgi-position-x-center pb-0">
+            <div class="mb-10">
+                <div class="fs-2hx fw-bold text-gray-800 text-center mb-13">
+                    <span class="me-2">
+                        <TL>Ooops. This page is crashed</TL>
+                    </span>
+                </div>
+                <div class="text-center">
+                    <TL>This page is crashed. The error has been reported to the moonlight team. Meanwhile you can try reloading the page</TL>
+                </div>
+            </div>
+        </div>
+    </div>
+}
+else
+{
+    @ChildContent
+}
 
 @code
 {
+    private bool Crashed = false;
+    
     protected override async Task OnErrorAsync(Exception exception)
     {
-        Logger.Debug(exception);
+        Logger.Warn(exception);
         
         if (exception is DisplayException displayException)
         {
@@ -49,7 +71,8 @@
         }
         else
         {
-            throw exception;
+            Crashed = true;
+            await InvokeAsync(StateHasChanged);
         }
     }
 }
\ No newline at end of file
diff --git a/Moonlight/Shared/Layouts/MainLayout.razor b/Moonlight/Shared/Layouts/MainLayout.razor
index 4c42ecb..46017b5 100644
--- a/Moonlight/Shared/Layouts/MainLayout.razor
+++ b/Moonlight/Shared/Layouts/MainLayout.razor
@@ -59,57 +59,55 @@
                             <div id="kt_app_content" class="app-content flex-column-fluid">
                                 <div id="kt_app_content_container" class="app-container container-fluid">
                                     <div class="mt-10">
-                                        <PageErrorBoundary>
-                                            <SoftErrorBoundary>
-                                                @if (uri.LocalPath != "/login" &&
-                                                     uri.LocalPath != "/passwordreset" &&
-                                                     uri.LocalPath != "/register")
+                                        <SoftErrorBoundary>
+                                            @if (uri.LocalPath != "/login" &&
+                                                 uri.LocalPath != "/passwordreset" &&
+                                                 uri.LocalPath != "/register")
+                                            {
+                                                if (User == null)
                                                 {
-                                                    if (User == null)
-                                                    {
-                                                        <Login></Login>
-                                                    }
-                                                    else
-                                                    {
-                                                        if (User.Status == UserStatus.Banned)
-                                                        {
-                                                            <BannedAlert></BannedAlert>
-                                                        }
-                                                        else if (User.Status == UserStatus.Disabled)
-                                                        {
-                                                            <DisabledAlert></DisabledAlert>
-                                                        }
-                                                        else if (User.Status == UserStatus.PasswordPending)
-                                                        {
-                                                            <PasswordChangeView></PasswordChangeView>
-                                                        }
-                                                        else if (User.Status == UserStatus.DataPending)
-                                                        {
-                                                            <UserDataSetView></UserDataSetView>
-                                                        }
-                                                        else
-                                                        {
-                                                            @Body
-                                                        }
-                                                    }
+                                                    <Login></Login>
                                                 }
                                                 else
                                                 {
-                                                    if (uri.LocalPath == "/login")
+                                                    if (User.Status == UserStatus.Banned)
                                                     {
-                                                        <Login></Login>
+                                                        <BannedAlert></BannedAlert>
                                                     }
-                                                    else if (uri.LocalPath == "/register")
+                                                    else if (User.Status == UserStatus.Disabled)
                                                     {
-                                                        <Register></Register>
+                                                        <DisabledAlert></DisabledAlert>
                                                     }
-                                                    else if (uri.LocalPath == "/passwordreset")
+                                                    else if (User.Status == UserStatus.PasswordPending)
                                                     {
-                                                        <PasswordReset></PasswordReset>
+                                                        <PasswordChangeView></PasswordChangeView>
+                                                    }
+                                                    else if (User.Status == UserStatus.DataPending)
+                                                    {
+                                                        <UserDataSetView></UserDataSetView>
+                                                    }
+                                                    else
+                                                    {
+                                                        @Body
                                                     }
                                                 }
-                                            </SoftErrorBoundary>
-                                        </PageErrorBoundary>
+                                            }
+                                            else
+                                            {
+                                                if (uri.LocalPath == "/login")
+                                                {
+                                                    <Login></Login>
+                                                }
+                                                else if (uri.LocalPath == "/register")
+                                                {
+                                                    <Register></Register>
+                                                }
+                                                else if (uri.LocalPath == "/passwordreset")
+                                                {
+                                                    <PasswordReset></PasswordReset>
+                                                }
+                                            }
+                                        </SoftErrorBoundary>
                                     </div>
                                 </div>
                             </div>
@@ -161,8 +159,8 @@
                 await JsRuntime.InvokeVoidAsync("document.body.removeAttribute", "data-kt-app-page-loading");
                 await JsRuntime.InvokeVoidAsync("KTMenu.createInstances");
                 await JsRuntime.InvokeVoidAsync("KTDrawer.createInstances");
-                
-                //await JsRuntime.InvokeVoidAsync("createSnow");
+
+    //await JsRuntime.InvokeVoidAsync("createSnow");
 
                 await SessionService.Register();
 
diff --git a/Moonlight/Shared/Views/Test.razor b/Moonlight/Shared/Views/Test.razor
index 88c12f0..906f424 100644
--- a/Moonlight/Shared/Views/Test.razor
+++ b/Moonlight/Shared/Views/Test.razor
@@ -1,39 +1,12 @@
 @page "/test"
-@using Moonlight.App.Repositories
-@using Moonlight.App.Database.Entities
-@using Moonlight.App.Models.Forms
-
-@inject UserRepository UserRepository
 
 <LazyLoader Load="Load">
-    <SmartForm Model="Model" OnValidSubmit="OnValidSubmit">
-        <div class="mb-3">
-            <SmartDropdown 
-                Items="Users" 
-                @bind-Value="Model.User" 
-                DisplayFunc="@(x => x.Email)" 
-                SearchProp="@(x => x.Email)" />
-        </div>
-        <div>
-            <button class="btn btn-primary" type="submit">Submit</button>
-        </div>
-    </SmartForm>
 </LazyLoader>
 
 @code
 {
-    private User[] Users;
-    private TestDataModel Model = new();
-
     private Task Load(LazyLoader arg)
     {
-        Users = UserRepository.Get().ToArray();
-        
-        return Task.CompletedTask;
-    }
-
-    private Task OnValidSubmit()
-    {
-        return Task.CompletedTask;
+        throw new Exception("Nein");
     }
 }