You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When loading the following problem: Library/MC/PreAlgebra/setPreAlgebraC01S06/SUbtractingIntegers03.pg
I'm getting an error:
VM1878:1 Uncaught TypeError: Cannot read properties of undefined (reading 'style')
at <anonymous>:1:61
at eval (Problem.vue?3854:162)
at NodeList.forEach (<anonymous>)
at eval (Problem.vue?3854:157)
at Generator.next (<anonymous>)
at fulfilled (VM1642 Problem.vue:16)
## Add the problemRandomize message and data#if ($isWhen && !$okDate) {
$result->{msg} .= "</i><br /><b>".$main::PG->maketext("Note:")."</b> <i>"if$result->{msg};
$result->{msg} .= $main::PG->maketext("You can get a new version of this problem after the due date.");
}
if (!$result->{msg}) {
# hack to remove unwanted "<b>Note: </b>" from the problem# (it is inserted automatically by Problem.pm when {msg} is non-emtpy).$result->{msg} .= '<script>var bb = document.getElementsByTagName("b");'
. 'bb[bb.length-1].style.display="none"</script>';
}
$result->{msg} .= qq!<input type="hidden" name="_status" value="$data" />!;
Even webwork2 has a built-in way to do this without problemRandomize.pl (though it has issues). I agree with @drdrew42 that there is no reason to support problemRandomize.pl going forward. In the past the decision was made by the OPL board to not accept problems that include it, so I would like to see it removed from problems in the OPL. Just don't tell @drjt I said that, since it looks like it's mostly MC problems that include it.
Activity
pstaabp commentedon Nov 18, 2021
I just dug into this a bit more. In webwork 2, at the bottom of the problem, the following is in the rendered problem:
However, in webwork3, the same block is:
so it's missing the
<b style="display: none;">Note: </b>so there is an error thrown when the script is run.I'm not sure why this is like this. What's the point of printing: "Note:" then hiding it.
drdrew42 commentedon Nov 18, 2021
Yet another reason to ditch
problemRandomize.plpstaabp commentedon Nov 19, 2021
I think having problem versions will allow us to do this in a built-in way, correct?
dlglin commentedon Nov 19, 2021
Even webwork2 has a built-in way to do this without problemRandomize.pl (though it has issues). I agree with @drdrew42 that there is no reason to support problemRandomize.pl going forward. In the past the decision was made by the OPL board to not accept problems that include it, so I would like to see it removed from problems in the OPL. Just don't tell @drjt I said that, since it looks like it's mostly MC problems that include it.