Fix an "always OSUL" thinko
authorDan Scott <dscott@laurentian.ca>
Mon, 28 Sep 2015 19:41:54 +0000 (15:41 -0400)
committerDan Scott <dscott@laurentian.ca>
Mon, 28 Sep 2015 19:41:54 +0000 (15:41 -0400)
A conditional that always returns true but looks conditional is a little
nasty!

Signed-off-by: Dan Scott <dscott@laurentian.ca>
tools/patron-load/ldap_osrf_sync

index 3d9245c..3796c28 100755 (executable)
@@ -204,7 +204,7 @@ class User:
         Map LDAP record to Evergreen home library
         """
 
-        if 'laurentian.ca' or 'laurentienne.ca' in self.email:
+        if 'laurentian.ca' in self.email or 'laurentienne.ca' in self.email:
             return 103
         elif 'huntingtonu.ca' in self.email:
             return 104