Fix for admin having no (execute) rights on private projects/tplans#111
Open
kostgr wants to merge 1 commit intoTestLinkOpenSourceTRMS:testlink_1_9from
Open
Fix for admin having no (execute) rights on private projects/tplans#111kostgr wants to merge 1 commit intoTestLinkOpenSourceTRMS:testlink_1_9from
kostgr wants to merge 1 commit intoTestLinkOpenSourceTRMS:testlink_1_9from
Conversation
Contributor
|
Need to recheck and do some tests before accepting this merge request. |
Contributor
Author
|
If you have too much testcases in a suite - split up a suite, use filters (not assigned testcases, testcases status etc.), take a look at the settings (something with loading of tree and of the testcases for subsuites). |
Contributor
|
Issues has to be openen on http://mantis.testlink.org |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


We've experienced issue with global admins having no testplan execution right on some private projects / testplans (he could go to execution, but at the bottom of the Testcase description there was no area with execution notes and with buttons for marking test as blocked/successfull/failed).
For some reason the global admins are not always assigned an admin role on all of the projects/testplans. For the public projects and public testplans it is not an issue. But if project of testplan is private, than the $user->hasRight() returns false (I'm not sure about it => has debugged with vim and by trial and error, but that was my conclusion).
In the execSetResults controller it lead to an effect, that the user was identified as having no execution right.
It would be possibly better to ignore public-Attributes for global admins, but I've taken the simpler way (I'm not an php-Developer, so I've searched for the simplest way).
I had to add exception for exec_ro_access, because otherwise the wrong description (wrong menu item) was shown on the home for admins - "Show the exections RO" or something like this - instead of "Execute tests" (I've used the german language and haven't noticed the text exactly).