View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003663 | Plugins | LuaMacro / Macro Browser | public | 2018-10-30 10:35 | 2018-12-17 17:03 |
Reporter | JohnDoe | Assigned To | shmuel | ||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | fixed | ||
Platform | x86 | OS | Windows | OS Version | XP SP3 |
Product Version | 3.0 | ||||
Summary | 0003663: LuaFAR: возможность опустить параметры-флаги может быть нежелательна | ||||
Description | Несколько методов объекта Settings Control принимают параметр Root, тип которого в мануале указан как integer. Однако если передать nil, то это не приведёт к ошибке.
На мой взгляд, пользы от возможности опустить этот параметр (в данном случае) мало, и даже напротив: так потенциальные баги с неинициализированными значениями проявляются позже. | ||||
Additional Information | С другой стороны, допускаю что точечная правка в Settings Control и не имеет большого смысла.
https://forum.farmanager.com/viewtopic.php?p=151128#p151128 | ||||
Tags | SettingsControl | ||||
|
Собственно, приведу реальный пример, который заставил меня обратить внимание на проблему:
Если пользователь не убедился в успехе операции OpenSubkey, то в Enum передаётся nil.А в результате ошибка остаётся незамеченной, потому что Enum воспринял nil как корень, и мы получили с виду нормальную таблицу items. Или вот другой пример, из чужого кода:
|
|
LuaFAR build 679. |
|
Спасибо, работает. Один вопрос: может стоило сохранить возможность принимать флаги-строки? settings:OpenSubkey("FSSF_ROOT",... |
|
> может стоило сохранить возможность принимать флаги-строки? Эту возможность никто не убирал, просто в приведенном вами примере её и не было. Надо будет поправить. |
|
Вот сейчас сделал параметр Root флагом (везде, где он есть). |
|
Спасибо. Благодаря этому тикету вот уже сходу нашёл у себя один баг с неинициализированным значением) |
Date Modified | Username | Field | Change |
---|---|---|---|
2018-10-30 10:35 | JohnDoe | New Issue | |
2018-10-30 14:41 | JohnDoe | Tag Attached: SettingsControl | |
2018-11-07 13:21 | JohnDoe | Note Added: 0016297 | |
2018-11-08 10:02 | JohnDoe | Note Edited: 0016297 | |
2018-12-15 14:47 | shmuel | Assigned To | => shmuel |
2018-12-15 14:47 | shmuel | Status | new => feedback |
2018-12-15 14:47 | shmuel | Note Added: 0016383 | |
2018-12-16 12:43 | JohnDoe | Note Added: 0016384 | |
2018-12-16 12:43 | JohnDoe | Status | feedback => assigned |
2018-12-16 12:54 | shmuel | Note Added: 0016385 | |
2018-12-16 14:43 | shmuel | Note Added: 0016386 | |
2018-12-17 08:52 | JohnDoe | Status | assigned => resolved |
2018-12-17 08:52 | JohnDoe | Note Added: 0016387 | |
2018-12-17 17:03 | shmuel | Status | resolved => closed |
2018-12-17 17:03 | shmuel | Resolution | open => fixed |