Refactor IAP extension for Android and iOS platforms

This commit is contained in:
Nick Leeman 2024-04-28 10:07:39 +02:00
parent 6e27d8ad2e
commit b358e0bce4
5 changed files with 15 additions and 15 deletions

View File

@ -352,7 +352,7 @@ static void HandlePurchaseResult(const IAPCommand* cmd)
if (!dmScript::SetupCallback(cmd->m_Callback)) if (!dmScript::SetupCallback(cmd->m_Callback))
{ {
dmLogError("No callback set for purchase result!"); dmLogError("Failed to setup callback for purchase result!");
assert(top == lua_gettop(L)); assert(top == lua_gettop(L));
return; return;
} }
@ -368,7 +368,7 @@ static void HandlePurchaseResult(const IAPCommand* cmd)
IAP_PushError(L, "purchase response was null", REASON_UNSPECIFIED); IAP_PushError(L, "purchase response was null", REASON_UNSPECIFIED);
} }
} else if (cmd->m_ResponseCode == BILLING_RESPONSE_RESULT_USER_CANCELED) { } else if (cmd->m_ResponseCode == BILLING_RESPONSE_RESULT_USER_CANCELED) {
dmLogError("User canceled purchase"); dmLogInfo("User canceled purchase");
lua_pushnil(L); lua_pushnil(L);
IAP_PushError(L, "user canceled purchase", REASON_USER_CANCELED); IAP_PushError(L, "user canceled purchase", REASON_USER_CANCELED);
} else { } else {
@ -389,9 +389,7 @@ static dmExtension::Result InitializeIAP(dmExtension::Params* params)
dmAndroid::ThreadAttacher threadAttacher; dmAndroid::ThreadAttacher threadAttacher;
JNIEnv* env = threadAttacher.GetEnv(); JNIEnv* env = threadAttacher.GetEnv();
const char* provider = dmConfigFile::GetString(params->m_ConfigFile, "android.iap_provider", "GooglePlay");
const char* class_name = "com.defold.iap.IapGooglePlay"; const char* class_name = "com.defold.iap.IapGooglePlay";
jclass iap_class = dmAndroid::LoadClass(env, class_name); jclass iap_class = dmAndroid::LoadClass(env, class_name);
jclass iap_jni_class = dmAndroid::LoadClass(env, "com.defold.iap.IapJNI"); jclass iap_jni_class = dmAndroid::LoadClass(env, "com.defold.iap.IapJNI");
@ -426,15 +424,16 @@ static void IAP_OnCommand(IAPCommand* cmd, void*)
{ {
switch (cmd->m_Command) switch (cmd->m_Command)
{ {
case IAP_PRODUCT_RESULT: case IAP_PRODUCT_RESULT:
HandleProductResult(cmd); HandleProductResult(cmd);
break; break;
case IAP_PURCHASE_RESULT:
HandlePurchaseResult(cmd); case IAP_PURCHASE_RESULT:
break; HandlePurchaseResult(cmd);
break;
default: default:
assert(false); assert(false);
} }
if (cmd->m_Data) { if (cmd->m_Data) {

View File

@ -325,6 +325,7 @@ static void HandlePurchaseResult(IAPCommand* cmd)
if (!dmScript::SetupCallback(cmd->m_Callback)) if (!dmScript::SetupCallback(cmd->m_Callback))
{ {
dmLogError("Failed to setup callback for purchase result");
assert(top == lua_gettop(L)); assert(top == lua_gettop(L));
return; return;
} }

View File

@ -1,4 +1,4 @@
#if !defined(DM_PLATFORM_HTML5) && !defined(DM_PLATFORM_ANDROID) && !defined(DM_PLATFORM_IOS) #if !defined(DM_PLATFORM_ANDROID) && !defined(DM_PLATFORM_IOS)
extern "C" void IAPExt() extern "C" void IAPExt()
{ {

View File

@ -1,4 +1,4 @@
#if defined(DM_PLATFORM_HTML5) || defined(DM_PLATFORM_ANDROID) || defined(DM_PLATFORM_IOS) #if defined(DM_PLATFORM_ANDROID) || defined(DM_PLATFORM_IOS)
#include <dmsdk/sdk.h> #include <dmsdk/sdk.h>

View File

@ -1,4 +1,4 @@
#if defined(DM_PLATFORM_HTML5) || defined(DM_PLATFORM_ANDROID) || defined(DM_PLATFORM_IOS) #if defined(DM_PLATFORM_ANDROID) || defined(DM_PLATFORM_IOS)
#ifndef IAP_PRIVATE_H #ifndef IAP_PRIVATE_H
#define IAP_PRIVATE_H #define IAP_PRIVATE_H