Post-release version bump

Release version 0.6.2.1

IndividualAggregator: copy the list of stores before looping over it

Because a Backend's list of PersonaStores can change while we are
iterating it, we do a copy before starting.

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=658580

NEWS: add new section for next release

Post-release version bump

Release version 0.6.2

Update @since tags

Updated galician translations

Updated NEWS file

tracker: update link personas test

tracker: update IM addresses updates test

tracker: update add persona tes

tracker: update set-im-addresses test

Because IM addresses are linkable property, a new Individual
is created when they are modified.

e-d-s: update set-im-addresses test

Because IM addresses are linkable property, a new Individual
is created when they are modified.

e-d-s: only emit notifications if linkable properties changed

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=658411

e-d-s: move Set comparison method into it's own class

This refactoring is so we can reuse the method with other
types.

Bug 656689 — Re-link personas on linkable properties being changed

Listen for notifications of changes to the linkable properties of every
persona in the IndividualAggregator. When we receive such a notification,
relink that persona by artificially removing them from the aggregator and
immediately re-adding them.

Like many things in the IndividualAggregator, this is ugly — but it appears
to work, and I can get my head around it.

Closes: bgo#656689

core: Fix individuals_changed_detailed mappings for unlinked individuals

It turns out that we can do a post-processing phase on the mappings to be
emitted by IndividualAggregator.individuals_changed_detailed and ensure that
individuals who've been unlinked are represented by a mapping from themselves
to the set of individuals replacing them, rather than a mapping from
themselves to null, and null to the set of individuals replacing them.

This probably isn't the most efficient way to fix the problem, but it works
and I can just about understand it.

Helps: bgo#656689, bgo#657282

core: Be more thorough when removing individuals from the link map

Since the values of personas' linkable properties could've changed since we
added them to the link map, we have to be more thorough when removing
individuals from the map. We therefore now remove them by examining all the
values in the map, and removing every mapping to the individual we're
interested in.

Helps: bgo#656689

core: Add more debug output and validation to the IndividualAggregator

Everyone loves debug output, and validating that we aren't leaving stale
entries in the link map which will later cause whoever's looking into
aggregator bugs to hide in a corner and cry.

Helps: bgo#656689

core: Clarify that Persona.linkable_properties doesn't change at runtime

Helps: bgo#656689

core: Ensure we always notify of new Individuals

In the following situation, it was possible for
IndividualAggregator.individuals_changed_detailed to not emit a notification
that an individual was added:

If two personas were added by a given store in the same emission of
PersonaStore.personas_changed, the IA would create an Individual, i1, from
the first and add a mapping (null → i1) to the change set. It would then
process the second, destroying the first individual and creating a new
Individual, i2, (correctly) containing both personas. In doing so, it would
remove the mapping (null → i1) from the change set, but would incorrectly
not add a mapping (null → i2) in its place.

This situation can be extended to others where a single new Individual is
formed from multiple new personas coming from a single emission of
PersonaStore.personas_changed.

Closes: bgo#657282 (again)

e-d-s: kill gconfd-2 after running a test

I thought we were properly cleaning up the GConf entries
generated by each test run but it turns out that sometimes
gconfd-2 exits /after/ we've supposedly nuked it's directory.
So before cleaning up GConf's files we now kill gconfd-2
to prevent it from sync-ing to disc afterwards.

Helps: https://bugzilla.gnome.org/show_bug.cgi?id=658327

Bump required Vala version.

This avoids breaking the build due to valac --fatal-warnings for our
internal code.

Update for translatable strings in group-details

Fix birthday calculation in eds tests.

It's not nice to discriminate against people born outside of UTC.

Updated Slovenian translation

Added British English translation

e-d-s: remove invalid assertion in set-roles test case

e-d-s: free resources properly between tests cases on link-persona test unit

e-d-s: eliminate another case of possible WebServiceFieldDetails with null value

e-d-s: don't instantiate WebServiceFieldDetails with null values

Helps: https://bugzilla.gnome.org/show_bug.cgi?id=658327

libsocialweb: update bug link in lsw for quiescence support

Bug 657783 — Prefer data from primary store when picking Individual values

Use a consistent high-level order over personas in an individual to choose
which one to use for the value of single-valued properties such as alias,
avatar and gender.

Closes: bgo#657783

core: Clarify nullability of a few properties

AliasDetails.alias, NameDetails.nickname and NameDetails.full_name must never
be set to null. The empty string represents an unset value for these
properties.

Updated Slovenian translation

Added sl for Slovenian translation

Added Slovenian translation

aggregator: Fix a typo in emit_individuals_changed()

aggregator: Remove a useless duplicated set

This appears to be lingering from the dark days when we used
GLib.List<Persona>.

aggregator: Factor out code to add personas to the link map

This doesn't change behaviour at all (apart from debug messages), but should
make the code a bit more maintainable.

Workaround for Tpf.PersonaStores with no stored channels support

Helps: https://bugzilla.gnome.org/show_bug.cgi?id=656184

e-d-s: add test case for settings roles

e-d-s: extend add persona test to handle Roles

e-d-s: implement support for RoleDetails

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=657969

build: tests don't use <folks/*.h> so add path to folks/

libsocialweb: make PersonaStores immediately quiescent

Because we don't know if a libsocialweb service has 0
contacts or is not well configured or is not authenticated, etc.;
we might end up forever for contacts to arrive and never
achieve quiescence. Until libsocialweb provides us with more
information about the status of service (is it configured and
authenticated? How many contacts should we receive? etc.) we
immediately declare it quiescent after preparing it.

telepathy: Potentially fix is-quiescent

See: bgo#656184

eds: Immediately make non-browseable Edsf.PersonaStores reach quiescence

For example, some LDAP servers may not be browseable (i.e. they're search
only), so should reach quiescence immediately. This prevents the
IndividualAggregator waiting for ever for them to become quiescent.

Helps: bgo#656184

[l10n] Updated German translation

Updated Spanish translation

Updated galician translations

tests: Port the remaining tests to use individuals_changed_detailed

Helps: bgo#657282

tests: Expand the aggregation test to check both individuals_changed signals

Helps: bgo#657282

Bug 657282 — Add an IndividualAggregator.individuals_changed_detailed signal

This supersedes IndividualAggregator.individuals_changed, providing more
information about the relationships between the added and removed individuals.

Closes: bgo#657282

Bug 657789 — Don't claim uneditable eds fields as writable

Support setting the writeable-properties property of Edsf.Persona and the
always-writeable-properties property of Edsf.PersonaStore from the set of
supported fields exported by the address book.

Closes: bgo#657789

core: Add GROUPS and INVALID to PersonaDetail

One is obviously necessary. The other is necessary for use as an error return
from a function in the fix for bgo#657789.

This also adds some bounds checking to PersonaStore.detail_key(), which can
now return null if the PersonaDetail wasn't recognised.

Helps: bgo#657789

e-d-s: simplify property checking in add-persona test

Updated Spanish translation

Added Spanish translation to LINGUAS

Updated Spanish translation

Fixed typo in string («paramter»)

e-d-s: add birthday to list of writeable properties

Update NEWS file

e-d-s: add test for setting birthday on an e-d-s Persona

Helps: https://bugzilla.gnome.org/show_bug.cgi?id=657971

e-d-s: extend add-persona test to include the BirthdayDetails interface

e-d-s: implement BirthdayDetails

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=657971

Documentation: state the birthday's DateTime objects are UTC

Bug 656184 — Add is-quiescent property

This allows clients to determine when the IndividualAggregator has reached
a quiescent state — it's received and linked all of the personas it should
do at startup, and from that point onwards will only relink personas in
response to (for example), persona stores dynamically being added and removed
at runtime.

Closes: bgo#656184

eds/tracker: Ensure the PersonaStores are removed on error

key-file: Yield when setting properties on new personas

This fixes the libsocialweb aggregation test.

Helps: bgo#657510

eds: Ensure that property changes are signalled before .change_*() returns

We can't rely on E.BookClientView.objects_modified being emitted before
E.BookClient.modify_contact() returns. In order to guarantee that the
relevant properties will be notified before Edsf.Persona.change_*() returns,
we therefore move all calls to modify_contact() into a new
Edsf.PersonaStore._commit_modified_property() method, and block returning
from that on receipt of a property change notification from e-d-s.

However, we also add a timeout to _commit_modified_property() (defaulting to
30 seconds), so that if we don't receive a property change notification in
that time period, we error out from the .change_*() method.

Helps: bgo#657510

core: Add GroupDetails.change_groups()

This allows the groups of an implementing class to all be changed
simultaneously, asynchronously and with proper error notification.

Closes: bgo#657510

core: Add WebServiceDetails.change_web_service_addresses()

This allows the web service addresses of an implementing class to be changed
asynchronously with proper error notification.

Helps: bgo#657510

core: Add UrlDetails.change_urls()

This allows the URLs of an implementing class to be changed asynchronously
with proper error notification.

Helps: bgo#657510

core: Add RoleDetails.change_roles()

This allows the roles of an implementing class to be changed asynchronously
with proper error notification.

Helps: bgo#657510

core: Add PostalAddressDetails.change_postal_addresses()

This allows the postal addresses of an implementing class to be changed
asynchronously with proper error notification.

Helps: bgo#657510

core: Add PhoneDetails.change_phone_numbers()

This allows the phone numbers of an implementing class to be changed
asynchronously with proper error notification.

Helps: bgo#657510

core: Add NoteDetails.change_notes()

This allows the notes of an implementing class to be changed asynchronously
with proper error notification.

Helps: bgo#657510

core: Add NameDetails.change_*() methods

This allows the names of an implementing class to be changed asynchronously
with proper error notification.

Helps: bgo#657510

core: Make NameDetails.structured_name nullable

It effectively was before; now it's official.

core: Add LocalIdDetails.change_local_ids()

This allows the local IDs of an implementing class to be changed asynchronously
with proper error notification.

Helps: bgo#657510

eds: Fix a typo in a doc comment

core: Add GenderDetails.change_gender()

This allows the gender of an implementing class to be changed asynchronously
with proper error notification.

Helps: bgo#657510

core: Add FavouriteDetails.change_is_favourite()

This allows the favouriteness of an implementing class to be changed
asynchronously with proper error notification.

Helps: bgo#657510

core: Add EmailDetails.change_email_addresses()

This allows the e-mail addresses of an implementing class to be changed
asynchronously with proper error notification.

Helps: bgo#657510

core: Add BirthdayDetails.change_birthday() and .change_calendar_event_id()

These allows the birthday (and its calendar event) of an implementing class
to be changed asynchronously with proper error notification.

Helps: bgo#657510

core: Make BirthdayDetails.birthday nullable

It effectively was before; now it's official.

core: Add AvatarDetails.change_avatar()

This allows the avatar of an implementing class to be changed asynchronously
with proper error notification.

Helps: bgo#657510

core: Add ImDetails.change_im_addresses()

This allows the IM addresses of an implementing class to be changed
asynchronously with proper error notification.

Helps: bgo#657510

core: Add AliasDetails.change_alias()

This allows the alias of an implementing class to be changed asynchronously
with proper error notification.

Helps: bgo#657510

core: Add Folks.PropertyError

Helps: bgo#657510

e-d-s: make sure we also remove GConf's configs before running tests

Commit e0b99581 tries to remove GConf's config *after* a test
execution. But if the test fails, clean up might not happen. So
this commit adds try to remove configs *before* we start.

IndividualAggregator: check for the writeable store was wrong

We were checking for matching type_id to see if a PersonaStore
was the configured writeable store, instead we should do a
pointer comparison.

build: previous commit doesn't cover the tracker disabled case

tracker: conditional compilation of GLib.SpawnError exception handling

Starting Tracker 0.10.6, the GLib.SpawnError is exception is thrown
when calling Tracker.Sparql.Connection.get (). So we need to test
for the installed version and catch or not the exception, accordingly.

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=657650

e-d-s: more coding style cleanup in the persona store test

e-d-s: remove unused variable from persona store test

e-d-s: make the persona stores test follow our coding style

interface: throw some light on LocalIdDetail's cryptic comments