From 24f8595501ca5225088cce4106c8822d7bd899cc Mon Sep 17 00:00:00 2001
From: erickson <erickson@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Thu, 27 May 2010 13:18:46 +0000
Subject: [PATCH] added note about new direction of opt-in circ history
maintenance; minor cleanup
git-svn-id: svn://svn.open-ils.org/ILS/trunk@16517 dcc99617-32d9-48b4-a31d-7c20da2025e4
---
.../perlmods/OpenILS/Application/Circ/Circulate.pm | 21 ++++++++++-----------
1 file changed, 10 insertions(+), 11 deletions(-)
diff --git a/Open-ILS/src/perlmods/OpenILS/Application/Circ/Circulate.pm b/Open-ILS/src/perlmods/OpenILS/Application/Circ/Circulate.pm
index d8366e20cb..0fd1dd52fb 100644
--- a/Open-ILS/src/perlmods/OpenILS/Application/Circ/Circulate.pm
+++ b/Open-ILS/src/perlmods/OpenILS/Application/Circ/Circulate.pm
@@ -3064,6 +3064,10 @@ sub run_renew_permit {
}
+# XXX: The primary mechanism for storing circ history is now handled
+# by tracking real circulation objects instead of bibs in a bucket.
+# However, this code is disabled by default and could be useful
+# some day, so may as well leave it for now.
sub append_reading_list {
my $self = shift;
@@ -3073,16 +3077,14 @@ sub append_reading_list {
$self->copy and
!$self->is_noncat;
- my $e = new_editor(xact => 1, requestor => $self->editor->requestor);
# verify history is globally enabled and uses the bucket mechanism
my $htype = OpenSRF::Utils::SettingsClient->new->config_value(
apps => 'open-ils.circ' => app_settings => 'checkout_history_mechanism');
- unless($htype eq 'bucket') {
- $e->rollback;
- return undef;
- }
+ return undef unless $htype and $htype eq 'bucket';
+
+ my $e = new_editor(xact => 1, requestor => $self->editor->requestor);
# verify the patron wants to retain the hisory
my $setting = $e->search_actor_user_setting(
@@ -3130,12 +3132,9 @@ sub append_reading_list {
sub make_trigger_events {
my $self = shift;
return unless $self->circ;
- my $ses = OpenSRF::AppSession->create('open-ils.trigger');
- $ses->request('open-ils.trigger.event.autocreate', 'checkout', $self->circ, $self->circ_lib) if $self->is_checkout;
- $ses->request('open-ils.trigger.event.autocreate', 'checkin', $self->circ, $self->circ_lib) if $self->is_checkin;
- $ses->request('open-ils.trigger.event.autocreate', 'renewal', $self->circ, $self->circ_lib) if $self->is_renewal;
-
- # ignore response
+ $U->create_events_for_hook('checkout', $self->circ, $self->circ_lib) if $self->is_checkout;
+ $U->create_events_for_hook('checkin', $self->circ, $self->circ_lib) if $self->is_checkin;
+ $U->create_events_for_hook('renewal', $self->circ, $self->circ_lib) if $self->is_renewal;
}
--
2.11.0