Skip to content

MIT Demo App Fixes - #359

Merged
RafalOzogJudopay merged 2 commits into
masterfrom
feature/CT-3272/recurring-payments-fixes
Aug 17, 2026
Merged

MIT Demo App Fixes#359
RafalOzogJudopay merged 2 commits into
masterfrom
feature/CT-3272/recurring-payments-fixes

Conversation

@RafalOzogJudopay

@RafalOzogJudopay RafalOzogJudopay commented Aug 16, 2026

Copy link
Copy Markdown
Contributor
  1. Fixes demo app treatment of Google Pay empty properties as strings.

Google Pay validation fails when optional values come from Demo App as empty strings. They are expected to come as nulls.

Now it passes empty Google Pay properties as null if it's just empty editText, not as empty string.

PS: Simple removal of default values from google_pay_preferences.xml wasn't enough, because whenever user opens it and saves (as an empty value), then it is automatically saved as empty string.

  1. Fixes crash bug (ClassNotFoundException) caused by Proguard that strips references to classes that are used/mentioned only in the preferences XML. It couldn't see GooglePaySettingsFragment then (in signed builds).

Google Pay validation fails when optional values come from Demo App as empty strings, they should come as nulls as SDK expects. Otherwise there is no way of passing null value from the Demo app.
Proguard removes references to classes that are used only exclusively in the prefrences XML, as in this case.
It was causing crash due to "ClassNotFoundException: com.judokit.android.examples.settings.fragments.GooglePaySettingsFragment".
@RafalOzogJudopay RafalOzogJudopay changed the title Fixes demo app treatment of Google Pay empty properties as strings MIT Fixes Aug 17, 2026
@RafalOzogJudopay RafalOzogJudopay changed the title MIT Fixes MIT Demo App Fixes Aug 17, 2026
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@RafalOzogJudopay
RafalOzogJudopay merged commit f0fcc8e into master Aug 17, 2026
2 of 3 checks passed
@RafalOzogJudopay
RafalOzogJudopay deleted the feature/CT-3272/recurring-payments-fixes branch August 17, 2026 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants