FarSettingsHistory - а где Type?
- zg
- zero-gravity
- Posts: 6035
- Joined: Tue 15 Mar, 2005 17:36
- Location: riga
- Been thanked: 185 times
- Contact:
Re: FarSettingsHistory - а где Type?
а смысл? оно в базе лишнее, это да.Nikolay wrote:Собственно, почему поля type из базы нету в этой структуре?
- zg
- zero-gravity
- Posts: 6035
- Joined: Tue 15 Mar, 2005 17:36
- Location: riga
- Been thanked: 185 times
- Contact:
Re: FarSettingsHistory - а где Type?
хотя вообще я наверное тут не прав. но с type как-то всё странно, если в код посмотреть. и это скорее к алексу, что он там имел в виду в 850 билде.
- 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?
ну это там всегда как бы было, type нужен только для истории просмотра:
case 0: // вьювер
case 1: // обычное открытие в редакторе
case 4: // открытие в редакторе открытие с локом
ProcessExternal()
case 2: // external - без ожидания
case 3: // external - AlwaysWaitFinish
case 0: // вьювер
case 1: // обычное открытие в редакторе
case 4: // открытие в редакторе открытие с локом
ProcessExternal()
case 2: // external - без ожидания
case 3: // external - AlwaysWaitFinish
- zg
- zero-gravity
- Posts: 6035
- Joined: Tue 15 Mar, 2005 17:36
- Location: riga
- Been thanked: 185 times
- Contact:
Re: FarSettingsHistory - а где Type?
в 850 добавлено:t-rex wrote:ну это там всегда как бы было, type нужен только для истории просмотра:
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))
case 1: // обычное открытие в редакторе
Code: Select all
BOOL FileList::ChangeDir(const wchar_t *NewDir,BOOL IsUpdated)
…
CtrlObject->FolderHistory->AddToHistory(strInfoCurDir,1,&PluginManager::GetGUID(hPlugin),strInfoHostFile,strInfoData);
- 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?
Да, тут вроде баг, наверно должно быть TypeHistory!=HISTORYTYPE_FOLDER, хотя сразу вообще не особо понятно что тут за логика.zg wrote:в 850 добавлено:t-rex wrote:ну это там всегда как бы было, type нужен только для истории просмотра:HISTORYTYPE_FOLDER тут явно не в тему.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))
Тока о каком 850, мы говорим? В 850 билде нифига такого нету.
Вообще смотрю, этот код ты написал в 2239, в 2238 был вот такой правильный:
if (TypeHistory == HISTORYTYPE_FOLDER && Prefix && *Prefix)
Это я смотрю с 1.7 как минимум так, видимо тут должно быть 0.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);
- zg
- zero-gravity
- Posts: 6035
- Joined: Tue 15 Mar, 2005 17:36
- Location: riga
- Been thanked: 185 times
- Contact:
Re: FarSettingsHistory - а где Type?
ну значит меня вызов из FileList::ChangeDir в заблуждение ввёл.t-rex wrote:Вообще смотрю, этот код ты написал в 2239, в 2238 был вот такой правильный:
if (TypeHistory == HISTORYTYPE_FOLDER && Prefix && *Prefix)