merge from cvs:

[mjr] Do not remove history entries when removing user data (Bug: 8755).

diff --git a/kronolith/docs/CHANGES b/kronolith/docs/CHANGES
index 940f837..960f782 100644 (file)
--- a/kronolith/docs/CHANGES
+++ b/kronolith/docs/CHANGES
@@ -32,7 +32,7 @@ v3.0-git
 v2.3.5-cvs
 ----------
 
-
+[mjr] Do not remove history entries when removing user data (Bug: 8755).
 
 
 ------

diff --git a/kronolith/lib/Application.php b/kronolith/lib/Application.php
index 5b4be45..2eec8bc 100644 (file)
--- a/kronolith/lib/Application.php
+++ b/kronolith/lib/Application.php
@@ -437,9 +437,6 @@ class Kronolith_Application extends Horde_Registry_Application
         /* Remove all events owned by the user in all calendars. */
         $result = Kronolith::getDriver()->removeUserData($user);
 
-        /* Now delete history as well. */
-        $GLOBALS['injector']->getInstance('Horde_History')->removeByParent('kronolith:' . $user);
-
         /* Get the user's default share */
         try {
             $share = $GLOBALS['kronolith_shares']->getShare($user);

diff --git a/mnemo/docs/CHANGES b/mnemo/docs/CHANGES
index 4b2035d..ff3a25b 100644 (file)
--- a/mnemo/docs/CHANGES
+++ b/mnemo/docs/CHANGES
@@ -10,7 +10,7 @@ v3.0-cvs
 v2.2.4-cvs
 ----------
 
-
+[mjr] Do not remove history entries when removing user data (Bug: 8755).
 
 ------
 v2.2.3

diff --git a/mnemo/lib/Api.php b/mnemo/lib/Api.php
index 9b9eacc..0b5d2a2 100644 (file)
--- a/mnemo/lib/Api.php
+++ b/mnemo/lib/Api.php
@@ -126,23 +126,6 @@ function _mnemo_removeUserData($user)
             }
         }
 
-        /* Now delete history as well. */
-        $history = $GLOBALS['injector']->getInstance('Horde_History');
-        if (method_exists($history, 'removeByParent')) {
-            $histories = $history->removeByParent('mnemo:' . $user);
-        } else {
-            /* Remove entries 100 at a time. */
-            $all = $history->getByTimestamp('>', 0, array(), 'mnemo:' . $user);
-            if (is_a($all, 'PEAR_Error')) {
-                Horde::logMessage($all, 'ERR');
-            } else {
-                $all = array_keys($all);
-                while (count($d = array_splice($all, 0, 100)) > 0) {
-                    $history->removebyNames($d);
-                }
-            }
-        }
-
         /* Remove the share itself */
         if (!empty($share)) {
             $result = $GLOBALS['mnemo_shares']->removeShare($share);

diff --git a/nag/docs/CHANGES b/nag/docs/CHANGES
index 5a06533..aa96666 100644 (file)
--- a/nag/docs/CHANGES
+++ b/nag/docs/CHANGES
@@ -15,7 +15,7 @@ v3.0-git
 v2.3.6-cvs
 ----------
 
-
+[mjr] Do not remove history entries when removing user data (Bug: 8755).
 
 
 ------

diff --git a/nag/lib/Application.php b/nag/lib/Application.php
index 36f154b..0205bfb 100644 (file)
--- a/nag/lib/Application.php
+++ b/nag/lib/Application.php
@@ -266,9 +266,6 @@ class Nag_Application extends Horde_Registry_Application
             }
         }
 
-        /* Now delete history as well. */
-        $GLOBALS['injector']->getInstance('Horde_History')->removeByParent('nag:' . $user);
-
         /* ...and finally, delete the actual share */
         if (!empty($share)) {
             $result = $GLOBALS['nag_shares']->removeShare($share);