Skip to content

Commit 5dcc6b7

Browse files
author
Mike Kozicki
authored
Merge pull request #2989 from alectejada/AZD177372
AZD177372: hide cancel button when test starts running or is cancelled
2 parents a159137 + 6ad4b5a commit 5dcc6b7

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

www/running.inc

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,9 @@
3939
<div id="runningHeader" class="box">
4040
<?php
4141
$status = GetTestStatus($id, true);
42-
if (!array_key_exists('startTime', $status) || !$status['startTime']) {
42+
if ($status['statusCode'] < 400 && (!array_key_exists('startTime', $status) || !$status['startTime'])) {
4343
?>
44-
<form action="/cancelTest.php" method="get">
44+
<form class="cancel_form" action="/cancelTest.php" method="get">
4545
<input type="hidden" name="test" value="<?php echo $id;?>">
4646
<input type="submit" value="Cancel Test"/>
4747
</form>
@@ -137,9 +137,11 @@
137137
if (response['statusCode'] != undefined &&
138138
response['statusCode'] != lastStatusCode) {
139139
lastStatusCode = response['statusCode'];
140+
var showCancelButton = false;
140141
if (lastStatusCode == 101) {
141142
document.getElementById('waiting').classList.add('is-active');
142143
document.getElementById('testing').classList.remove('is-active');
144+
showCancelButton = true;
143145
} else if (lastStatusCode == 100) {
144146
document.getElementById('waiting').classList.remove('is-active');
145147
document.getElementById('testing').classList.add('is-active');
@@ -148,7 +150,14 @@
148150
} else {
149151
reload = true;
150152
}
153+
154+
if (!showCancelButton) {
155+
$('.cancel_form').hide();
156+
} else {
157+
$('.cancel_form').show();
158+
}
151159
}
160+
152161
if (response['statusText'] != undefined &&
153162
response['statusText'] != lastStatusText) {
154163
lastStatusText = response['statusText'];

0 commit comments

Comments
 (0)