Merge pull request #3494 from sap-labs-france/master-qa

Version 2.7.5 - hot fixes - ocpi endpoint UI
sap-labs-france · Jan 10, 2023 · d2fd800 · d2fd800
2 parents 47d3c46 + b1b3569
commit d2fd800
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 11 deletions.
diff --git a/.../pages/settings-integration/ocpi/endpoints/endpoint/settings-ocpi-endpoint.component.html b/.../pages/settings-integration/ocpi/endpoints/endpoint/settings-ocpi-endpoint.component.html
@@ -98,7 +98,7 @@
                   </div>
                 </div>
                 <div class="col-md-4 text-end">
-                  <button mat-raised-button color="primary" *ngIf="currentEndpoint.canGenerateLocalToken" (click)="generateLocalToken(formGroup.value)">
+                  <button mat-raised-button color="primary" *ngIf="canGenerateLocalToken" (click)="generateLocalToken(formGroup.value)">
                     <mat-icon>vpn_key</mat-icon>
                     <span>{{'ocpiendpoints.generate_new_token' | translate}}</span>
                   </button>
@@ -117,7 +117,7 @@
                   </div>
                 </div>
                 <div class="col-md-4 text-end">
-                  <button mat-raised-button color="primary" *ngIf="currentEndpoint.canPing" (click)="testConnection(formGroup.value)"
+                  <button mat-raised-button color="primary" *ngIf="canPing" (click)="testConnection(formGroup.value)"
                     [disabled]="!token.value || baseUrl.errors?.required || baseUrl.errors?.pattern">
                     <mat-icon>touch_app</mat-icon>
                     <span>{{'ocpiendpoints.test_connection' | translate}}</span>

diff --git a/...pp/pages/settings-integration/ocpi/endpoints/endpoint/settings-ocpi-endpoint.component.ts b/...pp/pages/settings-integration/ocpi/endpoints/endpoint/settings-ocpi-endpoint.component.ts
@@ -4,14 +4,14 @@ import { MatDialogRef } from '@angular/material/dialog';
 import { Router } from '@angular/router';
 import { TranslateService } from '@ngx-translate/core';
 import { StatusCodes } from 'http-status-codes';
-import { DialogMode } from 'types/Authorization';
-import { OCPIEndpoint } from 'types/ocpi/OCPIEndpoint';
 
 import { CentralServerService } from '../../../../../services/central-server.service';
 import { DialogService } from '../../../../../services/dialog.service';
 import { MessageService } from '../../../../../services/message.service';
 import { SpinnerService } from '../../../../../services/spinner.service';
+import { DialogMode, OcpiEndpointsAuthorizations } from '../../../../../types/Authorization';
 import { RestResponse } from '../../../../../types/GlobalType';
+import { OCPIEndpoint } from '../../../../../types/ocpi/OCPIEndpoint';
 import { Constants } from '../../../../../utils/Constants';
 import { Utils } from '../../../../../utils/Utils';
 
@@ -24,6 +24,7 @@ export class SettingsOcpiEndpointComponent implements OnInit {
   @Input() public currentEndpoint!: OCPIEndpoint;
   @Input() public dialogRef!: MatDialogRef<any>;
   @Input() public dialogMode!: DialogMode;
+  @Input() public authorizations!: OcpiEndpointsAuthorizations;
 
   public formGroup!: FormGroup;
   public id!: AbstractControl;
@@ -37,7 +38,8 @@ export class SettingsOcpiEndpointComponent implements OnInit {
   public token!: AbstractControl;
   public isBackgroundPatchJobActive!: AbstractControl;
   public readOnly = true;
-
+  public canGenerateLocalToken: boolean;
+  public canPing: boolean;
 
   // eslint-disable-next-line no-useless-constructor
   public constructor(
@@ -97,6 +99,8 @@ export class SettingsOcpiEndpointComponent implements OnInit {
     this.localToken = this.formGroup.controls['localToken'];
     this.token = this.formGroup.controls['token'];
     this.isBackgroundPatchJobActive = this.formGroup.controls['backgroundPatchJob'];
+    this.canGenerateLocalToken = Utils.convertToBoolean(this.authorizations.canGenerateLocalToken);
+    this.canPing = Utils.convertToBoolean(this.authorizations.canPing);
     // Handle Dialog mode
     this.readOnly = this.dialogMode === DialogMode.VIEW;
     Utils.handleDialogMode(this.dialogMode, this.formGroup);

diff --git a/...s/settings-integration/ocpi/endpoints/endpoint/settings-ocpi-endpoint.dialog.component.ts b/...s/settings-integration/ocpi/endpoints/endpoint/settings-ocpi-endpoint.dialog.component.ts
@@ -1,25 +1,26 @@
 import { AfterViewInit, Component, Inject, ViewChild } from '@angular/core';
 import { MAT_DIALOG_DATA, MatDialogRef } from '@angular/material/dialog';
-import { DialogMode, DialogParams } from 'types/Authorization';
 
+import { DialogMode, DialogParamsWithAuth, OcpiEndpointsAuthorizations } from '../../../../../types/Authorization';
 import { OCPIEndpoint } from '../../../../../types/ocpi/OCPIEndpoint';
 import { Utils } from '../../../../../utils/Utils';
 import { SettingsOcpiEndpointComponent } from './settings-ocpi-endpoint.component';
 
 @Component({
-  template: '<app-ocpi-endpoint #appRef [currentEndpoint]="currentEndpoint" [dialogMode]="dialogMode" [dialogRef]="dialogRef"></app-ocpi-endpoint>',
+  template: '<app-ocpi-endpoint #appRef [currentEndpoint]="currentEndpoint" [dialogMode]="dialogMode" [dialogRef]="dialogRef" [authorizations]="authorizations"></app-ocpi-endpoint>',
 })
 export class SettingsOcpiEndpointDialogComponent implements AfterViewInit {
   @ViewChild('appRef') public appRef!: SettingsOcpiEndpointComponent;
   public currentEndpoint!: OCPIEndpoint;
   public dialogMode!: DialogMode;
-
+  public authorizations!: OcpiEndpointsAuthorizations;
 
   public constructor(
     public dialogRef: MatDialogRef<SettingsOcpiEndpointDialogComponent>,
-    @Inject(MAT_DIALOG_DATA) dialogParams: DialogParams<OCPIEndpoint>) {
+    @Inject(MAT_DIALOG_DATA) dialogParams: DialogParamsWithAuth<OCPIEndpoint, OcpiEndpointsAuthorizations>) {
     this.currentEndpoint = dialogParams.dialogData;
     this.dialogMode = dialogParams.dialogMode;
+    this.authorizations = dialogParams.authorizations;
   }
 
   public ngAfterViewInit() {

diff --git a/...pp/pages/settings-integration/ocpi/endpoints/settings-ocpi-endpoints-table-data-source.ts b/...pp/pages/settings-integration/ocpi/endpoints/settings-ocpi-endpoints-table-data-source.ts
@@ -61,7 +61,9 @@ export class SettingsOcpiEndpointsTableDataSource extends TableDataSource<OCPIEn
         next: (ocpiEndpoints) => {
           // Init auth
           this.authorizations = {
-            canCreate: Utils.convertToBoolean(ocpiEndpoints.canCreate)
+            canCreate: Utils.convertToBoolean(ocpiEndpoints.canCreate),
+            canGenerateLocalToken: Utils.convertToBoolean(ocpiEndpoints.canGenerateLocalToken),
+            canPing: Utils.convertToBoolean(ocpiEndpoints.canPing),
           };
           // Set visibility
           this.createAction.visible = this.authorizations.canCreate;
@@ -241,7 +243,10 @@ export class SettingsOcpiEndpointsTableDataSource extends TableDataSource<OCPIEn
     const dialogConfig = new MatDialogConfig();
     dialogConfig.minWidth = '50vw';
     dialogConfig.panelClass = 'transparent-dialog-container';
-    dialogConfig.data = { dialogData: endpoint, dialogMode };
+    dialogConfig.data = {
+      dialogData: endpoint,
+      dialogMode,
+      authorizations: this.authorizations };
     // disable outside click close
     dialogConfig.disableClose = true;
     // Open

diff --git a/src/app/types/Authorization.ts b/src/app/types/Authorization.ts
@@ -454,6 +454,8 @@ export interface OcpiEndpointsAuthorizations extends AuthorizationAttributes, Oc
 }
 
 export interface OcpiEndpointsAuthorizationActions extends DataResultAuthorizationActions {
+  canPing?: boolean;
+  canGenerateLocalToken?: boolean;
 }
 
 export interface OcpiEndpointAuthorizationActions extends AuthorizationActions {