Fixed
Status Update
No update yet.
Comments
su...@google.com
After some internal discussion, we think that the best path forward is to close this bug as infeasible. The only realistic change we can offer here is to rename Result to something else like WorkResult. That being said, this error will not occur when you are creating a new ListenableWorker, so most people will never hit this. Only people updating an existing WorkManager implementation will encounter this bug.
ch...@gmail.com
I think WorkResult would be clearer regardless of the conflict. I think assuming most people are using ListenableWorker out of the box is not safe assumption.
[Deleted User] <[Deleted User]>
+1 for renaming to `WorkResult`
ro...@gmail.com
+1 for renaming to `WorkResult`
su...@google.com
Reopening so I can just rename it to WorkResult.
ch...@instacart.com
Thanks Sumir!
Description
Version used: 1.0.0-alpha12
After upgrading to alpha12, all of my Workers written in Kotlin (and CoroutineWorkers, FWIW), complain about the 'Result' return value with an error that says: "One type argument expected for class Result<out T>"
It appears that the Kotlin standard library contains a public inline Result class already:
The Kotlin Result class is automatically available in every file, so Android Studio (3.2.1) doesn't offer to import the androidx.work.Result class, causing the error since it assumes you want to use the already available kotlin Result.
There were two workarounds, neither very friendly:
1) Use the fully qualified androidx.work.Result
2) Manually adding the androidx.work.Result import statement
Note that:
- Using the fully qualified name, then using the Android Studio option to add import did not work
- Removing the doWork method entirely and using Android Studio's quick fix to implement the method did add an import to androidx.work.Result, so new classes would avoid running into this issue.