Merge pull request #427 from Automattic/hamorillo/426-bugfix-oauth-denied

hamorillo · web-flow · commit 860f0b19319c · 2024-11-12T09:40:17.000+01:00
QuickEditor: Check not null when getting "access_token" from OAuth flow
diff --git a/gravatar-quickeditor/src/main/java/com/gravatar/quickeditor/ui/oauth/OAuthPage.kt b/gravatar-quickeditor/src/main/java/com/gravatar/quickeditor/ui/oauth/OAuthPage.kt
@@ -77,7 +77,7 @@ internal fun OAuthPage(
                         split.first() to split.last()
                     }
                     ?.get("access_token")
-                    .let { URLDecoder.decode(it, "UTF-8") }
+                    ?.let { URLDecoder.decode(it, "UTF-8") }
 
                 if (token != null) {
                     viewModel.tokenReceived(email, token)

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,7 @@ internal fun OAuthPage(`
`77`	`77`	`split.first() to split.last()`
`78`	`78`	`}`
`79`	`79`	`?.get("access_token")`
`80`		`- .let { URLDecoder.decode(it, "UTF-8") }`
	`80`	`+ ?.let { URLDecoder.decode(it, "UTF-8") }`
`81`	`81`
`82`	`82`	`if (token != null) {`
`83`	`83`	`viewModel.tokenReceived(email, token)`