Warren - I just looked over the code.  It looks good - except for the $user->email line in the processILS method.  We need to make sure to capture the user's email address.

We also need to refactor the user data in VuFind a bit so that we are not storing the ILS username/password.  Just some way of connecting the the user account in VuFind with the user account in the ILS.


On Mon, Dec 15, 2008 at 3:26 PM, Warren Layton <warren.layton@gmail.com> wrote:
On Mon, Dec 15, 2008 at 2:57 PM, Andrew Nagy <asnagy@gmail.com> wrote:
> As to the authentication mechanism - are you using the loginILS method in
> the vufind/web/sys/User.php class?  If so, then the configuration settings
> should go in the driver specific ini file (ie Evergreen.ini)

Yes, I'm using the loginILS method. However, it seemed like a bit of
an empty function (for instance, there was no processILSUser), unlike
loginSIP2 and loginLDAP. Also, I believe that the config.ini file has
config options for SIP2 and LDAP, but not for ILS.

I was a bit unsure if what I'm doing in loginILS is correct, so
feedback is welcome. That's actually why I submitted the driver before
I was finished, and submitted a separate patch for sys/User.php.