Merge branch 'development' of github.com:open-keychain/open-keychain into development

3 files changed, 234 insertions, 141 deletions

diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/KeyListActivity.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/KeyListActivity.java
index c05393f6d..ba03400d7 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/KeyListActivity.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/KeyListActivity.java
@@ -206,7 +206,6 @@ public class KeyListActivity extends DrawerActivity {
 
         // start service with intent
         startService(intent);
-
     }
 
     @Override
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/KeyListFragment.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/KeyListFragment.java
index 0b28a2594..eba16e55d 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/KeyListFragment.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/KeyListFragment.java
@@ -115,10 +115,6 @@ public class KeyListFragment extends LoaderFragment
 
     boolean hideMenu = false;
 
-    Long mExchangeMasterKeyId = null;
-
-    private static final int REQUEST_CODE_SAFE_SLINGER = 2;
-
     /**
      * Load custom layout with StickyListView from library
      */
@@ -603,7 +599,9 @@ public class KeyListFragment extends LoaderFragment
                 @Override
                 public void onClick(View v) {
                     if (holder.mMasterKeyId != null) {
-                        startExchange(holder.mMasterKeyId);
+                        Intent safeSlingerIntent = new Intent(getActivity(), SafeSlingerActivity.class);
+                        safeSlingerIntent.putExtra(SafeSlingerActivity.EXTRA_MASTER_KEY_ID, holder.mMasterKeyId);
+                        startActivity(safeSlingerIntent);
                     }
                 }
             });
@@ -842,136 +840,6 @@ public class KeyListFragment extends LoaderFragment
 
     }
 
-    @Override
-    public void onActivityResult(int requestCode, int resultCode, Intent data) {
-        if (requestCode == REQUEST_CODE_SAFE_SLINGER) {
-            if (resultCode == ExchangeActivity.RESULT_EXCHANGE_CANCELED) {
-                return;
-            }
-
-            final FragmentActivity activity = getActivity();
-
-            // Message is received after importing is done in KeychainIntentService
-            KeychainIntentServiceHandler saveHandler = new KeychainIntentServiceHandler(
-                    activity,
-                    getString(R.string.progress_importing),
-                    ProgressDialog.STYLE_HORIZONTAL,
-                    true) {
-                public void handleMessage(Message message) {
-                    // handle messages by standard KeychainIntentServiceHandler first
-                    super.handleMessage(message);
-
-                    if (message.arg1 == KeychainIntentServiceHandler.MESSAGE_OKAY) {
-                        // get returned data bundle
-                        Bundle returnData = message.getData();
-                        if (returnData == null) {
-                            return;
-                        }
-                        final ImportKeyResult result =
-                                returnData.getParcelable(OperationResult.EXTRA_RESULT);
-                        if (result == null) {
-                            Log.e(Constants.TAG, "result == null");
-                            return;
-                        }
-
-                        if (!result.success()) {
-                            result.createNotify(activity).show();
-                            return;
-                        }
-
-                        if (mExchangeMasterKeyId == null) {
-                            return;
-                        }
-
-                        Intent certifyIntent = new Intent(activity, MultiCertifyKeyActivity.class);
-                        certifyIntent.putExtra(MultiCertifyKeyActivity.EXTRA_RESULT, result);
-                        certifyIntent.putExtra(MultiCertifyKeyActivity.EXTRA_KEY_IDS, result.getImportedMasterKeyIds());
-                        certifyIntent.putExtra(MultiCertifyKeyActivity.EXTRA_CERTIFY_KEY_ID, mExchangeMasterKeyId);
-                        startActivityForResult(certifyIntent, KeyListActivity.REQUEST_CODE_RESULT_TO_LIST);
-
-                        mExchangeMasterKeyId = null;
-                    }
-                }
-            };
-
-            Log.d(Constants.TAG, "importKeys started");
-
-            // Send all information needed to service to import key in other thread
-            Intent intent = new Intent(activity, KeychainIntentService.class);
-
-            intent.setAction(KeychainIntentService.ACTION_IMPORT_KEYRING);
-
-            // instead of giving the entries by Intent extra, cache them into a
-            // file to prevent Java Binder problems on heavy imports
-            // read FileImportCache for more info.
-            try {
-                // import exchanged keys
-                ArrayList<ParcelableKeyRing> it = getSlingedKeys(data.getExtras());
-
-                // We parcel this iteratively into a file - anything we can
-                // display here, we should be able to import.
-                ParcelableFileCache<ParcelableKeyRing> cache =
-                        new ParcelableFileCache<ParcelableKeyRing>(activity, "key_import.pcl");
-                cache.writeCache(it.size(), it.iterator());
-
-                // fill values for this action
-                Bundle bundle = new Bundle();
-                intent.putExtra(KeychainIntentService.EXTRA_DATA, bundle);
-
-                // Create a new Messenger for the communication back
-                Messenger messenger = new Messenger(saveHandler);
-                intent.putExtra(KeychainIntentService.EXTRA_MESSENGER, messenger);
-
-                // show progress dialog
-                saveHandler.showProgressDialog(activity);
-
-                // start service with intent
-                activity.startService(intent);
-            } catch (IOException e) {
-                Log.e(Constants.TAG, "Problem writing cache file", e);
-                Notify.showNotify(activity, "Problem writing cache file!", Notify.Style.ERROR);
-            }
-        }
-
-        super.onActivityResult(requestCode, resultCode, data);
-    }
-
-    private static ArrayList<ParcelableKeyRing> getSlingedKeys(Bundle extras) {
-
-        ArrayList<ParcelableKeyRing> list = new ArrayList<ParcelableKeyRing>();
-
-        if (extras != null) {
-            byte[] d;
-            int i = 0;
-            do {
-                d = extras.getByteArray(ExchangeConfig.extra.MEMBER_DATA + i);
-                if (d != null) {
-                    list.add(new ParcelableKeyRing(d));
-                    i++;
-                }
-            } while (d != null);
-        }
-
-        return list;
-
-    }
-
-    private void startExchange(long masterKeyId) {
-        mExchangeMasterKeyId = masterKeyId;
-        // retrieve public key blob and start SafeSlinger
-        Uri uri = KeychainContract.KeyRingData.buildPublicKeyRingUri(masterKeyId);
-        try {
-            byte[] keyBlob = (byte[]) new ProviderHelper(getActivity()).getGenericData(
-                    uri, KeychainContract.KeyRingData.KEY_RING_DATA, ProviderHelper.FIELD_TYPE_BLOB);
-
-            Intent slingerIntent = new Intent(getActivity(), ExchangeActivity.class);
-            slingerIntent.putExtra(ExchangeConfig.extra.USER_DATA, keyBlob);
-            slingerIntent.putExtra(ExchangeConfig.extra.HOST_NAME, Constants.SAFESLINGER_SERVER);
-            startActivityForResult(slingerIntent, REQUEST_CODE_SAFE_SLINGER);
-        } catch (ProviderHelper.NotFoundException e) {
-            Log.e(Constants.TAG, "personal key not found", e);
-        }
-    }
 
 
 }
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/SafeSlingerActivity.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/SafeSlingerActivity.java
index b8e3dd590..9d4bfbda5 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/SafeSlingerActivity.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/SafeSlingerActivity.java
@@ -17,27 +17,253 @@
 
 package org.sufficientlysecure.keychain.ui;
 
+import android.annotation.TargetApi;
+import android.app.ProgressDialog;
+import android.content.Intent;
+import android.graphics.PorterDuff;
+import android.net.Uri;
+import android.os.Build;
 import android.os.Bundle;
+import android.os.Message;
+import android.os.Messenger;
+import android.support.v4.app.FragmentActivity;
 import android.support.v7.app.ActionBarActivity;
+import android.view.View;
+import android.widget.AdapterView;
+import android.widget.ArrayAdapter;
+import android.widget.ImageButton;
+import android.widget.ImageView;
 import android.widget.NumberPicker;
+import android.widget.Spinner;
 
+import org.sufficientlysecure.keychain.Constants;
 import org.sufficientlysecure.keychain.R;
+import org.sufficientlysecure.keychain.keyimport.ParcelableKeyRing;
+import org.sufficientlysecure.keychain.operations.results.ImportKeyResult;
+import org.sufficientlysecure.keychain.operations.results.OperationResult;
+import org.sufficientlysecure.keychain.provider.KeychainContract;
+import org.sufficientlysecure.keychain.provider.ProviderHelper;
+import org.sufficientlysecure.keychain.service.KeychainIntentService;
+import org.sufficientlysecure.keychain.service.KeychainIntentServiceHandler;
+import org.sufficientlysecure.keychain.ui.util.Notify;
+import org.sufficientlysecure.keychain.util.Log;
+import org.sufficientlysecure.keychain.util.ParcelableFileCache;
 
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
 
+import edu.cmu.cylab.starslinger.exchange.ExchangeActivity;
+import edu.cmu.cylab.starslinger.exchange.ExchangeConfig;
+
+@TargetApi(Build.VERSION_CODES.HONEYCOMB)
 public class SafeSlingerActivity extends ActionBarActivity {
 
+    private static final int REQUEST_CODE_SAFE_SLINGER = 211;
+
+    public static final String EXTRA_MASTER_KEY_ID = "master_key_id";
+
+    private long mMasterKeyId;
+    private int mSelectedNumber;
+
     @Override
     public void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
 
         setContentView(R.layout.safe_slinger_activity);
 
-        NumberPicker numberPicker = (NumberPicker) findViewById(R.id.safe_slinger_number_picker);
+        mMasterKeyId = getIntent().getLongExtra(EXTRA_MASTER_KEY_ID, 0);
+
+        // NOTE: there are two versions of this layout, for API >= 11 and one for < 11
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.HONEYCOMB) {
+            NumberPicker picker = (NumberPicker) findViewById(R.id.safe_slinger_picker);
+            picker.setMinValue(2);
+            picker.setMaxValue(10);
+            picker.setOnValueChangedListener(new NumberPicker.OnValueChangeListener() {
+                @Override
+                public void onValueChange(NumberPicker picker, int oldVal, int newVal) {
+                    mSelectedNumber = newVal;
+                }
+            });
+        } else {
+            Spinner spinner = (Spinner) findViewById(R.id.safe_slinger_spinner);
+
+            List<String> list = new ArrayList<String>();
+            list.add("2");
+            list.add("3");
+            list.add("4");
+            list.add("5");
+            list.add("6");
+            list.add("7");
+            list.add("8");
+            list.add("9");
+            list.add("10");
+
+            ArrayAdapter<String> dataAdapter = new ArrayAdapter<String>(this,
+                    android.R.layout.simple_spinner_item, list);
+            dataAdapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item);
+            spinner.setAdapter(dataAdapter);
+            spinner.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
+                @Override
+                public void onItemSelected(AdapterView<?> parent, View view, int position, long id) {
+                    mSelectedNumber = position + 2;
+                }
+
+                @Override
+                public void onNothingSelected(AdapterView<?> parent) {
+
+                }
+            });
+        }
+
+        ImageView buttonIcon = (ImageView) findViewById(R.id.safe_slinger_button_image);
+        buttonIcon.setColorFilter(getResources().getColor(R.color.tertiary_text_light),
+                PorterDuff.Mode.SRC_IN);
+
+        View button = findViewById(R.id.safe_slinger_button);
+        button.setOnClickListener(new View.OnClickListener() {
+            @Override
+            public void onClick(View v) {
+                startExchange(mMasterKeyId);
+            }
+        });
+    }
+
+    private void startExchange(long masterKeyId) {
+        // retrieve public key blob and start SafeSlinger
+        Uri uri = KeychainContract.KeyRingData.buildPublicKeyRingUri(masterKeyId);
+        try {
+            byte[] keyBlob = (byte[]) new ProviderHelper(this).getGenericData(
+                    uri, KeychainContract.KeyRingData.KEY_RING_DATA, ProviderHelper.FIELD_TYPE_BLOB);
+
+            Intent slingerIntent = new Intent(this, ExchangeActivity.class);
+            slingerIntent.putExtra(ExchangeConfig.extra.NUM_USERS, mSelectedNumber);
+            slingerIntent.putExtra(ExchangeConfig.extra.USER_DATA, keyBlob);
+            slingerIntent.putExtra(ExchangeConfig.extra.HOST_NAME, Constants.SAFESLINGER_SERVER);
+            startActivityForResult(slingerIntent, REQUEST_CODE_SAFE_SLINGER);
+        } catch (ProviderHelper.NotFoundException e) {
+            Log.e(Constants.TAG, "personal key not found", e);
+        }
+    }
+
+
+    @Override
+    public void onActivityResult(int requestCode, int resultCode, Intent data) {
+        if (requestCode == REQUEST_CODE_SAFE_SLINGER) {
+            if (resultCode == ExchangeActivity.RESULT_EXCHANGE_CANCELED) {
+                return;
+            }
+
+            final FragmentActivity activity = SafeSlingerActivity.this;
+
+            // Message is received after importing is done in KeychainIntentService
+            KeychainIntentServiceHandler saveHandler = new KeychainIntentServiceHandler(
+                    activity,
+                    getString(R.string.progress_importing),
+                    ProgressDialog.STYLE_HORIZONTAL,
+                    true) {
+                public void handleMessage(Message message) {
+                    // handle messages by standard KeychainIntentServiceHandler first
+                    super.handleMessage(message);
+
+                    if (message.arg1 == KeychainIntentServiceHandler.MESSAGE_OKAY) {
+                        // get returned data bundle
+                        Bundle returnData = message.getData();
+                        if (returnData == null) {
+                            return;
+                        }
+                        final ImportKeyResult result =
+                                returnData.getParcelable(OperationResult.EXTRA_RESULT);
+                        if (result == null) {
+                            Log.e(Constants.TAG, "result == null");
+                            return;
+                        }
+
+                        if (!result.success()) {
+//                            result.createNotify(activity).show();
+                            // only return if no success...
+                            Intent data = new Intent();
+                            data.putExtras(returnData);
+                            setResult(RESULT_OK, data);
+                            finish();
+                            return;
+                        }
+
+//                        if (mExchangeMasterKeyId == null) {
+//                            return;
+//                        }
+
+                        Intent certifyIntent = new Intent(activity, MultiCertifyKeyActivity.class);
+                        certifyIntent.putExtra(MultiCertifyKeyActivity.EXTRA_RESULT, result);
+                        certifyIntent.putExtra(MultiCertifyKeyActivity.EXTRA_KEY_IDS, result.getImportedMasterKeyIds());
+                        certifyIntent.putExtra(MultiCertifyKeyActivity.EXTRA_CERTIFY_KEY_ID, mMasterKeyId);
+                        startActivityForResult(certifyIntent, KeyListActivity.REQUEST_CODE_RESULT_TO_LIST);
+
+//                        mExchangeMasterKeyId = null;
+                    }
+                }
+            };
+
+            Log.d(Constants.TAG, "importKeys started");
+
+            // Send all information needed to service to import key in other thread
+            Intent intent = new Intent(activity, KeychainIntentService.class);
+
+            intent.setAction(KeychainIntentService.ACTION_IMPORT_KEYRING);
+
+            // instead of giving the entries by Intent extra, cache them into a
+            // file to prevent Java Binder problems on heavy imports
+            // read FileImportCache for more info.
+            try {
+                // import exchanged keys
+                ArrayList<ParcelableKeyRing> it = getSlingedKeys(data.getExtras());
+
+                // We parcel this iteratively into a file - anything we can
+                // display here, we should be able to import.
+                ParcelableFileCache<ParcelableKeyRing> cache =
+                        new ParcelableFileCache<ParcelableKeyRing>(activity, "key_import.pcl");
+                cache.writeCache(it.size(), it.iterator());
+
+                // fill values for this action
+                Bundle bundle = new Bundle();
+                intent.putExtra(KeychainIntentService.EXTRA_DATA, bundle);
+
+                // Create a new Messenger for the communication back
+                Messenger messenger = new Messenger(saveHandler);
+                intent.putExtra(KeychainIntentService.EXTRA_MESSENGER, messenger);
+
+                // show progress dialog
+                saveHandler.showProgressDialog(activity);
+
+                // start service with intent
+                activity.startService(intent);
+            } catch (IOException e) {
+                Log.e(Constants.TAG, "Problem writing cache file", e);
+                Notify.showNotify(activity, "Problem writing cache file!", Notify.Style.ERROR);
+            }
+        } else {
+            // give everything else down to KeyListActivity!
+            setResult(resultCode, data);
+            finish();
+        }
+    }
+
+    private static ArrayList<ParcelableKeyRing> getSlingedKeys(Bundle extras) {
+        ArrayList<ParcelableKeyRing> list = new ArrayList<ParcelableKeyRing>();
+
+        if (extras != null) {
+            byte[] d;
+            int i = 0;
+            do {
+                d = extras.getByteArray(ExchangeConfig.extra.MEMBER_DATA + i);
+                if (d != null) {
+                    list.add(new ParcelableKeyRing(d));
+                    i++;
+                }
+            } while (d != null);
+        }
 
-        numberPicker.setDisplayedValues(new String[]{"2","3","4","5","6","7","8","9","10"});
-        numberPicker.setValue(0);
-//        numberPicker.setMaxValue(8);
-//        numberPicker.setMinValue(0);
+        return list;
     }
 
 }