From ea766d970f340fff7d2395aa2035b2d51d82d6c7 Mon Sep 17 00:00:00 2001 From: Daniel Gultsch Date: Thu, 27 Mar 2014 16:09:23 +0100 Subject: [PATCH] crash reporter --- res/values/strings.xml | 4 + res/xml/preferences.xml | 8 ++ .../ui/ConversationActivity.java | 2 + .../conversations/utils/ExceptionHelper.java | 80 +++++++++++++++++++ 4 files changed, 94 insertions(+) diff --git a/res/values/strings.xml b/res/values/strings.xml index faf8fff47..d9f6b9b5a 100644 --- a/res/values/strings.xml +++ b/res/values/strings.xml @@ -40,4 +40,8 @@ Invitation sent Account offline You have to be online to invite people to conferences + Conversations has crashed + By sending in stack traces you are helping the ongoing development of Conversations\nWarning: This will use your XMPP account to send the stack trace to the developer. + Send now + Never ask again diff --git a/res/xml/preferences.xml b/res/xml/preferences.xml index 49b3e5803..7339565ed 100644 --- a/res/xml/preferences.xml +++ b/res/xml/preferences.xml @@ -55,4 +55,12 @@ android:summary="Use room’s subject to identify Conferences" android:defaultValue="true"/> + + + diff --git a/src/eu/siacs/conversations/ui/ConversationActivity.java b/src/eu/siacs/conversations/ui/ConversationActivity.java index 4acddf8a6..45a3376a5 100644 --- a/src/eu/siacs/conversations/ui/ConversationActivity.java +++ b/src/eu/siacs/conversations/ui/ConversationActivity.java @@ -8,6 +8,7 @@ import eu.siacs.conversations.entities.Account; import eu.siacs.conversations.entities.Contact; import eu.siacs.conversations.entities.Conversation; import eu.siacs.conversations.entities.Message; +import eu.siacs.conversations.utils.ExceptionHelper; import eu.siacs.conversations.utils.UIHelper; import android.os.Bundle; import android.preference.PreferenceManager; @@ -454,6 +455,7 @@ public class ConversationActivity extends XmppActivity { selectedConversation = conversationList.get(0); swapConversationFragment(); } + ExceptionHelper.checkForCrash(this,this.xmppConnectionService); } } } diff --git a/src/eu/siacs/conversations/utils/ExceptionHelper.java b/src/eu/siacs/conversations/utils/ExceptionHelper.java index def184d0d..63c4ab6a6 100644 --- a/src/eu/siacs/conversations/utils/ExceptionHelper.java +++ b/src/eu/siacs/conversations/utils/ExceptionHelper.java @@ -1,6 +1,25 @@ package eu.siacs.conversations.utils; +import java.io.BufferedInputStream; +import java.io.BufferedReader; +import java.io.FileInputStream; +import java.io.FileNotFoundException; +import java.io.IOException; +import java.io.InputStreamReader; +import java.util.List; + +import eu.siacs.conversations.R; +import eu.siacs.conversations.entities.Account; +import eu.siacs.conversations.entities.Conversation; +import eu.siacs.conversations.entities.Message; +import eu.siacs.conversations.services.XmppConnectionService; +import android.app.AlertDialog; import android.content.Context; +import android.content.DialogInterface; +import android.content.SharedPreferences; +import android.content.DialogInterface.OnClickListener; +import android.preference.PreferenceManager; +import android.util.Log; public class ExceptionHelper { public static void init(Context context) { @@ -8,4 +27,65 @@ public class ExceptionHelper { Thread.setDefaultUncaughtExceptionHandler(new ExceptionHandler(context)); } } + + public static void checkForCrash(Context context, final XmppConnectionService service) { + try { + final SharedPreferences preferences = PreferenceManager.getDefaultSharedPreferences(context); + boolean neverSend = preferences.getBoolean("never_send",false); + if (neverSend) { + return; + } + FileInputStream file = context.openFileInput("stacktrace.txt"); + InputStreamReader inputStreamReader = new InputStreamReader( + file); + BufferedReader bufferedReader = new BufferedReader( + inputStreamReader); + final StringBuilder stacktrace = new StringBuilder(); + String line; + while((line = bufferedReader.readLine()) != null) { + stacktrace.append(line); + stacktrace.append('\n'); + } + file.close(); + context.deleteFile("stacktrace.txt"); + AlertDialog.Builder builder = new AlertDialog.Builder(context); + builder.setTitle(context.getString(R.string.crash_report_title)); + builder.setMessage(context.getText(R.string.crash_report_message)); + builder.setPositiveButton(context.getText(R.string.send_now), new OnClickListener() { + + @Override + public void onClick(DialogInterface dialog, int which) { + List accounts = service.getAccounts(); + Account account = null; + for(int i = 0; i < accounts.size(); ++i) { + if (!accounts.get(i).isOptionSet(Account.OPTION_DISABLED)) { + account = accounts.get(i); + break; + } + } + if (account!=null) { + Log.d("xmppService","using account="+account.getJid()+" to send in stack trace"); + Conversation conversation = service.findOrCreateConversation(account, "bugs@siacs.eu", false); + Message message = new Message(conversation, stacktrace.toString(), Message.ENCRYPTION_NONE); + service.sendMessage(message, null); + } + } + }); + builder.setNegativeButton(context.getText(R.string.send_never),new OnClickListener() { + + @Override + public void onClick(DialogInterface dialog, int which) { + preferences.edit().putBoolean("never_send", true).commit(); + } + }); + builder.create().show(); + } catch (FileNotFoundException e) { + // TODO Auto-generated catch block + e.printStackTrace(); + } catch (IOException e) { + // TODO Auto-generated catch block + e.printStackTrace(); + } + + } }