• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • core
    • None

      The delete link in the sidepanel is not checked for the right permisson. It checks if permission is configure and not delete.
      See
      main/core/src/main/resources/hudson/model/View/sidepanel.jelly
      *-----
      53 <j:if test="${it.owner.canDelete(it)}">
      54 <l:task icon="images/24x24/edit-delete.gif" href="delete" title="${%Delete View}" permission="${it.CONFIGURE}" />
      55 </j:if>
      *-----
      Should be
      *+++++
      53 <j:if test="${it.owner.canDelete(it)}">
      54 <l:task icon="images/24x24/edit-delete.gif" href="delete" title="${%Delete View}" permission="${it.DELETE}" />
      55 </j:if>
      *+++++

          [JENKINS-7605] View Delete Permisson not checked properly

          lukasfunk created issue -
          lukasfunk made changes -
          Description Original: The delete link in the sidepanel is not checked for the right permisson. It checks if permission is configure and not delete.
          See
          main/core/src/main/resources/hudson/model/View/sidepanel.jelly
          *-----
          53 <j:if test="${it.owner.canDelete(it)}">
          54 <l:task icon="images/24x24/edit-delete.gif" href="delete" title="${%Delete View}" permission="${it.CONFIGURE}" />
          55 </j:if>
          *-----
          New: The delete link in the sidepanel is not checked for the right permisson. It checks if permission is configure and not delete.
          See
          main/core/src/main/resources/hudson/model/View/sidepanel.jelly
          *-----
          53 <j:if test="${it.owner.canDelete(it)}">
          54 <l:task icon="images/24x24/edit-delete.gif" href="delete" title="${%Delete View}" permission="${it.CONFIGURE}" />
          55 </j:if>
          *-----
          Should be
          *+++++
          53 <j:if test="${it.owner.canDelete(it)}">
          54 <l:task icon="images/24x24/edit-delete.gif" href="delete" title="${%Delete View}" permission="${it.DELETE}" />
          55 </j:if>
          *+++++
          Romain Seguy made changes -
          Assignee New: Romain Seguy [ rseguy ]
          Romain Seguy made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]

          Code changed in hudson
          User: : rseguy
          Path:
          trunk/hudson/main/core/src/main/resources/hudson/model/View/sidepanel.jelly
          http://jenkins-ci.org/commit/36980
          Log:
          [FIXED JENKINS-7605]

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : rseguy Path: trunk/hudson/main/core/src/main/resources/hudson/model/View/sidepanel.jelly http://jenkins-ci.org/commit/36980 Log: [FIXED JENKINS-7605]
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]

          Code changed in hudson
          User: : rseguy
          Path:
          trunk/www/changelog.html
          http://jenkins-ci.org/commit/36981
          Log:
          [FIXED JENKINS-7605]

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : rseguy Path: trunk/www/changelog.html http://jenkins-ci.org/commit/36981 Log: [FIXED JENKINS-7605]
          Romain Seguy made changes -
          Status Original: Resolved [ 5 ] New: Closed [ 6 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 137704 ] New: JNJira + In-Review [ 204574 ]

            rseguy Romain Seguy
            lukasfunk lukasfunk
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: