Skip to content
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

Closed
bozana opened this issue Mar 9, 2016 · 6 comments
Closed

Consider user rights/role for the "New Submission" button display #1248

bozana opened this issue Mar 9, 2016 · 6 comments
Assignees
Milestone

Comments

@bozana
Copy link
Collaborator

bozana commented Mar 9, 2016

Display the "New Submission" button only if the user is allowed to make a submission. S. #975 (comment).

@bozana bozana added this to the OMP 1.2 milestone Mar 9, 2016
@bozana
Copy link
Collaborator Author

bozana commented Mar 14, 2016

@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?

@asmecher
Copy link
Member

@bozana, the submitting user currently needs to be in a user group that's got an author role:
https://github.com/pkp/pkp-lib/blob/master/classes/submission/form/PKPSubmissionSubmitStep1Form.inc.php#L34

@bozana
Copy link
Collaborator Author

bozana commented Mar 17, 2016

@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

@bozana
Copy link
Collaborator Author

bozana commented Mar 17, 2016

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.

@asmecher
Copy link
Member

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.

asmecher added a commit that referenced this issue Mar 17, 2016
#1248 consider user role for the new submission button dis…
@asmecher
Copy link
Member

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants
@asmecher @bozana and others