Skip to content

fix(ohos): bind WXApi event handler on registerApp so the first callback after cold start is not lost (#773) #775

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.

Sign up for GitHub

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

Merged
JarvanMo merged 1 commit into from
Sep 23, 2026
Merged

fix(ohos): bind WXApi event handler on registerApp so the first callback after cold start is not lost (#773) #775

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion packages/_shared/ohos/src/main/ets/components/plugin/FluwxPlugin.ets
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ export default class FluwxPlugin implements FlutterPlugin, MethodCallHandler, Ab
WXAPiHandler.checkWeChatInstallation(result);
break;
case "registerApp":
WXAPiHandler.registerApp(call, result);
WXAPiHandler.registerApp(call, result, this);
break;
case "sendAuth":
this.authHandler?.sendAuth(call, result);
Expand Down Expand Up @@ -262,6 +262,7 @@ export default class FluwxPlugin implements FlutterPlugin, MethodCallHandler, Ab

attemptToResumeMsgFromWx(result: MethodResult) {
WXAPiHandler.wxApi?.handleWant(this.binding?.getAbility().launchWant, this)
result.success(null)
}

async launchMiniProgram(call: MethodCall, result: MethodResult) {
Expand Down
35 changes: 23 additions & 12 deletions packages/_shared/ohos/src/main/ets/components/plugin/handlers/WXAPiHandler.ets
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import * as wechatOpenSDK from "@tencent/wechat_open_sdk"
import { MethodCall, MethodResult } from '@ohos/flutter_ohos'
import { bundleManager, common } from '@kit.AbilityKit'
import { bundleManager, common, Want } from '@kit.AbilityKit'

export class WXAPiHandler {
static wxApi: wechatOpenSDK.WXApi | null = null
Expand All @@ -20,22 +20,33 @@ export class WXAPiHandler {
WXAPiHandler.context = context
}

static registerApp(call: MethodCall, result: MethodResult) {
if (WXAPiHandler.wxApi != null) {
result.success(true)
return
static registerApp(call: MethodCall, result: MethodResult, eventHandler: wechatOpenSDK.WXApiEventHandler) {
if (WXAPiHandler.wxApi == null) {
const appId: string | null = call.argument("appId")
if (!appId) {
result.error("invalid app id", "are you sure your app id is correct ?", appId)
return
}
WXAPiHandler.registerWxAPIInternal(appId)
}
const appId: string | null = call.argument("appId")
if (!appId) {
result.error("invalid app id", "are you sure your app id is correct ?", appId)
return
}

WXAPiHandler.registerWxAPIInternal(appId)

WXAPiHandler.bindEventHandler(eventHandler)
result.success(WXAPiHandler.registered)
}

// The SDK dispatches every response to the handler it received in the most recent handleWant() call,
// and handleWant() is the only way to hand it one. Responses coming back through the openLink channel
// (current WeChat versions) never go through onNewWant, so until something calls handleWant() the
// handler is null and the SDK drops them silently (`this.eventHandler?.onResp(...)`). That is what
// made the first auth / pay after every cold start time out (#773).
//
// An empty Want carries no WeChat payload, so this only binds the handler: nothing is dispatched and the
// launch Want is left for attemptToResumeMsgFromWx(), which apps call once their subscribers are ready.
private static bindEventHandler(eventHandler: wechatOpenSDK.WXApiEventHandler) {
const noPayload: Want = {}
WXAPiHandler.wxApi?.handleWant(noPayload, eventHandler)
}

static checkWeChatInstallation(result: MethodResult) {
const isInstalled = bundleManager.canOpenLink("weixin://") || WXAPiHandler.wxApi?.isWXAppInstalled() === true;
result.success(isInstalled);
Expand Down