RE: [Ruqueue-devel] bulk add users?

[Montana Quiring]

I was tinkering with those settings too and your email confirmed I was on
the right tack, but still no joy.
Here's what I did:

Add_user.php:
Line 91
Changed: if ((empty($iid)) || (!(empty($id)))) {
To: if ((empty($uid)) || (!(empty($id)))) {

Line 98
Changed:   $ldap_query = "iid=$iid";
To:   $ldap_query = "iid=$uid";
Also tried:   $ldap_query = "uid=$uid";
Also tried:   $ldap_query = "uid=$iid";


Could it be I need to change something in row62?:
   $user_row = GetFirstRowFromTable("user where id = '$id'");

Also.. I'm thinking I may have to tweak:
functions-ldap.php

I'll keep working at it, if you have any other suggestions, please let me
know.

My guess is that $id becomes $iid and I need to change $id to become $uid

-Montana Quiring


-----Original Message-----
From: address@hidden [mailto:address@hidden On Behalf Of John
Fulton
Sent: Wednesday, November 01, 2006 8:17 PM
To: Montana Quiring
Cc: address@hidden
Subject: Re: [Ruqueue-devel] bulk add users?

Based on what you said, I think I know what the issue is.  The "add by
last name" uses the iid in the URL.  So if you mouse over the "click
here", you should see something like:

add_user.php?iid=

such that the iid is empty, and thus when it gets to the next page
there is no iid in the URL and the lookup is unable to move forward
since it has nothing to do the search on.  I recommend modifying the
add_user.php script to use the uid instead of the iid.  This shouldnt'
be too hard.  Look for the "if ((empty($iid)) || (!(empty($id)))) {"
in the code below (near the end):

http://cvs.savannah.nongnu.org/viewcvs/ruqueue/ruqueue/web-interface/add_use
r.php?rev=1.2&root=ruqueue&view=markup

If you modify this to use the uid instead and then modify the code
that does the LDAP search to use the uid as well, you should be done
with modifying add_user.php.  Then you just need to modify
lookup_name.php to display the uid instead of the iid when you hover
the mouse over the "click here".  You might as well change the IID in
the table above to UID too.

Thanks for helping me understand what your issue was.  Since the IID
is a Rutgers thing (yet another unique identifier) this change could
probably be made to ruQueue to make it more general in the LDAP sense.
 It also shouldn't break how it works in theory since it just needs a
uniqe identifier to search on.

  John

On 11/1/06, Montana Quiring <address@hidden> wrote:

> Sorry. I sent this directly to John and forgot to send it to the list.
>
> Hmmm.. It does pull up the appropriate results (via LDAP) when I do a
"Name" search. It's just that when I click "click here" under "Add this
user" it doesn't populate the fields.
>
> I noticed that the iid field is empty in the LDAP results... could that be
the problem?
>
>
> There's no iid field in the actual LDAP entries but there is a uid.
>
> -Montana Quiring
> _______________________________________________
> Ruqueue-devel mailing list
> address@hidden
> http://lists.nongnu.org/mailman/listinfo/ruqueue-devel
>
-- 
http://jfulton.org