Bug 25599: Fix default value placeholders for UseACQFrameworkForBiblioRecords
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 26 May 2020 08:50:31 +0000 (10:50 +0200)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 15 Jun 2020 08:30:13 +0000 (10:30 +0200)
The feature was there but a condition disabled it.

Test plan:
- modify the default value for 008@ or another subfield in the ACQ framework using one or more of the placeholders above
- activate UseACQFrameworkForBiblioRecords
- create a basket
- create an order from a new record
- verify the values have been replaced

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

acqui/neworderempty.pl

index 9414f64..cf2e7ee 100755 (executable)
@@ -226,7 +226,7 @@ if ( not $ordernumber ) {    # create order
                     }
                 }
 
-                if ( $value eq '' ) {
+                if ( $value ) {
 
                     # get today date & replace <<YYYY>>, <<MM>>, <<DD>> if provided in the default value
                     my $today_dt = dt_from_string;