Изменение look_for_matches в block_formal_langs_token_base

[Nikita Penskoy]

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

Соответственно, исправить вызовы функции и тесты. Проверить, что другие тесты работают.

[Nikita Penskoy]

Не могу назначить issue на себя.

[Nikita Penskoy]

Изменил, репозиторий https://bitbucket.org/ShadowGorn/moodle-plugins-formallangs-stanford-english

[Oleg Sychev]

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

[Oleg Sychev]

Пожалуйста, отучитесь коммитить отладочную печать, даже в закомментированном виде - и удалите из коммита. В юнит-тестах иногда это уместно,в коде - ни в коем случае...

Также желательно переписать комментарии (которые вы меняете или вводите) в стиле Moodle - предложениями с большой буквы, заканчивающимися точкой.

[Nikita Penskoy]

Это, собственно, не мое. Так и было, просто из удаления if весь блок на табуляцию влево уехал. Убрать, это не нужные части?

[Oleg Sychev]

Вижу, но есть правило когда если затрагиваешь унаследованный код нарушающий правила кодирования, то исправляешь его при этом. Незатрагиваемый исправляется отдельными коммитами.

[Nikita Penskoy]

Необходимо сделать 2 коммита: первый со стилистическими правками, второй с изменениями, я правильно понял?

[Oleg Sychev]

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

Отдальный коммит для стилистических правок нужен если выправляется весь файл или вся функция, а не только затронутые изменением строки.

[Nikita Penskoy]

Исправил, можете посмотреть? https://bitbucket.org/ShadowGorn/moodle-plugins-formallangs-stanford-english

[Dmitry Mamontov]

Никита, а вы прогоняли тесты на функции, работа которых зависит от данной функции?

Навскидку: https://bitbucket.org/ShadowGorn/moodle-plugins-formallangs-stanford-english/src/fe5424e9fee3d162f00f6a062a4907a6829f7d19/blocks/formal_langs/tests/test_best_string_pairs.php?at=default&fileviewer=file-view-default

https://bitbucket.org/ShadowGorn/moodle-plugins-formallangs-stanford-english/src/fe5424e9fee3d162f00f6a062a4907a6829f7d19/blocks/formal_langs/tests/test_best_string_pairs_for_bypass.php?at=default&fileviewer=file-view-default

https://bitbucket.org/ShadowGorn/moodle-plugins-correctwriting-penskoy/src/290c9caa2856bdd255112c2668e553746d7099c2/question/type/correctwriting/tests/test_lexical_analyzer.php?at=default&fileviewer=file-view-default

https://bitbucket.org/ShadowGorn/moodle-plugins-correctwriting-penskoy/src/290c9caa2856bdd255112c2668e553746d7099c2/question/type/correctwriting/tests/test_stress_lexical_analyzer.php?at=default&fileviewer=file-view-default

Там не все тесты могут проходить, в частности, тому мешает отсутствие специфичных ошибок, но остальное должно работать.

[Nikita Penskoy]

Первые точно тестировал, вторые вроде тоже. Но все, что тестировал только в formal_langs.

Я посмотрю тесты в correctwriting, где еще может использоваться?

[Dmitry Mamontov]

Стоит просмотреть эти тесты в актуальном correctwriting.

Стоит ещё прогнать работу пользователя с тестовыми вопросами при включенном lexical_analyzer.

[Nikita Penskoy]

Прошу, поясните по поводу прогона работы пользователя.

Из данных четырех файлов разницы в ошибках с предыдущей версией нет.

[Oleg Sychev]

Дмитрий, я думаю теперь вам стоит это протестить.

[Dmitry Mamontov]

Хорошо. Протестирую в воскресенье - раньше просто не могу.

[Dmitry Mamontov]

Протестировал. Пока замечаний нет, вроде бы все работает верно.

[Oleg Sychev]

Владислав, если что-то непонятно по новому интерфейсу look_for_matches - пишите сюда как вариант вопросы.

[Nikita Penskoy]

https://bitbucket.org/ShadowGorn/moodle-plugins-formallangs-stanford-english