summaryrefslogtreecommitdiff
path: root/java
diff options
context:
space:
mode:
authormdooley <mdooley@google.com>2017-12-12 16:14:13 -0800
committerCopybara-Service <copybara-piper@google.com>2017-12-12 16:17:51 -0800
commitad2ea23c5762bee680a74d56664644b578625cd2 (patch)
treef4ee31279134b3e77880ca1675826bc46b51fa04 /java
parentd588387ecd4a26c0e5930652cf43371abe47ab74 (diff)
Cleanup. Changing voicemail_transcription_enabled to voicemail_transcription_available
Follow the convention that features allowed by server configuration flags use the word 'available' and when a feature is allowed by the user via settings then use the word 'enabled' Bug: 37340510 Test: manual and updated unit tests PiperOrigin-RevId: 178831467 Change-Id: I5691fd7db45a70908d32cdac2b049fe76e6a0680
Diffstat (limited to 'java')
-rw-r--r--java/com/android/dialer/app/voicemail/error/VoicemailTosMessageCreator.java13
-rw-r--r--java/com/android/voicemail/impl/VoicemailClientImpl.java2
-rw-r--r--java/com/android/voicemail/impl/transcribe/TranscriptionConfigProvider.java5
-rw-r--r--java/com/android/voicemail/impl/transcribe/TranscriptionService.java4
4 files changed, 13 insertions, 11 deletions
diff --git a/java/com/android/dialer/app/voicemail/error/VoicemailTosMessageCreator.java b/java/com/android/dialer/app/voicemail/error/VoicemailTosMessageCreator.java
index 2787320fa..a714b6d90 100644
--- a/java/com/android/dialer/app/voicemail/error/VoicemailTosMessageCreator.java
+++ b/java/com/android/dialer/app/voicemail/error/VoicemailTosMessageCreator.java
@@ -187,7 +187,7 @@ public class VoicemailTosMessageCreator {
return true;
}
- if (isVoicemailTranscriptionEnabled() && !isLegacyVoicemailUser()) {
+ if (isVoicemailTranscriptionAvailable() && !isLegacyVoicemailUser()) {
LogUtil.i(
"VoicemailTosMessageCreator.shouldShowTos", "showing TOS for Google transcription users");
return true;
@@ -203,7 +203,7 @@ public class VoicemailTosMessageCreator {
return false;
}
- if (isVoicemailTranscriptionEnabled()) {
+ if (isVoicemailTranscriptionAvailable()) {
LogUtil.i(
"VoicemailTosMessageCreator.shouldShowPromo",
"showing promo for Google transcription users");
@@ -227,9 +227,10 @@ public class VoicemailTosMessageCreator {
}
}
- private boolean isVoicemailTranscriptionEnabled() {
+ private boolean isVoicemailTranscriptionAvailable() {
return Build.VERSION.SDK_INT >= Build.VERSION_CODES.O
- && ConfigProviderBindings.get(context).getBoolean("voicemail_transcription_enabled", false);
+ && ConfigProviderBindings.get(context)
+ .getBoolean("voicemail_transcription_available", false);
}
private void showDeclineTosDialog(final PhoneAccountHandle handle) {
@@ -407,7 +408,7 @@ public class VoicemailTosMessageCreator {
}
private CharSequence getNewUserDialerTos() {
- if (!isVoicemailTranscriptionEnabled()) {
+ if (!isVoicemailTranscriptionAvailable()) {
return "";
}
@@ -416,7 +417,7 @@ public class VoicemailTosMessageCreator {
}
private CharSequence getExistingUserDialerTos() {
- if (!isVoicemailTranscriptionEnabled()) {
+ if (!isVoicemailTranscriptionAvailable()) {
return "";
}
diff --git a/java/com/android/voicemail/impl/VoicemailClientImpl.java b/java/com/android/voicemail/impl/VoicemailClientImpl.java
index 60fc80692..75d6dfc66 100644
--- a/java/com/android/voicemail/impl/VoicemailClientImpl.java
+++ b/java/com/android/voicemail/impl/VoicemailClientImpl.java
@@ -130,7 +130,7 @@ public class VoicemailClientImpl implements VoicemailClient {
}
TranscriptionConfigProvider provider = new TranscriptionConfigProvider(context);
- if (!provider.isVoicemailTranscriptionEnabled()) {
+ if (!provider.isVoicemailTranscriptionAvailable()) {
LogUtil.i(
"VoicemailClientImpl.isVoicemailTranscriptionAvailable", "feature disabled by config");
return false;
diff --git a/java/com/android/voicemail/impl/transcribe/TranscriptionConfigProvider.java b/java/com/android/voicemail/impl/transcribe/TranscriptionConfigProvider.java
index 3d1755b64..54a1ae4d4 100644
--- a/java/com/android/voicemail/impl/transcribe/TranscriptionConfigProvider.java
+++ b/java/com/android/voicemail/impl/transcribe/TranscriptionConfigProvider.java
@@ -28,9 +28,10 @@ public class TranscriptionConfigProvider {
this.context = context;
}
- public boolean isVoicemailTranscriptionEnabled() {
+ public boolean isVoicemailTranscriptionAvailable() {
return Build.VERSION.SDK_INT >= Build.VERSION_CODES.O
- && ConfigProviderBindings.get(context).getBoolean("voicemail_transcription_enabled", false);
+ && ConfigProviderBindings.get(context)
+ .getBoolean("voicemail_transcription_available", false);
}
public String getServerAddress() {
diff --git a/java/com/android/voicemail/impl/transcribe/TranscriptionService.java b/java/com/android/voicemail/impl/transcribe/TranscriptionService.java
index a19ab6208..0f5300358 100644
--- a/java/com/android/voicemail/impl/transcribe/TranscriptionService.java
+++ b/java/com/android/voicemail/impl/transcribe/TranscriptionService.java
@@ -142,8 +142,8 @@ public class TranscriptionService extends JobService {
public boolean onStartJob(JobParameters params) {
Assert.isMainThread();
LogUtil.enterBlock("TranscriptionService.onStartJob");
- if (!getConfigProvider().isVoicemailTranscriptionEnabled()) {
- LogUtil.i("TranscriptionService.onStartJob", "transcription not enabled, exiting.");
+ if (!getConfigProvider().isVoicemailTranscriptionAvailable()) {
+ LogUtil.i("TranscriptionService.onStartJob", "transcription not available, exiting.");
return false;
} else if (TextUtils.isEmpty(getConfigProvider().getServerAddress())) {
LogUtil.i("TranscriptionService.onStartJob", "transcription server not configured, exiting.");