DAO Implementation Executing Incorrect/Old Query [76031240]

Fixed

Bug

Status Update

No update yet.

Description

tr...@gmail.com

created issue #1

Mar 21, 2018 07:10PM

Component used: Room
Version used: 1.1.0-alpha3
Devices/Android versions reproduced on: Pixel 2 XL, Android 8.1.0

The screenshot shows this, but I've got a DAO method that returns a Single<List<RawMetadata>> and for some reason, it seems that the method occasionally runs an incorrect query (I'm guessing it's one that got pulled from RoomSQLiteQuery's pool). This is causing "column not found" issues because the DAO is looking for the columns that make sense for the particular DAO method that was called, but the cursor contains all of the columns from the incorrect/old query.

Note the mQuery value of _statement in the debugger variables pane and how it's different than the _sql value on line 227.

Screen Shot 2018-03-21 at 1.02.30 PM.png

874 KB

View

Download

Comments

yb...@google.com <yb...@google.com> #2Mar 21, 2018 07:28PM

Status is still Assigned, but Compose 1.1.0 was already released. Did the ETA change for this feature?

tr...@gmail.com <tr...@gmail.com> #3Mar 21, 2018 07:37PM

The Compose Animation APIs Navigation needs (such as AnimatedContent) are still @Experimental in Compose 1.1.0.

tr...@gmail.com <tr...@gmail.com> #4Mar 21, 2018 07:39PM

Update: Same case for Compose 1.2.0

yb...@google.com <yb...@google.com> #5Mar 21, 2018 07:43PM

why is this taking so long?

tr...@gmail.com <tr...@gmail.com> #6Mar 21, 2018 07:49PM

seriously no update on what is stable?

tr...@gmail.com <tr...@gmail.com> #7Mar 22, 2018 02:48PM

AnimatedContent is, as of today, still experimental and still cannot be used in androidx.navigation.

As soon as it is stable, we can kick off this work.

yb...@google.com <yb...@google.com> #8Mar 25, 2018 05:54PM

Assigned to yb...@google.com.

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

commit a3386225c952c42b7c687771ea31642d76f4df66
Author: Clara Fok <clarafok@google.com>
Date: Wed May 10 11:26:59 2023

Add Preview support in animated NavHost

Test: ./gradlew navigation:navigation-compose:cC
Bug: 197140101
Change-Id: I7b017913d96e31785f16eb66534321d05f29b53d

M navigation/navigation-compose/src/main/java/androidx/navigation/compose/NavHost.kt

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

tr...@gmail.com <tr...@gmail.com> #9Mar 26, 2018 05:44AM

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

commit 256f919c6b8d579bc55e19d829aed438422f463b
Author: Clara Fok <clarafok@google.com>
Date: Wed May 10 10:48:32 2023

Migrate accompanist NavHost and NavController into androidx

Test: ./gradlew navigation:navigation-compose:cC
Bug: 197140101
Relnote: "NavHost and NavController now supports navigating in compose with Animations."
Change-Id: Ica95e794ef1794df4d1bdb38719b55624f832868

M navigation/navigation-compose/api/current.txt
M navigation/navigation-compose/api/public_plus_experimental_current.txt
M navigation/navigation-compose/api/restricted_current.txt
M navigation/navigation-compose/src/androidTest/java/androidx/navigation/compose/NavHostControllerTest.kt
M navigation/navigation-compose/src/androidTest/java/androidx/navigation/compose/NavHostTest.kt
M navigation/navigation-compose/src/main/java/androidx/navigation/compose/ComposeNavigator.kt
M navigation/navigation-compose/src/main/java/androidx/navigation/compose/NavGraphBuilder.kt
M navigation/navigation-compose/src/main/java/androidx/navigation/compose/NavHost.kt

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

ya...@google.com <ya...@google.com> Mar 30, 2018 06:10AM

Reassigned to ya...@google.com.

yb...@google.com <yb...@google.com> #10Mar 30, 2018 09:05PM

Reassigned to yb...@google.com.

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

commit 682bae8ada2b2b29376003d55455230ff7320a69
Author: Clara Fok <clarafok@google.com>
Date: Tue May 09 15:13:14 2023

Migrate accompanist NavGraphBuilder into androidx

Test: ./gradlew navigation:navigation-compose:cC
Bug: 197140101
Relnote: "NavGraphBuilder now supports building composables and NavGraphs with transition animations."
Change-Id: I5ca9e22c8c3b02acaf80ddaa35b58ba7426c993a

M navigation/navigation-compose/src/main/java/androidx/navigation/compose/NavGraphBuilder.kt

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

yb...@google.com <yb...@google.com> #11Mar 30, 2018 09:06PM

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

commit 895c4bb35ee11fe1c54f538699a845ddfaf7e2eb
Author: Clara Fok <clarafok@google.com>
Date: Tue May 09 13:14:10 2023

Migrate accompanist ComposeNavigator into androidx

Test: ./gradlew navigation:navigation-compose:cC
Bug: 197140101
Relnote: "ComposeNavigator now supports Destinations with Animations."
Change-Id: Ie33b4c2da53ff6c89494332a06be8123be67e879

M navigation/navigation-compose/src/main/java/androidx/navigation/compose/ComposeNavigator.kt

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

tr...@gmail.com <tr...@gmail.com> #12Apr 2, 2018 10:02PM

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

commit 169254301b8aad0454c253a65abff7bf174ba164
Author: Clara Fok <clarafok@google.com>
Date: Tue May 09 11:11:42 2023

Bump compose version in navigation-compose

Bump from 1.0.1 to ToT project dependency. Project dependency because compose is currently in alpha.

Test: n/a
Bug: 197140101
Relnote: "navigation-compose has bumped compose dependency to use latest compose dependency"
Change-Id: Ide442548636637c2b75d46ef99c9913b0ebb60b8

M navigation/navigation-compose/build.gradle

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

yb...@google.com <yb...@google.com> Apr 21, 2018 10:36PM

Marked as fixed.