From b75cdd3708371470a0526a731804de7f26f4642f Mon Sep 17 00:00:00 2001
From: Ian Ibbotson <ianibbo@googlemail.com>
Date: Tue, 26 Jul 2022 12:07:07 +0100
Subject: [PATCH] Readonly transactional annotations

---
 .../org/olf/EntitlementController.groovy        | 11 +++++++++++
 .../olf/SubscriptionAgreementController.groovy  | 17 ++++++++++++++++-
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/service/grails-app/controllers/org/olf/EntitlementController.groovy b/service/grails-app/controllers/org/olf/EntitlementController.groovy
index 83bb3ccab..679f3af00 100644
--- a/service/grails-app/controllers/org/olf/EntitlementController.groovy
+++ b/service/grails-app/controllers/org/olf/EntitlementController.groovy
@@ -6,6 +6,7 @@ import com.k_int.okapi.OkapiTenantAwareController
 
 import grails.gorm.multitenancy.CurrentTenant
 import groovy.util.logging.Slf4j
+import grails.gorm.transactions.Transactional
 
 
 /**
@@ -18,6 +19,16 @@ class EntitlementController extends OkapiTenantAwareController<EntitlementContro
   EntitlementController() {
     super(Entitlement)
   }
+
+  @Transactional(readOnly=true)
+  def index(Integer max) {
+    super.index(max)
+  }
+
+  @Transactional(readOnly=true)
+  def show() {
+    super.show()
+  }
   
   def external() {
     Entitlement ent = new Entitlement ()
diff --git a/service/grails-app/controllers/org/olf/SubscriptionAgreementController.groovy b/service/grails-app/controllers/org/olf/SubscriptionAgreementController.groovy
index 7d8e85f7a..afeac90f7 100644
--- a/service/grails-app/controllers/org/olf/SubscriptionAgreementController.groovy
+++ b/service/grails-app/controllers/org/olf/SubscriptionAgreementController.groovy
@@ -35,7 +35,17 @@ class SubscriptionAgreementController extends OkapiTenantAwareController<Subscri
     super(SubscriptionAgreement)
   }
   
-  
+  @Transactional(readOnly=true)
+  def index(Integer max) {
+    super.index(max)
+  }
+
+  @Transactional(readOnly=true)
+  def show() {
+    super.show()
+  }
+
+  @Transactional(readOnly=true)
   def publicLookup () {
     final List<String> referenceIds = params.list('referenceId')
     final List<String> resourceIds = params.list('resourceId')
@@ -216,6 +226,7 @@ class SubscriptionAgreementController extends OkapiTenantAwareController<Subscri
     })
   } 
   
+  @Transactional(readOnly=true)
   def resources () {
     
     final String subscriptionAgreementId = params.get("subscriptionAgreementId")
@@ -284,6 +295,7 @@ class SubscriptionAgreementController extends OkapiTenantAwareController<Subscri
     }
   }
   
+  @Transactional(readOnly=true)
   def currentResources () {
     
     final String subscriptionAgreementId = params.get("subscriptionAgreementId")
@@ -383,6 +395,7 @@ class SubscriptionAgreementController extends OkapiTenantAwareController<Subscri
     }
   }
   
+  @Transactional(readOnly=true)
   def droppedResources () {
     
     final String subscriptionAgreementId = params.get("subscriptionAgreementId")
@@ -450,6 +463,7 @@ class SubscriptionAgreementController extends OkapiTenantAwareController<Subscri
     }
   }
   
+  @Transactional(readOnly=true)
   def futureResources () {
     
     final String subscriptionAgreementId = params.get("subscriptionAgreementId")
@@ -575,6 +589,7 @@ class SubscriptionAgreementController extends OkapiTenantAwareController<Subscri
     respond ([statusCode: 404])
   }
   
+  @Transactional(readOnly=true)
   def export () {
     final String subscriptionAgreementId = params.get("subscriptionAgreementId")
     if (subscriptionAgreementId) {