Double search and F4 cause file delete

[asvzzzz]

I encountered with such problem:
Sometimes you need to find a certain substring in the source files, and then another in these found files.
That is, in the folder with the files, press ALT+F7, the line “abcd” for example, search.
A list of found files with this string appears, click "Panel". Press ALT+F7, new substring "1234" for example, search.
A new list of files has appeared. In this list (without going to the "Panel" view) on any file, press F4, look at what you wanted, ESC, look at the next one, etc.
And all files opened using F4 during such double (triple, etc.) search will be DELETED!
I was upset after deleting some not committed files!
Version 2.6.1-2024-04-15-d6cc217-beta Linux x86_64

[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 и там некоторое количество людей из чята на ней сидят. Поглядим, не будет ли жалоб.