[atlas] 03/04: ATLAS-3693:UI:[Business Metadata] Unable to add negative values in multi-valued attributes for byte, short, int, long, float, double.

[nixon]

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

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

commit 1c2eedffcf51259b7855e5d1c032dc02322376e1
Author: prasad pawar <prasad.pa...@freestoneinfotech.com>
AuthorDate: Thu Mar 18 15:20:10 2021 +0530

    ATLAS-3693:UI:[Business Metadata] Unable to add negative values in 
multi-valued attributes for byte, short, int, long, float, double.
    
    Signed-off-by: nixonrodrigues <ni...@apache.org>
---
 .../js/views/entity/EntityBusinessMetaDataItemView.js      | 14 +++++++++++---
 .../js/views/entity/EntityBusinessMetaDataItemView.js      | 14 +++++++++++---
 2 files changed, 22 insertions(+), 6 deletions(-)

diff --git 
a/dashboardv2/public/js/views/entity/EntityBusinessMetaDataItemView.js 
b/dashboardv2/public/js/views/entity/EntityBusinessMetaDataItemView.js
index f641546..254c8ac 100644
--- a/dashboardv2/public/js/views/entity/EntityBusinessMetaDataItemView.js
+++ b/dashboardv2/public/js/views/entity/EntityBusinessMetaDataItemView.js
@@ -137,8 +137,7 @@ define(['require',
                     if ((typename === "float" || typename === "array<float>") 
&& event.which == 46) {
                         return;
                     }
-                    if ((event.which < 48 || event.which > 57)) {
-
+                    if ((event.which < 48 || event.which > 57) && event.key 
!== "-") {
                         event.preventDefault();
                     }
                 });
@@ -231,7 +230,16 @@ define(['require',
                         }
                         var opt = {
                             tags: selectEl.data("tags") ? true : false,
-                            multiple: selectEl.data("multi")
+                            multiple: selectEl.data("multi"),
+                            createTag: function(params) {
+                                var option = params.term;
+                                if ($.isNumeric(option)) {
+                                    return {
+                                        id: option,
+                                        text: option
+                                    }
+                                }
+                            }
                         }
                         if (!selectEl.data("enum")) {
                             opt.data = data;
diff --git 
a/dashboardv3/public/js/views/entity/EntityBusinessMetaDataItemView.js 
b/dashboardv3/public/js/views/entity/EntityBusinessMetaDataItemView.js
index 1cec55f..5202d01 100644
--- a/dashboardv3/public/js/views/entity/EntityBusinessMetaDataItemView.js
+++ b/dashboardv3/public/js/views/entity/EntityBusinessMetaDataItemView.js
@@ -137,8 +137,7 @@ define(['require',
                     if ((typename === "float" || typename === "array<float>") 
&& event.which == 46) {
                         return;
                     }
-                    if ((event.which < 48 || event.which > 57)) {
-
+                    if ((event.which < 48 || event.which > 57) && event.key 
!== "-") {
                         event.preventDefault();
                     }
                 });
@@ -232,7 +231,16 @@ define(['require',
                         }
                         var opt = {
                             tags: selectEl.data("tags") ? true : false,
-                            multiple: selectEl.data("multi")
+                            multiple: selectEl.data("multi"),
+                            createTag: function(params) {
+                                var option = params.term;
+                                if ($.isNumeric(option)) {
+                                    return {
+                                        id: option,
+                                        text: option
+                                    }
+                                }
+                            }
                         }
                         if (!selectEl.data("enum")) {
                             opt.data = data;