From 1335d2008edddcb482f19714c13afdc715364d26 Mon Sep 17 00:00:00 2001
From: Andrew McMillan <andrew@morphoss.com>
Date: Fri, 25 Sep 2009 14:41:32 +1200
Subject: [PATCH] More tweaks to the new delete user option.

---
 htdocs/usr.php      | 5 ++++-
 inc/DAViCalUser.php | 4 ++--
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/htdocs/usr.php b/htdocs/usr.php
index 41655800..29647033 100644
--- a/htdocs/usr.php
+++ b/htdocs/usr.php
@@ -10,6 +10,9 @@
 
   $user_no = intval(isset($_POST['user_no']) ? $_POST['user_no'] : (isset($_GET['user_no'])?$_GET['user_no']:0) );
   $user = new DAViCalUser($user_no);
+  if ( 'insert' == $user->WriteType && $user->user_no > 0 ) {
+    $user = new DAViCalUser(0);
+  }
   if ( $user->user_no == 0 ) {
     $c->page_title = ( $user_no != "" ? translate("User Unavailable") : translate("New User") );
   }
@@ -53,7 +56,7 @@
 
   if ( $user->user_no > 0 ) {
     if ( $user->AllowedTo('update') && ! $user->EditMode ) {
-    	$user_menu->AddOption( sprintf(translate("Edit %s"), $user->Values->fullname), "$c->base_url/usr.php?user_no=$user->user_no&edit=1", translate("Edit this user record"), true, 900 );
+    	$user_menu->AddOption( sprintf(translate("Edit %s"), $user->Get('fullname')), "$c->base_url/usr.php?user_no=$user->user_no&edit=1", translate("Edit this user record"), true, 900 );
     }
     else {
       $user_menu->AddOption( sprintf(translate("View %s"), $user->Get('fullname')), "$c->base_url/usr.php?user_no=$user->user_no", translate("View this user record"), true, 900 );
diff --git a/inc/DAViCalUser.php b/inc/DAViCalUser.php
index 41b8f1df..2aff318a 100644
--- a/inc/DAViCalUser.php
+++ b/inc/DAViCalUser.php
@@ -66,7 +66,7 @@ class DAViCalUser extends User
 
     if ( isset($this->delete_user_confirmation_required) ) {
       $html .= '<p class="error">';
-      $html .= sprintf('<b>User:</b> %s: <a class="error" href="%s&%s">%s</a>', $this->Get('username'), $_SERVER['REQUEST_URI'], $this->delete_user_confirmation_required, translate('Confirm Deletion of the User') );
+      $html .= sprintf('<b>%s</b> \'%s\' <a class="error" href="%s&%s">%s</a> %s', translate('Deleting User:'), $this->Get('username'), $_SERVER['REQUEST_URI'], $this->delete_user_confirmation_required, translate('Confirm Deletion of the User'), translate('All of the user\'s calendars and events will be unrecoverably deleted.') );
       $html .= "</p>\n";
     }
 
@@ -289,7 +289,7 @@ EOSQL;
     $html = ( $title == '' ? '' : $ef->BreakLine(translate($title)) );
     if ( isset($this->delete_collection_confirmation_required) ) {
       $html .= '<tr><td colspan="2" class="error">';
-      $html .= sprintf('<b>Collection:</b> %s: <a class="error" href="%s&%s">%s</a>', $_GET['dav_name'], $_SERVER['REQUEST_URI'], $this->delete_collection_confirmation_required, translate('Confirm Deletion of the Collection') );
+      $html .= sprintf('<b>%s</b> "%s" <a class="error" href="%s&%s">%s</a> %s', translate('Deleting Collection:'), $_GET['dav_name'], $_SERVER['REQUEST_URI'], $this->delete_collection_confirmation_required, translate('Confirm Deletion of the Collection'), translate('All collection data will be unrecoverably deleted.') );
       $html .= "</td></tr>\n";
     }
     $html .= '<tr><td colspan="2">';