Room compiler can't see the TypeConverter of a List<JavaClass> using kapt [111404868]

Fixed

Bug

Status Update

No update yet.

Description

ah...@gmail.com

created issue #1

Jul 13, 2018 08:58AM

Component used: Room
Version used: 1.1.1
Kotlin version: 1.2.51

Room can't detect the TypeConverter of a List<Model> if that Model itself is written in Java and if I convert this class to Kotlin, it works as expected!

I got this error no matter where I put the TypeConverters annotation!
error: Cannot figure out how to save this field into database. You can consider adding a type converter for it.

A project where this can be reproduced is attached!

RoomTypeConverterBug.zip

1 MB

Download

Comments

yb...@google.com <yb...@google.com> Aug 17, 2018 08:03PM

Assigned to da...@google.com.

ap...@google.com <ap...@google.com> #2Oct 10, 2018 09:24PM

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

commit e36c65f67e49ddbfa0080542295f4a4850ae17da
Author: Sumir Kataria <sumir@google.com>
Date: Tue Aug 07 13:04:03 2018

Enforce ConstraintTracker#setState is on the main thread.

- setState now asserts that we are on the main thread
- Moved assert*Thread methods into their own utils
- Annotated several tests manually calling onBroadcastReceive
as @UiThreadTest as our broadcast receivers only operate on
the main thread (they use the default registerReceiver
method)
- NetworkCallbacks are invoked on non-main threads by default;
in this case, we re-post them to the main thread

This is part of the solution to a concurrency exception in
ConstraintTracker.

Bug: 112272753
Test: Updated and ran tests
Change-Id: I7011e5993b814f4fcafe861e283a827cd1edee12

M work/workmanager/build.gradle
M work/workmanager/src/androidTest/java/androidx/work/impl/constraints/trackers/BatteryChargingTrackerTest.java
M work/workmanager/src/androidTest/java/androidx/work/impl/constraints/trackers/BatteryNotLowTrackerTest.java
M work/workmanager/src/androidTest/java/androidx/work/impl/constraints/trackers/ConstraintTrackerTest.java
M work/workmanager/src/androidTest/java/androidx/work/impl/constraints/trackers/StorageNotLowTrackerTest.java
M work/workmanager/src/androidTest/java/androidx/work/impl/workers/ConstraintTrackingWorkerTest.java
M work/workmanager/src/main/java/androidx/work/impl/WorkManagerImpl.java
M work/workmanager/src/main/java/androidx/work/impl/background/systemalarm/SystemAlarmDispatcher.java
M work/workmanager/src/main/java/androidx/work/impl/constraints/trackers/ConstraintTracker.java
M work/workmanager/src/main/java/androidx/work/impl/constraints/trackers/NetworkStateTracker.java
A work/workmanager/src/main/java/androidx/work/impl/utils/ThreadUtils.java

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

https://goto.google.com/android-sha1/e36c65f67e49ddbfa0080542295f4a4850ae17da

da...@google.com <da...@google.com> Oct 10, 2018 09:34PM

Marked as fixed.

[Deleted User] <[Deleted User]> #3Jan 2, 2019 02:12AM

Fixed in

https://android-review.googlesource.com/c/platform/frameworks/support/+/728180 (not sure why the automated message isn't being shown here). This should be available in alpha07.

da...@gmail.com <da...@gmail.com> #4Nov 15, 2019 09:58AM

room:2.2.1
kotlin: 1.3.50

I am still getting this error for:
var formIds: List<Long> = arrayListOf(),

Fails with an error:
Cannot figure out how to save this field into database. You can consider adding a type converter for it.
private java.util.List<java.lang.Long> formIds;

but it works for:
var formIds: MutableList<Long> = arrayListOf(),

da...@gmail.com <da...@gmail.com> #5Nov 15, 2019 10:06AM

sorry, it works for
var formIds: ArrayList<Long> = arrayListOf(),

Issue 111404868