Minor fixes

XRPLF · Nov 7, 2023 · bdb123d · bdb123d
1 parent f2619cb
commit bdb123d
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 11 deletions.
diff --git a/src/ripple/net/impl/RPCCall.cpp b/src/ripple/net/impl/RPCCall.cpp
@@ -1603,7 +1603,7 @@ fromCommandLine(
     Logs& logs)
 {
     auto const result =
-        rpcClient(vCmd, config, logs, RPC::apiMinimumSupportedVersion);
+        rpcClient(vCmd, config, logs, RPC::apiCommandLineVersion);
 
     std::cout << result.second.toStyledString();
 

diff --git a/src/ripple/rpc/impl/RPCHelpers.h b/src/ripple/rpc/impl/RPCHelpers.h
@@ -241,7 +241,8 @@ constexpr unsigned int apiInvalidVersion = 0;
 constexpr unsigned int apiVersionIfUnspecified = 1;
 constexpr unsigned int apiMinimumSupportedVersion = 1;
 constexpr unsigned int apiMaximumSupportedVersion = 2;
-constexpr unsigned int apiBetaVersion = 2;
+constexpr unsigned int apiCommandLineVersion = 1;  // TODO Bump to 2 later
+constexpr unsigned int apiBetaVersion = 3;
 constexpr unsigned int apiMaximumValidVersion = apiBetaVersion;
 
 static_assert(apiMinimumSupportedVersion >= apiVersionIfUnspecified);

diff --git a/src/test/jtx/Env.h b/src/test/jtx/Env.h
@@ -281,8 +281,8 @@ class Env
     */
     template <class... Args>
     Json::Value
-    rpc(std::unordered_map<std::string, std::string> const& headers,
-        unsigned apiVersion,
+    rpc(unsigned apiVersion,
+        std::unordered_map<std::string, std::string> const& headers,
         std::string const& cmd,
         Args&&... args);
 
@@ -666,8 +666,8 @@ class Env
 
     Json::Value
     do_rpc(
-        std::vector<std::string> const& args,
         unsigned apiVersion,
+        std::vector<std::string> const& args,
         std::unordered_map<std::string, std::string> const& headers = {});
 
     void
@@ -710,14 +710,14 @@ class Env
 template <class... Args>
 Json::Value
 Env::rpc(
-    std::unordered_map<std::string, std::string> const& headers,
     unsigned apiVersion,
+    std::unordered_map<std::string, std::string> const& headers,
     std::string const& cmd,
     Args&&... args)
 {
     return do_rpc(
-        std::vector<std::string>{cmd, std::forward<Args>(args)...},
         apiVersion,
+        std::vector<std::string>{cmd, std::forward<Args>(args)...},
         headers);
 }
 
@@ -726,8 +726,8 @@ Json::Value
 Env::rpc(unsigned apiVersion, std::string const& cmd, Args&&... args)
 {
     return rpc(
-        std::unordered_map<std::string, std::string>(),
         apiVersion,
+        std::unordered_map<std::string, std::string>(),
         cmd,
         std::forward<Args>(args)...);
 }
@@ -740,8 +740,8 @@ Env::rpc(
     Args&&... args)
 {
     return do_rpc(
+        RPC::apiCommandLineVersion,
         std::vector<std::string>{cmd, std::forward<Args>(args)...},
-        RPC::apiMinimumSupportedVersion,
         headers);
 }
 

diff --git a/src/test/jtx/impl/Env.cpp b/src/test/jtx/impl/Env.cpp
@@ -460,8 +460,8 @@ Env::st(JTx const& jt)
 
 Json::Value
 Env::do_rpc(
-    std::vector<std::string> const& args,
     unsigned apiVersion,
+    std::vector<std::string> const& args,
     std::unordered_map<std::string, std::string> const& headers)
 {
     return rpcClient(args, app().config(), app().logs(), apiVersion, headers)

diff --git a/src/test/rpc/LedgerRequestRPC_test.cpp b/src/test/rpc/LedgerRequestRPC_test.cpp
@@ -23,7 +23,6 @@
 #include <ripple/protocol/jss.h>
 #include <ripple/rpc/impl/RPCHelpers.h>
 #include <test/jtx.h>
-#include <test/jtx/Env.h>
 
 #include <functional>