View Issue Details

IDProjectCategoryView StatusLast Update
0002087Far ManagerCopy/Movepublic2012-05-17 14:22
ReporterMaximusAssigned To2useven10 
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version3.0 
Target VersionFixed in Version3.0 
Summary0002087: Некорректная обработка Esc в диалоге ошибки копирования
DescriptionПри ошибке копирования
«The process cannot access the file because it is being used by another process»
нажатие Esc приводит к ПРОДОЛЖЕНИЮ копирования.
С какой радости?

BTW, что значит код возврата "-2" для Message?
TagsNo tags attached.
Build2637

Activities

Maximus

2012-04-25 13:40

reporter  

120425.diff (388 bytes)
Index: copy.cpp
===================================================================
--- copy.cpp	(revision 7943)
+++ copy.cpp	(working copy)
@@ -2775,12 +2775,12 @@
 
 			switch (MsgCode)
 			{
-				case -1:
 				case  1:
 					return COPY_NEXT;
 				case  2:
 					SkipMode=1;
 					return COPY_NEXT;
+				case -1:
 				case -2:
 				case  3:
 					return COPY_CANCEL;
120425.diff (388 bytes)

2useven10

2012-04-25 14:16

developer   bugnote:0009049

-2 это Ctrl-Break

2useven10

2012-04-25 14:45

developer   bugnote:0009050

Last edited: 2012-04-25 15:04

View 3 revisions

2 Maximus:
Esc == [Skip]. т.е. отменяется копирование текущего файла.
Ctrl-Break == [Cancel] - отменяет весь процесс копирования.
Твой патч делает их равными и для [Skip] нет удобного хоткея.

add:
imho такое же поведение надо сделать и при отработке предыдущего диалога
(File already exist) а то явная непоследовательность...

Maximus

2012-04-25 15:20

reporter   bugnote:0009051

1. Для Skip - удобная кнопка S
2. Когда юзер видит в диалоге кнопку "Cancel", при нажатии Esc он ожидает выполнения "Cancel". Предлагать что-либо другое - интуитивно непонятно, и более того - обломно.
3. спорить не буду, в bis исправил, а вы как хотите

2useven10

2012-04-25 15:38

developer   bugnote:0009052

посмотрел код...
это единственное место приблизительно из полутора десятков, где Esc != Ctrl-Break.
так что проще изменить его.

2useven10

2012-04-25 22:40

developer   bugnote:0009057

2637

Issue History

Date Modified Username Field Change
2012-04-25 13:40 Maximus New Issue
2012-04-25 13:40 Maximus File Added: 120425.diff
2012-04-25 14:16 2useven10 Note Added: 0009049
2012-04-25 14:45 2useven10 Note Added: 0009050
2012-04-25 14:52 2useven10 Note Edited: 0009050 View Revisions
2012-04-25 15:04 2useven10 Note Edited: 0009050 View Revisions
2012-04-25 15:20 Maximus Note Added: 0009051
2012-04-25 15:38 2useven10 Note Added: 0009052
2012-04-25 22:40 2useven10 Note Added: 0009057
2012-04-25 22:40 2useven10 Assigned To => 2useven10
2012-04-25 22:40 2useven10 Status new => feedback
2012-05-17 14:22 2useven10 Build => 2637
2012-05-17 14:22 2useven10 Status feedback => closed
2012-05-17 14:22 2useven10 Resolution open => fixed
2012-05-17 14:22 2useven10 Fixed in Version => 3.0