There is a hidden-test specified, but I've seen inconsistent behavior where it sometimes works and sometimes doesn't. I suspect a bit of a race condition with permission lookup.
I've put the pull request tag back on. If anyone wants to remove the hidden test, I wouldn't object at this point.
I want to add that I disagree with not hiding inoperable locations. There's no point in showing the user something that they cannot use as it is just clutter. I don't buy the consistency argument.
In that case, I'll leave it alone.
There is a hidden-test specified, but I've seen inconsistent behavior where it sometimes works and sometimes doesn't. I suspect a bit of a race condition with permission lookup.
I've put the pull request tag back on. If anyone wants to remove the hidden test, I wouldn't object at this point.
I want to add that I disagree with not hiding inoperable locations. There's no point in showing the user something that they cannot use as it is just clutter. I don't buy the consistency argument.
I am capitulating in this specific case, however.