Double search and F4 cause file delete

[asvzzzz]

Double search and F4 cause file delete

[akruphi]

Note: Inside

far2l/far2l/src/findfile.cpp

Lines 1474 to 1700 in 2068c46

	case KEY_F3:
	case KEY_NUMPAD5:
	case KEY_SHIFTNUMPAD5:
	case KEY_F4: {
	if (!ListBox->GetItemCount()) {
	return TRUE;
	}
	size_t ItemIndex = reinterpret_cast<size_t>(ListBox->GetUserData(nullptr, 0));
	bool RemoveTemp = false;
	// Плагины надо закрывать, если открыли.
	bool ClosePlugin = false;
	FARString strSearchFileName;
	FARString strTempDir;
	FINDLIST FindItem;
	itd.GetFindListItem(ItemIndex, FindItem);
	if (FindItem.FindData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY) {
	return TRUE;
	}
	// FindFileArcIndex нельзя здесь использовать
	// Он может быть уже другой.
	if (FindItem.ArcIndex != LIST_INDEX_NONE) {
	ARCLIST ArcItem;
	itd.GetArcListItem(FindItem.ArcIndex, ArcItem);
	if (!(ArcItem.Flags & OPIF_REALNAMES)) {
	FARString strFindArcName = ArcItem.strArcName;
	if (ArcItem.hPlugin == INVALID_HANDLE_VALUE) {
	int SavePluginsOutput = DisablePluginsOutput;
	DisablePluginsOutput = TRUE;
	{
	PluginLocker Lock;
	ArcItem.hPlugin = CtrlObject->Plugins.OpenFilePlugin(strFindArcName,
	OPM_FIND, OFP_SEARCH);
	}
	itd.SetArcListItem(FindItem.ArcIndex, ArcItem);
	DisablePluginsOutput = SavePluginsOutput;
	if (ArcItem.hPlugin == (HANDLE)-2
	|| ArcItem.hPlugin == INVALID_HANDLE_VALUE) {
	ArcItem.hPlugin = INVALID_HANDLE_VALUE;
	itd.SetArcListItem(FindItem.ArcIndex, ArcItem);
	return TRUE;
	}
	ClosePlugin = true;
	}
	FarMkTempEx(strTempDir);
	apiCreateDirectory(strTempDir, nullptr);
	PluginLocker Lock;
	bool bGet = GetPluginFile(FindItem.ArcIndex, FindItem.FindData, strTempDir,
	strSearchFileName);
	itd.SetFindListItem(ItemIndex, FindItem);
	if (!bGet) {
	fprintf(stderr, "%s: GetPluginFile failed for '%ls'\n", __FUNCTION__,
	FindItem.FindData.strFileName.CPtr());
	apiRemoveDirectory(strTempDir);
	if (ClosePlugin) {
	CtrlObject->Plugins.ClosePlugin(ArcItem.hPlugin);
	ArcItem.hPlugin = INVALID_HANDLE_VALUE;
	itd.SetArcListItem(FindItem.ArcIndex, ArcItem);
	}
	return FALSE;
	} else {
	if (ClosePlugin) {
	CtrlObject->Plugins.ClosePlugin(ArcItem.hPlugin);
	ArcItem.hPlugin = INVALID_HANDLE_VALUE;
	itd.SetArcListItem(FindItem.ArcIndex, ArcItem);
	}
	}
	RemoveTemp = true;
	} else
	strSearchFileName = FindItem.FindData.strFileName;
	} else
	strSearchFileName = FindItem.FindData.strFileName;
	DWORD FileAttr = apiGetFileAttributes(strSearchFileName);
	if (FileAttr != INVALID_FILE_ATTRIBUTES) {
	FARString strOldTitle;
	Console.GetTitle(strOldTitle);
	if (Param2 == KEY_F3 || Param2 == KEY_NUMPAD5 || Param2 == KEY_SHIFTNUMPAD5) {
	int ListSize = ListBox->GetItemCount();
	NamesList ViewList;
	// Возьмем все файлы, которые имеют реальные имена...
	if (Opt.FindOpt.CollectFiles) {
	for (int I = 0; I < ListSize; I++) {
	FINDLIST FindItem;
	itd.GetFindListItem(reinterpret_cast<size_t>(
	ListBox->GetUserData(nullptr, 0, I)),
	FindItem);
	bool RealNames = true;
	if (FindItem.ArcIndex != LIST_INDEX_NONE) {
	ARCLIST ArcItem;
	itd.GetArcListItem(FindItem.ArcIndex, ArcItem);
	if (!(ArcItem.Flags & OPIF_REALNAMES)) {
	RealNames = false;
	}
	}
	if (RealNames) {
	if (!FindItem.FindData.strFileName.IsEmpty()
	&& !(FindItem.FindData.dwFileAttributes
	& FILE_ATTRIBUTE_DIRECTORY))
	ViewList.AddName(FindItem.FindData.strFileName);
	}
	}
	FARString strCurDir = FindItem.FindData.strFileName;
	ViewList.SetCurName(strCurDir);
	}
	SendDlgMessage(hDlg, DM_SHOWDIALOG, FALSE, 0);
	SendDlgMessage(hDlg, DM_ENABLEREDRAW, FALSE, 0);
	{
	FileViewer ShellViewer(std::make_shared<FileHolder>(strSearchFileName),
	FALSE, FALSE, FALSE, -1, nullptr, (FindItem.ArcIndex != LIST_INDEX_NONE)
	? nullptr
	: (Opt.FindOpt.CollectFiles ? &ViewList : nullptr));
	ShellViewer.SetDynamicallyBorn(FALSE);
	ShellViewer.SetEnableF6(TRUE);
	// FindFileArcIndex нельзя здесь использовать
	// Он может быть уже другой.
	if (FindItem.ArcIndex != LIST_INDEX_NONE) {
	ARCLIST ArcItem;
	itd.GetArcListItem(FindItem.ArcIndex, ArcItem);
	if (!(ArcItem.Flags & OPIF_REALNAMES)) {
	ShellViewer.SetSaveToSaveAs(true);
	}
	}
	FrameManager->ExecuteModalEV();
	// заставляем рефрешиться экран
	FrameManager->ProcessKey(KEY_CONSOLE_BUFFER_RESIZE);
	}
	SendDlgMessage(hDlg, DM_ENABLEREDRAW, TRUE, 0);
	SendDlgMessage(hDlg, DM_SHOWDIALOG, TRUE, 0);
	} else {
	SendDlgMessage(hDlg, DM_SHOWDIALOG, FALSE, 0);
	SendDlgMessage(hDlg, DM_ENABLEREDRAW, FALSE, 0);
	{
	/*
	$ 14.08.2002 VVM
	! Пока-что запретим из поиска переключаться в активный редактор.
	К сожалению, манагер на это не способен сейчас
	int FramePos=FrameManager->FindFrameByFile(MODALTYPE_EDITOR,SearchFileName);
	int SwitchTo=FALSE;
	if (FramePos!=-1)
	{
	if (!(*FrameManager)[FramePos]->GetCanLoseFocus(TRUE) ||
	Opt.Confirm.AllowReedit)
	{
	char MsgFullFileName[NM];
	far_strncpy(MsgFullFileName,SearchFileName,sizeof(MsgFullFileName));
	int MsgCode=Message(0,2,Msg::FindFileTitle,
	TruncPathStr(MsgFullFileName,ScrX-16),
	Msg::AskReload,
	Msg::Current,Msg::NewOpen);
	if (!MsgCode)
	{
	SwitchTo=TRUE;
	}
	else if (MsgCode==1)
	{
	SwitchTo=FALSE;
	}
	else
	{
	SendDlgMessage(hDlg,DM_ENABLEREDRAW,TRUE,0);
	SendDlgMessage(hDlg,DM_SHOWDIALOG,TRUE,0);
	return TRUE;
	}
	}
	else
	{
	SwitchTo=TRUE;
	}
	}
	if (SwitchTo)
	{
	(*FrameManager)[FramePos]->SetCanLoseFocus(FALSE);
	(*FrameManager)[FramePos]->SetDynamicallyBorn(FALSE);
	FrameManager->ActivateFrame(FramePos);
	FrameManager->ExecuteModalEV();
	// FrameManager->ExecuteNonModal();
	// заставляем рефрешиться экран
	FrameManager->ProcessKey(KEY_CONSOLE_BUFFER_RESIZE);
	}
	else
	*/
	{
	FileHolderPtr TFH;
	if (FindItem.ArcIndex != LIST_INDEX_NONE) {
	TFH = std::make_shared<FindDlg_TempFileHolder>(strSearchFileName,
	FindItem.ArcIndex, FindItem.FindData);
	} else {
	TFH = std::make_shared<FileHolder>(strSearchFileName);
	}
	FileEditor ShellEditor(TFH, CP_AUTODETECT, 0);
	ShellEditor.SetDynamicallyBorn(FALSE);
	ShellEditor.SetEnableF6(TRUE);
	// FindFileArcIndex нельзя здесь использовать
	// Он может быть уже другой.
	FrameManager->ExecuteModalEV();
	TFH->CheckForChanges();
	// заставляем рефрешиться экран
	FrameManager->ProcessKey(KEY_CONSOLE_BUFFER_RESIZE);
	}
	}
	SendDlgMessage(hDlg, DM_ENABLEREDRAW, TRUE, 0);
	SendDlgMessage(hDlg, DM_SHOWDIALOG, TRUE, 0);
	}
	Console.SetTitle(strOldTitle);
	}
	if (RemoveTemp) {
	DeleteFileWithFolder(strSearchFileName);
	}
	return TRUE;
	} break;

has depended on variable RemoveTemp deleting temporary files in

far2l/far2l/src/findfile.cpp

Lines 1696 to 1697 in 2068c46

	if (RemoveTemp) {
	DeleteFileWithFolder(strSearchFileName);

But in each press F3 or F4 RemoveTemp initialized as false and should no influence to real files.

[spnethw]

It seems that problem is really in findfile.cpp, but a little earlier in the code, namely here:

far2l/far2l/src/findfile.cpp

Lines 1673 to 1679 in 2068c46

	FileHolderPtr TFH;
	if (FindItem.ArcIndex != LIST_INDEX_NONE) {
	TFH = std::make_shared<FindDlg_TempFileHolder>(strSearchFileName,
	FindItem.ArcIndex, FindItem.FindData);
	} else {
	TFH = std::make_shared<FileHolder>(strSearchFileName);
	}

if FindItem.ArcIndex != LIST_INDEX_NONE, then the instance of FindDlg_TempFileHolder will be created. This class inherited from TempFileUploadHolder, which in turn is inherited from TempFileHolder.

The actual file deletion is done in the TempFileHolder::~TempFileHolder() which will be called as the base class destructor when the instance of FindDlg_TempFileHolder is destroyed:

far2l/far2l/src/fileholder.cpp

Lines 60 to 66 in 2068c46

	if (_delete == DELETE_FILE) {
	apiMakeWritable(_file_path_name);
	apiDeleteFile(_file_path_name);
	} else if (_delete == DELETE_FILE_AND_PARENT_DIR) {
	DeleteFileWithFolder(_file_path_name);
	}

[spnethw]

So, the problem is that the files found on a second search from "Temporary panel" are treated as temporary files, as if the search was done in an archive and they were extracted to /tmp.

UPD: perhaps subsequent digging should be made around this line of code (may be not).

---

Получается, причина в том, что при поиске файлов на "Временной панели" они рассматриваются как найденные в архиве, интерпретируются как временные, распакованные в /tmp, после чего благополучно грохаются (вместе с содержащей их директорией, если там больше ничего нет).

[spnethw]

Оставить просто

TFH = std::make_shared<FileHolder>(strSearchFileName);

мы не можем, хотя бы потому что таким образом мы убьём одну из фич far2l — а именно сохранение сделанных при редактировании изменений обратно в архив. Так как в ветке, получающей управление при запросе на редактирование по F4, потом дёргается

far2l/far2l/src/findfile.cpp

Line 1686 in 2068c46

TFH->CheckForChanges();

А она в базовом классе, FileHolder, не имеет реализации, а реализована только в непосредственном предке FindDlg_TempFileHolder — в TempFileUploadHolder .

[spnethw]

Есть ощущение, что можно попробовать пофиксить, изменив логику где-то в этой области:

far2l/far2l/src/findfile.cpp

Lines 2447 to 2464 in 2068c46

	if (v.PluginMode) {
	Panel *ActivePanel = CtrlObject->Cp()->ActivePanel;
	HANDLE hPlugin = ActivePanel->GetPluginHandle();
	OpenPluginInfo Info;
	{
	PluginLocker Lock;
	CtrlObject->Plugins.GetOpenPluginInfo(hPlugin, &Info);
	itd.SetFindFileArcIndex(itd.AddArcListItem(Info.HostFile, hPlugin, Info.Flags, Info.CurDir));
	}
	if (itd.GetFindFileArcIndex() == LIST_INDEX_NONE)
	return false;
	if (!(Info.Flags & OPIF_REALNAMES)) {
	FindDlg[FD_BUTTON_PANEL].Type = DI_TEXT;
	FindDlg[FD_BUTTON_PANEL].strData.Clear();
	}
	}

Именно тут функцией AddArcListItem файлы со временной панели незаслуженно отправляются в ArcList :

far2l/far2l/src/findfile.cpp

Line 2454 in 2068c46

itd.SetFindFileArcIndex(itd.AddArcListItem(Info.HostFile, hPlugin, Info.Flags, Info.CurDir));

Допустим, проверять, и если мы на tmppanel, а не multiarc, то всех этих телодвижений не выполнять?

[unxed]

Допустим, проверять, и если мы на tmppanel, а не multiarc, то всех этих телодвижений не выполнять?

А сделайте PR мне в staging, посмотрим как это у людей работать будет, если ничего нигде не сломается, зашлём сюда.

[akruphi]

Вроде #2229 исправляет без внесения новых глюков, но проверьте кто может крайние случаи и в них ситуацию перехода по F6 между редактором и просмоторщиком и обратно.

[unxed]

Забрал в свою ветку staging, у неё отдельный ppa и там некоторое количество людей из чята на ней сидят. Поглядим, не будет ли жалоб.