geary:wip/768431-forwarded-in-conversation

Last commit made on 2016-07-31
Get this branch:
git clone -b wip/768431-forwarded-in-conversation https://git.launchpad.net/geary

Branch merges

Branch information

Name:
wip/768431-forwarded-in-conversation
Repository:
lp:geary

Recent commits

852e814... by Michael Gratton

WIP implementation to include forwarded messages in conversation.

There's two parts to this: First, add a References header to the
forwarded messages for the conversation they are a part of. Second, check
for matching references when querying for related messages, in addition
to in-reply-to.

This would all be well and good, except that since the References header
is a list of message ids, but it is stored in the DB as a single string,
the query has to be a substring match, which is horrifically slow.

This WIP adds an index, but it doesn't help, since SQLite only uses the
index if using GLOB or case-insensitiive ILIKE, and only if there is no
wild card at the start of the string, which we need to do. So the correct
solution would be to split the References header into its own table.

Bug 768431

* sql/CMakeLists.txt: Add new SQL migration script.

* sql/version-025.sql: Create index for MessageTable.reference_ids.

* src/client/composer/composer-widget.vala
  (ComposerWidget::ComposerWidget): Add references to outoging forwarded
  messages.

* src/engine/imap-db/imap-db-account.vala
  (Geary.ImapDB.Account::search_message_id_async): Include references in
  when searching for messages.

6aacb75... by Michael Gratton

Fix infinite loop when sending long space-stuffed line. Bug 768642.

* src/client/util/util-webkit.vala (Util.DOM.html_to_flowed_text): If a
  line is space-stuffed, ignore the stuffing space when determining where
  to break the line.

83955b3... by Michael Gratton

Fix crash when parsing unknown accout info service provider. Bug 713985.

* src/engine/api/geary-service-provider.vala
  (ServiceProvider::from_string): Throw an error rather than an assert
  when the provided string is not found.

82faf2c... by Michael Gratton

Fix RTL email not being shown as RTL in conversation viewer. Bug 713062.

* theming/message-viewer.html: Set dir="auto" on template for email bodies.

37dde52... by Balázs Meskó

Updated Hungarian translation
(cherry picked from commit ccdc94cef121ddaddbbdc29d881394b3969cace5)

50a4cfd... by Michael Gratton

Add Enchant to build dependencies.

557a9e7... by Michael Gratton

Fix crash when replying to email with null from address.

15e6877... by Michael Gratton

Fix critical when email without any from addresses loaded from DB.

See bbf5f07 and Bug 768468.

8f7ff57... by Michael Gratton

Fix error loading SMTP and IMAP host names introduced by a59a399.

a59a399... by Michael Gratton

Don't try to load accounts with missing or broken config files. Bug 714044.

* src/engine/api/geary-account-information.vala (AccountInformation):
  Move settings defaults for all properties to their declarations. Add a
  new internal ctor for creating new accounts that does not attempt to
  load from the config file. Update AccountInformation.from_file to
  actually throw an error if one occurs.

* src/engine/api/geary-engine.vala (Engine::add_existing_accounts_async):
  Catch error loading config and print a warning rather than adding the
  account.
  (Engine::create_orphan_account): Use new ctor for creating a blank
  account.