-
Notifications
You must be signed in to change notification settings - Fork 452
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider user rights/role for the "New Submission" button display #1248
Comments
@beghelli, is it so that only author role can use this button? Manager too? Some other roles? Hmmm... Or all the roles that have the submission checkbox selected on the access setting page? |
@bozana, the submitting user currently needs to be in a user group that's got an author role: |
@asmecher, thanks for pointing to the place in the code -- that helps! So actually, it is all groups with author role (ROLE_ID_AUTHOR) and assigned to WORKFLOW_STAGE_ID_SUBMISSION + all groups with manager role (ROLE_ID_MANAGER), s. https://github.com/pkp/pkp-lib/blob/master/classes/submission/form/PKPSubmissionSubmitStep1Form.inc.php#L81-L104 |
Hmmm... If I only have translator role that is not assigned to WORKFLOW_STAGE_ID_SUBMISSION, I am able to enter the submit step 1 and then I get this warning: "Your account does not currently have a role permitting new submissions, such as Author. Please double-check your account; you may be able to add a suitable role yourself by editing your profile. ". Then I am able to give myself the author role and make a submission. Thus, actually, the new submission button should be displayed to all with the role ROLE_ID_AUTHOR and all with the role ROLE_ID_MANAGER. And if those with the ROLE_ID_AUTHOR are not assigned to WORKFLOW_STAGE_ID_SUBMISSION, they will get the warning. |
Yes, you're right -- managers and authors should be able to start submissions. We may get feedback that requires changes to this later, but I think that'll be a good start. |
#1248 consider user role for the new submission button dis…
Merged, thanks! |
Display the "New Submission" button only if the user is allowed to make a submission. S. #975 (comment).
The text was updated successfully, but these errors were encountered: