View Issue Details

IDProjectCategoryView StatusLast Update
0001697WishesMacropublic2013-02-03 00:34
ReporterDAtaMan Assigned Tovskirdin  
PrioritynormalSeverityfeatureReproducibilityhave not tried
Status closedResolutionfixed 
Product Version2.0 
Fixed in Version2.0 
Summary0001697: Доступ к Custom-колонке из функции panel.item
DescriptionСобственно, сабж. Патч прилагается.
Additional InformationЗаодно небольшое упрощение при использовании 11-ой колонки (описание файла).
TagsNo tags attached.
Build1811

Activities

DAtaMan

2011-02-02 13:20

reporter  

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;
 		}
 	}
 
macro.cpp.diff (712 bytes)   

DAtaMan

2011-02-02 13:24

reporter   bugnote:0006328

Может всё-таки перетасовать индексы колонок, чтобы однотипные (дата/время) колонки были рядышком ?
А то в Писании таблица не очень кузяво смотрится, да и при получении колонок в цикле от 0 до крайней, тоже.

JohnDoe

2011-02-02 13:46

updater   bugnote:0006329

Last edited: 2011-02-02 13:49

http://api.farmanager.com/ru2/macro/macrocmd/prop_func/panels.html#panel.item
Про таблицу:
Смысл не в том чтобы циферки шли стройными рядами, а в том чтобы человеку было легко ориентироваться.
А циферки по идее в чистом виде мало где нужны, вместо них правильнее использовать константы.

Про колонки:
Не понял зачем их получать в цикле.

DAtaMan

2011-02-02 14:47

reporter   bugnote:0006331

>чтобы человеку было легко ориентироваться
Да, легко при написании своих макросов, согласен - сортировка по описанию удобна.
При разборе же чужих, сортировка по индексу удобней. Разве нет ?

>Не понял зачем их получать в цикле.
Просто есть отладочный макрос, выводящий диалог со всеми колонками для текущего файла. Я им несколько багов по колонкам выявил.
Можно и без цикла обойтись, конечно, но с циклом макрос проще.
А если ещё будут добавляться колонки (а вдруг ? :) ), потребуется меньше изменений.
Само собой, это не причина для изменения индексов, т.к. это только мне (а может и не только) нужно.

vskirdin

2011-02-02 15:14

administrator   bugnote:0006332

DAtaMan, строку для Писания про 22-й индекс сюда пости

JohnDoe

2011-02-02 15:15

updater   bugnote:0006333

>При разборе же чужих, сортировка по индексу удобней. Разве нет ?
Вот для чего нужны константы: при разборе чужих макросов вообще не придётся заглядывать в Писание.

>Само собой, это не причина для изменения индексов
Лично меня это не коснётся, т.к. использую константы.

DAtaMan

2011-02-02 16:11

reporter   bugnote:0006335

>строку для Писания про 22-й индекс сюда пости
22 String Значение Custom-колонки, если установлены соответствующие плагины и она (C0) отображается в текущем режиме панели.

Так сойдёт или diff нужен ?

2 JohnDoe
>Вот для чего нужны константы: при разборе чужих макросов вообще не придётся заглядывать в Писание.
Эх, если бы все придерживались хорошего тона при написании макросов.
Но я не настаиваю - пью сырой. :)

JohnDoe

2011-02-02 16:36

updater   bugnote:0006336

>Эх, если бы все придерживались хорошего тона при написании макросов.
На форуме я при каждом удобном случае "пропагандирую" константы.

А если приходится применять чужой макрос, то первым делом заменяю значения константами.

>22 String Значение Custom-колонки, если установлены соответствующие плагины и она (C0) отображается в текущем режиме панели.
Звучит несколько странно, поскольку С0 нигде больше не документирована, ни в помощи, ни в Писании. (По крайней мере я нашёл только одно косвенное упоминание: http://api.farmanager.com/ru2/structures/pluginpanelitem.html)

DrKnS

2011-02-02 16:59

administrator   bugnote:0006337

> t-rex 12.06.2010 19:14:06 +0200 - build 1588
>
> 1. Экспериментально, неофициально и криво на посмотреть, поиграться и подумать:
> поддержка кастом колонки (C0) заполняемой плагинами в не плагиновых файловых > панелях

Куда вы так спешите, оно же недоделано ещё.

JohnDoe

2011-02-02 17:04

updater   bugnote:0006338

Ну t-rex вроде ничего больше доделывать не планировал, и остальные тоже вскоре затихли: http://forum.farmanager.com/viewtopic.php?p=64364#p64364

vskirdin

2011-02-03 10:37

administrator   bugnote:0006342

build 1811

DAtaMan

2011-02-03 13:11

reporter   bugnote:0006344

Всё ok, спасибо.

Пока "ломается" API, вишепатчи лучше не предлагать ?

Issue History

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
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