Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-4177

Improve error handling on person doc when checking permissions for adding roles

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Major
    • Resolution: Cannot Reproduce
    • Affects Version/s: None
    • Fix Version/s: 1.0.3
    • Component/s: Development
    • Labels:
      None
    • Rice Module:
      KIM
    • Application Requirement:
      KFS

      Description

      If you try to add a role on a person using the role # instead of using the lookup you get an error at the bottom of the page when you click "add":
      Not Authorized to assign role null null.

      I'm sure this is because it's trying to figure out if I have permission to add the role I've selected but hasn't figured out the namespace yet. It would be great if we could figure out a way for it to find the namespace before doing this check.

      At a minimum the error should be moved to the membership tab.

        Attachments

          Issue Links

            Activity

            Hide
            g1zhang Ge Zhang (Inactive) added a comment -

            Tried to repeat this by editing an existing person and by creating a new person, but can not repeat it, what I can see is the role can be added to a person doc by directly filling in a role number
            if the role can not be added(for example a derived role), the error message is shouwn on the membership tab.

            I guess this issue might be resolved by recent fix, please let me know if yo have any concerns. thanks

            Show
            g1zhang Ge Zhang (Inactive) added a comment - Tried to repeat this by editing an existing person and by creating a new person, but can not repeat it, what I can see is the role can be added to a person doc by directly filling in a role number if the role can not be added(for example a derived role), the error message is shouwn on the membership tab. I guess this issue might be resolved by recent fix, please let me know if yo have any concerns. thanks
            Hide
            g1zhang Ge Zhang (Inactive) added a comment -

            feel free to re-open it if this is still a problem

            Show
            g1zhang Ge Zhang (Inactive) added a comment - feel free to re-open it if this is still a problem
            Hide
            ddorsey Damon Dorsey added a comment -

            I confirmed that this can't be reproduced in KFS CNV, so I'm closing.
            thanks!
            Damon

            Show
            ddorsey Damon Dorsey added a comment - I confirmed that this can't be reproduced in KFS CNV, so I'm closing. thanks! Damon

              People

              • Assignee:
                g1zhang Ge Zhang (Inactive)
                Reporter:
                dlemus Dan Lemus (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 1 day
                  1d
                  Remaining:
                  Remaining Estimate - 1 day
                  1d
                  Logged:
                  Time Spent - Not Specified
                  Not Specified