From 34b5cd5267243867042325550ba50bd7a0ee46aa Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Tue, 26 May 2020 10:50:31 +0200
Subject: [PATCH] Bug 25599: Fix default value placeholders for
 UseACQFrameworkForBiblioRecords

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>
(cherry picked from commit b16e463173e8125396eaef3159d08df5d0d2cfb6)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
---
 acqui/neworderempty.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/acqui/neworderempty.pl b/acqui/neworderempty.pl
index 9414f64061..cf2e7ee945 100755
--- a/acqui/neworderempty.pl
+++ b/acqui/neworderempty.pl
@@ -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;
-- 
2.39.5