Bug 690876 — eds: fix various test failures

Note bug fixed by the last several commits

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

eds test: don't assume Individuals remain after changing a linkable property

eds test: create a new address book for each test to avoid inheriting state

This fixes a bug where the second anti-linking test would fail if it ran
after the first.

eds test: ensure basic anti-link removal works

eds test: add basic anti-linking test

eds test: remove unnecessarily-strict test for extra Individual add/remove

Ideally, we should be able to fulfill this test, but it's probably not worth
the extra code complexity.

eds test: support setting email addresses in test backend

eds test: rename _parse_addrs to _parse_im_addrs for specificity.

eds test: ensure changes to linkable properties aggregate Personas together

eds test: create a new address book for each test to avoid inheriting state

This fixes a bug where the web services linking test would fail if it ran
after the im address linking test.

eds test: add test for specific behavior of linkable properties

This splits out a subtle and complicated linkable-properties bug discovered
by the set-emails test where changing the emails (which happen to be a
linkable property in the EDS backend) causes an unnecessary Edsf.Persona
creation and removal cycle.

The set-* tests are meant to confirm that reading and writing the given
property works, so seperating this should let us simplify those tests
slightly yet focus on the secondary details we care about in a specific test
case.

HACKING: Update documentation for running single tests

build: Rewrite valgrind.mk to use GNU Make conditionals

This means we’re not overwriting rules and potentially changing their
prerequisites. Incorrectly overwriting check-* rules was causing tests to
be run twice in the tests/folks directory.

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

build: Fix cleaning of .stamp files in EDS tests directory

build: Tidy up test Makefiles a little

HACKING: document how to run a single test

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

build: Update valgrind.mk to match automake-generated ‘check’ rule

The dependencies and chained rule were wrong. Not that it seems to
make much difference.

Bug 690989 — "warning: copying delegates is discouraged"

Fix a warning introduced by commit 7088fbafafd3e75f02266aa8b093d69660311b36.
Whoops.

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

build: Add a missing bracket in configure.ac

Updated Hungarian translation

telepathy: Call continuations in idle callbacks rather than directly

The continuations will proceed to a g_simple_async_result_complete() call,
rather than a g_simple_async_result_complete_in_idle() call as I expected
(and didn’t check). This could result in the first continuation executing
a lot of code and starving the other continuations (and the original
prepare() call). By delaying the continuations to the idle queue, the
original prepare() call is guaranteed to finish first. Then the
continuations can only starve each other.

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

individual-aggregator: Fix some debug spew

documentation: Document which yielding methods are safe to call concurrently

This is a follow-up commit to ce55fa2bf2f5f8cf95532da585d835bafeeb3347.
I went through all methods in folks which yield to another async method,
and tried to work out whether the caller was safe to run multiple times
concurrently (e.g. begin a second asynchronous call to it between a previous
async call beginning and finishing).

I’ve marked all such methods as safe (or not safe) as appropriate. I
haven’t made any attempt to make the unsafe methods safe, except in one
case in backend-store.vala.

phone-details: get_normalised: keep the leading '+'

It's needed to be able to call phone numbers containing an international
prefix.

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

Updated Hebrew translation.

telepathy: Yield subsequent Logger.prepare() calls if one is pending

If one PersonaStore began to prepare its Logger, then yielded on a D-Bus
call, another PersonaStore could begin to prepare its Logger and also
try to create a static D-Bus proxy (which is only supposed to be created
once). This is because the original async call hasn’t returned and set
Logger._logger by that point.

Fix this by keeping a queue of pending prepare() calls which is signalled
by the initial prepare() call once it finishes yielding.

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

telepathy: Remove Logger.prepare() return value

It’s unused. It’s private. It’s a waste of code.

telepathy: Fix bus watching by multiple Tpf.PersonaStore instances

The bus needs to be watched by all Tpf.PersonaStores which are connected to
the Logger instance, so that they all get the ‘invalidated’ signal.

Updated Greek translation

Updated Galician translations

eds test: Use the same birthday for the test individual and expected value.

Tamil translation updated

Tamil Translation updated

Updated Spanish translation

folks.pc: remove telepathy-glib

Apps linking against just folks core libraries do not need
telepathy. Always including it as it was done before leads to a
folks.pc which is unusuable unless Telepathy is installed, which
partly defeats the purpose of --disable-telepathy-backend.

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

eds: handle removal of store without removal of personas

The assumption in the IndividualAggregator that all backends remove
their personas before removing a store did not hold for the EDS
backend when stores were removed via set_persona_stores() or
disable_store().

Fixing that in EDS is tricky, so better make the IndividualAggregator
more resilient and remove any remaining personas when the store gets
removed.

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

[l10n] Update Japanese translation

Added slovak translation

Updated Telugu Translations

Updated Telugu Translations

Updated Polish translation

Updated Malayalam translations

eds: remove urls

URLs which are represented by EContact fields cannot be removed. When
setting UrlFieldDetails, the _set_contact_urls method must remove
X-EVOLUTION-BLOG-URL, X-EVOLUTION-VIDEO-URL, FBURL and URL in addition
to X-URIS, which was already removed before setting the new set of
values.

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

eds: Fix some syntax errors in documentation comments

individual: avoid updating in get

Properties with delayed loading always went through an update check
each time the property was read. This slowed down reading
unnecessarily, because constructing the value is costly and necessary
updates are already dealt with via notifications.

Fixed by introducing a new parameter "force_update" which is true
by default (= same behavior as before) and false when used inside
get().

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

individual-aggregator: Fix a typo in a documentation comment

lsw: Add enable|disable_persona_store api to libsocialweb backend.

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

hindi translation update

Assamese translation updated

Added Bengali India to the list of languages

Added Bengali India Translation

Updated Odia Translation with FUEL implementation.

Added Odia language into the list.

Updated Odia Translation with FUEL implementation.

Updated Slovenian translation

Add bug fix to NEWS file.

eds: expose Google system groups in the public API

This will allow gnome-contacts to change it directly (to switch a contact
from My Contacts to Other and back)

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

eds: Put email_addresses_ro back into Edsf.Persona.
eds: Remove extra whitespace.

From Patrick Ohly <patrick.ohly@intel.com>
Like instant messaging handles, email addresses can be assumed to be
unique for individuals. Therefore they can be used for linking by
default. This commit enables that in the EDS backend.

With email addresses being a linkable property, delaying loading of
them no longer makes sense because they will always be loaded.

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

From Patrick Ohly <patrick.ohly@intel.com>

Reading converts from date-only to local time to UTC. Writing must do
the same conversion. Conversion from UTC to local time was missing,
therefore the date may have gotten shifted by a day (depending on the
local time zone).

This commit intentionally does not cache the local time zone (like
reading does), because this is a bit ugly and the assumption is that
writing occurs a lot less often than reading.

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

Fix build broken by zeitgeist optional bugfix.

Add fix of 686506 to NEWS

tracker: Add enable|disable_persona_store implementation to tracker backend.
core: Remove trailing whitespace in folks/backend.vala

Make zeitgeist optional again

This commit makes folks build without zeitgeist again.

Bug 685250 — make <test>.gdb deadlocks

Always set CHECK_VERBOSE when running tests under gdb so that output (from
gdb) isn’t redirected to a log file. Apparently, headless gdb is hard to
use.

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

Add bugfix and new create|remove_address_book API to NEWS.

eds: Add create_address_book and remove_address_book to Edsf.PersonaStore.
eds: Add create-remove-stores unit test.

Add Ofono backend to NEWS file.

ofono: Added new Ofono backend to read contacts from SIM cards.

telepathy: Implement set_persona_stores backend method.

eds: if set given to set_persona_stores is null, load any missing stores.

Add new api to NEWS file.

telepathy: Add enable_persona_store and disable_persona_store to telepathy backend.
telepathy: Add empty set_persona_stores to telepathy backend.

keyfile: Add enable/disable_persona_store to keyfile backend.
keyfile: Add set_persona_stores to keyfile backend.

eds: Add test for disabling and enabling persona stores.

core: Add set_persona_stores method to Backend.
eds: Implement set_persona_stores in eds backend.

core: Add disable_persona_store and enable_persona_store API to backend
eds: Implement new API in eds backend

core: Add method to create IndividualAggregator passing in a BackendStore.

Updated Bulgarian translation

Post-release version bump

Release version 0.8.0

core: Fix comparison of updated local IDs

We should be comparing a set of strings, not a set of AbstractFieldDetails.
This fixes a crash in the link-personas-via-local-ids Tracker test.

tests: Fix creation/deletion of E.Sources in EDS test backend

The E.Sources for testing were being created by calling `cp` in the wrapper
script running around the entire test suite. However, each test case was
tearing down the E.Sources by deleting them. This meant that no E.Source
existed for anything except the first test case in a given test suite.

The fix is to move the file copying inside the test suite’s setup method.

build: fix disabling telepathy backend

fixes 855803a62b3a89ddb3c6abd3fa316d80cd036baf

Signed-off-by: Marc-Antoine Perennou <Marc-Antoine@Perennou.com>

Bug 684954 — Debug message claim that Individual are created with no personas

Fix the debug message in the Individual constructor.

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

telepathy: Update behaviour of Zeitgeist properties

Counter works for last 30 days instead of all time since it can
mess up calculations easily.
Reduce query size to Zeitgeist since events are filtered in the
Persona class.

Closes: https://bugzilla.gnome.org/show_bug.cgi?id=672709 (again)

telepathy: Always notify of persona changes when disconnecting

This reverts commit af500a4bd9fe88ae9f3832a37e19a86a4273ba5d and applies
a different fix. The problem was that persona changes could get away without
being notified when disconnecting due to an account being disabled. This
is because, after storing the cache, an additional check is performed for
whether the account was enabled. If it is, the cache is loaded (and persona
changes notified). If not, the cache is not loaded, *and persona changes
were not notified*.

This situation could occur when a Telepathy account is disabled:
 1. _notify_connection_cb() is called, but the TpAccount::enabled property
    is still true.
 2. While in store_cache(), the TpAccount::enabled property goes false.
 3. The check before load_cache() would fail, so load_cache() would never
    be called.

Similarly, it could occur when a Telepathy account is removed, with
TpAccountManager:account-removed being emitted while in store_cache().

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

individual-aggregator: Clarify that is-quiescent never changes true → false

[l10n] Add Japanese translation

Updated Latvian translation

[l10n] Updated German translation

Updated Belarusian translation.

Updated Malayalam file

Updated Korean translation

[l10n]Added Catalan (Valencian) translation

[l10n] Added Catalan translation

individual: Clarify that Individuals must contain at least one Persona

Updated kn translation

Add hu to LINGUAS

Added Hungarian translation