mirror of
https://github.com/apache/superset.git
synced 2026-04-12 04:37:49 +00:00
[sql lab] only show single run query button (#1858)
* only show single run query button, allow async if possible * only pass the needed props, rather than entire objects to the component * add simple test * fix linting
This commit is contained in:
@@ -0,0 +1,71 @@
|
||||
import React, { PropTypes } from 'react';
|
||||
import Button from '../../components/Button';
|
||||
|
||||
const propTypes = {
|
||||
allowAsync: PropTypes.bool.isRequired,
|
||||
dbId: PropTypes.number.isRequired,
|
||||
queryState: PropTypes.string.isRequired,
|
||||
runQuery: PropTypes.func.isRequired,
|
||||
selectedText: PropTypes.string,
|
||||
stopQuery: PropTypes.func.isRequired,
|
||||
};
|
||||
|
||||
export default function RunQueryActionButton(props) {
|
||||
const runBtnText = props.selectedText ? 'Run Selected Query' : 'Run Query';
|
||||
const btnStyle = props.selectedText ? 'warning' : 'primary';
|
||||
const shouldShowStopBtn = ['running', 'pending'].indexOf(props.queryState) > -1;
|
||||
const asyncToolTip = 'Run query asynchronously';
|
||||
|
||||
const commonBtnProps = {
|
||||
bsSize: 'small',
|
||||
bsStyle: btnStyle,
|
||||
disabled: !(props.dbId),
|
||||
};
|
||||
|
||||
const syncBtn = (
|
||||
<Button
|
||||
{...commonBtnProps}
|
||||
onClick={() => props.runQuery(false)}
|
||||
key="run-btn"
|
||||
>
|
||||
<i className="fa fa-table" /> {runBtnText}
|
||||
</Button>
|
||||
);
|
||||
|
||||
const asyncBtn = (
|
||||
<Button
|
||||
{...commonBtnProps}
|
||||
onClick={() => props.runQuery(true)}
|
||||
key="run-async-btn"
|
||||
tooltip={asyncToolTip}
|
||||
>
|
||||
<i className="fa fa-table" /> {runBtnText}
|
||||
</Button>
|
||||
);
|
||||
|
||||
const stopBtn = (
|
||||
<Button
|
||||
{...commonBtnProps}
|
||||
onClick={props.stopQuery}
|
||||
>
|
||||
<i className="fa fa-stop" /> Stop
|
||||
</Button>
|
||||
);
|
||||
|
||||
let button;
|
||||
if (shouldShowStopBtn) {
|
||||
button = stopBtn;
|
||||
} else if (props.allowAsync) {
|
||||
button = asyncBtn;
|
||||
} else {
|
||||
button = syncBtn;
|
||||
}
|
||||
|
||||
return (
|
||||
<div className="inline m-r-5 pull-left">
|
||||
{button}
|
||||
</div>
|
||||
);
|
||||
}
|
||||
|
||||
RunQueryActionButton.propTypes = propTypes;
|
||||
Reference in New Issue
Block a user