From dfd9e08b6ba2da956bea3ab288857df2ee63914e Mon Sep 17 00:00:00 2001
From: Pedro Amorim <pedro.amorim@openfifth.co.uk>
Date: Mon, 13 Oct 2025 12:55:28 +0000
Subject: [PATCH] Bug 37893: Add warning notes to obsolete xml file sections

Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com>
Signed-off-by: Lisette Scheer <lisette@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
---
 debian/templates/SIPconfig.xml | 2 ++
 etc/SIPconfig.xml              | 3 +++
 2 files changed, 5 insertions(+)

diff --git a/debian/templates/SIPconfig.xml b/debian/templates/SIPconfig.xml
index f0a408c67b..0213eaae5e 100644
--- a/debian/templates/SIPconfig.xml
+++ b/debian/templates/SIPconfig.xml
@@ -43,6 +43,7 @@
 -->
   </listeners>
 
+<!-- WARNING: Accounts are now managed in the Staff UI SIP2 module -->
   <accounts>
       <login id="term1"  password="term1" delimiter="|" error-detect="enabled" institution="CPL" encoding="ascii" checked_in_ok="1" />
       <login id="koha"   password="koha"  delimiter="|" error-detect="enabled" institution="kohalibrary" encoding="utf8" />
@@ -71,6 +72,7 @@ The implementation attribute is actually used to find the code to run,
 in our case "ILS".
 -->
 
+<!-- WARNING: Institutions are now managed in the Staff UI SIP2 module -->
 <institutions>
     <institution id="MAIN" implementation="ILS" parms="">
           <policy checkin="true" renewal="true" checkout="true"
diff --git a/etc/SIPconfig.xml b/etc/SIPconfig.xml
index 1988477691..55a8a4718f 100644
--- a/etc/SIPconfig.xml
+++ b/etc/SIPconfig.xml
@@ -52,6 +52,7 @@
 -->
   </listeners>
 
+<!-- WARNING: Accounts are now managed in the Staff UI SIP2 module -->
   <accounts>
       <login id="staff"  password="staff" delimiter="|" error-detect="enabled" institution="CPL" encoding="ascii" checked_in_ok="1" payment_type_writeoff="06" disallow_overpayment="1" />
       <login id="koha"   password="koha"  delimiter="|" error-detect="enabled" institution="kohalibrary" encoding="utf8" />
@@ -112,6 +113,7 @@ The implementation attribute is actually used to find the code to run,
 in our case "ILS".
 -->
 
+<!-- WARNING: Institutions are now managed in the Staff UI SIP2 module -->
 <institutions>
     <institution id="MAIN" implementation="ILS" parms="">
           <policy checkin="true" renewal="true" checkout="true"
@@ -147,6 +149,7 @@ in our case "ILS".
      If the same system preference is overridden at the login level, the login
      version will take precedence.
 -->
+<!-- WARNING: Syspref overrides are now managed in the Staff UI SIP2 module -->
 <syspref_overrides>
       <AllFinesNeedOverride>0</AllFinesNeedOverride>
 </syspref_overrides>