commit f2a8491d71915c621518ac8a40bed3c2a35272a9
author Ihab Awad <ihab@google.com> Tue Jul 22 21:09:25 2014 -0700
committer Ihab Awad <ihab@google.com> Wed Jul 23 13:08:52 2014 -0700
tree be867e23835e16c1d9918702d0bb01fe2954ba72
parent 9d53f1392b051929915fcc2ea40ea7469a8f8da4

Improve wiring of Phone Accounts

Change-Id: I891224e09a1d2bf81ffa887e1d077f6f64f73022

src/com/android/telecomm/PhoneAccountRegistrar.java

diff --git a/src/com/android/telecomm/PhoneAccountRegistrar.java b/src/com/android/telecomm/PhoneAccountRegistrar.java
index 7fd000a..9c45c24 100644
--- a/src/com/android/telecomm/PhoneAccountRegistrar.java
+++ b/src/com/android/telecomm/PhoneAccountRegistrar.java
@@ -56,7 +56,32 @@
 
     public PhoneAccountHandle getDefaultOutgoingPhoneAccount() {
         State s = read();
-        return s.defaultOutgoingHandle;
+
+        if (s.defaultOutgoingHandle != null) {
+            // Return the registered outgoing default iff it still exists (we keep a sticky
+            // default to survive account deletion and re-addition)
+            for (int i = 0; i < s.accounts.size(); i++) {
+                if (s.accounts.get(i).getAccountHandle().equals(s.defaultOutgoingHandle)) {
+                    return s.defaultOutgoingHandle;
+                }
+            }
+            // At this point, there was a registered default but it has been deleted; remember
+            // it for the future, but return null from this method
+            return null;
+        } else {
+            List<PhoneAccountHandle> enabled = getEnabledPhoneAccounts();
+            switch (enabled.size()) {
+                case 0:
+                    // There are no accounts, so there can be no default
+                    return null;
+                case 1:
+                    // There is only one account, which is by definition the default
+                    return enabled.get(0);
+                default:
+                    // There are multiple accounts with no selected default
+                    return null;
+            }
+        }
     }
 
     public void setDefaultOutgoingPhoneAccount(PhoneAccountHandle accountHandle) {
@@ -88,7 +113,7 @@
 
     public List<PhoneAccountHandle> getEnabledPhoneAccounts() {
         State s = read();
-        return accountHandlesOnly(s);
+        return simSubscriptionAccountHandles(s);
     }
 
     public PhoneAccount getPhoneAccount(PhoneAccountHandle accountHandle) {
@@ -129,8 +154,6 @@
             }
         }
 
-        checkDefaultOutgoing(s);
-
         write(s);
     }
 
@@ -145,26 +168,15 @@
             }
         }
 
-        checkDefaultOutgoing(s);
-
         write(s);
     }
 
-    private void checkDefaultOutgoing(State s) {
-        // Check that, after an operation that removes accounts, the account set up as the "default
-        // outgoing" has not been deleted. If it has, then clear out the setting.
-        for (PhoneAccount m : s.accounts) {
-            if (Objects.equals(s.defaultOutgoingHandle, m.getAccountHandle())) {
-                return;
-            }
-        }
-        s.defaultOutgoingHandle = null;
-    }
-
-    private List<PhoneAccountHandle> accountHandlesOnly(State s) {
+    private List<PhoneAccountHandle> simSubscriptionAccountHandles(State s) {
         List<PhoneAccountHandle> accountHandles = new ArrayList<>();
         for (PhoneAccount m : s.accounts) {
-            accountHandles.add(m.getAccountHandle());
+            if ((m.getCapabilities() & PhoneAccount.CAPABILITY_SIM_SUBSCRIPTION) != 0) {
+                accountHandles.add(m.getAccountHandle());
+            }
         }
         return accountHandles;
     }