Bug 4047 - Move/Copy received message to Sent inconsistencies/bug (incl. loss of msg)
Summary: Move/Copy received message to Sent inconsistencies/bug (incl. loss of msg)
Status: RESOLVED INVALID
Alias: None
Product: Claws Mail
Classification: Unclassified
Component: Folders (show other bugs)
Version: 3.17.0
Hardware: PC Linux
: P3 normal
Assignee: users
URL:
Depends on:
Blocks:
 
Reported: 2018-07-04 13:07 CEST by Pierre Fortin
Modified: 2018-07-04 15:42 CEST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Pierre Fortin 2018-07-04 13:07:17 CEST
1. right-click on a "received" message in a folder, and:
  a) move to Sent: silently discarded (vanishes) - bug
  b) copy to Sent: silently ignored - ok (info dialog disabled?)
2. Drag & drop: moved to Sent - wrong IMO

Received messages should never be silently/unexpectedly discarded, or allowed to be moved to Sent.
Comment 1 Pierre Fortin 2018-07-04 14:29:19 CEST
Further testing...  I run two instances of CM. One exhibits 1.a), the other does not.  A test CM also worksforme.  On the failing instance, the message count on the source folder decreases by one; but the Sent count does not change, and searching for the "moved" message returns nothing.
version 3.16.0git163

before move:
$ ll Mail/inbox/ClarkHoward/807
-rw------- 1 pfortin pfortin 19012 Aug 27  2007 Mail/inbox/ClarkHoward/807
$ grep "Unlimited calling" Mail/inbox/ClarkHoward/8??
Mail/inbox/ClarkHoward/807:Subject: Clark Howard: Unlimited calling for $40/year - Clark Said - 8/27/07
$

after "move":
$ ll Mail/inbox/ClarkHoward/807
ls: cannot access 'Mail/inbox/ClarkHoward/807': No such file or directory
$ grep "Unlimited calling" Mail/inbox/ClarkHoward/8?? 
$ grep "Unlimited calling" Mail/sent/*
$

--debug on the failing instance:
[select a message]:
procheader.c:155:generic_get_one_field: empty line
procheader.c:155:generic_get_one_field: empty line
procheader.c:155:generic_get_one_field: empty line
    message/rfc822 (offset:0 length:19012 encoding: 6)
        multipart/alternative (offset:1986 length:17026 encoding: 6)
            text/plain (offset:2172 length:6977 encoding: 3)
            text/html (offset:9289 length:9672 encoding: 3)
procheader.c:155:generic_get_one_field: empty line
messageview.c:1310:found part 6/alternative
procheader.c:178:generic_get_one_field: empty line
textview.c:714:TIMING textview_add_part : 0s000ms
textview.c:680:TIMING textview_add_part : 0s000ms
textview.c:1198:Viewing text content of type: plain (length: 6150)
textview.c:830:TIMING textview_add_part : 0s006ms
textview.c:847:TIMING recursive_add_parts : 0s006ms
textview.c:893:TIMING recursive_add_parts : 0s006ms
textview.c:893:TIMING recursive_add_parts : 0s007ms
textview.c:653:TIMING textview_show_part : 0s007ms
summaryview.c:3763:TIMING summary_display_msg_full : 0s013ms

[Move to Sent]:
summaryview.c:4686:called inc_lock (lock count 1)
hooks.c:91:unregistered hook 7 in 'msginfo_update'
summaryview.c:3592:flags: doing change now
procmsg.c:2019:Changing flags for message 807 in folder inbox/ClarkHoward
summaryview.c:4659:Message 807 is set to move to sent
summaryview.c:4694:called inc_unlock (lock count 0)
folder.c:1239:Counting total number of messages...
main.c:910:The name com.google.code.Awn was not provided by any .service files
msgcache.c:277:TIMING msgcache_get_msg_list : 0s000ms
folder.c:1239:Counting total number of messages...
main.c:910:The name com.google.code.Awn was not provided by any .service files
msgcache.c:277:TIMING msgcache_get_msg_list : 0s000ms
hooks.c:70:registered new hook for 'msginfo_update' as id 8
summaryview.c:5552:Unthreading for execution...
summaryview.c:5554:called inc_lock (lock count 1)
hooks.c:91:unregistered hook 8 in 'msginfo_update'
summaryview.c:5560:called inc_unlock (lock count 0)
hooks.c:70:registered new hook for 'msginfo_update' as id 9
summaryview.c:5562:Unthreading for execution done.
hooks.c:91:unregistered hook 9 in 'msginfo_update'
folder.c:3703:called inc_lock (lock count 1)
mh.c:236:MH scan not required: /home/pfortin/0_TestCases/sent (1530705875 <= 1530705875)
mh.c:236:MH scan not required: /home/pfortin/Mail/inbox/ClarkHoward (1530706309 <= 1530706309)
mh.c:559:Copying message inbox/ClarkHoward/807 to sent ...
msgcache.c:277:TIMING msgcache_get_msg_list : 0s000ms
mh.c:1442:TIMING mh_write_sequences : 0s000ms
mh.c:1474:MH: forced mtime of sent to 1530706868
mh.c:1474:MH: forced mtime of ClarkHoward to 1530706868
mh.c:236:MH scan not required: /home/pfortin/Mail/inbox/ClarkHoward (1530706868 <= 1530706868)
mh.c:1474:MH: forced mtime of ClarkHoward to 1530706868
msgcache.c:197:Cache size: 837 messages, 399440 bytes
procheader.c:155:generic_get_one_field: empty line
msgcache.c:173:Cache size: 11 messages, 4880 bytes
procmsg.c:2019:Changing flags for message 11 in folder sent
folder.c:3705:called inc_unlock (lock count 0)
hooks.c:70:registered new hook for 'msginfo_update' as id 10
hooks.c:91:unregistered hook 10 in 'msginfo_update'
hooks.c:70:registered new hook for 'msginfo_update' as id 11
folder.c:1239:Counting total number of messages...
main.c:910:The name com.google.code.Awn was not provided by any .service files
msgcache.c:277:TIMING msgcache_get_msg_list : 0s000ms
folder.c:1239:Counting total number of messages...
main.c:910:The name com.google.code.Awn was not provided by any .service files
msgcache.c:277:TIMING msgcache_get_msg_list : 0s000ms
procheader.c:155:generic_get_one_field: empty line
procheader.c:155:generic_get_one_field: empty line
procheader.c:155:generic_get_one_field: empty line
    message/rfc822 (offset:0 length:17072 encoding: 6)
        multipart/alternative (offset:1987 length:15085 encoding: 6)
            text/plain (offset:2173 length:5982 encoding: 3)
            text/html (offset:8295 length:8726 encoding: 3)
procheader.c:155:generic_get_one_field: empty line
messageview.c:1310:found part 6/alternative
procheader.c:178:generic_get_one_field: empty line
textview.c:714:TIMING textview_add_part : 0s000ms
textview.c:680:TIMING textview_add_part : 0s000ms
textview.c:1198:Viewing text content of type: plain (length: 5339)
textview.c:830:TIMING textview_add_part : 0s010ms
textview.c:847:TIMING recursive_add_parts : 0s010ms
textview.c:893:TIMING recursive_add_parts : 0s010ms
textview.c:893:TIMING recursive_add_parts : 0s011ms
textview.c:653:TIMING textview_show_part : 0s011ms
summaryview.c:3763:TIMING summary_display_msg_full : 0s034ms
summaryview.c:5467:Building threads...
summaryview.c:5516:Building threads done.
summaryview.c:5543:TIMING summary_thread_init : 0s000ms
Comment 2 Paul 2018-07-04 15:42:06 CEST
looks like you're looking for the message in the wrong place:

> $ grep "Unlimited calling" Mail/sent/*

> mh.c:236:MH scan not required: /home/pfortin/0_TestCases/sent (1530705875 <= 1530705875)