From 7c4d4bae65a70789fa11746e3552b0ceaac12769 Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Fri, 11 Jun 2021 09:39:34 +0300
Subject: [PATCH] hide routes which have not path

#9289
---
 npm/ng-packs/apps/dev-app/src/app/app.module.ts    |  2 +-
 .../apps/dev-app/src/app/route.provider.ts         |  2 +-
 .../account/config/src/providers/route.provider.ts |  2 +-
 .../core/src/lib/services/routes.service.ts        |  3 ++-
 .../config/src/providers/route.provider.ts         |  2 +-
 .../config/src/providers/route.provider.ts         |  2 +-
 .../lib/components/routes/routes.component.html    | 14 ++++++++------
 7 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/npm/ng-packs/apps/dev-app/src/app/app.module.ts b/npm/ng-packs/apps/dev-app/src/app/app.module.ts
index dd3a9636db..94ebbb5cc0 100644
--- a/npm/ng-packs/apps/dev-app/src/app/app.module.ts
+++ b/npm/ng-packs/apps/dev-app/src/app/app.module.ts
@@ -1,3 +1,4 @@
+import { AccountConfigModule } from '@abp/ng.account/config';
 import { CoreModule } from '@abp/ng.core';
 import { registerLocale } from '@abp/ng.core/locale';
 import { IdentityConfigModule } from '@abp/ng.identity/config';
@@ -11,7 +12,6 @@ import { BrowserAnimationsModule } from '@angular/platform-browser/animations';
 import { InspectorModule } from '@ngneat/inspector';
 import { NgxsLoggerPluginModule } from '@ngxs/logger-plugin';
 import { NgxsModule } from '@ngxs/store';
-import { AccountConfigModule } from '@abp/ng.account/config';
 import { environment } from '../environments/environment';
 import { AppRoutingModule } from './app-routing.module';
 import { AppComponent } from './app.component';
diff --git a/npm/ng-packs/apps/dev-app/src/app/route.provider.ts b/npm/ng-packs/apps/dev-app/src/app/route.provider.ts
index 73865dfd0a..a03945ada0 100644
--- a/npm/ng-packs/apps/dev-app/src/app/route.provider.ts
+++ b/npm/ng-packs/apps/dev-app/src/app/route.provider.ts
@@ -1,4 +1,4 @@
-import { RoutesService, eLayoutType } from '@abp/ng.core';
+import { eLayoutType, RoutesService } from '@abp/ng.core';
 import { APP_INITIALIZER } from '@angular/core';
 
 export const APP_ROUTE_PROVIDER = [
diff --git a/npm/ng-packs/packages/account/config/src/providers/route.provider.ts b/npm/ng-packs/packages/account/config/src/providers/route.provider.ts
index c230d21ad8..9cc8f6fe82 100644
--- a/npm/ng-packs/packages/account/config/src/providers/route.provider.ts
+++ b/npm/ng-packs/packages/account/config/src/providers/route.provider.ts
@@ -10,7 +10,7 @@ export function configureRoutes(routes: RoutesService) {
   return () => {
     routes.add([
       {
-        path: '/account',
+        path: undefined,
         name: eAccountRouteNames.Account,
         invisible: true,
         layout: eLayoutType.application,
diff --git a/npm/ng-packs/packages/core/src/lib/services/routes.service.ts b/npm/ng-packs/packages/core/src/lib/services/routes.service.ts
index c4a35c27c1..02c88b4901 100644
--- a/npm/ng-packs/packages/core/src/lib/services/routes.service.ts
+++ b/npm/ng-packs/packages/core/src/lib/services/routes.service.ts
@@ -135,7 +135,8 @@ export abstract class AbstractTreeService<T extends object> {
 @Injectable()
 export abstract class AbstractNavTreeService<T extends ABP.Nav>
   extends AbstractTreeService<T>
-  implements OnDestroy {
+  implements OnDestroy
+{
   private subscription: Subscription;
   private permissionService: PermissionService;
   readonly id = 'name';
diff --git a/npm/ng-packs/packages/identity/config/src/providers/route.provider.ts b/npm/ng-packs/packages/identity/config/src/providers/route.provider.ts
index f5c2bc2327..26326138e6 100644
--- a/npm/ng-packs/packages/identity/config/src/providers/route.provider.ts
+++ b/npm/ng-packs/packages/identity/config/src/providers/route.provider.ts
@@ -12,7 +12,7 @@ export function configureRoutes(routesService: RoutesService) {
   return () => {
     routesService.add([
       {
-        path: '/identity',
+        path: undefined,
         name: eIdentityRouteNames.IdentityManagement,
         parentName: eThemeSharedRouteNames.Administration,
         requiredPolicy: eIdentityPolicyNames.IdentityManagement,
diff --git a/npm/ng-packs/packages/tenant-management/config/src/providers/route.provider.ts b/npm/ng-packs/packages/tenant-management/config/src/providers/route.provider.ts
index b191b46ce3..5b3d9c88fd 100644
--- a/npm/ng-packs/packages/tenant-management/config/src/providers/route.provider.ts
+++ b/npm/ng-packs/packages/tenant-management/config/src/providers/route.provider.ts
@@ -12,7 +12,7 @@ export function configureRoutes(routes: RoutesService) {
   return () => {
     routes.add([
       {
-        path: '/tenant-management',
+        path: undefined,
         name: eTenantManagementRouteNames.TenantManagement,
         parentName: eThemeSharedRouteNames.Administration,
         requiredPolicy: eTenantManagementPolicyNames.TenantManagement,
diff --git a/npm/ng-packs/packages/theme-basic/src/lib/components/routes/routes.component.html b/npm/ng-packs/packages/theme-basic/src/lib/components/routes/routes.component.html
index 0e2f67156c..aae02df783 100644
--- a/npm/ng-packs/packages/theme-basic/src/lib/components/routes/routes.component.html
+++ b/npm/ng-packs/packages/theme-basic/src/lib/components/routes/routes.component.html
@@ -62,12 +62,14 @@
   </ng-template>
 
   <ng-template #defaultChild let-child>
-    <div class="dropdown-submenu" *abpPermission="child.requiredPolicy">
-      <a class="dropdown-item" [routerLink]="[child.path]" (click)="closeDropdown()">
-        <i *ngIf="child.iconClass" [ngClass]="child.iconClass"></i>
-        {{ child.name | abpLocalization }}</a
-      >
-    </div>
+    <ng-container *ngIf="child.path">
+      <div class="dropdown-submenu" *abpPermission="child.requiredPolicy">
+        <a class="dropdown-item" [routerLink]="[child.path]" (click)="closeDropdown()">
+          <i *ngIf="child.iconClass" [ngClass]="child.iconClass"></i>
+          {{ child.name | abpLocalization }}</a
+        >
+      </div>
+    </ng-container>
   </ng-template>
 
   <ng-template #dropdownChild let-child>