Add 'event' parameter to 'handled' lambda in 'ViewEvent' 'peek()'

This is not an important change, but It’s an enhancement. Currently, the function peek, It’s the following:

viewModel.observeState(this) {
    it.event?.peek {
        when (it.event) {
            is CatViewEvent.Meow -> {
                // show Meow! dialog
                true
            }
        }
    }
}

But into the peek function, I need use again the “it.event”. It’s can resolve changing the parameter in the peek function:

fun peek(handled: ViewEvent.() -> Boolean) {
    if (!alreadyHandled) alreadyHandled = this.handled()
}

So, the new peek method would be used:

viewModel.observeState(this) {
    it.event?.peek {
        when (this) {
            is CatViewEvent.Meow -> {
                // show Meow! dialog
                true
            }
        }
    }
}

[etiennelenhart]

Normally executing an event multiple times should be accomplished by setting an updated ViewState with a new event.

I totally forgot about internal. That should accomplish pretty much exactly what I’m aiming for. I’ll probably keep this for a later update though since it’s a really minor thing.

Thanks for your input. 🙂

[eugenio1590]

Hi @etiennelenhart , I apologize for the delay. Thanks for the support. I was thinking about it too and I think that the public variable would help to execute events again. Similarly, the visibility “internal” would help a lot. Thanks again for the support, I will be testing as soon as possible.