Uploaded image for project: 'KFS Archive'
  1. KFS Archive
  2. KFSOLD-20158

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: 4.x
    • Labels:
      None
    • Sub-Committee:
      SYS
    • Impacted Modules:
      System

      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
            kymber Kymber Horn added a comment -

            Hi Damon, are you saying that this should not be fixed for release 3.0? Thanks, Kymber

            Show
            kymber Kymber Horn added a comment - Hi Damon, are you saying that this should not be fixed for release 3.0? Thanks, Kymber
            Hide
            ddorsey Damon Dorsey added a comment -

            Hey Kymber,
            Yes, it's really not stopping anything. You can still add the role you just need to use the lookup to do so and I don't suspect many users will remember the role ID numbers anyway. So this one doesn't seem like a deal breaker to me.
            Damon

            Show
            ddorsey Damon Dorsey added a comment - Hey Kymber, Yes, it's really not stopping anything. You can still add the role you just need to use the lookup to do so and I don't suspect many users will remember the role ID numbers anyway. So this one doesn't seem like a deal breaker to me. Damon
            Hide
            kymber Kymber Horn added a comment -

            Got it and you're right Thank you.

            Show
            kymber Kymber Horn added a comment - Got it and you're right Thank you.
            Hide
            ddorsey Damon Dorsey added a comment -

            Can no longer reproduce in KFS, the role is added successfully when I enter the number and click "add."

            Show
            ddorsey Damon Dorsey added a comment - Can no longer reproduce in KFS, the role is added successfully when I enter the number and click "add."

              People

              • Assignee:
                Unassigned
                Reporter:
                abyrne Ailish Byrne
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: