From 018a84c8d664d017f24fcf59b99f29944b853066 Mon Sep 17 00:00:00 2001
From: Dominik <dominik@dominikschuermann.de>
Date: Mon, 10 Sep 2012 11:50:41 +0200
Subject: [PATCH] qr code scanning basically works

---
 org_apg/res/layout/import_from_qr_code.xml    |   7 +-
 .../android/apg/helper/PGPMain.java           |   6 +-
 .../android/apg/service/ApgService.java       |   5 +
 .../apg/ui/ImportFromQRCodeActivity.java      | 158 ++++++++++++------
 .../android/apg/ui/KeyListActivity.java       |   7 +-
 .../apg/ui/KeyServerExportActivity.java       |   1 +
 6 files changed, 127 insertions(+), 57 deletions(-)

diff --git a/org_apg/res/layout/import_from_qr_code.xml b/org_apg/res/layout/import_from_qr_code.xml
index 54f4355b3..f20dbbb68 100644
--- a/org_apg/res/layout/import_from_qr_code.xml
+++ b/org_apg/res/layout/import_from_qr_code.xml
@@ -12,30 +12,33 @@
         android:orientation="vertical"
         android:paddingLeft="10dp"
         android:paddingRight="10dp" >
-        
-        
+
         <Button
             android:id="@+id/import_from_qr_code_import"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
+            android:onClick="importOnClick"
             android:text="@string/import_from_qr_code_import" />
 
         <Button
             android:id="@+id/import_from_qr_code_import_sign_and_upload"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
+            android:onClick="signAndUploadOnClick"
             android:text="@string/import_from_qr_code_import_sign_and_upload" />
 
         <Button
             android:id="@+id/import_from_qr_code_scan_again"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
+            android:onClick="scanAgainOnClick"
             android:text="@string/import_from_qr_code_scan_again" />
 
         <Button
             android:id="@+id/import_from_qr_code_finish"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
+            android:onClick="finishOnClick"
             android:text="@string/import_from_qr_code_finish" />
     </LinearLayout>
 
diff --git a/org_apg/src/org/thialfihar/android/apg/helper/PGPMain.java b/org_apg/src/org/thialfihar/android/apg/helper/PGPMain.java
index 3fcbeb7a7..960baea72 100644
--- a/org_apg/src/org/thialfihar/android/apg/helper/PGPMain.java
+++ b/org_apg/src/org/thialfihar/android/apg/helper/PGPMain.java
@@ -723,9 +723,9 @@ public class PGPMain {
             // nothing to do, we are done
         }
 
-        returnData.putInt("added", newKeys);
-        returnData.putInt("updated", oldKeys);
-        returnData.putInt("bad", badKeys);
+        returnData.putInt(ApgService.RESULT_IMPORT_ADDED, newKeys);
+        returnData.putInt(ApgService.RESULT_IMPORT_UPDATED, oldKeys);
+        returnData.putInt(ApgService.RESULT_IMPORT_BAD, badKeys);
 
         if (progress != null)
             progress.setProgress(R.string.progress_done, 100, 100);
diff --git a/org_apg/src/org/thialfihar/android/apg/service/ApgService.java b/org_apg/src/org/thialfihar/android/apg/service/ApgService.java
index 901097bbc..69c9e408c 100644
--- a/org_apg/src/org/thialfihar/android/apg/service/ApgService.java
+++ b/org_apg/src/org/thialfihar/android/apg/service/ApgService.java
@@ -161,6 +161,11 @@ public class ApgService extends IntentService implements ProgressDialogUpdater {
     public static final String RESULT_SIGNATURE_UNKNOWN = "signatureUnknown";
     public static final String RESULT_SIGNATURE_LOOKUP_KEY = "lookupKey";
 
+    // import
+    public static final String RESULT_IMPORT_ADDED = "added";
+    public static final String RESULT_IMPORT_UPDATED = "updated";
+    public static final String RESULT_IMPORT_BAD = "bad";
+
     Messenger mMessenger;
 
     public ApgService() {
diff --git a/org_apg/src/org/thialfihar/android/apg/ui/ImportFromQRCodeActivity.java b/org_apg/src/org/thialfihar/android/apg/ui/ImportFromQRCodeActivity.java
index 0b751f66c..67636686c 100644
--- a/org_apg/src/org/thialfihar/android/apg/ui/ImportFromQRCodeActivity.java
+++ b/org_apg/src/org/thialfihar/android/apg/ui/ImportFromQRCodeActivity.java
@@ -1,4 +1,5 @@
 /*
+ * Copyright (C) 2012 Dominik Schürmann <dominik@dominikschuermann.de>
  * Copyright (C) 2011 Senecaso
  * 
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -16,45 +17,39 @@
 
 package org.thialfihar.android.apg.ui;
 
-import java.io.ByteArrayInputStream;
-import java.io.IOException;
-
-import org.spongycastle.openpgp.PGPKeyRing;
-import org.spongycastle.openpgp.PGPPublicKeyRing;
 import org.thialfihar.android.apg.Constants;
-import org.thialfihar.android.apg.Id;
-import org.thialfihar.android.apg.helper.PGPHelper;
-import org.thialfihar.android.apg.helper.PGPMain;
-import org.thialfihar.android.apg.util.HkpKeyServer;
-import org.thialfihar.android.apg.util.KeyServer.QueryException;
+import org.thialfihar.android.apg.service.ApgHandler;
+import org.thialfihar.android.apg.service.ApgService;
+import org.thialfihar.android.apg.ui.dialog.ProgressDialogFragment;
 import org.thialfihar.android.apg.R;
 
+import android.app.AlertDialog;
+import android.app.ProgressDialog;
+import android.content.DialogInterface;
 import android.content.Intent;
 import android.os.Bundle;
 import android.os.Message;
+import android.os.Messenger;
+
 import org.thialfihar.android.apg.util.Log;
 
+import android.view.View;
 import android.widget.TextView;
 import android.widget.Toast;
 
+import com.actionbarsherlock.app.SherlockFragmentActivity;
 import com.google.zxing.integration.android.IntentIntegrator;
 import com.google.zxing.integration.android.IntentResult;
 
-public class ImportFromQRCodeActivity extends BaseActivity {
+public class ImportFromQRCodeActivity extends SherlockFragmentActivity {
 
     // Not used in sourcode, but listed in AndroidManifest!
     public static final String IMPORT_FROM_QR_CODE = Constants.INTENT_PREFIX
             + "IMPORT_FROM_QR_CODE";
 
-    public static final String EXTRA_KEY_ID = "keyId";
+    // public static final String EXTRA_KEY_ID = "keyId";
 
-    // TODO: remove when using new intentservice:
-    public static final String EXTRA_ERROR = "error";
-
-    private static final String TAG = "ImportFromQRCodeActivity";
-
-    private final Bundle status = new Bundle();
-    private final Message msg = new Message();
+    private String mScannedContent;
 
     @Override
     protected void onCreate(Bundle savedInstanceState) {
@@ -121,10 +116,101 @@ public class ImportFromQRCodeActivity extends BaseActivity {
     // }
     // }
 
-    private void importAndSign(final long keyId, final String expectedFingerprint) {
+    public void scanAgainOnClick(View view) {
+        new IntentIntegrator(this).initiateScan();
+    }
 
-        // setContentView(R.layout.import_from_qr_code);
+    public void finishOnClick(View view) {
+        finish();
+    }
 
+    public void importOnClick(View view) {
+        Log.d(Constants.TAG, "import key started");
+
+        // Send all information needed to service to import key in other thread
+        Intent intent = new Intent(this, ApgService.class);
+
+        intent.putExtra(ApgService.EXTRA_ACTION, ApgService.ACTION_IMPORT_KEY);
+
+        // fill values for this action
+        Bundle data = new Bundle();
+
+        // data.putInt(ApgService.IMPORT_KEY_TYPE, Id.type.public_key);
+
+        data.putInt(ApgService.TARGET, ApgService.TARGET_BYTES);
+        data.putByteArray(ApgService.IMPORT_BYTES, mScannedContent.getBytes());
+
+        intent.putExtra(ApgService.EXTRA_DATA, data);
+
+        // create progress dialog
+        ProgressDialogFragment importingDialog = ProgressDialogFragment.newInstance(
+                R.string.progress_importing, ProgressDialog.STYLE_HORIZONTAL);
+
+        // Message is received after importing is done in ApgService
+        ApgHandler saveHandler = new ApgHandler(this, importingDialog) {
+            public void handleMessage(Message message) {
+                // handle messages by standard ApgHandler first
+                super.handleMessage(message);
+
+                if (message.arg1 == ApgHandler.MESSAGE_OKAY) {
+                    // get returned data bundle
+                    Bundle returnData = message.getData();
+
+                    int added = returnData.getInt(ApgService.RESULT_IMPORT_ADDED);
+                    int updated = returnData.getInt(ApgService.RESULT_IMPORT_UPDATED);
+                    int bad = returnData.getInt(ApgService.RESULT_IMPORT_BAD);
+                    String toastMessage;
+                    if (added > 0 && updated > 0) {
+                        toastMessage = getString(R.string.keysAddedAndUpdated, added, updated);
+                    } else if (added > 0) {
+                        toastMessage = getString(R.string.keysAdded, added);
+                    } else if (updated > 0) {
+                        toastMessage = getString(R.string.keysUpdated, updated);
+                    } else {
+                        toastMessage = getString(R.string.noKeysAddedOrUpdated);
+                    }
+                    Toast.makeText(ImportFromQRCodeActivity.this, toastMessage, Toast.LENGTH_SHORT)
+                            .show();
+                    if (bad > 0) {
+                        AlertDialog.Builder alert = new AlertDialog.Builder(
+                                ImportFromQRCodeActivity.this);
+
+                        alert.setIcon(android.R.drawable.ic_dialog_alert);
+                        alert.setTitle(R.string.warning);
+                        alert.setMessage(ImportFromQRCodeActivity.this.getString(
+                                R.string.badKeysEncountered, bad));
+
+                        alert.setPositiveButton(android.R.string.ok,
+                                new DialogInterface.OnClickListener() {
+                                    public void onClick(DialogInterface dialog, int id) {
+                                        dialog.cancel();
+                                    }
+                                });
+                        alert.setCancelable(true);
+                        alert.create().show();
+                    }
+                }
+            };
+        };
+
+        // Create a new Messenger for the communication back
+        Messenger messenger = new Messenger(saveHandler);
+        intent.putExtra(ApgService.EXTRA_MESSENGER, messenger);
+
+        // show progress dialog
+        importingDialog.show(getSupportFragmentManager(), "importingDialog");
+
+        // start service with intent
+        startService(intent);
+    }
+
+    public void signAndUploadOnClick(View view) {
+        // first, import!
+        importOnClick(view);
+
+        // TODO: implement sign and upload!
+        Toast.makeText(ImportFromQRCodeActivity.this, "Not implemented right now!",
+                Toast.LENGTH_SHORT).show();
     }
 
     @Override
@@ -139,9 +225,9 @@ public class ImportFromQRCodeActivity extends BaseActivity {
                 setContentView(R.layout.import_from_qr_code);
                 TextView contentView = (TextView) findViewById(R.id.import_from_qr_code_content);
 
-                String content = scanResult.getContents();
+                mScannedContent = scanResult.getContents();
 
-                contentView.setText(content);
+                contentView.setText(mScannedContent);
                 // String[] bits = scanResult.getContents().split(",");
                 // if (bits.length != 2) {
                 // return; // dont know how to handle this. Not a valid code
@@ -157,35 +243,9 @@ public class ImportFromQRCodeActivity extends BaseActivity {
             break;
         }
 
-        case Id.request.sign_key: {
-            // signals the end of processing. Signature was either applied, or it wasnt
-            status.putInt(Constants.extras.STATUS, Id.message.done);
-
-            msg.setData(status);
-            sendMessage(msg);
-
-            break;
-        }
-
         default: {
             super.onActivityResult(requestCode, resultCode, data);
         }
         }
     }
-
-    @Override
-    public void doneCallback(Message msg) {
-        super.doneCallback(msg);
-
-        Bundle data = msg.getData();
-        String error = data.getString(EXTRA_ERROR);
-        if (error != null) {
-            Toast.makeText(this, getString(R.string.errorMessage, error), Toast.LENGTH_SHORT)
-                    .show();
-            return;
-        }
-
-        Toast.makeText(this, R.string.keySignSuccess, Toast.LENGTH_SHORT).show(); // TODO
-        finish();
-    }
 }
diff --git a/org_apg/src/org/thialfihar/android/apg/ui/KeyListActivity.java b/org_apg/src/org/thialfihar/android/apg/ui/KeyListActivity.java
index 1d41ab696..f7aeaee64 100644
--- a/org_apg/src/org/thialfihar/android/apg/ui/KeyListActivity.java
+++ b/org_apg/src/org/thialfihar/android/apg/ui/KeyListActivity.java
@@ -1,4 +1,5 @@
 /*
+ * Copyright (C) 2012 Dominik Schürmann <dominik@dominikschuermann.de>
  * Copyright (C) 2010 Thialfihar <thi@thialfihar.org>
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -312,9 +313,9 @@ public class KeyListActivity extends SherlockFragmentActivity {
                     // get returned data bundle
                     Bundle returnData = message.getData();
 
-                    int added = returnData.getInt("added");
-                    int updated = returnData.getInt("updated");
-                    int bad = returnData.getInt("bad");
+                    int added = returnData.getInt(ApgService.RESULT_IMPORT_ADDED);
+                    int updated = returnData.getInt(ApgService.RESULT_IMPORT_UPDATED);
+                    int bad = returnData.getInt(ApgService.RESULT_IMPORT_BAD);
                     String toastMessage;
                     if (added > 0 && updated > 0) {
                         toastMessage = getString(R.string.keysAddedAndUpdated, added, updated);
diff --git a/org_apg/src/org/thialfihar/android/apg/ui/KeyServerExportActivity.java b/org_apg/src/org/thialfihar/android/apg/ui/KeyServerExportActivity.java
index a15468bed..33f222644 100644
--- a/org_apg/src/org/thialfihar/android/apg/ui/KeyServerExportActivity.java
+++ b/org_apg/src/org/thialfihar/android/apg/ui/KeyServerExportActivity.java
@@ -1,4 +1,5 @@
 /*
+ * Copyright (C) 2012 Dominik Schürmann <dominik@dominikschuermann.de>
  * Copyright (C) 2011 Senecaso
  *
  * Licensed under the Apache License, Version 2.0 (the "License");