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

No reviewers returned from the API endoint users/reviewers #10739

Closed
bozana opened this issue Dec 19, 2024 · 3 comments
Closed

No reviewers returned from the API endoint users/reviewers #10739

bozana opened this issue Dec 19, 2024 · 3 comments
Assignees
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Milestone

Comments

@bozana
Copy link
Collaborator

bozana commented Dec 19, 2024

S. https://forum.pkp.sfu.ca/t/rest-api-reviewers-list/93608

The API call api/v1/users/reviewers without the parameter 'reviewStage' returns no items, because it is not checked if the parameter 'reviewStage' exists before applying the filter by workflow stage in this line: https://github.com/pkp/pkp-lib/blob/stable-3_4_0/api/v1/users/PKPUserHandler.php#L224.

In 3.3 the function filterByReviewStage() takes only one stage. This seems to be changed in 3.4, so that the function filterByWorkflowStageIds() takes an array, however the parameter is still only one stage and not an array.

PRs:
stable-3_4_0:

main:

@bozana bozana added the Bug:1:Low A bug that does not have a severe consequence or affects a small number of users. label Dec 19, 2024
@bozana bozana added this to the 3.4.0-9 milestone Dec 19, 2024
bozana added a commit to bozana/pkp-lib that referenced this issue Dec 19, 2024
bozana added a commit to bozana/ojs that referenced this issue Dec 19, 2024
@bozana
Copy link
Collaborator Author

bozana commented Dec 19, 2024

Hi @ewhanson, could you please review the fix above?
Thanks a lot!

@bozana
Copy link
Collaborator Author

bozana commented Jan 6, 2025

Hi @ewhanson, maybe just a reminder: if you could review the fix above?
Thanks a lot!

@ewhanson
Copy link
Collaborator

ewhanson commented Jan 9, 2025

Hi @bozana, apologies for the delay. Looks good to me.

bozana added a commit to bozana/pkp-lib that referenced this issue Jan 10, 2025
bozana added a commit to bozana/ojs that referenced this issue Jan 10, 2025
bozana added a commit to bozana/ojs that referenced this issue Jan 10, 2025
bozana added a commit to bozana/omp that referenced this issue Jan 10, 2025
bozana added a commit to bozana/ops that referenced this issue Jan 10, 2025
bozana added a commit that referenced this issue Jan 10, 2025
#10739 getReviewers(): check if param reviewStage exists f…
bozana added a commit to pkp/ojs that referenced this issue Jan 10, 2025
pkp/pkp-lib#10739 submodule update ##bozana/10739-3_4_0##
bozana added a commit to pkp/omp that referenced this issue Jan 10, 2025
pkp/pkp-lib#10739 submodule update ##bozana/10739-3_4_0##
bozana added a commit to pkp/ops that referenced this issue Jan 10, 2025
pkp/pkp-lib#10739 submodule update ##bozana/10739-3_4_0##
bozana added a commit to bozana/pkp-lib that referenced this issue Jan 10, 2025
bozana added a commit that referenced this issue Jan 10, 2025
#10739 getReviewers(): check if param reviewStage exists f…
@bozana bozana closed this as completed Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Projects
None yet
Development

No branches or pull requests

3 participants