Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MC deletes the wrong file #2736

Closed
mc-butler opened this issue Feb 21, 2012 · 10 comments
Closed

MC deletes the wrong file #2736

mc-butler opened this issue Feb 21, 2012 · 10 comments
Assignees
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress
Milestone

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/2736
Reporter nfl (nfl-mcbugreport@….com)

Based on https://bugs.launchpad.net/ubuntu/+source/mc/+bug/897942:

What version of Midnight Commander is used?
4.8.1

What steps will reproduce the problem?
1, Switch off delete confirmation.
2, Go to a directory with two files: "junk" and "vital".
3, Outside MC, delete "junk".
4, In MC, press F8 on "junk".

What is the expected output?
MC presents some error message that "junk" does not exist.

What do you see instead?
MC deletes "vital".

If you enable delete confirmation then MC asks if you want to delete "vital", but people don't expect such a tricky question.

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 23, 2012 at 9:30 UTC (comment 1)

  • Owner set to andrew_b
  • Milestone changed from Future Releases to 4.8.2
  • Keywords set to stable-candidate
  • Status changed from new to accepted
  • Branch state changed from no branch to on review

Branch: 2736_dont_panel_reload (parent: master).
[19528d71030e0daddb9376d6a0d834a6c030af25]

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Feb 25, 2012 at 12:29 UTC (comment 2)

  • Votes set to angel_il

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Feb 28, 2012 at 10:04 UTC (comment 3)

  • Branch state changed from on review to approved
  • Votes changed from angel_il to angel_il slavazanko

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Feb 28, 2012 at 16:46 UTC (comment 4)

  • Resolution set to fixed
  • Status changed from accepted to testing
  • Votes changed from angel_il slavazanko to committed-master
  • Branch state changed from approved to merged

Merged to master: [ebacf41].

@mc-butler
Copy link
Author

Changed by egmont (@egmontkob) on Mar 6, 2012 at 21:56 UTC (comment 5)

This seems to be a quite serious bug. Could you guys please backport this patch to the soon-to-be-stable 4.8.1 branch? Thanks a lot!

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Mar 6, 2012 at 22:13 UTC (comment 6)

Of course, patch will be applied to stable branch in any way. This ticket have keyword 'stable candidate'; this mean, before next stable release we will search tickets by this keyword and will apply all patches to stable branch (also, ticket will change state to 'closed' from 'testing').

@mc-butler
Copy link
Author

Changed by egmont (@egmontkob) on Mar 7, 2012 at 7:51 UTC (comment 7)

Thanks for the clarification, I didn't notice there was such a 'stable candidate' keyword in this bugtracker.

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Mar 7, 2012 at 8:32 UTC (comment 8)

  • Keywords stable-candidate deleted
  • Status changed from testing to closed

appied to stable:
[2d9e2bb]

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Mar 7, 2012 at 8:34 UTC (comment 9)

  • Votes changed from committed-master to committed-master committed-stable

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Mar 8, 2012 at 15:44 UTC (comment 10)

Quick fixes:
master: [da00fa0]
4.8.1-stable: [0932051]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues not related to a specific subsystem prio: medium Has the potential to affect progress
Development

No branches or pull requests

2 participants