background_locator_fixed icon indicating copy to clipboard operation
background_locator_fixed copied to clipboard

Published 20 hours ago • verified publisher icon Yukams

Type mismatch: inferred type is String? but Any was expected

Open sandysultan opened this issue 2 years ago • 10 comments

When building I got this error image

I got this fixed by changing Any to Any? in this codes below LocationParserUtil.kt (11,74) LocationParserUtil.kt (35,81) LocationUpdateListener (6, 50) IsolateHolderService (296, 59)

sandysultan avatar Jul 14 '22 16:07 sandysultan

Hi @sandysultan, thanks for reporting ! I'll have a look next week, it's strange that no one reported this before though, maybe a new version of something broke it. If you can push a pull request I'll review it on Monday and push it on the next update after testing

Yukams avatar Jul 15 '22 08:07 Yukams

Hi @sandysultan do you still have this problem with the newer version ? Can you provide a pull request to fix your problem ?

Yukams avatar Jul 20 '22 12:07 Yukams

have the same issue flutter/.pub-cache/hosted/pub.dartlang.org/background_locator_2-2.0.5/android/src/main/kotlin/yukams/app/background_locator_2/provider/LocationParserUtil.kt: (21, 20): Type mismatch: inferred type is String? but Any was expected

kalyujniy avatar Jul 21 '22 15:07 kalyujniy

Hi @kirillwebmail, have you tried the fix proposed by @sandysultan ? If yes, and if it's working, could you provide a pull request ?

Yukams avatar Jul 21 '22 15:07 Yukams

Hi @Yukams, please try this, I tested myself and worked #22

sandysultan avatar Jul 21 '22 16:07 sandysultan

Thanks @sandysultan ! I will try this tonight and if everything is fine I'll push it to a pre-release branch and I'll post here again so you can retrieve the newest version with your fix :)

Yukams avatar Jul 21 '22 16:07 Yukams

@Yukams great, now its work #22 fix the bug with type missmatch

kalyujniy avatar Jul 21 '22 16:07 kalyujniy

Hi @kirillwebmail and @sandysultan, sorry it took a bit more time than I expected, but it's up on pub.dev as 2.0.6-dev.1 https://pub.dev/packages/background_locator_2/versions/2.0.6-dev.1

Thanks to both of you !

I'll merge this in the main branch next week if my tests are fine :)

Yukams avatar Jul 24 '22 15:07 Yukams

For anyone waiting for this fix to come to the main branch : I forgot to merge it before going for a week of vacation, just use the pre-release branch for now, sorry !

Yukams avatar Jul 30 '22 18:07 Yukams

@Yukams This PR can probably be merged to master now, correct? I'd like to include it in the next 2.0.6, please confirm.

srmncnk avatar Oct 21 '22 14:10 srmncnk