there the captured username/password/url from the browser plugin are stored

I should make a screencast of that too

I'm not going to pretend understanding the nesting of groups here, but what about:

@User uploaded a file: Untitled

each group has a list of groups (parent->child)

but that code is cleaner

yours I mean

Helps a little. Ideally we'd find a nicer way to traverse those groups though.

this is not my thing, it comes from the openkeepass library

If

rootGrup

is not null, will the rest of the hierarchy be available?

yes

Then you can do

rootGroup?.apply { groups.first().groups.add(this) }

but it doesn't help much.

rootgroup always has a list with at least one child

But it can be null?

no, the createKeepassFile, specifically creates it

Then why do you need to use a null safe operator there?

kotlin forces it 😞

Then you can just do

rootGroup!!.groups.first().add(this)

That's at least just one ugly bit.

interesting this

val rootGroup: Group?
        get() = keepassFile?.root

seems to be evaluated only once

No, that would be evaluated every time. Must be something else causing you to believe that.

yes, the something else is me, I am stupid sometimes 😉

@carlw just made this recording to show the walk through

wha! I was about to search for a good password keeper! 😛

@ron

whenVisible

is available in the latest build of TornadoFX 🙂

@carlw I'll create the screencast, then add the acceptance test and if that goes well I'll merge it into the original repo 🙂

@edvin when you look at that youtube thingy when the dialog is opened it does not go to the front

I have to click on keepassFX to get it shown

do you have a trick?

Checking.