You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by zh...@apache.org on 2015/11/23 08:40:10 UTC

incubator-kylin git commit: KYLIN-919, do not validate model name when edit

Repository: incubator-kylin
Updated Branches:
  refs/heads/2.x-staging 1a1f2bedd -> dff549547


KYLIN-919, do not validate model name when edit


Project: http://git-wip-us.apache.org/repos/asf/incubator-kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kylin/commit/dff54954
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kylin/tree/dff54954
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kylin/diff/dff54954

Branch: refs/heads/2.x-staging
Commit: dff549547c524ee3b4b0a7f954de71bdb3eb361f
Parents: 1a1f2be
Author: jiazhong <ji...@ebay.com>
Authored: Mon Nov 23 15:39:41 2015 +0800
Committer: jiazhong <ji...@ebay.com>
Committed: Mon Nov 23 15:39:41 2015 +0800

----------------------------------------------------------------------
 webapp/app/js/controllers/modelSchema.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-kylin/blob/dff54954/webapp/app/js/controllers/modelSchema.js
----------------------------------------------------------------------
diff --git a/webapp/app/js/controllers/modelSchema.js b/webapp/app/js/controllers/modelSchema.js
index f4bae9e..9e9f09c 100644
--- a/webapp/app/js/controllers/modelSchema.js
+++ b/webapp/app/js/controllers/modelSchema.js
@@ -166,7 +166,7 @@ KylinApp.controller('ModelSchemaCtrl', function ($scope, QueryService, UserServi
 
     var modelName = $scope.modelsManager.selectedModel.name.toUpperCase();
     var models = $scope.modelsManager.modelNameList;
-    if (models.indexOf(modelName) != -1 || models.indexOf(modelName.toLowerCase()) !=-1) {
+    if ($scope.modelMode=="addNewModel"&&models.indexOf(modelName) != -1 || models.indexOf(modelName.toLowerCase()) !=-1) {
       SweetAlert.swal('', "Model named [" + modelName + "] already exist!", 'warning');
       return false;
     }