mirror of
https://github.com/moparisthebest/open-keychain
synced 2024-12-25 08:28:50 -05:00
Merge pull request #767 from mar-v-in/issue-763
Use dropdown in CertifyActivity
This commit is contained in:
commit
28df004cbb
@ -111,6 +111,7 @@ public class KeychainContract {
|
|||||||
public static final String HAS_ANY_SECRET = "has_any_secret";
|
public static final String HAS_ANY_SECRET = "has_any_secret";
|
||||||
public static final String HAS_ENCRYPT = "has_encrypt";
|
public static final String HAS_ENCRYPT = "has_encrypt";
|
||||||
public static final String HAS_SIGN = "has_sign";
|
public static final String HAS_SIGN = "has_sign";
|
||||||
|
public static final String HAS_CERTIFY = "has_certify";
|
||||||
public static final String PUBKEY_DATA = "pubkey_data";
|
public static final String PUBKEY_DATA = "pubkey_data";
|
||||||
public static final String PRIVKEY_DATA = "privkey_data";
|
public static final String PRIVKEY_DATA = "privkey_data";
|
||||||
|
|
||||||
|
@ -271,6 +271,8 @@ public class KeychainProvider extends ContentProvider {
|
|||||||
"kE." + Keys.KEY_ID + " AS " + KeyRings.HAS_ENCRYPT);
|
"kE." + Keys.KEY_ID + " AS " + KeyRings.HAS_ENCRYPT);
|
||||||
projectionMap.put(KeyRings.HAS_SIGN,
|
projectionMap.put(KeyRings.HAS_SIGN,
|
||||||
"kS." + Keys.KEY_ID + " AS " + KeyRings.HAS_SIGN);
|
"kS." + Keys.KEY_ID + " AS " + KeyRings.HAS_SIGN);
|
||||||
|
projectionMap.put(KeyRings.HAS_CERTIFY,
|
||||||
|
"kC." + Keys.KEY_ID + " AS " + KeyRings.HAS_CERTIFY);
|
||||||
projectionMap.put(KeyRings.IS_EXPIRED,
|
projectionMap.put(KeyRings.IS_EXPIRED,
|
||||||
"(" + Tables.KEYS + "." + Keys.EXPIRY + " IS NOT NULL AND " + Tables.KEYS + "." + Keys.EXPIRY
|
"(" + Tables.KEYS + "." + Keys.EXPIRY + " IS NOT NULL AND " + Tables.KEYS + "." + Keys.EXPIRY
|
||||||
+ " < " + new Date().getTime() / 1000 + ") AS " + KeyRings.IS_EXPIRED);
|
+ " < " + new Date().getTime() / 1000 + ") AS " + KeyRings.IS_EXPIRED);
|
||||||
@ -324,6 +326,15 @@ public class KeychainProvider extends ContentProvider {
|
|||||||
+ " AND ( kS." + Keys.EXPIRY + " IS NULL OR kS." + Keys.EXPIRY
|
+ " AND ( kS." + Keys.EXPIRY + " IS NULL OR kS." + Keys.EXPIRY
|
||||||
+ " >= " + new Date().getTime() / 1000 + " )"
|
+ " >= " + new Date().getTime() / 1000 + " )"
|
||||||
+ ")" : "")
|
+ ")" : "")
|
||||||
|
+ (plist.contains(KeyRings.HAS_CERTIFY) ?
|
||||||
|
" LEFT JOIN " + Tables.KEYS + " AS kC ON ("
|
||||||
|
+"kC." + Keys.MASTER_KEY_ID
|
||||||
|
+ " = " + Tables.KEYS + "." + Keys.MASTER_KEY_ID
|
||||||
|
+ " AND kC." + Keys.IS_REVOKED + " = 0"
|
||||||
|
+ " AND kC." + Keys.CAN_CERTIFY + " = 1"
|
||||||
|
+ " AND ( kC." + Keys.EXPIRY + " IS NULL OR kC." + Keys.EXPIRY
|
||||||
|
+ " >= " + new Date().getTime() / 1000 + " )"
|
||||||
|
+ ")" : "")
|
||||||
);
|
);
|
||||||
qb.appendWhere(Tables.KEYS + "." + Keys.RANK + " = 0");
|
qb.appendWhere(Tables.KEYS + "." + Keys.RANK + " = 0");
|
||||||
// in case there are multiple verifying certificates
|
// in case there are multiple verifying certificates
|
||||||
|
@ -56,6 +56,8 @@ import org.sufficientlysecure.keychain.service.OperationResultParcel;
|
|||||||
import org.sufficientlysecure.keychain.service.PassphraseCacheService;
|
import org.sufficientlysecure.keychain.service.PassphraseCacheService;
|
||||||
import org.sufficientlysecure.keychain.ui.adapter.UserIdsAdapter;
|
import org.sufficientlysecure.keychain.ui.adapter.UserIdsAdapter;
|
||||||
import org.sufficientlysecure.keychain.ui.dialog.PassphraseDialogFragment;
|
import org.sufficientlysecure.keychain.ui.dialog.PassphraseDialogFragment;
|
||||||
|
import org.sufficientlysecure.keychain.ui.widget.CertifyKeySpinner;
|
||||||
|
import org.sufficientlysecure.keychain.ui.widget.KeySpinner;
|
||||||
import org.sufficientlysecure.keychain.util.Log;
|
import org.sufficientlysecure.keychain.util.Log;
|
||||||
import org.sufficientlysecure.keychain.util.Notify;
|
import org.sufficientlysecure.keychain.util.Notify;
|
||||||
|
|
||||||
@ -64,18 +66,17 @@ import java.util.ArrayList;
|
|||||||
/**
|
/**
|
||||||
* Signs the specified public key with the specified secret master key
|
* Signs the specified public key with the specified secret master key
|
||||||
*/
|
*/
|
||||||
public class CertifyKeyActivity extends ActionBarActivity implements
|
public class CertifyKeyActivity extends ActionBarActivity implements LoaderManager.LoaderCallbacks<Cursor> {
|
||||||
SelectSecretKeyLayoutFragment.SelectSecretKeyCallback, LoaderManager.LoaderCallbacks<Cursor> {
|
|
||||||
private View mCertifyButton;
|
private View mCertifyButton;
|
||||||
private ImageView mActionCertifyImage;
|
private ImageView mActionCertifyImage;
|
||||||
private CheckBox mUploadKeyCheckbox;
|
private CheckBox mUploadKeyCheckbox;
|
||||||
private Spinner mSelectKeyserverSpinner;
|
private Spinner mSelectKeyserverSpinner;
|
||||||
|
|
||||||
private SelectSecretKeyLayoutFragment mSelectKeyFragment;
|
private CertifyKeySpinner mCertifyKeySpinner;
|
||||||
|
|
||||||
private Uri mDataUri;
|
private Uri mDataUri;
|
||||||
private long mPubKeyId = 0;
|
private long mPubKeyId = Constants.key.none;
|
||||||
private long mMasterKeyId = 0;
|
private long mMasterKeyId = Constants.key.none;
|
||||||
|
|
||||||
private ListView mUserIds;
|
private ListView mUserIds;
|
||||||
private UserIdsAdapter mUserIdsAdapter;
|
private UserIdsAdapter mUserIdsAdapter;
|
||||||
@ -89,8 +90,7 @@ public class CertifyKeyActivity extends ActionBarActivity implements
|
|||||||
|
|
||||||
setContentView(R.layout.certify_key_activity);
|
setContentView(R.layout.certify_key_activity);
|
||||||
|
|
||||||
mSelectKeyFragment = (SelectSecretKeyLayoutFragment) getSupportFragmentManager()
|
mCertifyKeySpinner = (CertifyKeySpinner) findViewById(R.id.certify_key_spinner);
|
||||||
.findFragmentById(R.id.sign_key_select_key_fragment);
|
|
||||||
mSelectKeyserverSpinner = (Spinner) findViewById(R.id.upload_key_keyserver);
|
mSelectKeyserverSpinner = (Spinner) findViewById(R.id.upload_key_keyserver);
|
||||||
mUploadKeyCheckbox = (CheckBox) findViewById(R.id.sign_key_upload_checkbox);
|
mUploadKeyCheckbox = (CheckBox) findViewById(R.id.sign_key_upload_checkbox);
|
||||||
mCertifyButton = findViewById(R.id.certify_key_certify_button);
|
mCertifyButton = findViewById(R.id.certify_key_certify_button);
|
||||||
@ -101,8 +101,12 @@ public class CertifyKeyActivity extends ActionBarActivity implements
|
|||||||
mActionCertifyImage.setColorFilter(getResources().getColor(R.color.tertiary_text_light),
|
mActionCertifyImage.setColorFilter(getResources().getColor(R.color.tertiary_text_light),
|
||||||
PorterDuff.Mode.SRC_IN);
|
PorterDuff.Mode.SRC_IN);
|
||||||
|
|
||||||
mSelectKeyFragment.setCallback(this);
|
mCertifyKeySpinner.setOnKeyChangedListener(new KeySpinner.OnKeyChangedListener() {
|
||||||
mSelectKeyFragment.setFilterCertify(true);
|
@Override
|
||||||
|
public void onKeyChanged(long masterKeyId) {
|
||||||
|
mMasterKeyId = masterKeyId;
|
||||||
|
}
|
||||||
|
});
|
||||||
|
|
||||||
ArrayAdapter<String> adapter = new ArrayAdapter<String>(this,
|
ArrayAdapter<String> adapter = new ArrayAdapter<String>(this,
|
||||||
android.R.layout.simple_spinner_item, Preferences.getPreferences(this)
|
android.R.layout.simple_spinner_item, Preferences.getPreferences(this)
|
||||||
@ -135,7 +139,6 @@ public class CertifyKeyActivity extends ActionBarActivity implements
|
|||||||
public void onClick(View v) {
|
public void onClick(View v) {
|
||||||
if (mPubKeyId != 0) {
|
if (mPubKeyId != 0) {
|
||||||
if (mMasterKeyId == 0) {
|
if (mMasterKeyId == 0) {
|
||||||
mSelectKeyFragment.setError(getString(R.string.select_key_to_certify));
|
|
||||||
Notify.showNotify(CertifyKeyActivity.this, getString(R.string.select_key_to_certify),
|
Notify.showNotify(CertifyKeyActivity.this, getString(R.string.select_key_to_certify),
|
||||||
Notify.Style.ERROR);
|
Notify.Style.ERROR);
|
||||||
} else {
|
} else {
|
||||||
@ -199,6 +202,7 @@ public class CertifyKeyActivity extends ActionBarActivity implements
|
|||||||
if (data.moveToFirst()) {
|
if (data.moveToFirst()) {
|
||||||
// TODO: put findViewById in onCreate!
|
// TODO: put findViewById in onCreate!
|
||||||
mPubKeyId = data.getLong(INDEX_MASTER_KEY_ID);
|
mPubKeyId = data.getLong(INDEX_MASTER_KEY_ID);
|
||||||
|
mCertifyKeySpinner.setHiddenMasterKeyId(mPubKeyId);
|
||||||
String keyIdStr = PgpKeyHelper.convertKeyIdToHex(mPubKeyId);
|
String keyIdStr = PgpKeyHelper.convertKeyIdToHex(mPubKeyId);
|
||||||
((TextView) findViewById(R.id.key_id)).setText(keyIdStr);
|
((TextView) findViewById(R.id.key_id)).setText(keyIdStr);
|
||||||
|
|
||||||
@ -367,14 +371,6 @@ public class CertifyKeyActivity extends ActionBarActivity implements
|
|||||||
startService(intent);
|
startService(intent);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* callback from select key fragment
|
|
||||||
*/
|
|
||||||
@Override
|
|
||||||
public void onKeySelected(long secretKeyId) {
|
|
||||||
mMasterKeyId = secretKeyId;
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public boolean onOptionsItemSelected(MenuItem item) {
|
public boolean onOptionsItemSelected(MenuItem item) {
|
||||||
switch (item.getItemId()) {
|
switch (item.getItemId()) {
|
||||||
|
@ -47,6 +47,7 @@ import org.sufficientlysecure.keychain.provider.CachedPublicKeyRing;
|
|||||||
import org.sufficientlysecure.keychain.provider.KeychainContract.KeyRings;
|
import org.sufficientlysecure.keychain.provider.KeychainContract.KeyRings;
|
||||||
import org.sufficientlysecure.keychain.provider.ProviderHelper;
|
import org.sufficientlysecure.keychain.provider.ProviderHelper;
|
||||||
import org.sufficientlysecure.keychain.ui.widget.EncryptKeyCompletionView;
|
import org.sufficientlysecure.keychain.ui.widget.EncryptKeyCompletionView;
|
||||||
|
import org.sufficientlysecure.keychain.ui.widget.KeySpinner;
|
||||||
import org.sufficientlysecure.keychain.util.Log;
|
import org.sufficientlysecure.keychain.util.Log;
|
||||||
|
|
||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
@ -54,22 +55,20 @@ import java.util.Iterator;
|
|||||||
import java.util.List;
|
import java.util.List;
|
||||||
|
|
||||||
public class EncryptAsymmetricFragment extends Fragment implements EncryptActivityInterface.UpdateListener {
|
public class EncryptAsymmetricFragment extends Fragment implements EncryptActivityInterface.UpdateListener {
|
||||||
public static final String ARG_SIGNATURE_KEY_ID = "signature_key_id";
|
|
||||||
public static final String ARG_ENCRYPTION_KEY_IDS = "encryption_key_ids";
|
|
||||||
|
|
||||||
ProviderHelper mProviderHelper;
|
ProviderHelper mProviderHelper;
|
||||||
|
|
||||||
// view
|
// view
|
||||||
private Spinner mSign;
|
private KeySpinner mSign;
|
||||||
private EncryptKeyCompletionView mEncryptKeyView;
|
private EncryptKeyCompletionView mEncryptKeyView;
|
||||||
private SelectSignKeyCursorAdapter mSignAdapter = new SelectSignKeyCursorAdapter();
|
|
||||||
|
|
||||||
// model
|
// model
|
||||||
private EncryptActivityInterface mEncryptInterface;
|
private EncryptActivityInterface mEncryptInterface;
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public void onNotifyUpdate() {
|
public void onNotifyUpdate() {
|
||||||
|
if (mSign != null) {
|
||||||
|
mSign.setSelectedKeyId(mEncryptInterface.getSignatureKey());
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
@ -101,17 +100,11 @@ public class EncryptAsymmetricFragment extends Fragment implements EncryptActivi
|
|||||||
public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
|
public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
|
||||||
View view = inflater.inflate(R.layout.encrypt_asymmetric_fragment, container, false);
|
View view = inflater.inflate(R.layout.encrypt_asymmetric_fragment, container, false);
|
||||||
|
|
||||||
mSign = (Spinner) view.findViewById(R.id.sign);
|
mSign = (KeySpinner) view.findViewById(R.id.sign);
|
||||||
mSign.setAdapter(mSignAdapter);
|
mSign.setOnKeyChangedListener(new KeySpinner.OnKeyChangedListener() {
|
||||||
mSign.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() {
|
|
||||||
@Override
|
@Override
|
||||||
public void onItemSelected(AdapterView<?> parent, View view, int position, long id) {
|
public void onKeyChanged(long masterKeyId) {
|
||||||
setSignatureKeyId(parent.getAdapter().getItemId(position));
|
setSignatureKeyId(masterKeyId);
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public void onNothingSelected(AdapterView<?> parent) {
|
|
||||||
setSignatureKeyId(Constants.key.none);
|
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
mEncryptKeyView = (EncryptKeyCompletionView) view.findViewById(R.id.recipient_list);
|
mEncryptKeyView = (EncryptKeyCompletionView) view.findViewById(R.id.recipient_list);
|
||||||
@ -128,42 +121,6 @@ public class EncryptAsymmetricFragment extends Fragment implements EncryptActivi
|
|||||||
// preselect keys given
|
// preselect keys given
|
||||||
preselectKeys();
|
preselectKeys();
|
||||||
|
|
||||||
getLoaderManager().initLoader(1, null, new LoaderManager.LoaderCallbacks<Cursor>() {
|
|
||||||
@Override
|
|
||||||
public Loader<Cursor> onCreateLoader(int id, Bundle args) {
|
|
||||||
// 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 = KeyRings.buildUnifiedKeyRingsUri();
|
|
||||||
|
|
||||||
// These are the rows that we will retrieve.
|
|
||||||
String[] projection = new String[]{
|
|
||||||
KeyRings._ID,
|
|
||||||
KeyRings.MASTER_KEY_ID,
|
|
||||||
KeyRings.KEY_ID,
|
|
||||||
KeyRings.USER_ID,
|
|
||||||
KeyRings.IS_EXPIRED,
|
|
||||||
KeyRings.HAS_SIGN,
|
|
||||||
KeyRings.HAS_ANY_SECRET
|
|
||||||
};
|
|
||||||
|
|
||||||
String where = KeyRings.HAS_ANY_SECRET + " = 1 AND " + KeyRings.HAS_SIGN + " NOT NULL AND "
|
|
||||||
+ KeyRings.IS_REVOKED + " = 0 AND " + KeyRings.IS_EXPIRED + " = 0";
|
|
||||||
|
|
||||||
// Now create and return a CursorLoader that will take care of
|
|
||||||
// creating a Cursor for the data being displayed.
|
|
||||||
return new CursorLoader(getActivity(), baseUri, projection, where, null, null);
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public void onLoadFinished(Loader<Cursor> loader, Cursor data) {
|
|
||||||
mSignAdapter.swapCursor(data);
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public void onLoaderReset(Loader<Cursor> loader) {
|
|
||||||
mSignAdapter.swapCursor(null);
|
|
||||||
}
|
|
||||||
});
|
|
||||||
mEncryptKeyView.setTokenListener(new TokenCompleteTextView.TokenListener() {
|
mEncryptKeyView.setTokenListener(new TokenCompleteTextView.TokenListener() {
|
||||||
@Override
|
@Override
|
||||||
public void onTokenAdded(Object token) {
|
public void onTokenAdded(Object token) {
|
||||||
@ -194,6 +151,7 @@ public class EncryptAsymmetricFragment extends Fragment implements EncryptActivi
|
|||||||
KeyRings.buildUnifiedKeyRingUri(signatureKey));
|
KeyRings.buildUnifiedKeyRingUri(signatureKey));
|
||||||
if(keyring.hasAnySecret()) {
|
if(keyring.hasAnySecret()) {
|
||||||
setSignatureKeyId(keyring.getMasterKeyId());
|
setSignatureKeyId(keyring.getMasterKeyId());
|
||||||
|
mSign.setSelectedKeyId(mEncryptInterface.getSignatureKey());
|
||||||
}
|
}
|
||||||
} catch (PgpGeneralException e) {
|
} catch (PgpGeneralException e) {
|
||||||
Log.e(Constants.TAG, "key not found!", e);
|
Log.e(Constants.TAG, "key not found!", e);
|
||||||
@ -233,95 +191,4 @@ public class EncryptAsymmetricFragment extends Fragment implements EncryptActivi
|
|||||||
setEncryptionKeyIds(keyIdsArr);
|
setEncryptionKeyIds(keyIdsArr);
|
||||||
setEncryptionUserIds(userIds.toArray(new String[userIds.size()]));
|
setEncryptionUserIds(userIds.toArray(new String[userIds.size()]));
|
||||||
}
|
}
|
||||||
|
|
||||||
private class SelectSignKeyCursorAdapter extends BaseAdapter implements SpinnerAdapter {
|
|
||||||
private CursorAdapter inner;
|
|
||||||
private int mIndexUserId;
|
|
||||||
private int mIndexKeyId;
|
|
||||||
private int mIndexMasterKeyId;
|
|
||||||
|
|
||||||
public SelectSignKeyCursorAdapter() {
|
|
||||||
inner = new CursorAdapter(null, null, 0) {
|
|
||||||
@Override
|
|
||||||
public View newView(Context context, Cursor cursor, ViewGroup parent) {
|
|
||||||
return getActivity().getLayoutInflater().inflate(R.layout.encrypt_asymmetric_signkey, null);
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public void bindView(View view, Context context, Cursor cursor) {
|
|
||||||
String[] userId = KeyRing.splitUserId(cursor.getString(mIndexUserId));
|
|
||||||
((TextView) view.findViewById(android.R.id.title)).setText(userId[2] == null ? userId[0] : (userId[0] + " (" + userId[2] + ")"));
|
|
||||||
((TextView) view.findViewById(android.R.id.text1)).setText(userId[1]);
|
|
||||||
((TextView) view.findViewById(android.R.id.text2)).setText(PgpKeyHelper.convertKeyIdToHex(cursor.getLong(mIndexKeyId)));
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public long getItemId(int position) {
|
|
||||||
mCursor.moveToPosition(position);
|
|
||||||
return mCursor.getLong(mIndexMasterKeyId);
|
|
||||||
}
|
|
||||||
};
|
|
||||||
}
|
|
||||||
|
|
||||||
public Cursor swapCursor(Cursor newCursor) {
|
|
||||||
if (newCursor == null) return inner.swapCursor(null);
|
|
||||||
|
|
||||||
mIndexKeyId = newCursor.getColumnIndex(KeyRings.KEY_ID);
|
|
||||||
mIndexUserId = newCursor.getColumnIndex(KeyRings.USER_ID);
|
|
||||||
mIndexMasterKeyId = newCursor.getColumnIndex(KeyRings.MASTER_KEY_ID);
|
|
||||||
if (newCursor.moveToFirst()) {
|
|
||||||
do {
|
|
||||||
if (newCursor.getLong(mIndexMasterKeyId) == mEncryptInterface.getSignatureKey()) {
|
|
||||||
mSign.setSelection(newCursor.getPosition() + 1);
|
|
||||||
}
|
|
||||||
} while (newCursor.moveToNext());
|
|
||||||
}
|
|
||||||
return inner.swapCursor(newCursor);
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public int getCount() {
|
|
||||||
return inner.getCount() + 1;
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public Object getItem(int position) {
|
|
||||||
if (position == 0) return null;
|
|
||||||
return inner.getItem(position - 1);
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public long getItemId(int position) {
|
|
||||||
if (position == 0) return Constants.key.none;
|
|
||||||
return inner.getItemId(position - 1);
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public View getView(int position, View convertView, ViewGroup parent) {
|
|
||||||
View v = getDropDownView(position, convertView, parent);
|
|
||||||
v.findViewById(android.R.id.text1).setVisibility(View.GONE);
|
|
||||||
return v;
|
|
||||||
}
|
|
||||||
|
|
||||||
@Override
|
|
||||||
public View getDropDownView(int position, View convertView, ViewGroup parent) {
|
|
||||||
View v;
|
|
||||||
if (position == 0) {
|
|
||||||
if (convertView == null) {
|
|
||||||
v = inner.newView(null, null, parent);
|
|
||||||
} else {
|
|
||||||
v = convertView;
|
|
||||||
}
|
|
||||||
((TextView) v.findViewById(android.R.id.title)).setText("None");
|
|
||||||
v.findViewById(android.R.id.text1).setVisibility(View.GONE);
|
|
||||||
v.findViewById(android.R.id.text2).setVisibility(View.GONE);
|
|
||||||
} else {
|
|
||||||
v = inner.getView(position - 1, convertView, parent);
|
|
||||||
v.findViewById(android.R.id.text1).setVisibility(View.VISIBLE);
|
|
||||||
v.findViewById(android.R.id.text2).setVisibility(View.VISIBLE);
|
|
||||||
}
|
|
||||||
return v;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
}
|
}
|
||||||
|
@ -0,0 +1,60 @@
|
|||||||
|
package org.sufficientlysecure.keychain.ui.widget;
|
||||||
|
|
||||||
|
import android.content.Context;
|
||||||
|
import android.database.Cursor;
|
||||||
|
import android.net.Uri;
|
||||||
|
import android.support.v4.content.CursorLoader;
|
||||||
|
import android.support.v4.content.Loader;
|
||||||
|
import android.util.AttributeSet;
|
||||||
|
import org.sufficientlysecure.keychain.Constants;
|
||||||
|
import org.sufficientlysecure.keychain.provider.KeychainContract;
|
||||||
|
import org.sufficientlysecure.keychain.provider.KeychainDatabase;
|
||||||
|
|
||||||
|
public class CertifyKeySpinner extends KeySpinner {
|
||||||
|
private long mHiddenMasterKeyId = Constants.key.none;
|
||||||
|
|
||||||
|
public CertifyKeySpinner(Context context) {
|
||||||
|
super(context);
|
||||||
|
}
|
||||||
|
|
||||||
|
public CertifyKeySpinner(Context context, AttributeSet attrs) {
|
||||||
|
super(context, attrs);
|
||||||
|
}
|
||||||
|
|
||||||
|
public CertifyKeySpinner(Context context, AttributeSet attrs, int defStyle) {
|
||||||
|
super(context, attrs, defStyle);
|
||||||
|
}
|
||||||
|
|
||||||
|
public void setHiddenMasterKeyId(long hiddenMasterKeyId) {
|
||||||
|
this.mHiddenMasterKeyId = hiddenMasterKeyId;
|
||||||
|
reload();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public Loader<Cursor> onCreateLoader() {
|
||||||
|
// 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();
|
||||||
|
|
||||||
|
// These are the rows that we will retrieve.
|
||||||
|
String[] projection = new String[]{
|
||||||
|
KeychainContract.KeyRings._ID,
|
||||||
|
KeychainContract.KeyRings.MASTER_KEY_ID,
|
||||||
|
KeychainContract.KeyRings.KEY_ID,
|
||||||
|
KeychainContract.KeyRings.USER_ID,
|
||||||
|
KeychainContract.KeyRings.IS_EXPIRED,
|
||||||
|
KeychainContract.KeyRings.HAS_CERTIFY,
|
||||||
|
KeychainContract.KeyRings.HAS_ANY_SECRET
|
||||||
|
};
|
||||||
|
|
||||||
|
String where = KeychainContract.KeyRings.HAS_ANY_SECRET + " = 1 AND "
|
||||||
|
+ KeychainContract.KeyRings.HAS_CERTIFY + " NOT NULL AND "
|
||||||
|
+ KeychainContract.KeyRings.IS_REVOKED + " = 0 AND "
|
||||||
|
+ KeychainContract.KeyRings.IS_EXPIRED + " = 0 AND " + KeychainDatabase.Tables.KEYS + "."
|
||||||
|
+ KeychainContract.KeyRings.MASTER_KEY_ID + " != " + mHiddenMasterKeyId;
|
||||||
|
|
||||||
|
// Now create and return a CursorLoader that will take care of
|
||||||
|
// creating a Cursor for the data being displayed.
|
||||||
|
return new CursorLoader(getContext(), baseUri, projection, where, null, null);
|
||||||
|
}
|
||||||
|
}
|
@ -111,7 +111,7 @@ public class EncryptKeyCompletionView extends TokenCompleteTextView {
|
|||||||
protected void onAttachedToWindow() {
|
protected void onAttachedToWindow() {
|
||||||
super.onAttachedToWindow();
|
super.onAttachedToWindow();
|
||||||
if (getContext() instanceof FragmentActivity) {
|
if (getContext() instanceof FragmentActivity) {
|
||||||
((FragmentActivity) getContext()).getSupportLoaderManager().initLoader(0, null, new LoaderManager.LoaderCallbacks<Cursor>() {
|
((FragmentActivity) getContext()).getSupportLoaderManager().initLoader(hashCode(), null, new LoaderManager.LoaderCallbacks<Cursor>() {
|
||||||
@Override
|
@Override
|
||||||
public Loader<Cursor> onCreateLoader(int id, Bundle args) {
|
public Loader<Cursor> onCreateLoader(int id, Bundle args) {
|
||||||
// These are the rows that we will retrieve.
|
// These are the rows that we will retrieve.
|
||||||
@ -143,6 +143,8 @@ public class EncryptKeyCompletionView extends TokenCompleteTextView {
|
|||||||
swapCursor(null);
|
swapCursor(null);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
} else {
|
||||||
|
Log.e(Constants.TAG, "EncryptKeyCompletionView must be attached to a FragmentActivity, this is " + getContext().getClass());
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -0,0 +1,215 @@
|
|||||||
|
package org.sufficientlysecure.keychain.ui.widget;
|
||||||
|
|
||||||
|
import android.content.Context;
|
||||||
|
import android.database.Cursor;
|
||||||
|
import android.os.Bundle;
|
||||||
|
import android.support.v4.app.FragmentActivity;
|
||||||
|
import android.support.v4.app.LoaderManager;
|
||||||
|
import android.support.v4.content.Loader;
|
||||||
|
import android.support.v4.widget.CursorAdapter;
|
||||||
|
import android.util.AttributeSet;
|
||||||
|
import android.view.View;
|
||||||
|
import android.view.ViewGroup;
|
||||||
|
import android.widget.AdapterView;
|
||||||
|
import android.widget.BaseAdapter;
|
||||||
|
import android.widget.Spinner;
|
||||||
|
import android.widget.SpinnerAdapter;
|
||||||
|
import android.widget.TextView;
|
||||||
|
|
||||||
|
import org.sufficientlysecure.keychain.Constants;
|
||||||
|
import org.sufficientlysecure.keychain.R;
|
||||||
|
import org.sufficientlysecure.keychain.pgp.KeyRing;
|
||||||
|
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 interface OnKeyChangedListener {
|
||||||
|
public void onKeyChanged(long masterKeyId);
|
||||||
|
}
|
||||||
|
|
||||||
|
private long mSelectedKeyId;
|
||||||
|
private SelectKeyAdapter mAdapter = new SelectKeyAdapter();
|
||||||
|
private OnKeyChangedListener mListener;
|
||||||
|
|
||||||
|
public KeySpinner(Context context) {
|
||||||
|
super(context);
|
||||||
|
initView();
|
||||||
|
}
|
||||||
|
|
||||||
|
public KeySpinner(Context context, AttributeSet attrs) {
|
||||||
|
super(context, attrs);
|
||||||
|
initView();
|
||||||
|
}
|
||||||
|
|
||||||
|
public KeySpinner(Context context, AttributeSet attrs, int defStyle) {
|
||||||
|
super(context, attrs, defStyle);
|
||||||
|
initView();
|
||||||
|
}
|
||||||
|
|
||||||
|
private void initView() {
|
||||||
|
setAdapter(mAdapter);
|
||||||
|
super.setOnItemSelectedListener(new OnItemSelectedListener() {
|
||||||
|
@Override
|
||||||
|
public void onItemSelected(AdapterView<?> parent, View view, int position, long id) {
|
||||||
|
if (mListener != null) {
|
||||||
|
mListener.onKeyChanged(id);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void onNothingSelected(AdapterView<?> parent) {
|
||||||
|
if (mListener != null) {
|
||||||
|
mListener.onKeyChanged(Constants.key.none);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
public abstract Loader<Cursor> onCreateLoader();
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void setOnItemSelectedListener(OnItemSelectedListener listener) {
|
||||||
|
throw new UnsupportedOperationException();
|
||||||
|
}
|
||||||
|
|
||||||
|
public void setOnKeyChangedListener(OnKeyChangedListener listener) {
|
||||||
|
mListener = listener;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
protected void onAttachedToWindow() {
|
||||||
|
super.onAttachedToWindow();
|
||||||
|
reload();
|
||||||
|
}
|
||||||
|
|
||||||
|
public void reload() {
|
||||||
|
if (getContext() instanceof FragmentActivity) {
|
||||||
|
((FragmentActivity) getContext()).getSupportLoaderManager().restartLoader(hashCode(), null, new LoaderManager.LoaderCallbacks<Cursor>() {
|
||||||
|
@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());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
public long getSelectedKeyId() {
|
||||||
|
return mSelectedKeyId;
|
||||||
|
}
|
||||||
|
|
||||||
|
public void setSelectedKeyId(long selectedKeyId) {
|
||||||
|
this.mSelectedKeyId = selectedKeyId;
|
||||||
|
}
|
||||||
|
|
||||||
|
private class SelectKeyAdapter extends BaseAdapter implements SpinnerAdapter {
|
||||||
|
private CursorAdapter inner;
|
||||||
|
private int mIndexUserId;
|
||||||
|
private int mIndexKeyId;
|
||||||
|
private int mIndexMasterKeyId;
|
||||||
|
|
||||||
|
public SelectKeyAdapter() {
|
||||||
|
inner = new CursorAdapter(null, null, 0) {
|
||||||
|
@Override
|
||||||
|
public View newView(Context context, Cursor cursor, ViewGroup parent) {
|
||||||
|
return View.inflate(getContext(), R.layout.keyspinner_key, null);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void bindView(View view, Context context, Cursor cursor) {
|
||||||
|
String[] userId = KeyRing.splitUserId(cursor.getString(mIndexUserId));
|
||||||
|
((TextView) view.findViewById(android.R.id.title)).setText(userId[2] == null ? userId[0] : (userId[0] + " (" + userId[2] + ")"));
|
||||||
|
((TextView) view.findViewById(android.R.id.text1)).setText(userId[1]);
|
||||||
|
((TextView) view.findViewById(android.R.id.text2)).setText(PgpKeyHelper.convertKeyIdToHex(cursor.getLong(mIndexKeyId)));
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public long getItemId(int position) {
|
||||||
|
try {
|
||||||
|
return ((Cursor) getItem(position)).getLong(mIndexMasterKeyId);
|
||||||
|
} catch (Exception e) {
|
||||||
|
// This can happen on concurrent modification :(
|
||||||
|
return Constants.key.none;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
};
|
||||||
|
}
|
||||||
|
|
||||||
|
public Cursor swapCursor(Cursor newCursor) {
|
||||||
|
if (newCursor == null) return inner.swapCursor(null);
|
||||||
|
|
||||||
|
mIndexKeyId = newCursor.getColumnIndex(KeychainContract.KeyRings.KEY_ID);
|
||||||
|
mIndexUserId = newCursor.getColumnIndex(KeychainContract.KeyRings.USER_ID);
|
||||||
|
mIndexMasterKeyId = newCursor.getColumnIndex(KeychainContract.KeyRings.MASTER_KEY_ID);
|
||||||
|
if (newCursor.moveToFirst()) {
|
||||||
|
do {
|
||||||
|
if (newCursor.getLong(mIndexMasterKeyId) == mSelectedKeyId) {
|
||||||
|
setSelection(newCursor.getPosition() + 1);
|
||||||
|
}
|
||||||
|
} while (newCursor.moveToNext());
|
||||||
|
}
|
||||||
|
return inner.swapCursor(newCursor);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public int getCount() {
|
||||||
|
return inner.getCount() + 1;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public Object getItem(int position) {
|
||||||
|
if (position == 0) return null;
|
||||||
|
return inner.getItem(position - 1);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public long getItemId(int position) {
|
||||||
|
if (position == 0) return Constants.key.none;
|
||||||
|
return inner.getItemId(position - 1);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public View getView(int position, View convertView, ViewGroup parent) {
|
||||||
|
try {
|
||||||
|
View v = getDropDownView(position, convertView, parent);
|
||||||
|
v.findViewById(android.R.id.text1).setVisibility(View.GONE);
|
||||||
|
return v;
|
||||||
|
} catch (NullPointerException e) {
|
||||||
|
// This is for the preview...
|
||||||
|
return View.inflate(getContext(), android.R.layout.simple_list_item_1, null);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public View getDropDownView(int position, View convertView, ViewGroup parent) {
|
||||||
|
View v;
|
||||||
|
if (position == 0) {
|
||||||
|
if (convertView == null) {
|
||||||
|
v = inner.newView(null, null, parent);
|
||||||
|
} else {
|
||||||
|
v = convertView;
|
||||||
|
}
|
||||||
|
((TextView) v.findViewById(android.R.id.title)).setText("None");
|
||||||
|
v.findViewById(android.R.id.text1).setVisibility(View.GONE);
|
||||||
|
v.findViewById(android.R.id.text2).setVisibility(View.GONE);
|
||||||
|
} else {
|
||||||
|
v = inner.getView(position - 1, convertView, parent);
|
||||||
|
v.findViewById(android.R.id.text1).setVisibility(View.VISIBLE);
|
||||||
|
v.findViewById(android.R.id.text2).setVisibility(View.VISIBLE);
|
||||||
|
}
|
||||||
|
return v;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
@ -0,0 +1,48 @@
|
|||||||
|
package org.sufficientlysecure.keychain.ui.widget;
|
||||||
|
|
||||||
|
import android.content.Context;
|
||||||
|
import android.database.Cursor;
|
||||||
|
import android.net.Uri;
|
||||||
|
import android.support.v4.content.CursorLoader;
|
||||||
|
import android.support.v4.content.Loader;
|
||||||
|
import android.util.AttributeSet;
|
||||||
|
import org.sufficientlysecure.keychain.provider.KeychainContract;
|
||||||
|
|
||||||
|
public class SignKeySpinner extends KeySpinner {
|
||||||
|
public SignKeySpinner(Context context) {
|
||||||
|
super(context);
|
||||||
|
}
|
||||||
|
|
||||||
|
public SignKeySpinner(Context context, AttributeSet attrs) {
|
||||||
|
super(context, attrs);
|
||||||
|
}
|
||||||
|
|
||||||
|
public SignKeySpinner(Context context, AttributeSet attrs, int defStyle) {
|
||||||
|
super(context, attrs, defStyle);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public Loader<Cursor> onCreateLoader() {
|
||||||
|
// 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();
|
||||||
|
|
||||||
|
// These are the rows that we will retrieve.
|
||||||
|
String[] projection = new String[]{
|
||||||
|
KeychainContract.KeyRings._ID,
|
||||||
|
KeychainContract.KeyRings.MASTER_KEY_ID,
|
||||||
|
KeychainContract.KeyRings.KEY_ID,
|
||||||
|
KeychainContract.KeyRings.USER_ID,
|
||||||
|
KeychainContract.KeyRings.IS_EXPIRED,
|
||||||
|
KeychainContract.KeyRings.HAS_SIGN,
|
||||||
|
KeychainContract.KeyRings.HAS_ANY_SECRET
|
||||||
|
};
|
||||||
|
|
||||||
|
String where = KeychainContract.KeyRings.HAS_ANY_SECRET + " = 1 AND " + KeychainContract.KeyRings.HAS_SIGN + " NOT NULL AND "
|
||||||
|
+ KeychainContract.KeyRings.IS_REVOKED + " = 0 AND " + KeychainContract.KeyRings.IS_EXPIRED + " = 0";
|
||||||
|
|
||||||
|
// Now create and return a CursorLoader that will take care of
|
||||||
|
// creating a Cursor for the data being displayed.
|
||||||
|
return new CursorLoader(getContext(), baseUri, projection, where, null, null);
|
||||||
|
}
|
||||||
|
}
|
@ -26,14 +26,10 @@
|
|||||||
android:layout_marginTop="14dp"
|
android:layout_marginTop="14dp"
|
||||||
android:text="@string/section_certification_key" />
|
android:text="@string/section_certification_key" />
|
||||||
|
|
||||||
<fragment
|
<org.sufficientlysecure.keychain.ui.widget.CertifyKeySpinner
|
||||||
android:id="@+id/sign_key_select_key_fragment"
|
android:id="@+id/certify_key_spinner"
|
||||||
android:name="org.sufficientlysecure.keychain.ui.SelectSecretKeyLayoutFragment"
|
|
||||||
android:layout_width="match_parent"
|
android:layout_width="match_parent"
|
||||||
android:layout_height="wrap_content"
|
android:layout_height="wrap_content" />
|
||||||
android:layout_marginBottom="4dp"
|
|
||||||
android:layout_marginTop="4dp"
|
|
||||||
tools:layout="@layout/select_secret_key_layout_fragment" />
|
|
||||||
|
|
||||||
<TextView
|
<TextView
|
||||||
style="@style/SectionHeader"
|
style="@style/SectionHeader"
|
||||||
|
@ -24,7 +24,7 @@
|
|||||||
android:textAppearance="?android:attr/textAppearanceMedium"
|
android:textAppearance="?android:attr/textAppearanceMedium"
|
||||||
android:text="@string/label_asymmetric_from" />
|
android:text="@string/label_asymmetric_from" />
|
||||||
|
|
||||||
<Spinner
|
<org.sufficientlysecure.keychain.ui.widget.SignKeySpinner
|
||||||
android:id="@+id/sign"
|
android:id="@+id/sign"
|
||||||
android:layout_width="match_parent"
|
android:layout_width="match_parent"
|
||||||
android:layout_height="wrap_content"
|
android:layout_height="wrap_content"
|
||||||
|
Loading…
Reference in New Issue
Block a user