Add color hints for paper status in AveragePaperScore.php
Status: Beta
Brought to you by:
dirkgrunwald
In Chair/AveragePaperScore.php, it would be a nice
visual clue to users that a paper has been accepted or
rejected by coloring the HTML row of that paper if the
'judgement' column has been selected.
(better yet, perhaps the row can be colored without the
need for a judgement column.)
The coloring effect can be achieved by replacing the
code around line 479:
print "<tr> <td $redText> $rowNum<BR>($paperId)</td> ";
with:
$colorRowOutcome = $row['outcome'];
if($colorRowOutcome == 'accepted')
{
print "<tr bgcolor=#009900>";
}else if($colorRowOutcome=='rejected'){
print "<tr bgcolor=#CC0000>";
}else{
print "<tr>";
}
print " <td $redText> $rowNum<BR>($paperId)</td> ";
Cheers,
Michael
mlocasto AT acm.org
Logged In: NO
After a bit more poking about, I realized the above solution
only works for the PC Chair role.
In order to let all PC members see the decision, then line
149 needs to be commented out and moved outside the
enclosing if() test for the PC Chair role:
if( $_SESSION['Me']->isChair ){
....
....
if( IsSet($_REQUEST['GradeCol']) ){
$judgeCol = 1;
//$judgeData = ", Paper.outcome "; // move to below
print " CHECKED";
}
print "> Judgement Column<BR>\n";
}
// put here so that PC members can see a colored outcome
// this also works if you are the PC Chair only.
$judgeData = ", Paper.outcome ";
if( IsSet($_REQUEST['COLS']) ){
....
This makes sure that judgeData is included in $query
variable, so that $result resultSet includes the outcome and
thus $row['outcome'] holds the string outcome result for the
test suggested by my previous result.
Logged In: NO
The above fix works only for PC Chairs. For PC Members to
see a colored outcome, the code around line 149 needs to be
adjusted.
if( IsSet($_REQUEST['GradeCol']) ){
$judgeCol = 1;
//$judgeData = ", Paper.outcome "; // move to below
print " CHECKED";
}
print "> Judgement Column<BR>\n";
}
// put here so that PC members can see a colored outcome
$judgeData = ", Paper.outcome ";