Ticket #2060 (closed defect: fixed)

Opened 3 years ago

Last modified 19 months ago

Copying files multiple times results in bogus names

Reported by: bernie Owned by: alsroot
Priority: Unspecified by Maintainer Milestone: Unspecified by Release Team
Component: journal Version: Unspecified
Severity: Unspecified Keywords: dextrose sugar-love dtst
Cc: aa, smparrish, bernie Distribution/OS: Unspecified
Bug Status: Assigned

Description

1. drag a file from the journal to a volume
2. drag it again and again
3. open the volume

The copies will be called foo, foo_1, foo_1_2, foo_1_2_3, and so on.

Change History

  Changed 3 years ago by alsroot

This is not a bug but a future :) which exists, afaik, for several sugar releases. I guess we need to discuss changing existed behaviour widely on sugar-devel@.

  Changed 3 years ago by bernie

  • keywords dextrose sugar-love added

  Changed 3 years ago by smparrish

  • cc smparrish added

I agree with alsroot. This is how things work in KDE and Gnome as well to prevent file clobbering.

  Changed 3 years ago by smparrish

  • seeta_dev set to Ishan

  Changed 3 years ago by sascha_silbe

  • cc bernie added

Copying CC list from #2453.

  Changed 3 years ago by bernie

My description of the problem was probably too vague.

This is the expected naming scheme: foo, foo_1, foo_2, foo_3, foo_4...

This is what we're doing instead: foo, foo_1, foo_1_2, foo_1_2_3, foo_1_2_3_4...

follow-up: ↓ 8   Changed 3 years ago by jtannenbaum

patch (untested):  http://pastebin.com/b5Km4EB4

note: first patch ever, expect catastrophic embarrassment

in reply to: ↑ 7   Changed 3 years ago by sascha_silbe

Replying to jtannenbaum:

patch (untested):  http://pastebin.com/b5Km4EB4

note: first patch ever, expect catastrophic embarrassment

Thanks for the patch! However there already is a  patch for this issue, it just needs a clean-up of the authorship.

  Changed 3 years ago by smparrish

  • keywords dst added
  • status_field changed from Unconfirmed to Assigned

This patch has been included in os5dx2 release of Dextrose

  Changed 3 years ago by smparrish

  • keywords dtst added; dst removed

follow-up: ↓ 12   Changed 19 months ago by godiard

The patch was already commited in Nov 24 2010 569c713e8300da659234b2192c78a1343f032144

Can we close this ticket?

in reply to: ↑ 11   Changed 19 months ago by bernie

  • status changed from new to closed
  • resolution set to fixed

Replying to godiard:

The patch was already commited in Nov 24 2010 569c713e8300da659234b2192c78a1343f032144

Can we close this ticket?

Sure. When you feel like closing a bug, you can just go ahead and do it. If the reporter or owner disagree, they can reopen it later.

Note: See TracTickets for help on using tickets.