Skip to content

Fix the maketext call added to lib/Value/AnswerChecker.pm in #1224.#1235

Merged
pstaabp merged 1 commit intoopenwebwork:PG-2.20from
drgrice1:fix-failing-maketext
May 13, 2025
Merged

Fix the maketext call added to lib/Value/AnswerChecker.pm in #1224.#1235
pstaabp merged 1 commit intoopenwebwork:PG-2.20from
drgrice1:fix-failing-maketext

Conversation

@drgrice1
Copy link
Copy Markdown
Member

Since that is outside of the safe compartment the maketext method can not be called directly. Instead it needs to be called via pgCall.

This fixes issue #1234.

…bwork#1224.

Since that is outside of the safe compartment the `maketext` method can
not be called directly.  Instead it needs to be called via `pgCall`.

This fixes issue openwebwork#1234.
Copy link
Copy Markdown
Contributor

@somiaj somiaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes the issue.

Copy link
Copy Markdown
Member

@pstaabp pstaabp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Issue fixed.

@pstaabp pstaabp merged commit 1392a55 into openwebwork:PG-2.20 May 13, 2025
3 checks passed
@drgrice1 drgrice1 deleted the fix-failing-maketext branch May 14, 2025 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants