Value cannot be cast from Boolean to String

[cheng-kang]

Hi,

I'm using react-native-mparticle@2.4.12 and experiencing the issue in the screenshot on Android. is_prepay is a boolean value attribute in the event, it works fine on iOS.

My workaround for now is:

track(event) {
    const payload = { ...event.payload };
    // MParticleModule for Android somehow tries to cast Boolean to String which would crash the app.
    if (Platform.OS === "android") {
      Object.keys(payload).forEach((key) => {
        if (typeof payload[key] === "boolean") {
          payload[key] = String(payload[key]);
        }
      })
    }
    MParticle.logEvent(event.name, MPaticle.EventType.Other, payload);
}

Could you help me to understand the situation here? Did I miss any docs about this issue? Or is this a bug?

[markgibaud-vtail]

I have encountered the same issue when trying to log a number in the event attributes.

UnexpectedNativeTypeException com.facebook.react.bridge.ReadableNativeMap in checkInstance
Value for numberOfImages cannot be cast from Double to String

On closer inspection of the source, it looks like only String is supported for now:

https://github.com/mParticle/react-native-mparticle/blob/master/android/src/main/java/com/mparticle/react/MParticleModule.java#L65

@willpassidomo any chance this can be fixed to facilitate boolean and number too? Would really help with downstream analytics querying capabilities (eg. range queries on event attribute data when logging numbers).

[willpassidomo]

Hey Mark, So, this is a bit unfortunate, but we our backend is setup to ingest all custom attribute values as Strings. We do have a number of integrations that, provided they are capable of accepting other primitive-like types, we will automatically convert into Number, Bool, Int etc if it can be converted. Do you have a particular integration in mind where it would help for these attributes to type cast? I could check if it is already available as a setting in our forwarder or at least give you an idea if it's something we could add to it. Thanks for reaching out, Will

…

On Thu, Sep 2, 2021, 7:23 PM Mark Gibaud ***@***.***> wrote: I have encountered the same issue when trying to log a number in the event attributes. On closer inspection of the source, it looks like only String is supported for now: https://github.com/mParticle/react-native-mparticle/blob/master/android/src/main/java/com/mparticle/react/MParticleModule.java#L65 @willpassidomo <https://github.com/willpassidomo> any chance this can be fixed to facilitate boolean and number too? Would really help with downstream analytics querying capabilities (eg. range queries on event attribute data when logging numbers). — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#45 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABXQCQPJ46U7VCUORWKRNQDUAABOXANCNFSM5BGFLP2A> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[willpassidomo]

@cheng-kang I'm so sorry this sat for so long! We were under the impression we had set up webhooks for all of our repositories, but this one seems to have slipped under the radar. I just opened a PR (referenced above) for this issue and it should be released shortly

[willpassidomo]

PR was merged and this will be included in the next release!

[markgibaud-vtail]

@willpassidomo thanks for coming back. As an example we're sending through the number of images (image) sent as attachments in a MessageSent event, and server-side forwarding to Mixpanel, but Mixpanel seems to ingest as a string and so we can't do range queries for example: