Merge pull request #2466 from sul-dlss/patronGroup

Apparently patron groups are optional in FOLIO
sul-dlss · May 14, 2024 · bc2b0f0 · bc2b0f0
2 parents 54d4e1b + 7bf9712
commit bc2b0f0
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/app/models/folio/patron.rb b/app/models/folio/patron.rb
@@ -40,12 +40,12 @@ def username
 
     def patron_group_id
       # FOLIO APIs return a UUID here
-      user_info.fetch('patronGroup')
+      user_info['patronGroup']
     end
 
     # this returns the full patronGroup object
     def patron_group
-      Folio::Types.patron_groups[patron_group_id]
+      Folio::Types.patron_groups[patron_group_id] if patron_group_id
     end
 
     def patron_group_name