Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Show Toast in UI thread and fix unpacking on Android 10 (#9900)
  • Loading branch information
ubulem committed Jun 5, 2020
1 parent 850af80 commit 87b25e5
Show file tree
Hide file tree
Showing 3 changed files with 29 additions and 12 deletions.
Expand Up @@ -20,11 +20,12 @@

package net.minetest.minetest;

import android.content.Context;
import android.content.Intent;
import android.os.AsyncTask;
import android.widget.Toast;

import androidx.appcompat.app.AppCompatActivity;

import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
Expand All @@ -33,10 +34,10 @@

public class CopyZipTask extends AsyncTask<String, Void, String> {

private final WeakReference<Context> contextRef;
private final WeakReference<AppCompatActivity> activityRef;

CopyZipTask(Context context) {
contextRef = new WeakReference<>(context);
CopyZipTask(AppCompatActivity activity) {
activityRef = new WeakReference<>(activity);
}

protected String doInBackground(String... params) {
Expand All @@ -51,11 +52,14 @@ protected void onPostExecute(String result) {

private void copyAsset(String zipName) {
String filename = zipName.substring(zipName.lastIndexOf("/") + 1);
try (InputStream in = contextRef.get().getAssets().open(filename);
try (InputStream in = activityRef.get().getAssets().open(filename);
OutputStream out = new FileOutputStream(zipName)) {
copyFile(in, out);
} catch (IOException e) {
Toast.makeText(contextRef.get(), e.getLocalizedMessage(), Toast.LENGTH_LONG).show();
AppCompatActivity activity = activityRef.get();
if (activity != null) {
activity.runOnUiThread(() -> Toast.makeText(activityRef.get(), e.getLocalizedMessage(), Toast.LENGTH_LONG).show());
}
cancel(true);
}
}
Expand All @@ -68,8 +72,11 @@ private void copyFile(InputStream in, OutputStream out) throws IOException {
}

private void startUnzipService(String file) {
Intent intent = new Intent(contextRef.get(), UnzipService.class);
Intent intent = new Intent(activityRef.get(), UnzipService.class);
intent.putExtra(UnzipService.EXTRA_KEY_IN_FILE, file);
contextRef.get().startService(intent);
AppCompatActivity activity = activityRef.get();
if (activity != null) {
activity.startService(intent);
}
}
}
Expand Up @@ -43,6 +43,7 @@
import java.util.Arrays;
import java.util.List;

import static net.minetest.minetest.UnzipService.ACTION_FAILURE;
import static net.minetest.minetest.UnzipService.ACTION_PROGRESS;
import static net.minetest.minetest.UnzipService.ACTION_UPDATE;
import static net.minetest.minetest.UnzipService.FAILURE;
Expand Down Expand Up @@ -71,6 +72,7 @@ public void onReceive(Context context, Intent intent) {
}
mTextView.setVisibility(View.VISIBLE);
} else if (progress == FAILURE) {
Toast.makeText(MainActivity.this, intent.getStringExtra(ACTION_FAILURE), Toast.LENGTH_LONG).show();
finish();
} else if (progress == SUCCESS)
startNative();
Expand Down Expand Up @@ -138,6 +140,11 @@ private void startNative() {
startActivity(intent);
}

@Override
public void onBackPressed() {
// Prevent abrupt interruption when copy game files from assets
}

@Override
protected void onDestroy() {
super.onDestroy();
Expand Down
Expand Up @@ -42,20 +42,21 @@
public class UnzipService extends IntentService {
public static final String ACTION_UPDATE = "net.minetest.minetest.UPDATE";
public static final String ACTION_PROGRESS = "net.minetest.minetest.PROGRESS";
public static final String ACTION_FAILURE = "net.minetest.minetest.FAILURE";
public static final String EXTRA_KEY_IN_FILE = "file";
public static final int SUCCESS = -1;
public static final int FAILURE = -2;
private static final String TAG = "UnzipService";
private final int id = 1;
private NotificationManager mNotifyManager;
private boolean isSuccess = true;
private String failureMessage;

public UnzipService() {
super("net.minetest.minetest.UnzipService");
}

private void isDir(String dir, String location) {
File f = new File(location + dir);
File f = new File(location, dir);
if (!f.isDirectory())
f.mkdirs();
}
Expand Down Expand Up @@ -99,7 +100,8 @@ private void createNotification() {

private void unzip(Intent intent) {
String zip = intent.getStringExtra(EXTRA_KEY_IN_FILE);
String location = Environment.getExternalStorageDirectory() + "/Minetest/";
isDir("Minetest", Environment.getExternalStorageDirectory().toString());
String location = Environment.getExternalStorageDirectory() + File.separator + "Minetest" + File.separator;
int per = 0;
int size = getSummarySize(zip);
File zipFile = new File(zip);
Expand All @@ -124,13 +126,14 @@ private void unzip(Intent intent) {
}
} catch (IOException e) {
isSuccess = false;
Toast.makeText(this, e.getLocalizedMessage(), Toast.LENGTH_LONG).show();
failureMessage = e.getLocalizedMessage();
}
}

private void publishProgress(int progress) {
Intent intentUpdate = new Intent(ACTION_UPDATE);
intentUpdate.putExtra(ACTION_PROGRESS, progress);
if (!isSuccess) intentUpdate.putExtra(ACTION_FAILURE, failureMessage);
sendBroadcast(intentUpdate);
}

Expand Down

0 comments on commit 87b25e5

Please sign in to comment.