You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datalab.apache.org by yt...@apache.org on 2021/06/04 08:35:52 UTC

[incubator-datalab] 01/01: [DATALAB-2387] fixed bug with installing libraries on Manage libraries pop up

This is an automated email from the ASF dual-hosted git repository.

ytykhun pushed a commit to branch DATALAB-2387
in repository https://gitbox.apache.org/repos/asf/incubator-datalab.git

commit a7003d3e2a32faa2d65048d72818a54a9389d2d2
Author: Yurii Tykhun <ty...@gmail.com>
AuthorDate: Fri Jun 4 11:35:24 2021 +0300

    [DATALAB-2387] fixed bug with installing libraries on Manage libraries pop up
---
 .../install-libraries/install-libraries.component.html            | 8 ++++----
 .../exploratory/install-libraries/install-libraries.component.ts  | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/services/self-service/src/main/resources/webapp/src/app/resources/exploratory/install-libraries/install-libraries.component.html b/services/self-service/src/main/resources/webapp/src/app/resources/exploratory/install-libraries/install-libraries.component.html
index 09eb859..7fe0d22 100644
--- a/services/self-service/src/main/resources/webapp/src/app/resources/exploratory/install-libraries/install-libraries.component.html
+++ b/services/self-service/src/main/resources/webapp/src/app/resources/exploratory/install-libraries/install-libraries.component.html
@@ -90,16 +90,16 @@
                     </div>
                   </mat-option>
                 </ng-template>
-                <mat-option *ngIf="!filteredList?.length && !validity_format && autoComplete === 'ENABLED'">
+                <mat-option *ngIf="!filteredList?.length && !validity_format && autoComplete === 'ENABLED' && lib.name?.length > 0" disabled>
                   <span class="configuring">No matches found</span>
                 </mat-option>
                 <mat-option *ngIf="validity_format?.length > 0 && autoComplete === 'ENABLED'">
                   <span class="configuring" >{{ validity_format }}</span >
                 </mat-option>
-                <mat-option *ngIf="autoComplete === 'NONE' && lib.name?.length > 1">
+                <mat-option *ngIf="autoComplete === 'NONE' && lib.name?.length > 1" disabled>
                   <span class="configuring" >Autocomplete is currently unavailable for {{groupsListMap[group]}} group</span >
                 </mat-option>
-                <mat-option *ngIf="autoComplete === 'UPDATING' && lib.name?.length > 1">
+                <mat-option *ngIf="autoComplete === 'UPDATING' && lib.name?.length > 1" disabled>
                   <span class="configuring" >Library list is being loaded at the moment. Please wait a minute.</span>
                 </mat-option>
               </mat-autocomplete>
@@ -112,7 +112,7 @@
                   class="library-input"
                   [placeholder]="'Enter library version (optional)'"
                   [(ngModel)]="lib.version"
-                  [disabled]="!lib.name.trim() || lib.name.trim().length < 2"
+                  [disabled]="!lib.name?.trim() || lib.name?.trim().length < 2"
                   (keyup)="validateVersion(lib.version)"
                   (keydown.enter)="addLibrary(lib)"
                 >
diff --git a/services/self-service/src/main/resources/webapp/src/app/resources/exploratory/install-libraries/install-libraries.component.ts b/services/self-service/src/main/resources/webapp/src/app/resources/exploratory/install-libraries/install-libraries.component.ts
index b97f583..1b44d99 100644
--- a/services/self-service/src/main/resources/webapp/src/app/resources/exploratory/install-libraries/install-libraries.component.ts
+++ b/services/self-service/src/main/resources/webapp/src/app/resources/exploratory/install-libraries/install-libraries.component.ts
@@ -112,7 +112,7 @@ export class InstallLibrariesComponent implements OnInit, OnDestroy {
       takeUntil(this.unsubscribe$)
       )
       .subscribe(value => {
-        if(!!value.match(/\s+/g)) {
+        if(!!value?.match(/\s+/g)) {
           this.libSearch.setValue(value.replace(/\s+/g, ''))
           this.lib.name = value.replace(/\s+/g, '');
         } else {

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datalab.apache.org
For additional commands, e-mail: commits-help@datalab.apache.org