Bug 684014 — A few outstanding issues in the internationalisation

Add/Expand some translator comments in the EDS backend.

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

Updated Brazilian Portuguese translation

Updated Brazilian Portuguese translation

Updated British English translation

Updated Russian translation

Updated Malayalam file

Updated Danish translation

Added da to list of languages

Added Korean translation

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Updated Serbian translation

Updated gujarati file

Updated Lithuanian translation

Updated kn translation

Updated kn translation

Update French translation

Updated kn translation

Updated Greek translation

Updated Slovenian translation

Updated Spanish translation

Assamese translation updated

Updated Marathi Translations

Updated Galician translations

Updated Indonesian translation

Added changes in 0.7.4.1 to NEWS file.

Post release bump in configure.ac

Bump release and update the fields for the release correctly.

Updated Polish translation

Updated Portuguese translation

Post release 0.7.4 changes.
Added NEWS section
Reset release bit and bump version to 0.7.4.1 for now

Merge branch '0.7.4-release'

telepathy: Fix individual-properties unit test, always emit notify for
  full-name and urls.

Assamese translation updated

telepathy: Fix WeakRef ambiguity.

Add individual to translation files.

.stamp file not needed for internal c only library.

Bump version numbers and set release flag.

Change UNRELEASED to 0.7.4 for release.

telepathy: Handle connections disappearing during a ‘yield’ while connecting

It seems it’s possible for the TpConnection to disconnect while in an async
function call below the _notify_connection_cb(). Handle this possibility
gracefully, rather than exploding.

Note: This fix is basically a guess. The crash can’t be reproduced, and
no debug output was available from folks for the crash.

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

Updated Galician translations

Updated Czech translation

Updated Lithuanian translation

Updated Russian translation

Updated Belarusian translation.

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.