498914 - moving imap folders generating gloda assertions

Status: RESOLVED FIXED

(MailNews Core :: Database, defect)

Description

[David :Bienvenu]

Standard8, in bug 496905, which involves moving imap folders within a server, reports :

I am seeing various gloda assertions such as:

************************************************************
* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "'[JavaScript Error: "srcURI is not defined" {file:
"file:///Users/moztest/comm/main/tb/mozilla/dist/ShredderDebug.app/Contents/MacOS/modules/gloda/indexer.js"
line: 2176}]' when calling method:
[nsIMsgFolderListener::folderMoveCopyCompleted]"  nsresult: "0x80570021
(NS_ERROR_XPC_JAVASCRIPT_ERROR_WITH_DETAILS)"  location: "<unknown>"  data:
yes]
************************************************************

and

************************************************************
* Call to xpconnect wrapped JSObject produced this error:  *
[Exception... "Component returned failure code: 0xc1f30001
(NS_ERROR_NOT_INITIALIZED) [mozIStorageStatement.bindInt64Parameter]" 
nsresult: "0xc1f30001 (NS_ERROR_NOT_INITIALIZED)"  location: "JS frame ::
file:///Users/moztest/comm/main/tb/mozilla/dist/ShredderDebug.app/Contents/MacOS/modules/gloda/datastore.js
:: gloda_ds_markMessagesDeletedByFolderID :: line 2220"  data: no]
************************************************************

I didn't see this for the case of dragging a single folder around, but when I moved a hierarchy, I did see these (actually, just the bindInt64 one...)

Comment 1

[David :Bienvenu]

Attachment: proposed fix

I don't know if gloda does the right thing on folder move/deletion now, but at least it has a fighting chance :-)

Comment 2

[David :Bienvenu]

fix checked in.