My conference workflow has four decision terms - Accept Full Paper, Accept Professional Poster, Accept Student Poster and Reject.
by editing the locale xml files I’ve edited the the standard terms (unsure about the implication further down the line however) but I appear to be restricted to only three decisions. How do I add/edit/remove decisions terms for reviewers and track directors?
Thanks
Hi @Paul_Coyne,
You can track this down by searching the codebase for the locale keys you’ve already been working with, e.g. in Linux:
find . -name \*.inc.php -exec grep -l director.paper.decision.accept "{}" ";"
This turns up…
classes/submission/trackDirector/TrackDirectorSubmission.inc.php
plugins/reports/papers/PaperReportPlugin.inc.php
In particular, you’ll need to work with TrackDirectorSubmission.inc.php
in the getDirectorDecisionOptions
function. (Other modifications will likely be required but that’s a good place to start.)
Regards,
Alec Smecher
Public Knowledge Project Team
Thanks,
I’ve made a start, but it’s hard to unpick the dependencies. For example I created 4 new decision terms as ACCEPT is too broad, but it looks as if the static constant SUBMISSION_DIRECTOR_ACCEPT is scattered liberally, upshot is I can see new decision terms on offer, but they’re not being recorded - though I’ve updated the DAO and Action file.
Isn’t there a dashboard setting somewhere allowing one to simply define a family of terms?
Thanks
Hi @Paul_Coyne,
Unfortunately director decisions are a little more involved than a vocabulary defined in one place, as they affect various areas of the workflow. Can you describe the behavior you’re experiencing a little more specifically? I’ll see if I can help track down the problem.
Regards,
Alec Smecher
Public Knowledge Project Team
Thanks Alec,
Here’s what I did, and the behaviour I now observe:
In the classes/common/Action class file I added these new decision terms
define(‘SUBMISSION_DIRECTOR_DECISION_ORAL’, 1);
define(‘SUBMISSION_DIRECTOR_DECISION_ACCEPT’, 2);
define(‘SUBMISSION_DIRECTOR_DECISION_PENDING_REVISIONS’, 3);
define(‘SUBMISSION_DIRECTOR_DECISION_BUFFER’, 4);
define(‘SUBMISSION_DIRECTOR_DECISION_DECLINE’, 5);
then I modified classes/submission/trackdirector/TrackDirectorSubmission.inc.php at line 145; I changed it to this -
if ($latestDecision[‘decision’] != SUBMISSION_DIRECTOR_DECISION_DECLINE)
And in the same file I modified the function
function &getDirectorDecisionOptions() {
$directorDecisionOptions = array(’’ => ‘common.chooseOne’);
$directorDecisionOptions[SUBMISSION_DIRECTOR_DECISION_ORAL] = ‘director.paper.decision.oralpresentation’;
$directorDecisionOptions[SUBMISSION_DIRECTOR_DECISION_ACCEPT] = ‘director.paper.decision.accept’;
$directorDecisionOptions[SUBMISSION_DIRECTOR_DECISION_PENDING_REVISIONS] = ‘director.paper.decision.pendingRevisions’;
$directorDecisionOptions[SUBMISSION_DIRECTOR_DECISION_PENDING_BUFFER] = ‘director.paper.decision.posterbuffer’;
$directorDecisionOptions[SUBMISSION_DIRECTOR_DECISION_DECLINE] = ‘director.paper.decision.decline’;
return $directorDecisionOptions;
}
Additionally, in classes/submissions/trackdirector/TrackDirectorAction.inc.php, at line 107 I made this change -
if($decision == SUBMISSION_DIRECTOR_DECISION_ACCEPT || $decision == SUBMISSION_DIRECTOR_DECISION_ORAL || $decision == SUBMISSION_DIRECTOR_DECISION_PENDING_REVISIONS || $decision == SUBMISSION_DIRECTOR_DECISION_BUFFER) {
// completeReview will take care of updating the
// submission with the new decision.
TrackDirectorAction::completeReview($trackDirectorSubmission);
} else {
// Insert the new decision.
$trackDirectorSubmissionDao->updateTrackDirectorSubmission($trackDirectorSubmission);
}
What I hoped/expected to happen was that
- The Track Director decision selection options would include the new Decision Terms. They do, so that’s OK
- When selecting a decision the decision would be recorded as is the case prior to the modifications. Only partially - the Accept As Oral and Accept As Buffer options do not call the recordDecision function and I don’t know why - I’m not sure where to look next so any help is gratefully received!
Best
Paul
Hi @Paul_Coyne,
Have you checked the edit_decisions
table in the database to see whether the decision is being recorded but perhaps not displayed?
Regards,
Alec Smecher
Public Knowledge Project Team
Hi @asmecher,
Can you please elaborate- how to create a new decision called Accept as Poster under abstract review- director decision.
Hi @Subrat_Kumar_Sahu,
I see you’ve already posted your question here. Please don’t double-post – it clutters the forum.
Regards,
Alec Smecher
Public Knowledge Project Team