From 915eb54c0a2a680a41c576557a88dcf777d87168 Mon Sep 17 00:00:00 2001
From: Vitaliy Zarubin <v.zarubin@omp.ru>
Date: Thu, 11 May 2023 23:03:57 +0300
Subject: [PATCH] [shared_preferences] save list

---
 .../aurora/shared_preferences_aurora_plugin.cpp | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/packages/shared_preferences/shared_preferences_aurora/aurora/shared_preferences_aurora_plugin.cpp b/packages/shared_preferences/shared_preferences_aurora/aurora/shared_preferences_aurora_plugin.cpp
index 295bf65..dd7f3c5 100644
--- a/packages/shared_preferences/shared_preferences_aurora/aurora/shared_preferences_aurora_plugin.cpp
+++ b/packages/shared_preferences/shared_preferences_aurora/aurora/shared_preferences_aurora_plugin.cpp
@@ -2,6 +2,8 @@
 #include <flutter/method-channel.h>
 #include <flutter/application.h>
 #include <QSettings>
+#include <QMetaType>
+#include <QVariant>
 
 SharedPreferencesAuroraPlugin::SharedPreferencesAuroraPlugin(): settings(
     QString::fromStdString(Application::GetID().orgname), 
@@ -117,9 +119,7 @@ void SharedPreferencesAuroraPlugin::onGetStringList(const MethodCall &call)
     const auto [key, value] = this->getArguments(call);
 
     std::vector<Encodable> vec;
-    QStringList list = settings.value(key, "")
-        .toString()
-        .split(",");
+    QStringList list = settings.value(key, {}).toStringList();
 
     for (const auto& item : list)
     {
@@ -137,18 +137,15 @@ void SharedPreferencesAuroraPlugin::onSetStringList(const MethodCall &call)
 {
     const auto [key, value] = this->getArguments(call);
     const auto vec = value.GetList();
-    std::stringstream ss;
+
+    QStringList strings;
 
     for (const auto& item : vec)
     {
-        if (ss.rdbuf()->in_avail() != 0) 
-        {
-            ss << ",";
-        }
-        ss << item;
+        strings.append(QString::fromStdString(item.GetString()));
     }
 
-    settings.setValue(key, QString::fromStdString(ss.str()));
+    settings.setValue(key, strings);
     call.SendSuccessResponse(true);
 }