Ticket #3149 (accepted defect)

Opened 3 years ago

Last modified 11 days ago

"compute totals" unnecessarily done for moves

Reported by: ossi Owned by: andrew_b
Priority: major Milestone: 4.8.20
Component: mc-core Version: master
Keywords: Cc: egmont, mooffie
Blocked By: Blocking:
Branch state: on review Votes for changeset:

Description

mc will compute the totals for a directory move as if it was a copy. this is obviously tremendously pointless if the move can be done cleanly with a single rename() call, and an annoying waste of time if the directory has a deep substructure.

i presume this is the result of #2075.

consequently the computation should be aware of the operation it targets.
related ticket: #20

Change History

comment:1 Changed 3 years ago by andrew_b

Ticket #3185 has been marked as a duplicate of this ticket.

comment:2 Changed 2 years ago by andrew_b

Ticket #3395 has been marked as a duplicate of this ticket.

comment:3 Changed 13 months ago by andrew_b

Ticket #3653 has been marked as a duplicate of this ticket.

comment:4 Changed 12 months ago by andrew_b

  • Owner set to andrew_b
  • Status changed from new to accepted

Branch: 3149_move_compute_total
Initial changeset:8cfd12d150910eb1ab73fc0c6fabf41fa92e4f9c

Last edited 6 weeks ago by andrew_b (previous) (diff)

comment:5 Changed 12 months ago by andrew_b

  • Branch state changed from no branch to on review
  • Milestone changed from Future Releases to 4.8.18

comment:6 Changed 3 months ago by andrew_b

  • Branch state changed from on review to on rework

comment:7 Changed 6 weeks ago by andrew_b

  • Branch state changed from on rework to on review

comment:8 Changed 11 days ago by andrew_b

Does anybody want review this branch after 4 weeks?

comment:9 Changed 11 days ago by zaytsev

  • Cc egmont, mooffie added

I'm completely overloaded @ $DAYJOB, so no chance I can get to it anytime soon :-/ Maybe egmont or mooffie? Or even the reporter?

Note: See TracTickets for help on using tickets.