Store adapted string in MemberProxy

src/ArduinoJson/Document/JsonDocument.hpp

@@ -186,19 +186,21 @@ class JsonDocument : public detail::VariantOperators<const JsonDocument&> {
   // Gets or sets a root object's member.
   // https://arduinojson.org/v7/api/jsondocument/subscript/
   template <typename TString>
-  detail::enable_if_t<detail::IsString<TString>::value,
-                      detail::MemberProxy<JsonDocument&, TString>>
+  detail::enable_if_t<
+      detail::IsString<TString>::value,
+      detail::MemberProxy<JsonDocument&, detail::AdaptedString<TString>>>
   operator[](const TString& key) {
-    return {*this, key};
+    return {*this, detail::adaptString(key)};
   }
 
   // Gets or sets a root object's member.
   // https://arduinojson.org/v7/api/jsondocument/subscript/
   template <typename TChar>
-  detail::enable_if_t<detail::IsString<TChar*>::value,
-                      detail::MemberProxy<JsonDocument&, TChar*>>
+  detail::enable_if_t<
+      detail::IsString<TChar*>::value,
+      detail::MemberProxy<JsonDocument&, detail::AdaptedString<TChar*>>>
   operator[](TChar* key) {
-    return {*this, key};
+    return {*this, detail::adaptString(key)};
   }
 
   // Gets a root object's member.

src/ArduinoJson/Object/JsonObject.hpp

@@ -102,31 +102,31 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
   // Gets or sets the member with specified key.
   // https://arduinojson.org/v7/api/jsonobject/subscript/
   template <typename TString>
-  detail::enable_if_t<detail::IsString<TString>::value,
-                      detail::MemberProxy<JsonObject, TString>>
+  detail::enable_if_t<
+      detail::IsString<TString>::value,
+      detail::MemberProxy<JsonObject, detail::AdaptedString<TString>>>
   operator[](const TString& key) const {
-    return {*this, key};
+    return {*this, detail::adaptString(key)};
   }
 
   // Gets or sets the member with specified key.
   // https://arduinojson.org/v7/api/jsonobject/subscript/
   template <typename TChar>
-  detail::enable_if_t<detail::IsString<TChar*>::value,
-                      detail::MemberProxy<JsonObject, TChar*>>
+  detail::enable_if_t<
+      detail::IsString<TChar*>::value,
+      detail::MemberProxy<JsonObject, detail::AdaptedString<TChar*>>>
   operator[](TChar* key) const {
-    return {*this, key};
+    return {*this, detail::adaptString(key)};
   }
 
   // Gets or sets the member with specified key.
   // https://arduinojson.org/v7/api/jsonobject/subscript/
   template <typename TVariant>
-  detail::enable_if_t<detail::IsVariant<TVariant>::value,
-                      detail::MemberProxy<JsonObject, JsonString>>
+  detail::enable_if_t<
+      detail::IsVariant<TVariant>::value,
+      detail::MemberProxy<JsonObject, detail::AdaptedString<JsonString>>>
   operator[](const TVariant& key) const {
-    if (key.template is<JsonString>())
-      return {*this, key.template as<JsonString>()};
-    else
-      return {*this, nullptr};
+    return {*this, detail::adaptString(key.template as<JsonString>())};
   }
 
   // Removes the member at the specified iterator.

src/ArduinoJson/Object/MemberProxy.hpp

@@ -10,14 +10,14 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE
 
 // A proxy class to get or set a member of an object.
 // https://arduinojson.org/v7/api/jsonobject/subscript/
-template <typename TUpstream, typename TStringRef>
+template <typename TUpstream, typename AdaptedString>
 class MemberProxy
-    : public VariantRefBase<MemberProxy<TUpstream, TStringRef>>,
-      public VariantOperators<MemberProxy<TUpstream, TStringRef>> {
+    : public VariantRefBase<MemberProxy<TUpstream, AdaptedString>>,
+      public VariantOperators<MemberProxy<TUpstream, AdaptedString>> {
   friend class VariantAttorney;
 
  public:
-  MemberProxy(TUpstream upstream, TStringRef key)
+  MemberProxy(TUpstream upstream, AdaptedString key)
       : upstream_(upstream), key_(key) {}
 
   MemberProxy(const MemberProxy& src)
@@ -47,7 +47,7 @@ class MemberProxy
 
   VariantData* getData() const {
     return VariantData::getMember(
-        VariantAttorney::getData(upstream_), adaptString(key_),
+        VariantAttorney::getData(upstream_), key_,
         VariantAttorney::getResourceManager(upstream_));
   }
 
@@ -55,13 +55,13 @@ class MemberProxy
     auto data = VariantAttorney::getOrCreateData(upstream_);
     if (!data)
       return nullptr;
-    return data->getOrAddMember(adaptString(key_),
+    return data->getOrAddMember(key_,
                                 VariantAttorney::getResourceManager(upstream_));
   }
 
  private:
   TUpstream upstream_;
-  TStringRef key_;
+  AdaptedString key_;
 };
 
 ARDUINOJSON_END_PRIVATE_NAMESPACE

src/ArduinoJson/Strings/StringAdapters.hpp

@@ -70,4 +70,8 @@ static void stringGetChars(TAdaptedString s, char* p, size_t n) {
   }
 }
 
+template <typename T>
+using AdaptedString =
+    typename StringAdapter<remove_reference_t<T>>::AdaptedString;
+
 ARDUINOJSON_END_PRIVATE_NAMESPACE

src/ArduinoJson/Variant/VariantRefBase.hpp

@@ -188,16 +188,16 @@ class VariantRefBase : public VariantTag {
   // Gets or sets an object member.
   // https://arduinojson.org/v7/api/jsonvariant/subscript/
   template <typename TString>
-  FORCE_INLINE
-      enable_if_t<IsString<TString>::value, MemberProxy<TDerived, TString>>
-      operator[](const TString& key) const;
+  FORCE_INLINE enable_if_t<IsString<TString>::value,
+                           MemberProxy<TDerived, AdaptedString<TString>>>
+  operator[](const TString& key) const;
 
   // Gets or sets an object member.
   // https://arduinojson.org/v7/api/jsonvariant/subscript/
   template <typename TChar>
-  FORCE_INLINE
-      enable_if_t<IsString<TChar*>::value, MemberProxy<TDerived, TChar*>>
-      operator[](TChar* key) const;
+  FORCE_INLINE enable_if_t<IsString<TChar*>::value,
+                           MemberProxy<TDerived, AdaptedString<TChar*>>>
+  operator[](TChar* key) const;
 
   // Gets an object member or an array element.
   // https://arduinojson.org/v7/api/jsonvariant/subscript/

src/ArduinoJson/Variant/VariantRefBaseImpl.hpp

@@ -123,17 +123,19 @@ inline ElementProxy<TDerived> VariantRefBase<TDerived>::operator[](
 }
 
 template <typename TDerived>
-template <typename TString>
-inline enable_if_t<IsString<TString*>::value, MemberProxy<TDerived, TString*>>
-VariantRefBase<TDerived>::operator[](TString* key) const {
-  return MemberProxy<TDerived, TString*>(derived(), key);
+template <typename TChar>
+inline enable_if_t<IsString<TChar*>::value,
+                   MemberProxy<TDerived, AdaptedString<TChar*>>>
+VariantRefBase<TDerived>::operator[](TChar* key) const {
+  return {derived(), adaptString(key)};
 }
 
 template <typename TDerived>
 template <typename TString>
-inline enable_if_t<IsString<TString>::value, MemberProxy<TDerived, TString>>
+inline enable_if_t<IsString<TString>::value,
+                   MemberProxy<TDerived, AdaptedString<TString>>>
 VariantRefBase<TDerived>::operator[](const TString& key) const {
-  return MemberProxy<TDerived, TString>(derived(), key);
+  return {derived(), adaptString(key)};
 }
 
 template <typename TDerived>