Skip to content
This repository has been archived by the owner on Feb 14, 2019. It is now read-only.

Don't work with Redmine revision 9241 and later #30

Closed
atsapov opened this issue Apr 4, 2012 · 1 comment
Closed

Don't work with Redmine revision 9241 and later #30

atsapov opened this issue Apr 4, 2012 · 1 comment
Assignees
@thorin
Labels
bug

Comments

@atsapov
Copy link

atsapov commented Apr 4, 2012

After updating the Redmine to revision 9241 I've get error

A ArgumentError occurred in account#login:

 wrong number of arguments (1 for 2)
 [RAILS_ROOT]/vendor/plugins/redmine_ldap_sync/lib/redmine_ldap_sync/redmine_ext/auth_source_ldap_patch.rb:121:in 'get_user_dn'

When I change 121 line:
attrs = get_user_dn(user.login)
by:
attrs = get_user_dn(user.login, "")
error is gone.
@ghost ghost assigned thorin Apr 4, 2012
thorin added a commit that referenced this issue Apr 4, 2012
@thorin
added support for revision 9241 (#30)
f5e15ac
@thorin
Copy link
Owner

thorin commented Apr 4, 2012

I should be fixed.

Feel free reopen the ticket if not.

@thorin thorin closed this as completed Apr 4, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@thorin thorin

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@thorin @atsapov