View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000908 | Far Manager | Macro | public | 2009-06-01 20:08 | 2010-02-11 18:46 |
Reporter | alexy | Assigned To | DrKnS | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 2.0 | ||||
Fixed in Version | 2.0 | ||||
Summary | 0000908: Проблемы с сообщением ошибок в макросе | ||||
Description | 1. Имеем макрос: $if (eval("%%calc="+replace(trim(Editor.Value),"=","")+";")==0) $Text %%calc $else $Text "Error in expresion" $end Открываем редактор с пустым файлом и пишем там: replace("string","i") Запускаем макрос. По заголовку окна консоли видим что должен был показатся диалог сообщения об ошибке но его ни где не видно. 2. Открываем MacroBrowser и создаём ошибочный макрос, например: replace("string","i") Говорим OK на сохранение. Выходим из меню плагина. Опа, неожидано видим сообщение (легитимное само по себе) что макрос ошибочный. Тут конечно надо и плагине чинить, чтоб шла проверка при добавлении, но и в фаре тут что то не так вроде бы. | ||||
Tags | No tags attached. | ||||
Build | 1390 | ||||
|
см. buil 974 |
|
Пункт 1 исправлен. Пункт 2, я думаю так, при "считвание" макросов не надо выводить ошибки ибо это совершенно неожидано для пользователя. Ошибку надо выводить при попытке выполнения такого кривого макроса - имея ввиду текущую реализацию надо просто для кривых макросов хранить инфу ошибку которая вышла в момент "компиляции" и показывать её когда такой макрос попытались вызвать. А то выходит что при считывание показывает ошибки (которые вообще совсем не понятно к какому макросу относятся) а в момент попытки запуска тишина и вообще не понятно почему не работает. |
|
>А то выходит что при считывание показывает ошибки (которые вообще совсем не понятно к какому макросу относятся) Можно ведь выводить так чтобы понятно было. |
|
Можно, но не нужно. Выводить надо при попытки запуска самого макроса. |
|
Валя, ты чинить будешь? Сделай хоть как JohnDoe предлагает. |
|
> Выводить надо при попытки запуска самого макроса. А зачем? Как увидели грабли, так и выводим, чего тянуть? Мессага про кривые плагины, которая вываливается _во_время_загрузки_ фара - никого же не смущает. |
|
Ну так я уже и не против, но надо хоть выводить хоткей и область макроса, а то не понятно же нифига что это за макрос такой глючный. |
|
> хоткей и область макроса А, ну это само собой. |
|
Как я понял это ты уже поправил, не? |
|
Ага. |
Date Modified | Username | Field | Change |
---|---|---|---|
2009-06-01 20:08 | alexy | New Issue | |
2009-06-02 05:47 | vskirdin | Status | new => assigned |
2009-06-02 05:47 | vskirdin | Assigned To | => vskirdin |
2009-06-02 05:58 | vskirdin | Note Added: 0003448 | |
2009-06-02 05:58 | vskirdin | Status | assigned => feedback |
2009-06-02 10:31 | alexy | Note Added: 0003453 | |
2009-06-02 10:43 | JohnDoe | Note Added: 0003455 | |
2009-06-02 10:53 | alexy | Note Added: 0003459 | |
2009-11-03 19:27 | alexy | Note Added: 0004488 | |
2009-11-03 19:28 | alexy | Status | feedback => assigned |
2009-11-04 13:18 | DrKnS | Note Added: 0004499 | |
2009-11-04 13:31 | alexy | Note Added: 0004501 | |
2009-11-04 13:45 | DrKnS | Note Added: 0004502 | |
2010-02-11 14:56 | alexy | Assigned To | vskirdin => DrKnS |
2010-02-11 14:56 | alexy | Note Added: 0004897 | |
2010-02-11 18:46 | DrKnS | Build | => 1390 |
2010-02-11 18:46 | DrKnS | Note Added: 0004903 | |
2010-02-11 18:46 | DrKnS | Status | assigned => closed |
2010-02-11 18:46 | DrKnS | Fixed in Version | => 2.0 |
2010-02-11 18:46 | DrKnS | Resolution | open => fixed |