thiago-android-terminal | Stripe Developers | Page 1

grim harbor Dec 21, 2021, 5:58 PM

#

hello, not sure off the top of my head but that sounds like a dependency issue, so lets start here

what stripe-terminal-android version do you have installed

strong orchid Dec 21, 2021, 5:59 PM

#

thanks a lot!

#

implementation "com.stripe:stripeterminal:2.2.0"

#

when i use "internet" config

#

works like a charm

#

any idea?

grim harbor Dec 21, 2021, 6:10 PM

#

still thinking, afaik that should still be the right import

try updating your dependency to v2.5.2 (latest) and sync gradle again and see: https://github.com/stripe/stripe-terminal-android/releases

GitHub

Releases · stripe/stripe-terminal-android

Stripe Terminal Android SDK. Contribute to stripe/stripe-terminal-android development by creating an account on GitHub.

strong orchid Dec 21, 2021, 6:18 PM

#

let me try

#

the same :/

river bison Dec 21, 2021, 6:26 PM

#

👋 @grim harbor had to leave, but I'm stepping in!

#

Just to make sure I understand everything correctly - what specifically are you doing differently in the "internet" config so that it works?

strong orchid Dec 21, 2021, 6:29 PM

#

one sec

#

val config = DiscoveryConfiguration(0, DiscoveryMethod.INTERNET, true) -> WORKS
val config = DiscoveryConfiguration(10, DiscoveryMethod.BLUETOOTH_SCAN, false) -> not work

river bison Dec 21, 2021, 6:38 PM

#

thanks for that - let me dig a bit more

strong orchid Dec 21, 2021, 6:42 PM

#

thanks @river bison

river bison Dec 21, 2021, 7:15 PM

#

I'm not seeing any reason why this shouldn't work... can you try forcing an update to refresh the dependencies? I wonder if somethign is still out of sync

strong orchid Dec 21, 2021, 7:49 PM

#

What dependencies?

#

I already set the terminal to the last one

grim harbor Dec 21, 2021, 8:00 PM

#

a couple of us are catching up, give us a second

#

karbi had to step away

strong orchid Dec 21, 2021, 8:00 PM

#

thanks!

mystic ermine Dec 21, 2021, 8:02 PM

#

👋 hey @strong orchid
do you wanna try force update/re-download the dependencies?

https://www.cloudhadoop.com/gradle-force-update/

strong orchid Dec 21, 2021, 8:07 PM

#

1 min

#

any ideas before that? because I'll need to update my gradle to version 7

#

and right now it wasn't the idea

grim harbor Dec 21, 2021, 8:21 PM

#

give me a sec to quickly repro this, one min

strong orchid Dec 21, 2021, 8:21 PM

#

thanks @grim harbor

grim harbor Dec 21, 2021, 9:23 PM

#

@strong orchid mind pasting the full import for BluetoothReaderListener that you're declaring in your Kotlin/Java file?

river bison Dec 22, 2021, 5:57 PM

#

Feel free to continue here @strong orchid

strong orchid Dec 22, 2021, 6:19 PM

#

@grim harbor do you mean about the #imports?

river bison Dec 22, 2021, 6:21 PM

#

Yes, he's asking for the full import line for BluetoothReaderListener

strong orchid Dec 22, 2021, 6:35 PM

#

Give me one min

#

It's weird, i don't have any imports for that

#

😐

#

Did i need any import?

grim harbor Dec 22, 2021, 7:05 PM

#

like aren't you using that in the file? where you need to implement BluetoothReaderListener?

#

like how the example App imports it here
https://github.com/stripe/stripe-terminal-android/blob/37df6222c3a48ffc6c076056fd01e3cd2e187489/Example/javaapp/src/main/java/com/stripe/example/javaapp/MainActivity.java#L34

then implements that Listener interface on MainActivity
https://github.com/stripe/stripe-terminal-android/blob/37df6222c3a48ffc6c076056fd01e3cd2e187489/Example/javaapp/src/main/java/com/stripe/example/javaapp/MainActivity.java#L51

then implements any of the ReaderListener callbacks like
https://github.com/stripe/stripe-terminal-android/blob/37df6222c3a48ffc6c076056fd01e3cd2e187489/Example/javaapp/src/main/java/com/stripe/example/javaapp/MainActivity.java#L51

strong orchid Dec 22, 2021, 7:13 PM

#

these are my imports

#

import android.os.Bundle
import android.os.Handler
import androidx.fragment.app.Fragment
import android.view.LayoutInflater
import android.view.View
import android.view.ViewGroup
import androidx.core.content.ContextCompat
import com.stripe.stripeterminal.Terminal
import com.stripe.stripeterminal.external.callable.*
import com.stripe.stripeterminal.external.models.*
import ironark.com.charge.R
import ironark.com.charge.activities.MainActivity
import ironark.com.charge.adapters.ReaderLocationsAdapter
import ironark.com.charge.api.StripeTerminalApplication.Companion.readerSelected
import ironark.com.charge.database.PrefsModels
import ironark.com.charge.database.PrefsModels.connectedReaderLocId
import ironark.com.charge.database.PrefsModels.connectedReaderSn
import ironark.com.charge.utils.StripeTerminalManager
import kotlinx.android.synthetic.main.fragment_reader_locations.*
import org.jetbrains.anko.*
import org.jetbrains.anko.support.v4.act
import org.jetbrains.anko.support.v4.ctx
import org.jetbrains.anko.support.v4.runOnUiThread
import org.jetbrains.anko.support.v4.toast

strong orchid Dec 22, 2021, 7:38 PM

#

just added this one import com.stripe.stripeterminal.external.callable.BluetoothReaderListener

#

and nothing

strong orchid Dec 22, 2021, 7:58 PM

#

any idea?

tribal anvil Dec 22, 2021, 8:02 PM

#

Catching up on this thread. One minute...

strong orchid Dec 22, 2021, 8:04 PM

#

thanks @tribal anvil

grim harbor Dec 22, 2021, 8:06 PM

#

where do you get this error @strong orchid , like can you share a screenshot of your Android Studio when you get this error?

#

to give you more context - a few of us tried a fresh project, integrated the stripe-terminal-android SDK from scratch and synced gradle and had no build or dependency issues. So we couldn't repro, so trying to get as much info from your dev/Android Studio set up to see what is different

strong orchid Dec 22, 2021, 8:07 PM

#

when I'm trying to connect to reader

#

#

Terminal.getInstance().connectBluetoothReader(reader, connectionConfig, activity?.get(), object : ReaderCallback {ReaderCallback {
override fun onSuccess(reader: Reader) {
mOnStripeTerminalListener.connectBluetoothReader(reader)
}
override fun onFailure(e: TerminalException) {
e.printStackTrace()
mOnStripeTerminalListener.connectBluetoothReader(reader, e.errorMessage)
}
})

#

when I was testing by internertConfig -> // Terminal.getInstance().connectInternetReader(reader, connectionConfig, object : ReaderCallback {

#

works

grim harbor Dec 22, 2021, 8:14 PM

#

in a pastebin, can you paste me the full activity where you implement BluetoothReaderListener?

#

and in the meantime, looking cause I haven't encountered this issue before (nothing reported on the github issues either)

strong orchid Dec 22, 2021, 8:18 PM

#

yep

#

📎 reader.rtf

#

I also didn't find anything reported on the gitub

grim harbor Dec 22, 2021, 8:47 PM

#

@strong orchid
what exact line does it crash on, is it sometime after it calls connectBluetoothReader() ? Or immediately on that line?

strong orchid Dec 22, 2021, 8:49 PM

#

It's not a crash, just an error when call this function

#

line 193 call this function

#

fun connectBluetoothReader(reader: Reader, activity: Activity? = null) {
val connectionConfig = ConnectionConfiguration.BluetoothConnectionConfiguration(reader.location?.id.orEmpty())
// val connectionConfig = ConnectionConfiguration.InternetConnectionConfiguration(true)

    Terminal.getInstance().connectBluetoothReader(reader, connectionConfig, activity?.get(), object : ReaderCallback {

// Terminal.getInstance().connectInternetReader(reader, connectionConfig, object : ReaderCallback {
override fun onSuccess(reader: Reader) {
mOnStripeTerminalListener.connectBluetoothReader(reader)
}
override fun onFailure(e: TerminalException) {
e.printStackTrace()
mOnStripeTerminalListener.connectBluetoothReader(reader, e.errorMessage)
}
})
}

#

ant it failed

grim harbor Dec 22, 2021, 8:53 PM

#

what is the class of activity on line 193, I assume ReaderLocationsFragment but want to check

strong orchid Dec 22, 2021, 8:54 PM

#

yes

#

it's Reader locationFragment

#

my manager

📎 manager.rtf

grim harbor Dec 22, 2021, 8:55 PM

#

can you send those as non-rtf files, easier to open

strong orchid Dec 22, 2021, 8:56 PM

#

📎 manager

#

📎 reader

grim harbor Dec 22, 2021, 8:58 PM

#

how far does your code get into connectBluetoothReader() on your manager class? any specific part it crashes on?

strong orchid Dec 22, 2021, 9:00 PM

#

the manager call connectBluetoothReader (Line 60) and it crashes on Reader(line 198)

grim harbor Dec 22, 2021, 10:43 PM

#

ah so some really helpful colleagues clued me in more, it looks like you're using koin for dep injection?

I'm not familiar with koin really but you might have to do something to get your activity's instance passed with koin, similar to this issue: https://stackoverflow.com/questions/61148093/koin-no-definition-found-for-class-but-already-declared

opaque pagoda Dec 23, 2021, 1:27 PM

#

@strong orchid unarchived, but not sure how much help I can be.

strong orchid Dec 23, 2021, 1:35 PM

#

Hey @opaque pagoda

#

thanks!

#

I was checking the sample app

#

and in StripeTerminalApplication we have

#

public void onCreate() {
// Should happen before super.onCreate()
StrictMode.setThreadPolicy(
new StrictMode.ThreadPolicy.Builder()
.detectDiskReads()
.detectDiskWrites()
.detectAll()
.penaltyLog()
.build());

    StrictMode.setVmPolicy(
            new StrictMode.VmPolicy.Builder()
                    .detectLeakedSqlLiteObjects()
                    .detectLeakedClosableObjects()
                    .penaltyLog()
                    .penaltyDeath()
                    .build());

    super.onCreate();

    TerminalApplicationDelegate.onCreate(this);
}

#

but if i use this, i have this error

#

E/StripeTerminal: KoinApplication has not been started

#

Starting Koin i have this:

#

E/StripeTerminal: No definition found for class:'com.stripe.stripeterminal.external.callable.BluetoothReaderListener'. Check your definitions!

opaque pagoda Dec 23, 2021, 1:39 PM

#

yeah to be clear, we looked at this with some colleagues and the only real sense we have here is _something _ about Koin and its dependency injection is causing a problem.

strong orchid Dec 23, 2021, 1:39 PM

#

I think my problem is the Koin

opaque pagoda Dec 23, 2021, 1:39 PM

#

but we don't use Koin or have any experience with it ourselves so it's hard to say

strong orchid Dec 23, 2021, 1:40 PM

#

We could use another DI

#

I'm using Koin because of the log

#

Did you understand it?

opaque pagoda Dec 23, 2021, 1:41 PM

#

understand what exactly?

strong orchid Dec 23, 2021, 1:42 PM

#

About the Koin, I using the Koin DI.. because of the log on Android Studio

#

Thanks a lot for your help

opaque pagoda Dec 23, 2021, 1:43 PM

#

I only found out Koin existed 5 minutes ago so I'm not too familiar with what advantages it has or how it adds logging capabilities 😅

strong orchid Dec 23, 2021, 1:43 PM

#

ahahaha

opaque pagoda Dec 23, 2021, 1:44 PM

#

as @grim harbor mentioned though maybe there's something special that needs to be done to pass the Activity instances to Koin like in that StackOverflow answer, not sure.

strong orchid Dec 23, 2021, 1:44 PM

#

and i found out when Stripe terminal "recommend" it

#

I don't know, but i think my issue in how startKoin with the module

opaque pagoda Dec 23, 2021, 1:51 PM

#

possibly!

strong orchid Dec 23, 2021, 2:07 PM

#

waiting for @grim harbor

#

if you have some idea @opaque pagoda, let me know

#

sill trying to find some

opaque pagoda Dec 23, 2021, 2:18 PM

#

I don't have any direct idea I'm afraid, no. Maybe try what's in that StackOverflow answer. I'm not sure anyone would have a better idea here beyond not using Koin, or trying to find out what is needed to make it work with our SDK, which you might have a better sense for, we don't use it ourselves and this is a new issue

strong orchid Dec 23, 2021, 2:21 PM

#

Got it, but i used tha same Application class in Sample, and i got the error "startKoin"... so i don't know

opaque pagoda Dec 23, 2021, 2:30 PM

#

I don't think you can just copy and paste our Application into an existing app using Koin, since doesn't Koin require you to call this startKoin function at the top level?

so really like you said: if you don't use startKoin (in our copy-pasted sample application) you get an error because you didn't start it; if you do start it, you get an error later about injecting that Bluetooth class.

So to me the options are to rip Koin out from your app entirely; or figure out what causes the injection error. I'd love to help you with the second option but nobody at Stripe is likely to have any great sense of how to proceed there (and it's a holiday for most of the company so there are not a lot of eyes to look at this right now unfortunately )

strong orchid Dec 23, 2021, 6:04 PM

#

ok thanks a lot

#

I'll keep digging, if I didn't solve I'll come back here after the next week

#

🙂

mystic ermine Dec 23, 2021, 6:14 PM

#

Happy Holidays! 🙂
The thread will be archived soon but if you have any follow ups later, you can ask the moderator (whoever is available) to re-open it.

#thiago-android-terminal