Skip to content

Add function and link for revise questionnaires with reload precedent…#5

Open
merluxx wants to merge 4 commits into
ACSPRI:masterfrom
merluxx:dev_revise
Open

Add function and link for revise questionnaires with reload precedent…#5
merluxx wants to merge 4 commits into
ACSPRI:masterfrom
merluxx:dev_revise

Conversation

@merluxx

@merluxx merluxx commented May 11, 2026

Copy link
Copy Markdown

Hello, some users have asked me for a way to review a completed and submitted questionnaire without losing the answers already submitted. I've therefore slightly modified the quexf code for this purpose and I'd like to know what you think of it and if it would be possible to integrate it into future releases if you think it's a good idea?

Sincerely.

@adamzammit

Copy link
Copy Markdown
Contributor

thanks for this request @merluxx .

Can you please explain to me how you use it in practice? It appears to alter the session information but from the administrative page. Do you click the revise button as the administrator, then go to the verification page as the administrator and complete the form?

@merluxx

merluxx commented May 12, 2026

Copy link
Copy Markdown
Author

Yes, that's exactly it. I had to add the session_reset; otherwise, after reviewing the first quiz, subsequent quizzes would display the pages from the first one.

@adamzammit

Copy link
Copy Markdown
Contributor

Thank you! I appreciate your efforts in sharing this patch. As it is, I won't include it in queXF as it relies on a side effect (setting the session from the administrator page) that could cause a data conflict if misused or misunderstood.

I don't expect you to re-write the patch if it works fine for you, but ideally instead of setting the session from the administrator page, it would set a flag on the form so that the next time the administrator accesses the form it will load the previous responses (similar to the reverify function, except instead of throwing away the first verification attempt, it keeps it).

I don't have much time to work on this at the moment myself but I'll leave this issue open to remind me / anyone else who wants to contribute.

Adam

…P session by using the flag `forms.done=4`.
@adamzammit

Copy link
Copy Markdown
Contributor

Thanks for the update @merluxx .

I can see that you are reassigning the formboxverifychar and formboxverifytext data from the first verifier to vid = 0 when moving to review.

I think the original vid = 0 records for that form probably need to be deleted first, then the reassignment is done.

This should be sufficient then to have the form in the state that the original verifier left it, without needing to use the session reload function at all, as records assigned to vid = 0 are automatically loaded as part of the verification process.

Also when using the T_() translation function please try to avoid putting in variables partway (eg T("test" . $something . " something else") as it won't be able to be translated later, it should just be a fixed string.

Julien Chavant added 2 commits June 2, 2026 11:31
…ck for easier identification when scanned files contain subject codes. Fixed print for translation
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.

2 participants