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

Bug #655190 - Sluggish performance interacting with calendar/tasks

Replace alarm by reminder for UI consistency. Fixes bug #340614

Bug 619135 — Add photo support to Google Contacts backend

Add support for getting and setting photos on contacts from Google Contacts,
including caching support.

Closes: bgo#619135

Updated Swedish translation

Bug #655499 - Don't crash when getting contacts before backend is opened

Updated Russian translation

Updated Spanish translation

Merge same backtraces into one e_pointer_tracker report

Be able to provide backtraces for e_ptr_tracker* functions

The e_ptr_tracker* functions from e-data-server-util.h track
pointers and reports those left in the queue at the end of
the application. With this change, when evolution-data-server
is configured with --enable-backtraces and required tools are
available, then also a backtrace when the pointer was added
to the queue is shown.

Thanks to Jan Kratochvil for his help with the dwfl part.

Introspection: state the element-type when using GSLists

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Introspection: fix annotation in e_client_get_capabilities

Fix stray unref of server in imapx add_folders_to_summary()

This was causing a double (well, multiple) free and use-after-free of the
server; it has no business here.

It was actually seen when a broken Yahoo server gave a namespace with NIL
for the dir_sep, thus causing problems with subfolders.

Make e-{addressbook,calendar}-factory supersede old factory at startup.

I've lost count of the number of times I've run a factory for debugging
purposes but actually discovered that there's another one already running,
so my new one isn't being used. It's particularly likely because when you
*kill* an existing factory Evolution will bitch about how calendars will
never work again until you restart Evolution... but it *will* restart the
factory automatically!

So make the new factory supersede an old one, and make the old one quit
when it's superseded. This will make debugging a whole lot saner.

Introspection: fix annotation in e_book_client_get_contacts_finish

Introspection: fix annotation in e_book_client_get_self

In the previous commit I forgot to annotate the fact that
the EBookClient param is an out param too.

Introspection: fix annotation in e_book_client_get_self

Add missing \n in a debug print

Introspection: fix annotation in e_book_client_get_contacts_sync

Bug #654893 - Fix one missed rename of 'msg_security' to 'dirty'

Bug #655414 - Need translation comments

Bug #655409 - Extra flags in CamelStoreInfo break folder type check

Bug #654472 - Crash on async error return over GDBus

Be able to fetch all vCards/uids from an SQLite book backend cache

Use same type for fields_of_interest arguments

The fields_of_interest argument of e_book_backend_sqlitedb_search()
was of a different type than the one provided by EDataBookView.

Return back camel_stream_reset() functionality where required

Bug #655331 - pgp/gpg signature is not shown

Bug 654480 - [imapx] Mark as Junk is not reliable

When comparing server-side message flags to local flags, disregard
non-permanent flags that appear to have been cleared on the server.

This solves the problem of IMAP servers not storing junk-related flags
permanently, then when we synchronize message flags with the server, our
local junk-related flags (which -are- stored permanently) get clobbered.

IMAPX: Delete CamelIMAPXViewSummary.

After spending nearly an hour trying to understand how this code is
used... only to realize it's not.

Post-release version bump.

NEWS update for 3.1.4 release.

Coding style and whitespace cleanups.

CamelPOP3Store cleanups.

Update API documentation.

Update POTFILES.in for camel-junk-filter.c.