Fixed unsupported variant type: 1024 (QJSValue)

[obscuren]

Added proper casting from QMetaType::User (1024) with userType 1034 (i.e.,
QJSValue) to QVariant so it can be handled properly by the packDataValue
method.

[pjoe]

I tried this with limetext/qml .. however for me the userType is 1051 ... will try a way to not hardcode this

[pjoe]

Here is what i ended up with, which seems to work:

diff --git a/cpp/capi.cpp b/cpp/capi.cpp
index 55ac585..c2255aa 100644
--- a/cpp/capi.cpp
+++ b/cpp/capi.cpp
@@ -745,7 +745,8 @@ void packDataValue(QVariant_ *var, DataValue *value)
         break;
     case QMetaType::User:
         {
-            if (qvar->userType() == 1034) {
+            static const int qjstype = QVariant::fromValue(QJSValue()).userType();
+            if (qvar->userType() == qjstype) {
                 auto var = qvar->value<QJSValue>().toVariant();
                 packDataValue(&var, value);
             }

[obscuren]

Confirmed. @pjoe change works. Updated PR.

[niemeyer]

We need a test case exercising the issue to make sure we understand the problem and it doesn't not regress.

Also, can you please sign the agreement so we can get the change in:

http://www.ubuntu.com/legal/contributors

[tgerring]

I don't think it was c288002 was intended to be part of the original PR, but it may be obsoleted by 154c0d1 which was merged in 2ee7e5f

[immesys]

@Niemyer, please can you merge this in. It's really useful.