From 6fdb19b0f97f477dc83afb5e2517f2537e6602de Mon Sep 17 00:00:00 2001 From: Zhong Date: Wed, 2 Dec 2015 15:02:03 +0800 Subject: [PATCH] KYLIN-919 model name validate when edit model,add lowercase check --- webapp/app/js/controllers/modelSchema.js | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/webapp/app/js/controllers/modelSchema.js b/webapp/app/js/controllers/modelSchema.js index 9e9f09c..682ce5e 100644 --- a/webapp/app/js/controllers/modelSchema.js +++ b/webapp/app/js/controllers/modelSchema.js @@ -166,9 +166,11 @@ KylinApp.controller('ModelSchemaCtrl', function ($scope, QueryService, UserServi var modelName = $scope.modelsManager.selectedModel.name.toUpperCase(); var models = $scope.modelsManager.modelNameList; - if ($scope.modelMode=="addNewModel"&&models.indexOf(modelName) != -1 || models.indexOf(modelName.toLowerCase()) !=-1) { - SweetAlert.swal('', "Model named [" + modelName + "] already exist!", 'warning'); - return false; + if ($scope.modelMode=="addNewModel") { + if(models.indexOf(modelName) != -1 || models.indexOf(modelName.toLowerCase()) !=-1){ + SweetAlert.swal('', "Model named [" + modelName + "] already exist!", 'warning'); + return false; + } } return true; } -- 2.4.9 (Apple Git-60)