Merge pull request #615 from felixbuenemann/patch-1

Update README LDAP instructions

Merge pull request #615 from felixbuenemann/patch-1
Update README LDAP instructions
Arthur Nogueira Neves
2 parents 569b8491 6a58cb55
Showing 1 changed file with 7 additions and 3 deletions Show diff stats
README.md
@@ -323,9 +323,13 @@ Errbit::Config.devise_modules &lt;&lt; :ldap_authenticatable
   before authentication. You must add the following lines to `app/models/user.rb`:
  
 ```ruby
-  before_save :set_ldap_email
-  def set_ldap_email
-    self.email = Devise::LdapAdapter.get_ldap_param(self.username, "mail")
+  def ldap_before_save
+    name = Devise::LDAP::Adapter.get_ldap_param(self.username, "givenName")
+    surname = Devise::LDAP::Adapter.get_ldap_param(self.username, "sn")
+    mail = Devise::LDAP::Adapter.get_ldap_param(self.username, "mail")
+
+    self.name = (name + surname).join ' '
+    self.email = mail.first
   end
 ```
...	...	@@ -323,9 +323,13 @@ Errbit::Config.devise_modules << :ldap_authenticatable
323	323	before authentication. You must add the following lines to `app/models/user.rb`:
324	324
325	325	```ruby
326		- before_save :set_ldap_email
327		- def set_ldap_email
328		- self.email = Devise::LdapAdapter.get_ldap_param(self.username, "mail")
	326	+ def ldap_before_save
	327	+ name = Devise::LDAP::Adapter.get_ldap_param(self.username, "givenName")
	328	+ surname = Devise::LDAP::Adapter.get_ldap_param(self.username, "sn")
	329	+ mail = Devise::LDAP::Adapter.get_ldap_param(self.username, "mail")
	330	+
	331	+ self.name = (name + surname).join ' '
	332	+ self.email = mail.first
329	333	end
330	334	```
331	335
...	...