i18n: Update British English translation

telepathy: Fix weak_unref() warnings

If a TpContact is destroyed and our weak reference handler is invoked, it
could previously not remove the TpContact's pointer from the map of weak
references held by the persona store. When the persona was later destroyed,
it would call weak_unref() on the stale pointer and cause a warning.

This patch ensures we always remove pointers from the map when their
weak notification functions are called.

i18n: Add a missing file to POTFILES.in

docs: Various documentation fixes

Bug 660235 — Consistent signal handlers to a terminal would be cool

Fix folks-inspect’s handling of SIGINT, SIGTERM and EOF. In the former case,
we want to clear the current input buffer and display a new prompt. In
the case of SIGTERM we want to exit cleanly, and in the case of EOF we want
to exit if the input buffer is empty.

The combination of readline, Unix signals and threads in this was unfun.

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

inspect: Move readline() loop into the GLib main loop

The readline() loop is more pain than it’s worth, so we now use readline’s
callback interface for command line handling, allowing us to use a single
main loop and a single thread for both the aggregator and readline.

This should make signal handling a lot easier.

Note that this bumps our Vala requirement from 0.15.1 to 0.15.2.

telepathy: Handle contact removals from groups

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

Bug 670872 — Should be less sensitive to accentuated chars

Strip accented characters from names and make them all lower case when
searching for potential matches between individuals.

This uses a modified version of Empathy's live search code to allow for
string matches regardless of case or accentuation.

The commit also fixes the potential match code to handle UTF-8 properly,
which it wasn't doing before.

This includes a test case.

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

tests: Tidy up the tracker/match-name test a little

Reduce code duplication somewhat.

Bug 671900 — Folks should not suggest linking contacts from telepathy-salut

Reject any possibility of a match between two individuals if either of them
have a trust level of NONE (which can happen if they contain link-local XMPP
personas, for example).

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

core: Make PotentialMatch._inspect_im_addresses() private

It was never meant to be public, and nobody should've been using it.
Consequently, let's pretend this isn't an API break.

telepathy: Handle TpProxy::invalidated signals in the Telepathy backend

If the TpConnection is invalidated (due to the CM crashing), we want to
reset the state of the TpfPersonaStore (but not remove it). If the
TpAccountManager is invalidated (due to it crashing), we assume that all
accounts have been invalidated, and remove all TpfPersonaStores. Same for
the TpAccount.

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

Bug 672373 — folks-import segfaults (Archlinux x86_64)

Fix a GValue boxed-type-mismatch between folks-import and ImDetails which
was causing crashes.

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

create_personas_from_contact_ids: always return all the personas

We always want to try adding the persona to Telepathy, even if it was already
existing in Folks because, for example, we look for the details of this
contact before adding it.

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

_add_persona_from_contact: always return the persona

In some case we may be interested by using the persona even if it was already
existing.

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

Updated Spanish translation

Updated Polish translation

Updated Galician translations

Signed-off-by: Fran Diéguez <fran.dieguez@mabishu.com>

Updated Slovenian translation

core: Improve quiescence timeout

Ensure that we start the timeout in the case that all backends are marked
as quiescent (meaning that they've added all their persona stores), but none
of the persona stores are.

eds: Tie favourite status to “Starred in Android” group for Google Contacts

Link is-favourite to whether a “Starred in Android” group exists on a contact
(and vice-versa) if the contact is from a Google Contacts address book.

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

telepathy: Handle null birthday strings

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

Updated Swedish translation

build: Add missing entry to NEWS

core: Don’t modify a HashSet while iterating over it

Even though we were modifying it from the Iterator (which should’ve been OK),
we got hit by https://bugzilla.gnome.org/show_bug.cgi?id=671327 which meant
that some Personas weren't getting disconnected from Individuals. This is a
likely cause of bug #670523.

This also includes a related (but irrelevant to the bug itself) tidy-up to
Persona._individual.

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

Persona: make sure to NULL-ify the individual when it's destroyed

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

folks: Don't set G_MESSAGES_DEBUG unconditionally

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

build: Add --disable-fatal-warnings configure flag

Useful for builders who aren't necessarily developers.

Added uk translation

libsocialweb: Tidy up Swf.PersonaStore.prepare() to return asynchronously

Previously, the prepare() method would return synchronously, firing off
some asynchronous methods in libsocialweb, which would later return in
callbacks and (theoretically) cause the persona store to reach quiescence.

This behaviour was not playing nicely with the expected asynchronous
behaviour of prepare(), and wasn't handling errors properly — on an error,
the code would cancel preparation, but not throw an error or remove the
persona store.

The code now throws errors and removes the persona store if preparation
fails, which should hopefully help some of the not-reaching-quiescence issues
we’ve been seeing.

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

core: Force IndividualAggregator quiescence after a 5s timeout

This prevents clients being (too) adversely affected (e.g. not being able
to do anything) by buggy Backends and PersonaStores which never reach
quiescence.

Note that this patches the symptoms, not the cause of the problem. Good as
a safety net, though.

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

Bump Tracker version requirements to match current Tracker

Factor out the tracker-sparql major number.

This changes fairly often, so this will make it easier to adjust it all
in one place.

Post-release version bump

Release version 0.6.7

Temporarily disable troublesome tests due to bgo#668415

Cut invalid overly-specific type cast

The Vala compiler now correctly warns that typeof(Foo<Bar>) is invalid,
so this stops pretending we can be that specific.

(The generated C code can't make a GValue as specific as the above Vala
code fragment suggests; historically, the compiler would let you get
away with this, likely with the false assumption that the generic type
would ever be considered again.)

Build with vala 0.15

Note fix for bgo#670196 in NEWS

Updated Czech translation

add missing config.h includes

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

Disable glib deprecated warnings

We need GValueArray for dbus-glib, and it got deprecated in GLib 2.31.x

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

Add zh_CN to LINGUAS

Add Simplified Chinese translation.

[l10n] Updated German translation

Updated Polish translation

Updated Slovenian translation

Updated Spanish translation

Only add non-empty Role or PostalAddress fields in Tracker backend

This prevents some newly-exposed test failures.

Don't assume every Tpf.Persona has a contact

Fix the nullity of test functions.

Make Individual implement PresenceDetails properly

Make Tpf.Persona properly implement the PresenceDetails properties.

Match names better for PotentialMatches

We match alias as well as full name, to e.g. match an EDS contact (full name)
with an IM contact (alias).

Also, we make exact name matches bring the priority to HIGH alone, which
is needed in the IM case since the alias is the only piece of data
we have to match on.

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

autogen.sh: Honor NOCONFIGURE=1

See http://people.gnome.org/~walters/docs/build-api.txt

Updated Galician translations

Updated Czech translation

eds: Add a dummy authentication handler

This will deny any authentication requests, which goes some way to stopping
address books remaining in the “opening” state indefinitely. However, it
(curiously) doesn’t entirely eliminate the problem.

It also looks like this is a situation where the ::opened signal can return
an error while the async open() call doesn’t. Joy.

telepathy: Fix weak notification of TpContacts

By the time our weak notification callback was called, the TpContact’s
handle had been set to 0, which mucked things up for us.

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

telepathy: Keep Tpf.PersonaStore.personas up to date for cached stores

When loading a Tpf.PersonaStore from its cache, ensure Personas are inserted
into both the set of personas _and_ the map of IIDs to personas.

This fixes Tpf.PersonaStore.personas being empty for cached stores.

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

Bug 667410 — A second aggregator instance only fetches a subset of contacts

This was happening because the initial BackendStore was hanging around across
multiple IndividualAggregator instances, keeping all the Backends,
PersonaStores and Personas alive.

The IndividualAggregator didn’t have code to deal with pre-prepared Backends
and PersonaStores, meaning it never realised the Personas existed (because
they weren’t announced via personas-changed signals), and thus never created
Individuals out of them.

This commit fixes the problem by having IndividualAggregator check for
existing Backends, PersonaStores and Personas when prepare() is called.

It also adds a test case to the folks test suite, based on the one written
by Guillaume in bgo#667410.

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

eds: Mark a few more contact fields as being handled

This should prevent a few debug messages about unhandled fields.

core: Set G_MESSAGES_DEBUG when parsing debug domains

This stops GLib from dropping all our debug output (unless the user has
remembered to set G_MESSAGES_DEBUG manually in addition to FOLKS_DEBUG).

build: Add folks-internal files to MAINTAINERCLEANFILES etc.

docs: Add NEWS for the previous two commits

build: Add lcov make targets from telepathy-glib

Add lcov.mk, copied from telepathy-glib:
http://cgit.freedesktop.org/telepathy/telepathy-glib/tree/tools/lcov.am
with the tp-glib–specific bits removed.

These targets are only useful if folks is compiled with
--enable-code-coverage.

To run the test suite and generate a report, run `make lcov-check`. To just
generate a report without running the test suite, run `make lcov-report`.

The .info file and generated report are only deleted by
`make maintainer-clean`, since the developer might want to refer to them
across other cleans.

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

Bug 657063 — Allow to pass a command to folks-inspect

Add a non-interactive mode to folks-inspect which executes the command
given on the command line synchronously, and then exits.

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

inspect: Set up locale in main()

tests: Add a quiescence test for the Telepathy backend

This just checks that an aggregator with a single Tpf.PersonaStore reaches
quiescence in a reasonable time.

tests: Add a fake-tp-backend program in the Telepathy tests

This is not an automated test, and is intended to be run manually on a D-Bus
bus specified using the DBUS_SESSION_BUS_ADDRESS environment variable,
together with a client program (for example, a test program) to be run against
it.

build: Add an --enable-code-coverage configure parameter

When building with code coverage enabled, -fprofile-arcs and -ftest-coverage
will be passed to gcc during compilation, and -lgcov will be passed during
linking.

Code coverage is disabled by default for the core of folks, since it has to
turn off valac’s ‘-g’ option to output #line directives in the generated C
code. This is necessary because lcov gets confused by execution passing
between .vala and .c files. More useful results can be obtained by just
looking at the code coverage of the .c files and ignoring the boilerplate
in them.

(More specifically, lcov reports on .vala files aren’t especially useful
because they erroneously mark many lines as being executed which actually
weren’t. This happens because one Vala line may be compiled into several
lines of C, some of which will be executed unconditionally even if the Vala
line itself isn’t executed. This will fool lcov into thinking the Vala line
was actually executed.)

Code coverage is automatically turned off when making a release.

build: Tidy up use of AM_* variables in Makefile.ams

eds: Cope with opening buggy EDS address book backends

Turns out that almost as soon as I’d pushed the previous commit, one of my
address books proved me wrong and returned successfully from the async open()
call without being open (according to is_opened()) afterwards. Bother.

This replaces the assertion that the above holds with some code which will
throw an error instead, which should be a bit more robust in the case of
buggy EDS address books.

eds: Handle BUSY errors when opening address books better

If another process is in the middle of opening an address book when we come
to try and open it, our open() call will return immediately with a BUSY
error.

In this case, we need to wait for the EClient::opened signal to know when
the open operation has finished.

Hopefully, this will fix those irritating ‘blah is busy’ errors I frequently
get when running the EDS backend test suite. Hopefully it also won’t
introduce too many other problems.

tests: Fix the EDS link-personas-diff-stores test

Due to the tightening up of checks for PersonaStores being prepared before
PersonaStore.add_persona_from_details() is called, this test was failing
because it didn’t wait for its persona stores to get prepared.

This fixes the problem by waiting for quiescence before launching into the
test code proper.

eds: Reach quiescence in Edsf.PersonaStore for address books with no contacts

It’s possible for an address book to have the “do-initial-query” capability
yet not emit any ::objects-added signals on its view, if the address book has
no contacts.

This commit fixes that by listening for the ::complete signal on the address
book view and using that to determine quiescence instead.

This should theoretically also catch cases where the address book encounters
an error part-way through initialisation, since that will be reported via
the ::complete signal.

tests: Fix printerr() handler in TestCase

It was always getting a null message due to having an extraneous extra
parameter (in its C form), meaning that stacktraces from failed assertions
in test cases were never getting printed.

eds: Nullability fixes

With the EDS annotation fixes in
https://bugzilla.gnome.org/show_bug.cgi?id=667388, this fixes all of the
nullability problems found by compiling with --enable-experimental-non-null.

As mentioned in the previous commit, we can’t use
--enable-experimental-non-null by default yet, but the fixes should work by
themselves.

core: Nullability fixes

Almost all of these are just the necessary ‘(!)’ annotations to allow the
nullability check to pass. There were few, if any, actual bugs found by the
check (which either means folks is perfect, or Vala's nullability checking
is imperfect).

This brings us down from 296 nullability errors to just below 50.

The work was done by compiling folks with valac’s
--enable-experimental-non-null flag. We’re not ready to add the flag to
VALAFLAGS permanently yet, since this commit depends on various annotation
fixes in GLib (and similarly, the next one depends on several in EDS).
However, the fixes themselves should be valid without the flag.

This depends on (at least):
 • https://bugzilla.gnome.org/show_bug.cgi?id=666700
 • https://bugzilla.gnome.org/show_bug.cgi?id=666699

Updated Slovenian translation

Updated German translation

Updated Spanish translation

Updated Spanish translation

Updated Polish translation

aggregator: Expand the ‘no primary persona store’ error messages

Try and be a little more helpful.

docs: Add NEWS for commit a99f47a196bee5a5ab258a010f7dcc173bbf7a42

eds: Ensure the default role is non-empty

If the default role is empty (e.g. because EDS returns empty strings as the
contact's title, org or role; rather than null), don't add it to the
Persona.

Helps: bgo#666540

core: Warn on passing empty values to AbstractFieldDetails subclasses

There are no situations where passing an empty value (string or object) to
an AbstractFieldDetails subclass makes sense. It just introduces potential
bugs with mishandling empty strings (etc.) in code which consumes the
AbstractFieldDetails.

This commit adds warnings to all the AbstractFieldDetails subclasses’
constructors which check for empty inputs. We can't use assert()s here
because that would be an API break.

Helps: bgo#666540

Bug 666540 — Segfault on empty e-mail addresses with potential match

Gracefully handle invalid e-mail addresses (such as empty strings or strings
not containing an ‘@’) when searching for potential matches.

Closes: bgo#666540

eds: Ignore empty values when creating AbstractFieldDetails instances

We don't want to be passing around (e.g.) empty strings as e-mail addresses,
or we'll cause bugs like bgo#666540.

This modifies the EDS backend to check E.VCardAttribute.get_value() is not
null or the empty string whenever it's called, and skip the attribute as
appropriate.

Helps: bgo#666540

Bug 666528 — Can't convert from type 'GFile' to 'gchararray'

Add pretty-printing support for a couple of new properties to folks-inspect.

Closes: bgo#666528

inspect: Fix use of a deprecated NoteFieldDetails property

Added Brazilian Portuguese translation

Add nb

Added Norwegian bokmål translation

Add missing #include <config.h> to make translations work

Bug 666310 — Crash in Tracker backend by unsetting an entry in a read-only…

Closes: bgo#666310

Post-release version bump.

Release version 0.6.6

Bug 665376 — Add API to get a TpfPersona from a TpContact

Add static functions to quickly look up Tpf.PersonaStores and Tpf.Personas
from Tp.Accounts and Tp.Contacts (respectively).

Closes: bgo#665376