View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001697 | Wishes | Macro | public | 2011-02-02 13:20 | 2013-02-03 00:34 |
Reporter | DAtaMan | Assigned To | vskirdin | ||
Priority | normal | Severity | feature | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Product Version | 2.0 | ||||
Fixed in Version | 2.0 | ||||
Summary | 0001697: Доступ к Custom-колонке из функции panel.item | ||||
Description | Собственно, сабж. Патч прилагается. | ||||
Additional Information | Заодно небольшое упрощение при использовании 11-ой колонки (описание файла). | ||||
Tags | No tags attached. | ||||
Build | 1811 | ||||
|
macro.cpp.diff (712 bytes)
Index: macro.cpp =================================================================== --- macro.cpp (revision 5499) +++ macro.cpp (working copy) @@ -3561,11 +3561,8 @@ Ret=TVar((__int64)filelistItem.SortGroup); break; case 11: // DizText - { - const wchar_t *LPtr=filelistItem.DizText; - Ret=TVar(LPtr); + Ret=TVar(filelistItem.DizText); break; - } case 12: // Owner Ret=TVar(filelistItem.strOwner); break; @@ -3599,6 +3596,9 @@ case 21: // ChangeTime (FILETIME) Ret=TVar((__int64)FileTimeToUI64(&filelistItem.ChangeTime)); break; + case 22: // CustomData + Ret=TVar(filelistItem.strCustomData); + break; } } |
|
Может всё-таки перетасовать индексы колонок, чтобы однотипные (дата/время) колонки были рядышком ? А то в Писании таблица не очень кузяво смотрится, да и при получении колонок в цикле от 0 до крайней, тоже. |
|
http://api.farmanager.com/ru2/macro/macrocmd/prop_func/panels.html#panel.item Про таблицу: Смысл не в том чтобы циферки шли стройными рядами, а в том чтобы человеку было легко ориентироваться. А циферки по идее в чистом виде мало где нужны, вместо них правильнее использовать константы. Про колонки: Не понял зачем их получать в цикле. |
|
>чтобы человеку было легко ориентироваться Да, легко при написании своих макросов, согласен - сортировка по описанию удобна. При разборе же чужих, сортировка по индексу удобней. Разве нет ? >Не понял зачем их получать в цикле. Просто есть отладочный макрос, выводящий диалог со всеми колонками для текущего файла. Я им несколько багов по колонкам выявил. Можно и без цикла обойтись, конечно, но с циклом макрос проще. А если ещё будут добавляться колонки (а вдруг ? :) ), потребуется меньше изменений. Само собой, это не причина для изменения индексов, т.к. это только мне (а может и не только) нужно. |
|
DAtaMan, строку для Писания про 22-й индекс сюда пости |
|
>При разборе же чужих, сортировка по индексу удобней. Разве нет ? Вот для чего нужны константы: при разборе чужих макросов вообще не придётся заглядывать в Писание. >Само собой, это не причина для изменения индексов Лично меня это не коснётся, т.к. использую константы. |
|
>строку для Писания про 22-й индекс сюда пости 22 String Значение Custom-колонки, если установлены соответствующие плагины и она (C0) отображается в текущем режиме панели. Так сойдёт или diff нужен ? 2 JohnDoe >Вот для чего нужны константы: при разборе чужих макросов вообще не придётся заглядывать в Писание. Эх, если бы все придерживались хорошего тона при написании макросов. Но я не настаиваю - пью сырой. :) |
|
>Эх, если бы все придерживались хорошего тона при написании макросов. На форуме я при каждом удобном случае "пропагандирую" константы. А если приходится применять чужой макрос, то первым делом заменяю значения константами. >22 String Значение Custom-колонки, если установлены соответствующие плагины и она (C0) отображается в текущем режиме панели. Звучит несколько странно, поскольку С0 нигде больше не документирована, ни в помощи, ни в Писании. (По крайней мере я нашёл только одно косвенное упоминание: http://api.farmanager.com/ru2/structures/pluginpanelitem.html) |
|
> t-rex 12.06.2010 19:14:06 +0200 - build 1588 > > 1. Экспериментально, неофициально и криво на посмотреть, поиграться и подумать: > поддержка кастом колонки (C0) заполняемой плагинами в не плагиновых файловых > панелях Куда вы так спешите, оно же недоделано ещё. |
|
Ну t-rex вроде ничего больше доделывать не планировал, и остальные тоже вскоре затихли: http://forum.farmanager.com/viewtopic.php?p=64364#p64364 |
|
build 1811 |
|
Всё ok, спасибо. Пока "ломается" API, вишепатчи лучше не предлагать ? |
Date Modified | Username | Field | Change |
---|---|---|---|
2011-02-02 13:20 | DAtaMan | New Issue | |
2011-02-02 13:20 | DAtaMan | File Added: macro.cpp.diff | |
2011-02-02 13:24 | DAtaMan | Note Added: 0006328 | |
2011-02-02 13:46 | JohnDoe | Note Added: 0006329 | |
2011-02-02 13:49 | JohnDoe | Note Edited: 0006329 | View Revisions |
2011-02-02 14:47 | DAtaMan | Note Added: 0006331 | |
2011-02-02 15:14 | vskirdin | Note Added: 0006332 | |
2011-02-02 15:14 | vskirdin | Assigned To | => vskirdin |
2011-02-02 15:14 | vskirdin | Status | new => assigned |
2011-02-02 15:15 | JohnDoe | Note Added: 0006333 | |
2011-02-02 16:11 | DAtaMan | Note Added: 0006335 | |
2011-02-02 16:36 | JohnDoe | Note Added: 0006336 | |
2011-02-02 16:59 | DrKnS | Note Added: 0006337 | |
2011-02-02 17:04 | JohnDoe | Note Added: 0006338 | |
2011-02-03 10:37 | vskirdin | Note Added: 0006342 | |
2011-02-03 10:37 | vskirdin | Status | assigned => feedback |
2011-02-03 13:11 | DAtaMan | Note Added: 0006344 | |
2011-02-03 13:11 | DAtaMan | Status | feedback => assigned |
2011-02-03 13:25 | vskirdin | Build | => 1811 |
2011-02-03 13:25 | vskirdin | Status | assigned => closed |
2011-02-03 13:25 | vskirdin | Resolution | open => fixed |
2011-02-03 13:25 | vskirdin | Fixed in Version | => 2.0 |
2013-02-03 00:34 | alexy | Severity | minor => feature |