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

tracker: update linking test to use iid instead of uid

tracker: use iid instead of uid when linking via local ids

e-d-s: add test to link personas from different stores

This test checks what was fixed in:

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

e-d-s: make address book uri a settable property in the test Backend

This allows us to instantiate multiple Backends, each one associated
to a it's own address book, which comes in handy for tests involving
multiple e-d-s PersonaStores.

e-d-s: update link-personas test to use iid instead of contact_id

e-d-s: use this.local_ids in linkable_property_to_links ()

Otherwise, this method can have a different  behaviour depending
on the state of the current Persona depending on whether
this.local_ids was called before or not.

e-d-s: we should use the Persona's iid as local-ids

If we don't and use the contact_id as before, linking Personas
from different PersonaStores won't work.

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

e-d-s: default trust level for PersonaStores is PARTIAL

Without this, we can't link personas from different
(e-d-s) PersonaStores.

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

e-d-s: make sure we remove GConf's configs between tests

When exercising GConf more than the usual, by creating multiple
address books or such, we might leave things in an inconsistent
state that might affect the tests that come afterwards.

tracker: Fix the match-im-addresses test

Due to the Tracker tests now running with the tracker persona store as the
writeable store, Tracker personas in the tests are now linked by their
linkeable properties (as their persona store is now fully trusted due to
being the writeable store).

This caused the two personas in the match-im-addresses test to be linked
together implicitly by the aggregator, when the test case assumed they
weren't. This broke the test.

This commit fixes the test permanently by changing the way we consider
potential matches based on IM addresses. Previously, we only indicated a
potential match if the two individuals under consideration shared a common
IM address + protocol combination. This commit changes that so that two
individuals are now considered to potentially match if they share a common
IM address *only*. Protocols are no longer considered.

This allows us to change the way the personas are set up in the
match-im-addresses test so that they're no longer implicitly linked by the
aggregator.

e-d-s: fix indentation in tests backend

build: Bump the e-d-s dependency to 3.1.5

We totally missed this on previous releases and
it is now causing problems for packagers.

Bug 653777 — Add a helper function to create a writable persona

This adds IndividualAggregator.ensure_individual_property_writeable(), which
returns a persona which has the specified property writeable in the specified
individual. If no such persona exists already, a new one is created and
linked to the individual. If that all fails, an error is returned.

This allows for clients to write to properties of personas with confidence
that the updates will be written out to the backend stores. (Achieved by
calling IndividualAggregator.ensure_individual_property_writeable() first
and only writing to the property if that call succeeds.)

Closes: bgo#653777

key-file: Allow empty Kf.Personas to be added to the persona store

This is necessary for the tests for bgo#653777. It removes the error which
would previously be thrown when trying to add a persona to Kf.PersonaStore
with no properties.

Helps: bgo#653777

core: Add PersonaStore:always-writeable-properties

This complements Persona:writeable-properties, listing the properties which
are guaranteed to always be writeable on the personas in a given persona
store. This is in contrast to Persona:writeable-properties, which may list
extra properties which are only writeable on that particular persona.

This could be the case with, for example, personas representing the user,
which might have extra writeable properties to allow the user to change their
alias or avatar.

Helps: bgo#653777

Fix NEWS: there never was a 0.5.3 release.

Use the key-file backend for writes in general tests.

See commit ef36f347 for more details.

Post-release version bump

build: Fix the NEWS file

More haste, less speed…

Bug 645056 — TpLowlevel library should have only static public functions

Remove the TpLowlevel class from the TpLowlevel library and instead place
all the functions statically in the “FolksTpLowlevel” namespace.

Closes: bgo#645056

Release version 0.6.1

tracker: catch GLib.SpawnError when getting a connection

Add myself as maintainer so I can upload tarballs, etc

tracker: Remove “alias” from the writeable properties list of Trf.Persona

Trf.Persona no longer implements AliasDetails.

Bug 657332 — Add linking link-individuals

Add a `linking link-individuals` subcommand to folks-inspect. Note: this
appears to break readline command completion for the `linking` command in
certain situations (e.g. if you tab-complete the name of the command first).

I can't fix this for love or money. Pigs will fly before I manage to work
out why readline hates me so much.

Closes: bgo#657332

Don't require that zero Individuals be removed after auto-linking.

Make sure Tracker tests actually use the Tracker backend for writes.

Ignore edsf-persona-store.c for translations.

Properly handle 'url' TYPE for URLs in Tracker backend.

Correctly spell ERROR_VALAFLAGS.

Don't use ERROR_CFLAGS yet. Vala doesn't guarantee warning-free C.

Use the key-file backend for writes in tests for non-writeable backends.

We could have used the EDS backend, but this helps keep the execution
of the tests as simple as possible.

This fixes a bug in the libsocialweb aggregation test where it was
using the EDS backend from the Aggregator default but its
FOLKS_BACKEND_STORE_KEY_FILE_PATH didn't include the EDS backend, so
it had nowhere to store its linking personas.

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

IndividualAggregator: initialize primary-store property to NULL