commit d741a4afb795019d8b252b36289977e48f44dcee
author Edwin Kempin <ekempin@google.com> Fri Dec 08 13:57:26 2017 +0100
committer Edwin Kempin <ekempin@google.com> Mon Dec 18 17:12:18 2017 +0100
tree 25d4f17aeda42f3f500ce682eade922102dd6a88
parent 350e3e263f2991cd1bd36207d2d56ea78482df80

Add an InternalAccountUpdate class to prepare updates to accounts

Instead of doing account updates directly on the Account instance have
an InternalAccountUpdate class with a builder for preparing account
updates.

This is done for the following reasons:

1. More consistency with how group updates are done (group updates are
   prepared with an InternalGroupUpdate class).

2. It's a preparation to make updates of different account data types
   atomic (e.g. update of account properties and external IDs should
   become atomic).

3. This change will allow us to replace the Account entity class from
   ReviewDb times with an AutoValue type (similar to how groups are
   represented by the InternalGroup AutoValue type).

To achieve 2. the new InternalAccountUpdate class can be extended to
host updates for further account data like external IDs, preferences
etc. Since all account data is stored in the All-Users repository the
updates in NoteDb can be atomic across different account data types.
In particular this will simplify the account creation which first
creates the external ID and then inserts the account, but then has to
rollback the external ID creation if the account insertion fails.

In contrast to GroupsUpdate AccountsUpdate continues to require a
consumer for updates (instead of an InternalAccountUpdate instance).
This is because some callers of the AccountsUpdate class need to have
access to the loaded account to decide which updates should be done
(e.g. ReceiveCommits wants to set a full name only if a full name wasn't
set yet). This is why the consumer is invoked with a new AccountUpdate
that provides both, access to the current account (for reading) and
access to the InternalAccountUpdate builder (for updates).

Change-Id: I057361b9ca4d3a61475c77a6ff06f40ebedaec0f
Signed-off-by: Edwin Kempin <ekempin@google.com>