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

Added by Michaël Cadilhac about 1 year ago. Updated 15 days ago.

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

100%
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 about 1 year 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.

#2
Updated by Alexis Bienvenüe 15 days ago

Should not be neede, but included with git:888793f88

  • Status changed from New to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF