Make it easier to react to Lifecycle State changes in Compose [235529345]

Fixed

Feature Request

Status Update

No update yet.

Description

il...@google.com

created issue #1

Jun 9, 2022 11:56PM

Component used: Lifecycle

Version used: 2.5.0-rc01

Reacting to a Lifecycle.State changes is currently fairly complicated in a Jetpack Compose based app, requiring a DisposableEffect based approach with registering a LifecycleObserver with the LocalLifecycleOwner.

Ideally, this would come in two forms:

I'd like to see a way to capture the current Lifecycle.State as State, perhaps something like LocalLifecycleOwner.current.lifecycle.currentStateFlow.collectAsState()
I'd like a way to run side effects when the Lifecycle changes, perhaps using a custom LifecycleEventEffect that would only run the effect when the Lifecycle.State changes.

Comments

il...@google.com <il...@google.com> Sep 13, 2022 12:14AM

Assigned to sa...@google.com.

tr...@gmail.com <tr...@gmail.com> #2Oct 26, 2022 07:14AM

Note for dev team to check: If this isn't an error, CameraX should not log.

a....@gmail.com <a....@gmail.com> #3Nov 4, 2022 10:30PM

Hello,

Thanks for filing this issue, this issue can be safely ignored - we have a todo for removing unnecessarily logging. This should be fixed shortly.

If there are other issues, please let us know.

sj...@gmail.com <sj...@gmail.com> #4Feb 23, 2023 07:56AM

Fixed to debug in https://android-review.googlesource.com/c/platform/frameworks/support/+/1291317

sj...@gmail.com <sj...@gmail.com> #5Mar 7, 2023 11:30AM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 404aab04fc5ede6314fa93ff7b62431f9646e0e9
Author: Trevor McGuire <trevormcguire@google.com>
Date: Mon Apr 20 15:49:47 2020

Switch "Failed to set already detached use case online" to debug log

This can occur if the use case is detached before the camera's thread
has a chance to add the use case to the capture session. This should
not affect normal operation, so this error log message can be changed
to a debug log.

Relnote: "Converted error log related to detached use case to a debug log on Camera2CameraImpl."

Test: ./gradlew bOS
Bug: 154422490
Change-Id: I1a565e55af6e63792287221f8cbd9e7747fd6875

M camera/camera-camera2/src/main/java/androidx/camera/camera2/internal/Camera2CameraImpl.java

https://android-review.googlesource.com/1291317

sa...@google.com <sa...@google.com> Mar 14, 2023 08:18PM

Accepted by sa...@google.com.

ap...@google.com <ap...@google.com> #6Mar 16, 2023 07:03PM

Can this log error message also be safely ignored?

Failed to set already detached use case online

[Deleted User] <[Deleted User]> #7Mar 22, 2023 05:51PM

Sorry, I meant this one:

Failed to set already detached use case active

[Deleted User] <[Deleted User]> #8Mar 22, 2023 05:51PM

Yes, it can be safely ignored. I will switch it to a debug log as well.

[Deleted User] <[Deleted User]> #9Mar 22, 2023 05:51PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 0cff18a13097a05812b60c8bfb7968baf79464b4
Author: Trevor McGuire <trevormcguire@google.com>
Date: Mon Apr 27 11:52:06 2020

Switch "Failed to set already detached use case active" to debug log

This can occur if the use case is detached before the camera's thread
has a chance react to the use case being marked active. This should
not affect normal operation, so this error log message can be changed
to a debug log.

Bug: 154422490
Test: ./gradlew bOS
Change-Id: I2a99128be65c03af955753c230cb798789f8769f

M camera/camera-camera2/src/main/java/androidx/camera/camera2/internal/Camera2CameraImpl.java

https://android-review.googlesource.com/1296396

ap...@google.com <ap...@google.com> #10May 2, 2023 10:39PM

Project: platform/frameworks/support
Branch: androidx-main

commit e9afa3a28b32897fef6717fea079c6296f3a2e5a
Author: sanura <sanura@google.com>
Date: Tue Apr 25 01:07:17 2023

Create LifecycleEventEffect composable

Adding LifecycleEventEffect composable API to
be used when running SideEffects based on a
particular Lifecycle.Event.

RelNote: "`LifecycleEventEffect` API has been
added to run side effects based on `Lifecycle.Event`."
Test: LifecycleEventEffectTest
Bug: 235529345

Change-Id: Ic9794d7e59b1a66cc7f31cbaebed9b3d3dee5dd3

M lifecycle/lifecycle-runtime-compose/api/current.txt
M lifecycle/lifecycle-runtime-compose/api/public_plus_experimental_current.txt
M lifecycle/lifecycle-runtime-compose/api/restricted_current.txt
M lifecycle/lifecycle-runtime-compose/src/androidTest/java/androidx/lifecycle/compose/CollectAsStateWithLifecycleTests.kt
A lifecycle/lifecycle-runtime-compose/src/androidTest/java/androidx/lifecycle/compose/LifecycleEffectTest.kt
M lifecycle/lifecycle-runtime-compose/src/androidTest/java/androidx/lifecycle/compose/LifecycleExtTest.kt
A lifecycle/lifecycle-runtime-compose/src/main/java/androidx/lifecycle/compose/LifecycleEffect.kt

https://android-review.googlesource.com/2560450

ap...@google.com <ap...@google.com> #11May 4, 2023 10:09PM

Project: platform/frameworks/support
Branch: androidx-main

commit 25a2002ab933ea551295483aa3f7e278d60dbe75
Author: sanura <sanura@google.com>
Date: Wed May 03 20:20:56 2023

Create LifecycleResumeEffect composable

Adding LifecycleResumeEffect composable API to be used when
running SideEffects based on ON_RESUME/ON_PAUSE event callbacks.

RelNote: "`LifecycleResumeEffect` API has been added to run side
effects based on both `Lifecycle.Event.ON_RESUME` and
`Lifecycle.Event.ON_PAUSE` event callbacks."
Test: lifecycleResumeEffectTest
Bug: 235529345

Change-Id: I603860f21eb6a433c909d7ae6f8120f6d23179aa

M lifecycle/lifecycle-runtime-compose/api/current.txt
M lifecycle/lifecycle-runtime-compose/api/public_plus_experimental_current.txt
M lifecycle/lifecycle-runtime-compose/api/restricted_current.txt
M lifecycle/lifecycle-runtime-compose/src/androidTest/java/androidx/lifecycle/compose/LifecycleEffectTest.kt
M lifecycle/lifecycle-runtime-compose/src/main/java/androidx/lifecycle/compose/LifecycleEffect.kt

https://android-review.googlesource.com/2577033

ap...@google.com <ap...@google.com> #12May 4, 2023 10:09PM

Project: platform/frameworks/support
Branch: androidx-main

commit 2b090a54b8182927eb0e96062e0a14d065a8561a
Author: sanura <sanura@google.com>
Date: Tue May 02 19:53:55 2023

Create LifecycleStartEffect composable

Adding LifecycleStartEffect composable API to be used when
running SideEffects based on ON_START/ON_STOP event callbacks.

RelNote: "`LifecycleStartEffect` API has been added to run side
effects based on `Lifecycle.Event.ON_START` and
`Lifecycle.Event.ON_STOP` event callbacks."
Test: lifecycleStartEffectTest
Bug: 235529345

Change-Id: I5a8d1e6f152b7083e086146ba97044c4b454db84

M lifecycle/lifecycle-runtime-compose/api/current.txt
M lifecycle/lifecycle-runtime-compose/api/public_plus_experimental_current.txt
M lifecycle/lifecycle-runtime-compose/api/restricted_current.txt
M lifecycle/lifecycle-runtime-compose/src/androidTest/java/androidx/lifecycle/compose/LifecycleEffectTest.kt
M lifecycle/lifecycle-runtime-compose/src/main/java/androidx/lifecycle/compose/LifecycleEffect.kt

https://android-review.googlesource.com/2576490

st...@gmail.com <st...@gmail.com> #13May 29, 2023 02:24PM

Is there a target API version where this may be released?

il...@google.com <il...@google.com> #14May 30, 2023 10:58PM

This is still in progress for inclusion in Lifecycle 2.7.0-alpha01.

ap...@google.com <ap...@google.com> #15Jun 1, 2023 11:36PM

Project: platform/frameworks/support
Branch: androidx-main

commit ad273fa4882a38a5a2f72004044ce3173b2c06a9
Author: sanura <sanura@google.com>
Date: Fri May 05 02:02:59 2023

Add samples for LifecycleEffects

Adding samples for LifecycleEventEffect,
LifecycleStartEffect, and LifecycleResumeEffect.

RelNote: "All composables in `LifecycleEffect` now
link sample usages."
Test: ./gradlew bOS
Bug: 235529345

Change-Id: Ib4786f4b586cf9390f3dbe6f3b0fc86c60267a86

M lifecycle/lifecycle-runtime-compose/samples/src/main/java/androidx/lifecycle/compose/samples/LifecycleComposeSamples.kt
M lifecycle/lifecycle-runtime-compose/src/main/java/androidx/lifecycle/compose/LifecycleEffect.kt

https://android-review.googlesource.com/2579164

sa...@google.com <sa...@google.com> Jun 21, 2023 07:39PM

Marked as fixed.

[Deleted User] <[Deleted User]> #16Jun 22, 2023 04:31AM

فخاامه

في الأربعاء، 21 يونيو، 2023 10:40 م <buganizer-system@google.com> كتب:

- Show quoted text -

ka...@gmail.com <ka...@gmail.com> #17Jul 7, 2023 08:12AM

When will this be released?

na...@google.com <na...@google.com> #18Jul 26, 2023 05:21PM

The following release(s) address this bug.It is possible this bug has only been partially addressed:

androidx.lifecycle:lifecycle-runtime-compose:2.7.0-alpha01

sj...@gmail.com <sj...@gmail.com> #19Jan 11, 2024 06:40AM

Just want to say that Lifecycle 2.7.0 stable has been finally released. Thank you very much 🙏🏼

ey...@gmail.com <ey...@gmail.com> #20Jan 11, 2024 02:10PM

I'm getting some strange results from `currentStateFlow` as opposed to setting a regular lifecycle observer. Basically, for ON_PAUSE and ON_STOP I get STARTED and CREATED, respectively.

jb...@google.com <jb...@google.com> #21Jan 11, 2024 02:20PM

Yes, those are the Lifecycle.States that reflect those particular Lifecycle.Events. Please take a look at the Lifecycle guidance for more details.

ey...@gmail.com <ey...@gmail.com> #22Jan 11, 2024 03:21PM

Sorry, my bad, didn't realize that. But since this is a flow, shouldn't it emit events rather than states? What is the usecase to react to two states that never become invalid? The docs claim this is in order to "read changes in the lifecycle" but it would require further processing to be able to realize what's going on.

jb...@google.com <jb...@google.com> #23Jan 11, 2024 04:32PM

Receiving a state from the flow would be the signal that a change has occurred and the new state is that particular change. There are cases where you don't care how you get to a state, just that you are there. We also offer LifecycleEffects to assist with use cases for Lifecycle.Events.

il...@google.com <il...@google.com> #24Jan 11, 2024 05:31PM

If you look at the release notes, you'll find two separate sections - one on Lifecycle Events (where you can use Lifecycle.asFlow() to get a Flow of Lifecycle.Event) and one for Lifecycle State (where you can use Lifecycle.currentStateFlow to get the current Lifecycle.State).

Make sure you are using the right API.

ey...@gmail.com <ey...@gmail.com> #25Jan 12, 2024 09:20AM

Thank you both for all the guidance. I guess my issue is that I'm trying to use it outside of Compose, to replace a simple lifecycle event observer, since it just feels like a nicer interface to collect a flow. For Compose, the effects work fine, indeed.

il...@google.com <il...@google.com> #26Jan 16, 2024 08:25PM

Lifecycle.asFlow() works outside of Compose too and gives you Lifecycle.Event instances as they occur. That's why that API exists separately from currentStateFlow.