GitLab

Previously the dirty state is cleared by marking the voicemail as read
again. This confuses the VoicemailContentProvider. Sending a update
with no values is enough to clean the flag.

The dirty flag is used to track external changes like read flag that
should be uploaded to the server.

+ Reverts ag/1314867, the "new" column is not updatable through the
  VoicemailContentProvider

Change-Id: I17a3c50622d98397fd27dc1097c15219fcd1196c
Fixes: 30105817

Change-Id: I9ae799cb895e136b3fe348ac3267bb038828f0a4
Auto-generated-cl: translation import

The voicemail notifications are tracking new voicemails. Once the
voicemail is set as read on the server it should no longer be treated
as new.

Change-Id: I503a8f0eae3aece5388ce8586082b7613ffebd72
Fixes: 30105817

Bug: 30702393
Change-Id: Ie3c3697ce021808469d08f52d353fbdc9524ec19

The original assumption that a participant's endpoint is unique is
invalid. Instead, we now key on a Pair containing both the User and
Endpoint Uris of a conference participant.

Bug: 30777779
Change-Id: I2bc7b1a694ddbdb18284e28bbf31dacff5c0d679

This CL allows the default dialer/apps with carrier privileges/apps
with MODIFY_PHONE_STATE permission enable or disable the visual
voicemail client.

+ Fix the bug where the configuration state error was not cleared
  for subsequent activation due to the edit was not applied.

Bug: 30709089
Change-Id: Ie3e4a8b48a6862e10b7381520d07d0f879b1884b

Fix recalculateConferenceable() to not differentiate Connferenceables by
whether or not they are currently in the active/background, since it has
no bearing on whether or not a Conferenceable is mergeable with another
Conferenceable. This allows us to send less updates to the
InCallServices when swapping calls in IMS.

Bug: 30676004
Change-Id: I6190364e902941e2969cced7a66e52f00be57064

A previous bug added logic to prevent the "add call" button in dialer from
showing in some situations for video calls.  This CL adds in logic to
ensure you also can't dial (via call log, contacts, etc) another call
in this situation as well.

Bug: 30427546
Change-Id: Id731f3761304bbac233c4eef429a2275fb4a0108

Change-Id: I0586f39e9ade9a93e46bf1ceea37ee3d908cc64b
Auto-generated-cl: translation import

Send a telephony-specific connection event so the InCallService can
display a message.

Bug: 30697460
Change-Id: I9df78ca49edfd41a5081a93d64301c4671f6ec09

When the SIM is removed the VVM config is no longer valid so the event
cannot be handled.

In this CL the status will directly be disabled.

Change-Id: I7ce50734435b2378779b464da3364031efff1f1f
Fixes:30678043

Change-Id: Id4d3e0df4b64be41e96a65cedf361fbf72620757
Auto-generated-cl: translation import

Bug: 30629746
Change-Id: Id3df02c2c06e02fe48b7877aafa1c55c28cebafa

Although this isn't explicitly used in Dialer at the moment, it was
something defined and forgotten to be used.

Bug: 29906222
Change-Id: I6b722a8fb0ddd82fa1600ac81526f650b8aa5466

The VVM client only supports audio messages. Most other message types
are filtered the moment the SYNC SMS is received, but some message type
such as VVM3 DSN is indistinguishable from audio message and will pass
through.

DSN (Delivery Status Notification) is the notification when sending a
email to some address has failed. VVM3 DSN triggers a SYNC SMS that is
completely identical to normal audio messages, and can only be
identified after accessing the IMAP server.

Although DSN is not supported in the VVM client, some carrier has this
feature in their voicemail telephone interface, and will send the DSN
to the client, this must be handled or a unplayable message will be
shown with no indication of errors.

In this CL, if the client is unable to find the audio data for a
voicemail, it will be deemed a an unsupported message, and its'
transcription field will be replaced by an error message. This covers
not only DSN but also any other unplayable message.

While this solution is far from ideal, it is not expected to have a lot
of users that use the DSN feature or other unknown message types.

Change-Id: I482479b746776b563ebf5694a58a8c2cf534b0f2
Fixes: 30588635

b/30513337 NPE when connection lost
ImapConnection will try to log out when the connection closes, even
when the connection is interrupted and logging out cannot be done.
Doing so will cause a NPE.

In this CL log out will only commence if the transport is still valid.

b/30514504 Don't show "Activating" if VVM has been activated before
The activating message is shown upon every boot but most of the time
the account should be already activated, activation is just for book
keeping.

After this CL the activating message will only be shown if the account
is never activated before on the device. Error message will still be
posted if activation fails.

b/30483028 getCallCapablePhoneAccounts() returns null
This is working as intended, comments updated

b/29363739 Unable to fetch voicemail from Marshmallow
PhoneAccountHandle is stored in the database to determine which account
to fetch voicemail from. Between M and N the PhoneAccountHandle value
has changed, causing the entry in M voicemails to mismatch.

In this CL if the phone account of the voicemail cannot be found, the
client will try to search it as the M format

+ Fix NPE while updating quota when the preceding operations on the
  ImapHelper has already failed.

Change-Id: Ia17e345c772892e17a005ae8f0c6c08255a0ef1f
Fixes: 29363739
Fixes: 30513337
Fixes: 30514504

In some cases TelecomManager.getCallCapablePhoneAccounts() might return
a PhoneAccountHandle with a invalid subId on boot, which will cause
a NPE when trying to convert the subId back to a PhoneAccountHandle
in SimChangeReceiver.processSubId().

SimChangeReceiver also might be triggered with a invalid subId that
is not SubscriptionManager.INVALID_SUBSCRIPTION_ID.
SubscriptionManager.isValidSubscriptionId() should be used instead of
comparing with the constant.

In this CL the subId is validated on the above events.
PhoneAccountHandleConverter.fromSubId() will also handle invalid subId
gracefully and return null. The null return value is checked in all
usages.

Change-Id: Ie703b1a2a826a2951e8cc90fccc6badd76928bd3
Fixes: 30474294

In some cases a phone account might have already written to the status
table, but the feature is later removed through changes in
CarrierConfig. Previously the VVM client will just stop handling the
account and the status will be stuck in the table. Sometimes it would
leave a error that which will be shown to the user forever.

After this CL, if the phone account has no VVM support, the status
table entry will be reset to the disabled state, which will make the
UI ignore the account.

Bug: 30314643
Change-Id: I0b67decd62d021de90d7d0ff0f5209e7f3e624ee

On Vzw when there are multiple anonymous participants, the SIP uris
returned in the CEP are different from what was expected ; eg:
sip:anonymous1@anonymous.invalid;legid=1
sip:anonymous2@anonymous.invalid;legid=2

The CEP parsing code did not handle the ; and extra parameter.
Per RFC3261, when we parse this string:
anonymous1@anonymous.invalid;legid=1
We first separate out and ; parameters (which are guaranteed to be at
the end of the address.

Bug: 30434469
Change-Id: I2f7224f434a0b01f54d6815276a8bb365bd53ac1

Makes sure that PROPERTY_IS_DOWNGRADED_CONFERENCE and
CAPABILITY_CONFERENCE_HAS_NO_CHILDREN are not erased when
updateConnectionProperties and updateConnectionCapabilities are called.

Bug: 29806833
Change-Id: Id8e3a81845df0bc144a040c68709db111f467473

If the logged in user is not the primary user, then video will be disabled
on the phone accounts registered.  This solves the problem that video calls
fail for secondary users.

Bug: 27616192
Change-Id: Ic91854cbc7d692c7c3a8319e98fbb4526781becd

This reverts commit 1e68844e.

Change-Id: I4974fa654ed53ea1038f56d958d065c1bf1c0e30