Add a groups api method on the Identity provider and change access_Core::_get_all_groups() to use this to get the defined groups.

2 files changed, 14 insertions, 2 deletions

diff --git a/modules/gallery/libraries/Identity.php b/modules/gallery/libraries/Identity.php
index 3fcb6756..3ef13a6d 100644
--- a/modules/gallery/libraries/Identity.php
+++ b/modules/gallery/libraries/Identity.php
@@ -65,11 +65,11 @@ class Identity_Core {
    * @return  void
    */
   public function __construct() {
-    $name = $config = module::get_var("gallery", "identity_provider", "user");
+    $config = module::get_var("gallery", "identity_provider", "user");
 
     // Test the config group name
     if (($this->config = Kohana::config("identity.".$config)) === NULL) {
-      throw new Exception("@todo NO USER LIBRARY CONFIGURATION FOR: $name");
+      throw new Exception("@todo NO USER LIBRARY CONFIGURATION FOR: $config");
     }
 
     // Set driver name
@@ -205,4 +205,11 @@ class Identity_Core {
   static function get_user_list($ids) {
     return self::instance()->driver->get_user_list($ids);
   }
+
+  /**
+   * @see Identity_Driver::groups.
+   */
+  static function groups() {
+    return self::instance()->driver->groups();
+  }
 } // End Identity
diff --git a/modules/gallery/libraries/drivers/Identity.php b/modules/gallery/libraries/drivers/Identity.php
index 2fc4d349..b77eb6be 100644
--- a/modules/gallery/libraries/drivers/Identity.php
+++ b/modules/gallery/libraries/drivers/Identity.php
@@ -97,6 +97,11 @@ interface Identity_Driver {
    */
   public function get_user_list($ids);
 
+  /**
+   * List the groups defined in the Identity Provider
+   */
+  static function groups();
+
 } // End Identity Driver Definition
 
 interface Group_Definition {}