Обзор и интеграция кода специального списка группы
Issue #419
resolved
Была проведена очень полезная разработка по коду, позволяющему составлять группы интенсива - из конкретного списка студентов.
Нужно посмотреть качество кода, Анастасия - хочу услышать ваше мнение - если примем, нам его поддерживать. Сейчас код должен быть работоспособен.
Что точно надо сделать - поправить терминологию на экране и в коде, там сейчас называется это internship почему-то.
Репозиторий с кодом - https://bitbucket.org/saodem74/moodle-plugins-supervised/commits/all
Comments (14)
-
-
reporter
Этот файл скопирован и переделан под наши таблицы с аналогичного файла Moodle, ответственного за редактирование состава групп и когорт - например moodle/group/members.php Там похожая структура. Если сама переделка нормальна (можете сравнить файлы через diff - что изменилось), то в данном случае я не вижу проблемы. Лучшим решением было бы только абстрагировать код в Moodle и сделать его реюзабельным, но это уже отдельный проект...
-
Вопрос по поводу требований для блока. Блоку supervised определенно нужна версия Moodle 2016032400? У меня стоит более ранняя версия, но Moodle 3. Если мы делаем блок для Moodle 3, то можно ли сделать так, чтобы блок работал и на более ранних версиях, начиная от Moodle 3.0?
У меня на данный момент стоит версия Moodle 2015111602. (Moodle 3.0.2)
-
reporter
Попробуйте чисто у себя на тестировочном сайте уменьшить версию (без репозитория) и посмотреть, все ли будет нормально.
-
Вот на такую ошибку наткнулась сейчас. Если я выбираю в списке Internship group, но у меня нет ни одной, то возникает ошибка. Я думаю, в первое время все пользователи, не знавшие о новом функционале, и о том, что нужно сделать для работы с internship группами, натыкались бы на эту ошибку. В перенаправлении на создание группы internship тоже возникает ошибка в строке https://bitbucket.org/saodem74/moodle-plugins-supervised/src/011cd1c91705bac6a3caad128b23ed5ee9ad27c7/blocks/supervised/groups/members.php?at=default&fileviewer=file-view-default#members.php-203 , где используется неинициализированная переменная.
-
Еще одна особенность. Данное дополнение каждый раз при запуске блока с Internship Group создает новую Internship Group, даже если у меня уже есть такая. Названия групп одинаковые, никаких отличительных признаков нет. Таким образом после нескольких запусков блока, у меня вот такая ситуация в списке групп. И параллельно вопрос возникает - можно ли вообще разрешать занятие, если в эту Internship group еще никто не добавлен. На данный момент просто создается пустая группа, и с пустой группой начинается занятие.
-
По основному функционалу протестировала: взяла двух пользователей, забросила их в разные студенческие группы, потом сначала в одну internship группу, затем в две разные. Доступ к тестам работает как и должен. Единственный вопрос возник. Такая ситуация - есть преподаватель курса и админ. Оба начали сессию в одном курсе, преподаватель начал сессию, в котором может решать тест студент А и студент Б, у админа начат курс, в котором может решать тест только студент Б. Студент А заходит на курс и может решать тест. Но там стоит в супервайзерах преподаватель. У нас преподаватель по статусу выше, чем админ? (Сессия преподавателя началась раньше, чем сессия админа).
-
reporter
Насчет одинаковых названий - это почти нормально, единственный способ сделать их разными - это заставить пользователя ввести название группы, но я не думаю что кто-то будет это делать под каждое занятие интенсива. Можно сделать настройку на курс - как должны называться такие группы в рамках данного курса.
Два преподавателя могут вести сессии параллельно, хотя ситуация с сессиями в одной аудитории и одними студентами немного вырожденная. По идее не вижу проблемы, к кому именно попадет студент в этом случае - разрешено ему по-любому.
-
reporter
А вот группу бы надо переименовать - в ad hoc например, синонимов я по тезаурусу не нашел. И в интерфейсе, и в названии переменных в коде программы.
-
ad hoc подходит по смыслу, я думаю, что specific group или special group меньше подойдут.
-
reporter
Анастасия, можете посмотреть исправления в репозитории - там три коммита - по исправлению самих ошибок? И само переименование группы в adhoc в сообщениях и коде программы, чтобы нам не иметь проблем? Чтобы мы где-нибудь в течении недели-двух сделали релиз...
-
да, я посмотрю, конечно.
-
Нашла еще интересный момент - при нажатии Update Session идет переадресация на страницу редактирования членов Internship Group.
-
reporter
- changed status to resolved
- Log in to comment
Олег Александрович, я вот не видела еще ни разу, чтоб в файлах в moodle совмещали php и html, мы разве не с помощью php делали формы?
https://bitbucket.org/saodem74/moodle-plugins-supervised/commits/3bf65ce61e9983a999e1ab73cad8b715fcd9c452#Lblocks/supervised/groups/members.phpT140