SetDirectoryInfo::UserData

Разработка Far.
User avatar
NightRoman
Posts: 6034
Joined: Fri 18 Aug, 2006 13:52
Location: Cambridge, UK
Has thanked: 20 times
Been thanked: 49 times
Contact:

Re: SetDirectoryInfo::UserData

Post by NightRoman »

Latest post of the previous page:

Без поиска все тесты отработали. Это уже хорошо. С поиском тестов нет, естественно, так как он был сознательно вырублен. Сейчас что-нибудь руками попробую потестировать, если найдутся подходящие панели и данные.
User avatar
NightRoman
Posts: 6034
Joined: Fri 18 Aug, 2006 13:52
Location: Cambridge, UK
Has thanked: 20 times
Been thanked: 49 times
Contact:

Re: SetDirectoryInfo::UserData

Post by NightRoman »

Глянул в панели и глянул в код. Вероятно, поисковик быстро попробовать не могу. Код Фарнет панелей сейчас слишком повязан на то, что родного поиска не было, надо дорабатывать без суеты и смотреть, что получится. Так что пробовать пока можно на любой плагиновой панели с поиском (на падения, не на новую функциональность, конечно). А я буду думать, как доработать панели Фарнет для нового поиска, тогда и видно будет, работает или нет.
User avatar
Skirda
Posts: 3475
Joined: Wed 16 Mar, 2005 11:01
Been thanked: 5 times

Re: SetDirectoryInfo::UserData

Post by Skirda »

Надо определиться и с FreeData.
Когда он должен зваться?
We are the Borg. You will be assimilated.
So Say We All!
User avatar
NightRoman
Posts: 6034
Joined: Fri 18 Aug, 2006 13:52
Location: Cambridge, UK
Has thanked: 20 times
Been thanked: 49 times
Contact:

Re: SetDirectoryInfo::UserData

Post by NightRoman »

Когда файл плагина гарантированно больше не будет использоваться Фаром.
User avatar
NightRoman
Posts: 6034
Joined: Fri 18 Aug, 2006 13:52
Location: Cambridge, UK
Has thanked: 20 times
Been thanked: 49 times
Contact:

Re: SetDirectoryInfo::UserData

Post by NightRoman »

Или даже точнее - передаваться обратно в плагин (но это, суть, одно и то же, просто иначе названо).
User avatar
NightRoman
Posts: 6034
Joined: Fri 18 Aug, 2006 13:52
Location: Cambridge, UK
Has thanked: 20 times
Been thanked: 49 times
Contact:

Re: SetDirectoryInfo::UserData

Post by NightRoman »

И все это касается даже не самого файла (его-то данные просто скопированы), а юзер дата, конечно. У юзер дата есть "жизненный цикл", в отличие от остальных данных файла.
User avatar
NightRoman
Posts: 6034
Joined: Fri 18 Aug, 2006 13:52
Location: Cambridge, UK
Has thanked: 20 times
Been thanked: 49 times
Contact:

Re: SetDirectoryInfo::UserData

Post by NightRoman »

Протестировал и 3040 из базы. То, что используется у меня, работает нормально. Пошел думать, как научить панели использовать новые возможности.
User avatar
zg
zero-gravity
Posts: 5878
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 158 times
Contact:

Re: SetDirectoryInfo::UserData

Post by zg »

Skirda wrote:Надо определиться и с FreeData.
Когда он должен зваться?
при удалении списка.
User avatar
NightRoman
Posts: 6034
Joined: Fri 18 Aug, 2006 13:52
Location: Cambridge, UK
Has thanked: 20 times
Been thanked: 49 times
Contact:

Re: SetDirectoryInfo::UserData

Post by NightRoman »

zg
При поиске - с оговоркой. Найденные файлы не должны удаляться пока. Они должны удаляться, когда диалог результатов поиска будет закрыт. В диалоге поиска юзер данные все еще нужны для разных операций (просмотр, редактирование, возможно, и переход на найденный файл).
User avatar
zg
zero-gravity
Posts: 5878
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 158 times
Contact:

Re: SetDirectoryInfo::UserData

Post by zg »

NightRoman wrote:zg
При поиске - с оговоркой. Найденные файлы не должны удаляться пока. Они должны удаляться, когда диалог результатов поиска будет закрыт. В диалоге поиска юзер данные все еще нужны для разных операций (просмотр, редактирование, возможно, и переход на найденный файл).
я не очень понимаю, как при удалённом списке над его элементами можно совершать операции.
User avatar
NightRoman
Posts: 6034
Joined: Fri 18 Aug, 2006 13:52
Location: Cambridge, UK
Has thanked: 20 times
Been thanked: 49 times
Contact:

Re: SetDirectoryInfo::UserData

Post by NightRoman »

Удаление списка и его элементов это разные операции. Именно для этого и нужен персональный колбэк каждому файлу (а иначе и FreeFindDataW было бы достаточно). Странно, что именно ты не понимаешь, ведь ты уже именно так и сделал, можно найти на форуме наше обсуждение. Я тестировал и оно работало правильно - вызывался FreeFindDataW и удаление элементов, но только тех, которые не прошли поиск. Те, которые прошли, удаляются при закрытии диалога результатов поиска.

Либо мы говорим про разные списки.

В любом случае. Если в диалоге результатов поиска будут уже удаленные юзер дата у файлов, то это приведет к падению при операциях плагина над ними.
User avatar
zg
zero-gravity
Posts: 5878
Joined: Tue 15 Mar, 2005 17:36
Location: riga
Been thanked: 158 times
Contact:

Re: SetDirectoryInfo::UserData

Post by zg »

NightRoman wrote:Либо мы говорим про разные списки.
очевидно, что я говорю про внутренний фаровский список. говорить в таком контексте о списке, возвращаемом плагином не имеет никакого смысла.
NightRoman wrote:В любом случае. Если в диалоге результатов поиска будут уже удаленные юзер дата у файлов, то это приведет к падению при операциях плагина над ними.
не надо 100 раз писать одно и то же. персональный калбэк был введён именно для того, чтобы не падало. есть конкретные ошибки, пиши багрепорт на мантис. если к багрепорту будет приложен простой легкособирающийся тестовый плагин с исходниками, шанс исправления возрастает в 100 от базового. если для воспроизведения нужно поставить фарнет — шанс соответственно падает в 100 раз.
User avatar
NightRoman
Posts: 6034
Joined: Fri 18 Aug, 2006 13:52
Location: Cambridge, UK
Has thanked: 20 times
Been thanked: 49 times
Contact:

Re: SetDirectoryInfo::UserData

Post by NightRoman »

Звучит так, словно это мне надо, а не пользователям Фара пригодится, через Фарнет или нет. Фича разве не для плагинов? И если плагин показывает ошибку, то этого недостаточно?
User avatar
Skirda
Posts: 3475
Joined: Wed 16 Mar, 2005 11:01
Been thanked: 5 times

Re: SetDirectoryInfo::UserData

Post by Skirda »

2All: по этой теме пока не юзайте SetDirectoryInfo::UserData до особой отмашки.
We are the Borg. You will be assimilated.
So Say We All!
Post Reply

Return to “Development”