From 7e488f53b356e04b83ea34549ffcba27d850eb9a Mon Sep 17 00:00:00 2001 From: Anne Rong Date: Wed, 19 Aug 2015 18:32:08 -0700 Subject: FilteredNumberProvider authority name & filtered_number_table changes Authority name for FilteredNumberProvider changed so it can be overriden in GoogleDialer. Added UNIQUE constraint to NORMALIZED_NUMBER field. Added NUMBER field to filtered_number_table (un-normalized number). Bug: 23424013 Bug: 23424011 Change-Id: I3156cee70ddcaea15b7272650d5f255782e34520 --- .../android/dialer/database/DialerDatabaseHelper.java | 14 +++++++++----- .../dialer/database/FilteredNumberContract.java | 19 ++++++++++++++++--- .../dialer/database/FilteredNumberProvider.java | 15 +++++++-------- src/com/android/dialerbind/ObjectFactory.java | 4 ++++ 4 files changed, 36 insertions(+), 16 deletions(-) (limited to 'src') diff --git a/src/com/android/dialer/database/DialerDatabaseHelper.java b/src/com/android/dialer/database/DialerDatabaseHelper.java index 8fcbb9208..d36a0f6d8 100644 --- a/src/com/android/dialer/database/DialerDatabaseHelper.java +++ b/src/com/android/dialer/database/DialerDatabaseHelper.java @@ -75,7 +75,7 @@ public class DialerDatabaseHelper extends SQLiteOpenHelper { * 0-98 KitKat * */ - public static final int DATABASE_VERSION = 5; + public static final int DATABASE_VERSION = 6; public static final String DATABASE_NAME = "dialer.db"; /** @@ -400,7 +400,8 @@ public class DialerDatabaseHelper extends SQLiteOpenHelper { // and to be able to guarantee the state of the DB at each upgrade step. db.execSQL("CREATE TABLE " + Tables.FILTERED_NUMBER_TABLE + " (" + FilteredNumberColumns._ID + " INTEGER PRIMARY KEY AUTOINCREMENT," - + FilteredNumberColumns.NORMALIZED_NUMBER + " TEXT," + + FilteredNumberColumns.NORMALIZED_NUMBER + " TEXT UNIQUE," + + FilteredNumberColumns.NUMBER + " TEXT," + FilteredNumberColumns.COUNTRY_ISO + " TEXT," + FilteredNumberColumns.TIMES_FILTERED + " INTEGER," + FilteredNumberColumns.LAST_TIME_FILTERED + " LONG," @@ -418,6 +419,7 @@ public class DialerDatabaseHelper extends SQLiteOpenHelper { db.execSQL("DROP TABLE IF EXISTS " + Tables.PREFIX_TABLE); db.execSQL("DROP TABLE IF EXISTS " + Tables.SMARTDIAL_TABLE); db.execSQL("DROP TABLE IF EXISTS " + Tables.PROPERTIES); + db.execSQL("DROP TABLE IF EXISTS " + Tables.FILTERED_NUMBER_TABLE); } @Override @@ -438,10 +440,12 @@ public class DialerDatabaseHelper extends SQLiteOpenHelper { return; } - if (oldVersion < 5) { + if (oldVersion < 6) { + db.execSQL("DROP TABLE IF EXISTS " + Tables.FILTERED_NUMBER_TABLE); db.execSQL("CREATE TABLE " + Tables.FILTERED_NUMBER_TABLE + " (" + FilteredNumberColumns._ID + " INTEGER PRIMARY KEY AUTOINCREMENT," - + FilteredNumberColumns.NORMALIZED_NUMBER + " TEXT," + + FilteredNumberColumns.NORMALIZED_NUMBER + " TEXT UNIQUE," + + FilteredNumberColumns.NUMBER + " TEXT," + FilteredNumberColumns.COUNTRY_ISO + " TEXT," + FilteredNumberColumns.TIMES_FILTERED + " INTEGER," + FilteredNumberColumns.LAST_TIME_FILTERED + " LONG," @@ -449,7 +453,7 @@ public class DialerDatabaseHelper extends SQLiteOpenHelper { + FilteredNumberColumns.TYPE + " INTEGER," + FilteredNumberColumns.SOURCE + " INTEGER" + ");"); - oldVersion = 5; + oldVersion = 6; } if (oldVersion != DATABASE_VERSION) { diff --git a/src/com/android/dialer/database/FilteredNumberContract.java b/src/com/android/dialer/database/FilteredNumberContract.java index 1fb2363d0..0ec171b15 100644 --- a/src/com/android/dialer/database/FilteredNumberContract.java +++ b/src/com/android/dialer/database/FilteredNumberContract.java @@ -16,8 +16,15 @@ package com.android.dialer.database; +import android.content.ContentResolver; +import android.content.ContentUris; +import android.content.ContentValues; +import android.database.Cursor; import android.net.Uri; import android.provider.BaseColumns; +import android.telephony.PhoneNumberUtils; + +import com.android.dialerbind.ObjectFactory; /** *

@@ -29,7 +36,7 @@ import android.provider.BaseColumns; public final class FilteredNumberContract { /** The authority for the filtered numbers provider */ - public static final String AUTHORITY = "com.android.dialer"; + public static final String AUTHORITY = ObjectFactory.getFilteredNumberProviderAuthority(); /** A content:// style uri to the authority for the filtered numbers provider */ public static final Uri AUTHORITY_URI = Uri.parse("content://" + AUTHORITY); @@ -61,6 +68,13 @@ public final class FilteredNumberContract { * TYPE: TEXT */ static final String NORMALIZED_NUMBER = "normalized_number"; + /** + * Represents the number to be filtered, for formatting and + * used with country iso for contact lookups. + * + * TYPE: TEXT + */ + static final String NUMBER = "number"; /** * The country code representing the country detected when * the phone number was added to the database. @@ -111,7 +125,7 @@ public final class FilteredNumberContract { *

Operations

*
*
Insert
- *
Required fields: NORMALIZED_NUMBER, TYPE, SOURCE. + *
Required fields: NUMBER, NORMALIZED_NUMBER, TYPE, SOURCE. * A default value will be used for the other fields if left null.
*
Update
*
Delete
@@ -144,6 +158,5 @@ public final class FilteredNumberContract { */ public static final String CONTENT_ITEM_TYPE = "vnd.android.cursor.item/filtered_numbers_table"; - } } diff --git a/src/com/android/dialer/database/FilteredNumberProvider.java b/src/com/android/dialer/database/FilteredNumberProvider.java index 2bacd89b2..2bea7cae1 100644 --- a/src/com/android/dialer/database/FilteredNumberProvider.java +++ b/src/com/android/dialer/database/FilteredNumberProvider.java @@ -33,6 +33,7 @@ import android.util.Log; import com.android.contacts.common.GeoUtil; import com.android.dialer.database.FilteredNumberContract.FilteredNumberColumns; import com.android.dialerbind.DatabaseHelperManager; +import com.android.dialerbind.ObjectFactory; import com.google.common.annotations.VisibleForTesting; import java.util.Arrays; @@ -50,14 +51,6 @@ public class FilteredNumberProvider extends ContentProvider { private static final int FILTERED_NUMBERS_TABLE_ID = 2; private static final UriMatcher sUriMatcher = new UriMatcher(UriMatcher.NO_MATCH); - static { - sUriMatcher.addURI(FilteredNumberContract.AUTHORITY, - FilteredNumberContract.FilteredNumber.FILTERED_NUMBERS_TABLE, - FILTERED_NUMBERS_TABLE); - sUriMatcher.addURI(FilteredNumberContract.AUTHORITY, - FilteredNumberContract.FilteredNumber.FILTERED_NUMBERS_TABLE + "/#", - FILTERED_NUMBERS_TABLE_ID); - } @Override public boolean onCreate() { @@ -65,6 +58,12 @@ public class FilteredNumberProvider extends ContentProvider { if (mDialerDatabaseHelper == null) { return false; } + sUriMatcher.addURI(ObjectFactory.getFilteredNumberProviderAuthority(), + FilteredNumberContract.FilteredNumber.FILTERED_NUMBERS_TABLE, + FILTERED_NUMBERS_TABLE); + sUriMatcher.addURI(ObjectFactory.getFilteredNumberProviderAuthority(), + FilteredNumberContract.FilteredNumber.FILTERED_NUMBERS_TABLE + "/#", + FILTERED_NUMBERS_TABLE_ID); return true; } diff --git a/src/com/android/dialerbind/ObjectFactory.java b/src/com/android/dialerbind/ObjectFactory.java index 1a36b05d6..bace0a675 100644 --- a/src/com/android/dialerbind/ObjectFactory.java +++ b/src/com/android/dialerbind/ObjectFactory.java @@ -35,6 +35,10 @@ public class ObjectFactory { return null; } + public static String getFilteredNumberProviderAuthority() { + return "com.android.dialer.database.filterednumberprovider"; + } + /** * Create a new instance of the call log adapter. * @param context The context to use. -- cgit v1.2.3