From: Raphaël Barrois <raphael.barrois@polytechnique.org>
Date: Thu, 4 Feb 2010 18:02:09 +0000 (+0100)
Subject: Don't fetch data until needed
X-Git-Tag: xorg/1.0.0~332^2~254
X-Git-Url: http://git.polytechnique.org/?a=commitdiff_plain;h=9ed7f5bcaa0b9b264ef005baf55b3511a4327c69;p=platal.git

Don't fetch data until needed

Signed-off-by: Raphaël Barrois <raphael.barrois@polytechnique.org>
---

diff --git a/include/directory.enums.inc.php b/include/directory.enums.inc.php
index ad7d694..538c8d2 100644
--- a/include/directory.enums.inc.php
+++ b/include/directory.enums.inc.php
@@ -114,7 +114,7 @@ abstract class DirEnumeration
 
     /** An internal array of ID => optionTxt
      */
-    protected $options;
+    protected $options = null;
 
     /** Description of the MySQL storage of the fields
      */
@@ -134,12 +134,16 @@ abstract class DirEnumeration
     protected $ac_distinct = true; // Whether we want to keep only distinct valfield value
     protected $ac_withid = true; // Do we want to fetch id too ?
 
-    public function __construct() {
-        $this->loadOptions();
+    protected function _fetchOptions()
+    {
+        if (is_null($this->options)) {
+            $this->loadOptions();
+        }
     }
 
     public function getOptions()
     {
+        $this->_fetchOptions();
         return $this->options;
     }
 
@@ -310,6 +314,7 @@ class DE_Degrees extends DirEnumeration
 
     public function getOptions($eduid = null)
     {
+        $this->_fetchOptions();
         if ($eduid == null) {
             return PlIteratorUtils::fromArray($this->options, 1, true);
         }
@@ -397,6 +402,8 @@ class DE_AdminAreas extends DirEnumeration
 
     public function getOptions($country = null)
     {
+        $this->_fetchOptions();
+
         if ($country == null) {
             return PlIteratorUtils::fromArray($this->options, 1, true);
         }