From 3eb4e6920daa5da441f06f4894ea9cdad72de6d7 Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Mon, 11 Jan 2010 11:03:34 -0800
Subject: [PATCH] Implement Ldap_User::groups() to match K24 APIs.

---
 modules/ldap/libraries/drivers/IdentityProvider/Ldap.php | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/modules/ldap/libraries/drivers/IdentityProvider/Ldap.php b/modules/ldap/libraries/drivers/IdentityProvider/Ldap.php
index c3eb1dd1..69440199 100644
--- a/modules/ldap/libraries/drivers/IdentityProvider/Ldap.php
+++ b/modules/ldap/libraries/drivers/IdentityProvider/Ldap.php
@@ -239,9 +239,6 @@ class Ldap_User implements User_Definition {
     case "id":
       return $this->ldap_entry["uidnumber"][0];
 
-    case "groups":
-      return IdentityProvider_Ldap_Driver::groups_for($this);
-
     case "locale":  // @todo
       return null;
 
@@ -266,6 +263,10 @@ class Ldap_User implements User_Definition {
     }
   }
 
+  public function groups() {
+      return IdentityProvider_Ldap_Driver::groups_for($this);
+  }
+
   public function avatar_url($size=80, $default=null) {
     return sprintf("http://www.gravatar.com/avatar/%s.jpg?s=%d&r=pg%s",
                    md5($this->email), $size, $default ? "&d=" . urlencode($default) : "");