In Thunderbird  45.0 with quickFilters enabled, the context menu in the message
pane (Move To > folder) does not work anymore. It does work if quickFilters is
disabled.

Apparently the wrapped function MsgMoveMessage is being called recursively. I
see this when I enable debugging with the switch:

extensions.quickfilters.debug.msgMove

I am getting an endless stream of log messages which looks to me like the same
procedure is called over and over again:


 {MSGMOVE} 12:47:37.548  [0 ms]   
 setTimeout(..) for final move / copy call
 ----------
quickFilters {MSGMOVE} 12:47:37.548  [0 ms]   
After original Move/CopyMessage.]]
 ----------
quickFilters {MSGMOVE} 12:47:38.21  [473 ms]   
Executing original MoveMessage [[
 ----------
quickFilters {MSGMOVE} 12:47:38.22  [1 ms]   
 quickFilters.executeMoveMessage == quickFilters.MsgMove_Wrapper :false
 ----------
quickFilters {MSGMOVE} 12:47:38.22  [0 ms]   
Executing wrapped MsgMoveMessage
 ----------
quickFilters {MSGMOVE} 12:47:38.22  [0 ms]   
 setTimeout(..) for final move / copy call
 ----------
quickFilters {MSGMOVE} 12:47:38.22  [0 ms]   
After original Move/CopyMessage.]]
 ----------
quickFilters {MSGMOVE} 12:47:38.473  [451 ms]   
Executing original MoveMessage [[
 ----------
quickFilters {MSGMOVE} 12:47:38.473  [0 ms]   
 quickFilters.executeMoveMessage == quickFilters.MsgMove_Wrapper :false
 ----------
quickFilters {MSGMOVE} 12:47:38.473  [0 ms]   
Executing wrapped MsgMoveMessage

After getting rid of the recall - apparently this function calls MsgMoveMessage
in the Thunderbird module mailWindowsOverlay.js

it passes is the destination folder correctly:

function MsgMoveMessage(aDestFolder)
{
  gFolderDisplay.hintAboutToDeleteMessages();
  gDBView.doCommandWithFolder(nsMsgViewCommandType.moveMessages, aDestFolder);
  Services.prefs.setCharPref("mail.last_msg_movecopy_target_uri",
aDestFolder.URI);
  Services.prefs.setBoolPref("mail.last_msg_movecopy_was_move", true);
}

the big question is why gDBView.doCommandWithFolder doesn't do anything...

After getting rid of the recall - apparently this function calls MsgMoveMessage
in the Thunderbird module mailWindowsOverlay.js

it passes is the destination folder correctly:

function MsgMoveMessage(aDestFolder)
{
  gFolderDisplay.hintAboutToDeleteMessages();
  gDBView.doCommandWithFolder(nsMsgViewCommandType.moveMessages, aDestFolder);
  Services.prefs.setCharPref("mail.last_msg_movecopy_target_uri",
aDestFolder.URI);
  Services.prefs.setBoolPref("mail.last_msg_movecopy_was_move", true);
}

the big question is why gDBView.doCommandWithFolder doesn't do anything...

Created an attachment (id=8171) [details]
quickFilters 3.2 Prerelease 3

This make the endless calling loop go away. Also, I think the move to is back
to working.

Created an attachment (id=8197) [details]
quickFilters 3.2 Prerelease 17

Just a newer version with some added bug fixes so we are all testing the same
things.

Fixed in Release 3.2