From bf1e43fe989b25dccad5afa8c17ca95283b63ebc Mon Sep 17 00:00:00 2001
From: Dinh Nguyen <dinh.nguyentrinhquoc@axonactive.com>
Date: Tue, 23 Jul 2024 23:15:25 +0700
Subject: [PATCH] refactor service

---
 marketplace-ui/src/app/app.component.html        |  4 ++--
 marketplace-ui/src/app/app.component.ts          | 16 +++++-----------
 .../src/app/cookie.management.service.ts         | 14 +++++++++-----
 .../product-detail-version-action.component.ts   |  2 +-
 4 files changed, 17 insertions(+), 19 deletions(-)

diff --git a/marketplace-ui/src/app/app.component.html b/marketplace-ui/src/app/app.component.html
index 86dd5834e..fa248498d 100644
--- a/marketplace-ui/src/app/app.component.html
+++ b/marketplace-ui/src/app/app.component.html
@@ -1,4 +1,4 @@
-@if(!isDesignerViewer()){
+@if(!cookieManagementService.isDesignerEnv()){
 <header class="d-flex align-items-center">
   <div class="container">
     <app-header></app-header>
@@ -10,7 +10,7 @@
     <router-outlet />
   </div>
 </main>
-@if(!isDesignerViewer()){
+@if(!cookieManagementService.isDesignerEnv()){
 <footer>
   <div class="container">
     <app-footer></app-footer>
diff --git a/marketplace-ui/src/app/app.component.ts b/marketplace-ui/src/app/app.component.ts
index 924f496b5..323bdc0d7 100644
--- a/marketplace-ui/src/app/app.component.ts
+++ b/marketplace-ui/src/app/app.component.ts
@@ -1,9 +1,7 @@
-import { Component, inject, signal } from '@angular/core';
+import { Component, inject } from '@angular/core';
 import {
   RouterOutlet,
-  ActivatedRoute,
-  Router,
-  NavigationEnd,
+  ActivatedRoute
 } from '@angular/router';
 import { FooterComponent } from './shared/components/footer/footer.component';
 import { HeaderComponent } from './shared/components/header/header.component';
@@ -19,21 +17,17 @@ import { CookieManagementService } from './cookie.management.service';
 })
 export class AppComponent {
   loadingService = inject(LoadingService);
-  isDesignerViewer = signal(false);
+  cookieManagementService = inject( CookieManagementService);
 
   constructor(
-    private router: Router,
     private route: ActivatedRoute,
-    private readonly cookieManagementService: CookieManagementService
   ) { }
 
   ngOnInit(): void {
     this.cookieManagementService.getNavigationEndEvents().subscribe(() => {
-      if (this.cookieManagementService.isBrowserHaveDesignerEnvCookie()) {
-        this.isDesignerViewer.set(true);
-      } else {
+      if (!this.cookieManagementService.isDesignerEnv()) {
         this.route.queryParams.subscribe(params => {
-          this.cookieManagementService.checkCookieForDesignerEnv(params, this.isDesignerViewer);
+          this.cookieManagementService.checkCookieForDesignerEnv(params);
           this.cookieManagementService.checkCookieForDesignerVersion(params);
         })
       }
diff --git a/marketplace-ui/src/app/cookie.management.service.ts b/marketplace-ui/src/app/cookie.management.service.ts
index d80ef1965..6e9889b6f 100644
--- a/marketplace-ui/src/app/cookie.management.service.ts
+++ b/marketplace-ui/src/app/cookie.management.service.ts
@@ -1,5 +1,5 @@
 
-import { Injectable, signal } from '@angular/core';
+import { Injectable, Signal, signal } from '@angular/core';
 import { CookieService } from 'ngx-cookie-service';
 import { DESIGNER_COOKIE_VARIABLE } from './shared/constants/common.constant';
 import {
@@ -13,6 +13,7 @@ import { filter } from 'rxjs/operators';
   providedIn: 'root'
 })
 export class CookieManagementService {
+  private isDesigner = signal(false);
   constructor(private cookieService: CookieService, private router: Router) { }
 
   checkCookieForDesignerVersion(params: Params) {
@@ -22,16 +23,19 @@ export class CookieManagementService {
     }
   }
 
-  checkCookieForDesignerEnv(params: Params, isDesignerViewerSignal: any) {
+  checkCookieForDesignerEnv(params: Params) {
     const ivyViewerParam = params[DESIGNER_COOKIE_VARIABLE.ivyViewerParamName];
     if (ivyViewerParam == DESIGNER_COOKIE_VARIABLE.defaultDesignerViewer) {
       this.cookieService.set(DESIGNER_COOKIE_VARIABLE.ivyViewerParamName, ivyViewerParam);
-      isDesignerViewerSignal.set(true);
+      this.isDesigner.set(true);
     }
   }
 
-  isBrowserHaveDesignerEnvCookie() {
-    return this.cookieService.get(DESIGNER_COOKIE_VARIABLE.ivyViewerParamName) == DESIGNER_COOKIE_VARIABLE.defaultDesignerViewer;
+  isDesignerEnv() {
+    if(!this.isDesigner()) {
+      this.isDesigner.set(this.cookieService.get(DESIGNER_COOKIE_VARIABLE.ivyViewerParamName) == DESIGNER_COOKIE_VARIABLE.defaultDesignerViewer);
+    }
+    return this.isDesigner();
   }
 
   getNavigationEndEvents(): Observable<NavigationEnd> {
diff --git a/marketplace-ui/src/app/modules/product/product-detail/product-detail-version-action/product-detail-version-action.component.ts b/marketplace-ui/src/app/modules/product/product-detail/product-detail-version-action/product-detail-version-action.component.ts
index e6f8489d6..f6041e906 100644
--- a/marketplace-ui/src/app/modules/product/product-detail/product-detail-version-action/product-detail-version-action.component.ts
+++ b/marketplace-ui/src/app/modules/product/product-detail/product-detail-version-action/product-detail-version-action.component.ts
@@ -57,7 +57,7 @@ export class ProductDetailVersionActionComponent implements AfterViewInit {
     tooltipTriggerList.forEach(
       tooltipTriggerEl => new Tooltip(tooltipTriggerEl)
     );
-    this.isDesignerEnvironment.set(this.cookieService.isBrowserHaveDesignerEnvCookie());
+    this.isDesignerEnvironment.set(this.cookieService.isDesignerEnv());
   }
 
   onShowVersions() {