build: Bump the Vala dependency to 0.13.3

This is due to commit c0ecad63015078231934e63ec63f02b97defb730.

e-d-s: Vala dropped support for reordering GCancellable params

A couple of e-d-s methods (get_contact, modify_contact,
get_backend_property, etc) experienced an automatic API
change in it's Vala binding due to Vala dropping support
for reordering GCancellable params.

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

core: Fix some whitespace

e-d-s: ignore the type when comparing images

e-d-s: update the Avatar Details test

Now that we've reworked avatar handling and we've
introduced an Avatar cache, the avatars associated
to a newly created e-d-s contact might not be immediately
available after the corresponding Edsf.Persona is created.
Hence we need to listen to notifications for the avatar
property.

e-d-s: Only set avatar if store_avatar () was successful

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

e-d-s: When an avatar has been removed, set it to null

Though new_avatar was null, I prefer to even more
obvious about what is going on.

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

e-d-s: Don't infer we already have an avatar in our cache by its URI

From libfolks we should discourage using local URIs to
set an EContactPhoto so even if we get inlined EContactPhoto
just do a full content comparison.

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

Don't manipulate the Avatar Cache before e-d-s acks changes

Also, lets set EContactPhotos as INLINED because it doesn't
make sense to use URIs for local files. In the future, when
we support gravatars, we'll have to handle the case of
setting ContactPhotos as URIs for those.

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

Do Tracker tests last, since they're most likely to fail uncontrollably.

Helps: bgo#655374 - Un-break Tracker avatar tests

Add utility function to compare LoadableIcon content.

Unlike the various "equal()" functions for LoadableIcon and related classes,
this function actually compares image content (which can be necessary when
testing vs. images in the avatar cache, which may have a different path than an
otherwise-identical image).

Helps: bgo#655374 - Un-break Tracker avatar tests

Update NEWS file

eds: Support writing postal addresses as vcards

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

Generalize Edsf.PersonaStore._set_contact_attributes

We need to handle non-string field details values when we implement
postal addresses.

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

Read postal addresses as vcards

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

Break out common code in edsf backend

Most types use the same code to update the vcard params, so we
can share this code.

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

inspect: Ensure to initialise out variables

This fixes a few compiler warnings.

core: Fix use of deprecated Vala API

eds: Don't copy EVCardAttributes at all

libebook does some icky things with EVCardAttributes it's passed, especially
when calling e_vcard_remove_attribute() (it removes by pointer comparison
and then unconditionally frees the passed-in attribute). Consequently, we
need to do everything we can to avoid copying EVCardAttributes unless we
want double-frees and memory leaks.

Don't use const string[] , Vala doesn't handle them correctly

Make sure we pass ENABLE_EDS to vala flags

inspect: Remove workarounds for bgo#638363

See also: bgo#656467.

Post-release version bump.

Release version 0.6.0

Update the checked files for valac requirement.

Distribute EDS test tools.

Un-break Docs.

Write support for URLs in Edsf.Persona

Closes: bgo#655609 - Implement setter for URLs

inspect: Fix listing GStrv properties

We have to work around bgo#656467 for the moment.

inspect: Update folks-inspect for the FieldDetails changes

Make the EDS backend's system address book the default writeable

Closes: bgo#648818 - Support the EDS backend being a primary

Trivial NEWS clean-up.

Remove FieldDetails (obsoleted by AbstractFieldDetails and implementors)

Support vCard-like parameters for URLs

Closes: bgo#655922 - Rebase UrlDetails.urls upon an
AbstractFieldDetails-derived class

Support vCard-like parameters for Roles

Closes: bug#655921 - Change RoleDetails.roles to support vCard-like arbitrary parameters

Support vCard-like parameters for web service addresses

Closes: bgo#653682 - Change WebServiceDetails.web_service_addresses to support
vCard-like arbitrary parameters

Rebase PhoneDetails.phone_numbers upon PhoneFieldDetails

Closes: bgo#655920 - Rebase PhoneDetails.phone_numbers upon an
AbstractFieldDetails-derived class

Allow for Individual replacement in the EDS add-persona test.

Rebase NoteDetails.notes upon NoteFieldDetails

Closes: bgo#655919 - Rebase NoteDetails.notes upon an
AbstractFieldDetails-derived class

Set the eds backend as the primary for the eds tests.

This fixes the eds/link-personas test.

Rebase EmailDetails.email_addresses upon EmailFieldDetails

Closes: bgo#655917 - Rebase EmailDetails.email_addresses upon an
AbstractFieldDetails-derived class

Run key-file tests later, since it's a low-priority backend.

Run Tracker tests last, since it's a low-priority backend.

They're also most likely to fail for reasons outside of our control.

Add utility function to compare MultiMaps of AbstractFieldDetails.

Clarify parameters for PostalAddressOwnerDetails.

Add basic equality testing for ImFieldDetails.

Helps: bug#653680 - Change ImDetails.im_addresses to support vCard-like
arbitrary parameters

Support vCard-like parameters for IM addresses

Helps: bug#653680 - Change ImDetails.im_addresses to support vCard-like
arbitrary parameters

Clean up the way the Tracker backend tracks IM addresses.

Helps: bug#653680 - Change ImDetails.im_addresses to support vCard-like
arbitrary parameters

Don't spill over 80 characters.

Support vCard-like parameters for addresses in PostalAddressDetails.

This also removes the older, less-functional, less-consistent
PostalAddress.types.

Helps: bgo#653679 - Change PostalAddressDetails.postal_addresses to
support vCard-like arbitrary parameters

Add test for parameter equality in FieldDetails.

Helps: bgo#653679 - Change PostalAddressDetails.postal_addresses to
support vCard-like arbitrary parameters

Add basic equality testing for FieldDetails.

Helps: bgo#653679 - Change PostalAddressDetails.postal_addresses to
support vCard-like arbitrary parameters

Add parameters as an optional argument to the FieldDetails constructor.

Helps: bgo#653679 - Change PostalAddressDetails.postal_addresses to
support vCard-like arbitrary parameters

Implement FieldDetails.equal()/hash() in terms of the default.

This allows explicit references to FieldDetails.equal()/hash() for
structures storing them (in case it changes from the
AbstractFieldDetails implementation in the future).

Helps: bgo#653679 - Change PostalAddressDetails.postal_addresses to
support vCard-like arbitrary parameters

Add equal() and hash() functions to AbstractFieldDetails.

Helps: bgo#653679 - Change PostalAddressDetails.postal_addresses to
support vCard-like arbitrary parameters

Centralize parameters to AbstractFieldDetails.

This cuts some lines of code from the derived classes.

Helps: bgo#653679 - Change PostalAddressDetails.postal_addresses to
support vCard-like arbitrary parameters

Rebase FieldDetails upon an abstract and generic version of itself.

This should be useful for cases where the value has a very specific type
(eg, for PostalAddressDetails.postal_addresses). We'll create subclasses of
FieldDetails for those cases.

When we've created all the appropriate derived classes, we'll remove
FieldDetails entirely.

Helps: bgo#653679 - Change PostalAddressDetails.postal_addresses to
support vCard-like arbitrary parameters

core: Expand the documentation of EmailDetails.email_addresses

build: Add VAPI dependencies for eds backend

core: Fix a typo in NoteDetails documentation

Bug 653619 — Add company details

I dislike Tracker, ontologies, Sparql and drinking games.

Closes: bgo#653619

eds: Fix a minor type problem in the add-contacts-stress-test

build: Build eds backend by default

Signed-off-by: Raul Gutierrez Segales <rgs@collabora.co.uk>

e-d-s: fix link personas test

We've changed the way we set up the writeable store in
commit 5c062183b (bug #654907) and that broke this test.

Don't set a timeout for contacts adding stress test

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

Bug 655008 — Tidy up error handling in eds backend

Return more specific error codes from Edsf.PersonaStore methods where
possible, and in all cases make sure we've got a switch statement on the
incoming error code. Closes: bgo#655008

Clarify the documentation of PersonaStore.is_writeable

Closes: bgo#652657

Bug 649296 — Folks backend files shouldn't begin with "lib"

Rename the backend library files from “libfolks-backend-${BACKEND_NAME}.so”
to “${BACKEND_NAME}.so”.

Closes: bgo#649296

eds: Minor fixes to documentation comment syntax

e-d-s: add test for Gender property

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

e-d-s: Implement read/write support for gender property

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

Configure the writeable store by type_id and store id

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

Add rules for uploading generated docs to freedesktop.org

In the future, we should copy the remaining rules from telepathy-glib's setup,
which restrict doc uploads to releases (as well as simplifying other details of
releases).

Build eds docs along with others.

Fix syntax error in valadoc for Edsf.PersonaStore.remove_persona.

Fix syntax error in valadoc for Edsf.PersonaStore.prepare.

Fix link in Edsf.Persona docs to Folks.AvatarDetails.avatar.

Build libsocialweb docs along with others.

Fix link in Swf.Persona docs to Folks.AvatarDetails.avatar.

Fix link in Trf.Persona docs to Folks.AvatarDetails.avatar.

Centralize .PHONY doc targets.

Centralize definition of doc dirs to distclean.

Bump the required version of valadoc.

Actually check for the valadoc version when building docs.

Work around valadoc parsing bug with indented comments.

Give public access to properties for which there is write support

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

Make the Edsf.Persona class follow our coding style

Besides adhering to our coding style, this also
works around a bug in Vala that allows accessing

public string foo { get; set; }

through _foo.

Works around: https://bugzilla.gnome.org/show_bug.cgi?id=655741

core: Remove unused PotentialMatch.result_to_string() method

docs: Fix up lots of minor issues in the documentation comments

core: Make PotentialMatch.jaro_dist() private

inspect: Add a linking command

individual: Fix linking when personas are moved between individuals weirdly

It's possible (likely, even) for a set of personas to move between two
individuals during linking. Previously, bad things would happen to
persona.individual if the personas weren't correctly removed from the first
individual before being added to the second (e.g. if they were added to the
second individual before being removed from the first). This fixes that.

tests: Add a basic refcounting test for linked individuals

individual: Add more debug output to Individual

aggregator: Ensure we don't remove Personas we're adding

If a two Personas with the same UID are in the added and removed sets for a
single call to personas_changed_cb() in the individual aggregator, make sure
that we remove the old instance and replace it with the new instance, instead
of just removing both instances. This covers the case where the real instance
of a (Tpf.) Persona disappears and is immediately replaced by its cached
copy.

NOTE: This needs more testing.

Bug 652643 — Add PersonaStore cache

Use ObjectCache in Tpf.PersonaStore.

Closes: bgo#652643

Add a generic caching object to the core of folks

This adds the ObjectCache API.

Helps: bgo#652643

Unset the self-contact on Tpf.PersonaStore when resetting the store

Bug 650414 — Need better APIs to handle image data

Change AvatarDetails.avatar to have type LoadableIcon. By introducing
a libfolks-wide avatar cache, propagate this change to all the backends.

This breaks the API of AvatarDetails.

Closes: bgo#650414

Add missing copyright/license notices to tests.