Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent scaffolds from revealing correct answers when feedback is not permitted (really) #1163

Merged

Conversation

drgrice1
Copy link
Member

In #986 I believed that this was set. But either something has changed since then, or I didn't have it right at that point.

In that pull request I claimed that when an answer preview occurs, scaffolds will open but not receive the green color if all answers within are correct. In particular, if a preview occurs it should not reveal that answers are correct. Unfortunately, it still is.

This fixes that by only giving the correct colors when attempt results are allowed.

@drgrice1
Copy link
Member Author

Note that as stated in #986, the main point of this is for tests. In homework, although the correct colors will not be shown on preview, there is still an indication of correctness that the savvy student can figure out (depending on the scaffold settings of the problem). That is that later scaffolds will open when previous parts are correct. For tests even that will not reveal that parts are correct since all scaffolds are forced open in tests to begin with.

… permitted (really)

In openwebwork#986 I believed that this was set.  But either something has changed
since then, or I didn't have it right at that point.

In that pull request I claimed that when an answer preview occurs,
scaffolds will open but not receive the green color if all answers
within are correct.  In particular, if a preview occurs it should not
reveal that answers are correct.  Unfortunately, it still is.

This fixes that by only giving the correct colors when attempt results
are allowed.
@drgrice1 drgrice1 force-pushed the scaffolds-no-reveal-correct-really branch from eeda98b to 4aee938 Compare December 14, 2024 15:19
Copy link
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.

Works as described.

I'll close #1105 when this is merged, as I didn't realize that the correct color wasn't suppose to be shown which is part of what that issue was about. Sure a clever student who realizes the state of the scaffolds in homework can still use preview to figure out correct answers, but this is no longer possible in tests with my testing.

I'd suggest a hotfix for this as it is small and probably worthwhile to have in 2.19 for the time.

@drgrice1
Copy link
Member Author

I would agree to a hot fix since this was supposed to be something new for 2.19.

@Alex-Jordan Alex-Jordan merged commit cda6a90 into openwebwork:develop Dec 17, 2024
3 checks passed
@drgrice1 drgrice1 deleted the scaffolds-no-reveal-correct-really branch December 17, 2024 00:33
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