***SPAM*** [OpenSRF-GIT] OpenSRF branch master updated. 6ed6f2bc6bf56d0834ae43f00c193bce5a3867a1

Evergreen Git git at git.evergreen-ils.org
Thu Aug 21 12:05:38 EDT 2014


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "OpenSRF".

The branch, master has been updated
       via  6ed6f2bc6bf56d0834ae43f00c193bce5a3867a1 (commit)
      from  f0c02bf3cc4e8e2c1049fd2ba0fd24079b5aeba6 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 6ed6f2bc6bf56d0834ae43f00c193bce5a3867a1
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Thu Aug 21 08:58:18 2014 -0700

    Revert "LP#1350457: Pass caller's session to subrequests called via method_lookup"
    
    This reverts commit 96917538c6b6fad688dd40c2a8f0cc9127053b24.

diff --git a/src/perl/lib/OpenSRF/Application.pm b/src/perl/lib/OpenSRF/Application.pm
index d409957..5ae98bf 100644
--- a/src/perl/lib/OpenSRF/Application.pm
+++ b/src/perl/lib/OpenSRF/Application.pm
@@ -171,7 +171,6 @@ sub handler {
 
 		if (ref $coderef) {
 			my @args = $app_msg->params;
-			$coderef->session( $session );
 			my $appreq = OpenSRF::AppRequest->new( $session );
 			$appreq->max_chunk_size( $coderef->max_chunk_size );
 			$appreq->max_chunk_count( $coderef->max_chunk_count );
@@ -559,7 +558,6 @@ sub method_lookup {
 		$meth = $self->method_lookup($method,$proto,1);
 	}
 
-	$meth->session($self->session) if $meth; # Pass the caller's session
 	return $meth;
 }
 
@@ -573,7 +571,9 @@ sub run {
 	if ( !UNIVERSAL::isa($req, 'OpenSRF::AppRequest') ) {
 		$log->debug("Creating a SubRequest object", DEBUG);
 		unshift @params, $req;
-		$req = OpenSRF::AppSubrequest->new( session => $self->session );
+		$req = OpenSRF::AppSubrequest->new;
+		$req->session( $self->session ) if ($self->session);
+
 	} else {
 		$log->debug("This is a top level request", DEBUG);
 	}

-----------------------------------------------------------------------

Summary of changes:
 src/perl/lib/OpenSRF/Application.pm |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
OpenSRF


More information about the opensrf-commits mailing list