Release version 0.6.3.2

Require Tracker 0.12

Closes: bgo#660051 - require tracker 0.12

Don't assume that "CONNECTED" Tp.Accounts have valid Tp.Connections

There's an in-between state where the Tp.Account is "CONNECTED" but hasn't
finished preparing its Tp.Connection (and thus, it's null).

Closes: bgo#659041 — tpf-persona-store.vala: try to connect to a NULL object

Ensure we prepare all TpAccounts before trying to use them.

Helps: bgo#659041 - tpf-persona-store.vala: try to connect to a NULL object

More-tightly scope variable in EDS backend.

Actually fall back to eds:system in case there's no default address book.

Closes: bgo#660140 - contacts_contact_find_persona_from_store: assertion `store
!= NULL' failed

[l10n] Updated German translation

Trivial NEWS fix.

Note in the NEWS that we fixed bgo#659732 before the 0.6.3.1 release.

Post-release version bump.

Release version 0.6.3.1

Make sure we source GConf files installed next to us.

Deal with lingering callbacks in the eds/set-avatar test.

These can be called after we've set this._main_loop = null,
so if we don't check it first, we'll hit warnings (which get
treated as errors).

This isn't an ideal solution, but there doesn't seem to be a
great way to avoid the handlers from firing after we no
longer care (we need something like thaw/freeze_notify(),
except for general signals, not just notify).

Serialize checks for completion in EDS test.

We were interleaving checks which resulted in inconsistent behavior.

Exit EDS test early in case multiple signal emissions are in-flight.

This fixes a race between the mainloop being nullified and a dependent
signal handler calling quit() again (no idea how the signals were
getting issued and handled after the initial quit() though).

eds: split up link-personas test and cut relative URI dependency

Though some code might be duplicated, it's healthy to split tests
into different executables to assure our clean-up code (wiping out
GConf entries, shutting down our own session Bus, etc) runs between
tests.

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

eds: rework tests so that they don't depend on relative URIs

Previously, tests were build around the assumption that each
Edsf.PersonaStore had an id == ESource.relative_uri. That
stopped being true (because you'll get swallowed by a black
hole in pwithnall's garden if you use relative URIs).

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

e-d-s: use UID instead of relative URI to track ESources

Some e-d-s backends (i.e.: the Exchange one) might not have
a relative URI so we can't rely on it as an address book
identifier. So from now on we use the ESource's UID.

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

tpf: no need to use a lambda since callback exists

tpf: don't unprepare the backend if it isn't prepared

Clean up variables in Tpf test to match Edsf test.

Check for expected-missing capabilities in PersonaStore tests.

Use PersonaStore.always_writeable_properties.

This removes references to the deprecated
PersonaStore.can_alias_personas and PersonaStore.can_group_personas

Closes: bgo#659737 - tests fail due to usage of deprecated methods

build: Fix regular expression to extract dlname

The regex writer unintentially made a character range in ".-+".
This was always wrong, but it actively fails when LANG=C.

Closes: bgo#658002 - defective typelib

Deprecate PersonaStore.can-[alias|group]-personas

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

Updated Slovenian translation

Updated Slovenian translation

Updated Slovenian translation

Update NEWS file for next release

build: Post-release version bump

build: Release version 0.6.3

url-details: s/HOMEPAGE/HOME_PAGE/

Quick, before this gets released and frozen!

eds: Fix potential false matches in _is_in_source_list()

It's possible for the relative URIs of two ESources to be equal (though it's
unlikely). Consequently, we need to compare by UID as well.

Bug 653623 — Would like a set of standard "type" parameters for UrlDetails

Add some consts for TYPE and some common values for it for UrlFieldDetails.

Closes: bgo#653623

individual: Prefer set properties over unset ones

I can't believe I didn't notice/think of this when I first wrote this code.
I think my licence to write software should be revoked.

Update British English translation

Bug 657738 — Favorite people are not always displayed

Ensure the TpLogger connection in Tpf.PersonaStore is always prepared before
we first attempt to use favourite contact data.

This should fix the situation where IM connections are already prepared before
folks is prepared (i.e. _connection_ready_cb() is called as soon as the
Tpf.PersonaStore is prepared).

Closes: bgo#657738

Updated Czech translation

Updated Polish translation

i18n: Remove untranslatable identifiers from translatable strings

Also remove some trailing newline characters which don't need translating.

i18n: Fix a few en_GB-isms in C locale strings

Bug 658161 — Consistently use “contact” in translatable strings

Use “contact” and “Telepathy contact” where appropriate instead of “persona”,
to try and reduce the amount of technical vocabulary in the translatable
strings.

Closes: bgo#658161

Bug 646847 — Rename _update () in Trf.Persona

…or just remove it entirely, since it was only called from one place.

Closes: bgo#646847

eds: Add gender as an always-writeable property

This is because it uses a custom vCard attribute, so no EDS backend actually
says it's supported. We assume that the backends will save these custom
vCard attributes safely.

Bug 658328 — Return PropertyError.NOT_WRITEABLE from Persona property setters

Only the EDS backend was affected, and it's not been changed to check whether
properties are writeable before attempting to set them.

Closes: bgo#658328

eds: Remove duplicate change checks from Edsf.Persona.change_avatar()

Helps: bgo#658328

Bug 658531 — Sometimes avatar caching doesn't work

Work around bug #659324 (in GIO) to ensure that we never close the same FD
more than once when storing avatars to the cache.

Closes: bgo#658531

eds: Re-use a GTimeZone instance for converting birthday timezones

Every time a GTimeZone is created, /etc/localtime is opened. Reduce this
overhead by sharing a single GTimeZone instance between all Edsf.Personas
for all time.

This makes our strace a little tidier.

Updated Slovenian translation

Update British English translation

Updated Czech translation

abstract-field-details: Minor documentation fix

e-d-s: test case for changing primary store

e-d-s: allow setting default address books in tests

e-d-s: notify when an address book is configured as default

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

core: add Folks.PersonaStore.user-set-default property

With this property we can tell if a Store (i.e.: an e-d-s address
book) has been marked by the user as his default store (either from
Evo or from GOA, etc).

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

IndividualAggregator: improve naming for primary store related methods

Renamed _set_primary_store () to _configure_primary_store () and refactor
the code that actually sets the primary store into _set_primary_store ().

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

Let each PersonaStore manage its trust-level

Previously, the IndividualAggregator was in charge of
downgrading and upgrading the trust-level of each
PersonaStore. From now on, we let each Store manage its
trust-level since it's the natural way to do it.

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

Bug 659128 — If a persona store goes away we don't remove its personas

Handle removal of the address books backing Edsf.PersonaStores by notifying
of the removal of all the personas in the store, then emitting
PersonaStore.removed.

Test included.

Closes: bgo#659128

individual: Prefer data from personas with writeable properties

As an addition to the fix for bgo#657783, if two personas come from stores
which have equal is-primary-store values, we now prefer the persona which
lists the given property as writeable. If both personas list (or don't list)
the property as writeable, we fall back to the given ordering function as
before.

Closes: bgo#657783

tracker: configure primary store for tests via env var

No need for GConf to set up primary store for tests, doing it
via en env var is good enough.

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

e-d-s: configure primary store for link-personas test via env var

For tests, it's much simpler to do it via an env variable than via
GConf and we avoid ourselves a dependency.

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

Updated Polish translation

Folks.Individual: don't use is-writeable in change_avatar

This is a follow-up for commit 96539acd, using PersonaStore.is_writeable
is deprecated in favour of checking Persona.writeable_properties

build: Clean the .typelib file in clean rather than maintainer-clean

It's built by `make` not the maintainer, so according to
http://www.gnu.org/software/hello/manual/automake/Clean.html it should be
cleaned by `make clean`.

Also remove it from BUILT_SOURCES, since it doesn't belong there.

Don't distribute the typelib file.

Typelib files are architecture-specific, so they shouldn't be distributed.

Closes: bgo#659095 - Don't distribute typelib file

Folks.Individual: use is-primary-store instead of is-writeable

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

IndividualAggregator: s/writeable_store/primary_store/g

Renamed _writeable_store to _primary_store to augument clarity.
Also, stop listening and setting is-writeable properties and
work with is-primary-store properties.

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

Folks.PersonaStore: add is-primary-store and deprecate is-writeable

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

core: use always-writeable instead of is-writeable

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

core: deprecate FOLKS_WRITEABLE_STORE for FOLKS_PRIMARY_STORE

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

e-d-s: always open address books (regardless of it existing or not)

This helps Gnome Contacts which needs to make sure that at least
the system address book exists.

Bug 657065 — Cache keeps around contacts from disabled accounts

Add some extra conditions on loading and storing the Tpf.PersonaStore cache,
so as to prevent loading it when the account is disabled or invalid, and to
prevent storing it unnecessarily.

This also adds some extra debug output to Tpf.PersonaStore's caching methods.

Closes: bgo#657065

Bug 658002 — defective typelib

Pass the libtool dlname for libfolks.so to g-ir-compiler rather than
libfolks.la.

Closes: bgo#658002

Updated Slovenian translation

Remove extra definition of BACKEND_EDS

e-d-s: add test for auto linking via emails and IM addresses

e-d-s: assume certain e-mail addresseses are IM IDs too

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

avatar-cache: Add some more debug output

Updated Polish translation

Bug 633781 — Allow to set avatar on individuals

Make Individual.avatar writeable and write new avatars back to the personas
from writeable persona stores in an individual.

Closes: bgo#633781

build: use AM_MAINTAINER_MODE([enable]) instead of AM_MAINTAINER_MODE

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

Bug 658730 — ngettext (Plural forms) needed…

Use ngettext() instead of gettext() for a plural string.

Closes: bgo#658730

Bug 658874 — Typo?

Fix a typo in a translatable string in the Tracker backend.

Closes: bgo#658874

Bug 658331 — Ensure read-only PersonaStores have empty writeable-properties

Ensure that Edsf.PersonaStore.always-writeable-properties is empty for
EDS address books which are read-only. Similarly, ensure that
Edsf.Persona.writeable-properties is also empty for such address books (which
follows because it passes through the value of always-writeable-properties).

No other backends need modifying for this, since no others have a concept
of “read only” except libsocialweb, which already has empty
writeable-properties properties.

Closes: bgo#658331

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

This is a follow-up for what was suggested in:

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

about dealing with updating properties and emitting notifications.

Bug 658631 — crash at empathy startup

In the case that a persona is removed and the individual containing that
persona is replaced by a different non-null individual which doesn't contain
that persona (because it's been removed), we need to set the persona's
individual pointer to null rather than the replacement individual.

This stops us tripping the assertion in the setter for Persona.individual.

Closes: bgo#658631

individual-aggregator: Add some more debug output

It turns out to be really useful to see what's being emitted by
IndividualAggregator.individuals-changed-detailed.

Added Swedish translation

Czech translation

Added Czech language

e-d-s: Make sure consecutive property changes work

Don't send any notification due to property changes until we've
done updating an entire persona, otherwise code that listens to
such notifies will see the persona in an intermediate state, whic
can cause problems. For instance, if such a callback tries to
change a property it might not recieve change notification for it.

e-d-s: Make sure we clear old attributes when setting im_addresses

The previous code called e_contact_get_attributes() which returns
a copy of the attributes, and tried to remove it with
e_vcard_remove_attribute () which matches on pointer values, so
it always failed.

Added Polish translation

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