Add core.open_url() to main menu API (#8592)

This commit is contained in:
rubenwardy 2020-05-17 19:09:10 +01:00 committed by GitHub
parent c1ce4be756
commit a9c3a42323
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
10 changed files with 98 additions and 96 deletions

@ -22,6 +22,7 @@ package net.minetest.minetest;
import android.app.NativeActivity; import android.app.NativeActivity;
import android.content.Intent; import android.content.Intent;
import android.net.Uri;
import android.os.Build; import android.os.Build;
import android.os.Bundle; import android.os.Bundle;
import android.view.View; import android.view.View;
@ -117,4 +118,9 @@ public class GameActivity extends NativeActivity {
public int getDisplayWidth() { public int getDisplayWidth() {
return getResources().getDisplayMetrics().widthPixels; return getResources().getDisplayMetrics().widthPixels;
} }
public void openURL(String url) {
Intent browserIntent = new Intent(Intent.ACTION_VIEW, Uri.parse(url));
startActivity(browserIntent);
}
} }

@ -16,7 +16,6 @@
--51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. --51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
local store = { packages = {}, packages_full = {} } local store = { packages = {}, packages_full = {} }
local package_dialog = {}
-- Screenshot -- Screenshot
local screenshot_dir = core.get_cache_path() .. DIR_DELIM .. "cdb" local screenshot_dir = core.get_cache_path() .. DIR_DELIM .. "cdb"
@ -44,8 +43,6 @@ local filter_types_type = {
} }
local function download_package(param) local function download_package(param)
if core.download_file(param.package.url, param.filename) then if core.download_file(param.package.url, param.filename) then
return { return {
@ -195,74 +192,6 @@ local function get_screenshot(package)
return defaulttexturedir .. "loading_screenshot.png" return defaulttexturedir .. "loading_screenshot.png"
end end
function package_dialog.get_formspec()
local package = package_dialog.package
store.update_paths()
local formspec = {
"size[9,4;true]",
"image[0,1;4.5,3;", core.formspec_escape(get_screenshot(package)), ']',
"label[3.8,1;",
minetest.colorize(mt_color_green, core.formspec_escape(package.title)), "\n",
minetest.colorize('#BFBFBF', "by " .. core.formspec_escape(package.author)), "]",
"textarea[4,2;5.3,2;;;", core.formspec_escape(package.short_description), "]",
"button[0,0;2,1;back;", fgettext("Back"), "]",
}
if not package.path then
formspec[#formspec + 1] = "button[7,0;2,1;install;"
formspec[#formspec + 1] = fgettext("Install")
formspec[#formspec + 1] = "]"
elseif package.installed_release < package.release then
-- The install_ action also handles updating
formspec[#formspec + 1] = "button[7,0;2,1;install;"
formspec[#formspec + 1] = fgettext("Update")
formspec[#formspec + 1] = "]"
formspec[#formspec + 1] = "button[5,0;2,1;uninstall;"
formspec[#formspec + 1] = fgettext("Uninstall")
formspec[#formspec + 1] = "]"
else
formspec[#formspec + 1] = "button[7,0;2,1;uninstall;"
formspec[#formspec + 1] = fgettext("Uninstall")
formspec[#formspec + 1] = "]"
end
return table.concat(formspec, "")
end
function package_dialog.handle_submit(this, fields)
if fields.back then
this:delete()
return true
end
if fields.install then
start_install(this, package_dialog.package)
return true
end
if fields.uninstall then
local dlg_delmod = create_delete_content_dlg(package_dialog.package)
dlg_delmod:set_parent(this)
this:hide()
dlg_delmod:show()
return true
end
return false
end
function package_dialog.create(package)
package_dialog.package = package
return dialog_create("package_view",
package_dialog.get_formspec,
package_dialog.handle_submit,
nil)
end
function store.load() function store.load()
local tmpdir = os.tempfolder() local tmpdir = os.tempfolder()
local target = tmpdir .. DIR_DELIM .. "packages.json" local target = tmpdir .. DIR_DELIM .. "packages.json"
@ -462,44 +391,45 @@ function store.get_formspec(dlgdata)
minetest.colorize("#BFBFBF", " by " .. package.author)) minetest.colorize("#BFBFBF", " by " .. package.author))
formspec[#formspec + 1] = "]" formspec[#formspec + 1] = "]"
-- description
if package.path and package.installed_release < package.release then
formspec[#formspec + 1] = "textarea[1.25,0.3;7.5,1;;;"
else
formspec[#formspec + 1] = "textarea[1.25,0.3;9,1;;;"
end
formspec[#formspec + 1] = core.formspec_escape(package.short_description)
formspec[#formspec + 1] = "]"
-- buttons -- buttons
local description_width = 7.5
if not package.path then if not package.path then
formspec[#formspec + 1] = "button[9.9,0;1.5,1;install_" formspec[#formspec + 1] = "button[8.4,0;1.5,1;install_"
formspec[#formspec + 1] = tostring(i) formspec[#formspec + 1] = tostring(i)
formspec[#formspec + 1] = ";" formspec[#formspec + 1] = ";"
formspec[#formspec + 1] = fgettext("Install") formspec[#formspec + 1] = fgettext("Install")
formspec[#formspec + 1] = "]" formspec[#formspec + 1] = "]"
else else
if package.installed_release < package.release then if package.installed_release < package.release then
description_width = 6
-- The install_ action also handles updating -- The install_ action also handles updating
formspec[#formspec + 1] = "button[8.4,0;1.5,1;install_" formspec[#formspec + 1] = "button[6.9,0;1.5,1;install_"
formspec[#formspec + 1] = tostring(i) formspec[#formspec + 1] = tostring(i)
formspec[#formspec + 1] = ";" formspec[#formspec + 1] = ";"
formspec[#formspec + 1] = fgettext("Update") formspec[#formspec + 1] = fgettext("Update")
formspec[#formspec + 1] = "]" formspec[#formspec + 1] = "]"
end end
formspec[#formspec + 1] = "button[9.9,0;1.5,1;uninstall_" formspec[#formspec + 1] = "button[8.4,0;1.5,1;uninstall_"
formspec[#formspec + 1] = tostring(i) formspec[#formspec + 1] = tostring(i)
formspec[#formspec + 1] = ";" formspec[#formspec + 1] = ";"
formspec[#formspec + 1] = fgettext("Uninstall") formspec[#formspec + 1] = fgettext("Uninstall")
formspec[#formspec + 1] = "]" formspec[#formspec + 1] = "]"
end end
--formspec[#formspec + 1] = "button[9.9,0;1.5,1;view_" formspec[#formspec + 1] = "button[9.9,0;1.5,1;view_"
--formspec[#formspec + 1] = tostring(i) formspec[#formspec + 1] = tostring(i)
--formspec[#formspec + 1] = ";" formspec[#formspec + 1] = ";"
--formspec[#formspec + 1] = fgettext("View") formspec[#formspec + 1] = fgettext("View")
--formspec[#formspec + 1] = "]" formspec[#formspec + 1] = "]"
-- description
formspec[#formspec + 1] = "textarea[1.25,0.3;"
formspec[#formspec + 1] = tostring(description_width)
formspec[#formspec + 1] = ",1;;;"
formspec[#formspec + 1] = core.formspec_escape(package.short_description)
formspec[#formspec + 1] = "]"
formspec[#formspec + 1] = "container_end[]" formspec[#formspec + 1] = "container_end[]"
end end
@ -576,10 +506,9 @@ function store.handle_submit(this, fields)
end end
if fields["view_" .. i] then if fields["view_" .. i] then
local dlg = package_dialog.create(package) local url = ("%s/packages/%s?protocol_version=%d"):format(
dlg:set_parent(this) core.settings:get("contentdb_url"), package.id, core.get_max_supp_proto())
this:hide() core.open_url(url)
dlg:show()
return true return true
end end
end end

@ -101,8 +101,8 @@ return {
local logofile = defaulttexturedir .. "logo.png" local logofile = defaulttexturedir .. "logo.png"
local version = core.get_version() local version = core.get_version()
return "image[0.5,1;" .. core.formspec_escape(logofile) .. "]" .. return "image[0.5,1;" .. core.formspec_escape(logofile) .. "]" ..
"label[0.5,3.2;" .. version.project .. " " .. version.string .. "]" .. "label[0.5,2.8;" .. version.project .. " " .. version.string .. "]" ..
"label[0.5,3.5;http://minetest.net]" .. "button[0.5,3;2,2;homepage;minetest.net]" ..
"tablecolumns[color;text]" .. "tablecolumns[color;text]" ..
"tableoptions[background=#00000000;highlight=#00000000;border=false]" .. "tableoptions[background=#00000000;highlight=#00000000;border=false]" ..
"table[3.5,-0.25;8.5,6.05;list_credits;" .. "table[3.5,-0.25;8.5,6.05;list_credits;" ..
@ -115,5 +115,10 @@ return {
"#FFFF00," .. fgettext("Previous Contributors") .. ",," .. "#FFFF00," .. fgettext("Previous Contributors") .. ",," ..
buildCreditList(previous_contributors) .. "," .. buildCreditList(previous_contributors) .. "," ..
";1]" ";1]"
end end,
cbf_button_handler = function(this, fields, name, tabdata)
if fields.homepage then
core.open_url("https://www.minetest.net")
end
end,
} }

@ -234,6 +234,11 @@ core.get_min_supp_proto()
core.get_max_supp_proto() core.get_max_supp_proto()
^ returns the maximum supported network protocol version ^ returns the maximum supported network protocol version
Other:
core.open_url(url)
^ opens the URL in a web browser, returns false on failure.
^ Must begin with http:// or https://
Async: Async:
core.handle_async(async_job,parameters,finished) core.handle_async(async_job,parameters,finished)
^ execute a function asynchronously ^ execute a function asynchronously

@ -33,22 +33,28 @@ with this program; if not, write to the Free Software Foundation, Inc.,
#include <wincrypt.h> #include <wincrypt.h>
#include <algorithm> #include <algorithm>
#include <shlwapi.h> #include <shlwapi.h>
#include <shellapi.h>
#endif #endif
#if !defined(_WIN32) #if !defined(_WIN32)
#include <unistd.h> #include <unistd.h>
#include <sys/utsname.h> #include <sys/utsname.h>
#if !defined(__ANDROID__)
#include <spawn.h>
#endif
#endif #endif
#if defined(__hpux) #if defined(__hpux)
#define _PSTAT64 #define _PSTAT64
#include <sys/pstat.h> #include <sys/pstat.h>
#endif #endif
#if defined(__ANDROID__)
#include "porting_android.h"
#endif
#include "config.h" #include "config.h"
#include "debug.h" #include "debug.h"
#include "filesys.h" #include "filesys.h"
#include "log.h" #include "log.h"
#include "util/string.h" #include "util/string.h"
#include "settings.h"
#include <list> #include <list>
#include <cstdarg> #include <cstdarg>
#include <cstdio> #include <cstdio>
@ -697,6 +703,28 @@ int mt_snprintf(char *buf, const size_t buf_size, const char *fmt, ...)
return c; return c;
} }
bool openURL(const std::string &url)
{
if ((url.substr(0, 7) != "http://" && url.substr(0, 8) != "https://") ||
url.find_first_of("\r\n") != std::string::npos) {
errorstream << "Invalid url: " << url << std::endl;
return false;
}
#if defined(_WIN32)
return (intptr_t)ShellExecuteA(NULL, NULL, url.c_str(), NULL, NULL, SW_SHOWNORMAL) > 32;
#elif defined(__ANDROID__)
openURLAndroid(url);
return true;
#elif defined(__APPLE__)
const char *argv[] = {"open", url.c_str(), NULL};
return posix_spawnp(NULL, "open", NULL, NULL, (char**)argv, environ) == 0;
#else
const char *argv[] = {"xdg-open", url.c_str(), NULL};
return posix_spawnp(NULL, "xdg-open", NULL, NULL, (char**)argv, environ) == 0;
#endif
}
// Load performance counter frequency only once at startup // Load performance counter frequency only once at startup
#ifdef _WIN32 #ifdef _WIN32

@ -329,6 +329,9 @@ bool secure_rand_fill_buf(void *buf, size_t len);
void attachOrCreateConsole(); void attachOrCreateConsole();
int mt_snprintf(char *buf, const size_t buf_size, const char *fmt, ...); int mt_snprintf(char *buf, const size_t buf_size, const char *fmt, ...);
bool openURL(const std::string &url);
} // namespace porting } // namespace porting
#ifdef __ANDROID__ #ifdef __ANDROID__

@ -213,6 +213,18 @@ void showInputDialog(const std::string &acceptButton, const std::string &hint,
jacceptButton, jhint, jcurrent, jeditType); jacceptButton, jhint, jcurrent, jeditType);
} }
void openURLAndroid(const std::string &url)
{
jmethodID url_open = jnienv->GetMethodID(nativeActivity, "openURL",
"(Ljava/lang/String;)V");
FATAL_ERROR_IF(url_open == nullptr,
"porting::openURLAndroid unable to find java openURL method");
jstring jurl = jnienv->NewStringUTF(url.c_str());
jnienv->CallVoidMethod(app_global->activity->clazz, url_open, jurl);
}
int getInputDialogState() int getInputDialogState()
{ {
jmethodID dialogstate = jnienv->GetMethodID(nativeActivity, jmethodID dialogstate = jnienv->GetMethodID(nativeActivity,

@ -58,6 +58,8 @@ void initializePathsAndroid();
void showInputDialog(const std::string &acceptButton, void showInputDialog(const std::string &acceptButton,
const std::string &hint, const std::string &current, int editType); const std::string &hint, const std::string &current, int editType);
void openURLAndroid(const std::string &url);
/** /**
* WORKAROUND for not working callbacks from java -> c++ * WORKAROUND for not working callbacks from java -> c++
* get current state of input dialog * get current state of input dialog

@ -1063,6 +1063,14 @@ int ModApiMainMenu::l_get_max_supp_proto(lua_State *L)
return 1; return 1;
} }
/******************************************************************************/
int ModApiMainMenu::l_open_url(lua_State *L)
{
std::string url = luaL_checkstring(L, 1);
lua_pushboolean(L, porting::openURL(url));
return 1;
}
/******************************************************************************/ /******************************************************************************/
int ModApiMainMenu::l_do_async_callback(lua_State *L) int ModApiMainMenu::l_do_async_callback(lua_State *L)
{ {
@ -1125,6 +1133,7 @@ void ModApiMainMenu::Initialize(lua_State *L, int top)
API_FCT(get_screen_info); API_FCT(get_screen_info);
API_FCT(get_min_supp_proto); API_FCT(get_min_supp_proto);
API_FCT(get_max_supp_proto); API_FCT(get_max_supp_proto);
API_FCT(open_url);
API_FCT(do_async_callback); API_FCT(do_async_callback);
} }

@ -145,6 +145,9 @@ private:
static int l_get_max_supp_proto(lua_State *L); static int l_get_max_supp_proto(lua_State *L);
// other
static int l_open_url(lua_State *L);
// async // async
static int l_do_async_callback(lua_State *L); static int l_do_async_callback(lua_State *L);