Fix scenario causing termux-dialog to hang (#200)

If termux-dialog is called while Termux already has a dialog open, the new dialog fails to open and the command will hang until it is manually terminated. This fix allows multiple dialogs to be open at once, ensuring that termux-dialog will not hang.
termux · Jul 26, 2018 · da5b911 · da5b911
1 parent fc8daea
commit da5b911
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/app/src/main/java/com/termux/api/DialogActivity.java b/app/src/main/java/com/termux/api/DialogActivity.java
@@ -55,7 +55,7 @@
  */
 public class DialogActivity extends AppCompatActivity {
 
-    static boolean resultReturned = false;
+    private boolean resultReturned = false;
 
     @Override
     protected void onCreate(Bundle savedInstanceState) {
@@ -82,15 +82,6 @@ protected void onNewIntent(Intent intent) {
         setIntent(intent);
     }
 
-    @Override
-    protected void onPause() {
-        super.onPause();
-
-        if (!resultReturned) {
-            postResult(this, null);
-        }
-    }
-
     @Override
     protected void onDestroy() {
         super.onDestroy();

diff --git a/app/src/main/java/com/termux/api/TermuxApiReceiver.java b/app/src/main/java/com/termux/api/TermuxApiReceiver.java
@@ -64,7 +64,7 @@ public void onReceive(Context context, Intent intent) {
                 }
                 break;
             case "Dialog":
-                context.startActivity(new Intent(context, DialogActivity.class).putExtras(intent.getExtras()).addFlags(Intent.FLAG_ACTIVITY_NEW_TASK));
+                context.startActivity(new Intent(context, DialogActivity.class).putExtras(intent.getExtras()).addFlags(Intent.FLAG_ACTIVITY_MULTIPLE_TASK));
                 break;
             case "Download":
                 DownloadAPI.onReceive(this, context, intent);