Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0-b3, 2.0
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-6293add loadAgenda to AgendaTypeService to allow pluggable qualifier matching
      KULRICE-6624AgendaTypeService impls aren't being used to translate AgendaDefinitions to Agendas
      KULRICE-6625AgendaTypeService impls aren't being used to translate AgendaDefinitions to Agendas
      KULRICE-13235Clear up the KIM qualifier question as to whether they should match by wildcard
      KULRICE-5315KimRoleTypeService (if configured) should get to decide how to handle null qualifier match
      KULRICE-3679Improve the performance of qualified role matching in KIM
      KULRICE-3848Delegation membership check not working for non-qualified roles
      KULRICE-6923Agenda selection should support selecting agenda by name
      KULRICE-9835removePrincipalFromRole uses attribute id instead of attribute name in qualifier
      KULRICE-12992Attribute matching now does exact match and does not take wildcards into account breaking KIM checks
    • Rice Module:
      KRMS
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      The way agenda qualifiers are being matched by default is by iterating through the attributes on the agenda and checking each against the qualifiers in the selection criteria. With that algorithm, if the agenda lacks an attribute that is in the qualifiers it won't ever check that one and so it will still match.

        Activity

        Hide
        Rice-CI User (Inactive) added a comment -

        Integrated in rice-trunk-nightly #269 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/269/)
        KULRICE-6098: Agenda qualifier matching is backwards

        BasicAgenda.java:

        • corrected qualifier matching logic

        SelectionCriteria.java:

        • Minor javadoc improvements

        AgendaTest.java:

        • added test to detect regression
        Show
        Rice-CI User (Inactive) added a comment - Integrated in rice-trunk-nightly #269 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/269/ ) KULRICE-6098 : Agenda qualifier matching is backwards BasicAgenda.java: corrected qualifier matching logic SelectionCriteria.java: Minor javadoc improvements AgendaTest.java: added test to detect regression
        Hide
        Jessica Coltrin (Inactive) added a comment -

        Closing since these items are now in the release notes.

        Show
        Jessica Coltrin (Inactive) added a comment - Closing since these items are now in the release notes.

          People

          • Assignee:
            Peter Giles (Inactive)
            Reporter:
            Peter Giles (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Structure Helper Panel