View Issue Details

IDProjectCategoryView StatusLast Update
0000908Far ManagerMacropublic2010-02-11 18:46
Reporteralexy Assigned ToDrKnS  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version2.0 
Fixed in Version2.0 
Summary0000908: Проблемы с сообщением ошибок в макросе
Description1. Имеем макрос:
$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 на сохранение. Выходим из меню плагина. Опа, неожидано видим сообщение (легитимное само по себе) что макрос ошибочный.

Тут конечно надо и плагине чинить, чтоб шла проверка при добавлении, но и в фаре тут что то не так вроде бы.
TagsNo tags attached.
Build1390

Activities

vskirdin

2009-06-02 05:58

administrator   bugnote:0003448

см. buil 974

alexy

2009-06-02 10:31

administrator   bugnote:0003453

Пункт 1 исправлен.

Пункт 2, я думаю так, при "считвание" макросов не надо выводить ошибки ибо это совершенно неожидано для пользователя. Ошибку надо выводить при попытке выполнения такого кривого макроса - имея ввиду текущую реализацию надо просто для кривых макросов хранить инфу ошибку которая вышла в момент "компиляции" и показывать её когда такой макрос попытались вызвать.
А то выходит что при считывание показывает ошибки (которые вообще совсем не понятно к какому макросу относятся) а в момент попытки запуска тишина и вообще не понятно почему не работает.

JohnDoe

2009-06-02 10:43

updater   bugnote:0003455

>А то выходит что при считывание показывает ошибки (которые вообще совсем не понятно к какому макросу относятся)
Можно ведь выводить так чтобы понятно было.

alexy

2009-06-02 10:53

administrator   bugnote:0003459

Можно, но не нужно. Выводить надо при попытки запуска самого макроса.

alexy

2009-11-03 19:27

administrator   bugnote:0004488

Валя, ты чинить будешь? Сделай хоть как JohnDoe предлагает.

DrKnS

2009-11-04 13:18

administrator   bugnote:0004499

> Выводить надо при попытки запуска самого макроса.

А зачем? Как увидели грабли, так и выводим, чего тянуть?

Мессага про кривые плагины, которая вываливается _во_время_загрузки_ фара - никого же не смущает.

alexy

2009-11-04 13:31

administrator   bugnote:0004501

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

DrKnS

2009-11-04 13:45

administrator   bugnote:0004502

> хоткей и область макроса

А, ну это само собой.

alexy

2010-02-11 14:56

administrator   bugnote:0004897

Как я понял это ты уже поправил, не?

DrKnS

2010-02-11 18:46

administrator   bugnote:0004903

Ага.

Issue History

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