https://kotlinlang.org logo
Channels
Powered by
#tornadofx
  • e

    edvin

    05/13/2017, 8:00 PM
    It was fixed in 1.7.3.
  • e

    edvin

    05/13/2017, 8:00 PM
    So you don't need to go bleeding edge 🙂 But update to 1.7.4 perhaps, and do a clean/build?
  • k

    kastork

    05/13/2017, 8:01 PM
    Well much of what I'm doing has been addressed in commits over the last few days... like that 
    String
    case in 
    toJson
  • e

    edvin

    05/13/2017, 8:02 PM
    Just go for 1.7.5-SNAPSHOT then 🙂 It is stable, we'll release 1.7.5 shortly.
  • k

    kastork

    05/13/2017, 8:02 PM
    from today
  • e

    edvin

    05/13/2017, 8:02 PM
    I can push a new snapshot now if you don't want to build yourself.
  • k

    kastork

    05/13/2017, 8:02 PM
    I've been running off head for a while now, its no problem
  • e

    edvin

    05/13/2017, 8:03 PM
    OK.
  • e

    edvin

    05/13/2017, 8:03 PM
    How's it looking now?
  • k

    kastork

    05/13/2017, 8:06 PM
    got interrupted... one sec
  • k

    kastork

    05/13/2017, 8:10 PM
    @edvin still nulls, I must have something wrong in my entity class
  • e

    edvin

    05/13/2017, 8:12 PM
    I think so too. Will you post it?
  • k

    kastork

    05/13/2017, 8:12 PM
    Let me abbreviate it a bit
  • e

    edvin

    05/13/2017, 8:13 PM
    Yeah, one prop should be enough.
  • k

    kastork

    05/13/2017, 8:14 PM
    @User uploaded a file: Untitled
    Untitled.kt
  • k

    kastork

    05/13/2017, 8:15 PM
    It serializes to JSON just fine
  • e

    edvin

    05/13/2017, 8:18 PM
    Hm.. I didn't even write the JsonModelAuto myself, so I have no experience with it, but let me check something here.
  • e

    edvin

    05/13/2017, 8:20 PM
    My little test works with the auto model as well:
  • k

    kastork

    05/13/2017, 8:20 PM
    On line 5 I'm getting null for 
    pr
    -.kt
  • e

    edvin

    05/13/2017, 8:21 PM
    Hang on, trying with your object
  • e

    edvin

    05/13/2017, 8:22 PM
    Yours didn't work for the 
    configurationName
    property, only for 
    isGlobalDefault
    for some reason.
  • k

    kastork

    05/13/2017, 8:23 PM
    @User uploaded a file:

    Pasted image at 2017-05-13, 1:23 PM▾

  • e

    edvin

    05/13/2017, 8:25 PM
    Yeah, the auto model is not up to snuff, that's for sure.
  • e

    edvin

    05/13/2017, 8:25 PM
    It needs to differentiate between several different types of properties.
  • k

    kastork

    05/13/2017, 8:26 PM
    ok, more debugging. The delegateField is "lazy value not initialized yet
  • k

    kastork

    05/13/2017, 8:26 PM
    "
  • e

    edvin

    05/13/2017, 8:27 PM
    You have lazy fields in the model? The JsonModelAuto makes no distinction on such properties. This will take some doing, and unfortunately I won't be able to do that tonight.
  • k

    kastork

    05/13/2017, 8:27 PM
    Since 
    pr
    is null, none of the cases match
  • e

    edvin

    05/13/2017, 8:27 PM
    I would implement updateModel and toJSON myself.
  • e

    edvin

    05/13/2017, 8:28 PM
    But I will try to look at it tomorrow, since it's in the framework, it should work correctly 🙂 I kind of regret accepting that PR now 🙂 hehe
1...117811791180...2007Latest