Open defstat opened 5 months ago
@defstat, while testing this I noticed another problem - unable to see any files under that grid. My steps:
SUBMISSION_FILE_REVIEW_ATTACHMENT
I can reproduce the bug described here by assigning a reviewer and declining this request. But I'm confused. I don't remember this part of the code at all. So, under that grid all files at the stage SUBMISSION_FILE_REVIEW_ATTACHMENT
should be shown if the review is open, right? But author is able to read the review and see the file through that form, so it's duplication of the functionality?
@defstat Also run into that - and was not aware of this issue. So I created new one at that time https://github.com/pkp/pkp-lib/issues/10453
I will keep it opened as well as it offers alternative solution.
What I propose as fix that display correctly the review assignment also if its not submitted yet. As result author (for open review) basically sees exactly the same thing as editors, because from the code that seemed to be the original intention.
@Vitaliy-1 @Devika008 @asmecher Any opinion on this?
Context is that when the review is opened one - we provide similar reviewer listing to author as the editor can see. Just with option to Read the review and no other actions. Which means that the author see whole process of reviewers are being requested, and they responded etc.
Question is whether we want to reduce it and display maybe only the reviews that has been submitted (which is closer to @defstat PR) or keep the whole process transparent.
@jardakotesovec, I'm hesitant to make a major change without some pretty wide user testing. We've run into complaints before from both authors and editors. Authors want more information (they want to know what's happening with their submission), but editors want authors to see less information (they are tired of authors bothering them over any perceived change in the display). So authors and editors want opposite things in this case, and the current balance (minus the bug) is a careful one.
@asmecher @Devika008 have made bit of research about this, which she will share here and we will align it with that.
My intention here was not to change the behaviour - was really trying to understand the original intention to align fix with it. But having some additional feedback from outside will help us to fix that correctly.
I think we will go likely with conservative approach to make sure that the authors can see the reviews when they are in, but not revealing much of the process itself.
Describe the bug In the Author's workflow->review stage of a given submission, the Reviewer Grid fails to be displayed.
To Reproduce Steps to reproduce the behavior:
What application are you using? OJS
main
branchAdditional information The underling error is
Fatal error: Uncaught TypeError: count(): Argument #1 ($value) must be of type Countable|array, null given
and it is referring to theactions
template variable of theAuthorReviewerGridCellProvider
.The problem seems to be related to this. Here, the
stable
branch defines the query like soThe way that is implemented in the
main
branch, it seems that it is missing ther.date_confirmed IS NOT NULL AND r.declined <> 1
part of the query.PRs
@Vitaliy-1 I have crafted this suggested PR for that.
PKP-LIB: #9958