From 7709e53c077c4eda203f8dc8057e2fa30e126dfb Mon Sep 17 00:00:00 2001
From: Jan Schneider <jan@horde.org>
Date: Mon, 1 Nov 2010 19:45:02 +0100
Subject: [PATCH] Remove setGeoLocation()/getGeoLocation().

---
 kronolith/lib/Event.php           | 44 ++++++++++-----------------------------
 kronolith/lib/Resource/Single.php |  2 +-
 2 files changed, 12 insertions(+), 34 deletions(-)

diff --git a/kronolith/lib/Event.php b/kronolith/lib/Event.php
index 46bca8a42..f057f2e8d 100644
--- a/kronolith/lib/Event.php
+++ b/kronolith/lib/Event.php
@@ -365,7 +365,13 @@ abstract class Kronolith_Event
             }
             return $this->_tags;
         case 'geoLocation':
-            return $this->getGeolocation();
+            if (!isset($this->_geoLocation) &&
+                ($gDriver = Kronolith::getGeoDriver())) {
+                try {
+                    $this->_geoLocation = $gDriver->getLocation($this->id);
+                } catch (Exception $e) {}
+            }
+            return $this->_geoLocation;
         }
 
         $trace = debug_backtrace();
@@ -2341,13 +2347,13 @@ abstract class Kronolith_Event
         }
 
         // Tags.
-        $this->_tags = Horde_Util::getFormData('tags', $this->tags);
+        $this->tags = Horde_Util::getFormData('tags', $this->tags);
 
         // Geolocation
         if (Horde_Util::getFormData('lat') && Horde_Util::getFormData('lon')) {
-            $this->setGeoLocation(array('lat' => Horde_Util::getFormData('lat'),
-                                        'lon' => Horde_Util::getFormData('lon'),
-                                        'zoom' => Horde_Util::getFormData('zoom')));
+            $this->geoLocation = array('lat' => Horde_Util::getFormData('lat'),
+                                       'lon' => Horde_Util::getFormData('lon'),
+                                       'zoom' => Horde_Util::getFormData('zoom'));
         }
 
         $this->initialized = true;
@@ -2771,34 +2777,6 @@ abstract class Kronolith_Event
         return 'kronolithEvent';
     }
 
-    /**
-     * Setter for geo data
-     *
-     * @param array $data  An array of lat/lng data.
-     */
-    public function setGeoLocation($data)
-    {
-        $this->_geoLocation = $data;
-    }
-
-    /**
-     * Getter for geo data
-     *
-     * @return array  An array of lat/lng data.
-     */
-    public function getGeolocation()
-    {
-        /* Get geolocation data */
-        if (!isset($this->_geoLocation) &&
-            ($gDriver = Kronolith::getGeoDriver())) {
-            try {
-                $this->_geoLocation = $gDriver->getLocation($this->id);
-            } catch (Exception $e) {}
-        }
-
-        return $this->_geoLocation;
-    }
-
     private function _formIDEncode($id)
     {
         return str_replace(array('[', ']'),
diff --git a/kronolith/lib/Resource/Single.php b/kronolith/lib/Resource/Single.php
index 8fe826304..2ac8f0402 100644
--- a/kronolith/lib/Resource/Single.php
+++ b/kronolith/lib/Resource/Single.php
@@ -157,7 +157,7 @@ class Kronolith_Resource_Single extends Kronolith_Resource_Base
         $to->description = $from->description;
         $to->url = $from->url;
         $to->tags = $from->tags;
-        $to->setGeoLocation($from->geoLocation);
+        $to->geoLocation = $from->geoLocation;
         $to->first = $from ->first;
         $to->last = $from->last;
         $to->start = $from->start;
-- 
2.11.0