Merge branch 'emit-notification'
Reviewed-by: Philip Withnall

core: Added emit_notification parameter to _update methods.
telepathy: Added emit_notification parameter to lazy load _update methods.
eds: Added emit_notification parameter to lazy load _update methods.

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

Updated Slovenian translation

tpf-persona-store: call _emit_personas_changed() when reseting _persona_set

We need to notify when we reset the persona set.

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

Signed-off-by: Jeremy Whiting <jpwhiting@kde.org>

Updated German translation

Updated Serbian translation

Merge branch 'command-set'
Reviewed-by: Philip Withnall

inspect: Add set command with set alias capability.

Updated Slovenian translation

Updated Spanish translation

Update French translation

l10n: Added Persian translation

Updated Indonesian translation

Bug 683452 — gnome-contacts starts up with an empty address book

Edsf.Persona.is_google_personal_group and .is_favourite can both be set by
the _update_groups() method. Groups must be lazily initialised when
accessing these properties, therefore.

Based on a patch by Giovanni Campagna <scampa.giovanni@gmail.com>.

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

Bump vala required version.

Bug 681726 — empathy crashed with SIGABRT in g_assertion_message()

Handle the case where calling a change_prop() method on an Individual
doesn’t find a suitable Persona to change the property on. Previously, the
code assumed that not successfully changing the property meant an error
must’ve been set. This was obviously wrong.

This patch changes it to only try and throw an error if an error has actually
been set. If no suitable Personas exist to write a property change to, the
property change will fail with PropertyError.NOT_WRITEABLE.

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

Updated Polish translation

telepathy: Clarify nullability of Tpf.PersonaStore._conn

telepathy: Correctly maintain strong refs on TpContacts in property accessors

While performing operations on a TpContact’s properties, folks should hold
a strong ref on the TpContact (even if it doesn’t normally hold one).
This expands on commit b251bcb92a343b051e62d458b66de4a9d3011f82 to fix folks
to correctly hold a strong reference in such situations.

This also fixes a potential bug where changing a Tpf.Persona’s groups while
the persona was being served out of the cache would cause a crash (null
pointer dereference).

This adds the following new API:
 • PropertyError.UNAVAILABLE

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

telepathy: Fix a race condition in the handling of weak refs to TpContacts

Use a WeakRef to maintain the pointer to a TpContact in Tpf.Persona,
rather than g_object_weak_[un]ref() and manually nullifying the pointer.

See the comments in bug #680335 for a full explanation.

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

Added bugfix to NEWS.

Merge branch 'inspect-linking'
Reviewed-by: Philip Withnall

Bug 683319 — Crash in individual-aggregator: _personas_changed_cb

Keep a reference to the Tpf.PersonaStore’s self persona when emitting
a ‘personas changed’ notification about it, so that it doesn’t get unset
in the store’s member variable in the mean time, resulting in a null persona
being emitted in the ‘personas changed’ signal.

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

Updated British English translation

Fix build error

inspect: Make Command.run async.
inspect: Make linking folks-inspect command run not return until done.

Updated Polish translation

Bug 682719 — eds test fails to compile

Don’t use deprecated EDS API in the EDS tests. The replacement API doesn’t
require a dependency version bump.

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

core: Only emit notifications for multi-valued properties when necessary

Previously, notifications were emitted whenever a related Persona’s property
changed, even if it had no effect on the Individual’s property. This was
causing infinite loops of notifications-then-getters-being-called in some
of the EDS tests. Bad.

This is fixed by only emitting notifications on Individuals’ multi-valued
properties if the properties have actually changed. Of course, this
requires lots of copying and pasting the same code because Vala doesn’t
have a preprocessor and delegates aren’t up to the job. Sigh.

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

core: Tidy up multi-valued property update functions

Create a new _update_multi_valued_property() helper function and
corresponding delegates, to mirror the existing
_update_single_valued_property() helper.

This new function is now used for most multi-valued property updaters in
Individual. It’s not used for groups, because they have weird signalling.

telepathy: Support lazy initialisation of properties

See commit 303547fec56e416f57f73643e2afb7bb4e4a8a7f. This adds support for
lazy initialisation of multi-valued properties to the Telepathy backend.

eds: Support lazy initialisation of properties

See commit 303547fec56e416f57f73643e2afb7bb4e4a8a7f. This adds support for
lazy initialisation of multi-valued properties to the EDS backend.

core: Support lazy initialisation of properties

Creating several HashSets and HashMultiMaps for every Individual turns
out to be quite wasteful, especially when most of them will typically be
empty (as address books are generally quite sparse on properties) and never
accessed (since most clients don’t need local IDs or web service addresses).

This commit delays initialisation of various multi-valued Individual
properties to the first time they’re accessed, giving them a null value
before that time. It preserves the existing API for Individual, i.e. the
properties themselves remain non-nullable, and only the object members
backing them become nullable.

This does introduce a slight behaviour change, in that an Individual will
now emit change notifications for a non-initialised multi-valued property if
*any* of the Personas in the Individual emit a notification for that
property. This is because the Individual can’t compare the current value of
its property to the new one resulting from the change in the Persona’s
property value to determine if a change has really occurred. Therefore the
Individual makes a safe over-estimate and emits notifications which might
be false positives.

This shouldn’t be a problem: if a client is interested in the property,
they will have already queried it and caused it to be initialised.
Initialised properties have the same notification behaviour as before.
If a client isn’t interested in the property, it won’t be connected to the
property notifications anyway.

This change roughly quarters the number of GObjects being created when
opening folks-inspect with the Telepathy, key-file and EDS backends enabled
and ~115 personas in the system.

core: Expanding a profiling message slightly

eds: Batch up emissions of personas-changed while starting up

There’s no point in emitting personas-changed multiple times before the EDS
persona store has reached quiescence — it just means the individual
aggregator has to re-link more personas with each emission.

This patch batches up emissions of personas-changed from the EDS persona
store until is-quiescent is reached, then notifies about all personas in
a single emission.

eds: Remove unnecessary locking

It didn’t achieve anything.

Implemented FUEL entries to Assamese translation

docs: Add some more rules to the HACKING coding guidelines

Because rules are good.

build: Switch to using AS_IF in configure.ac rather than shell script ‘if’

This is necessary because using certain autoconf macros inside a shell
script ‘if’ can cause problems if the macros perform a one-time
initialisation. The initialisation variables will be lost if the ‘if’ branch
isn’t taken.

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

build: Add DIST_SUBDIRS in the top-level Makefile.am

Just in case DISTCHECK_CONFIGURE_FLAGS gets changed at some point to no
longer build all subdirectories, we should list them all in DIST_SUBDIRS
so they still get distributed. It would be a shame to release a folks
tarball with half of the source code missing.

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

Updated Vietnamese translation

po/vi: import from Damned Lies

Update NEWS with fix for bug 682346

Forgot to push this out with the original commit.

build: Make telepathy backend optional

This is handy for while developing for those who don't always have
bleeding edge telepathy-glib. As a side-effect, tests are also made
optional (but on by default).

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

inspect: Fix crash when Utils.output_filestream is uninitialized.

Bug 682572 — Build failures due to accessing static members

Fix some more build failures due to accessing static members using an instance
variable. Also fix an instance of calling an async function with an implicit
‘.begin()’.

Hopefully that’s the last of them.

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

Updated Polish translation

Updated Indonesian translation

Bug 664072 — Folks should only use assert*() for critical, program-terminating errors

Turn various assert()s into return[_val]_if_fail()s.

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

Updated Galician translations

Note fixes for bgo#681420 in NEWS.

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

Access static class members through class; use explicit .begin()

This covers the libsocialweb backend tests.

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

Access static class members through class, not instance.

This covers the libsocialweb backend.

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

docs: Use correct Valadoc syntax for monospaced content

The syntax is ‘``monospaced text``’ rather than ‘`monospaced text`’, which
we were consistently getting wrong. Valadoc now produces the correct markup
for monospaced text, but it doesn’t appear differently from other text at
the moment due to Valadoc not having any CSS for it. That will change soon.

Similarly, Valadoc currently produces the wrong markup for monospaced text
with the gtkdoc doclet, but that will soon be fixed.

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

Add gir and .vapi files to DISTCLEANFILES if not building a release

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

eds: Add create_source_registry to eds tests.

tracker tests fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

tracker backend fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

docs: Add replacement symbols to ‘Deprecated’ annotations

This improves the documentation (and valac error messages) a little.

Added Marathi Translation

Update Punjabi Translation

eds: Add create_source_registry to eds-backend to fix build.

Prefer earlier alphabetical uid persona when two personas from the same persona store.
Fixes bug https://bugzilla.gnome.org/show_bug.cgi?id=681346

Updated gujarati file

Updated gujarati file

Updated gujarati file

docs: Various small fixes and expansions of documentation

 • Various links fixed to point to class documentation rather than its
   constructor documentation (this is a quirk of Valadoc; if using
   “{@link ClassName}” inside a method of ClassName, the link will point to
   ClassName’s constructor because symbols are resolved relatively and the
   class’ constructor is called “ClassName”).
 • Various bits of documentation expanded (mostly trivially) to shut gtk-doc
   up about missing long descriptions.
 • Some Vala code attributes moved around so the documentation comments are
   correctly associated with the code. (This shouldn’t change the behaviour
   of the attributes themselves.)
 • Some trivial constructors added to classes in order to give them
   documentation.
 • Constructor for Utils added and deprecated immediately. Utils should
   become a nested namespace on the next API break, since it will only ever
   contain static methods, and thus doesn’t need to be instantiable (which
   folks will make it at the moment).

docs: Remove protected symbols from the documentation

Folks isn’t really extensible outside its own source tree, so why clutter
up the documentation (and its build process/logs) with protected symbols?

eds: Minor API fixes in the EDS backend

These APIs, introduced in folks 0.7.1 (so this is an unstable API break,
but that’s OK), weren’t quite right in C. The SourceRegistry constructor
shouldn’t be exposed publicly as part of folks, and the PersonaStore
constructor had a stray extra parameter.

tests fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

Fixes to build with last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

telepathy backend fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

key-file backend fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.

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

eds backend fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.

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

Added Latvian translation

LINGUAS updated and new hindi translation added

Assamese translation completed

Added Assamese to LINGUAS

configure: -Werror= make no sense for "no" and "extra"

gcc error out when -Werror=extra -Werror=no-*
are used. And it is the most sensible thing to do.

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

Post-release version bump

build: Add a missing file to MAINTAINERCLEANFILES

build: Remove -Werror from CFLAGS

Replace it with a more targetted set of -Werror=[flag] flags, copied from
libsoup: http://git.gnome.org/browse/libsoup/tree/configure.ac?id=f5902fce98ae0314f0d9ca6e544895548c94a456#n339

Updated Galician translations

eds: Remove unnecessary valadoc taglet

The [Deprecated] code attribute serves the same purpose.

docs: Add “wiki” pages to the documentation

These, aptly enough, contain information moved from the wiki on
freedesktop.org about folks concepts, debugging and a glossary.

Release version 0.7.3

build: Add a missing file to the tests Makefile

Updated Russian translation

Updated German translation

Updated gujarati file  & Added LINGUAS file for gujarati

Revert "Change libgee dependency from 0.6 to 0.8."

This reverts commit 5b5360bff7a61ef17173cbe35cc32712ca0fc329.

Revert "Port folks to libgee 0.8."

This reverts commit e069bbe59dc54eb3595d8ee5c96aea4a36cff9aa.

Bug 660236 — Paging for long text in folks-inspect

Use $PAGER to allow paging of the output of really long commands in
folks-inspect.

Based on work by Raúl Gutierrez Segales and Jeremy Whiting.

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

Remove extra calls to set this._prepare_pending to false.

Removed locks in trf-persona-store that are not needed.
Minor style fixes based on review of previous changes.

Remove additional locks around this._is_prepared.

Cut unnecessary locking in Tracker backend tests.

Closes: bgo#652637 - Don't hold locks across async calls

Cut unnecessary Swf.PersonaStore._is_prepared locking.

Helps: bgo#652637 - Don't hold locks across async calls

Cut unnecessary Kf.PersonaStore._is_prepared locking.

Helps: bgo#652637 - Don't hold locks across async calls

Cut unnecessary Tpf.PersonaStore._is_prepared locking.

Helps: bgo#652637 - Don't hold locks across async calls