Инструменты авторинга при некорректном регексе

Issue #194 closed
Oleg Sychev repo owner created an issue

Originally reported on Google Code with ID 194

В отличие от матчера, инструменты авторинга должны работать при некорректном регексе.
Дерево может отображать почти любые ошибки - как и описание. Граф - некоторые (это
должно решаться аццептингом узла ошибки с вниманием к ее типу). Управлять возможность
отображения дерева/графа/описания должен аццептинг, а не наличие ошибок. 

Это помогает юзерам находить и исправлять ошибки. Поэтому с последним коммитом, отменяющим
построение конкретного дерева при ошибках я не согласен.

Давайте разберемся и наладим работу инструментов авторинга в случае некорректного регекса.
С алертами надо разбираться - система узлов дерева с ошибками и вводилась для того,
чтобы иметь возможность отобразить регекс при наличии ошибок.

P.S. Строки типа 
dotscript = 'digraph { "Ooops! Your regex contains errors, so I can\'t build the explaining
graph!" [color=white]; }'; 
вообще мне странны. Вроде все участники давно в курсе, что любая строка, выводимая
пользователем, должна идти через файл языка чтобы переводиться...

Reported by oasychev on 2013-04-21 21:23:01

Comments (6)

  1. Valeriy Streltsov
    Тогда 3 курсу нужно написать акцептинг. Более-менее адекватный сейчас только у Иванова,
    у остальных - принимается всё и всегда.
    
    Я кстати не уверен, что owner'ом вы меня правильно поставили, т.к. бОльшая часть задачи
    - именно аксептинг у инструментов авторинга. В хэндлере поправить обратно 2 строчки
    недолго.
    

    Reported by vostreltsov on 2013-04-21 21:45:04

  2. Oleg Sychev reporter
    На самом деле, если вы подумаете, то поймете что дерево и описание действительно могут
    отобразить все или почти все. Если чего-то нет ни в аццептинге, ни в инструментах -
    надо посмотреть, куда его добавить - может и в дерево/описание, а не откидывать.
    
    Вы начали, поэтому пока владелец. Вернуть строки мало, надо изменить функции сообщения
    об ошибке в регексе на функцию вывода сообщений о непрохождении аццептинга например...
    Ну и сделать анализ алертов, чтобы знать, кому их исправлять.
    

    Reported by oasychev on 2013-04-21 21:54:15

  3. Valeriy Streltsov
    Вернул построение dst как было; перенес проверку на якорение в матчер под условием что
    ошибок нет - алертов тоже нет, все хорошо.
    
    Поскольку заговорили про акцептинг, Терехову также нужно переопределить всякие функции
    типа получения имени компонента и т. д., у остальных вроде нормально.
    

    Reported by vostreltsov on 2013-04-21 22:20:46

  4. Oleg Sychev reporter
    Валерий, по этому issue есть еще проблемы? У Терехова вполне может быть аццептинг всего
    на свете - в дереве не проблема отобразить все...
    

    Reported by oasychev on 2013-07-02 14:21:46

  5. Oleg Sychev reporter
    Валерий, вы не ответили на последний комментарий.
    

    Reported by oasychev on 2013-07-12 14:45:21 - Status changed: Fixed

  6. Valeriy Streltsov
    Тут вроде всё нормально.
    

    Reported by vostreltsov on 2013-07-13 05:20:41 - Status changed: Done

  7. Log in to comment