From 892e75ca6fd36de7af3e9f52340afe47aca97306 Mon Sep 17 00:00:00 2001
From: David Anderson <davea@ssl.berkeley.edu>
Date: Wed, 11 Nov 2009 00:15:25 +0000
Subject: [PATCH] - client: the handling of GUI RPCs was inconsistent;     some
 expected <request>, some expected <request/>,     and some would accept
 either.     Changed them all to accept either.

svn path=/trunk/boinc/; revision=19535
---
 checkin_notes                 | 12 ++++++++++++
 client/gui_rpc_server_ops.cpp | 18 +++++++++---------
 2 files changed, 21 insertions(+), 9 deletions(-)

diff --git a/checkin_notes b/checkin_notes
index 0f9d38687e..7cf00614ab 100644
--- a/checkin_notes
+++ b/checkin_notes
@@ -9033,3 +9033,15 @@ David  10 Nov 2009
 
     lib/
         coproc.cpp
+
+David  10 Nov 2009
+    - client: the handling of GUI RPCs was inconsistent;
+        some expected <request>, some expected <request/>,
+        and some would accept either.
+        Changed them all to accept either.
+
+        NOTE: this means no request can be a substring of another
+
+    client/
+        gui_rpc_server_ops.cpp
+
diff --git a/client/gui_rpc_server_ops.cpp b/client/gui_rpc_server_ops.cpp
index ce6428cc7c..b02e9b0e92 100644
--- a/client/gui_rpc_server_ops.cpp
+++ b/client/gui_rpc_server_ops.cpp
@@ -1144,11 +1144,11 @@ int GUI_RPC_CONN::handle_rpc() {
         handle_get_host_info(request_msg, mf);
     } else if (match_tag(request_msg, "<get_statistics")) {
         handle_get_statistics(request_msg, mf);
-    } else if (match_tag(request_msg, "<get_newer_version>")) {
+    } else if (match_tag(request_msg, "<get_newer_version")) {
         handle_get_newer_version(mf);
     } else if (match_tag(request_msg, "<get_cc_status")) {
         handle_get_cc_status(this, mf);
-    } else if (match_tag(request_msg, "<get_all_projects_list/>")) {
+    } else if (match_tag(request_msg, "<get_all_projects_list")) {
         read_all_projects_list_file(mf);
 
     // Operations that require authentication start here
@@ -1197,7 +1197,7 @@ int GUI_RPC_CONN::handle_rpc() {
         handle_quit(request_msg, mf);
     } else if (match_tag(request_msg, "<acct_mgr_info")) {
         handle_acct_mgr_info(request_msg, mf);
-    } else if (match_tag(request_msg, "<read_global_prefs_override/>")) {
+    } else if (match_tag(request_msg, "<read_global_prefs_override")) {
         mf.printf("<success/>\n");
         gstate.read_global_prefs();
         gstate.request_schedule_cpus("Preferences override");
@@ -1216,7 +1216,7 @@ int GUI_RPC_CONN::handle_rpc() {
         handle_get_cc_config(mf);
     } else if (match_tag(request_msg, "<set_cc_config")) {
         handle_set_cc_config(request_msg, mf);
-    } else if (match_tag(request_msg, "<read_cc_config/>")) {
+    } else if (match_tag(request_msg, "<read_cc_config")) {
         mf.printf("<success/>\n");
         read_config_file(false);
         msg_printf(0, MSG_INFO, "Re-read config file");
@@ -1243,23 +1243,23 @@ int GUI_RPC_CONN::handle_rpc() {
             handle_project_op(request_msg, mf, "reset");
         } else if (match_tag(request_msg, "<project_update")) {
             handle_project_op(request_msg, mf, "update");
-        } else if (match_tag(request_msg, "<get_project_config>")) {
+        } else if (match_tag(request_msg, "<get_project_config")) {
             handle_get_project_config(request_msg, mf);
         } else if (match_tag(request_msg, "<get_project_config_poll")) {
             handle_get_project_config_poll(request_msg, mf);
-        } else if (match_tag(request_msg, "<lookup_account>")) {
+        } else if (match_tag(request_msg, "<lookup_account")) {
             handle_lookup_account(request_msg, mf);
         } else if (match_tag(request_msg, "<lookup_account_poll")) {
             handle_lookup_account_poll(request_msg, mf);
-        } else if (match_tag(request_msg, "<create_account>")) {
+        } else if (match_tag(request_msg, "<create_account")) {
             handle_create_account(request_msg, mf);
         } else if (match_tag(request_msg, "<create_account_poll")) {
             handle_create_account_poll(request_msg, mf);
-        } else if (match_tag(request_msg, "<project_attach>")) {
+        } else if (match_tag(request_msg, "<project_attach")) {
             handle_project_attach(request_msg, mf);
         } else if (match_tag(request_msg, "<project_attach_poll")) {
             handle_project_attach_poll(request_msg, mf);
-        } else if (match_tag(request_msg, "<acct_mgr_rpc>")) {
+        } else if (match_tag(request_msg, "<acct_mgr_rpc")) {
             handle_acct_mgr_rpc(request_msg, mf);
         } else if (match_tag(request_msg, "<acct_mgr_rpc_poll")) {
             handle_acct_mgr_rpc_poll(request_msg, mf);