Реализовать форму редактирования вопроса correctwriting

[Oleg Sychev]

Originally reported on Google Code with ID 100

Реализовать форму редактирования вопроса correctwriting

Reported by oasychev on 2012-03-20 13:28:23

[Former user]

``` Попытался рассмотреть форму, которую создал Коновалов, и в связи с этим возникло множество вопросов. Дело в том, что код в его репозитории не работает (там множество ошибок синтаксического характера, помимо просто ошибок совместимости с новыми версиями PHP), но это не важно - это удалось исправить. Не совсем понятна форма ввода лексем. Кнопка "разбить на лексемы" не делает ничего, а формат ввода лексем непонятен. Можете кратко объяснить, что там происходит и как вводить лексемы? ```

Reported by `mamontov.dp` on 2012-04-14 13:07:51

[Oleg Sychev]

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

Суть была примерно такая: по кнопке или первой попытке сохранения (через ошибку валидации) разбить каждый ответ на лексемы (при наличии парсера - определить узлы, требующие имена) и выдать TextEdit в котором названием (меткой) являлись бы строки лексем, разделенные тегами переноса строки (каждая лексема на отдельной строке) - соответственно юзер записывал бы напротив нее описание (т.е. описания вводятся через перевод строки). Идея была та, что так легче, чем генерировать на каждую лексему отдельную строку ввода, переключаться между ними табуляцией или мышью и т.д.... ```

Reported by `oasychev` on 2012-04-14 20:27:43

[Oleg Sychev]

``` Дмитрий, а почему вы не наследуете форму от shortanswer как это делает preg? Вас не учили, что копировать код - не самая лучшая практика? ```

Reported by `oasychev` on 2012-04-16 14:41:32

[Former user]

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

Reported by `mamontov.dp` on 2012-04-16 15:45:57

[Oleg Sychev]

``` Смысл в наследовании - не дублировать код тех полей/методов, которые общие - и не переносить вручную изменения в них когда они сделаны в shortanswer.

Работая в параллель с такой командой как Moodle HQ вы можете, если хотите, на своей шкуре прочуствовать весь вред копирования кода. Гораздо лучше когда эти изменения наследуются и редко требуют вашего внимания... ```

Reported by `oasychev` on 2012-04-17 11:43:40

[Former user]

``` Хорошо, я посмотрю, как можно изменить код, чтобы можно было использовать поля от shortanswer. ```

Reported by `mamontov.dp` on 2012-04-17 14:46:09

[Oleg Sychev]

``` 34. Форма: подумать и проставить Advanced для более тонких настроек... ```

Reported by `oasychev` on 2012-08-09 15:37:30

[Former user]

Вроде как выполнено, имеет смысл проверить и закрыть

Reported by mamontov.dp on 2012-10-26 09:25:51

[Oleg Sychev]

34 (см. выше) выполнено?

И если где-то нет описаний или проблемы с ними - надо выводить ошибку не для всего
вопроса, а для конкретной формы с описаниями!

Reported by oasychev on 2012-10-26 12:04:21

[Oleg Sychev]

"И если где-то нет описаний или проблемы с ними - надо выводить ошибку не для всего
вопроса, а для конкретной формы с описаниями!" - это сейчас работает.

Но вот пустой список лексем при вводе описания остается почему-то, хотя обновил по
последним данным в коде....

Reported by oasychev on 2012-11-01 11:53:05

[Former user]

Для решения этой проблемы мне необходима версия вашего Moodle и PHP. На текущий момент
у меня стоит PHP 5.3.10-1ubuntu3.4 with Suhosin-Patch (cli) и Moodle 2.4(Build: 20120802).
Вполне возможно, что у вас новее, поэтому мне важно знать вашу версию, т.к. у меня
все это работает. 

Reported by mamontov.dp on 2012-11-01 16:39:40

[Oleg Sychev]

Для срочной проблемы я #160 сделал и предложил вам там помочь с отладочной печатью чтобы
понять проблему. 

А в этом issue остается вопрос advanved настроек. А то нормальный учитель если в первый
раз увидит форму со столькими настройками, он закроет ее не создав свой первый вопрос...

Reported by oasychev on 2012-11-04 12:01:32

[Oleg Sychev]

34 выполнено, пока претензий к форме больше нет.

Reported by oasychev on 2012-11-05 12:26:07 - Status changed: Fixed

[Oleg Sychev]

При выделении жирным - разве в qtype_poasquestion_string нет подходящих функций? Это
лучше, чем напрямую вызывать mb_string. 
Я же просил посмотреть, как выделяет preg (поищите функцию highlight_regex)...

Reported by oasychev on 2012-11-08 15:40:50 - Status changed: InProgress

[Oleg Sychev]

Я бы еще добавил простую проверку, чтобы не отправлять на лексинг пустые ответы (там,
где trim дает пустую строку trim($answer)=='' ) и не создавать для них окошек для ввода
лексем.

Reported by oasychev on 2012-11-15 10:51:12

[Oleg Sychev]

Дальнейшие усовершенствования переносятся в отдельные задачи

Reported by oasychev on 2012-12-14 17:10:48 - Status changed: Done