e_async_closure_callback: Do not test 'object' parameter validity

It is unnecessary to test validity of the 'object' parameter,
especially when this callback does nothing with it. The reason
is that the secret_password_store() calls the callback with NULL
'object', which leads to a deadlock on the source registry side.

Bug #685090 - GOA EWS module doesn't save all required values

Collection's child source Enabled should match collections Enabled

Revert "Revert "Bug #678400 - ECollectionBackend populates disabled accounts""

This reverts commit 1c4fb39c131a03428035a2035fca8ea462fc7714,
because the initial revert was a misunderstanding of the change.

Add AllowAuthPromptAll() method to SourceManager interface.

Equivalent to calling AllowAuthPrompt() on each exported data source,
but cuts way down on the D-Bus traffic.  Since the registry service is
commonly activated during a desktop session startup, this can actually
clog up the session bus and slow down login.

This leaves the source-specific AllowAuthPrompt() method unused, but
there are still use cases for it which we haven't implemented yet, so
leave it be.

Remove version number from ${privincludedir}.

Last man standing, so may as well.  Version not needed there either.

Remove erroneous version number from ${privlibdir}.

This slipped through because we don't actually reference ${privlibdir}
in our backend/module/provider directory definitions.  Fix that too.

Also, some pkg-config files needed to be tweaked as a result of this.

Remove version number from ${privdatadir}.

In talking to some people at Red Hat about how to deploy pre-configured
Evolution accounts, one thing I suggested is to provide a package which
installs .source files to one of the system-wide locations:

    /usr/share/evolution-data-server-3.8/ro-sources
    /usr/share/evolution-data-server-3.8/rw-sources

But that's not going to work in general with the version number in the
path.  Upgrading E-D-S to, say, 3.10 would invalidate such a package.

Remove the version number from our ${privdatadir}.  It's not needed.

Bug #681962 - Identify server certificates by hostname and fingerprint

Bug #685034 - [imapx] Read status of email is not persisted correctly

Bug #687100 - Crash under cal_backend_contacts_remove_book_record()

Bug #686924 - Doesn't parse Gerrit encoded From address correctly

imapx_command_start: Improve handling of write-to-stream error.

For some reason we were calling camel_stream_write_string() without
passing a GError, and on failure setting a very generic error message
and losing potentially useful information about the actual error.

Use code coverage macros from gnome-common.

The GNOME_CODE_COVERAGE macro in gnome-common automates the process
nicely.  Just run "make check-code-coverage" from the top-level source
directory (or from any of the "tests" subdirectories) and open the nice
coverage analysis report:

    evolution-data-server-x.y.z-coverage/index.html

Bug 685986 - ESourceRegistry: Wait for signals after creating sources

e_source_registry_create_sources_sync() now waits for "object-added"
signals from its GDBusObjectManagerClient before returning.  This allows
the newly-created ESources to be obtained immediately after the function
returns.  Prior to this there was a small delay before the newly-created
ESources became available through the ESourceRegistry API.

There is a caveat to this.  If any of the scratch ESources reference a
parent UID which is not presently exported by the registry service, the
registry service will refuse to export the newly-created ESource until
its parent becomes available.

In this scenario, to avoid waiting forever for an "object-added" signal
that isn't coming, we place a limit on the wait time (two seconds).  If
this time limit expires, we return from the function regardless.

Add a test case for bug 685986.

Managed to reproduce the bug using the test case.

Prototype a D-Bus unit test harness.

Add functions for programmatically overriding XDG base directories
and launching a private D-Bus session bus using GTestDBus.

This requires Evolution-Data-Server to be installed, since the test
harness references D-Bus service files by the installation prefix.

Bump minimum GLib requirement to 2.34.

Need to utilize the GTestDBus API, which is new in 2.34.

Updated Indonesian translation

Bug 686719 - Missing linker flags in Camel tests

Updated Slovenian translation

Prefer e_source_registry_check_enabled().

Prefer e_source_registry_check_enabled() over e_source_get_enabled()
for most instances where we want to exclude disabled data sources.

Add e_source_registry_check_enabled().

Checks the enabled state of a given #ESource and of its ancestors to
determine if the #ESource should be displayed or acted on.

This is somewhat similar to widget sensitivity in GTK+.
cf. gtk_widget_get_sensitive() versus gtk_widget_is_sensitive()

Bug #680497 - POP3 re-adds messages to local Inbox

IMAPX: Explicitly pass GCancellable and GError to callbacks.

Further error handling improvements.  This involves some public API
changes, but the impact is limited to evolution-kolab.

 * CamelIMAPXCommandFunc now takes a GCancellable argument in addition
   to its GError argument.

 * The public GCancellable member of CamelIMAPXJob is now private, and
   the public GError member is removed.

 * The start() method of CamelIMAPXJob now takes a GCancellable and
   GError argument.  The GCancellable pass through this method call
   is always the CamelIMAPXJob's own private GCancellable.

 * A new camel_imapx_job_cancel() function which simply cancels the
   CamelIMAPXJob's internal GCancellable.

Additional changes internal to CamelIMAPXServer are:

 * Adapt operations to the new callback function signatures.

 * imapx_command_queue() now takes a GCancellable and GError argument
   and returns a 'success' boolean.  Other core job and command routing
   functions also now have similar function signatures.

 * Add error handling where needed, so that no failable function call
   goes unchecked.

Bug #686727 - POP3 provider deadlocks on finalize

Updated German translation

CamelIMAPXServer: Handle errors in imapx_continuation().

There was no error handling at all on the write-to-stream operations.

CamelIMAPXServer: Return a boolean from imapx_command_start_next().

The function is failable and takes a GError, yet returns void.  This
is how GErrors pile up.  Return a boolean and add the necessary error
checks where the function is called.

Revert "Bug #678400 - ECollectionBackend populates disabled accounts"

This reverts commit 9a7948409852e57721dcb2e9e63ae69d8ac78d40.

Need to find a better solution than corrupting what are supposed to be
separate state flags.

Bug #678400 - ECollectionBackend populates disabled accounts

CamelIMAPXServer: Improve error handling around stopping IDLE.

imapx_stop_idle() returned a boolean but TRUE had an ambiguous
interpretation.  It either meant the DONE command was sent or an I/O
error occurred.  Define a new enum type (CamelIMAPXIdleStopResult) to
resolve the ambiguity.

Also imapx_command_idle_stop() returned a boolean but FALSE had an
ambiguous interpretation.  It either meant an I/O error occurred, or
there was no available CamelIMAPXStream from which to issue the DONE
command.  Resolve the ambiguity by forcing callers to explicitly pass
in a CamelIMAPXStream (and a GCancellable, while we're at it).

CamelIMAPXServer: Declare imapx_uidset_* functions as static.

These functions were never meant to be exported.  The "static" keyword
seems to have been omitted by accident.  Can't find any evidence of them
being called from outside camel-imapx-server.c, so just make them static.

CamelIMAPXServer: Clean up static function declarations.

One argument per line.

CamelIMAPXServer: Make sure all disabled bits have a comment.

Normally I throw away all disabled-with-no-explanation-why code chunks,
but in this case they all seem related, so clarify that in the code.

Updated Hungarian translation

EAuthenticationSession: Close prompt before storing password.

Close our prompt before storing the password in the keyring.
If the keyring is locked, it will need to prompt the user for
a keyring password, but it can't do that if our password prompt
is still open since both prompts are system-modal.  Not sure what
would happen next; probably the store operation would either fail
or deadlock.

Bug #640594 - Save also UID/REV in WebDAV backend (for CardDAV)

Post-release version bump.

NEWS update for 3.7.1 release.

Coding style and whitespace cleanup.

evolution-source-viewer: Select next source after deleting.

Deleting a list of sources is less painful when the next source is
automatically selected.

Further test code cleanups.

Bug 686421 - Restore libebook tests to minimum working condition

Merge cleanups from 'gdbus-codegen' branch.

Merge cosmetic cleanups from my 'gdbus-codegen' branch so I can more
easily review the important changes.

Updated Spanish translation

Updated Slovenian translation

configure.ac: Revert unnecessary soname bumps.

Adding new functions does not break backward-compatibility of the
programming or binary interfaces for these libraries.

libedataserver soname bumps are particularly disruptive to other GNOME
modules, and should be avoided if at all possible.

Bug #268618 - Add: Save replies in the folder of the message being replied to

Bug #310978 - USENET always requires authentication

Updated Slovenian translation

Bug #201807 - Cannot specify sendmail parameters

camel_junk_filter_classify(): Fix return values.

Bug #641825 - Crash in poll_ldap() when finalizing the backend

Updated Greek translation

Updated Czech translation

Change camel_junk_filter_classify() signature.

Add CAMEL_JUNK_STATUS_ERROR to the CamelJunkStatus enum and have
camel_junk_filter_classify() return the status directly, rather than
forcing callers to check both a status value and a 'success' boolean.

Minor API break, but should only affect Evolution.

Updated Galician translations

Updated Slovenian translation

Updated Slovenian translation

Updated Indonesian translation

Updated Indonesian translation

Updated Polish translation

Bug 684655 - Files in data/sources are not getting translated

Bug #681314 - Add address dialog should follow composer's view settings

Fix code examples in ESourceExtension API docs.

The examples did not follow our single-include policy.

Updated POTFILES.in

Rename 'online-accounts' module to 'gnome-online-accounts'.

For better or worse, GNOME Online Accounts isn't the only accounts
service in town anymore.  Use a less generic module name.

ESourceRegistryServer: File monitor should set flags on new sources.

ESourceRegistryServer forgot to set WRITABLE and REMOVABLE flags on
new ESources created when a GFileMonitor detects a new .source file.
Consequently the new ESource could neither be modified nor removed,
which causes problems for unit tests that use predefined key files.

Silence libsecret unstable API warnings.

Deprecate e_source_extension_get_source().

Replace it with a thread-safe version: e_source_extension_ref_source()

Also use GWeakRef within ESourceExtension, which itself is thread-safe.

Remove e_data_cal_view_register_gdbus_object().

Similar to EDataCal, the first thing we do after creating a new
EDataCalView is export its D-Bus interface.  This was a separate and
failable operation.  If the export fails, the EDataCalView is useless.
Perfect use case for GInitable.

Now we pass the GDBusConnection and object path directly to
e_data_cal_view_new(), and if exporting fails the function sets a
GError and returns NULL.

This also introduces a couple accessor functions:

    e_data_cal_view_get_connection()
    e_data_cal_view_get_object_path()

Remove e_data_book_view_register_gdbus_object().

Similar to EDataBook, the first thing we do after creating a new
EDataBookView is export its D-Bus interface.  This was a separate and
failable operation.  If the export fails, the EDataBookView is useless.
Perfect use case for GInitable.

Now we pass the GDBusConnection and object path directly to
e_data_book_view_new(), and if exporting fails the function sets a
GError and returns NULL.

This also introduces a couple accessor functions:

    e_data_book_view_get_connection()
    e_data_book_view_get_object_path()

EDataCalView cleanups.

Some minor API changes with this:

    e_data_cal_view_get_text() is dropped.  Instead use:

        sexp = e_data_cal_view_get_sexp (view);
        query = e_cal_backend_sexp_text (sexp);

EDataBookView cleanups.

Some minor API changes with this:

   e_data_book_view_get_card_sexp() is now e_data_book_view_get_sexp().

   e_data_book_view_get_card_query() is dropped.  Instead use:

       sexp = e_data_book_view_get_sexp (view);
       query = e_book_backend_sexp_text (sexp);

Add e_book_backend_sexp_text().

Similar to e_cal_backend_sexp_text(), returns the original textual
expression used to build the EBookBackendSExp object.

e-cal.c cleanups.

e-book.c cleanups.

EGDataGoaAuthorizer: OAuth 2.0 fixes.

Changes based on Google's OAuth 2.0 Playground.

Authorization type is "OAuth" instead of "Bearer", and no Base64
encoding of the access token is required.

Still no successful test with GOA 3.7, so somewhat flying blind.

online-accounts: Refresh mail account settings on startup.

When matching GOA accounts to existing EDS accounts on startup, refresh
both the collection and mail account settings.  This is primarily to aid
the transition from XOAUTH to XOAUTH2, but can serve as a means of other
types of migration in the future.

EGDataGoaAuthorizer: Prototype OAuth 2.0 support.

Untested, but conforms to the specification as well as I understand it.

vala: Add libedataserver-1.2 to libebook-1.2.deps file.
Fixes part of folks bug https://bugzilla.gnome.org/show_bug.cgi?id=685084
so folks ofono backend will only need --pkg libebook-1.2
not libedataserver-1.2 explicitly.

Drop oauth package requirement for GOA support.

On second thought, Camel's XOAUTH SASL module is just using
g_random_int() to generate the "oauth_nonce" parameter.

Good enough, I suppose.  Do the same for EGDataGoaAuthorizer.

EGDataGoaAuthorizer: Use GHmac to sign the OAuth parameters.

Use GLib's GHmac API in place of oauth_sign_hmac_sha1().

Unfortunately we still have to keep the liboauth requirement for its
oauth_gen_nonce() function.  Considered just copying it out of oauth.c
but it looks rather messy with OpenSSL and NSS dependencies.

configure.ac: Tweak GOA-related error messages.

Drop the check for libgdata >= 0.9.1 for GOA support.

We already require libgdata >= 0.10, so the check is redundant.

Eliminate the assertion in camel_provider_register().

camel_provider_register() should automatically set up the module and
provider hash tables if needed instead of aborting, but the problem
is the GOnce function that sets up these hash tables also calls
camel_provider_register().

Break this cycle by moving the bulk of camel_provider_register() to a
new internal function (provider_register_internal) and calling it from
the GOnce function (provider_setup).

The public camel_provider_register() function is then simply:

    g_once (&setup_once, provider_setup, NULL);

    provider_register_internal (provider);

Bug 685713 - ESourceRegistry's manager thread aborts on error

If the ESourceRegistry's manager thread fails to create a new
GDBusObjectManagerClient during initialization, propagate the GError
back to the caller of e_source_registry_new() instead of aborting the
whole process with a g_error() call.

Bug 685725 - Handle "muser" attributes in %gconf.xml files

The migration of account data from GConf to key files parses raw
%gconf.xml files.  The parsing code neglected to handle "muser"
attributes in <entry> tags, so that when an "muser" attribute was
encountered, g_markup_collect_attributes() would throw an error an
cause the whole GConf migration phase to be aborted.

Don't migrate sources with empty attributes

vala: Don't ignore CalClientSourceType

It's needed by the API after the calendar refactoring.

Signed-off-by: Colin Walters <walters@verbum.org>

Updated Slovak translation

Remove e_data_cal_register_gdbus_object().

The first thing we do after creating a new EDataCal is export its D-Bus
interface.  This was a separate and failable operation.  If the exporting
fails, the EDataCal is useless.  Perfect use case for GInitable.

Now we pass the GDBusConnection and object path directly to
e_data_cal_new(), and if the exporting fails the function sets a GError
and returns NULL.

This also introduces a couple accessor functions:

    e_data_cal_get_connection()
    e_data_cal_get_object_path()

Deprecate e_cal_backend_foreach_view().

Use e_cal_backend_list_views() instead.

ECalBackend: Prefer e_cal_backend_list_views().

Less klunky than e_cal_backend_foreach_view().

e_cal_backend_foreach_view: Use e_cal_backend_list_views().

Holding a locked mutex while invoking callback functions we don't
control raises the potential for deadlock.

Add e_cal_backend_list_views().

Alternative to e_cal_backend_foreach_view().

ECalBackend: Prefer GList over GSList.

EGdbusCal: Remove the remove() method.

Not used anymore.  Remove the ESource instead.

This also takes out the following backend functions:

    e_cal_backend_remove()
    e_cal_backend_sync_remove()
    e_data_cal_respond_remove()

ECal: Reimplement "remove" functions.

Have them just call e_source_remove() or e_source_remove_sync().