From 0c11cadfcc44302e8df62bc331ae1eaa3c00a76c Mon Sep 17 00:00:00 2001
From: Sebastian Stehle <sebastian@squidex.io>
Date: Mon, 21 Jan 2019 17:46:39 +0100
Subject: [PATCH] Logging fixed.

---
 src/Squidex/Pipeline/RequestLogPerformanceMiddleware.cs   | 2 +-
 .../pages/restore/restore-page.component.scss             | 6 +++---
 .../dashboard/pages/dashboard-page.component.html         | 4 +++-
 .../dashboard/pages/dashboard-page.component.scss         | 4 ++++
 .../features/dashboard/pages/dashboard-page.component.ts  | 8 ++++++++
 5 files changed, 19 insertions(+), 5 deletions(-)

diff --git a/src/Squidex/Pipeline/RequestLogPerformanceMiddleware.cs b/src/Squidex/Pipeline/RequestLogPerformanceMiddleware.cs
index 59630c31d..87915019a 100644
--- a/src/Squidex/Pipeline/RequestLogPerformanceMiddleware.cs
+++ b/src/Squidex/Pipeline/RequestLogPerformanceMiddleware.cs
@@ -66,7 +66,7 @@ namespace Squidex.Pipeline
 
             var clientId = context.User.OpenIdClientId();
 
-            if (!string.IsNullOrWhiteSpace(userId))
+            if (!string.IsNullOrWhiteSpace(clientId))
             {
                 c.WriteProperty(nameof(clientId), clientId);
             }
diff --git a/src/Squidex/app/features/administration/pages/restore/restore-page.component.scss b/src/Squidex/app/features/administration/pages/restore/restore-page.component.scss
index ccd571a75..f4cc3a431 100644
--- a/src/Squidex/app/features/administration/pages/restore/restore-page.component.scss
+++ b/src/Squidex/app/features/administration/pages/restore/restore-page.component.scss
@@ -47,15 +47,15 @@ h3 {
             color: $color-dark-foreground;
             vertical-align: middle;
         }
-    
+
         &-pending {
             color: inherit;
         }
-    
+
         &-failed {
             background: $color-theme-error;
         }
-    
+
         &-success {
             background: $color-theme-green;
         }
diff --git a/src/Squidex/app/features/dashboard/pages/dashboard-page.component.html b/src/Squidex/app/features/dashboard/pages/dashboard-page.component.html
index 32dede6dd..03b8b2286 100644
--- a/src/Squidex/app/features/dashboard/pages/dashboard-page.component.html
+++ b/src/Squidex/app/features/dashboard/pages/dashboard-page.component.html
@@ -73,7 +73,9 @@
                         API Calls
 
                         <div class="float-right">
-                            <a class="force" (click)="downloadLog()">
+                            <img *ngIf="isLoadingLog" class="loader mr-2" src="/images/loader.gif" />
+
+                            <a class="force" (click)="downloadLog()" [class.disabled]="isLoadingLog">
                                 <small>Download Log</small>
                             </a>
                         </div>
diff --git a/src/Squidex/app/features/dashboard/pages/dashboard-page.component.scss b/src/Squidex/app/features/dashboard/pages/dashboard-page.component.scss
index 936b11b25..6e3dcd637 100644
--- a/src/Squidex/app/features/dashboard/pages/dashboard-page.component.scss
+++ b/src/Squidex/app/features/dashboard/pages/dashboard-page.component.scss
@@ -31,6 +31,10 @@
     color: $color-text-decent;
 }
 
+.loader {
+    height: 12px;
+}
+
 .card {
     & {
         margin-right: 1rem;
diff --git a/src/Squidex/app/features/dashboard/pages/dashboard-page.component.ts b/src/Squidex/app/features/dashboard/pages/dashboard-page.component.ts
index 836767c3f..0e2c851ba 100644
--- a/src/Squidex/app/features/dashboard/pages/dashboard-page.component.ts
+++ b/src/Squidex/app/features/dashboard/pages/dashboard-page.component.ts
@@ -99,6 +99,8 @@ export class DashboardPageComponent implements OnDestroy, OnInit {
     public callsCurrent = 0;
     public callsMax = 0;
 
+    public isLoadingLog = false;
+
     constructor(
         public readonly appsState: AppsState,
         public readonly authState: AuthService,
@@ -209,9 +211,15 @@ export class DashboardPageComponent implements OnDestroy, OnInit {
     }
 
     public downloadLog() {
+        this.isLoadingLog = true;
+
         this.usagesService.getLog(this.appsState.appName)
             .subscribe(buffer => {
                 saveAs(buffer, 'Log.csv');
+            }, () => {
+                this.isLoadingLog = false;
+            }, () => {
+                this.isLoadingLog = false;
             });
     }
 }