diff options
author | Vincent Breitmoser <valodim@mugenguild.com> | 2014-09-22 16:47:29 +0200 |
---|---|---|
committer | Vincent Breitmoser <valodim@mugenguild.com> | 2014-09-22 16:47:29 +0200 |
commit | f6ac887ce0e440d19273889ff10bc1306d8bed50 (patch) | |
tree | 08326d15b03cd2dc0f082cd511a58d057d4821fb | |
parent | 029ce91670952128b231de7bfd7240a5bc29f056 (diff) | |
download | open-keychain-f6ac887ce0e440d19273889ff10bc1306d8bed50.tar.gz open-keychain-f6ac887ce0e440d19273889ff10bc1306d8bed50.tar.bz2 open-keychain-f6ac887ce0e440d19273889ff10bc1306d8bed50.zip |
nicer code in key import for flag log types
-rw-r--r-- | OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java | 102 |
1 files changed, 27 insertions, 75 deletions
diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java index 767ab588c..6856f087d 100644 --- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java +++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/ProviderHelper.java @@ -293,6 +293,30 @@ public class ProviderHelper { } } + // bits, in order: CESA. make SURE these are correct, we will get bad log entries otherwise!! + static final LogType LOG_TYPES_FLAG_MASTER[] = new LogType[] { + LogType.MSG_IP_MASTER_FLAGS_XXXX, LogType.MSG_IP_MASTER_FLAGS_CXXX, + LogType.MSG_IP_MASTER_FLAGS_XEXX, LogType.MSG_IP_MASTER_FLAGS_CEXX, + LogType.MSG_IP_MASTER_FLAGS_XXSX, LogType.MSG_IP_MASTER_FLAGS_CXSX, + LogType.MSG_IP_MASTER_FLAGS_XESX, LogType.MSG_IP_MASTER_FLAGS_CESX, + LogType.MSG_IP_MASTER_FLAGS_XXXA, LogType.MSG_IP_MASTER_FLAGS_CXXA, + LogType.MSG_IP_MASTER_FLAGS_XEXA, LogType.MSG_IP_MASTER_FLAGS_CEXA, + LogType.MSG_IP_MASTER_FLAGS_XXSA, LogType.MSG_IP_MASTER_FLAGS_CXSA, + LogType.MSG_IP_MASTER_FLAGS_XESA, LogType.MSG_IP_MASTER_FLAGS_CESA + }; + + // same as above, but for subkeys + static final LogType LOG_TYPES_FLAG_SUBKEY[] = new LogType[] { + LogType.MSG_IP_SUBKEY_FLAGS_XXXX, LogType.MSG_IP_SUBKEY_FLAGS_CXXX, + LogType.MSG_IP_SUBKEY_FLAGS_XEXX, LogType.MSG_IP_SUBKEY_FLAGS_CEXX, + LogType.MSG_IP_SUBKEY_FLAGS_XXSX, LogType.MSG_IP_SUBKEY_FLAGS_CXSX, + LogType.MSG_IP_SUBKEY_FLAGS_XESX, LogType.MSG_IP_SUBKEY_FLAGS_CESX, + LogType.MSG_IP_SUBKEY_FLAGS_XXXA, LogType.MSG_IP_SUBKEY_FLAGS_CXXA, + LogType.MSG_IP_SUBKEY_FLAGS_XEXA, LogType.MSG_IP_SUBKEY_FLAGS_CEXA, + LogType.MSG_IP_SUBKEY_FLAGS_XXSA, LogType.MSG_IP_SUBKEY_FLAGS_CXSA, + LogType.MSG_IP_SUBKEY_FLAGS_XESA, LogType.MSG_IP_SUBKEY_FLAGS_CESA + }; + /** Saves an UncachedKeyRing of the public variant into the db. * * This method will not delete all previous data for this masterKeyId from the database prior @@ -363,83 +387,11 @@ public class ProviderHelper { values.put(Keys.CAN_AUTHENTICATE, a); values.put(Keys.IS_REVOKED, key.isRevoked()); + // see above if (masterKeyId == keyId) { - // yes it's ugly. got any better ideas? - if (a) { - if (c) { - if (e) { - log(s ? LogType.MSG_IP_MASTER_FLAGS_CESA - : LogType.MSG_IP_MASTER_FLAGS_CEXA); - } else { - log(s ? LogType.MSG_IP_MASTER_FLAGS_CXSA - : LogType.MSG_IP_MASTER_FLAGS_CXXA); - } - } else { - if (e) { - log(s ? LogType.MSG_IP_MASTER_FLAGS_XESA - : LogType.MSG_IP_MASTER_FLAGS_XEXA); - } else { - log(s ? LogType.MSG_IP_MASTER_FLAGS_XXSA - : LogType.MSG_IP_MASTER_FLAGS_XXXA); - } - } - } else { - if (c) { - if (e) { - log(s ? LogType.MSG_IP_MASTER_FLAGS_CESX - : LogType.MSG_IP_MASTER_FLAGS_CEXX); - } else { - log(s ? LogType.MSG_IP_MASTER_FLAGS_CXSX - : LogType.MSG_IP_MASTER_FLAGS_CXXX); - } - } else { - if (e) { - log(s ? LogType.MSG_IP_MASTER_FLAGS_XESX - : LogType.MSG_IP_MASTER_FLAGS_XEXX); - } else { - log(s ? LogType.MSG_IP_MASTER_FLAGS_XXSX - : LogType.MSG_IP_MASTER_FLAGS_XXXX); - } - } - } + log(LOG_TYPES_FLAG_MASTER[(c?1:0) + (e?2:0) + (s?4:0) + (a?8:0)]); } else { - if (a) { - if (c) { - if (e) { - log(s ? LogType.MSG_IP_SUBKEY_FLAGS_CESA - : LogType.MSG_IP_SUBKEY_FLAGS_CEXA); - } else { - log(s ? LogType.MSG_IP_SUBKEY_FLAGS_CXSA - : LogType.MSG_IP_SUBKEY_FLAGS_CXXA); - } - } else { - if (e) { - log(s ? LogType.MSG_IP_SUBKEY_FLAGS_XESA - : LogType.MSG_IP_SUBKEY_FLAGS_XEXA); - } else { - log(s ? LogType.MSG_IP_SUBKEY_FLAGS_XXSA - : LogType.MSG_IP_SUBKEY_FLAGS_XXXA); - } - } - } else { - if (c) { - if (e) { - log(s ? LogType.MSG_IP_SUBKEY_FLAGS_CESX - : LogType.MSG_IP_SUBKEY_FLAGS_CEXX); - } else { - log(s ? LogType.MSG_IP_SUBKEY_FLAGS_CXSX - : LogType.MSG_IP_SUBKEY_FLAGS_CXXX); - } - } else { - if (e) { - log(s ? LogType.MSG_IP_SUBKEY_FLAGS_XESX - : LogType.MSG_IP_SUBKEY_FLAGS_XEXX); - } else { - log(s ? LogType.MSG_IP_SUBKEY_FLAGS_XXSX - : LogType.MSG_IP_SUBKEY_FLAGS_XXXX); - } - } - } + log(LOG_TYPES_FLAG_SUBKEY[(c?1:0) + (e?2:0) + (s?4:0) + (a?8:0)]); } Date creation = key.getCreationTime(); |