From 4efb86aec85730ee3e3648b343ec18bbe4f14d06 Mon Sep 17 00:00:00 2001
From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Date: Thu, 12 Jun 2025 11:15:09 +0200
Subject: [PATCH] Bug 40119: (QA follow-up) Small additional comment

The reference to this bug report may be helpful to understand
later why we did so.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 01526ef545b28a79555756e9ae9b02f8a63cd525)
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
---
 C4/AuthoritiesMarc.pm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/C4/AuthoritiesMarc.pm b/C4/AuthoritiesMarc.pm
index 4cbed9e95d..dfc0749861 100644
--- a/C4/AuthoritiesMarc.pm
+++ b/C4/AuthoritiesMarc.pm
@@ -1749,7 +1749,8 @@ sub merge {
                 if ( exists $controlled_ind->{sub2} ) {    # thesaurus info
                     if ( defined $controlled_ind->{sub2} ) {
 
-                        # Add or replace
+                        # Add or replace when not empty. If empty, an existing
+                        # $2 value is retained (see also bug 40119).
                         if ( $controlled_ind->{sub2} ne q{} ) {
                             $field_to->update( 2 => $controlled_ind->{sub2} );
                         }