EBookBackendFile: Avoid lost data in contact modifications

When modifying contacts, ensure that the contact revisions are in
sync with the existing ones, otherwise report the E_DATA_BOOK_STATUS_OUT_OF_SYNC
error. This indicates that the client should refresh it's local copy
of the contact before trying to modify it again.

Conflicts:

addressbook/backends/file/e-book-backend-file.c

Adding test case proving there is a write-write race condition to fix

The test case creates a number of threads all trying to modify the
same contact in the same book each concurrently with a different field,
if the modification fails it retries. If all the fields (one for each
thread) are not properly set at the end of all thread execution, then
the test is a failure.

Adding ESourceRevisionGuards

The ESourceRevisionGuards extension configures whether an ESource backend
should check revisions on object modifications and produce an
E_CLIENT_ERROR_OUT_OF_SYNC error upon revision conflicts.

EClient: Adding E_CLIENT_ERROR_OUT_OF_SYNC

An error to descibe out of sync data between the client and backend.

Clean up the mess that is CalObjModType.

Deprecate both CalObjModType and EDataCalObjModType.

Introduce ECalObjModType with equivalent but properly namespaced enum
values, generate a GType for this enum, and swap out CalObjModType for
ECalObjModType throughout the calendar API.

Trying not to break the API here, but this is a freakin mess.

Deprecate EDataCalObjType.

No longer needed with the new D-Bus factory APIs.

Deprecate CalMode and cal_mode_to_corba().

e-cal-types.h cleanups.

e-cal-util.h cleanups.

Updated Slovenian translation

Bug #651399 - [maildir] Escape dots in folder display names

GOA: Support the upcoming "imap_smtp" provider type.

gnome_online_accounts_get_backend_name() cleanups.

Make ECollectionBackend and ECollectionBackendFactory instantiable.

ECollectionBackendFactory will use a backend name of "none" and will
instantiate an ECollectionBackend, which takes no action on its own.

Believe it or not, there's actually a use case for this.

GOA will soon add a provider for generic IMAP/SMTP accounts.  Our GOA
module will support these accounts by configuring a collection source
with a backend name of "none" as well as mail account, mail identity,
and mail transport sources.

The mechanics of ESourceRegistryServer do require a collection backend
to support a collection source but the backend will have nothing to do.

Updated Spanish translation

tests: Fix srcdir != builddir

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

ECalClient: Add a "source-type" construct-only property.

Restore test case for bug 685986.

EClient: Remove unused 'uri' member.

EClient: Minor cleanups.

Deprecate e_cal_backend_sync_set_backend_property().

No longer used.

Deprecate e_cal_backend_set_backend_property().

No longer used.

Deprecate e_data_cal_respond_set_backend_property().

No longer used.

EDataCal: Remove OP_SET_BACKEND_PROPERTY.

No longer used.

EDataCal: Remove "handle-set-backend-property" signal handler.

No longer used.

Deprecate e_book_backend_sync_set_backend_property().

No longer used.

Deprecate e_book_backend_set_backend_property().

No longer used.

Deprecate e_data_book_respond_set_backend_property().

No longer used.

EDataBook: Remove OP_SET_BACKEND_PROPERTY.

No longer used.

EDataBook: Remove "handle-set-backend-property" signal handler.

No longer used.

EClient: Deprecate e_client_set_backend_property_sync().

Clients cannot set backend properties.  Any attempt will fail with an
E_CLIENT_ERROR_NOT_SUPPORTED error.

Also deprecate the asynchronous variant:

    e_client_set_backend_property()
    e_client_set_backend_property_finish()

ECalClient: Handle set_backend_property() requests directly.

Neither ECalBackend itself nor any known ECalBackend subclass allows
clients to set backend properties, so forego the D-Bus message and fail
immediately on the client side with an E_CLIENT_ERROR_NOT_SUPPORTED.

EBookClient: Handle set_backend_property() requests directly.

Neither EBookBackend itself nor any known EBookBackend subclass allows
clients to set backend properties, so forego the D-Bus message and fail
immediately on the client side with an E_CLIENT_ERROR_NOT_SUPPORTED.

EClient: Add default implementations for async virtual methods.

Subclasses will need only implement the synchronous virtual methods.

Updated Serbian translation

Deprecate e_cal_backend_is_opening().

This always returns FALSE now.

Deprecate e_book_backend_is_opening().

This always returns FALSE now.

Deprecate e_client_is_opened().

This always returns TRUE now.

Deprecate CLIENT_BACKEND_PROPERTY_OPENING.

Clients don't need to care if they're fully opened anymore.

Deprecate CLIENT_BACKEND_PROPERTY_OPENED.

Clients don't need to care if they're fully opened anymore.

ECalBackend: Add a "writable" boolean property.

EBookBackend: Remove redundant "online" flag.

EBookBackend: Add a "writable" boolean property.

EDataCal: Give OperationData a GDBusMethodInvocation.

Not used just yet, but will be.

EDataBook: Give OperationData a GDBusMethodInvocation.

Not used just yet, but will be.

EDataCal: operation_thread() cleanups.

EDataBook: operation_thread() cleanups.

EGdbusTemplates: Address crash on operation cancel

In situations when a synchronous operation was cancelled, but the response
was already piled in main context the client could receive two replies, one
from the reply, the other from the cancelled operation, effectively accessing
invalid memory in the second response. This may address also other similar
situations caused by cancelled operations.

EDataCal: Have pending_ops hold OperationData references.

EDataBook: Have pending_ops hold OperationData references.

EDataCal: Give OperationData a reference count.

EDataBook: Give OperationData a reference count.

EDataCal: Rename signal handlers.

EDataBook: Rename signal handlers.

Avoid crash in e_book_backend_file_load_revision()

There could happen a crash in e_book_backend_file_load_revision()
due to NULL 'error' when an application did a quick open/close
operations, in which case a backend was reused, with already set
priv::revision, and the call failed on a runtime constraint, but
didn't set the 'error', which led to the crash.

EDataCal: Add argument guards where needed.

EDataBook: Add argument guards where needed.

Updated Spanish translation

Assamese translation updated

Replace EGdbusCalFactory with EDBusCalendarFactory.

Replace EGdbusBookFactory with EDBusAddressBookFactory.

Generate D-Bus factory interfaces with gdbus-codegen.

Bump D-Bus service names for new factory APIs.

Updated POTFILES.in

E_CLIENT_ERROR_BUSY is no longer used.

EDataCal: Queue operations while opening.

When an open operation is in progress, queue any subsequent operations
and dispatch them after the current open operation is complete.

The old behavior returned a BUSY error and forced the client to retry.
Clients are not involved in authentication anymore, so they should not
have to retry operations nor worry about whether the backend is opened.

This should really be handled directly by ECalBackend, but because of
the weird async semantics between ECalBackend and EDataCal, the queue
needs to live in EDataCal for now.

EDataBook: Queue operations while opening.

When an open operation is in progress, queue any subsequent operations
and dispatch them after the current open operation is complete.

The old behavior returned a BUSY error and forced the client to retry.
Clients are not involved in authentication anymore, so they should not
have to retry operations nor worry about whether the backend is opened.

This should really be handled directly by EBookBackend, but because of
the weird async semantics between EBookBackend and EDataBook, the queue
needs to live in EDataBook for now.

EDataBook/EDataCal: Minor cleanups.

EBookBackend: Initialize "cache-dir" from constructed().

Also tweak ECalBackend's constructed() method to match.

Updated Spanish translation

Revert accidential commit from a761eade

libebook: Avoid accidental implicit const-cast

Introduced by accident with 663c169e

Make ETestServerFixture more reliable.

Since e_source_registry_commit_source_sync() does not delay execution
until the source is actually added, use the "source-added" signal instead
to wait for the source to actually be added.

Also, created loops in the fixture test and run more iterations
(should help to catch problems with the fixture early, in case
there are still races).

Make use of G_DEFINE_QUARK()

This reduces boilerplate and ensures that quark definition functions make
proper use of the G_UNLIKELY macro.

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

Updated Lithuanian translation

libebook: Support phone number specific queries

This adds new values to the EBookQueryTest enumeration:

 * E_BOOK_QUERY_EQUALS_PHONE_NUMBER
 * E_BOOK_QUERY_EQUALS_NATIONAL_PHONE_NUMBER
 * E_BOOK_QUERY_EQUALS_SHORT_PHONE_NUMBER

This values get translated to the binary sexp functions "eqphone",
"eqphone_national" and "eqphone_short".

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

libebook: Add phone number utilities

This functionality covers parsing, formatting and comparing
phone numbers.

Note that this commit introduces an optional depedency on C++.

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

Fixed regression in e_book_get_book_view_async().

The reference counting scheme on the returned view was changed while
porting the EBook apis to use EBookClient apis internally, the result
is that the async callback no longer receives ownership of the passed
EBookView (this commit reverts to the old behaviour).

test-ecal-get-objects-for-uid: Fixed assertions in test case.

Similar to test-ecal-create-object, this test was comparing a component
built from scratch (with no UID) against a component fetched from
the calendar (with a UID).

Fixed test case to compare the components in the returned list against
a component fetched from the calendar (both with UIDs).

test-ecal-create-object: Fixed to assert something that makes sense.

The old assertions were broken in that it compares the scratch component
with no UID with the newly created and fetched component.

Now simply check that the newly created component can be fetched and that
it is 'valid'.

test-ecal-get-ldap-attribute: Changing test to check the reported error code.

As e_cal_get_ldap_attribute() docs mention that the function has been dropped
completely, this test (instead of removing) now simply tests for the expected
error code.

Additionally, the unused function ecal_test_utils_cal_get_ldap_attribute()
has been removed from in ecal-test-utils.[ch].

ECal test, test-ecal-get-free-busy: Create a valid user identity and account

Without this, the backend does not report any free-busy information,
with this commit the backend actually reports the free-busy data.

NOTE: This test still fails, because e_cal_get_free_busy() does not report
free busy information from the ECalClient API properly (see FIXME comment
in e_cal_get_free_busy()).

ECalClient test, test-client-get-free-busy: Create a valid user identity and account

Without this, the backend does not report any free-busy information,
with this commit the backend actually reports the free-busy data and
the test now additionally asserts that the free-busy data was reported.

ECalClient test test-client-refresh: Only test the refresh APIs if it is supported.

Newer versions of the local calendar backend dont support the explicit refresh,
so failing the explicit refresh is not an error.

EBookBackendFile: Report the proper error if a contact is not found by UID.

Conflicts:

addressbook/backends/file/e-book-backend-file.c

EBookBackendSqliteDB: Added contact-not-found error type.

Report this error if a contact was requested by UID and not found.

Improving test-client-remove-contact

Check for error type is more clear and safe now.

Bug #692278 - LDAP backend mutex deadlock on finalize

CamelIMAPXStore: Configure folder flags on folder creation.

Immediately after CamelFolder creation, check whether the store is
configured to use real Junk/Trash folders and set CamelFolderFlags
appropriately.

This solves the problem of Evolution not displaying junk messages in a
real Junk folder.  This does NOT solve the restart problem, as noted in
the comment.

CamelIMAPXStore: Pass a GError to get_folder_offline().

This is just silly.  get_folder_offline() already sets the GError on
failure.  There's no reason for imapx_store_get_folder_sync() to not
pass the GError to get_folder_offline() and then set its own generic
error message on failure.

CamelIMAPXStore: get_folder_offline() cleanups.

Fixing test-ebook-get-supported-auth-methods.c

This tests that unsupported code paths actually run as expected
(the problem was that the async callback was called before entering
the mainloop and waiting for a reply or timeout).

Added test case ensuring that new contacts have thier UID preserved.

Asserts both that newly added contacts preserve the assigned UID and
also that 2 contacts with the same UID cannot be added.

EBookBackendFile: Don't overwrite UIDs on incomming/new contacts.

  o Only assign a generated UID if the new contact does not have any
  o If the UID already exists in the SQLite at creation time, then
    report an E_DATA_BOOK_STATUS_CONTACTID_ALREADY_EXISTS error.

EBookBackendSqliteDB: Added e_book_backend_new_contact[s]() APIs

The new API allows control on whether or not to overwrite existing contacts
with the same UID.

Deprecated the older e_book_backend_add_contact[s]() apis which include
the unused 'partial_contact' parameter and have no control on whether
or not to overwrite existing contacts.

Additionally this patch exposes the E_BOOK_SDB_ERROR domain and a couple
of error codes to distinguish if an error occurred because of a constraint
or other errors (UID conflicts at contact addition time are constraint
errors).

Conflicts:

addressbook/libedata-book/e-book-backend-sqlitedb.c
addressbook/libedata-book/e-book-backend-sqlitedb.h

build: Fix parallel make issue in GResource change

We need deps on the generated files for both header and source.

Bundle built-in data sources as a GResource.

This was primarily meant to aid our test suite, whose goal is to run
successfully without requiring evolution-data-server (or its data files)
to be installed.  The tests inadvertently depend on these built-in data
sources being present.

But this is good to do anyway.  Bundling the data sources directly in
the evolution-source-registry binary guarantees they're always present,
which the E-D-S APIs already assume, and avoids the usual problems with
installed data files: I/O errors, user meddling, etc.

Deprecate e_source_registry_server_load_all().

It was a mistake to encode this much file location policy directly into
libebackend's API.  Instead, evolution-source-registry.c will implement
its own function that uses e_source_registry_server_load_resource().

Add e_source_registry_server_load_resource().

Loads data source key files from a GResource by enumerating the children
at the given pathname and calling e_source_registry_server_load_file()
on each child.

Fix the copyright in test-client-custom-summary.c