notmuch_message_get_filename: Support old-style filename storage.

When a notmuch database is upgraded to the new database format, (to
support file rename and deletion), any message documents corresponding
to deleted files will not currently be upgraded. This means that a
search matching these documents will find no filenames in the expected
place.

Go ahead and return the filename as originally stored, (rather than
aborting with an internal error), in this case.
This commit is contained in:
Carl Worth 2010-01-07 09:22:34 -08:00
parent 1a38cb841c
commit a274848f95

View file

@ -441,7 +441,19 @@ notmuch_message_get_filename (notmuch_message_t *message)
if (i == message->doc.termlist_end () || if (i == message->doc.termlist_end () ||
strncmp (direntry, prefix, prefix_len)) strncmp (direntry, prefix, prefix_len))
{ {
INTERNAL_ERROR ("message with no filename"); /* A message document created by an old version of notmuch
* (prior to rename support) will have the filename in the
* data of the document rather than as a file-direntry term. */
const char *data;
data = message->doc.get_data ().c_str ();
if (data == NULL)
INTERNAL_ERROR ("message with no filename");
message->filename = talloc_strdup (message, data);
return message->filename;
} }
direntry += prefix_len; direntry += prefix_len;