Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
When a server error is caught by applyErrorMessage (“ERROR” link), do…
… not render the whole oops.jelly page.

Just pick out the <div id="error-description"> from JENKINS-20772.
  • Loading branch information
jglick committed Oct 14, 2014
1 parent eaf7a26 commit 9620057
Showing 1 changed file with 16 additions and 5 deletions.
21 changes: 16 additions & 5 deletions war/src/main/webapp/scripts/hudson-behavior.js
Expand Up @@ -2202,11 +2202,22 @@ function validateButton(checkUrl,paramList,button) {
}

function applyErrorMessage(elt, rsp) {
var i;
elt.innerHTML = rsp.status==200 ? rsp.responseText
: '<a href="" onclick="document.getElementById(\'valerr' + (i=iota++)
+ '\').style.display=\'block\';return false">ERROR</a><div id="valerr'
+ i + '" style="display:none">' + rsp.responseText + '</div>';
if (rsp.status == 200) {
elt.innerHTML = rsp.responseText;
} else {
var id = 'valerr' + (iota++);
elt.innerHTML = '<a href="" onclick="document.getElementById(\'' + id
+ '\').style.display=\'block\';return false">ERROR</a><div id="'
+ id + '" style="display:none">' + rsp.responseText + '</div>';
var error = document.getElementById('error-description'); // cf. oops.jelly
if (error) {
var div = document.getElementById(id);
while (div.firstChild) {
div.removeChild(div.firstChild);
}
div.appendChild(error);
}
}
Behaviour.applySubtree(elt);
}

Expand Down

0 comments on commit 9620057

Please sign in to comment.