doc_indiv_solution not given an appropriate default value (Bug #917)

Added by Michaël Cadilhac 8 months ago. Updated 8 months ago.

Status:New Start date:03/13/2024
Priority:Low Due date:
Assignee:- % Done:

0%
Category:-
Target version:-

Description

That's a very minor bug:
https://gitlab.com/a10684/auto-multiple-choice/-/blob/master/AMC-prepare.pl.in?ref_type=heads#L958
I believe missing there, for consistency, is

$out_corrige_indiv = $prefix . "indiv-solution.pdf" if ( !$out_corrige_indiv );

History

#1
Updated by Michaël Cadilhac 8 months ago

That being said, I'm not 100% sure that this code is ever needed. It seems that setting just project_dir (a.k.a. prefix) in options.xml has little impact, since the doc_ variables are set to a default value in Config.pm. For this to work, we would need a "pre" default and a "post" default. The "pre" would be the current Config::defaults, and the "post" would infer the value of fields that are not set based on the ones that are.

Also available in: Atom PDF