** Using WeBWorK **

The line numbers are off in the warning messages for files loaded via loadMacros(). This is because loadMacros() inserts two extra lines into the code being processed, and so those lines add to the line numbering. The actual error is two lines earlier, and probably is due to the $inputs_ref->{action} test. Not sure why, though, as that is the value of the submit button being pressed.

In any case, that part of the code is in the older non-MathObjects version of fun_cmp(), which is not used by default. Have you set the $useOldAnswerMacros variable in global.conf or your course.conf (or in the problem files themselves)? That part of the code should not have been called unless you set that variable to a non-zero value somewhere. Try setting it back to zero and see if that helps.

Davide

UPDATE: The $inputs_ref->{action} variable is actually the old WeBWorK1.x method of passing the submit button, (the $inputs_ref->{previewAnswer} earlier in line 1037 is the current method). So that would account for it not being defined. You could change it to

    my $isPreview = $inputs_ref->{previewAnswers} || ($inputs_ref->{action} && $inputs_ref->{action} =~ m/^Preview/);

Davide