Hi,
I am trying to figure out how to correctly react to cancelled one-click reviews. The current logic in OJS 2.4.x is to display an inactive review page which puzzles some of our reviewers – imagine that the review invitation is two months old, you click on it and the landing page looks like a review, but it is in fact a relict of a cancelled review.
Before trying to investigate it further in 2.x context, I have tested it with OJS3, which unfortunately does not display anything at all due to a fatal error in ReviewHandler::_validateAccessKey()
in /pages/reviewer/ReviewHandler.inc.php
. The result is a blank page.
After some tweaks I am in a situation where the modified OJS3 reacts the same on (a) attempt to access a non-existent review, (b) attempt to access an existent review with an incorrect access key, and (c) attempt to access a cancelled review with a correct access key – in all cases the user lands on the login page without any additional information whatsoever. I would accept that for cases (a) and (b), but the case (c) deserves in my opinion at least a terse message saying something like “Too late, sorry!”
I suppose I will have to dig deeper into how ReviewHandler
is implemented and where is the redirect to login page implemented. Am I right?
Thanks.
– jan