View Issue Details

IDProjectCategoryView StatusLast Update
0001270Far ManagerMacropublic2010-02-12 19:40
ReporterNightRoman Assigned Tovskirdin  
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version2.0 
Fixed in Version2.0 
Summary0001270: MCMD_LOADALL не возвращает ошибку при наличии ошибок
DescriptionЕсли в реестре есть макрос с синтаксическими ошибками, то при вызове
MCMD_LOADALL есть два недостатка

1) код возврата не зависит от наличия или отсутствия ошибок, у вызывающего
плагина нет шансов понять, что что-то не в порядке

2) сообщение об ошибке не указывает макрос, это отдельный тикет:
http://bugs.farmanager.com/view.php?id=1269
но в идеале эта инфа должна бы не только в UI показываться, но и возвращаться
вызывающему, а вот мессаджбокс при вызове MCMD_LOADALL должен бы быть
опциональным.
TagsNo tags attached.
Build1397

Activities

vskirdin

2010-02-11 18:10

administrator   bugnote:0004898

1. MCMD_LOADALL вернет TRUE или FALSE. В случае FALSE вообще никакие макросы не будут загружены.
Хмм... т.е. плагину нужно что-то типа MCMD_GETLASTERROR?

NightRoman

2010-02-11 18:23

reporter   bugnote:0004899

> 1. MCMD_LOADALL вернет TRUE или FALSE. В случае FALSE вообще никакие макросы не будут загружены.
Это уже гуд.

> Хмм... т.е. плагину нужно что-то типа MCMD_GETLASTERROR?
Не знаю даже. А чего тут плуг может сделать? Знать, что был сбой и не начать что-то делать, что не надо при таком сбое - уже можно. Может этого и достаточно. Сообщение о сбое юзер видел, имя макроса теперь там показано, пусть фиксит свои макросы и потом уже работает с плугом нормально.

vskirdin

2010-02-11 18:36

administrator   bugnote:0004901

тогда Инц закрываем?

NightRoman

2010-02-11 18:40

reporter   bugnote:0004902

думаю, да, чего тут мудрить. но не протестировано еще :)

vskirdin

2010-02-12 13:52

administrator   bugnote:0004914

протестил?

NightRoman

2010-02-12 13:57

reporter   bugnote:0004916

не понял :) а что оно уже там?

NightRoman

2010-02-12 14:01

reporter   bugnote:0004917

ты либо не зачекинил (и я не видел в ченжлоге), либо не работает

vskirdin

2010-02-12 14:51

administrator   bugnote:0004920

я описал как есть, если что-то не так - давай подробности

NightRoman

2010-02-12 14:58

reporter   bugnote:0004921

создаю в реестре мусорный макрос с кодом zzz.
гружу фар, он ругается (диалог), хорошо.

потом вызываю MCMD_LOADALL - она всегда возвращает true, даже при наличии проблемы выше (тоже ругается - диалогом, но не возвратом false).

vskirdin

2010-02-12 15:18

administrator   bugnote:0004922

Нашел.
Вот же срань :-(

build 1397

NightRoman

2010-02-12 15:50

reporter   bugnote:0004924

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

Issue History

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