search

sumup / sumup-android-sdk

Sample App for the SumUp Android SDK
Other
100 stars 29 forks source link

NumberFormatException after failed checkout #134

Open danielebart opened 4 years ago

danielebart commented 4 years ago

Hello, we have a few crashes with the following ST:

Caused by java.lang.NumberFormatException: s == null
       at java.lang.Integer.parseInt(Integer.java:577)
       at java.lang.Integer.valueOf(Integer.java:801)
       at com.sumup.merchant.controllers.CheckoutFlowController.sendCancelCheckout(SourceFile:184)
       at java.lang.reflect.Method.invoke(Method.java)
       at org.greenrobot.eventbus.EventBus.invokeSubscriber(EventBus.java:507)
       at org.greenrobot.eventbus.EventBus.postToSubscription(EventBus.java:434)
       at org.greenrobot.eventbus.EventBus.postSingleEventForEventType(EventBus.java:411)
       at org.greenrobot.eventbus.EventBus.postSingleEvent(EventBus.java:384)
       at org.greenrobot.eventbus.EventBus.post(EventBus.java:265)
       at com.sumup.merchant.ui.Fragments.MobilePaymentFragment.sendCancelCheckoutRequest(SourceFile:347)
       at com.sumup.merchant.ui.Fragments.MobilePaymentFragment.access$1500(SourceFile:84)
       at com.sumup.merchant.ui.Fragments.MobilePaymentFragment$2.onClick(SourceFile:560)
       at androidx.appcompat.app.AlertController$ButtonHandler.handleMessage(AlertController.java:167)
       at android.os.Handler.dispatchMessage(Handler.java:107)
       at android.os.Looper.loop(Looper.java:237)
       at android.app.ActivityThread.main(ActivityThread.java:7948)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1075)

From our logs this crash seems to happen after retrying a previously failed payment. In all our crashes we clearly see that there are two attempt to start a payment before this exception is thrown.

SDK Version: 3.2.2 Android Version: we are receiving crashes on Android 9 & 10

Mike-Lopez commented 4 years ago

@danielebart

I will make a ticket for our developers and come back to you with an update soon.

Milszym commented 3 years ago

@Mike-Lopez Hey, have it been already fixed? Can you update us with a progress on this matter?

Mike-Lopez commented 3 years ago

Hello,

My apologies for the late reply.

Once I get this information, I will look further into the issue and get back to you.

Best,

Mike

On Mon, Sep 21, 2020 at 7:25 AM Milszym notifications@github.com wrote:

@Mike-Lopez https://github.com/Mike-Lopez Hey, have it been already fixed? Can you update us with a progress on this matter?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/sumup/sumup-android-sdk/issues/134#issuecomment-695906118, or unsubscribe https://github.com/notifications/unsubscribe-auth/APYALARLJZEA3LNJCVUWNLLSG3PTVANCNFSM4OSUJVCA .

-- Michael Lopez Technical Implementation Associate Sales and Partners

Milszym commented 3 years ago

@Mike-Lopez, Hey, thanks for the reply. I'd rather send it via email. Can you give me your email please?

Mike-Lopez commented 3 years ago

Please email integration@sumup.com.

Milszym commented 3 years ago

Ok sure, thanks I have emailed you with the data that you have asked for.