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

akuznetsov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite.git


The following commit(s) were added to refs/heads/master by this push:
     new 6fd5a76  IGNITE-11711 Web console: Fixed visibility logic for "JDBC 
driver download link".
6fd5a76 is described below

commit 6fd5a7622e3ad5e8c0b02a302b7a7a5d0d4dd007
Author: Vasiliy Sisko <[email protected]>
AuthorDate: Fri Apr 12 13:34:42 2019 +0700

    IGNITE-11711 Web console: Fixed visibility logic for "JDBC driver download 
link".
---
 .../components/cache-edit-form/templates/store.pug                | 4 ++--
 .../components/cluster-edit-form/templates/checkpoint/jdbc.pug    | 2 +-
 .../cluster-edit-form/templates/general/discovery/jdbc.pug        | 2 +-
 modules/web-console/frontend/app/configuration/services/Caches.ts | 8 ++++----
 .../web-console/frontend/app/configuration/services/Clusters.ts   | 8 ++++----
 5 files changed, 12 insertions(+), 12 deletions(-)

diff --git 
a/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cache-edit-form/templates/store.pug
 
b/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cache-edit-form/templates/store.pug
index b4baac5..42153d8 100644
--- 
a/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cache-edit-form/templates/store.pug
+++ 
b/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cache-edit-form/templates/store.pug
@@ -86,7 +86,7 @@ panel-collapsible(ng-form=form 
on-open=`ui.loadPanel('${form}')`)
                             
change:`$ctrl.clearImplementationVersion(${pojoStoreFactory})`
                         })
                     
.pc-form-grid-col-60(ng-if=`$ctrl.Caches.requiresProprietaryDrivers(${pojoStoreFactory})`)
-                        a.link-success(ng-href=`{{ 
$ctrl.Caches.JDBCDriverURL(${pojoStoreFactory}) }}` target='_blank')
+                        a.link-success(ng-href=`{{ 
$ctrl.Caches.jdbcDriverURL(${pojoStoreFactory}) }}` target='_blank')
                             | Download JDBC drivers?
                     .pc-form-grid-col-30
                         +form-field__number({
@@ -223,7 +223,7 @@ panel-collapsible(ng-form=form 
on-open=`ui.loadPanel('${form}')`)
                             placeholder: 'Choose database'
                         })
                     
.pc-form-grid-col-60(ng-if=`$ctrl.Caches.requiresProprietaryDrivers(${blobStoreFactory})`)
-                        a.link-success(ng-href=`{{ 
$ctrl.Caches.JDBCDriverURL(${blobStoreFactory}) }}` target='_blank')
+                        a.link-success(ng-href=`{{ 
$ctrl.Caches.jdbcDriverURL(${blobStoreFactory}) }}` target='_blank')
                             | Download JDBC drivers?
                     .pc-form-grid-col-60
                         +form-field__checkbox({
diff --git 
a/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cluster-edit-form/templates/checkpoint/jdbc.pug
 
b/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cluster-edit-form/templates/checkpoint/jdbc.pug
index ed5f148..945f54d 100644
--- 
a/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cluster-edit-form/templates/checkpoint/jdbc.pug
+++ 
b/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cluster-edit-form/templates/checkpoint/jdbc.pug
@@ -38,7 +38,7 @@ include /app/helpers/jade/mixins
         placeholder: 'Choose JDBC dialect'
     })
 
.pc-form-grid-col-60(ng-if='$ctrl.Clusters.requiresProprietaryDrivers($checkpointSPI.JDBC)')
-    a.link-success(ng-href='{{ 
$ctrl.Clusters.JDBCDriverURL($checkpointSPI.JDBC) }}' target='_blank')
+    a.link-success(ng-href='{{ 
$ctrl.Clusters.jdbcDriverURL($checkpointSPI.JDBC) }}' target='_blank')
         | Download JDBC drivers?
 .pc-form-grid-col-60
     +form-field__java-class({
diff --git 
a/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cluster-edit-form/templates/general/discovery/jdbc.pug
 
b/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cluster-edit-form/templates/general/discovery/jdbc.pug
index a285408..e82814f 100644
--- 
a/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cluster-edit-form/templates/general/discovery/jdbc.pug
+++ 
b/modules/web-console/frontend/app/configuration/components/page-configure-advanced/components/cluster-edit-form/templates/general/discovery/jdbc.pug
@@ -48,5 +48,5 @@ mixin discovery-jdbc(modelAt = '$ctrl.clonedCluster')
                 tip: 'Flag indicating whether DB schema should be initialized 
by Ignite or was explicitly created by user'
             })
         
.pc-form-grid-col-30(ng-if=`$ctrl.Clusters.requiresProprietaryDrivers(${modelAt}.discovery.Jdbc)`)
-            a.link-success(ng-href=`{{ 
$ctrl.Clusters.JDBCDriverURL(${modelAt}.discovery.Jdbc) }}` target='_blank')
+            a.link-success(ng-href=`{{ 
$ctrl.Clusters.jdbcDriverURL(${modelAt}.discovery.Jdbc) }}` target='_blank')
                 | Download JDBC drivers?
\ No newline at end of file
diff --git a/modules/web-console/frontend/app/configuration/services/Caches.ts 
b/modules/web-console/frontend/app/configuration/services/Caches.ts
index 8ee2b48..d1ea010 100644
--- a/modules/web-console/frontend/app/configuration/services/Caches.ts
+++ b/modules/web-console/frontend/app/configuration/services/Caches.ts
@@ -226,11 +226,11 @@ export default class Caches {
         return cache && cache.cacheMode === 'PARTITIONED';
     }
 
-    requiresProprietaryDrivers(storeFactory) {
-        return ['Oracle', 'DB2', 'SQLServer'].includes(get(storeFactory, 
'dialect'));
+    jdbcDriverURL(storeFactory) {
+        return this.JDBC_LINKS[get(storeFactory, 'dialect')];
     }
 
-    JDBCDriverURL(storeFactory) {
-        return this.JDBC_LINKS[get(storeFactory, 'dialect')];
+    requiresProprietaryDrivers(storeFactory) {
+        return !!this.jdbcDriverURL(storeFactory);
     }
 }
diff --git 
a/modules/web-console/frontend/app/configuration/services/Clusters.ts 
b/modules/web-console/frontend/app/configuration/services/Clusters.ts
index 5aaa3f1..ba6ddc6 100644
--- a/modules/web-console/frontend/app/configuration/services/Clusters.ts
+++ b/modules/web-console/frontend/app/configuration/services/Clusters.ts
@@ -226,12 +226,12 @@ export default class Clusters {
         };
     }
 
-    requiresProprietaryDrivers(dataSrc) {
-        return ['Oracle', 'DB2', 'SQLServer'].includes(get(dataSrc, 
'dialect'));
+    jdbcDriverURL(dataSrc) {
+        return this.JDBC_LINKS[get(dataSrc, 'dialect')];
     }
 
-    JDBCDriverURL(dataSrc) {
-        return this.JDBC_LINKS[get(dataSrc, 'dialect')];
+    requiresProprietaryDrivers(dataSrc) {
+        return !!this.jdbcDriverURL(dataSrc);
     }
 
     dataRegion = {

Reply via email to