test-client-view-operations.c: Now test both direct access and indirect access

This works properly now since we fixed (or worked around) issues in the
test fixtures. Since we use a separate ESource UID (sandbox) for each
test case this now works properly.

Some changes to the isolated test fixture

 a.) Setup EDS_REGISTRY_MODULES to use the local cache-reaper module
 b.) Setup EDS_CAMEL_PROVIDER_DIR to use the local provider
 c.) Added "source_name" and use separate ESource uids between tests,
     each test in the suite uses a separate ESource name

Note that 'c' is a stop-gap solution to the problem of not being able to
properly shut-down the whole test D-Bus environment between tests, it should
be removed once we can address that problem.

Make the Camel provider modules relocatable

This simply adds an environment variable allowing us to load the local
provider from a relocated location in the case we run 'make check' without
installing (this avoids some warnings and helps to isolate the test
environment).

Make the ESourceRegistryServer modules relocatable like addressbook & calendar

This simply adds an environment variable allowing us to load registry modules
from a relocated location in the case we run 'make check' without installing
(also ensuring that we test the not-yet-installed environment properly).

Do not update EBackend::online in a separate thread

Let the backend decide whether update its online state in a separate thread
or not. The initial idea was to have this done on a central place, but
it seems to be useless for most of the backends, thus do not do that.

Updated Uyghur translation

Signed-off-by: Gheyret Kenji <gheyret@gmail.com>

ECalClient: Remove global "cal_factory" variable.

Have cal_client_init_in_dbus_thread() create its own factory proxy,
obtain an object path for the desired backend, and then immediately
destroy the factory proxy object.

This eliminates the global EDBusCalendarFactory and associated
variables and macros and reduces the potential for reference leaks.

EBookClient: Remove global "book_factory" variable.

Have book_client_init_in_dbus_thread() create its own factory proxy,
obtain an object path for the desired backend, and then immediately
destroy the factory proxy object.

This eliminates the global EDBusAddressBookFactory and associated
variables and macros and reduces the potential for reference leaks.

EBookBackendWebdav: Make error msgs with HTTP status more consistent.

Fix comments in e_contact_map_google_with_evo_group().

Fix typo in e_phone_number_error_to_string().

Fix translator comment in camel_session_build_password_prompt().

Updated Russian translation

Updated POTFILES.in

EClient: Add a read-only "main-context" property.

This is the GMainContext on which to attach client event sources.

New functions:

    e_client_ref_main_context()

EClient cleanups.

Add e_source_registry_dup_unique_display_name().

Compares a source's display name against other sources having an
extension with the given extension name, or else against all other
sources in the registry if no extension name was given.

If the source's display name is unique among these other sources, the
function will return the source's display name verbatim.  Otherwise the
function will construct a string that includes the source's own display
name as well as those of its ancestors.

The functions's return value is intended to be used in messages shown to
the user to help clarify which source is being referred to.  It assumes
the source's display name is at least unique among its siblings.

ecalclient: Fix uninitialized list

This would likely crash.

evolution-user-prompter: Add a --keep-running option.

This thing keeps quitting on me when run from the command-line.

CamelSMIMEContext: Support IANA registered hash function names.

Use and recognize standard-compliant hash function names in the
hash_to_id() and id_to_hash() methods.

CamelSMIMEContext cleanups.

Fix build break from libebook-contacts.

None of the backends were linking to libebook-contacts-1.2.la.

evo_phonenumber.m4: Don't stomp on LDFLAGS.

Deprecate e_client_error_create().

Just use the GError API directly.

Deprecate e_client_util_unwrap_dbus_error().

No longer used.

EClient: Deprecate unwrap_dbus_error() method.

The generated D-Bus proxy classes return errors according to the usual
GLib conventions, and EDataBook and EDataCal convert server-side error
codes to client-side error codes before completing method invocations,
so we don't need this method anymore.

Added a default implementation to EClient which is just a pass-through.

EClient: Deprecate get_dbus_proxy() method.

This method is not used anymore.  Leave the implementations in place,
but tag the method as deprecated in the EClientClass struct so it can
be purged at some future date.

EBackend: Update online state from the correct main loop context.

Use a low-priority idle callback on the backend's main loop context to
reduce the number of times we cancel g_network_monitor_can_reach_async().

ESourceMailComposition: Escape brackets in documentation.

Add libebook-contacts-overrides.txt to source control.

Add camel_session_ref_main_context().

Replaces camel_session_get_main_context().

ESource: Improve "main-context" property description.

Finally found a wording for the property that I'm happy with.

CamelSession: Improve "main-context" property description.

Finally found a wording for the property that I'm happy with.

cal_client_add_cached_timezone: Wrong GDestroyNotify callback.

ECalBackend: Follow up on a timezone-related FIXME comment.

Emit the "timezone-added" signal from an idle callback on ECalBackend's
GMainContext, instead of from whichever thread the timezone was added.
This makes life easier for any handlers of that signal.

EBackend: Add a read-only "main-context" property.

This is the GMainContext on which to attach backend event sources.

New functions:

    e_backend_ref_main_context()

EBookQuery/EPhoneNumber: Enhanced documentation

EBookQuery documentation simplified to reference documentation
from EPhoneNumber docs specifically. EPhoneNumber documentation
enhanced to be more descriptive in terms of the match type strengths
possible.

Documentation Makefiles: Added MKDB_OPTIONS

Added the MKDB_OPTIONS to all gtk-doc makefiles, this makes
the <note><para></para></note> semantics in gtk-doc statements
work properly (along with probably a few other features).

test-client-custom-summary.c: Adding more tests regarding phone numbers.

The nature of phone number matching is complex, some additional
tests are required to ensure that this works to spec (perhaps more
tests will be added here as well).

e_cal_client_modify_objects_sync: Fix infinite loop.

e_source_registry_server_load_directory: Monitor failure is non-fatal.

Directory monitoring is a nice-to-have feature, but is not supported in
some contexts.  If we fail to create a GFileMonitor, leave a breadcrumb
on the console to indicate something whent wrong, but don't return an
error status.  That would cause the whole registry process to terminate.

EBookBackendFile: Report the correct error from remove_contacts()

If the EBookBackenSqliteDB reports CONTACT_NOT_FOUND, then use
E_DATA_BOOK_STATUS_CONTACT_NOT_FOUND, this causes test-client-remove-contact
to pass again.

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

test-client-remove-contact.c: Enhanced test case

Check that removing a removed contact reports the correct error code

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

CamelSmtpTransport: Override new CamelNetworkService properties.

CamelPOP3Store: Override new CamelNetworkService properties.

CamelNNTPStore: Override new CamelNetworkService properties.

CamelIMAPXStore: Override new CamelNetworkService properties.

CamelNetworkService: Add a read-only "host-reachable" property.

This will be updated automatically as network conditions change.

New functions:

    camel_network_service_get_host_reachable()

CamelNetworkService: Add a "connectable" property (GSocketConnectable).

This is the socket endpoint for the network service to which the
CamelNetworkService is a client.

This also adds a new_connectable() method to the interface, which by
default creates a new GNetworkAddress from the CamelNetworkSettings.

New functions:

    camel_network_service_ref_connectable()
    camel_network_service_set_connectable()

Give CamelNetworkService a private structure.

CamelNetworkService will do its own networking monitoring.

The interface is initialized during CamelService construction.

Do not compile code which might be unnecessary with GLib 2.35.9+

This is the only relevant part of the code which should be "removed"
from the previous commit due to changes in GNetworkManager's bug:
https://bugzilla.gnome.org/show_bug.cgi?id=694181

Once eds will depends on such GLib the conditioned code can be
completely removed.

Revert "Revert "Make network connection change listener actually working""

This reverts commit 51f5b1201bb713243b0cb2e160c18ee17d3a0baf.

It's inappropriate to revert whole commit, the relevant part is much
much much smaller.

Revert "Make network connection change listener actually working"

This reverts commit b64b51e0e969fbf355d3a2bc1c69f4123c33078f.

These changes are unnecessary given recent fixes in GNetworkMonitor.

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

Make network connection change listener actually working

With this it's properly recognized when network connection changes,
like when user connects to VPN, or disconnects from it, and the backend's
online state is properly updated (if is provides connection address).

test-client-view-operations.c: For now disable the regular access test

This test was written to protect against deadlocks in EBookBackendSqliteDB
with concurrent read access. Currently two in a row fails when fetching
the newly added contacts in the test book, still dont know why but it's
orthogonal to the reason for this test.

Bug #694223 - The "message-location" search never matches on IMAP+

EDataBook: Fix e_data_book_new_direct()

Since recent backends in master open thier persistance at initable_init()
time, it became necessary to splitup g_initable_new() into g_object_new()
and g_initable_init(), calling e_book_backend_configure_direct() in between.

Note: this fixes Direct Read Access where the XDG directories might not
be the same in the server context as in the client context.

Bug #694152 - Compile errors in C++ due to G_BEGIN/END_DECLS

Fix a memory leak in EBookQuery with field_test.locale

test-client-custom-summary.c: Fixup test case to also try Default summary

This patch makes the whole test battery run with a default summary
as well, ensuring the expected results work properly regardless
of whether the summary information is stored in SQLite or if it
falls back to full vcard parsing.

EBookBackendSexp: Restore suffix search functionality

All suffix searches using EBookBackendSexp had been broken as
of commit 5236e0b1f82ec1ef9d2e87e2f8e493471b8cd57e. This patch
restores suffix searches to work properly again.

EBackend: Use e_source_authentication_ref_connectable().

Initialize the GSocketConnectable directly from ESourceAuthentication's
GSocketConnectable, rather than building a new instance from its "host"
and "port" properties.

ESourceAuthentication: Add a read-only "connectable" property.

This is an internally cached GSocketConnectable instance constructed
from ESourceAuthentication's "host" and "port" properties.

New functions:

    e_source_authentication_ref_connectable()

Local Delivery not updated on start and missing Receiving Options page

This was reported at Red Hat's bugzilla:
https://bugzilla.redhat.com/show_bug.cgi?id=910252

To have this fully working, users should edit the account and save
the changes (even when not done any) by clicking the OK button
in the account editor, thus the source will receive its extension
and the next start the account will be recognized and auto-updated.

Do not use SSL_V2_COMPATIBLE_HELLO by default

Zarafa server with disabled SSL v2 rejects connection when
SSL v2 compatible hello is sent, thus disabled this by default.
After all, SSL v3 should be used in general these days anyway.

Use environment variable CAMEL_SSL_V2_HELLO=1 in case your server
requires the old behavior.

Extending test-client-custom-summary.c

Added test to ensure that phone numbers are not accidentally normalized,
and that we dont get a result for ENDS_WITH "8899" if a phone number
ending with "88 99" exists in the addressbook.

Ported various EBookClient tests to additionally test Direct Read Access variants

This patch basically adds additional invocations of the same test code
in direct read access mode. Only for tests concerned with read access
to the addressbook.

Note that the test-client-uid-only-view and test-client-revision-view
tests are not ported, they are not expected to pass in Direct Read Access
mode (since the EBookClientView in Direct Read Access mode fetches contact
data directly from the backend, any contact field filtering would be a
useless overhead).

e-test-server-utils: Add support for testing Direct Read Access books.

source_remove_sync: Fail gracefully when given a scratch source.

Apparently e_dbus_object_get_source_removable() does not check its
argument for NULL, and scratch sources have no internal EDBusObject.

source_write_sync: Fail gracefully when given a scratch source.

Apparently e_dbus_object_get_source_writable() does not check its
argument for NULL, and scratch sources have no internal EDBusObject.

Updated Serbian translation

Fixing typo in previous commit fixing G_BEGIN/END_DECLS

Fixing imbalanced G_BEGIN_DECLS/G_END_DECLS in EDS headers.

modified:   addressbook/libebook/e-book-types.h
modified:   camel/camel-imapx-search.h

Bug #693101 - IMAPx can vanish and redownload folder summary

Fixing pkg-config files for libebook & libedata-book

This part was missed while the libebook / libebook-contacts library
splitup was done manually.

Cleanup: Remove double definition of E_BOOK_CLIENT_ERROR

The EBookClientError definition has been moved to libebook-contacts,
this is just removing a stale #define E_BOOK_CLIENT_ERROR which
was remaining in e-book-client.h

Added documentation for new Direct Read Access related apis.

Also fixed up gtk-doc build for libedata-book to add EDataBookDirect
to the sections, types and libedata-book-docs.sgml files.

Migrating documentation into new libebook-contacts book.

This documentation splitup was missed in the recent libebook / libebook-contacts
library splitup.

EBackend: Utilize g_network_monitor_can_reach_async().

If the backend has a GSocketConnectable, determine the "online" state
using g_network_monitor_can_reach_async().

The "online" state is automatically updated on instance initialization,
when the GSocketConnectable is replaced, and when GNetworkMonitor emits
a "network-changed" signal.

EBackend: Keep a reference on GNetworkMonitor.

Keep a strong reference on the default GNetworkMonitor, and connect to
its "network-changed" signal.  Our signal handler does nothing for the
moment, but shortly it will update the "online" state.

EBackend: Add a "connectable" property (GSocketConnectable).

This is the socket endpoint for the network service to which the
EBackend is a client.  This can be NULL if the EBackend does not
use network sockets.

The initial value of the "connectable" property is derived from the
ESourceAuthentication extension of the EBackend's "source" property,
if the extension is present.

EBackend: Use g_clear_object() in dispose().

Post-release version bump.

Fix distcheck error.

NEWS update for 3.7.90 release.

data_cal_handle_remove_objects_cb: Fix GVariant format string.

e_cal_client_remove_objects_sync: Fix an infinite loop.

ECalObjModType is a flags type, not an enum type.

Transmit flags over D-Bus as a string of colon-separated nicknames.

Updated Polish translation

Add optimized indexing capabilities for phone number values.

Creating indexes with normalized phone numbers permits fast phone number
lookup even if the stored vCards or the queried phone numbers are freely
formatted or incomplete.

The numbers are formatted according to E.164 when building the index.
Incomplete numbers are prefixed with a country-code matching the current
locale settings while processing query results.

Updated Mathias Hasselmann's patch on bug 689622 to apply to current master.

Store E.164 phone number values automatically in the vCard

The E.164 normalized phone number is of interest for handset related
applications. With this change a X-EVOLUTION-E164 attribute is added
to each TEL attribute if the contact summary contains a E.164 formatted
variant of the phone number. This shall avoid overhead and inconsistency
that would occur if clients would use their own mechanism to compute
that already stored information.

Updated Mathias Hasselmann's patch on bug 689622 to apply to current master.

e_extensible_load_extensions: Allow for retries.

If e_extensible_load_extensions() does not find any extensions, remove
the GPtrArray from the extensible object.  It may be the extension types
have not been registered yet.  Subsequent e_extensible_load_extensions()
calls on the same object should rescan until extensions are found.

ECalClient: Add a "default-timezone" pointer property.

This just uses the existing e_cal_client_get_default_timezone() and
e_cal_client_set_default_timezone() functions, but the property can
be a useful target for bindings.

Updated POTFILES.in

ebook: make EContactGeo usable in Vala

"new E.ContactGeo()" in Vala fails because the struct has no
constructor. Introducing e_contact_geo_new() fixes that problem.

This extends the API instead of changing it. It is still valid
to allocated EContactGeo instances via some other means.

ebook: make EContactCert usable in Vala

"new E.ContactCert()" in Vala fails because the struct has no
constructor. Introducing e_contact_cert_new() fixes that problem.

This extends the API instead of changing it. It is still valid
to allocated EContactCert instances via some other means.

ebook: avoid repeatedly creating GSettings in e_book_client_is_self

When receiving the complete address book, a client has to use
e_book_client_is_self() on every EContact to find the one which is the
"self" contact. Calling e_book_client_get_self() instead does not
work, because that would create the self contact if none exists (not
desired for an app which just reads!).

The problem with e_book_client_is_self() is that it creates and
destroys a GSettings instance for the self UID each time the method is
called. In addition to reading the value over and over again, this
also triggers two D-Bus messages (AddMatch and RemoveMatch) - clearly
bad for performance.

To solve this problem, this patch caches the GSettings instance in a
static variable. It is protected by a mutex, to keep the function
thread-safe. The downside is that the instance is never going to be
freed.

It would be better to attach the GSettings instance to the EBookClient
instance, but because e_book_client_is_self() doesn't get a pointer to
that, this is not possible without an API change.

Added concurrent view test: test-client-view-operations.c

This tests concurrent access to EBookClientView, 20 threads each
race to receive notifications from the same addressbook and the
test asserts that all 5 contacts in the test addressbook are found.

This is particularly interesting with DEBUG_DIRECT set (Direct Read Access),
as it ensures that concurrent direct reads of the addressbook do not
cause any lockups.

e-test-server-utils: Added DEBUG_DIRECT env var

This is a temporary workaround to test Direct Read Access,
a later patch will ensure all tests check both Direct/Normal read access
modes.

EBookBackendFile: Add support for Direct Read Access.

This patch simply implements the added backend vfuncs to advertize
and configure itself in direct read access mode.