Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Android Permission bug #29

Closed
bobosette opened this issue Jan 5, 2022 · 10 comments
Closed

Android Permission bug #29

bobosette opened this issue Jan 5, 2022 · 10 comments

Comments

@bobosette
Copy link

Hi @Caldarie . I found another issue to fix. On my app it's the dart code which ask for permission (It asks for all the permissions inside the home page > I need to do this cause tflite_audio is not the only package that need permissions). When the app ask for the permission and user grants them, I don't know why but tflite_audio shows a dialog with this message: 'Microphone permission denied. Go to settings etc..'. But it isn't true cause user granted that permission. After a lot of time, I found the issue inside TfliteAudioPlugin.java, raw 330 (inside onRequestPermissionResult() method). I don't know why but it seems like that method doesn't understand that permission has been already granted.
Can you provide a little update on this thing?
Thank you

@Caldarie
Copy link
Owner

Caldarie commented Jan 5, 2022

hi @bobosette

Many thanks for reporting various bug. Really appreciate it.

I will investigate the issue. It seems that Java/android has been giving me alot of bugs lately compared to iOS. I may consider rewriting the whole plugin to kotlin in the future 😆

@bobosette
Copy link
Author

Incredible. Usually is the opposite. Let me know if you fix it

@Caldarie
Copy link
Owner

Caldarie commented Jan 5, 2022

Yeah.. i feel java is overly complicated and rigid. I end up writing perhaps 30 to 40% more code in java compared to swift..

@bobosette
Copy link
Author

I don't know cause I didn't learn swift. Only a little bit to work with flutter platform channel.

@Caldarie
Copy link
Owner

Caldarie commented Jan 5, 2022

Hi, it seems you are right. There are some strange behaviours with the master version.

Fortunately, this branch im working on does not seem to exhibit the same problem.

As you have a different device than I, perhaps you can test this branch for me to see if you still have the same problem?

@bobosette
Copy link
Author

bobosette commented Jan 5, 2022 via email

@Caldarie
Copy link
Owner

Caldarie commented Jan 5, 2022

Thanks. Please do let me know. I do hope to publish new update very soon.

@bobosette
Copy link
Author

bobosette commented Jan 6, 2022 via email

@Caldarie
Copy link
Owner

Caldarie commented Jan 6, 2022

Many thanks for the test. I will release an update very soon. Includes many stability fixes as well.

@Caldarie
Copy link
Owner

Caldarie commented Jan 7, 2022

Hi, i have published the new package 0.2.2+2. Please let me know if you run into any bugs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants