authpds-nyu 0.2.4 → 0.2.5

data/lib/authpds-nyu/session/aleph.rb

@@ -1,29 +1,33 @@
 module AuthpdsNyu
   module Session
     module Aleph
+      require 'exlibris-aleph'
       def aleph_bor_auth_permissions(bor_id=nil, verification=nil, adm=nil, sublibrary=nil)
         bor_auth = aleph_bor_auth(bor_id, verification, adm, sublibrary)
         return (bor_auth.nil? or bor_auth.error) ? {} : bor_auth.permissions
       end
 
       def aleph_bor_auth(bor_id=nil, verification=nil, adm=nil, sublibrary=nil)
-        bor_id = pds_user.id if bor_id.nil? unless pds_user.nil?
+        if bor_id.nil? and pds_user
+          bor_id = pds_user.id
+          verification = pds_user.verification
+        end
         raise ArgumentError.new("Argument Error in #{self.class}. bor_id not specified.") if bor_id.nil?;
-        verification = pds_user.verification if verification.nil? unless pds_user.nil?
         raise ArgumentError.new("Argument Error in #{self.class}. verification not specified.") if verification.nil?;
         adm = aleph_default_adm if adm.nil?
         sublibrary = aleph_default_sublibrary if sublibrary.nil?
         # Call X-Service
-        bor_auth =
-          Exlibris::Aleph::BorAuth.
-            new(aleph_url, adm, sublibrary, "N", bor_id, verification)
-        controller.logger.error(
-            "Error in #{self.class}. "+
-            "No permissions returned from Aleph bor-auth for user with bor_id #{bor_id}."+
-            "Error: #{(bor_auth.nil?) ? "bor_auth is nil." : bor_auth.error.inspect}"
-          ) and return nil if bor_auth.nil? or bor_auth.error
+        bor_auth = Exlibris::Aleph::BorAuth.new(aleph_url, adm, sublibrary, "N", bor_id, verification)
+        log_error(bor_id, bor_auth) and return nil if bor_auth.nil? or bor_auth.error
         return bor_auth
       end
+      
+      def log_error bor_id, bor_auth
+        controller.logger.error "Error in #{self.class}. "+
+          "No permissions returned from Aleph bor-auth for user with bor_id #{bor_id}."+
+            "Error: #{(bor_auth.nil?) ? "bor_auth is nil." : bor_auth.error.inspect}"
+      end
+      private :log_error
     end
   end
 end

data/lib/authpds-nyu/session/url_handling.rb

@@ -1,7 +1,7 @@
 module AuthpdsNyu
   module Session
     module CoreAttributes
-      def logout_url
+      def logout_url *args
         "#{pds_url}/logout?url=#{CGI::escape(user_session_redirect_url(redirect_logout_url))}"
       end
     end

data/lib/authpds-nyu/version.rb

@@ -1,3 +1,3 @@
 module AuthpdsNyu
-  VERSION = "0.2.4"
+  VERSION = "0.2.5"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: authpds-nyu
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-13 00:00:00.000000000 Z
+date: 2012-12-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: require_all
@@ -172,7 +172,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 4056362359527365106
+      hash: -1766004451307924140
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -181,7 +181,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 4056362359527365106
+      hash: -1766004451307924140
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.24