View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001270 | Far Manager | Macro | public | 2010-02-11 11:46 | 2010-02-12 19:40 |
Reporter | NightRoman | Assigned To | vskirdin | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 2.0 | ||||
Fixed in Version | 2.0 | ||||
Summary | 0001270: MCMD_LOADALL не возвращает ошибку при наличии ошибок | ||||
Description | Если в реестре есть макрос с синтаксическими ошибками, то при вызове MCMD_LOADALL есть два недостатка 1) код возврата не зависит от наличия или отсутствия ошибок, у вызывающего плагина нет шансов понять, что что-то не в порядке 2) сообщение об ошибке не указывает макрос, это отдельный тикет: http://bugs.farmanager.com/view.php?id=1269 но в идеале эта инфа должна бы не только в UI показываться, но и возвращаться вызывающему, а вот мессаджбокс при вызове MCMD_LOADALL должен бы быть опциональным. | ||||
Tags | No tags attached. | ||||
Build | 1397 | ||||
|
1. MCMD_LOADALL вернет TRUE или FALSE. В случае FALSE вообще никакие макросы не будут загружены. Хмм... т.е. плагину нужно что-то типа MCMD_GETLASTERROR? |
|
> 1. MCMD_LOADALL вернет TRUE или FALSE. В случае FALSE вообще никакие макросы не будут загружены. Это уже гуд. > Хмм... т.е. плагину нужно что-то типа MCMD_GETLASTERROR? Не знаю даже. А чего тут плуг может сделать? Знать, что был сбой и не начать что-то делать, что не надо при таком сбое - уже можно. Может этого и достаточно. Сообщение о сбое юзер видел, имя макроса теперь там показано, пусть фиксит свои макросы и потом уже работает с плугом нормально. |
|
тогда Инц закрываем? |
|
думаю, да, чего тут мудрить. но не протестировано еще :) |
|
протестил? |
|
не понял :) а что оно уже там? |
|
ты либо не зачекинил (и я не видел в ченжлоге), либо не работает |
|
я описал как есть, если что-то не так - давай подробности |
|
создаю в реестре мусорный макрос с кодом zzz. гружу фар, он ругается (диалог), хорошо. потом вызываю MCMD_LOADALL - она всегда возвращает true, даже при наличии проблемы выше (тоже ругается - диалогом, но не возвратом false). |
|
Нашел. Вот же срань :-( build 1397 |
|
работает. закрываю тикет. если что не так, то это уже другая тема будет. |
Date Modified | Username | Field | Change |
---|---|---|---|
2010-02-11 11:46 | NightRoman | New Issue | |
2010-02-11 15:09 | vskirdin | Status | new => assigned |
2010-02-11 15:09 | vskirdin | Assigned To | => vskirdin |
2010-02-11 18:10 | vskirdin | Note Added: 0004898 | |
2010-02-11 18:23 | NightRoman | Note Added: 0004899 | |
2010-02-11 18:36 | vskirdin | Note Added: 0004901 | |
2010-02-11 18:40 | NightRoman | Note Added: 0004902 | |
2010-02-12 13:52 | vskirdin | Note Added: 0004914 | |
2010-02-12 13:52 | vskirdin | Status | assigned => feedback |
2010-02-12 13:57 | NightRoman | Note Added: 0004916 | |
2010-02-12 14:01 | NightRoman | Note Added: 0004917 | |
2010-02-12 14:51 | vskirdin | Note Added: 0004920 | |
2010-02-12 14:58 | NightRoman | Note Added: 0004921 | |
2010-02-12 15:18 | vskirdin | Note Added: 0004922 | |
2010-02-12 15:50 | NightRoman | Note Added: 0004924 | |
2010-02-12 15:51 | NightRoman | Status | feedback => resolved |
2010-02-12 19:40 | vskirdin | Build | => 1397 |
2010-02-12 19:40 | vskirdin | Status | resolved => closed |
2010-02-12 19:40 | vskirdin | Fixed in Version | => 2.0 |
2010-02-12 19:40 | vskirdin | Resolution | open => fixed |