Post-release version bump.

NEWS update for 3.1.90 release.

Update API documentation.

[l10n] Updated German translation

Add E_TYPE_CAL_CLIENT_SOURCE_TYPE macro.

Check ic->status is not NULL imapx_command_select_done ()

There are ocassions (are they even valid?) in which ic->status
might be NULL, so we check for that before trying to access
ic->status->text when calling g_set_error ().

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

Use cached value in e_cal_client_get_default_object()

Avoid use-after-free by unsetting open_finished flag sooner

Updated Russian translation

Updated Spanish translation

Updated Galician translations

Camel: Fix linking issue.

camel-enumtypes.c is linked into libcamel.so, but is needed by
camel-network-settings.c which is part of libcamel-provider.so,
and libcamel-provider.so is lower in the dependency chain.

Move camel-enumtypes.c into libcamel-provider.so.

ed. I don't understand why libcamel-provider.so even exists as a
    separate library, unless there's some hidden circular dependency
    I haven't tripped over yet.  Tempted to merge it into libcamel.so
    and see how it goes...

Updated Spanish translation

Updated Norwegian bokmål translation

Add Google Talk field to the list of IM information

Bug #657181 - Backends can be left half-opened when using ECal

Updated Norwegian bokmål translation

Bug #652914 - IMAP: moving mail puts a copy into real Trash too

Also fixes issue when CamelSettings are populated too late for
the initable.

Updated Swedish translation

Avoid a crash in gdata_goa_authorizer_get_parameters().

I had assumed the SoupURI always has a query part, which is wrong,
and soup_form_decode() chokes on NULL strings.

Updated Serbian translation

Bug #655997 - Unneeded context specifiers

Remove more similar strings from translation

I forgot of these in the previous commit.

Bug #651849 - Remove string from translation

Bug #576398 - vfolder not showing new messages from nntp group

Bug #562912 - Unread vfolder marks unread messages as read

Bug #652437 - NNTP messages is sometimes displayed as grey

[IMAPX] Do not overwrite errors on reconnect

Pretend successful read when skipping 3+ part of multipart/signed

Incorrect message signature verification failure

Updated Bulgarian translation

Updated Bulgarian translation

Updated Indonesian translation

Bug #651693 - Decode QP encoded names when invoking 'Expand list Inline'

Updated Dutch translation by Wouter Bolsterlee

Updated Dutch translation by Wouter Bolsterlee

google: Fix various data loss bugs with organisations/jobs

For example:
 • Failing to save if an organisation department isn't entered (but a
   company name is).
 • Losing organisation department if no company name is entered.
 • Losing all details except title and role from the primary organisation.

Closes: bgo#651451

Added UG translation

Bug #651469 - Folders don't update after moving mails in maildir

Camel: Add CAMEL_STORE_CAN_EDIT_FOLDERS flag.

Basically for the NNTP provider to clear, or I suppose if you're
proxying someone else's account.  Evolution can check for this flag
and disable folder create/delete/rename actions in the context of a
newsgroup server.

Related to bug 656720.

CamelService: Add a "display-name" property.

Mostly as a convenience to clients.  Camel itself doesn't use this for
anything at present, although I suppose it could come in handy for some
error messages or user prompts.

Don't use our own DISABLE_DEPRECATED macros in .c files.

Those macros are meant for E-D-S -client- programs, not for E-D-S
itself.  As such, they should only appear in public header files.

google: Tidy up finish_operation()

Merge on_sequence_complete() into it and ensure that all errors get
propagated. See: bgo#619135

google: Fix cancellation and progress reporting for cold-cache queries

See: https://bugzilla.gnome.org/show_bug.cgi?id=619135#c3

Closes: bgo#619135

Bug #656051 - Preserve mbox file attributes after rewrite

Bug #655272 - IMAPX: Leaking file descriptors from open pipes

Bug #655121 - CalDAV: Can't reopen attachments saved on a server

Do not keep Google progress notifications in book view forever

Bug 655833 — Mailing addresses for Google Contacts not saving if PO Box…

Fix editing postal addresses of contacts where the PO box field is not
filled in.

Closes: bgo#655833

Updated Spanish translation

google: Add a missing NULL pointer check

This fixes a crash when editing contacts without avatars (and not adding an
avatar).

Updated Norwegian bokmål translation.

Updated Galician translations

Camel: Remove unused CAMEL_STORE_ASYNC flag.

CamelStore: Split subscription API into a separate interface.

The CAMEL_STORE_SUBSCRIPTIONS flag basically means the CamelStore
instance supports the folder subscription portion of its API.  This
would be better handled by splitting the subscription portion of its
API into a separate GTypeInterface for providers to implement.  Then
we don't need the subscriptions flag.

The CamelSubscribable interface does exactly that.  To check if a
CamelStore supports folder subscriptions, just do:

   if (CAMEL_IS_SUBSCRIBABLE (store)) ...

Implement CamelSubscribable in the IMAP, IMAPX and NNTP providers.

Add CamelSmtpSettings.

CamelSmtpSettings replaces the various URL parameters used in
CamelSmtpTransport with equivalent GObject properties.

Adapt the smtp provider to use CamelSmtpSettings.

Add CamelPOP3Settings.

CamelPOP3Settings replaces the various URL parameters used in
CamelPOP3Store with equivalent GObject properties.

Adapt the pop3 provider to use CamelPOP3Settings.

Add CamelNNTPSettings.

CamelNNTPSettings replaces the various URL parameters used in
CamelNNTPStore with equivalent GObject properties.

Adapt the nntp provider to use CamelNNTPSettings.

Add CamelSpoolSettings.

CamelSpoolSettings replaces the various URL parameters used in
CamelSpoolStore with equivalent GObject properties.

Adapt the spool provider to use CamelSpoolSettings.

Add CamelMhSettings.

CamelMhSettings replaces the various URL parameters used in CamelMhStore
with equivalent GObject properties.

Adapt the mh provider to use CamelMhSettings.

CamelLocalStore: Add a "need-summary-check" property.

Use g_object_set() instead of camel_url_set_param() to change the
need-summary-check flag common to all local store types.

Add CamelIMAPXSettings.

CamelIMAPXSettings replaces the various URL parameters used in
CamelIMAPXStore with equivalent GObject properties.

Adapt the imapx provider to use CamelIMAPXSettings.

Add CamelImapSettings.

CamelImapSettings replaces the various URL parameters used in
CamelImapStore with equivalent GObject properties.

Adapt the imap provider to use CamelImapSettings.

Add CamelNetworkSettings.

Move the "security-method" property from CamelNetworkService to
CamelNetworkSettings.  If a given CamelService subclass implements
CamelNetworkService, its corresponding CamelSettings subclass should
implement CamelNetworkSettings.

Add CamelOfflineSettings.

CamelOfflineSettings replaces the "sync_offline" URL parameter used in
CamelOfflineStore and the "offline_sync" URL parameter used in
CamelDiscoStore with a "stay-synchronized" GObject boolean property.

Adapt various providers to use CamelOfflineSettings.

Add CamelStoreSettings.

CamelStoreSettings replaces the "filter" URL parameter used in
CamelStore with a with a "filter-inbox" GObject boolean property.
Also remove CAMEL_STORE_FILTER_INBOX flag since it's now redundant.

Adapt various providers to use CamelStoreSettings.

Add CamelSettings.

CamelSettings is an abstract base class for holding settings values for
various CamelService subclasses.  In general each CamelService subclass
should define a custom CamelSettings subclass.  It's critical that all
settings values have a corresponding GObject property.

Camel: Fix enumeration regressions.

post release version bump

News update for 3.1.5 release

Bug 656490 - imapx: Memory leak of stream tokenbuf

Bug 656487 - Memory leak in imapx fetch_folders_for_namespaces()

We need to free the GSList but the namespaces themselves are fine.

Bug #656480 - Memory leak in camel_folder_search_search()

We don't unref sexp on failure.

Updated Spanish translation

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Coding style and whitespace cleanup.

Update API docs.

Bug #656267 - Checking for auth types doesn't work properly

libecal: e_cal_remove_object() must remove *all* recurrences

Traditionally, e_cal_remove_object() has always removed all
recurrences, despite the use of MOD_THIS underneath. That was due to
the uncertain semantic of MOD_THIS without rid.

Since clarifying that semantic and fixing the (file) backend
accordingly, e_cal_remove_object() started to behave differently: of
an event series with detached recurrences, only the parent event was
removed, which then caused the failures fixed by the previous commits.

This commit fixes that by switching to MOD_ALL, which properly
reflects the semantic of the API call. It was tested successfully with
the file backend.

calendar file backend: fixed incomplete sanity check in e_cal_create_object()

e_cal_create_object() traditionally is used for a new events which are
unrelated to anything in the calendar. Adding detached recurrences
to an existing meeting series has to be done with e_cal_modify_object().

The code did not correctly reject the addition of a parent event for a
previously added child event event because lookup_component() returned
NULL in that case.

This commit renames lookup_component() and redefines the return value
to match what it is used for: checking for the existance of a UID.

ecal file backend: avoid manipulating the UID inside component_add()

This commit fixes the following memory handling problem:
==10069== Invalid read of size 1
==10069==    at 0x4C25812: __GI_strlen (mc_replace_strmem.c:284)
==10069==    by 0x8EF011E: g_strdup (gstrfuncs.c:99)
==10069==    by 0xF4E08B6: e_cal_backend_file_create_object (e-cal-backend-file.c:2363)
==10069==    by 0x93E6061: e_cal_backend_sync_create_object (e-cal-backend-sync.c:214)
==10069==    by 0x93E86D3: _e_cal_backend_create_object (e-cal-backend-sync.c:630)
==10069==    by 0x93DD40B: e_cal_backend_create_object (e-cal-backend.c:1017)
==10069==    by 0x93F0C34: impl_Cal_createObject (e-data-cal.c:401)
==10069==    by 0x4E75383: _e_gdbus_gdbus_cclosure_marshaller_BOOLEAN__OBJECT_STRING (e-gdbus-marshallers.c:377)
==10069==    by 0x820999E: g_closure_invoke (gclosure.c:773)
==10069==    by 0x8225972: signal_emit_unlocked_R (gsignal.c:3256)
==10069==    by 0x82248D0: g_signal_emit_valist (gsignal.c:2997)
==10069==    by 0x8224DBC: g_signal_emit (gsignal.c:3044)
==10069==  Address 0x1499c7b0 is 0 bytes inside a block of size 39 free'd
==10069==    at 0x4C240FD: free (vg_replace_malloc.c:366)
==10069==    by 0x9DE952C: icalvalue_free (in /usr/lib/libical.so.0.44.0)
==10069==    by 0x9DDB796: icalproperty_set_value (in /usr/lib/libical.so.0.44.0)
==10069==    by 0x4E4FFA2: e_cal_component_set_uid (e-cal-component.c:1479)
==10069==    by 0xF4DB8F3: check_dup_uid (e-cal-backend-file.c:498)
==10069==    by 0xF4DBD9B: add_component (e-cal-backend-file.c:614)
==10069==    by 0xF4E0894: e_cal_backend_file_create_object (e-cal-backend-file.c:2356)
==10069==    by 0x93E6061: e_cal_backend_sync_create_object (e-cal-backend-sync.c:214)
==10069==    by 0x93E86D3: _e_cal_backend_create_object (e-cal-backend-sync.c:630)
==10069==    by 0x93DD40B: e_cal_backend_create_object (e-cal-backend.c:1017)
==10069==    by 0x93F0C34: impl_Cal_createObject (e-data-cal.c:401)
==10069==    by 0x4E75383: _e_gdbus_gdbus_cclosure_marshaller_BOOLEAN__OBJECT_STRING (e-gdbus-marshallers.c:377)

This occurs when a client (incorrectly) tries to create a VEVENT with
RECURRENCE-ID for a UID which already exists. The sequence of events is this:
- e_cal_backend_file_create_object() calls lookup_component(),
  which returns NULL because it only checks for the parent event
  (will be fixed separately).
- e_cal_backend_file_create_object() keeps a pointer to the UID.
- check_dup_uid() repeats the UID check, but this time finds that it
  is already taken and replaces the existing UID in the component
  before adding it. The pointer in e_cal_backend_file_create_object()
  points to freed memory.

I've seen cases where the hash ended up using the original UID as key,
with a component inside that had the new, replaced UID. As a result,
retrieving the event as reported by e_cal_get_object_list() (= rewritten UID)
failed in e_cal_get_object() (= original UID).

The UID should not be overwritten. I can't verify it anymore (events where it occured
have already been deleted), but this rewriting might explain why some of my
meeting update emails couldn't be applied to previously imported events.

Therefore this patch moves check_dup_uid() out of component_add(). This check
and rewriting only makes sense when reading the existing calendar file,
as a safe-guard against on-disk corruption. When adding or modifying events
via the API, the right reaction is to add a missing UID or or reject the
operation with an error.

All places where component_add() is used should have the necessary checks
or are preceeded by a remove_component(), which removes the UID first.

Updated Norwegian bokmål translation

complete_get_object_master: Fix the NULL recurrence-id check

The check was checking that the recurrence-id was valid instead
of making sure that it is NULL (opposite).

Add a bit more type checking in ESourceList

e_client_utils_open_new() could use already freed memory

Backends can notify that they are "opened" multiple times, but only
the first notification is significant for e_client_utils_open_new().

Bug #656058 - Improve EBookBackendSqliteDB searching API

Updated Galician translations

Properly manage boolean values in SQLite contacts cache

Vala bindings: work around owned not being honored in metadata

Works around: https://bugzilla.gnome.org/show_bug.cgi?id=655996

Bug #652172 - Fix a regression from the previous commit

Call gtk_init_check() in factories with enabled maintainer mode

That's to have loaded gtk-modules, like gnomesegvhandler
from bug-buddy, if it's possible. Factories can still be
run without gtk environment.

Add missing G_BEGIN_DECLS macro to libedataserver/e-client.h

Bug #655748 - rdate parsing failure: unknown value for period 20068

Updated Swedish translation

Adding test-client-revision-view.c

This test asserts that e_book_client_view_set_fields_of_interest()
is working properly with the local addressbook backend with regards
to views setup to only notify with the UID+REVISION.

Added test-client-uid-only-view.c test case.

This test case ensures that an EBookClientView with
the e_book_client_view_set_feilds_of_interest() set to
only the E_CONTACT_UID field, notifies with shallow
vcards holding only the contact UID.

Handle fields-of-interest for local addressbook backend.

This patch refactors the local addressbook backend to use
the new sqlitedb api instead of the old summary apis.
The result is that vcards are virtually built from the
sqlitedb cache when fields-of-interest is set.

Additionally, the patch adds the EBookBackend->notify_update()
vfunc which can be implemented for e_book_backend_notify_update().
The file backend uses this to notify with virtually created vcards
from the sqlite cache in response to contact additions and modifications.

This patch should address both bugs:
  https://bugzilla.gnome.org/show_bug.cgi?id=652179
  https://bugzilla.gnome.org/show_bug.cgi?id=652172

Handle summary fields and fields of interest better in e-book-backend-sqlitedb.c

This patch dramatically changes the sqlitedb cache code by introducing
a table (array of structures) describing all of the fields which should
be included in the (summary) cache. Thus, all code that treats the
summary fields by hand previously now consults the cache generically.

The REV field is added to the summary table, the UID is always returned
in any results from e_book_backend_sqlitedb_search() and when
'fields_of_interest' is specified then the sqlite3 db will only
be queried for the fields_of_interest + UID (thus only those fields
will be present in any virtually created vcard objects).

Additionally, e_book_backend_sqlitedb_get_vcard_string() and _get_contact()
take a new 'GHashTable *fields_of_interest' argument for field filtering and
e_book_backend_sqlitedb_is_summary_query() is an exported api which can be
tested before calling e_book_backend_sqlitedb_get_vcard_string().

Do not cancel cancellable in free_get_objects_async_data()

Introspection: fix annotation in e_vcard_remove_attributes