FarSettingsHistory - а где Type?

Место, где разработчики плагинов могут поделиться своим опытом.
Post Reply
Nikolay
Posts: 602
Joined: Sat 14 Jan, 2006 17:17
Has thanked: 1 time
Been thanked: 34 times

FarSettingsHistory - а где Type?

Post by Nikolay »

Собственно, почему поля type из базы нету в этой структуре?
User avatar
zg
zero-gravity
Posts: 6035
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 185 times
Contact:

Re: FarSettingsHistory - а где Type?

Post by zg »

Nikolay wrote:Собственно, почему поля type из базы нету в этой структуре?
а смысл? оно в базе лишнее, это да.
User avatar
zg
zero-gravity
Posts: 6035
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 185 times
Contact:

Re: FarSettingsHistory - а где Type?

Post by zg »

хотя вообще я наверное тут не прав. но с type как-то всё странно, если в код посмотреть. и это скорее к алексу, что он там имел в виду в 850 билде.
User avatar
t-rex
Страшный и ужасный
Posts: 4910
Joined: Tue 15 Mar, 2005 16:17
Location: Tel-Aviv
Has thanked: 1 time
Been thanked: 8 times
Contact:

Re: FarSettingsHistory - а где Type?

Post by t-rex »

ну это там всегда как бы было, type нужен только для истории просмотра:

case 0: // вьювер

case 1: // обычное открытие в редакторе
case 4: // открытие в редакторе открытие с локом

ProcessExternal()
case 2: // external - без ожидания
case 3: // external - AlwaysWaitFinish
User avatar
zg
zero-gravity
Posts: 6035
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 185 times
Contact:

Re: FarSettingsHistory - а где Type?

Post by zg »

t-rex wrote:ну это там всегда как бы было, type нужен только для истории просмотра:
в 850 добавлено:

Code: Select all

void History::AddToHistory(const wchar_t *Str, int Type, const GUID* Guid, const wchar_t *File, const wchar_t *Data, bool SaveForbid)
…
	if ((Type!=HISTORYTYPE_FOLDER || !Guid || IsEqualGUID(FarGuid,*Guid)) && (!Str || !*Str))
HISTORYTYPE_FOLDER тут явно не в тему.
case 1: // обычное открытие в редакторе

Code: Select all

BOOL FileList::ChangeDir(const wchar_t *NewDir,BOOL IsUpdated)
…
		CtrlObject->FolderHistory->AddToHistory(strInfoCurDir,1,&PluginManager::GetGUID(hPlugin),strInfoHostFile,strInfoData);
тоже противоречие.
Nikolay
Posts: 602
Joined: Sat 14 Jan, 2006 17:17
Has thanked: 1 time
Been thanked: 34 times

Re: FarSettingsHistory - а где Type?

Post by Nikolay »

t-rex wrote: case 4: // открытие в редакторе открытие с локом
Собственно, из-за этого сыр-бор. Я сначала логически подумал, что поле Lock - это оно, но обломался.
User avatar
t-rex
Страшный и ужасный
Posts: 4910
Joined: Tue 15 Mar, 2005 16:17
Location: Tel-Aviv
Has thanked: 1 time
Been thanked: 8 times
Contact:

Re: FarSettingsHistory - а где Type?

Post by t-rex »

zg wrote:
t-rex wrote:ну это там всегда как бы было, type нужен только для истории просмотра:
в 850 добавлено:

Code: Select all

void History::AddToHistory(const wchar_t *Str, int Type, const GUID* Guid, const wchar_t *File, const wchar_t *Data, bool SaveForbid)
…
	if ((Type!=HISTORYTYPE_FOLDER || !Guid || IsEqualGUID(FarGuid,*Guid)) && (!Str || !*Str))
HISTORYTYPE_FOLDER тут явно не в тему.
Да, тут вроде баг, наверно должно быть TypeHistory!=HISTORYTYPE_FOLDER, хотя сразу вообще не особо понятно что тут за логика.
Тока о каком 850, мы говорим? В 850 билде нифига такого нету.
Вообще смотрю, этот код ты написал в 2239, в 2238 был вот такой правильный:
if (TypeHistory == HISTORYTYPE_FOLDER && Prefix && *Prefix)
zg wrote:
case 1: // обычное открытие в редакторе

Code: Select all

BOOL FileList::ChangeDir(const wchar_t *NewDir,BOOL IsUpdated)
…
		CtrlObject->FolderHistory->AddToHistory(strInfoCurDir,1,&PluginManager::GetGUID(hPlugin),strInfoHostFile,strInfoData);
тоже противоречие.
Это я смотрю с 1.7 как минимум так, видимо тут должно быть 0.
User avatar
zg
zero-gravity
Posts: 6035
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 185 times
Contact:

Re: FarSettingsHistory - а где Type?

Post by zg »

t-rex wrote:Вообще смотрю, этот код ты написал в 2239, в 2238 был вот такой правильный:
if (TypeHistory == HISTORYTYPE_FOLDER && Prefix && *Prefix)
ну значит меня вызов из FileList::ChangeDir в заблуждение ввёл.
Nikolay
Posts: 602
Joined: Sat 14 Jan, 2006 17:17
Has thanked: 1 time
Been thanked: 34 times

Re: FarSettingsHistory - а где Type?

Post by Nikolay »

Дык, возвращаясь к сути вопроса - плагинам будет позволено узнать признак залоченности файла в истории редактирования?
Post Reply

Return to “Разработка плагинов”