From ad19d8249231d79db4e84bc71ce98f24f22ef2d7 Mon Sep 17 00:00:00 2001
From: chenzhx <346839943@qq.com>
Date: Thu, 13 Oct 2016 10:30:26 +0800
Subject: [PATCH] KYLIN-2052-the-group-by-column-wasn-t-displayed

---
 webapp/app/js/controllers/cubeMeasures.js | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/webapp/app/js/controllers/cubeMeasures.js b/webapp/app/js/controllers/cubeMeasures.js
index 2f191f9..ae53a6d 100644
--- a/webapp/app/js/controllers/cubeMeasures.js
+++ b/webapp/app/js/controllers/cubeMeasures.js
@@ -42,6 +42,14 @@ KylinApp.controller('CubeMeasuresCtrl', function ($scope, $modal,MetaModel,cubes
     }
     if($scope.newMeasure.function.expression=="TOP_N"){
       $scope.convertedColumns=[];
+       if($scope.newMeasure.function.configuration==null){
+              var GroupBy = {
+                  name:$scope.newMeasure.function.parameter.next_parameter.value,
+                  encoding:"dict",
+                  valueLength:0,
+                  }
+              $scope.convertedColumns.push(GroupBy);
+            }
       for(var configuration in $scope.newMeasure.function.configuration) {
         var _name=configuration.slice(14);
         var item=$scope.newMeasure.function.configuration[configuration];
@@ -131,6 +139,7 @@ KylinApp.controller('CubeMeasuresCtrl', function ($scope, $modal,MetaModel,cubes
   $scope.saveNewMeasure = function () {
 
     if ($scope.newMeasure.function.expression === 'TOP_N' ) {
+
       if($scope.newMeasure.function.parameter.value == ""){
         SweetAlert.swal('', '[TOP_N] ORDER|SUM by Column  is required', 'warning');
         return false;
-- 
2.7.2.windows.1