aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDominik Schürmann <dominik@dominikschuermann.de>2015-05-10 13:16:03 +0200
committerDominik Schürmann <dominik@dominikschuermann.de>2015-05-10 13:16:03 +0200
commita35594201a7b96d091b0a1a7ef46fb90ff5e0bb5 (patch)
tree2459af27361ce5e17ca1f50c7963f445db55ce6e
parente4b7920edcc04bed6524929b85ef1acb2c6a01d6 (diff)
downloadopen-keychain-a35594201a7b96d091b0a1a7ef46fb90ff5e0bb5.tar.gz
open-keychain-a35594201a7b96d091b0a1a7ef46fb90ff5e0bb5.tar.bz2
open-keychain-a35594201a7b96d091b0a1a7ef46fb90ff5e0bb5.zip
Fixing rotate crashes in create key
-rw-r--r--OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CreateKeyEmailFragment.java6
-rw-r--r--OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/dialog/ProgressDialogFragment.java8
2 files changed, 5 insertions, 9 deletions
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CreateKeyEmailFragment.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CreateKeyEmailFragment.java
index dbff4fb9f..597f04d6b 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CreateKeyEmailFragment.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/CreateKeyEmailFragment.java
@@ -48,7 +48,7 @@ import java.util.regex.Pattern;
public class CreateKeyEmailFragment extends Fragment {
private CreateKeyActivity mCreateKeyActivity;
private EmailEditText mEmailEdit;
- private ArrayList<EmailAdapter.ViewModel> mAdditionalEmailModels;
+ private ArrayList<EmailAdapter.ViewModel> mAdditionalEmailModels = new ArrayList<>();
private EmailAdapter mEmailAdapter;
// NOTE: Do not use more complicated pattern like defined in android.util.Patterns.EMAIL_ADDRESS
@@ -119,10 +119,6 @@ public class CreateKeyEmailFragment extends Fragment {
emailsRecyclerView.setLayoutManager(new LinearLayoutManager(getActivity()));
emailsRecyclerView.setItemAnimator(new DefaultItemAnimator());
- // initial values
- if (mAdditionalEmailModels == null) {
- mAdditionalEmailModels = new ArrayList<>();
- }
if (mEmailAdapter == null) {
mEmailAdapter = new EmailAdapter(mAdditionalEmailModels, new View.OnClickListener() {
@Override
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/dialog/ProgressDialogFragment.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/dialog/ProgressDialogFragment.java
index b58f584c8..545500cd4 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/dialog/ProgressDialogFragment.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/dialog/ProgressDialogFragment.java
@@ -94,12 +94,12 @@ public class ProgressDialogFragment extends DialogFragment {
/** Updates progress of dialog */
public void setProgress(String message, int progress, int max) {
- if (mIsCancelled) {
- return;
- }
-
ProgressDialog dialog = (ProgressDialog) getDialog();
+ if (mIsCancelled || dialog == null) {
+ return;
+ }
+
dialog.setMessage(message);
dialog.setProgress(progress);
dialog.setMax(max);