Hi again,

On 3/28/07, Kristis Makris <[EMAIL PROTECTED]> wrote:
> > How do I debug this further?  This is against bugzilla 2.20.

Dmitri, thanks SO MUCH for looking into this.

Hey - you want it to work, I want it to work, win-win.  (o:

[perl stuff snipped]

Thanks for the debug help.

My guess is that Bugzilla.pm.in:integration_add_comment falls to the
case where ($userid <= 0):

       if ( $userid > 0 ) {
           bugzilla_2_20_fixed_AppendComment( $bugid, $userid, $comment );
           return 0;
       } else {
           # This should never happen. Each user should have a
           # corresponding userid in the database schema
           return 1;
       }

*oops*.... In an effort to get more debug I had changed that (and
broken it).  Fixed now.

Somehow the userid supplied does not map to a userid in Bugzilla.

Please debug this for us!! It seems to be a real bug that we haven't
fixed yet.

Nup... My problem was that I _do_ need the username translation.  I
don't see a way to do what I need with the current config.  Someone
has hacked our bugzilla impl so that we have firstname.lastname as our
usernames in the profiles table.  This means I can't use the default
translation impl and so I have changed Daemon.pm

           #$request->{ translated_username } =
$LDAP_entry->get_value( $LDAP_bugtracking_username_attribute );
           $request->{ translated_username } =
$LDAP_entry->get_value( "givenName" ) . "." . $LDAP_entry->get_value(
"sn" );

Which now works for me.  Thanks a heap for your help.  I will
endeavour to uninstall and reinstall so that I can confirm the error
messages (without my code breaking debugging efforts).  I suspect the
initial problem was really one of misconfiguration on my part - a
better message for the situation where the username doesn't exist in
bugzilla would be nice, I will look into this and try to provide a
patch.

cheers
dim
_______________________________________________
scmbug-users mailing list
[email protected]
http://lists.mkgnu.net/cgi-bin/mailman/listinfo/scmbug-users

Reply via email to