Commit Graph
100 Commits
Author SHA1 Message Date
Taesu Lee f2d9199bfb Fix NoSuchMethodError for getTypeInfo()
Load an icon image for audio files from app resources instead because
ContentResolver#getTypeInfo() is for API level >= 29.
And it includes size-fit ic_photo_library_light images also.

Bug: 159987488
Test: Manual

Change-Id: Ie8f766f148a30e105e598e4b7e6b1b22bccda6ae
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-06-29 15:51:07 +09:00
Taesu Lee 5538d47259 Remove advanced category if no preference am: 392850a825 am: 34ebed2bba am: 45cd97eabf
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1351522

Change-Id: Ic6105759c8b90ffcce82c12e70e962e19b50bbd5
2020-06-26 17:15:23 +00:00
Taesu Lee 45cd97eabf Remove advanced category if no preference am: 392850a825 am: 34ebed2bba
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1351522

Change-Id: I40e97e119cfefaf90e14efad2760e9e31a46a9b8
2020-06-26 17:00:47 +00:00
Taesu Lee 34ebed2bba Remove advanced category if no preference am: 392850a825
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1351522

Change-Id: Icbed01452b088198a11436aad233c281b4068067
2020-06-26 16:48:17 +00:00
Taesu Lee 392850a825 Remove advanced category if no preference
Test: In debug mode, turn off all the configs for advanced category

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: I52ea0905b9d4f03a57b4849e5b575bc5e508dd2d
2020-06-26 19:25:05 +09:00
Taesu Lee 43f98b38f0 Fix app initialization issue if no permissions am: 297a6f7047 am: ba59a36fdb am: 5c393695c6
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1345448

Change-Id: Ia252373a2a6109a881feee44a5d20be6e5aec464
2020-06-22 17:21:11 +00:00
Taesu Lee 5c393695c6 Fix app initialization issue if no permissions am: 297a6f7047 am: ba59a36fdb
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1345448

Change-Id: I2dd1a3cc46f51f2ca743674bab363c6e90f5f0ad
2020-06-22 17:06:36 +00:00
Taesu Lee ba59a36fdb Fix app initialization issue if no permissions am: 297a6f7047
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1345448

Change-Id: If592a8e5431035c8a369917f24dbdd73e28bae4d
2020-06-22 16:53:14 +00:00
Taesu Lee 615ebb9dd6 Fix SecurityException at getActiveSubscriptionInfoList() on N and O
Creating ConnectivityUtil instances after checking the minimum set of
permissions since SubscriptionManager#getActiveSubscriptionInfoList()
requires READ_PHONE_STATE and users can revoke the permission.
On P and beyond, the subscription list will be filtered instead when no
permission.

Bug: 159421050
Test: Manual

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: Iafc3c57060e1893a91e629b1b3e3ba9ad78967cb
2020-06-22 19:15:39 +09:00
Taesu Lee 297a6f7047 Fix app initialization issue if no permissions
Call onRequiredPermissionsAcquired() before calling redirect() so that
BugleApplication initialization can be done completed always even
PermissionCheckActivity#onResume() case.

Test: Manual

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: I01bb681442e0b391238bc162ce139ff40d947eaf
2020-06-22 19:14:52 +09:00
Taesu Lee 298adffd13 Remove only one Lambda Exp for JAVA 7 am: 0c7ab8c20a am: 5db150a742 am: 619f7547d3
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1343464

Change-Id: I5433d17c76decd475b357d0ebefc83b53dc29098
2020-06-19 17:30:36 +00:00
Taesu Lee 619f7547d3 Remove only one Lambda Exp for JAVA 7 am: 0c7ab8c20a am: 5db150a742
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1343464

Change-Id: I7ea06fcff9be951ae5994ca1d9fc5d7d1efe648e
2020-06-19 17:16:27 +00:00
Taesu Lee 5db150a742 Remove only one Lambda Exp for JAVA 7 am: 0c7ab8c20a
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1343464

Change-Id: If165c85253bfd9bdcc2c3939def095968796fe83
2020-06-19 17:04:20 +00:00
Taesu Lee 0c7ab8c20a Remove only one Lambda Exp for JAVA 7
Remove a Lambda Exp and replace ConcurrentHashMap#computeIfAbsent() with
containsKey() and put() so that this app can be built using JAVA 7.

Test: Build

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: I7ff8a3b5fc4673dfe05c5ecc9418d377f3ed4f2e
2020-06-19 19:30:58 +09:00
Taesu Lee bd3b711aca Fix handling format based on API level accordingly.
Use SmsMessage#createFromPdu(pdu, format) at least on M and replace
SmsMessage#FORMAT_3GPP2 with "3gpp2".
 - SmsMessage#createFromPdu(pdu, format) is added in API level 23.
 - SmsMessage#FORMAT_3GPP2 is added in API level 28.

Test: Manual

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: Icab2c6c3c86ee8b6597881bfac6066fed60db4d2
2020-06-19 18:34:20 +09:00
Taesu Lee d5a77b3938 Fix creating ConnectivityUtil to work on any platform am: dcfe928ff3 am: 86314ff8e6 am: d810c8b49e
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1343457

Change-Id: I46cbce580c764748726666b3febe3b07f4633b5a
2020-06-18 17:53:52 +00:00
Taesu Lee d810c8b49e Fix creating ConnectivityUtil to work on any platform am: dcfe928ff3 am: 86314ff8e6
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1343457

Change-Id: Icc9cbfc22b10f73726ea7230c0e463d459c1830b
2020-06-18 17:40:18 +00:00
Taesu Lee 86314ff8e6 Fix creating ConnectivityUtil to work on any platform am: dcfe928ff3
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1343457

Change-Id: Id737b6239df8b85888e7832ab6f5e157be94de1f
2020-06-18 17:27:52 +00:00
Taesu Lee dcfe928ff3 Fix creating ConnectivityUtil to work on any platform
After this CL, ConnectivityUtil for default subId is used again on pre
N because TelephonyManager has createForSubscriptionId() for a specific
subId from N.
Nevertheless, listen() uses the default subId on PhoneStateListener
instead of the subId on the manager still even N. On O and beyond,
ConnectivityUtil works correctly on multi-sim devices.

Revert "Fix missing computeIfAbsent() method in L_MR1 and M"

This reverts commit f0ccb76d8d.

Reason for revert: Manage ConnectivityUtil instances for subIds at least
                   on N instead.

Test: Manual

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: I2f08928a6798a2ce275c5c75569ad379999d274c
2020-06-18 19:24:27 +09:00
Taesu Lee f0ccb76d8d Fix missing computeIfAbsent() method in L_MR1 and M
Replace ConcurrentHashMap#computeIfAbsent() with containsKey() and put()
since computeIfAbsent() is added in API level 24.

Bug: 158846265
Test: Build

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: I3c43f0d7b1f289f0442c9359de4f099217244387
2020-06-17 19:03:26 +09:00
Taesu Lee 3aae6cf088 Remove unnecessary methods from MmsUtils am: c9e85fd21a am: b9dd11dd77 am: 742b382d83
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1339795

Change-Id: I42852de2576cff4a5582e5b05623cbf22716a9dc
2020-06-16 16:52:42 +00:00
Taesu Lee 742b382d83 Remove unnecessary methods from MmsUtils am: c9e85fd21a am: b9dd11dd77
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1339795

Change-Id: Ied8b839de8e07d04c89be7c24aaf8ea387a663d1
2020-06-16 16:45:35 +00:00
Taesu Lee b9dd11dd77 Remove unnecessary methods from MmsUtils am: c9e85fd21a
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1339795

Change-Id: I6359187593ac39b7bbd1628cab743586515f72e8
2020-06-16 16:23:53 +00:00
Taesu Lee c9e85fd21a Remove unnecessary methods from MmsUtils
Same methods in PhoneUtils are used everywhere instead.

Test: Build

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: Ia17c1f17511c53a9235af462fa0809146726e89f
2020-06-16 11:00:42 +09:00
Taesu Lee d851c2f7f0 Allow call to phone numbers or voice mail numbers only am: 8b347325fa am: d067695cd8 am: c3e125df64
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1331094

Change-Id: Ie80fb55d3080d7f7f1f4c4dcb49aef84d09e2ee8
2020-06-11 19:29:09 +00:00
Taesu Lee c3e125df64 Allow call to phone numbers or voice mail numbers only am: 8b347325fa am: d067695cd8
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1331094

Change-Id: I4310dfdbf854572427d43d87a4c31702a36a1598
2020-06-11 19:16:35 +00:00
Taesu Lee d067695cd8 Allow call to phone numbers or voice mail numbers only am: 8b347325fa
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1331094

Change-Id: Iaa1761772b4e7991e009fce48e692a99ebe09150
2020-06-11 19:05:57 +00:00
Taesu Lee 8b347325fa Allow call to phone numbers or voice mail numbers only
It allows to call to well-formatted phone numbers or carrier specific
phone numbers starting with star sign such as *20 so that user can make
a call directly in a conversation for voice mail SMSs.

Revert "Check whether dialable only"

This reverts commit dbc43316cf.

Reason for revert: a call icon is disappeared for well-formatted phone
                   number with parentheses, dash and etc.

Test: Check the call icon in conversations for phone numbers and voice
      mail numbers.

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: I1b279f8a5372b00a5a0c572f9969731f1e633389
2020-06-11 18:56:04 +09:00
Taesu Lee 8a2358c4d5 Check CONFIG_SEND_MULTIPART_SMS_AS_SEPARATE_MESSAGES always am: f3ea99181b am: 638c7a6817 am: 2f60034413
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1325631

Change-Id: I55dfb1611ea32b9318cae568a75f89b4ba77635d
2020-06-08 17:40:30 +00:00
Taesu Lee 2f60034413 Check CONFIG_SEND_MULTIPART_SMS_AS_SEPARATE_MESSAGES always am: f3ea99181b am: 638c7a6817
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1325631

Change-Id: Idd83c8097c70305e74bf86bf0b61df48e3115fd3
2020-06-08 17:25:09 +00:00
Taesu Lee 638c7a6817 Check CONFIG_SEND_MULTIPART_SMS_AS_SEPARATE_MESSAGES always am: f3ea99181b
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1325631

Change-Id: If82c2db8dc767a806360c91c55a056cddd2f06a9
2020-06-08 17:08:29 +00:00
Taesu Lee f3ea99181b Check CONFIG_SEND_MULTIPART_SMS_AS_SEPARATE_MESSAGES always
Check CONFIG_SEND_MULTIPART_SMS_AS_SEPARATE_MESSAGES directly for each
sending messages always so that the correct config can be used for each
SIM cards in multi-sim devices.

Test: Manual

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: Iff80d093f43014ff2fe97caf6929aefe43911192
2020-06-08 16:12:02 +09:00
Taesu Lee f682a11332 Fix ConcurrentModificationException am: 845aa217fc am: 93c591bdb0 am: 1fd2c4656a
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1324972

Change-Id: I5a8f884cce556db540b98523e5aba69e8b6cbae1
2020-06-05 17:57:44 +00:00
Taesu Lee 1fd2c4656a Fix ConcurrentModificationException am: 845aa217fc am: 93c591bdb0
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1324972

Change-Id: Ia4b4c5dbe19435fc34f09bafebb0641934eec389
2020-06-05 17:45:24 +00:00
Taesu Lee 93c591bdb0 Fix ConcurrentModificationException am: 845aa217fc
Original change: https://android-review.googlesource.com/c/platform/packages/apps/Messaging/+/1324972

Change-Id: Ia9e1ac9906beace9db944fcbfcf7235a6c0f29ac
2020-06-05 17:31:20 +00:00
Taesu Lee 845aa217fc Fix ConcurrentModificationException
ConcurrentModificationException could be happened during iterating if
parts are added newly.

Test: Select several images on Gallery app and attach them to a message
via Sharing

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: Idb96239ef2c05a08fc0727797b97739aa89f12a4
2020-06-05 14:46:51 +09:00
Taesu Lee 5729626c46 Parse recipients from RESPOND_VIA_MESSAGE correctly am: fcf6699742 am: 4e646c4948 am: 1efb769573
Change-Id: I3081b98fcb924bf4e92571fcc02662fdc2d74716
2020-05-28 21:14:48 +00:00
Taesu Lee 1efb769573 Parse recipients from RESPOND_VIA_MESSAGE correctly am: fcf6699742 am: 4e646c4948
Change-Id: Ibf098b5ea21f8277a142dde46793a7b9085f624c
2020-05-28 20:58:51 +00:00
Taesu Lee 4e646c4948 Parse recipients from RESPOND_VIA_MESSAGE correctly am: fcf6699742
Change-Id: Ie293c1c84489e4a1f20972bc056f6d2af67b79b4
2020-05-28 20:42:10 +00:00
Taesu Lee fcf6699742 Parse recipients from RESPOND_VIA_MESSAGE correctly
UriUtil#parseRecipientsFromSmsMmsUri() will returns a comma-separated
recipients for common use so that InsertNewMessageAction#createMessage()
can create or get a conversation correctly for RESPOND_VIA_MESSAGE.
In addition, mms/mmsto schemes are added. And it includes small cleanup
intent-filters for LaunchConversationActivity.

Test: adb shell am startservice
-a android.intent.action.RESPOND_VIA_MESSAGE -d "sms:12345678,87654321"
-e android.intent.extra.TEXT "Text"

Change-Id: I291dc765a46846982d059016d44f90873c687867
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-05-28 16:43:29 +09:00
Taesu Lee 395876eb17 Check whether dialable only am: dbc43316cf am: 45741a4489 am: f1f488257c
Change-Id: Ie4dba4cb8405e839e8927d0346d1a4c377bf0e82
2020-05-25 21:36:43 +00:00
Taesu Lee f1f488257c Check whether dialable only am: dbc43316cf am: 45741a4489
Change-Id: Ib9aa219781543d90c309df876ba588dbc38faf3d
2020-05-25 21:20:14 +00:00
Taesu Lee 45741a4489 Check whether dialable only am: dbc43316cf
Change-Id: I517a73f1dcae564e177db748361c1f508bb426a0
2020-05-25 21:05:07 +00:00
Taesu Lee ea623cfb18 Remove dead code
No need code to handle any retry case after downloading MMS.

Test: No behavior change

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: I053f85e0260f7c4cd76d52114077d220ac0b25ba
2020-05-25 18:30:12 +09:00
Taesu Lee 9a0a21e836 Disallow making a call to emergency numbers
Show an error toast instead since ACTION_CALL intent cannot be used to
call emergency numbers.

Test: Make a call to an emergency number.

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: Ia42c588848b643def068dec1d73e2e917a4b8214
2020-05-25 17:52:44 +09:00
Taesu Lee dbc43316cf Check whether dialable only
It allows to call to any sender such as *20(Voice Mail Number for
Chile Claro).

Test: Manual

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: Ifca83c5f05018de627fa38318fd4b6db20370910
2020-05-25 15:22:26 +09:00
Taesu Lee 6bef426148 Auto re-sending MMS for all the transient failures
All the transient failures should be treated as same even reserved
values for future purposes.
(OMA-MMS-ENC-V1_2, 7.2.37. X-Mms-Response-Status field)

Test: Manual

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: Ia2739baebbf156b6fadee0fee71956b044c14234
2020-05-25 14:05:31 +09:00
Taesu Lee 925e555dd6 Rescheduling pending messages after outside window failure am: 0ff700cea4 am: 3bf899c587 am: 1e144034d1
Change-Id: I573a2677ae98dbab953bb5bef67b99d95610ff41
2020-05-19 01:08:32 +00:00
Taesu Lee 1e144034d1 Rescheduling pending messages after outside window failure am: 0ff700cea4 am: 3bf899c587
Change-Id: I5604cfaa4a27fdf75cc4c092650988161b18d23c
2020-05-19 00:52:59 +00:00
Taesu Lee 3bf899c587 Rescheduling pending messages after outside window failure am: 0ff700cea4
Change-Id: I84881b6b958ea11fc42c2e8c6b21d0b29b8fef64
2020-05-19 00:39:33 +00:00
Taesu Lee 0ff700cea4 Rescheduling pending messages after outside window failure
No more rescheduling for pending messages after DownloadMmsAction is
failed caused by outside window. Any pending download messages will be
remained as the DOWNLOADING state without actual downloading operations
and sending a DEFERRED M-NotifyResp.ind even.
It will reschedule pending messages whenever sending or downloading is
failed including outside window failure case. And it also removes dup
scheduling and fix some typo.

Test: Manual

Change-Id: Ie37088347c69f25d838875830fc65ce7727fefe6
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-05-18 13:16:50 +09:00
Taesu Lee 2393df67e4 Fix NPE on MultiConversationNotificationState am: 9635b646d8 am: 2cbca58166 am: fc61c91c8a
Change-Id: Ib018f30bd8592fa61eada8ce5c81ce59b8d60de6
2020-05-08 18:25:33 +00:00
Taesu Lee fc61c91c8a Fix NPE on MultiConversationNotificationState am: 9635b646d8 am: 2cbca58166
Change-Id: I7a06e4b055021fce0a34bb1d03f7ab629c46c832
2020-05-08 18:07:30 +00:00
Taesu Lee 2cbca58166 Fix NPE on MultiConversationNotificationState am: 9635b646d8
Change-Id: If8596050a9463200ea5244db1051a6ce4082e047
2020-05-08 17:55:19 +00:00
Taesu Lee 9635b646d8 Fix NPE on MultiConversationNotificationState
Use the most recent notification's state to print out log because
mContent is null in case of a notification for multiple messages in at
least 2 different conversations.
And no need sanitizePII() since this log is only for VERBOSE already.

Test: 1) adb shell setprop log.tag.MessagingAppNotif VERBOSE
      2) Receive SMS from different senders

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: Ie300620f0014577ca65b9a7e3a4c397a39717992
2020-05-08 19:01:59 +09:00
Taesu Lee 3466711399 Hide APN prefs if platform APIs are used am: 95ceb12cdb am: 78df54ed76 am: 9d2baa4eba
Change-Id: Ic5f1b270b6d667bd281cc9f2db44d3de081cb8e8
2020-05-05 02:55:28 +00:00
Taesu Lee d982d11e82 Use ic_add_white for consistency instead am: 116073d016 am: ef2f7ec468 am: 07c87d9e1b
Change-Id: I7896b05e32fc5e2c554e4c56358e4367acec3437
2020-05-05 02:55:22 +00:00
Taesu Lee 797aa42f35 Fix to show APN prefs correctly am: 14853cb490 am: 9af5c7183b am: e6c7e23828
Change-Id: I59e8fe10a0e419d6ddb008edf1533d43164714ba
2020-05-05 02:55:19 +00:00
Taesu Lee 9d2baa4eba Hide APN prefs if platform APIs are used am: 95ceb12cdb am: 78df54ed76
Change-Id: I8e2d4c95480dce4b8d7a38a7f0c887074844d342
2020-05-05 02:32:49 +00:00
Taesu Lee 07c87d9e1b Use ic_add_white for consistency instead am: 116073d016 am: ef2f7ec468
Change-Id: I64a6bd67b9b2efde2c0dbdb2b90eb35c07073404
2020-05-05 02:32:44 +00:00
Taesu Lee e6c7e23828 Fix to show APN prefs correctly am: 14853cb490 am: 9af5c7183b
Change-Id: I9eb3ce74f6381cdd289527d36e2ea02bfb47d345
2020-05-05 02:32:42 +00:00
Taesu Lee 78df54ed76 Hide APN prefs if platform APIs are used am: 95ceb12cdb
Change-Id: Ibd5a687a22678136634a80fb86dde41f94c76e2d
2020-05-05 02:14:59 +00:00
Taesu Lee ef2f7ec468 Use ic_add_white for consistency instead am: 116073d016
Change-Id: Ied38bf821dc213af56717f8563129245714c0cc8
2020-05-05 02:14:53 +00:00
Taesu Lee 9af5c7183b Fix to show APN prefs correctly am: 14853cb490
Change-Id: Ife9a0aa4272d91766b146d241fec43184e59862f
2020-05-05 02:14:51 +00:00
Taesu Lee 95ceb12cdb Hide APN prefs if platform APIs are used
The APN prefs is available if legacy APIs are used with local APN db
only.

Test: Manual

Change-Id: I5883dcb2c406acc601787edd02e0d3272732e6d7
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-05-04 19:30:12 +09:00
Taesu Lee 116073d016 Use ic_add_white for consistency instead
Test: Check "New APN" icon in ApnSettingsActivity.

Change-Id: I0fc29384e0714a780842652178e3e88ff311456a
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-05-04 18:48:30 +09:00
Taesu Lee 14853cb490 Fix to show APN prefs correctly
Test: Settings -> Advanced -> Access Point Names

Change-Id: Ia3ad443ac056a877abd6cd258c2219d8ee6ff181
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-05-04 18:48:30 +09:00
Taesu Lee 032518bd77 Fix NPE am: 1a0f1591de am: 23197e5729 am: 1292ae05a2
Change-Id: Ifdb16efdab9954f59ef972506fab0646ead68ab8
2020-04-28 16:23:19 +00:00
Taesu Lee 1292ae05a2 Fix NPE am: 1a0f1591de am: 23197e5729
Change-Id: Ifd170a74c57a08120b485fa8bd970c4ed3b96928
2020-04-28 16:11:47 +00:00
Taesu Lee 23197e5729 Fix NPE am: 1a0f1591de
Change-Id: Iff2bc970a78fc37f2b7721dbd651983dcf692b52
2020-04-28 15:57:27 +00:00
Taesu Lee 1a0f1591de Fix NPE
Check whether SMS delivery report preference is available first because
it could be removed if carriers don't support it.

Test: Manual

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: I9e0049dd61713f7a9638fb19a37aaa261f35d768
2020-04-28 17:28:42 +09:00
Taesu Lee 948781c1ca Handle delivered messages as completed messages am: 7ccb748e61 am: cbcb1628e5 am: 3548c16d15
Change-Id: Iee9c4c2d5f59e755c319e14e19ff57a7f0312dd1
2020-04-27 18:46:02 +00:00
Taesu Lee 3548c16d15 Handle delivered messages as completed messages am: 7ccb748e61 am: cbcb1628e5
Change-Id: I81749b4559f16731a009772cb52bc83e63056d80
2020-04-27 18:33:10 +00:00
Taesu Lee cbcb1628e5 Handle delivered messages as completed messages am: 7ccb748e61
Change-Id: I202ce5f8a2a72f763a56c98708045c10550a8d73
2020-04-27 18:21:48 +00:00
Taesu Lee 7ccb748e61 Handle delivered messages as completed messages
Delivered messages could be forwarded and sent time can be shown on
MessageDetailsDialog.

Test: Manual

Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
Change-Id: I6e24bdfe6a037ab588914327dbd8abf3f30512ea
2020-04-24 10:22:39 +09:00
Taesu Lee 2a8c11cecc Dismiss SnackBar when detached am: aa23161c03 am: 5c67dbb0d2 am: 57e6f108c7
Change-Id: I3ce6cf06f3bff5ef07bd49df8a16ed059b60315e
2020-04-02 21:59:09 +00:00
Taesu Lee 2ead48c6da Dismiss SnackBar when detached am: aa23161c03 am: 5c67dbb0d2
Change-Id: I92ebff9808c8df5625a3e542d0cae6e28a9a9c80
2020-04-02 21:37:41 +00:00
Taesu Lee 57e6f108c7 Dismiss SnackBar when detached am: aa23161c03 am: 5c67dbb0d2
Change-Id: Ib551d5788a2294374b892e8ad88448ba420b8aa7
2020-04-02 21:36:48 +00:00
Taesu Lee 5c67dbb0d2 Dismiss SnackBar when detached am: aa23161c03
Change-Id: Ib76ae11b7456549bfcb64710486da59bd56c2b34
2020-04-02 21:17:52 +00:00
Taesu Lee aa23161c03 Dismiss SnackBar when detached
SnackBarManager state is not cleared when SnackBar is detached from a
window without dismiss(). After then it is remained as dismissing state
forever.

Test: 1) Long-press a conversation and click archive action.
      2) Press back key before the snackbar is dismissed.
      3) Repeat 1) and check the snackbar is shown or not.

Change-Id: Icf2cdcd30aea42685db68e5ac36436a34c9c94ae
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-04-01 09:58:13 +09:00
Taesu Lee b41ce08cab Fix assert fail during onSaveInstanceState()
Selected attachment from GalleryGridView will be destroyed when same
attachment is already existed in DraftMessageData. It will make assert
fail during onSaveInstanceState() because mDestroyed in MessagePartData
is true already.
It destroys existing attachment and replaces with new attachment instead
so that selected attachments could be maintained correctly.

Test: 1) Select one image in GalleryMediaChooser.
      2) Reopen the chooser and long-press same image again to enter to
      the multi-selection mode.
      3) Long-press the attachment on draft message to open
      AttachmentChooser.
      4) Check Assert fail.

Change-Id: I029d778779260d0097ce30bf5338fceba05e1f77
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-03-31 18:34:54 +09:00
Taesu Lee 1412b40ec6 Fix WindowLeaked issue on AttachmentPreview
Stop attaching animation before removal attachment views and hiding the
AttachmentPreview.

Test: Long-press to attach a media from GalleryGridView and uncheck the
      media to detach immediately.

Change-Id: Ia1014609c056c9f35e428f8968b58dc2c1b704f9
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-03-06 10:55:44 +00:00
Taesu Lee e5c3d281ad Fix undesirable ripple effect on GalleryGridView
Set focusableInTouchMode = ture for workaround.

Test: Long-press a media and release it on GalleryGridView, check
      whether there is any undesirable ripple effect or not.

Change-Id: I076c47338d11b396210648a68ae393761d334c03
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-03-06 19:18:05 +09:00
Taesu Lee 45d1e1d044 New interface to pick a contact as attachment
New ContactMediaChooser allows the user to pick a contact from contacts
apps as attachment.

Test: Manual

Change-Id: Idb00cd8a3164a3cac859947affd5cbcc7e41b8c2
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-03-05 18:57:22 +09:00
Taesu Lee 4e99cc0a02 ProcessPendingMessagesAction works based on subscriptions
PorcessPendingMessagesAction queues one message for sending/downloading
associated with a subscription triggering current action at a time. And
the ConnectivityUtil also works based on subscriptions so that pending
messages can be processed regardless of other phones' state in multi-sim
case.

It includes cleanup code as well.

Test: Manual

Change-Id: Id6b4f4a0aa6a3291e7a4d8a5d3f0fbb9db3c5b86
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-28 18:16:54 +09:00
Taesu Lee a8b76eafc4 Allow acceptable media types only for sharing
Test: Manual

Change-Id: I20d5f8abf2ba7b7b620aa2a4ec1e0a43091f98c2
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-20 08:52:53 +09:00
Taesu Lee 2716c26c23 Support audio attachments through GalleryMediaChooser
GalleryMediaChooser supports audio types also. It displays simple audio
icon and file info for audio files on grid views.

Test: Manual

Change-Id: I98b605156af3c1909c0141a2b99380b5da11c1e2
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-20 08:52:44 +09:00
Taesu Lee 6c77158718 Support video attachments through GalleryMediaChooser
GalleryMediaChooser supports video types and more image types also. And
video thumbnails are overlaid with the play button image to distinguish
between images and videos.

Note:
 1. EXTRA_ALLOW_MULTIPLE is not specified for ACTION_GET_CONTENT.
 2. Files, methods and variables' name are still including "image".
 3. Content descriptions are not updated.

Test: Manual

Change-Id: I961928f150e4ae8ee80a1fba2f20c37fb5426669
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-19 18:36:32 +09:00
Taesu Lee a531636ae3 Do not show the fail icon for a group conversation
Test: Check an widget

Change-Id: Ie1ea35e0bd36891abf8b476b5409a99fa439a1b6
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-18 15:53:06 +09:00
Taesu Lee 519ae7e902 Show attachments size during composing MMS
It shows attachments size during composing MMS.
Note: Images could be resized before sending.

Test: Manual

Change-Id: I63eafe0d57259a7742ab839c4d96d595675e70e9
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-18 15:53:06 +09:00
Taesu Lee 013f61ff4b Actual attachments size is the minimum size except images am: c00d290df7 am: f731097e63 am: 6113906c54
Change-Id: I4097270e0b2ea482f3b222abc4019e7d9831e402
2020-02-17 23:47:18 +00:00
Taesu Lee 6113906c54 Actual attachments size is the minimum size except images am: c00d290df7 am: f731097e63
Change-Id: Ieedb788409e8ffb3178628cbc815891fc0ca1d29
2020-02-17 23:23:35 +00:00
Taesu Lee f731097e63 Actual attachments size is the minimum size except images am: c00d290df7
Change-Id: I482808786b8a16d529835c4afaf6bc37fc88012a
2020-02-17 23:09:30 +00:00
Taesu Lee c00d290df7 Actual attachments size is the minimum size except images
Only images can be compressed before sending. Actual size got from a
content uri is used for computing the minimum size for others.

Test: Manual

Change-Id: Id8d6ec24f80fe0eae70b6b1686ca7b199c70e313
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-17 12:08:45 +09:00
Taesu Lee 050bc461fc Fix android.os.strictmode.LeakedClosableViolation
Test: Check log.

Change-Id: I488fc6c4288bc3843a2be35a8e2988738441263e
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-12 09:54:01 +09:00
Taesu Lee 711bed5c23 Register implicit broadcasts at runtime for updating widgets
It resolves Broadcast Limitations issue if target API level is changed
as 26 or higher later.
It includes fixing code style also.

Test: Check messaging widgets' changes.

Change-Id: I6e08027b1d6a5a19cfd17f8ec3e9a895dbb3c44a
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-11 15:12:01 +09:00
Taesu Lee 1de47c7705 Fix to show PersonItemView for TYPE_CHOOSER_GRID correctly
Use match_parent instead.

Test: Check grid views for a vcard on AttachmentChooserFragment.

Change-Id: I60acc77d9c0cd9608187d86f71fc4d2c7e78afb7
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-11 15:03:12 +09:00
Taesu Lee 5e81c84ee9 Fix covering issue for video thumbnails on widget view
Use ic_video_play_light.png having background transparent to show the
video thumbnails correctly.

Test: Check a widget for conversations including a video attachment.

Change-Id: I1333a3be41e53ce7e8ad1a64849e726c73cda153
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-11 15:02:30 +09:00
Taesu Lee f3ae4c19fb Fix wrong margin on widget conversation view
Change layout_marginTop as -2dp to enclose message_list between top_line
and bottom_line.

Test: Check a widget for conversations.

Change-Id: I097303134e8ea079de39ac18746a51e1937b92c3
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-11 15:02:12 +09:00
Taesu Lee defea9dea1 Support sharing with image/video/audio/vcard and text fully
ACTION_SEND and ACTION_SEND_MULTIPLE support image, video, audio, text
and vcard files.

Test: Manual

Change-Id: I1c08070f89877140aba9087717bb663ee2990467
Signed-off-by: Taesu Lee <taesu82.lee@samsung.com>
2020-02-07 08:55:20 +09:00