Skip to content

Commit

Permalink
[JENKINS-32770] Fix jelly pages to enforce RUN_SCRIPTS
Browse files Browse the repository at this point in the history
  • Loading branch information
stephenc committed Feb 8, 2016
1 parent b9b6925 commit 4e5fa85
Showing 1 changed file with 3 additions and 3 deletions.
Expand Up @@ -27,14 +27,14 @@ THE SOFTWARE.
<j:when test="${!descriptor.isJob(it)}">
<!-- not a job so don't give job specific notices -->
</j:when>
<j:when test="${descriptor.permitReconfiguration and app.hasPermission(app.ADMINISTER)}">
<j:when test="${descriptor.permitReconfiguration and app.hasPermission(app.RUN_SCRIPTS)}">
<f:block>
<div class="warning">
${%Any non-administrators with permission to configure this job can retain this strategy while otherwise reconfiguring this job}
</div>
</f:block>
</j:when>
<j:when test="${instance == null and app.hasPermission(app.ADMINISTER)}">
<j:when test="${instance == null and app.hasPermission(app.RUN_SCRIPTS)}">
<f:block>
<div class="warning">
${%Only users with administrator permissions will be able to reconfigure this job while retaining this strategy}
Expand All @@ -43,7 +43,7 @@ THE SOFTWARE.
</j:when>
<j:when test="${instance == null}">
<f:block>
<div class="${it.hasPermission(app.ADMINISTER)?'warning':'error'}">
<div class="${it.hasPermission(app.RUN_SCRIPTS)?'warning':'error'}">
${%Only users with administrator permissions can select this strategy}
</div>
</f:block>
Expand Down

0 comments on commit 4e5fa85

Please sign in to comment.