better onActivityResult handling for qr code import, remove unimplemented import button

2 files changed, 18 insertions, 26 deletions

diff --git a/OpenPGP-Keychain/src/org/sufficientlysecure/keychain/ui/ImportKeysActivity.java b/OpenPGP-Keychain/src/org/sufficientlysecure/keychain/ui/ImportKeysActivity.java
index 698794e5c..790913d91 100644
--- a/OpenPGP-Keychain/src/org/sufficientlysecure/keychain/ui/ImportKeysActivity.java
+++ b/OpenPGP-Keychain/src/org/sufficientlysecure/keychain/ui/ImportKeysActivity.java
@@ -79,7 +79,8 @@ public class ImportKeysActivity extends DrawerActivity implements OnNavigationLi
     Fragment mCurrentFragment;
 
     BootstrapButton mImportButton;
-    BootstrapButton mImportSignUploadButton;
+
+    // BootstrapButton mImportSignUploadButton;
 
     @Override
     protected void onCreate(Bundle savedInstanceState) {
@@ -94,13 +95,13 @@ public class ImportKeysActivity extends DrawerActivity implements OnNavigationLi
                 importKeys();
             }
         });
-        mImportSignUploadButton = (BootstrapButton) findViewById(R.id.import_sign_and_upload);
-        mImportSignUploadButton.setOnClickListener(new OnClickListener() {
-            @Override
-            public void onClick(View v) {
-                signAndUploadOnClick();
-            }
-        });
+        // mImportSignUploadButton = (BootstrapButton) findViewById(R.id.import_sign_and_upload);
+        // mImportSignUploadButton.setOnClickListener(new OnClickListener() {
+        // @Override
+        // public void onClick(View v) {
+        // signAndUploadOnClick();
+        // }
+        // });
 
         getSupportActionBar().setDisplayShowTitleEnabled(false);
 
@@ -300,15 +301,6 @@ public class ImportKeysActivity extends DrawerActivity implements OnNavigationLi
     // }
     // }
 
-    @Override
-    protected void onActivityResult(int requestCode, int resultCode, Intent data) {
-        // required for qr code scanning
-        if (mCurrentFragment != null) {
-            mCurrentFragment.onActivityResult(requestCode, resultCode, data);
-        }
-        // super.onActivityResult(requestCode, resultCode, data);
-    }
-
     /**
      * Import keys with mImportData
      */
@@ -419,14 +411,14 @@ public class ImportKeysActivity extends DrawerActivity implements OnNavigationLi
         importKeys();
     }
 
-    public void signAndUploadOnClick() {
-        // first, import!
-        // importOnClick(view);
-
-        // TODO: implement sign and upload!
-        Toast.makeText(ImportKeysActivity.this, "Not implemented right now!", Toast.LENGTH_SHORT)
-                .show();
-    }
+    // public void signAndUploadOnClick() {
+    // // first, import!
+    // // importOnClick(view);
+    //
+    // // TODO: implement sign and upload!
+    // Toast.makeText(ImportKeysActivity.this, "Not implemented right now!", Toast.LENGTH_SHORT)
+    // .show();
+    // }
 
     /**
      * NFC
diff --git a/OpenPGP-Keychain/src/org/sufficientlysecure/keychain/ui/ImportKeysQrCodeFragment.java b/OpenPGP-Keychain/src/org/sufficientlysecure/keychain/ui/ImportKeysQrCodeFragment.java
index bdedbec0a..58a1400b0 100644
--- a/OpenPGP-Keychain/src/org/sufficientlysecure/keychain/ui/ImportKeysQrCodeFragment.java
+++ b/OpenPGP-Keychain/src/org/sufficientlysecure/keychain/ui/ImportKeysQrCodeFragment.java
@@ -91,7 +91,7 @@ public class ImportKeysQrCodeFragment extends Fragment {
 
     @Override
     public void onActivityResult(int requestCode, int resultCode, Intent data) {
-        switch (requestCode) {
+        switch (requestCode & 0xFFFF) {
         case IntentIntegrator.REQUEST_CODE: {
             IntentResult scanResult = IntentIntegrator.parseActivityResult(requestCode, resultCode,
                     data);