Bug #687670 - Signing with both pgp and S/MIME causes a broken signature

Bug #677378 - Crash in imapx_query_auth_types_sync()

Bug #688479 - Empty name selector dialog on open

Bug 688639 – SMTP: Correctly deal with 535 authentication error

Entering a wrong password, Evolution would fail to send messages using SMTP with the following error.

    Bad authentication response from server.

The log file of the mail server – Exim 4 in this case – contains the following error.

    cram_md5_server authenticator failed for myhost.example.org ([10.10.10.10]) [xx.xxx.xxx.xxx]: 535 Incorrect authentication data (set_id=joey)

Looking at the code after the rewrite in

    commit 68c269a66bd484768b9c1517e7dc2dd30f682485
    Author: Matthew Barnes <mbarnes@redhat.com>
    Date:   Sat Oct 15 09:51:35 2011 -0400

        smtp: Adapt to Camel's new authentication API.

present since git tag EVOLUTION_DATA_SERVER_3_3_1, accidentally changed the logic not acting correctly, when receiving a 535 authentication error response and therefore was overridden by the next code block and the variable `result` changed to `CAMEL_AUTHENTICATION_ERROR` instead of `CAMEL_AUTHENTICATION_REJECTED`, which resulted in the situation where the password was not reset and therefore the user not asked to reenter it.

Changing the logic, so the variable is not overridden successfully fixes the issue.

Bug #688660 - Crash on webdav contact cache update

Updated Spanish translation

Fixing e_book_backend_sexp_match_contact() for emails and phone numbers

This patch fixes the match function to match any phone number or email
found in a given vcard, not only the restrictive list of phone number
and email types defined in the EContactField enumeration.

EContact: Added boxed type E_CONTACT_TYPE_ATTR_LIST and e_contact_field_type()

This boxed type is used in place of pointer types for properties
installed on EContact which are returned lists of strings.

This does not effect the behavior of e_contact_set()/e_contact_get(),
only it allows e_contact_field_type() to return a distinct type
for those fields, allowing automated code to introspect those useful
contact field types.

The e_contact_field_type() convenience function is used to fetch the GType
for a given field when used with EContact (this allows more automated
routines to be written for EContactFields).

ECalClientView cleanups.

Let ECalClientView create its own D-Bus proxy object by implementing
GInitableIface and adding "connection" and "object-path" construct-only
GObject properties.

This eliminates e-cal-client-view-private.h and adds the following
public accessor functions:

   e_cal_client_view_get_connection()
   e_cal_client_view_get_object_path()

EBookClientView cleanups.

Let EBookClientView create its own D-Bus proxy object by implementing
GInitableIface and adding "connection" and "object-path" construct-only
GObject properties.

This eliminates e-book-client-view-private.h and adds the following
public accessor functions:

   e_book_client_view_get_connection()
   e_book_client_view_get_object_path()

Fix EBookClient property fetching crack.

Post-release version bump.

NEWS update for 3.7.2 release.

Missed a couple files when removing EBookBackendVCF.

Remove EBookBackendVCF.

Strolling through the commit history of this backend, I had to dig all
the way back to 2005 before I started seeing actual development work.
Since then the commits have all been code cleanups and adapting it to
backend API changes.

There is no way to create a VCF address book from Evolution, which means
this entire backend is dead code and we've been wasting our time with it.

Dump it.

Deprecate BOOK_BACKEND_PROPERTY_SUPPORTED_AUTH_METHODS.

This property is not used anywhere.

Remove all references to it from built-in backends.

The following EBook functions now simply return an empty list:

   e_book_get_supported_auth_methods
   e_book_async_get_supported_auth_methods
   e_book_get_supported_auth_methods_async

Bug #674454 - Crash in e_book_backend_ldap_authenticate_user()

Replace deprecated Gtk+ symbols (as of Gtk+ 3.6.x)

Bug #688146 - Crash in Contacts calendar backend

Bug #687634 - Improve camel console warning text

Bug #687865 - Shows old reminders for GOA accounts

Revert "EBookBackendGoogle: Make room for other authorizers."

This reverts commit a5dae1a780f898ab453f574440ca02e108b4ebef.

Actually no, don't want to do that.  Misread the code.

EBookBackendGoogle: Make room for other authorizers.

Update German translation

EDataBookFactory: Remove the GOA-specific bits.

Defer to the gnome-online-accounts module.  See the previous commit.

gnome-online-accounts: Add an EDataFactory extension.

This extension takes over for the GOA bits in EDataBookFactory.

Install this extension to both the addressbook and calendar backend
directories, even though only EBookBackendGoogle uses it at present.

EDataFactory: Add a "backend-created" signal.

Allows EDataFactory extensions to catch newly-created backends and
perform additional initialization on it.

The use case I have in mind is to move all the GOA initialization in
EDataBookFactory to the gnome-online-accounts module as an extension.

Note this uses one of the reserved pointer slots in EDataFactoryClass
and therefore does not break ABI.

Remove unused OAUTH_CFLAGS / OAUTH_LIBS.

Missed these when I removed the liboauth dependency.

Coding style and whitespace cleanup.

Updated POTFILES.in

EBookBackendSqliteDB cleanups.

Improve error handling and make commits/rollbacks less awkward.

Use g_type_ensure() where applicable.

CamelIMAPXServer: Work around crash in imapx_exit_idle().

g_cond_clear() doesn't like being given an uninitialized GCond.

The "start_watch_cond" and "start_watch_exit" structures are being
cleared unconditionally, so we should initialize them unconditionally.

Updated Slovenian translation

Restore e_flag_timed_wait().

Just deprecate the function, don't delete it.  GTimeVal isn't going
away any time soon.  It's not worth a libedataserver soname bump.

Disabled debug print in EBookBackendFile

This was somehow missed in my last commit.

Local file backend modified to use sqlite db exclusively

Patch includes a migration routine to migrate data from a previously
installed BDB.

EBookBackendSqliteDB: Added e_book_backend_sqlitedb_get/set_revision()

In preperation for the migration, there needs to be a way to persist the
overall addressbook revision. This patch also increments the internal
SQLite DB format version and adds the new revision column to for main
folders table of an addressbook (and increments the stored DB version)
if the DB version is out of date.

EBookBackendSqliteDB: Fixed sexp interpretation for 'id' field.

In the sqlite backend, the 'id' is stored as the 'uid' column.

Revert "calendar: Drop old ECalSourceTypeEnum from introspected API"

This reverts commit 3183733023f022a89980cbf3f7292bde0bbd9954.

We no longer introspect deprecated API.  Remove the cruft.

Disable runtime check in source_registry_initable_init().

Rationale explained in the source code.

Bug 687694 - imapx_untagged_vanished() is inefficient

For uid->len = 318320, timing the loop yields:

g_list_append(): 60+ seconds
g_list_prepend(): 0.3 seconds

Bug #687642 - Crash under caldav_synch_slave_loop()

Bug #669082 - Workaround crash in imapx_command_copy_messages_step_done()

Google: allow writing X-GOOGLE-SYSTEM-GROUP-IDS

We expose GData system groups as an extended attribute with that
name, but if we found it in the EContact we would turn it into
an extended property. Instead, marshal the system group IDs into
Evolution categories, and then those into GData groups or system
groups.

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

Bug #687776 - E-D-S crash in local address book

Introduce --enable-strict configure option

This is currently used to add extra AM_CPPFLAGS to disable
deprecated symbols from GLib.

Updated Norwegian bokmål translation

Use soup_message_headers_get_list() as a replacement of soup_message_headers_get()

CalDAV backend didn't receive proper value for access right, which
caused reject of the login to the server.

Replace deprecated soup_message_headers_get()

Replace deprecated GLib symbols (as of GLib 2.34.x)

[l10n] Fixed German translation, bug #687562

EBookBackendSqliteDB: Add argument guards.

Updated Russian translation

vala: Remove unused metadata.

Exclude deprecated API from introspection.

Deprecated APIs were deprecated for a reason.  The last thing we
should be doing is enabling their continued use in other languages.

Henceforth, any symbols tagged with EDS_DISABLED_DEPRECATED will be
automatically excluded from introspection and language bindings.

Introspection cleanups.

Remove E_BOOK_DISABLE_DEPRECATED and E_CAL_DISABLE_DEPRECATED.

We already use EDS_DISABLE_DEPRECATED, these other definitions only
confuse matters.

Suppress g-ir-scanner warnings.

They're far too noisy, and make it difficult to spot new C compiler
warnings, which are the only ones I care about.  We can enable them
again when we're serious about cleaning them up.  In the meantime,
they're just getting in the way.

calendar: Drop old ECalSourceTypeEnum from introspected API

It clashes when generating the vala bindings, and we're just keeping
it around for C API consumers.

e-cal-view.c: Remove unnecessary #includes.

e-cal.c: Remove unnecessary #includes.

e-book-view.c: Remove unnecessary #includes.

e-book.c: Remove unnecessary #includes.

Update libedataserverui API documentation.

Fix invalid Gtk-Doc syntax in comments.

Not certain this is everything, just what I could find.

Reimplement ECal using ECalClient.

This is primarily so I don't have to duplicate work while overhauling
the D-Bus API, but it's good to do anyway; it simplifies the deprecated
logic and helps ensure the two APIs behave consistently.

Reimplement ECalView using ECalClientView.

This is primarily so I don't have to duplicate work while overhauling
the D-Bus API, but it's good to do anyway; it simplifies the deprecated
logic and helps ensure the two APIs behave consistently.

Reimplement EBook using EBookClient.

This is primarily so I don't have to duplicate work while overhauling
the D-Bus API, but it's good to do anyway: it simplifies the deprecated
logic and helps ensure the two APIs behave consistently.

Reimplement EBookView using EBookClientView.

This is primarily so I don't have to duplicate work while overhauling
the D-Bus API, but it's good to do anyway: it simplifies the deprecated
logic and helps ensure the two APIs behave consistently.

e-cal-view.h cleanups.

e-book-view.h cleanups.

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.