if there is only one private key, use it fo certification by default

OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/widget/CertifyKeySpinner.java

@@ -20,6 +20,7 @@ package org.sufficientlysecure.keychain.ui.widget;
 import android.content.Context;
 import android.database.Cursor;
 import android.net.Uri;
+import android.os.Bundle;
 import android.support.v4.content.CursorLoader;
 import android.support.v4.content.Loader;
 import android.util.AttributeSet;
@@ -48,7 +49,7 @@ public class CertifyKeySpinner extends KeySpinner {
     }
 
     @Override
-    public Loader<Cursor> onCreateLoader() {
+    public Loader<Cursor> onCreateLoader(int loaderId, Bundle data) {
         // This is called when a new Loader needs to be created. This
         // sample only has one Loader, so we don't care about the ID.
         Uri baseUri = KeychainContract.KeyRings.buildUnifiedKeyRingsUri();
@@ -74,4 +75,14 @@ public class CertifyKeySpinner extends KeySpinner {
         // creating a Cursor for the data being displayed.
         return new CursorLoader(getContext(), baseUri, projection, where, null, null);
     }
+
+    @Override
+    public void onLoadFinished(Loader<Cursor> loader, Cursor data) {
+        super.onLoadFinished(loader, data);
+        // If there is only one choice, pick it by default
+        if (mAdapter.getCount() == 2) {
+            setSelection(1);
+        }
+    }
+
 }

OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/widget/KeySpinner.java

@@ -40,14 +40,14 @@ import org.sufficientlysecure.keychain.pgp.PgpKeyHelper;
 import org.sufficientlysecure.keychain.provider.KeychainContract;
 import org.sufficientlysecure.keychain.util.Log;
 
-public abstract class KeySpinner extends Spinner {
+public abstract class KeySpinner extends Spinner implements LoaderManager.LoaderCallbacks<Cursor> {
     public interface OnKeyChangedListener {
         public void onKeyChanged(long masterKeyId);
     }
 
-    private long mSelectedKeyId;
+    protected long mSelectedKeyId;
-    private SelectKeyAdapter mAdapter = new SelectKeyAdapter();
+    protected SelectKeyAdapter mAdapter = new SelectKeyAdapter();
-    private OnKeyChangedListener mListener;
+    protected OnKeyChangedListener mListener;
 
     public KeySpinner(Context context) {
         super(context);
@@ -83,8 +83,6 @@ public abstract class KeySpinner extends Spinner {
         });
     }
 
-    public abstract Loader<Cursor> onCreateLoader();
-
     @Override
     public void setOnItemSelectedListener(OnItemSelectedListener listener) {
         throw new UnsupportedOperationException();
@@ -102,27 +100,22 @@ public abstract class KeySpinner extends Spinner {
 
     public void reload() {
         if (getContext() instanceof FragmentActivity) {
-            ((FragmentActivity) getContext()).getSupportLoaderManager().restartLoader(hashCode(), null, new LoaderManager.LoaderCallbacks<Cursor>() {
+            ((FragmentActivity) getContext()).getSupportLoaderManager().restartLoader(0, null, this);
-                @Override
-                public Loader<Cursor> onCreateLoader(int id, Bundle args) {
-                    return KeySpinner.this.onCreateLoader();
-                }
-
-                @Override
-                public void onLoadFinished(Loader<Cursor> loader, Cursor data) {
-                    mAdapter.swapCursor(data);
-                }
-
-                @Override
-                public void onLoaderReset(Loader<Cursor> loader) {
-                    mAdapter.swapCursor(null);
-                }
-            });
         } else {
             Log.e(Constants.TAG, "KeySpinner must be attached to FragmentActivity, this is " + getContext().getClass());
         }
     }
 
+    @Override
+    public void onLoadFinished(Loader<Cursor> loader, Cursor data) {
+        mAdapter.swapCursor(data);
+    }
+
+    @Override
+    public void onLoaderReset(Loader<Cursor> loader) {
+        mAdapter.swapCursor(null);
+    }
+
     public long getSelectedKeyId() {
         return mSelectedKeyId;
     }
@@ -131,7 +124,7 @@ public abstract class KeySpinner extends Spinner {
         this.mSelectedKeyId = selectedKeyId;
     }
 
-    private class SelectKeyAdapter extends BaseAdapter implements SpinnerAdapter {
+    protected class SelectKeyAdapter extends BaseAdapter implements SpinnerAdapter {
         private CursorAdapter inner;
         private int mIndexUserId;
         private int mIndexKeyId;

OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/widget/SignKeySpinner.java

@@ -20,6 +20,7 @@ package org.sufficientlysecure.keychain.ui.widget;
 import android.content.Context;
 import android.database.Cursor;
 import android.net.Uri;
+import android.os.Bundle;
 import android.support.v4.content.CursorLoader;
 import android.support.v4.content.Loader;
 import android.util.AttributeSet;
@@ -39,7 +40,7 @@ public class SignKeySpinner extends KeySpinner {
     }
 
     @Override
-    public Loader<Cursor> onCreateLoader() {
+    public Loader<Cursor> onCreateLoader(int loaderId, Bundle data) {
         // This is called when a new Loader needs to be created. This
         // sample only has one Loader, so we don't care about the ID.
         Uri baseUri = KeychainContract.KeyRings.buildUnifiedKeyRingsUri();

OpenKeychain/src/main/res/layout/keyspinner_key.xml

@@ -4,6 +4,7 @@
               android:orientation="vertical"
               android:layout_width="wrap_content"
               android:padding="4dp"
+              android:minHeight="24dp"
               android:layout_height="wrap_content">
     <TextView android:id="@android:id/title"
               android:textColor="?android:attr/textColorSecondary"
@@ -11,7 +12,8 @@
               android:layout_width="wrap_content"
               android:layout_height="wrap_content"
               android:singleLine="true"
-              android:ellipsize="end"/>
+              android:ellipsize="end"
+        />
     <TextView android:id="@android:id/text1"
               android:textColor="?android:attr/textColorTertiary"
               android:textSize="14sp"
@@ -19,7 +21,8 @@
               android:layout_height="wrap_content"
               android:singleLine="true"
               android:ellipsize="end"
-              android:layout_marginTop="-4dip"/>
+              android:layout_marginTop="-4dip"
+              android:text=".com"/>
     <TextView android:id="@android:id/text2"
               android:textColor="?android:attr/textColorTertiary"
               android:textSize="14sp"
@@ -27,5 +30,6 @@
               android:layout_height="wrap_content"
               android:singleLine="true"
               android:ellipsize="end"
-              android:layout_marginTop="-4dip"/>
+              android:layout_marginTop="-4dip"
+        />
 </LinearLayout>