Webstaff: Can't remove registered workstation at root of user's org/permission tree.

[Jason Stephenson]

While looking into a webstaff client issue encountered by another, I registered a workstation for the admin user in a concerto database at the CONS.

I discovered that I could not later remove it, though I could register and remove any other workstation. The Remove button is dimmed out for this workstation. (Screen shots attached.)

The reason is that the CONS org_unit is first (i.e. zero) in the perm ou array and so the check for indexOf() returns 0, a false value. The check should be indexOf() != -1.

git branch is at working/user/dyrcona/lp1648918-remove-workstation-fix
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/dyrcona/lp1648918-remove-workstation-fix

[Jason Stephenson (jstephenson)]

Here's a screenshot showing that I can remove the BR1 workstation.

[Michele Morgan (mmorgan)]

Tested this as follows:

Registered a workstation at CONS level
Registered a workstation at SYS1 level
Selected the SYS1 registration, clicked Use Now
Redirected to Login screen, logged in as SYS1 workstation
Navigated to registered workstations
Chose CONS
Clicked Remove, the workstation was removed

Also:

Registered a workstation at CONS level
Selected the CONS registration, clicked Use Now
Redirected to Login screen, logged in as CONS workstation
Navigated to registered workstations
Chose CONS
Clicked Remove, the workstation was removed
Redirected to register a workstation

In all test cases I was able to successfully remove the CONS workstation

My signoff branch is at:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mmorgan/lp_1648918-signoff

[Galen Charlton (gmc)]

Pushed to master for inclusion in 2.12. Thanks, Jason and Michele!