From 49cdad1af8ba2178e26c0779b38dc76c373c5f5c Mon Sep 17 00:00:00 2001
From: Scott Schurr <scott@ripple.com>
Date: Thu, 26 Jan 2023 13:19:23 -0800
Subject: [PATCH] Fix a case where ripple::Expected returned a json array, not
 a value

---
 src/ripple/basics/Expected.h      |  6 +++---
 src/test/basics/Expected_test.cpp | 13 +++++++++++++
 2 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/src/ripple/basics/Expected.h b/src/ripple/basics/Expected.h
index 8dc368eefd7..bb699579b17 100644
--- a/src/ripple/basics/Expected.h
+++ b/src/ripple/basics/Expected.h
@@ -137,14 +137,14 @@ class [[nodiscard]] Expected
 public:
     template <typename U>
     requires std::convertible_to<U, T> constexpr Expected(U && r)
-        : Base(T{std::forward<U>(r)})
+        : Base(T(std::forward<U>(r)))
     {
     }
 
     template <typename U>
         requires std::convertible_to<U, E> &&
         (!std::is_reference_v<U>)constexpr Expected(Unexpected<U> e)
-        : Base(E{std::move(e.value())})
+        : Base(E(std::move(e.value())))
     {
     }
 
@@ -220,7 +220,7 @@ class [[nodiscard]] Expected<void, E>
     template <typename U>
         requires std::convertible_to<U, E> &&
         (!std::is_reference_v<U>)constexpr Expected(Unexpected<U> e)
-        : Base(E{std::move(e.value())})
+        : Base(E(std::move(e.value())))
     {
     }
 
diff --git a/src/test/basics/Expected_test.cpp b/src/test/basics/Expected_test.cpp
index 1f16e724de6..b89b9f6d309 100644
--- a/src/test/basics/Expected_test.cpp
+++ b/src/test/basics/Expected_test.cpp
@@ -20,6 +20,9 @@
 #include <ripple/basics/Expected.h>
 #include <ripple/beast/unit_test.h>
 #include <ripple/protocol/TER.h>
+#if BOOST_VERSION >= 107500
+#include <boost/json.hpp>  // Not part of boost before version 1.75
+#endif                     // BOOST_VERSION
 #include <array>
 #include <cstdint>
 
@@ -203,6 +206,16 @@ struct Expected_test : beast::unit_test::suite
             std::string const s(std::move(expected.error()));
             BEAST_EXPECT(s == "Not what is expected!");
         }
+        // Test a case that previously unintentionally returned an array.
+#if BOOST_VERSION >= 107500
+        {
+            auto expected = []() -> Expected<boost::json::value, std::string> {
+                return boost::json::object{{"oops", "me array now"}};
+            }();
+            BEAST_EXPECT(expected);
+            BEAST_EXPECT(!expected.value().is_array());
+        }
+#endif  // BOOST_VERSION
     }
 };