Uploaded image for project: 'Grouper'
  1. Grouper
  2. GRP-3152

AttributeDef trace privileges links back to subject page, not attributeDef page

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • 2.5.43
    • 2.4.0, 2.5.0
    • UI
    • None

    Description

      This is temporary code in trace privileges from March 2014:

      WEB-INF/grouperUi2/membership/traceAttributeDefPrivileges.jsp:56

                      <c:choose>
                        <%-- we only have subject at this point, take out the || true when we can do attribute def screen --%>
                        <c:when test="${grouperRequestContainer.membershipGuiContainer.traceMembershipFromSubject || true}">
                          <a href="#" onclick="return guiV2link('operation=UiV2Subject.thisSubjectsAttributeDefPrivileges&subjectId=${grouperRequestContainer.subjectContainer.guiSubject.subject.id}&sourceId=${grouperRequestCotake out the || true when we can do attribute def screenntainer.subjectContainer.guiSubject.subject.sourceId}');"
                             class="pull-right btn btn-primary btn-cancel">${textContainer.text['membershipTraceBackToSubjectButton']}</a>
                        </c:when>
                        <c:otherwise>
                          <a href="#" onclick="return guiV2link('operation=UiV2AttributeDef.attributeDefPrivileges&attributeDefId=${grouperRequestContainer.attributeDefContainer.guiAttributeDef.attributeDef.id}');"
                             class="pull-right btn btn-primary btn-cancel">${textContainer.text['membershipTraceBackToAttributeDefButton']}</a>
                        </c:otherwise>
                      </c:choose>
      

      The UiV2AttributeDef.attributeDefPrivileges screen was added in August 2014 (GRP-1021) so the `|| true` can be taken out. Note that membershipTraceBackToAttributeDefButton is currently undefined, but will be added with GRP-3016

      Attachments

        Activity

          People

            chad.redman@at.internet2.edu Chad Redman (unc.edu)
            chad.redman@at.internet2.edu Chad Redman (unc.edu)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: