From d73f8a6e9bf3956a9b1710ef60708e35c2bd20db Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Mon, 11 Nov 2019 14:11:42 +0300
Subject: [PATCH 1/9] chore: update npm versions

---
 npm/ng-packs/package.json                     | 26 +++++++++----------
 npm/ng-packs/packages/core/package.json       | 16 ++++++------
 .../packages/theme-shared/package.json        |  4 +--
 3 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/npm/ng-packs/package.json b/npm/ng-packs/package.json
index 84dec91088..6b028ddbbd 100644
--- a/npm/ng-packs/package.json
+++ b/npm/ng-packs/package.json
@@ -19,20 +19,20 @@
     "@abp/ng.tenant-management.config": "^1.0.2",
     "@abp/ng.theme.shared": "^1.0.2",
     "@angular-builders/jest": "^8.2.0",
-    "@angular-devkit/build-angular": "~0.803.15",
-    "@angular-devkit/build-ng-packagr": "~0.803.15",
-    "@angular/animations": "~8.2.12",
+    "@angular-devkit/build-angular": "~0.803.18",
+    "@angular-devkit/build-ng-packagr": "~0.803.18",
+    "@angular/animations": "~8.2.13",
     "@angular/cdk": "^8.2.3",
-    "@angular/cli": "~8.3.15",
-    "@angular/common": "~8.2.12",
-    "@angular/compiler": "~8.2.12",
-    "@angular/compiler-cli": "~8.2.12",
-    "@angular/core": "~8.2.12",
-    "@angular/forms": "~8.2.12",
-    "@angular/language-service": "~8.2.12",
-    "@angular/platform-browser": "~8.2.12",
-    "@angular/platform-browser-dynamic": "~8.2.12",
-    "@angular/router": "~8.2.12",
+    "@angular/cli": "~8.3.18",
+    "@angular/common": "~8.2.13",
+    "@angular/compiler": "~8.2.13",
+    "@angular/compiler-cli": "~8.2.13",
+    "@angular/core": "~8.2.13",
+    "@angular/forms": "~8.2.13",
+    "@angular/language-service": "~8.2.13",
+    "@angular/platform-browser": "~8.2.13",
+    "@angular/platform-browser-dynamic": "~8.2.13",
+    "@angular/router": "~8.2.13",
     "@fortawesome/fontawesome-free": "^5.11.2",
     "@ng-bootstrap/ng-bootstrap": "^5.1.0",
     "@ngneat/spectator": "^4.5.0",
diff --git a/npm/ng-packs/packages/core/package.json b/npm/ng-packs/packages/core/package.json
index 35ce648ffb..c1f4bd5a68 100644
--- a/npm/ng-packs/packages/core/package.json
+++ b/npm/ng-packs/packages/core/package.json
@@ -2,19 +2,19 @@
   "name": "@abp/ng.core",
   "version": "1.0.2",
   "dependencies": {
-    "@ngxs/router-plugin": "^3.5.0",
-    "@ngxs/storage-plugin": "^3.5.0",
-    "@ngxs/store": "^3.5.0",
-    "angular-oauth2-oidc": "^8.0.1",
+    "@ngxs/router-plugin": "^3.5.1",
+    "@ngxs/storage-plugin": "^3.5.1",
+    "@ngxs/store": "^3.5.1",
+    "angular-oauth2-oidc": "^8.0.4",
     "just-compare": "^1.3.0",
     "just-clone": "3.1.0",
     "snq": "^1.0.3"
   },
   "peerDependencies": {
-    "@angular/common": "~8.1.2",
-    "@angular/core": "~8.1.2",
-    "@angular/forms": "~8.1.2",
-    "@angular/router": "~8.1.2",
+    "@angular/common": ">=8.0.0 <9.0.0",
+    "@angular/core": ">=8.0.0 <9.0.0",
+    "@angular/forms": ">=8.0.0 <9.0.0",
+    "@angular/router": ">=8.0.0 <9.0.0",
     "rxjs": "~6.4.0"
   },
   "publishConfig": {
diff --git a/npm/ng-packs/packages/theme-shared/package.json b/npm/ng-packs/packages/theme-shared/package.json
index 7ffd4ea089..7a3bc3d6f0 100644
--- a/npm/ng-packs/packages/theme-shared/package.json
+++ b/npm/ng-packs/packages/theme-shared/package.json
@@ -5,10 +5,10 @@
     "@abp/ng.core": "^1.0.2",
     "@angular/cdk": "^8.2.3",
     "@fortawesome/fontawesome-free": "^5.11.2",
-    "@ng-bootstrap/ng-bootstrap": "^5.1.2",
+    "@ng-bootstrap/ng-bootstrap": "^5.1.4",
     "@ngx-validate/core": "^0.0.7",
     "bootstrap": "^4.3.1",
-    "chart.js": "^2.8.0",
+    "chart.js": "^2.9.2",
     "font-awesome": "^4.7.0",
     "ngx-perfect-scrollbar": "^8.0.0",
     "primeicons": "^2.0.0",

From d22e6b9b7b717e8aa215b9c7cbdb9c54b979e4c8 Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Mon, 11 Nov 2019 14:17:19 +0300
Subject: [PATCH 2/9] refactor(identity): roles save button

---
 .../src/lib/components/roles/roles.component.html      |  6 ++++--
 .../src/lib/components/roles/roles.component.ts        | 10 +++++++---
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.html b/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.html
index b5a3b42faf..d35a6c6541 100644
--- a/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.html
+++ b/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.html
@@ -106,7 +106,7 @@
   </ng-template>
 
   <ng-template #abpBody>
-    <form [formGroup]="form" (ngSubmit)="save()">
+    <form #formRef [formGroup]="form" (ngSubmit)="save()" validateOnSubmit>
       <div class="form-group">
         <label for="role-name">{{ 'AbpIdentity::RoleName' | abpLocalization }}</label
         ><span> * </span>
@@ -133,7 +133,9 @@
     <button type="button" class="btn btn-secondary" #abpClose>
       {{ 'AbpIdentity::Cancel' | abpLocalization }}
     </button>
-    <abp-button iconClass="fa fa-check" (click)="save()">{{ 'AbpIdentity::Save' | abpLocalization }}</abp-button>
+    <abp-button iconClass="fa fa-check" (click)="onClickSaveButton()">{{
+      'AbpIdentity::Save' | abpLocalization
+    }}</abp-button>
   </ng-template>
 </abp-modal>
 
diff --git a/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.ts b/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.ts
index 9973a1ebc0..4e3193ca25 100644
--- a/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.ts
+++ b/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.ts
@@ -1,6 +1,6 @@
 import { ABP } from '@abp/ng.core';
 import { ConfirmationService, Toaster } from '@abp/ng.theme.shared';
-import { Component, TemplateRef, ViewChild, OnInit } from '@angular/core';
+import { Component, TemplateRef, ViewChild, OnInit, ContentChild, ElementRef } from '@angular/core';
 import { FormBuilder, FormGroup, Validators, FormControl } from '@angular/forms';
 import { Select, Store } from '@ngxs/store';
 import { Observable } from 'rxjs';
@@ -40,8 +40,8 @@ export class RolesComponent implements OnInit {
 
   sortKey = '';
 
-  @ViewChild('modalContent', { static: false })
-  modalContent: TemplateRef<any>;
+  @ViewChild('formRef', { static: false, read: ElementRef })
+  formRef: ElementRef<HTMLFormElement>;
 
   constructor(private confirmationService: ConfirmationService, private fb: FormBuilder, private store: Store) {}
 
@@ -122,4 +122,8 @@ export class RolesComponent implements OnInit {
       .pipe(finalize(() => (this.loading = false)))
       .subscribe();
   }
+
+  onClickSaveButton() {
+    this.formRef.nativeElement.dispatchEvent(new Event('submit', { bubbles: true, cancelable: true }));
+  }
 }

From 3424730ca69df1162f35f6fbdc2dc20cf959539b Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Mon, 11 Nov 2019 16:03:46 +0300
Subject: [PATCH 3/9] refactor(identity): add disabled input to abp-button

---
 .../identity/src/lib/components/roles/roles.component.html      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.html b/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.html
index d35a6c6541..fa0ee93d1a 100644
--- a/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.html
+++ b/npm/ng-packs/packages/identity/src/lib/components/roles/roles.component.html
@@ -133,7 +133,7 @@
     <button type="button" class="btn btn-secondary" #abpClose>
       {{ 'AbpIdentity::Cancel' | abpLocalization }}
     </button>
-    <abp-button iconClass="fa fa-check" (click)="onClickSaveButton()">{{
+    <abp-button iconClass="fa fa-check" [disabled]="form?.invalid" (click)="onClickSaveButton()">{{
       'AbpIdentity::Save' | abpLocalization
     }}</abp-button>
   </ng-template>

From 0a5fab2816fdaf1eab1d7cf2bb721d58977dec64 Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Mon, 11 Nov 2019 16:06:37 +0300
Subject: [PATCH 4/9] refactor(tenant-management): add disabled input to
 abp-button add autofocus feature to defaultConnectionString input

---
 .../components/tenants/tenants.component.html |  9 ++++--
 .../components/tenants/tenants.component.ts   | 29 +++++++++++++++++--
 2 files changed, 32 insertions(+), 6 deletions(-)

diff --git a/npm/ng-packs/packages/tenant-management/src/lib/components/tenants/tenants.component.html b/npm/ng-packs/packages/tenant-management/src/lib/components/tenants/tenants.component.html
index ea59b87a78..4396f48d89 100644
--- a/npm/ng-packs/packages/tenant-management/src/lib/components/tenants/tenants.component.html
+++ b/npm/ng-packs/packages/tenant-management/src/lib/components/tenants/tenants.component.html
@@ -129,12 +129,14 @@
     <button #abpClose type="button" class="btn btn-secondary">
       {{ 'AbpTenantManagement::Cancel' | abpLocalization }}
     </button>
-    <abp-button iconClass="fa fa-check" (click)="save()">{{ 'AbpIdentity::Save' | abpLocalization }}</abp-button>
+    <abp-button iconClass="fa fa-check" (click)="save()" [disabled]="isDisabledSaveButton">{{
+      'AbpIdentity::Save' | abpLocalization
+    }}</abp-button>
   </ng-template>
 </abp-modal>
 
 <ng-template #tenantModalTemplate>
-  <form [formGroup]="tenantForm" (ngSubmit)="save()">
+  <form [formGroup]="tenantForm" (ngSubmit)="save()" validateOnSubmit>
     <div class="mt-2">
       <div class="form-group">
         <label for="name">{{ 'AbpTenantManagement::TenantName' | abpLocalization }}</label>
@@ -145,7 +147,7 @@
 </ng-template>
 
 <ng-template #connectionStringModalTemplate>
-  <form [formGroup]="defaultConnectionStringForm" (ngSubmit)="save()">
+  <form [formGroup]="defaultConnectionStringForm" (ngSubmit)="save()" validateOnSubmit>
     <label class="mt-2">
       <div class="form-group">
         <div class="custom-checkbox custom-control mb-2">
@@ -155,6 +157,7 @@
             class="custom-control-input"
             formControlName="useSharedDatabase"
             autofocus
+            (ngModelChange)="onSharedDatabaseChange($event)"
           />
           <label for="useSharedDatabase" class="custom-control-label">{{
             'AbpTenantManagement::DisplayName:UseSharedDatabase' | abpLocalization
diff --git a/npm/ng-packs/packages/tenant-management/src/lib/components/tenants/tenants.component.ts b/npm/ng-packs/packages/tenant-management/src/lib/components/tenants/tenants.component.ts
index f1a5615cef..18387ea784 100644
--- a/npm/ng-packs/packages/tenant-management/src/lib/components/tenants/tenants.component.ts
+++ b/npm/ng-packs/packages/tenant-management/src/lib/components/tenants/tenants.component.ts
@@ -1,6 +1,6 @@
 import { ABP } from '@abp/ng.core';
 import { ConfirmationService, Toaster } from '@abp/ng.theme.shared';
-import { Component, OnInit, TemplateRef, ViewChild } from '@angular/core';
+import { Component, OnInit, TemplateRef, ViewChild, ChangeDetectorRef } from '@angular/core';
 import { FormBuilder, FormGroup, Validators } from '@angular/forms';
 import { Select, Store } from '@ngxs/store';
 import { Observable } from 'rxjs';
@@ -16,7 +16,7 @@ import { TenantManagementService } from '../../services/tenant-management.servic
 import { TenantManagementState } from '../../states/tenant-management.state';
 
 interface SelectedModalContent {
-  type: string;
+  type: 'saveConnStr' | 'saveTenant';
   title: string;
   template: TemplateRef<any>;
 }
@@ -74,6 +74,18 @@ export class TenantsComponent implements OnInit {
   @ViewChild('connectionStringModalTemplate', { static: false })
   connectionStringModalTemplate: TemplateRef<any>;
 
+  get isDisabledSaveButton(): boolean {
+    if (!this.selectedModalContent) return false;
+
+    if (this.selectedModalContent.type === 'saveConnStr' && this.defaultConnectionStringForm.invalid) {
+      return true;
+    } else if (this.selectedModalContent.type === 'saveTenant' && this.tenantForm.invalid) {
+      return true;
+    } else {
+      return false;
+    }
+  }
+
   constructor(
     private confirmationService: ConfirmationService,
     private tenantService: TenantManagementService,
@@ -103,7 +115,7 @@ export class TenantsComponent implements OnInit {
     });
   }
 
-  openModal(title: string, template: TemplateRef<any>, type: string) {
+  openModal(title: string, template: TemplateRef<any>, type: 'saveConnStr' | 'saveTenant') {
     this.selectedModalContent = {
       title,
       template,
@@ -224,4 +236,15 @@ export class TenantsComponent implements OnInit {
       .pipe(finalize(() => (this.loading = false)))
       .subscribe();
   }
+
+  onSharedDatabaseChange(value: boolean) {
+    if (!value) {
+      setTimeout(() => {
+        const defaultConnectionString = document.getElementById('defaultConnectionString') as HTMLInputElement;
+        if (defaultConnectionString) {
+          defaultConnectionString.focus();
+        }
+      }, 0);
+    }
+  }
 }

From e7c4a38ab536e09fd9a616d680acde3ed08dd0c6 Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Mon, 11 Nov 2019 16:25:28 +0300
Subject: [PATCH 5/9] refactor(feature-management): add disabled input to
 button

---
 .../feature-management/feature-management.component.html      | 4 ++--
 .../feature-management/feature-management.component.ts        | 2 ++
 .../src/lib/states/feature-management.state.ts                | 2 +-
 3 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/npm/ng-packs/packages/feature-management/src/lib/components/feature-management/feature-management.component.html b/npm/ng-packs/packages/feature-management/src/lib/components/feature-management/feature-management.component.html
index 2dad2dfd92..44ff3f382f 100644
--- a/npm/ng-packs/packages/feature-management/src/lib/components/feature-management/feature-management.component.html
+++ b/npm/ng-packs/packages/feature-management/src/lib/components/feature-management/feature-management.component.html
@@ -4,7 +4,7 @@
   </ng-template>
 
   <ng-template #abpBody>
-    <form *ngIf="form" (ngSubmit)="save()" [formGroup]="form">
+    <form *ngIf="form" (ngSubmit)="save()" [formGroup]="form" validateOnSubmit>
       <div
         class="row my-3"
         *ngFor="let feature of features$ | async; let i = index"
@@ -25,7 +25,7 @@
     <button #abpClose type="button" class="btn btn-secondary">
       {{ 'AbpFeatureManagement::Cancel' | abpLocalization }}
     </button>
-    <abp-button iconClass="fa fa-check" (click)="save()">
+    <abp-button iconClass="fa fa-check" [disabled]="form?.invalid || modalBusy" (click)="save()">
       {{ 'AbpFeatureManagement::Save' | abpLocalization }}
     </abp-button>
   </ng-template>
diff --git a/npm/ng-packs/packages/feature-management/src/lib/components/feature-management/feature-management.component.ts b/npm/ng-packs/packages/feature-management/src/lib/components/feature-management/feature-management.component.ts
index a25d6b41ee..3a93380d39 100644
--- a/npm/ng-packs/packages/feature-management/src/lib/components/feature-management/feature-management.component.ts
+++ b/npm/ng-packs/packages/feature-management/src/lib/components/feature-management/feature-management.component.ts
@@ -76,6 +76,8 @@ export class FeatureManagementComponent {
   }
 
   save() {
+    if (this.modalBusy) return;
+
     this.modalBusy = true;
 
     let features = this.store.selectSnapshot(FeatureManagementState.getFeatures);
diff --git a/npm/ng-packs/packages/feature-management/src/lib/states/feature-management.state.ts b/npm/ng-packs/packages/feature-management/src/lib/states/feature-management.state.ts
index a02e075bdb..34dd78b0fb 100644
--- a/npm/ng-packs/packages/feature-management/src/lib/states/feature-management.state.ts
+++ b/npm/ng-packs/packages/feature-management/src/lib/states/feature-management.state.ts
@@ -11,7 +11,7 @@ import { FeatureManagementService } from '../services/feature-management.service
 export class FeatureManagementState {
   @Selector()
   static getFeatures({ features }: FeatureManagement.State) {
-    return features;
+    return features || [];
   }
 
   constructor(private featureManagementService: FeatureManagementService) {}

From d7207eb5782a729cd66f720aac0ca597229755d4 Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Mon, 11 Nov 2019 17:27:14 +0300
Subject: [PATCH 6/9] refactor(account): add inprogress variable to tenant-box
 add validateOnSubmit directive to forms

---
 .../lib/components/login/login.component.html |  2 +-
 .../register/register.component.html          |  2 +-
 .../tenant-box/tenant-box.component.html      |  6 +--
 .../tenant-box/tenant-box.component.ts        | 43 ++++++++-----------
 4 files changed, 22 insertions(+), 31 deletions(-)

diff --git a/npm/ng-packs/packages/account/src/lib/components/login/login.component.html b/npm/ng-packs/packages/account/src/lib/components/login/login.component.html
index 6f0902cd03..130b013d55 100644
--- a/npm/ng-packs/packages/account/src/lib/components/login/login.component.html
+++ b/npm/ng-packs/packages/account/src/lib/components/login/login.component.html
@@ -5,7 +5,7 @@
       {{ 'AbpAccount::AreYouANewUser' | abpLocalization }}
       <a class="text-decoration-none" routerLink="/account/register">{{ 'AbpAccount::Register' | abpLocalization }}</a>
     </strong>
-    <form [formGroup]="form" (ngSubmit)="onSubmit()" novalidate class="mt-4">
+    <form [formGroup]="form" (ngSubmit)="onSubmit()" validateOnSubmit class="mt-4">
       <div class="form-group">
         <label for="login-input-user-name-or-email-address">{{
           'AbpAccount::UserNameOrEmailAddress' | abpLocalization
diff --git a/npm/ng-packs/packages/account/src/lib/components/register/register.component.html b/npm/ng-packs/packages/account/src/lib/components/register/register.component.html
index fee24465c7..41d619a16a 100644
--- a/npm/ng-packs/packages/account/src/lib/components/register/register.component.html
+++ b/npm/ng-packs/packages/account/src/lib/components/register/register.component.html
@@ -5,7 +5,7 @@
       {{ 'AbpAccount::AlreadyRegistered' | abpLocalization }}
       <a class="text-decoration-none" routerLink="/account/login">{{ 'AbpAccount::Login' | abpLocalization }}</a>
     </strong>
-    <form [formGroup]="form" (ngSubmit)="onSubmit()" novalidate class="mt-4">
+    <form [formGroup]="form" (ngSubmit)="onSubmit()" validateOnSubmit class="mt-4">
       <div class="form-group">
         <label for="input-user-name">{{ 'AbpAccount::UserName' | abpLocalization }}</label
         ><span> * </span
diff --git a/npm/ng-packs/packages/account/src/lib/components/tenant-box/tenant-box.component.html b/npm/ng-packs/packages/account/src/lib/components/tenant-box/tenant-box.component.html
index 86730983f2..007f45f5a9 100644
--- a/npm/ng-packs/packages/account/src/lib/components/tenant-box/tenant-box.component.html
+++ b/npm/ng-packs/packages/account/src/lib/components/tenant-box/tenant-box.component.html
@@ -25,7 +25,7 @@
   </div>
 </div>
 
-<abp-modal [(visible)]="isModalVisible" size="md">
+<abp-modal size="md" [(visible)]="isModalVisible" [busy]="inProgress">
   <ng-template #abpHeader>
     <h5>Switch Tenant</h5>
   </ng-template>
@@ -44,8 +44,8 @@
     <button #abpClose type="button" class="btn btn-secondary">
       {{ 'AbpTenantManagement::Cancel' | abpLocalization }}
     </button>
-    <button type="button" class="btn btn-primary" (click)="save()">
+    <abp-button buttonType="button" buttonClass="btn btn-primary" (click)="save()">
       <i class="fa fa-check mr-1"></i> <span>{{ 'AbpTenantManagement::Save' | abpLocalization }}</span>
-    </button>
+    </abp-button>
   </ng-template>
 </abp-modal>
diff --git a/npm/ng-packs/packages/account/src/lib/components/tenant-box/tenant-box.component.ts b/npm/ng-packs/packages/account/src/lib/components/tenant-box/tenant-box.component.ts
index c114095adc..9300b1d248 100644
--- a/npm/ng-packs/packages/account/src/lib/components/tenant-box/tenant-box.component.ts
+++ b/npm/ng-packs/packages/account/src/lib/components/tenant-box/tenant-box.component.ts
@@ -3,31 +3,27 @@ import { ToasterService } from '@abp/ng.theme.shared';
 import { Component, OnInit } from '@angular/core';
 import { Store } from '@ngxs/store';
 import { throwError } from 'rxjs';
-import { catchError, take } from 'rxjs/operators';
+import { catchError, take, finalize } from 'rxjs/operators';
 import snq from 'snq';
 import { AccountService } from '../../services/account.service';
 
 @Component({
   selector: 'abp-tenant-box',
-  templateUrl: './tenant-box.component.html'
+  templateUrl: './tenant-box.component.html',
 })
 export class TenantBoxComponent implements OnInit {
-  constructor(
-    private store: Store,
-    private toasterService: ToasterService,
-    private accountService: AccountService
-  ) {}
-
   tenant = {} as ABP.BasicItem;
 
   tenantName: string;
 
   isModalVisible: boolean;
 
+  inProgress: boolean;
+
+  constructor(private store: Store, private toasterService: ToasterService, private accountService: AccountService) {}
+
   ngOnInit() {
-    this.tenant =
-      this.store.selectSnapshot(SessionState.getTenant) ||
-      ({} as ABP.BasicItem);
+    this.tenant = this.store.selectSnapshot(SessionState.getTenant) || ({} as ABP.BasicItem);
     this.tenantName = this.tenant.name || '';
   }
 
@@ -36,38 +32,33 @@ export class TenantBoxComponent implements OnInit {
   }
 
   save() {
-    if (this.tenant.name) {
+    if (this.tenant.name && !this.inProgress) {
+      this.inProgress = true;
       this.accountService
         .findTenant(this.tenant.name)
         .pipe(
+          finalize(() => (this.inProgress = false)),
           take(1),
           catchError(err => {
             this.toasterService.error(
-              snq(
-                () => err.error.error_description,
-                'AbpUi::DefaultErrorMessage'
-              ),
-              'AbpUi::Error'
+              snq(() => err.error.error_description, 'AbpUi::DefaultErrorMessage'),
+              'AbpUi::Error',
             );
             return throwError(err);
-          })
+          }),
         )
         .subscribe(({ success, tenantId }) => {
           if (success) {
             this.tenant = {
               id: tenantId,
-              name: this.tenant.name
+              name: this.tenant.name,
             };
             this.tenantName = this.tenant.name;
             this.isModalVisible = false;
           } else {
-            this.toasterService.error(
-              'AbpUiMultiTenancy::GivenTenantIsNotAvailable',
-              'AbpUi::Error',
-              {
-                messageLocalizationParams: [this.tenant.name]
-              }
-            );
+            this.toasterService.error('AbpUiMultiTenancy::GivenTenantIsNotAvailable', 'AbpUi::Error', {
+              messageLocalizationParams: [this.tenant.name],
+            });
             this.tenant = {} as ABP.BasicItem;
           }
           this.store.dispatch(new SetTenant(success ? this.tenant : null));

From c7c1efe378cd345a40f6af894c58c648fc7618e8 Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Mon, 11 Nov 2019 17:28:20 +0300
Subject: [PATCH 7/9] feat(theme-shared): add id input to button.component

---
 .../src/lib/components/button/button.component.ts             | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/npm/ng-packs/packages/theme-shared/src/lib/components/button/button.component.ts b/npm/ng-packs/packages/theme-shared/src/lib/components/button/button.component.ts
index 610b22bc5d..131eea86bd 100644
--- a/npm/ng-packs/packages/theme-shared/src/lib/components/button/button.component.ts
+++ b/npm/ng-packs/packages/theme-shared/src/lib/components/button/button.component.ts
@@ -7,6 +7,7 @@ import { ABP } from '@abp/ng.core';
   template: `
     <button
       #button
+      [id]="buttonId"
       [attr.type]="buttonType"
       [ngClass]="buttonClass"
       [disabled]="loading || disabled"
@@ -19,6 +20,9 @@ import { ABP } from '@abp/ng.core';
   `,
 })
 export class ButtonComponent implements OnInit {
+  @Input()
+  buttonId = '';
+
   @Input()
   buttonClass = 'btn btn-primary';
 

From ba9ab6fe2982475c3f29b06023c1ff80e5ea8d6d Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Mon, 11 Nov 2019 17:29:30 +0300
Subject: [PATCH 8/9] refactor(core): remove markAsDirty loop

---
 .../lib/directives/form-submit.directive.ts   | 19 ++++---------------
 1 file changed, 4 insertions(+), 15 deletions(-)

diff --git a/npm/ng-packs/packages/core/src/lib/directives/form-submit.directive.ts b/npm/ng-packs/packages/core/src/lib/directives/form-submit.directive.ts
index 00982bbbb9..5d0c6ef7a1 100644
--- a/npm/ng-packs/packages/core/src/lib/directives/form-submit.directive.ts
+++ b/npm/ng-packs/packages/core/src/lib/directives/form-submit.directive.ts
@@ -7,7 +7,7 @@ import {
   OnDestroy,
   OnInit,
   Output,
-  Self
+  Self,
 } from '@angular/core';
 import { FormControl, FormGroup, FormGroupDirective } from '@angular/forms';
 import { fromEvent } from 'rxjs';
@@ -18,7 +18,7 @@ type Controls = { [key: string]: FormControl } | FormGroup[];
 
 @Directive({
   // tslint:disable-next-line: directive-selector
-  selector: 'form[ngSubmit][formGroup]'
+  selector: 'form[ngSubmit][formGroup]',
 })
 export class FormSubmitDirective implements OnInit, OnDestroy {
   @Input()
@@ -31,7 +31,7 @@ export class FormSubmitDirective implements OnInit, OnDestroy {
   constructor(
     @Self() private formGroupDirective: FormGroupDirective,
     private host: ElementRef<HTMLFormElement>,
-    private cdRef: ChangeDetectorRef
+    private cdRef: ChangeDetectorRef,
   ) {}
 
   ngOnInit() {
@@ -44,7 +44,7 @@ export class FormSubmitDirective implements OnInit, OnDestroy {
       .pipe(
         debounceTime(200),
         filter((key: KeyboardEvent) => key && key.key === 'Enter'),
-        takeUntilDestroy(this)
+        takeUntilDestroy(this),
       )
       .subscribe(() => {
         if (!this.executedNgSubmit) {
@@ -53,17 +53,6 @@ export class FormSubmitDirective implements OnInit, OnDestroy {
 
         this.executedNgSubmit = false;
       });
-
-    fromEvent(this.host.nativeElement, 'submit')
-      .pipe(
-        takeUntilDestroy(this),
-        filter(() => !this.notValidateOnSubmit && typeof this.notValidateOnSubmit !== 'string')
-      )
-      .subscribe(() => {
-        if (!this.executedNgSubmit) {
-          this.markAsDirty();
-        }
-      });
   }
 
   ngOnDestroy(): void {}

From c84bff65129a3823681ddaccf0750bf376cc3ffe Mon Sep 17 00:00:00 2001
From: mehmet-erim <bilgi@mehmeterim.com>
Date: Mon, 11 Nov 2019 17:30:16 +0300
Subject: [PATCH 9/9] refactor(account): add disabled to
 personal-settings.component

---
 .../components/change-password/change-password.component.html  | 3 ++-
 .../personal-settings/personal-settings.component.html         | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/npm/ng-packs/packages/account/src/lib/components/change-password/change-password.component.html b/npm/ng-packs/packages/account/src/lib/components/change-password/change-password.component.html
index 5f70e699ba..15b7ca7dd4 100644
--- a/npm/ng-packs/packages/account/src/lib/components/change-password/change-password.component.html
+++ b/npm/ng-packs/packages/account/src/lib/components/change-password/change-password.component.html
@@ -1,4 +1,4 @@
-<form [formGroup]="form" (ngSubmit)="onSubmit()" [mapErrorsFn]="mapErrorsFn">
+<form [formGroup]="form" (ngSubmit)="onSubmit()" [mapErrorsFn]="mapErrorsFn" validateOnSubmit>
   <div class="form-group">
     <label for="current-password">{{ 'AbpIdentity::DisplayName:CurrentPassword' | abpLocalization }}</label
     ><span> * </span
@@ -18,6 +18,7 @@
     buttonClass="btn btn-primary color-white"
     buttonType="submit"
     [loading]="inProgress"
+    [disabled]="form?.invalid"
     >{{ 'AbpIdentity::Save' | abpLocalization }}</abp-button
   >
 </form>
diff --git a/npm/ng-packs/packages/account/src/lib/components/personal-settings/personal-settings.component.html b/npm/ng-packs/packages/account/src/lib/components/personal-settings/personal-settings.component.html
index 38c62301af..5e7b292703 100644
--- a/npm/ng-packs/packages/account/src/lib/components/personal-settings/personal-settings.component.html
+++ b/npm/ng-packs/packages/account/src/lib/components/personal-settings/personal-settings.component.html
@@ -1,4 +1,4 @@
-<form novalidate *ngIf="form" [formGroup]="form" (ngSubmit)="submit()">
+<form validateOnSubmit *ngIf="form" [formGroup]="form" (ngSubmit)="submit()">
   <div class="form-group">
     <label for="username">{{ 'AbpIdentity::DisplayName:UserName' | abpLocalization }}</label
     ><span> * </span
@@ -38,6 +38,7 @@
     iconClass="fa fa-check"
     buttonClass="btn btn-primary color-white"
     [loading]="inProgress"
+    [disabled]="form?.invalid || form?.pristine"
   >
     {{ 'AbpIdentity::Save' | abpLocalization }}</abp-button
   >