Fixing overly aggressive removal of json::details::_Value::_copy_value API.

Author: stgates

Release/include/cpprest/json.h

@@ -1418,6 +1418,8 @@ namespace json
         class _Value
         {
         public:
+            virtual std::unique_ptr<_Value> _copy_value() = 0;
+
             virtual bool has_field(const utility::string_t &) const { return false; }
             virtual value get_field(const utility::string_t &) const { throw json_exception(_XPLATSTR("not an object")); }
             virtual value get_element(array::size_type) const { throw json_exception(_XPLATSTR("not an array")); }
@@ -1487,6 +1489,10 @@ namespace json
         class _Null : public _Value
         {
         public:
+            virtual std::unique_ptr<_Value> _copy_value()
+            {
+                return utility::details::make_unique<_Null>();
+            }
             virtual json::value::value_type type() const { return json::value::Null; }
         };
 
@@ -1499,6 +1505,11 @@ namespace json
             _Number(int64_t value) : m_number(value) { }
             _Number(uint64_t value) : m_number(value) { }
 
+            virtual std::unique_ptr<_Value> _copy_value()
+            {
+                return utility::details::make_unique<_Number>(*this);
+            }
+
             virtual json::value::value_type type() const { return json::value::Number; }
 
             virtual bool is_integer() const { return m_number.is_integral(); }
@@ -1532,6 +1543,11 @@ namespace json
         public:
             _Boolean(bool value) : m_value(value) { }
 
+            virtual std::unique_ptr<_Value> _copy_value()
+            {
+                return utility::details::make_unique<_Boolean>(*this);
+            }
+
             virtual json::value::value_type type() const { return json::value::Boolean; }
 
             virtual bool as_bool() const { return m_value; }
@@ -1577,6 +1593,11 @@ namespace json
             { }
 #endif
 
+            virtual std::unique_ptr<_Value> _copy_value()
+            {
+                return utility::details::make_unique<_String>(*this);
+            }
+
             virtual json::value::value_type type() const { return json::value::String; }
 
             virtual const utility::string_t & as_string() const;
@@ -1643,6 +1664,11 @@ namespace json
             _Object(bool keep_order) : m_object(keep_order) { }
             _Object(object::storage_type fields, bool keep_order) : m_object(std::move(fields), keep_order) { }
 
+            virtual std::unique_ptr<_Value> _copy_value()
+            {
+                return utility::details::make_unique<_Object>(*this);
+            }
+
             virtual json::object& as_object() { return m_object; }
 
             virtual const json::object& as_object() const { return m_object; }
@@ -1749,6 +1775,11 @@ namespace json
             _Array(array::size_type size) : m_array(size) {}
             _Array(array::storage_type elements) : m_array(std::move(elements)) { }
 
+            virtual std::unique_ptr<_Value> _copy_value()
+            {
+                return utility::details::make_unique<_Array>(*this);
+            }
+
             virtual json::value::value_type type() const { return json::value::Array; }
 
             virtual json::array& as_array() { return m_array; }

Release/src/json/json.cpp

@@ -123,7 +123,7 @@ m_value(utility::details::make_unique<web::json::details::_String>(utility::stri
 { }
 
 web::json::value::value(const value &other) :
-    m_value(other.m_value.get())
+    m_value(other.m_value->_copy_value())
 #ifdef ENABLE_JSON_VALUE_VISUALIZER
     ,m_kind(other.m_kind)
 #endif
@@ -133,7 +133,7 @@ web::json::value &web::json::value::operator=(const value &other)
 {
     if(this != &other)
     {
-        m_value = std::unique_ptr<details::_Value>(other.m_value.get());
+        m_value = std::unique_ptr<details::_Value>(other.m_value->_copy_value());
 #ifdef ENABLE_JSON_VALUE_VISUALIZER
         m_kind = other.m_kind;
 #endif