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

    g4sarma

    12/28/2017, 10:19 PM
    Yes
  • g

    g4sarma

    12/28/2017, 10:19 PM
    @carlw
  • g

    g4sarma

    12/28/2017, 10:21 PM
    replaceWith(DashboardView::class, ViewTransition.Slide(Duration(1200.00), ViewTransition.Direction.LEFT))
  • e

    edvin

    12/28/2017, 10:41 PM
    @g4sarma Do you have a complete app committed somewhere I can look at it?
  • g

    g4sarma

    12/28/2017, 10:42 PM
    Unfortunately no 😞
  • c

    carlw

    12/28/2017, 10:44 PM
  • c

    carlw

    12/28/2017, 10:45 PM
    @g4sarma try reconciling your code with my example
  • e

    edvin

    12/28/2017, 10:46 PM
    Thanks @carlw 🙂
  • g

    g4sarma

    12/28/2017, 10:47 PM
    @carlw thanks for sharing that reference. I will do it 🙂
  • r

    ron

    12/29/2017, 1:50 PM
    @carlw @edvin I was under the impression that using a class like this:
  • r

    ron

    12/29/2017, 1:50 PM
    @ron uploaded a file: Untitled
    -.kt
  • r

    ron

    12/29/2017, 1:51 PM
    the 
    childrenProperty
    and the 
    photoProperty
    are initialized
  • r

    ron

    12/29/2017, 1:51 PM
    however they are not when its called from the updateModel
  • c

    carlw

    12/29/2017, 1:58 PM
    you get an error on line 39 when you deserialize?
  • r

    ron

    12/29/2017, 1:58 PM
    no, I get a nullpointer error when I run toString from a testcase
  • r

    ron

    12/29/2017, 1:59 PM
    @ron uploaded a file: Untitled
    -.kt
  • r

    ron

    12/29/2017, 1:59 PM
    the whole project is in https://github.com/ronsmits/KPhoto
  • r

    ron

    12/29/2017, 2:00 PM
    the error occurs on line 17
  • c

    carlw

    12/29/2017, 2:05 PM
    photolist.size is the NPE?
  • r

    ron

    12/29/2017, 2:05 PM
    yup
  • e

    edvin

    12/29/2017, 2:06 PM
    @ron That sounds like magic. In any case, you should probably use 
    SimpleListProperty
    - does that make a difference?
  • c

    carlw

    12/29/2017, 2:07 PM
    @edvin should it be photolist.get().size()?
  • r

    ron

    12/29/2017, 2:08 PM
    cleans up the code, does not fix the problem
  • e

    edvin

    12/29/2017, 2:08 PM
    @carlw Checking out the project and looking at it now
  • e

    edvin

    12/29/2017, 2:08 PM
    @ron 
    <tornadofx.version>1.7.2</tornadofx.version>
    ?? Madness 🙂
    🙀 1
  • r

    ron

    12/29/2017, 2:09 PM
    I cant keep up Edvin 🙂
  • e

    edvin

    12/29/2017, 2:09 PM
    @ron We had a bug, or rather no support for list style properties a while back, this might be it.
  • e

    edvin

    12/29/2017, 2:10 PM
    IDEA doesn't recognize the test, running from command line
  • e

    edvin

    12/29/2017, 2:11 PM
    Ah, because you have the tests in src/main/kotlin and didn't configure that output folder. Also, junit should have scope test
  • e

    edvin

    12/29/2017, 2:17 PM
    @ron Here is an updated deployment descriptor that fixes the test issue and updates versions
1...161316141615...2007Latest