@edvin uploaded a file: Build Descriptor with Kotlin Tests

fixed it

and I set the tornadofx version to 1.7.15-SNAPSHOT (I already had that)

I picked up this project again to play with dhash and hamming distances

@ron The issue is that you deserialize from json, and some of the json structures doesn't have a photolist entry, so you set the value to null

I took the liberty to highlight/semi-fix the issue:

@edvin uploaded a file: Untitled

Here I clear the list instead of setting the list property to null

You might instead accept null values, but just check for it in your app.

Instead of clear you could also set it to a new list, if that's important (change event fired)

If that's not important, you might as well do

photolist.setAll(json.getJsonArray("photolist")?.toModel() ?: emptyList<Photo>())

😁

hmmz if I change to SimpleListProperty<Photo>() I now get nullpointer exceptions

I pushed the changes

just run the test and it will bomb in the constructor of the where I want to clear the lists

hmmz the only way I can make it work is like this and I hate it:

@ron uploaded a file: Untitled

@ron This is as it should be, I’ll explain later, at dinner party :)

Is there a way to express the following idea in an input control (like a spinner or a text input) that is bound to an ItemViewModel property: "When the value is edited, commit the ItemViewModel if the new value is valid"

@ron Remember, a SimpleListProperty is a property that can hold a list object, just like any other javafx property and that list object will be null until you set it. There is nothing wrong about that. If you haven't set it, you just need to keep in mind that it might be null.

@kastork So you want to autocommit, basically?

I've set the autocommit option in the ItemViewModel property, but it doesn't seem to have the desired effect

@kastork Can you whip up a small test case?

@edvin yeah I got that after thinking a bit more about it. I am currently so drugged up, I dont always think completely straight

@ron By the way,

val name = bind { SimpleStringProperty(item?.name) }

can be expressed as

val name = bind(Photo::name)

with the new syntax.

@ron Drugged up??

@edvin yes, eventually

for some weird reason my blood pressure is through the roof and I blacked out while driving

Oh man.. sorry to hear that! What kind of medication are you on?

yikes