diff options
| -rw-r--r-- | app/models/auth_source_ldap.rb | 8 | ||||
| -rw-r--r-- | test/unit/auth_source_ldap_test.rb | 2 |
2 files changed, 9 insertions, 1 deletions
diff --git a/app/models/auth_source_ldap.rb b/app/models/auth_source_ldap.rb index 91942b282..92bcd4d65 100644 --- a/app/models/auth_source_ldap.rb +++ b/app/models/auth_source_ldap.rb @@ -29,6 +29,14 @@ class AuthSourceLdap < AuthSource before_validation :strip_ldap_attributes + def self.human_attribute_name(attribute_key_name, *args) + attr_name = attribute_key_name.to_s + if attr_name == "filter" + attr_name = "ldap_filter" + end + super(attr_name, *args) + end + def initialize(attributes=nil, *args) super self.port = 389 if self.port == 0 diff --git a/test/unit/auth_source_ldap_test.rb b/test/unit/auth_source_ldap_test.rb index 4955a4dc8..1662042b2 100644 --- a/test/unit/auth_source_ldap_test.rb +++ b/test/unit/auth_source_ldap_test.rb @@ -51,7 +51,7 @@ class AuthSourceLdapTest < ActiveSupport::TestCase a = AuthSourceLdap.new(:name => 'My LDAP', :host => 'ldap.example.net', :port => 389, :attr_login => 'sn') a.filter = "(mail=*@redmine.org" assert !a.valid? - assert_include "is invalid", a.errors[:filter] + assert_include "LDAP filter is invalid", a.errors.full_messages a.filter = "(mail=*@redmine.org)" assert a.valid? |