Skip to content

Commit de366a9

Browse files
author
minjk-bl
committed
Fix Subset - Condition select form
1 parent f2be407 commit de366a9

File tree

3 files changed

+17
-5
lines changed

3 files changed

+17
-5
lines changed

html/m_apps/subset.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
<td colspan="2">
88
<!-- Preview code board -->
99
<div class="vp-ds-preview">
10-
<textarea id="vp_previewCode" name="code"># Code preview</textarea>
10+
<textarea id="vp_ssPreviewCode" name="code"># Code preview</textarea>
1111
</div>
1212
</td>
1313
</tr>

js/com/com_Kernel.js

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,19 @@ define([
141141
})
142142
});
143143
}
144+
145+
getColumnCategory(dataframe, columnName) {
146+
var that = this;
147+
return new Promise(function(resolve, reject) {
148+
that.execute(com_util.formatString('_vp_print(_vp_get_column_category({0}, {1}))', dataframe, columnName))
149+
.then(function(resultObj) {
150+
resolve(resultObj);
151+
}).catch(function(err) {
152+
// reject
153+
reject(err);
154+
})
155+
});
156+
}
144157

145158
getRowList(dataframe) {
146159
var that = this;

js/m_apps/Subset.js

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ define([
7575
...this.state
7676
};
7777

78-
this._addCodemirror('previewCode', this.wrapSelector('#vp_previewCode'), 'readonly');
78+
this._addCodemirror('previewCode', this.wrapSelector('#vp_ssPreviewCode'), 'readonly');
7979
}
8080

8181
render() {
@@ -1331,14 +1331,13 @@ define([
13311331

13321332
$(document).on('change', this.wrapSelector('.vp-ds-cond-tbl .vp-col-list'), function () {
13331333
var thisTag = $(this);
1334-
var varName = $(this).closest('td').find('.vp-cond-var').val();
1334+
var varName = that.state.pandasObject;
13351335
var colName = $(this).find('option:selected').attr('data-code');
13361336

13371337
var condTag = $(this).closest('td').find('.vp-condition');
13381338

1339-
var code = com_util.formatString('_vp_print(_vp_get_column_category({0}, {1}))', varName, colName);
13401339
// get result and load column list
1341-
vpKernel.execute(code).then(function (resultObj) {
1340+
vpKernel.getColumnCategory(varName, colName).then(function (resultObj) {
13421341
let { result } = resultObj;
13431342
var category = JSON.parse(result);
13441343
if (category && category.length > 0) {

0 commit comments

Comments
 (0)