moparisthebest/open-keychain
1
0
Fork 0
mirror of https://github.com/moparisthebest/open-keychain synced 2025-01-31 15:10:19 -05:00
Code Issues Releases Wiki Activity

Always use INTEGER instead of BOOLEAN in database to avoid confusion

Browse Source
This commit is contained in:
Dominik Schürmann 2014-10-29 23:41:10 +01:00
parent 3e601431ea
commit 30c3021fea
1 changed files with 10 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
OpenKeychain/src/main/java/org/sufficientlysecure/keychain/provider/KeychainDatabase.java
View File

@ -49,8 +49,6 @@ import java.io.IOException;
* - REAL. The value is a floating point value, stored as an 8-byte IEEE floating point number. * - REAL. The value is a floating point value, stored as an 8-byte IEEE floating point number.
* - TEXT. The value is a text string, stored using the database encoding (UTF-8, UTF-16BE or UTF-16LE). * - TEXT. The value is a text string, stored using the database encoding (UTF-8, UTF-16BE or UTF-16LE).
* - BLOB. The value is a blob of data, stored exactly as it was input. * - BLOB. The value is a blob of data, stored exactly as it was input.
*
* Adding BOOLEAN results in an INTEGER, but we keep BOOLEAN to indicate the usage!
*/ */
public class KeychainDatabase extends SQLiteOpenHelper { public class KeychainDatabase extends SQLiteOpenHelper {
private static final String DATABASE_NAME = "openkeychain.db"; private static final String DATABASE_NAME = "openkeychain.db";
@ -93,12 +91,12 @@ public class KeychainDatabase extends SQLiteOpenHelper {
+ KeysColumns.ALGORITHM + " INTEGER, " + KeysColumns.ALGORITHM + " INTEGER, "
+ KeysColumns.FINGERPRINT + " BLOB, " + KeysColumns.FINGERPRINT + " BLOB, "
+ KeysColumns.CAN_CERTIFY + " BOOLEAN, " + KeysColumns.CAN_CERTIFY + " INTEGER, "
+ KeysColumns.CAN_SIGN + " BOOLEAN, " + KeysColumns.CAN_SIGN + " INTEGER, "
+ KeysColumns.CAN_ENCRYPT + " BOOLEAN, " + KeysColumns.CAN_ENCRYPT + " INTEGER, "
+ KeysColumns.CAN_AUTHENTICATE + " BOOLEAN, " + KeysColumns.CAN_AUTHENTICATE + " INTEGER, "
+ KeysColumns.IS_REVOKED + " BOOLEAN, " + KeysColumns.IS_REVOKED + " INTEGER, "
+ KeysColumns.HAS_SECRET + " BOOLEAN, " + KeysColumns.HAS_SECRET + " INTEGER, "
+ KeysColumns.CREATION + " INTEGER, " + KeysColumns.CREATION + " INTEGER, "
+ KeysColumns.EXPIRY + " INTEGER, " + KeysColumns.EXPIRY + " INTEGER, "
@ -113,8 +111,8 @@ public class KeychainDatabase extends SQLiteOpenHelper {
+ UserIdsColumns.MASTER_KEY_ID + " INTEGER, " + UserIdsColumns.MASTER_KEY_ID + " INTEGER, "
+ UserIdsColumns.USER_ID + " TEXT, " + UserIdsColumns.USER_ID + " TEXT, "
+ UserIdsColumns.IS_PRIMARY + " BOOLEAN, " + UserIdsColumns.IS_PRIMARY + " INTEGER, "
+ UserIdsColumns.IS_REVOKED + " BOOLEAN, " + UserIdsColumns.IS_REVOKED + " INTEGER, "
+ UserIdsColumns.RANK+ " INTEGER, " + UserIdsColumns.RANK+ " INTEGER, "
+ "PRIMARY KEY(" + UserIdsColumns.MASTER_KEY_ID + ", " + UserIdsColumns.USER_ID + "), " + "PRIMARY KEY(" + UserIdsColumns.MASTER_KEY_ID + ", " + UserIdsColumns.USER_ID + "), "
@ -214,7 +212,7 @@ public class KeychainDatabase extends SQLiteOpenHelper {
case 1: case 1:
// add has_secret for all who are upgrading from a beta version // add has_secret for all who are upgrading from a beta version
try { try {
db.execSQL("ALTER TABLE keys ADD COLUMN has_secret BOOLEAN"); db.execSQL("ALTER TABLE keys ADD COLUMN has_secret INTEGER");
} catch (Exception e) { } catch (Exception e) {
// never mind, the column probably already existed // never mind, the column probably already existed
} }
@ -232,7 +230,7 @@ public class KeychainDatabase extends SQLiteOpenHelper {
// fall through // fall through
case 4: case 4:
try { try {
db.execSQL("ALTER TABLE keys ADD COLUMN can_authenticate BOOLEAN"); db.execSQL("ALTER TABLE keys ADD COLUMN can_authenticate INTEGER");
} catch (Exception e) { } catch (Exception e) {
// never mind, the column probably already existed // never mind, the column probably already existed
} }