Fixed
Status Update
Comments
mg...@google.com <mg...@google.com> #2
Project: platform/frameworks/support
Branch: androidx-master-dev
commit b90079595f33f58fece04026a97faa0d243acdb1
Author: Yuichi Araki <yaraki@google.com>
Date: Wed Sep 18 16:55:49 2019
Change the way to detect mismatch between POJO and query
This fixes cursor mismatch warnings with expandProjection.
Bug: 140759491
Test: QueryMethodProcessorTest
Change-Id: I7659002e5e0d1ef60fc1af2a625c0c36da0664d8
M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt
M room/compiler/src/test/kotlin/androidx/room/processor/QueryMethodProcessorTest.kt
M room/compiler/src/test/kotlin/androidx/room/testing/TestProcessor.kt
https://android-review.googlesource.com/1123258
https://goto.google.com/android-sha1/b90079595f33f58fece04026a97faa0d243acdb1
Branch: androidx-master-dev
commit b90079595f33f58fece04026a97faa0d243acdb1
Author: Yuichi Araki <yaraki@google.com>
Date: Wed Sep 18 16:55:49 2019
Change the way to detect mismatch between POJO and query
This fixes cursor mismatch warnings with expandProjection.
Bug: 140759491
Test: QueryMethodProcessorTest
Change-Id: I7659002e5e0d1ef60fc1af2a625c0c36da0664d8
M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt
M room/compiler/src/test/kotlin/androidx/room/processor/QueryMethodProcessorTest.kt
M room/compiler/src/test/kotlin/androidx/room/testing/TestProcessor.kt
ap...@google.com <ap...@google.com> #3
ap...@google.com <ap...@google.com> #4
Project: platform/frameworks/support
Branch: androidx-master-dev
commit bdde5a1a970ddc9007b28de4aa29d60ffa588f08
Author: Yigit Boyar <yboyar@google.com>
Date: Thu Apr 16 16:47:05 2020
Re-factor how errors are dismissed when query is re-written
This CL changes how we handle errors/warnings if query is
re-written.
There was a bug in expandProjection where we would report warnings
for things that Room already fixes automatically ( b/140759491 ).
The solution to that problem (I7659002e5e0d1ef60fc1af2a625c0c36da0664d8)
solved it by deferring validating of columns until after re-write
decision is made. Unfortunately, this required changing PojoRowAdapter
to have a dummy mapping until it is validating, make it hard to use
as it does have a non-null mapping which is not useful.
This CL partially reverts that change and instead rely on the log
deferring logic we have in Context. This way, we don't need to break
the stability of PojoRowAdapter while still having the ability to
drop warnings that room fixes. This will also play nicer when we
have different query re-writing options that can use more information
about the query results.
Bug: 153387066
Bug: 140759491
Test: existing tests pass
Change-Id: I2ec967c763d33d7a3ff02c1a13c6953b460d1e5f
M room/compiler/src/main/kotlin/androidx/room/log/RLog.kt
M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt
https://android-review.googlesource.com/1288456
Branch: androidx-master-dev
commit bdde5a1a970ddc9007b28de4aa29d60ffa588f08
Author: Yigit Boyar <yboyar@google.com>
Date: Thu Apr 16 16:47:05 2020
Re-factor how errors are dismissed when query is re-written
This CL changes how we handle errors/warnings if query is
re-written.
There was a bug in expandProjection where we would report warnings
for things that Room already fixes automatically (
The solution to that problem (I7659002e5e0d1ef60fc1af2a625c0c36da0664d8)
solved it by deferring validating of columns until after re-write
decision is made. Unfortunately, this required changing PojoRowAdapter
to have a dummy mapping until it is validating, make it hard to use
as it does have a non-null mapping which is not useful.
This CL partially reverts that change and instead rely on the log
deferring logic we have in Context. This way, we don't need to break
the stability of PojoRowAdapter while still having the ability to
drop warnings that room fixes. This will also play nicer when we
have different query re-writing options that can use more information
about the query results.
Bug: 153387066
Bug: 140759491
Test: existing tests pass
Change-Id: I2ec967c763d33d7a3ff02c1a13c6953b460d1e5f
M room/compiler/src/main/kotlin/androidx/room/log/RLog.kt
M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt
ap...@google.com <ap...@google.com> #5
Project: platform/frameworks/support
Branch: androidx-main
commit 89a169b4573e1b28c7a5a73fd52d294bd0596a79
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:33:41 2024
Move `LocalViewModelStoreOwner` to common on `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I1ead8036fc8b3c49d5846c417d301f281cff90e3
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.android.kt
A lifecycle/lifecycle-viewmodel-compose/src/commonMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.kt
A lifecycle/lifecycle-viewmodel-compose/src/desktopMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.desktop.kt
https://android-review.googlesource.com/3007638
Branch: androidx-main
commit 89a169b4573e1b28c7a5a73fd52d294bd0596a79
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:33:41 2024
Move `LocalViewModelStoreOwner` to common on `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I1ead8036fc8b3c49d5846c417d301f281cff90e3
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.android.kt
A lifecycle/lifecycle-viewmodel-compose/src/commonMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.kt
A lifecycle/lifecycle-viewmodel-compose/src/desktopMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.desktop.kt
ap...@google.com <ap...@google.com> #6
Project: platform/frameworks/support
Branch: androidx-main
commit b8e5ea51a234c4cdbbed82e41f6ffd28f630dcd9
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:22:15 2024
Rename source from `viewmodel-compose` to follow KMP convention
Test: N/A
Bug: 330323282
Change-Id: Ic417903127417b53f0888263a35fc5e4a82d57cc
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaverTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/TestViewModelClasses.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInAppCompatActivityTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInComponentActivityTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInFragmentTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaveableApi.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaver.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
https://android-review.googlesource.com/3007635
Branch: androidx-main
commit b8e5ea51a234c4cdbbed82e41f6ffd28f630dcd9
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:22:15 2024
Rename source from `viewmodel-compose` to follow KMP convention
Test: N/A
Bug: 330323282
Change-Id: Ic417903127417b53f0888263a35fc5e4a82d57cc
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaverTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/TestViewModelClasses.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInAppCompatActivityTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInComponentActivityTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInFragmentTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaveableApi.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaver.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
ap...@google.com <ap...@google.com> #7
Project: platform/frameworks/support
Branch: androidx-main
commit 37991db14175338fe6beac23bae696090689d418
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:24:42 2024
Update Compose UI and Runtime to 1.6.0 in `viewmodel-compose`
* Required to use the `Composable` annotation from the common source set.
Test: N/A
Bug: 330323282
Change-Id: I879d9bd2c904524f157cf24068130d7cfc1cce45
M lifecycle/lifecycle-viewmodel-compose/build.gradle
https://android-review.googlesource.com/3007636
Branch: androidx-main
commit 37991db14175338fe6beac23bae696090689d418
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:24:42 2024
Update Compose UI and Runtime to 1.6.0 in `viewmodel-compose`
* Required to use the `Composable` annotation from the common source set.
Test: N/A
Bug: 330323282
Change-Id: I879d9bd2c904524f157cf24068130d7cfc1cce45
M lifecycle/lifecycle-viewmodel-compose/build.gradle
ap...@google.com <ap...@google.com> #8
Project: platform/frameworks/support
Branch: androidx-main
commit 288826d6e5ecd81e1300562feb62ceedf6ecc912
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:44:25 2024
Move `ViewModel.kt` to common on `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I0cec3da46577619bdf51a57976fa726ce961ad93
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
A lifecycle/lifecycle-viewmodel-compose/src/commonMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.kt
https://android-review.googlesource.com/3007642
Branch: androidx-main
commit 288826d6e5ecd81e1300562feb62ceedf6ecc912
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:44:25 2024
Move `ViewModel.kt` to common on `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I0cec3da46577619bdf51a57976fa726ce961ad93
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
A lifecycle/lifecycle-viewmodel-compose/src/commonMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.kt
ap...@google.com <ap...@google.com> #9
Project: platform/frameworks/support
Branch: androidx-main
commit 61e7561c841336e65be6c18fa7fd8d8b003fb2a4
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:26:09 2024
Move dependencies to `commonMain` on `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I9ccddf090462f16a71dabe9a0f687c7e91ffb5f6
M lifecycle/lifecycle-viewmodel-compose/build.gradle
https://android-review.googlesource.com/3007637
Branch: androidx-main
commit 61e7561c841336e65be6c18fa7fd8d8b003fb2a4
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:26:09 2024
Move dependencies to `commonMain` on `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I9ccddf090462f16a71dabe9a0f687c7e91ffb5f6
M lifecycle/lifecycle-viewmodel-compose/build.gradle
ap...@google.com <ap...@google.com> #10
Project: platform/frameworks/support
Branch: androidx-main
commit a1f7b7585088dd00015c61bfa10053e76f77cc85
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:41:44 2024
Use `ViewModelProvider.create` method on `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I056165d298392ddae01f170bd7912196207c30ea
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
https://android-review.googlesource.com/3007641
Branch: androidx-main
commit a1f7b7585088dd00015c61bfa10053e76f77cc85
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:41:44 2024
Use `ViewModelProvider.create` method on `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I056165d298392ddae01f170bd7912196207c30ea
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
ap...@google.com <ap...@google.com> #11
Project: platform/frameworks/support
Branch: androidx-main
commit c7d8ed902005764c8df4fef833aba0bb525084ea
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:36:08 2024
Add `factory` param to KDocs on `viewmodel-compose`
* Required for moving from `docs` to `kmpDocs`.
* Fixes `UnknownLine` Missing `@param` tag for parameter `factory` in DFunction `viewModel`
Test: N/A
Bug: 330323282
Change-Id: I216b5bbe5ada1909b24e86aa72236a4471fd334a
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
https://android-review.googlesource.com/3007639
Branch: androidx-main
commit c7d8ed902005764c8df4fef833aba0bb525084ea
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:36:08 2024
Add `factory` param to KDocs on `viewmodel-compose`
* Required for moving from `docs` to `kmpDocs`.
* Fixes `UnknownLine` Missing `@param` tag for parameter `factory` in DFunction `viewModel`
Test: N/A
Bug: 330323282
Change-Id: I216b5bbe5ada1909b24e86aa72236a4471fd334a
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
ap...@google.com <ap...@google.com> #12
Project: platform/frameworks/support
Branch: androidx-main
commit 1305e970c8a7187208cc2bbb0932104276bd2aea
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:41:44 2024
Use `ViewModelProvider.create` method in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: Ia866dfe300c361ceb3ffeb2a88d6b63f38b09aec
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
https://android-review.googlesource.com/3011213
Branch: androidx-main
commit 1305e970c8a7187208cc2bbb0932104276bd2aea
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:41:44 2024
Use `ViewModelProvider.create` method in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: Ia866dfe300c361ceb3ffeb2a88d6b63f38b09aec
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
ap...@google.com <ap...@google.com> #13
Project: platform/frameworks/support
Branch: androidx-main
commit ac869c99d561c43b5235bf58a35b859bb36e326f
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:33:41 2024
Move `LocalViewModelStoreOwner` to common in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I3561ea2d7e70ee2741bdbefdd6c1625413994aeb
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.android.kt
A lifecycle/lifecycle-viewmodel-compose/src/commonMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.kt
A lifecycle/lifecycle-viewmodel-compose/src/desktopMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.desktop.kt
https://android-review.googlesource.com/3006457
Branch: androidx-main
commit ac869c99d561c43b5235bf58a35b859bb36e326f
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:33:41 2024
Move `LocalViewModelStoreOwner` to common in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I3561ea2d7e70ee2741bdbefdd6c1625413994aeb
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.android.kt
A lifecycle/lifecycle-viewmodel-compose/src/commonMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.kt
A lifecycle/lifecycle-viewmodel-compose/src/desktopMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.desktop.kt
ap...@google.com <ap...@google.com> #14
Project: platform/frameworks/support
Branch: androidx-main
commit 358101c3e4ee1dd87464c1aec84de41309f706ac
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:22:15 2024
Rename source from `viewmodel-compose` to follow KMP convention
Test: N/A
Bug: 330323282
Change-Id: I688e85f81215bcc9bd05829e1ea2f78fb09c4c50
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaverTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/TestViewModelClasses.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInAppCompatActivityTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInComponentActivityTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInFragmentTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaveableApi.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaver.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
https://android-review.googlesource.com/3006454
Branch: androidx-main
commit 358101c3e4ee1dd87464c1aec84de41309f706ac
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:22:15 2024
Rename source from `viewmodel-compose` to follow KMP convention
Test: N/A
Bug: 330323282
Change-Id: I688e85f81215bcc9bd05829e1ea2f78fb09c4c50
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaverTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/TestViewModelClasses.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInAppCompatActivityTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInComponentActivityTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInFragmentTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelTest.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaveableApi.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaver.android.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
ap...@google.com <ap...@google.com> #15
Project: platform/frameworks/support
Branch: androidx-main
commit b7acab4368186972bb3138c45628cabc971b6fe2
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:24:42 2024
Update Compose UI and Runtime to 1.6.0 in `viewmodel-compose`
* Required to use the `Composable` annotation from the common source set.
Test: N/A
Bug: 330323282
Change-Id: I664449a501ef1d274b91cb3ad196cf5802d5f74b
M lifecycle/lifecycle-viewmodel-compose/build.gradle
https://android-review.googlesource.com/3006455
Branch: androidx-main
commit b7acab4368186972bb3138c45628cabc971b6fe2
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:24:42 2024
Update Compose UI and Runtime to 1.6.0 in `viewmodel-compose`
* Required to use the `Composable` annotation from the common source set.
Test: N/A
Bug: 330323282
Change-Id: I664449a501ef1d274b91cb3ad196cf5802d5f74b
M lifecycle/lifecycle-viewmodel-compose/build.gradle
ap...@google.com <ap...@google.com> #16
Project: platform/frameworks/support
Branch: androidx-main
commit 87fc3ece00231d3310d3f08746b6890c908722e0
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 27 14:53:26 2024
Fix corrupt serialized resolution result in `viewmodel-compose`
* Converting `lifecycle-viewmodel-compose` to Kotlin Multiplatform, and transitively depending on `lifecycle-livedata-core` triggered a bug in Gradle (https://github.com/gradle/gradle/issues/14220 ).
* Directly adding `lifecycle-livedata-core` to `androidMain` allowed us to workaround the issue.
Test: `./gradlew :lifecycle:lifecycle-viewmodel-compose:compileCommonMainKotlinMetadata -Pandroidx.constraints=true`
Bug: 330323282
Change-Id: I0eb4c79efeb65e049f51706c6dcefce0d1ab83a4
M lifecycle/lifecycle-viewmodel-compose/build.gradle
https://android-review.googlesource.com/3017197
Branch: androidx-main
commit 87fc3ece00231d3310d3f08746b6890c908722e0
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 27 14:53:26 2024
Fix corrupt serialized resolution result in `viewmodel-compose`
* Converting `lifecycle-viewmodel-compose` to Kotlin Multiplatform, and transitively depending on `lifecycle-livedata-core` triggered a bug in Gradle (
* Directly adding `lifecycle-livedata-core` to `androidMain` allowed us to workaround the issue.
Test: `./gradlew :lifecycle:lifecycle-viewmodel-compose:compileCommonMainKotlinMetadata -Pandroidx.constraints=true`
Bug: 330323282
Change-Id: I0eb4c79efeb65e049f51706c6dcefce0d1ab83a4
M lifecycle/lifecycle-viewmodel-compose/build.gradle
ap...@google.com <ap...@google.com> #17
Project: platform/frameworks/support
Branch: androidx-main
commit fc493abaa7ecb38655922406691d1454f084b6ec
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 27 14:57:55 2024
Align all `implementation` declarations in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: Ice8418438e70d633b59ae52cf3071d7b8c70e863
M lifecycle/lifecycle-viewmodel-compose/build.gradle
https://android-review.googlesource.com/3017198
Branch: androidx-main
commit fc493abaa7ecb38655922406691d1454f084b6ec
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 27 14:57:55 2024
Align all `implementation` declarations in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: Ice8418438e70d633b59ae52cf3071d7b8c70e863
M lifecycle/lifecycle-viewmodel-compose/build.gradle
ap...@google.com <ap...@google.com> #18
Project: platform/frameworks/support
Branch: androidx-main
commit 7a8b0a8f054a488f1fa3762269b3c812d5c8fbf1
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:26:09 2024
Move dependencies to `commonMain` in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: Icdf9d69a73de2ffc780355b2a84f21387ec4550c
M lifecycle/lifecycle-viewmodel-compose/build.gradle
https://android-review.googlesource.com/3006456
Branch: androidx-main
commit 7a8b0a8f054a488f1fa3762269b3c812d5c8fbf1
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:26:09 2024
Move dependencies to `commonMain` in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: Icdf9d69a73de2ffc780355b2a84f21387ec4550c
M lifecycle/lifecycle-viewmodel-compose/build.gradle
ap...@google.com <ap...@google.com> #19
Project: platform/frameworks/support
Branch: androidx-main
commit 923f14b702f2b1cd7c2e1a626a3320737ea88e3a
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:39:47 2024
Add `KClass` method variants in `viewmodel-compose`
RelNote: "Composable `viewModel` functions now accept a `KClass` for Multiplatform compatibility. The `Class` variants are kept for Android compatibility."
Test: N/A
Bug: 330323282
Change-Id: I309315a069205acd87f7752d31c8a91ffc5ffbe7
M lifecycle/lifecycle-viewmodel-compose/api/current.txt
M lifecycle/lifecycle-viewmodel-compose/api/restricted_current.txt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
https://android-review.googlesource.com/3006459
Branch: androidx-main
commit 923f14b702f2b1cd7c2e1a626a3320737ea88e3a
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:39:47 2024
Add `KClass` method variants in `viewmodel-compose`
RelNote: "Composable `viewModel` functions now accept a `KClass` for Multiplatform compatibility. The `Class` variants are kept for Android compatibility."
Test: N/A
Bug: 330323282
Change-Id: I309315a069205acd87f7752d31c8a91ffc5ffbe7
M lifecycle/lifecycle-viewmodel-compose/api/current.txt
M lifecycle/lifecycle-viewmodel-compose/api/restricted_current.txt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
ap...@google.com <ap...@google.com> #20
Project: platform/frameworks/support
Branch: androidx-main
commit ee955583f497b4c4ed60b95703282db85319534a
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:18:32 2024
Configure KMP in `viewmodel-compose`
* Setup the build scripts.
* Setup folder hierarchy.
Test: N/A
Bug: 330323282
Change-Id: I7461c27e0e90774c5391b971bfd3e58772cc0c9e
M docs-tip-of-tree/build.gradle
M lifecycle/lifecycle-viewmodel-compose/build.gradle
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/AndroidManifest.xml
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaverTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/TestViewModelClasses.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInAppCompatActivityTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInComponentActivityTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInFragmentTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaveableApi.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaver.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.kt
https://android-review.googlesource.com/3006453
Branch: androidx-main
commit ee955583f497b4c4ed60b95703282db85319534a
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:18:32 2024
Configure KMP in `viewmodel-compose`
* Setup the build scripts.
* Setup folder hierarchy.
Test: N/A
Bug: 330323282
Change-Id: I7461c27e0e90774c5391b971bfd3e58772cc0c9e
M docs-tip-of-tree/build.gradle
M lifecycle/lifecycle-viewmodel-compose/build.gradle
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/AndroidManifest.xml
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaverTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/TestViewModelClasses.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInAppCompatActivityTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInComponentActivityTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelInFragmentTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidInstrumentedTest/kotlin/androidx/lifecycle/viewmodel/compose/ViewModelTest.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/LocalViewModelStoreOwner.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaveableApi.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/SavedStateHandleSaver.kt
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.kt
ap...@google.com <ap...@google.com> #21
Project: platform/frameworks/support
Branch: androidx-main
commit 7a8b46834cd8070ef519ac1d118115bcb62924ad
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:44:25 2024
Move `ViewModel.kt` to common in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I88d7c2a876b7469f301870a139b733c9db15cb8d
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
A lifecycle/lifecycle-viewmodel-compose/src/commonMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.kt
https://android-review.googlesource.com/3011214
Branch: androidx-main
commit 7a8b46834cd8070ef519ac1d118115bcb62924ad
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:44:25 2024
Move `ViewModel.kt` to common in `viewmodel-compose`
Test: N/A
Bug: 330323282
Change-Id: I88d7c2a876b7469f301870a139b733c9db15cb8d
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
A lifecycle/lifecycle-viewmodel-compose/src/commonMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.kt
ap...@google.com <ap...@google.com> #22
Project: platform/frameworks/support
Branch: androidx-main
commit 22bf2247c81917a7d8e08d5d58e1a1b82c4fd6d9
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:36:08 2024
Add `factory` param to KDocs in `viewmodel-compose`
* Required for moving from `docs` to `kmpDocs`.
* Fixes `UnknownLine` Missing `@param` tag for parameter `factory` in DFunction `viewModel`
Test: N/A
Bug: 330323282
Change-Id: Iea301a001d6e47e038aa1e03cb5bc8e5c3b28bc2
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
https://android-review.googlesource.com/3006458
Branch: androidx-main
commit 22bf2247c81917a7d8e08d5d58e1a1b82c4fd6d9
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 20 11:36:08 2024
Add `factory` param to KDocs in `viewmodel-compose`
* Required for moving from `docs` to `kmpDocs`.
* Fixes `UnknownLine` Missing `@param` tag for parameter `factory` in DFunction `viewModel`
Test: N/A
Bug: 330323282
Change-Id: Iea301a001d6e47e038aa1e03cb5bc8e5c3b28bc2
M lifecycle/lifecycle-viewmodel-compose/src/androidMain/kotlin/androidx/lifecycle/viewmodel/compose/ViewModel.android.kt
ap...@google.com <ap...@google.com> #23
Project: platform/frameworks/support
Branch: androidx-main
commit 8c39ccf27d8a9656c2d22a464a9863e9af2e293e
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 27 17:17:46 2024
Set LiveData to `compile` in `viewmodel-compose`
* Replace `implementation(projectOrArtifact(":lifecycle:lifecycle-livedata-core"))` by `compileOnly(projectOrArtifact(":lifecycle:lifecycle-livedata-core"))`
Test: `./gradlew :lifecycle:lifecycle-viewmodel-compose:compileCommonMainKotlinMetadata -Pandroidx.constraints=true`
Bug: 330323282
Change-Id: I1e3c65fb268b91ae489650995caa3cd33a452f57
M lifecycle/lifecycle-viewmodel-compose/build.gradle
https://android-review.googlesource.com/3017501
Branch: androidx-main
commit 8c39ccf27d8a9656c2d22a464a9863e9af2e293e
Author: Marcello Galhardo <mgalhardo@google.com>
Date: Wed Mar 27 17:17:46 2024
Set LiveData to `compile` in `viewmodel-compose`
* Replace `implementation(projectOrArtifact(":lifecycle:lifecycle-livedata-core"))` by `compileOnly(projectOrArtifact(":lifecycle:lifecycle-livedata-core"))`
Test: `./gradlew :lifecycle:lifecycle-viewmodel-compose:compileCommonMainKotlinMetadata -Pandroidx.constraints=true`
Bug: 330323282
Change-Id: I1e3c65fb268b91ae489650995caa3cd33a452f57
M lifecycle/lifecycle-viewmodel-compose/build.gradle
Description
Now that b/214568825 ), we should follow-up migrating
lifecycle-viewmodel
has been migrated to Kotlin Multiplatform (seelifecycle-viewmodel-compose
too.