From e987eb096f40ab987690e9bd10b058ef2dfe2de3 Mon Sep 17 00:00:00 2001 From: linyuh Date: Thu, 8 Mar 2018 10:28:04 -0800 Subject: Implement logic for blocking a number from the new call log's bottom sheet. Bug: 70989547 Test: ShowBlockReportSpamDialogNotifierEndToEndTest PiperOrigin-RevId: 188351591 Change-Id: I634fb821592bdc890df291fdf83cdf307c94ffa9 --- .../blockreportspam/BlockReportSpamDialogs.java | 102 ++++++++++++++++----- .../ShowBlockReportSpamDialogNotifier.java | 13 +++ .../ShowBlockReportSpamDialogReceiver.java | 61 +++++++++--- .../dialer/blockreportspam/res/values/strings.xml | 2 +- 4 files changed, 141 insertions(+), 37 deletions(-) (limited to 'java/com/android/dialer/blockreportspam') diff --git a/java/com/android/dialer/blockreportspam/BlockReportSpamDialogs.java b/java/com/android/dialer/blockreportspam/BlockReportSpamDialogs.java index b75669fa3..2ec5dbd29 100644 --- a/java/com/android/dialer/blockreportspam/BlockReportSpamDialogs.java +++ b/java/com/android/dialer/blockreportspam/BlockReportSpamDialogs.java @@ -29,9 +29,7 @@ import android.widget.CheckBox; import android.widget.TextView; import com.android.dialer.blocking.FilteredNumberCompat; -/** - * Helper class for creating dialog fragments to block a number and/or report it as spam/not spam. - */ +/** Creates dialog fragments to block a number and/or report it as spam/not spam. */ public final class BlockReportSpamDialogs { public static final String BLOCK_REPORT_SPAM_DIALOG_TAG = "BlockReportSpamDialog"; @@ -39,7 +37,7 @@ public final class BlockReportSpamDialogs { public static final String UNBLOCK_DIALOG_TAG = "UnblockDialog"; public static final String NOT_SPAM_DIALOG_TAG = "NotSpamDialog"; - /** Creates a dialog with the default cancel button listener (dismisses dialog). */ + /** Creates a dialog with the default cancel button listener (which dismisses the dialog). */ private static AlertDialog.Builder createDialogBuilder( Activity activity, final DialogFragment fragment) { return new AlertDialog.Builder(activity, R.style.AlertDialogTheme) @@ -70,36 +68,36 @@ public final class BlockReportSpamDialogs { } /** - * Listener passed to block/report spam dialog for positive click in {@link - * BlockReportSpamDialogFragment}. + * Positive listener for the "Block/Report spam" dialog {@link + * DialogFragmentForBlockingNumberAndOptionallyReportingAsSpam}. */ public interface OnSpamDialogClickListener { /** - * Called when user clicks on positive button in block/report spam dialog. + * Called when the user clicks on the positive button of the "Block/Report spam" dialog. * * @param isSpamChecked Whether the spam checkbox is checked. */ void onClick(boolean isSpamChecked); } - /** Listener passed to all dialogs except the block/report spam dialog for positive click. */ + /** Positive listener for dialogs other than the "Block/Report spam" dialog. */ public interface OnConfirmListener { - /** Called when user clicks on positive button in the dialog. */ + /** Called when the user clicks on the positive button of the dialog. */ void onClick(); } - /** Contains the common attributes between all block/unblock/report spam dialog fragments. */ + /** Contains common attributes shared among all dialog fragments. */ private abstract static class CommonDialogsFragment extends DialogFragment { /** The number to display in the dialog title. */ protected String displayNumber; - /** Called when dialog positive button is pressed. */ + /** Listener for the positive button. */ protected OnConfirmListener positiveListener; - /** Called when dialog is dismissed. */ + /** Listener for when the dialog is dismissed. */ @Nullable protected DialogInterface.OnDismissListener dismissListener; @Override @@ -121,8 +119,14 @@ public final class BlockReportSpamDialogs { } } - /** Dialog for block/report spam with the mark as spam checkbox. */ - public static class BlockReportSpamDialogFragment extends CommonDialogsFragment { + /** + * Dialog for blocking a number and optionally reporting it as spam. + * + *

This dialog is for a number that is neither blocked nor marked as spam. It has a checkbox + * that allows the user to report a number as spam when they block it. + */ + public static class DialogFragmentForBlockingNumberAndOptionallyReportingAsSpam + extends CommonDialogsFragment { /** Called when dialog positive button is pressed. */ private OnSpamDialogClickListener onSpamDialogClickListener; @@ -135,7 +139,8 @@ public final class BlockReportSpamDialogs { boolean spamChecked, OnSpamDialogClickListener onSpamDialogClickListener, @Nullable DialogInterface.OnDismissListener dismissListener) { - BlockReportSpamDialogFragment fragment = new BlockReportSpamDialogFragment(); + DialogFragmentForBlockingNumberAndOptionallyReportingAsSpam fragment = + new DialogFragmentForBlockingNumberAndOptionallyReportingAsSpam(); fragment.spamChecked = spamChecked; fragment.displayNumber = displayNumber; fragment.onSpamDialogClickListener = onSpamDialogClickListener; @@ -173,8 +178,14 @@ public final class BlockReportSpamDialogs { } } - /** Dialog for blocking a number. */ - public static class BlockDialogFragment extends CommonDialogsFragment { + /** + * Dialog for blocking a number and reporting it as spam. + * + *

This dialog is for the migration of blocked numbers. Its positive action should block a + * number, and also marks it as spam if the spam feature is enabled. + */ + public static class DialogFragmentForBlockingNumberAndReportingAsSpam + extends CommonDialogsFragment { private boolean isSpamEnabled; @@ -183,7 +194,8 @@ public final class BlockReportSpamDialogs { boolean isSpamEnabled, OnConfirmListener positiveListener, @Nullable DialogInterface.OnDismissListener dismissListener) { - BlockDialogFragment fragment = new BlockDialogFragment(); + DialogFragmentForBlockingNumberAndReportingAsSpam fragment = + new DialogFragmentForBlockingNumberAndReportingAsSpam(); fragment.displayNumber = displayNumber; fragment.positiveListener = positiveListener; fragment.dismissListener = dismissListener; @@ -212,8 +224,51 @@ public final class BlockReportSpamDialogs { } } - /** Dialog for unblocking a number. */ - public static class UnblockDialogFragment extends CommonDialogsFragment { + /** + * Dialog for blocking a number. + * + *

This dialog is for a spam number that hasn't been blocked. For example, if the user receives + * a spam call, this dialog will be shown if they would like to block the number. + */ + public static class DialogFragmentForBlockingNumber extends CommonDialogsFragment { + + public static DialogFragment newInstance( + String displayNumber, + OnConfirmListener positiveListener, + @Nullable DialogInterface.OnDismissListener dismissListener) { + DialogFragmentForBlockingNumberAndReportingAsSpam fragment = + new DialogFragmentForBlockingNumberAndReportingAsSpam(); + fragment.displayNumber = displayNumber; + fragment.positiveListener = positiveListener; + fragment.dismissListener = dismissListener; + return fragment; + } + + @Override + public Dialog onCreateDialog(Bundle savedInstanceState) { + super.onCreateDialog(savedInstanceState); + // Return the newly created dialog + AlertDialog.Builder alertDialogBuilder = createDialogBuilder(getActivity(), this); + Dialog dialog = + alertDialogBuilder + .setTitle(getString(R.string.block_number_confirmation_title, displayNumber)) + .setMessage(getString(R.string.block_report_number_alert_details)) + .setPositiveButton( + R.string.block_number_ok, createGenericOnClickListener(this, positiveListener)) + .create(); + dialog.setCanceledOnTouchOutside(true); + return dialog; + } + } + + /** + * Dialog for unblocking a number and marking it as not spam. + * + *

This dialog is used in the old call log, where unblocking a number will also marking it as + * not spam. + */ + public static class DialogFragmentForUnblockingNumberAndReportingAsNotSpam + extends CommonDialogsFragment { /** Whether or not the number is spam. */ private boolean isSpam; @@ -223,7 +278,8 @@ public final class BlockReportSpamDialogs { boolean isSpam, OnConfirmListener positiveListener, @Nullable DialogInterface.OnDismissListener dismissListener) { - UnblockDialogFragment fragment = new UnblockDialogFragment(); + DialogFragmentForUnblockingNumberAndReportingAsNotSpam fragment = + new DialogFragmentForUnblockingNumberAndReportingAsNotSpam(); fragment.displayNumber = displayNumber; fragment.isSpam = isSpam; fragment.positiveListener = positiveListener; @@ -255,13 +311,13 @@ public final class BlockReportSpamDialogs { } /** Dialog for reporting a number as not spam. */ - public static class ReportNotSpamDialogFragment extends CommonDialogsFragment { + public static class DialogFragmentForReportingNotSpam extends CommonDialogsFragment { public static DialogFragment newInstance( String displayNumber, OnConfirmListener positiveListener, @Nullable DialogInterface.OnDismissListener dismissListener) { - ReportNotSpamDialogFragment fragment = new ReportNotSpamDialogFragment(); + DialogFragmentForReportingNotSpam fragment = new DialogFragmentForReportingNotSpam(); fragment.displayNumber = displayNumber; fragment.positiveListener = positiveListener; fragment.dismissListener = dismissListener; diff --git a/java/com/android/dialer/blockreportspam/ShowBlockReportSpamDialogNotifier.java b/java/com/android/dialer/blockreportspam/ShowBlockReportSpamDialogNotifier.java index 58e1988fb..2fccdd115 100644 --- a/java/com/android/dialer/blockreportspam/ShowBlockReportSpamDialogNotifier.java +++ b/java/com/android/dialer/blockreportspam/ShowBlockReportSpamDialogNotifier.java @@ -47,6 +47,19 @@ public final class ShowBlockReportSpamDialogNotifier { LocalBroadcastManager.getInstance(context).sendBroadcast(intent); } + /** Notifies that a dialog for blocking a number should be shown. */ + public static void notifyShowDialogToBlockNumber( + Context context, BlockReportSpamDialogInfo blockReportSpamDialogInfo) { + LogUtil.enterBlock("ShowBlockReportSpamDialogNotifier.notifyShowDialogToBlockNumber"); + + Intent intent = new Intent(); + intent.setAction(ShowBlockReportSpamDialogReceiver.ACTION_SHOW_DIALOG_TO_BLOCK_NUMBER); + ProtoParsers.put( + intent, ShowBlockReportSpamDialogReceiver.EXTRA_DIALOG_INFO, blockReportSpamDialogInfo); + + LocalBroadcastManager.getInstance(context).sendBroadcast(intent); + } + /** Notifies that a dialog for reporting a number as not spam should be shown. */ public static void notifyShowDialogToReportNotSpam( Context context, BlockReportSpamDialogInfo blockReportSpamDialogInfo) { diff --git a/java/com/android/dialer/blockreportspam/ShowBlockReportSpamDialogReceiver.java b/java/com/android/dialer/blockreportspam/ShowBlockReportSpamDialogReceiver.java index f24bb1c06..6b8f81908 100644 --- a/java/com/android/dialer/blockreportspam/ShowBlockReportSpamDialogReceiver.java +++ b/java/com/android/dialer/blockreportspam/ShowBlockReportSpamDialogReceiver.java @@ -21,7 +21,10 @@ import android.content.BroadcastReceiver; import android.content.Context; import android.content.Intent; import android.content.IntentFilter; -import android.widget.Toast; +import com.android.dialer.blocking.FilteredNumberAsyncQueryHandler; +import com.android.dialer.blockreportspam.BlockReportSpamDialogs.DialogFragmentForBlockingNumber; +import com.android.dialer.blockreportspam.BlockReportSpamDialogs.DialogFragmentForBlockingNumberAndOptionallyReportingAsSpam; +import com.android.dialer.blockreportspam.BlockReportSpamDialogs.DialogFragmentForReportingNotSpam; import com.android.dialer.blockreportspam.BlockReportSpamDialogs.OnConfirmListener; import com.android.dialer.blockreportspam.BlockReportSpamDialogs.OnSpamDialogClickListener; import com.android.dialer.common.Assert; @@ -31,7 +34,6 @@ import com.android.dialer.logging.Logger; import com.android.dialer.protos.ProtoParsers; import com.android.dialer.spam.Spam; import com.android.dialer.spam.SpamComponent; -import java.util.Locale; /** * A {@link BroadcastReceiver} that shows an appropriate dialog upon receiving notifications from @@ -39,6 +41,7 @@ import java.util.Locale; */ public final class ShowBlockReportSpamDialogReceiver extends BroadcastReceiver { + static final String ACTION_SHOW_DIALOG_TO_BLOCK_NUMBER = "show_dialog_to_block_number"; static final String ACTION_SHOW_DIALOG_TO_BLOCK_NUMBER_AND_OPTIONALLY_REPORT_SPAM = "show_dialog_to_block_number_and_optionally_report_spam"; static final String ACTION_SHOW_DIALOG_TO_REPORT_NOT_SPAM = "show_dialog_to_report_not_spam"; @@ -51,6 +54,7 @@ public final class ShowBlockReportSpamDialogReceiver extends BroadcastReceiver { public static IntentFilter getIntentFilter() { IntentFilter intentFilter = new IntentFilter(); intentFilter.addAction(ACTION_SHOW_DIALOG_TO_BLOCK_NUMBER_AND_OPTIONALLY_REPORT_SPAM); + intentFilter.addAction(ACTION_SHOW_DIALOG_TO_BLOCK_NUMBER); intentFilter.addAction(ACTION_SHOW_DIALOG_TO_REPORT_NOT_SPAM); return intentFilter; } @@ -66,6 +70,9 @@ public final class ShowBlockReportSpamDialogReceiver extends BroadcastReceiver { String action = intent.getAction(); switch (Assert.isNotNull(action)) { + case ACTION_SHOW_DIALOG_TO_BLOCK_NUMBER: + showDialogToBlockNumber(context, intent); + break; case ACTION_SHOW_DIALOG_TO_BLOCK_NUMBER_AND_OPTIONALLY_REPORT_SPAM: showDialogToBlockNumberAndOptionallyReportSpam(context, intent); break; @@ -87,6 +94,8 @@ public final class ShowBlockReportSpamDialogReceiver extends BroadcastReceiver { intent, EXTRA_DIALOG_INFO, BlockReportSpamDialogInfo.getDefaultInstance()); Spam spam = SpamComponent.get(context).spam(); + FilteredNumberAsyncQueryHandler filteredNumberAsyncQueryHandler = + new FilteredNumberAsyncQueryHandler(context); // Set up the positive listener for the dialog. OnSpamDialogClickListener onSpamDialogClickListener = @@ -111,19 +120,16 @@ public final class ShowBlockReportSpamDialogReceiver extends BroadcastReceiver { dialogInfo.getContactSource()); } - // TODO(a bug): Block the number. - Toast.makeText( - context, - String.format( - Locale.ENGLISH, - "TODO: " + "Block number %s.", - dialogInfo.getNormalizedNumber()), - Toast.LENGTH_SHORT) - .show(); + filteredNumberAsyncQueryHandler.blockNumber( + unused -> + Logger.get(context) + .logImpression(DialerImpression.Type.USER_ACTION_BLOCKED_NUMBER), + dialogInfo.getNormalizedNumber(), + dialogInfo.getCountryIso()); }; // Create and show the dialog. - BlockReportSpamDialogs.BlockReportSpamDialogFragment.newInstance( + DialogFragmentForBlockingNumberAndOptionallyReportingAsSpam.newInstance( dialogInfo.getNormalizedNumber(), spam.isDialogReportSpamCheckedByDefault(), onSpamDialogClickListener, @@ -131,6 +137,35 @@ public final class ShowBlockReportSpamDialogReceiver extends BroadcastReceiver { .show(fragmentManager, BlockReportSpamDialogs.BLOCK_REPORT_SPAM_DIALOG_TAG); } + private void showDialogToBlockNumber(Context context, Intent intent) { + LogUtil.enterBlock("ShowBlockReportSpamDialogReceiver.showDialogToBlockNumber"); + + Assert.checkArgument(intent.hasExtra(EXTRA_DIALOG_INFO)); + BlockReportSpamDialogInfo dialogInfo = + ProtoParsers.getTrusted( + intent, EXTRA_DIALOG_INFO, BlockReportSpamDialogInfo.getDefaultInstance()); + + FilteredNumberAsyncQueryHandler filteredNumberAsyncQueryHandler = + new FilteredNumberAsyncQueryHandler(context); + + // Set up the positive listener for the dialog. + OnConfirmListener onConfirmListener = + () -> { + LogUtil.i("ShowBlockReportSpamDialogReceiver.showDialogToBlockNumber", "block number"); + filteredNumberAsyncQueryHandler.blockNumber( + unused -> + Logger.get(context) + .logImpression(DialerImpression.Type.USER_ACTION_BLOCKED_NUMBER), + dialogInfo.getNormalizedNumber(), + dialogInfo.getCountryIso()); + }; + + // Create and show the dialog. + DialogFragmentForBlockingNumber.newInstance( + dialogInfo.getNormalizedNumber(), onConfirmListener, /* dismissListener = */ null) + .show(fragmentManager, BlockReportSpamDialogs.BLOCK_DIALOG_TAG); + } + private void showDialogToReportNotSpam(Context context, Intent intent) { LogUtil.enterBlock("ShowBlockReportSpamDialogReceiver.showDialogToReportNotSpam"); @@ -158,7 +193,7 @@ public final class ShowBlockReportSpamDialogReceiver extends BroadcastReceiver { }; // Create & show the dialog. - BlockReportSpamDialogs.ReportNotSpamDialogFragment.newInstance( + DialogFragmentForReportingNotSpam.newInstance( dialogInfo.getNormalizedNumber(), onConfirmListener, /* dismissListener = */ null) .show(fragmentManager, BlockReportSpamDialogs.NOT_SPAM_DIALOG_TAG); } diff --git a/java/com/android/dialer/blockreportspam/res/values/strings.xml b/java/com/android/dialer/blockreportspam/res/values/strings.xml index e803a162f..aface9268 100644 --- a/java/com/android/dialer/blockreportspam/res/values/strings.xml +++ b/java/com/android/dialer/blockreportspam/res/values/strings.xml @@ -18,7 +18,7 @@ - Block%1$s? + Block %1$s?