Не обновляется Quick View при вычеркивании файла из TEMP:

Здесь можно сообщить о найденных багах в Far.
Post Reply
User avatar
buniak_a_h
Posts: 4234
Joined: Sat 20 Apr, 2013 00:17
Location: Санкт-Петробад
Has thanked: 222 times
Been thanked: 642 times
Contact:

Не обновляется Quick View при вычеркивании файла из TEMP:

Post by buniak_a_h »

Far 3.0.3333 x86.
На одной панели - временная панель; на другой - Quick View. При вычеркивании текущего файла из временной панели (по F7) содержимое Quick View не обновляется
User avatar
buniak_a_h
Posts: 4234
Joined: Sat 20 Apr, 2013 00:17
Location: Санкт-Петробад
Has thanked: 222 times
Been thanked: 642 times
Contact:

Взаимодействие временной панели и QView

Post by buniak_a_h »

Взаимодействие временной панели и QView

При удалении элемента временной панели не обновляется информация, отображающаяся на панели быстрого просмотра.
Шаги к воспроизведению.
  1. Встать на любой элемент временной панели
  2. На пассивной панели открыть быстрый просмотр (Ctrl+Q)
  3. удалить элемент (F7). При этом текущим становится другой элемент, а на панели быстрого просмотра информация не обновляется.
Last edited by HaRT on Wed 26 Oct, 2022 17:53, edited 1 time in total.
Reason: Тема подклеена, заголовок продублирован в сообщение.
Shmuel
Posts: 6815
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 41 times
Been thanked: 526 times

Не обновляется панель QuickView

Post by Shmuel »

Активная панель - TmpPanel с файлами, пассивная - QuickView.
Если двигать курсор вверх или вниз, QuickView обновляется, как и положено.
Если же нажимать F7, убирая файлы с временной панели, QuickView не обновляется.
То же поведение и в Фар 2, а вот Фар 1 ведёт себя правильно.
Предполагаю, что проблема в Фаре, а не в плагине.
User avatar
buniak_a_h
Posts: 4234
Joined: Sat 20 Apr, 2013 00:17
Location: Санкт-Петробад
Has thanked: 222 times
Been thanked: 642 times
Contact:

Не обновляется панель QuickView

Post by buniak_a_h »

А ведь я об этом писал здесь и здесь
Shmuel
Posts: 6815
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 41 times
Been thanked: 526 times

Не обновляется панель QuickView

Post by Shmuel »

В своём форке far2l я починил только что, починка оказалось простой.
User avatar
HaRT
Moderator
Posts: 10806
Joined: Tue 30 Aug, 2005 17:21
Has thanked: 220 times
Been thanked: 357 times

Не обновляется Quick View при вычеркивании файла из TEMP:

Post by HaRT »

 ! Message from: HaRT
buniak_a_h wrote: Tue 25 Oct, 2022 23:46 А ведь я об этом писал здесь и здесь
Спасибо, я всё объединил.
Фар есть инструмент, а не нянька. © 2009 DrKnS
User avatar
buniak_a_h
Posts: 4234
Joined: Sat 20 Apr, 2013 00:17
Location: Санкт-Петробад
Has thanked: 222 times
Been thanked: 642 times
Contact:

Не обновляется Quick View при вычеркивании файла из TEMP:

Post by buniak_a_h »

Господа авторы HaRT, DrKnS, zg!
Есть ошибка, о которой я сообщал аж 9 лет назад, есть решение, опубликованное неделю назад. С тех пор уже вышла ночнушка. Но исправление в неё не вошло. Что так?
User avatar
HaRT
Moderator
Posts: 10806
Joined: Tue 30 Aug, 2005 17:21
Has thanked: 220 times
Been thanked: 357 times

Не обновляется Quick View при вычеркивании файла из TEMP:

Post by HaRT »

buniak_a_h wrote: Tue 01 Nov, 2022 09:48 есть решение, опубликованное неделю назад. С тех пор уже вышла ночнушка. Но исправление в неё не вошло. Что так?
Подозреваю, что шансы бы повысились, если бы хоть кто-нибудь оформил это решение в виде Pull Request'а в репозиторий Фара на GitHub'е.
Фар есть инструмент, а не нянька. © 2009 DrKnS
Shmuel
Posts: 6815
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 41 times
Been thanked: 526 times

Не обновляется Quick View при вычеркивании файла из TEMP:

Post by Shmuel »

HaRT wrote: Tue 01 Nov, 2022 10:59 Подозреваю, что шансы бы повысились, если бы хоть кто-нибудь оформил это решение в виде Pull Request'а
Мне кажется, что для изменения в одной строчке кода можно обойтись без пул-реквеста.
Но возможно то изменение, что я сделал, рассматривается как не вполне корректное. Если это так, было бы любопытно узнать, почему.
User avatar
DrKnS
Posts: 6114
Joined: Thu 04 Aug, 2005 06:44
Location: Kyiv
Has thanked: 12 times
Been thanked: 426 times

Не обновляется Quick View при вычеркивании файла из TEMP:

Post by DrKnS »

Shmuel, I would've happily changed it already if it was just a recent typo.

However, this logic has not changed since the Big Bang.

Of course, it could be Roshal's typo.
But that does not explain why 1.x works as expected, as you say.

Possibly it interferes with something else, or maybe the number of bugs was even again.

I have neither time nor inspiration to investigate it myself and have no idea whether it is correct or not.
If you want me to make the change, please at least explain why do you think the fix is correct and how did you come to that conclusion (even if it is obvious).
Shmuel
Posts: 6815
Joined: Thu 23 Mar, 2006 21:36
Location: Israel
Has thanked: 41 times
Been thanked: 526 times

Не обновляется Quick View при вычеркивании файла из TEMP:

Post by Shmuel »

DrKnS, отвечаю на русском, ибо мой английский оставляет желать лучшего.

Как и вы, я также проверил, что в Фар1 вроде работало и без этого изменения.
И также не смог позволить себе роскошь докопаться точно, где это сломалось.
Теперь о том, в чём (примитивная) логика моего изменения.
  1. Плагин TmpPanel после нажатия F7 и убирания файлов со своей (активной) панели вызывает FCTL_UPDATEPANEL и FCTL_REDRAWPANEL для противоположной (пассивной) панели.
  2. При этом в QuickView вызывается ф-ция Update().
  3. Но настоящий update происходит при вызове Parent()->GetAnotherPanel(this)->UpdateViewPanel(), а этот вызов почему-то обусловлен тем, что имя файла на активной панели - пустое. Я хотел этот if вообще убрать, но потом оставил, изменив условие на противоположное.
User avatar
zg
zero-gravity
Posts: 6015
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 185 times
Contact:

Не обновляется Quick View при вычеркивании файла из TEMP:

Post by zg »

6053 — сломано аж в 165.
Last edited by zg on Tue 15 Nov, 2022 22:10, edited 1 time in total.
Post Reply

Return to “Глюки и баги”