Use update_printer query, rather than implementing updates as a

delete_printer followed by an add_printer.

diff --git a/clients/moira/printer.c b/clients/moira/printer.c
index 983305c620c9b28c68b2a03a973e108e54ac9c09..12d888ffedf2ccd8e054a9cdb07738aff54260c1 100644 (file)
--- a/clients/moira/printer.c
+++ b/clients/moira/printer.c
@@ -203,10 +203,10 @@ static char *PrintPrnInfo(char **info)
  *     Returns: none.
  */
 
-static char **AskPrnInfo(char **info)
+static char **AskPrnInfo(char **info, Bool name)
 {
   char temp_buf[BUFSIZ];
-  char *args[3], *lpc_acl;
+  char *args[3], *lpc_acl, *newname;
   int status;
 
   Put_message("");
@@ -214,6 +214,18 @@ static char **AskPrnInfo(char **info)
   Put_message(temp_buf);
   Put_message("");
 
+  if (name)
+    {
+      while (1)
+       {
+         newname = strdup(info[PRN_NAME]);
+         if (GetValueFromUser("The new name for this printer? ", &newname) ==
+             SUB_ERROR)
+           return NULL;
+         if (ValidName(newname))
+           break;
+       }
+    }
   if (GetTypeFromUser("Type of Printer", "printertype", &info[PRN_TYPE]) ==
       SUB_ERROR)
     return NULL;
@@ -267,6 +279,9 @@ static char **AskPrnInfo(char **info)
   FreeAndClear(&info[PRN_MODBY], TRUE);
   FreeAndClear(&info[PRN_MODWITH], TRUE);
 
+  if (name)
+    SlipInNewName(info, newname);
+
   return info;
 }
 
@@ -399,7 +414,7 @@ int AddPrn(int argc, char **argv)
       return DM_NORMAL;
     }
 
-  args = AskPrnInfo(SetDefaults(info, argv[1]));
+  args = AskPrnInfo(SetDefaults(info, argv[1]), FALSE);
   if (!args)
     {
       Put_message("Aborted.");
@@ -426,25 +441,16 @@ int AddPrn(int argc, char **argv)
 void ChangePrn(char **info, Bool one_item)
 {
   int stat;
-  char **oldinfo;
 
-  oldinfo = CopyInfo(info);
-  if (!AskPrnInfo(info))
-    return;
-  if ((stat = do_mr_query("delete_printer", 1, &info[PRN_NAME], NULL, NULL)))
+  if (!AskPrnInfo(info, TRUE))
     {
-      com_err(program_name, stat, " printer not updated.");
+      Put_message("Aborted.");
       return;
     }
-  if ((stat = do_mr_query("add_printer", CountArgs(info), info, NULL, NULL)))
-    {
-      com_err(program_name, stat, " in ChngPrn");
-      if ((stat = do_mr_query("add_printer", CountArgs(oldinfo) - 3,
-                             oldinfo, NULL, NULL)))
-       com_err(program_name, stat, " while attempting to put old info back");
-    }
-  FreeInfo(oldinfo);
-  return;
+  if ((stat = do_mr_query("update_printer", CountArgs(info), info, NULL, NULL)))
+    com_err(program_name, stat, " in ChngPrn");
+  else
+    Put_message("Printer successfully updated.");
 }