Оценка по НИР за 1-й семестр 1-го курса

Issue #317 closed
Oleg Sychev repo owner created an issue

Originally reported on Google Code with ID 317

- правила должны быть утверждены
- юнит-тесты на правила реализованы в коде
- проект интерфейса утвержден

Reported by oasychev on 2014-12-24 13:51:59

Comments (13)

  1. Oleg Sychev reporter
    А чего комментарий удален?
    1. По чарсетам мы просто сравниваем интервалы. Наоборот - если есть альтернативы длиной
    в один символ, их можно свести к чарсетам.
    2. По простым ассертам типа ^ $ - уточните у Валерия, без preserveallnodes там уже
    преобразование производится с учетом опций, должно хватить.
    
    
    И что там с юнит-тестами на это все?
    

    Reported by oasychev on 2015-01-20 14:19:20

  2. Former user Account Deleted
    Комментарий не я оставлял, это исью для генерации подсказок, видимо автор ошибся и удалил
    его. Возможно ваш ответ тоже должен быть вот здесь: https://code.google.com/p/oasychev-moodle-plugins/issues/detail?id=316&colspec=ID%20Type%20Status%20Priority%20Component%20Milestone%20Owner%20Summary&start=100
    
    А теперь мой вопрос. Moodle поддерживает bootstrap 2 (https://docs.moodle.org/dev/Bootstrap).
    Могу ли я, используя bootstrap определять виджеты для формы авторинга? А то стандартные
    возможности из Form API далеко не позволяют сделать то, что можно быстро, просто и
    красиво сделать с помощью bootstrap.
    

    Reported by grvlter on 2015-01-21 21:38:06

  3. Former user Account Deleted
    Поправленные правила можно посмотреть тут: https://drive.google.com/drive/#folders/0B0Xn1SbBj6Nic1psck53cTRvd2c/0B0Xn1SbBj6NieTIxeE43SGpZNXM
    

    Reported by grvlter on 2015-01-25 19:04:37

  4. Oleg Sychev reporter
    Насчет bootstrap - я не уверен что " Moodle now has some themes that build upon the
    best practices, documentation, tools and resources that it provides." означает "поддерживает
    bootstrap 2". Вы можете найти код bootstrap в репозитории Moodle? Если есть, тогда
    можно пользоваться. 
    

    Reported by oasychev on 2015-01-25 21:58:27

  5. Former user Account Deleted
    По поводу интерфейса.
    Ниже приведен просто захардкоденый пример, что бы можно было что-то показать, никак
    с самим регурялным выражением не связанный.
    В свернутом положении выглядит так:
    http://prntscr.com/5x205r
    Лейблы рядом с названием инструмента собственно отображают тип ошибки и их количество.
    Предполагается, что если лейбла нет, то нет и ошибки.
    При раскрытии окошка будет следующее:
    http://prntscr.com/5x23j8
    Имеется список подсказок к регулярному выражению на фоне цвета, соответствующего типу
    подсказки. Для удобства кнопочки принять и отколнить (выделены красным) рядом с самой
    подсказкой, продублированы рядом с подробным описанием (выделены зеленым) для удобства.
    Что касается подробного описания, то оно отображается при нажатии на цветную подсказку.
    Так же сделаю настройки включения/отключения типа подсказок и справку.
    Для дополниельного удобства предлагаю убирать вообще с формы этот инструмент, если
    не найдено подсказок для регулярного выражения.
    

    Reported by grvlter on 2015-01-25 22:17:19

  6. Former user Account Deleted
    Проблема в том, что в репозитории мудл есть... бутстрап адаптированный под мудл. Это
    значит, что я могу им верстать (задавать стили для элементов), но в мудле я не нашел
    нужные мне скрипты бутстрапа. Думаю, не сильно кому испортим жизнь, если положим в
    poasquestion 1 js файлик...
    

    Reported by grvlter on 2015-01-25 22:19:34

  7. Former user Account Deleted
    Так же предлагаю к ближайшему релизу немного адаптировать нашу форму под bootstrap.
    Ну он реально удобен и реально помогает делать быстро клевые штуки. Например виджет
    yes/no можно сделать так: http://prntscr.com/5x286t . Это не займет много времени.
    И мне кажется это действительно удобным.
    

    Reported by grvlter on 2015-01-25 22:21:27

  8. Former user Account Deleted
    Тестовые ситуации описаны в коде, когда будет инструмент - ситуации будут вызваны для
    него. Смотреть тут: https://code.google.com/r/toptiiiii-preg-description-28/source/browse/question/type/preg/tests/equality_test.php
    вот этот метод: public function complex_provider() 
    

    Reported by grvlter on 2015-01-28 08:01:13 - Status changed: Fixed

  9. Oleg Sychev reporter
    Давайте разбираться: у вас с Пахомовым РАЗНЫЕ правила и разные тесты (и разные методы).
    Ни в коем случае нельзя их смешивать. Потому что цели разные.
    
    Если точнее, Пахомов может использовать тесты от Терехова но не может и не должен ими
    ограничиваться, потому что его понятие эквивалентности шире. 
    

    Reported by oasychev on 2015-01-28 11:32:36

  10. Oleg Sychev reporter
    Вам с Пахомовым надо иметь два набора тестов: один общий, другой у каждого разный.
    У вас есть "рекомендательные" правила с неточными преобразованиями (например пробел
    в \s и т.д.), которые Пахомову не подходят. Где на них тесты?
    

    Reported by oasychev on 2015-02-04 17:34:39

  11. Former user Account Deleted
    Добавли тесты, посмотреть тут: https://code.google.com/r/grvlter-preg-29/source/detail?r=74a069a95ed57e6290569f20c3d4c899b1fc5cb9
    

    Reported by grvlter on 2015-02-04 22:13:39

  12. Former user Account Deleted
    Вот ещё пачка новеньких тестовых ситуаций: https://code.google.com/r/grvlter-preg-29/source/detail?r=2265801d50bf5075bdb7763abf76cdcf843c58a9
    

    Reported by grvlter on 2015-02-05 22:08:00

  13. Log in to comment