View Issue Details

IDProjectCategoryView StatusLast Update
0013029mantisbtldappublic2011-08-05 02:41
ReporterBascy Assigned Torombert  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionduplicate 
Product Version1.2.5 
Summary0013029: Error when LDAP search doesnt return any objects
Description

When function ldap_get_field_from_username() in ldap_api.php doesnt get any objects back from the ldap server, and error occurs online 259 stating parameter 2 cannot be null.

Additional Information

Solution:
in ldap_api.php line 259 should read


if ( $t_info[count] == 0 ) {

TagsNo tags attached.

Relationships

duplicate of 0012960 closedrombert Email / Realname lookups against LDAP spew warning when account is not in LDAP 

Activities

rombert

rombert

2011-05-26 02:32

reporter   ~0028837

Thanks for the report. Would you we willing to attach a git patch, for proper attribution on this fix? See http://docs.mantisbt.org/master/en/developers.html#DEV.CONTRIB for instructions on how to do that.

Lapinkiller

Lapinkiller

2011-05-26 03:06

reporter   ~0028838

same bug as 0012960
no ?

rombert

rombert

2011-05-26 03:12

reporter   ~0028839

Yup, that seems to be the case.

rombert

rombert

2011-05-26 03:13

reporter   ~0028840

Yup, that seems to be the case.

Bascy

Bascy

2011-05-26 03:26

reporter   ~0028841

So no git patch required anymore?

rombert

rombert

2011-05-26 03:42

reporter   ~0028842

That patch is still open, so it would be good if you could attach a patch to it.

Bascy

Bascy

2011-05-26 03:49

reporter   ~0028844

sorry, but don't have Git here, and not gonna install it.
I documented the needed change in Addition information

Lapinkiller

Lapinkiller

2011-05-26 04:04

reporter   ~0028846

Last edited: 2011-05-26 04:05

I'll try to submit a patch when I'll back to home... ;)
Here, I cannot connect to git through proxy...

rombert

rombert

2011-05-26 04:21

reporter   ~0028847

@Bas : yes, I know it's harder at first, but it might be worth it if you can submit small changes ...
@Lapinkiller: great, looking forward to your patch. Please attach it to bug 0012960 .