1. Oleg Sychev Avatar Oleg Sychev
  2. POAS Moodle plugins
  3. Moodle Plugins
  4. Issues

Доработка block supervised к релизу

Issue #258 closed
Oleg Sychev repo owner created an issue

Originally reported on Google Code with ID 258

Вытолкнул небольшие изменения в языковые файлы.

Вопросы и проблемы:
1. Для логов вроде были viewallsessions/viewownsessions. Какие были особые причины
давать смотреть свои логи если есть supervise но нет viewownsessions?

2. Подчистка строк
а) удалить строки, имеющиеся в файлах Moodle, и перенаправить get_string туда (например
IP list и его справку можно взять из quiz). Копипаст - зло; Moodle HQ будет поддерживать
эти строки лучше нашего...

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

в) в длинных фразах типа писем избегать дублирования - выделить общую часть (данные
о сессии и т.д.) и подставлять ее через $a в сообщения. Также строки типа You can edit
this session ... You can delete it должны добавляться не всегда, а только если пользователь
имеет на это право.

г) строковые файлы принято упорядочивать по алфавиту ключей - так легче искать нужную
строку; в крайнем случае несколько секций по алфавиту ключей (но это у нас в preg где
этих строк жуткое количество) - отдельным коммитом от других изменений!

д) заменить teacher на superviser в строках при создании сесиии?

3. Русский язык - как минимум для строк студента, лучше и для учителя тоже... Администрирование
можно оставить

4. Кое-где в коде - двойные кавычки без надобности (например \blocks\supervised\logs\view.php
строка 78) - оставить двойные только там, где нужны спец символы или подстановки переменных.

Reported by oasychev on 2014-02-15 16:26:28

Comments (18)

  2. Oleg Sychev reporter

    Reported by oasychev on 2014-02-15 17:05:25 - Labels added: Priority-High - Labels removed: Priority-Medium

  3. Former user Account Deleted 
    1. Пользователь с правом supervise может начинать занятие из блока. В теле блока при
этом появляется ссылка "Просмотр логов сессии". Т.е. он может смотреть логи текущей
сессии.

2.
а) Сделано. Для строки 'IP list' нет прямого аналога в quiz. Там есть 'Require network
address', что в нам не подходит. Справки по формату IP адресов также нет, я ее составлял
по документации Moodle.

в) Сделано

г) Сделано

д) Мне кажется, термин 'teacher' более понятен, т.к. создается занятие, где этот самый
учитель есть.

3. Сделано

4. Что попалось на глаза - исправил, по всему коду пока не стал искать...

    Reported by ushakov.andrey.34 on 2014-02-17 10:58:09

  4. Oleg Sychev reporter 
    2 д) дело в том, что teacher отнюдь не универсальное слово в английском языке, и может
напрягать - там может быть и professor и assistant, и laborant (например если человек
тупо следит чтобы не списывали сдавая тесты, так делается) и много что еще. Это напрягает
людей, которые названы неверно - и может сбить с толку. И роли соответствующие на сайте
тоже могут называться по разному (точно также ученик может быть student pupil learned).
Используя понятие superviser мы добиваемся единства терминологии в рамках блока - используя
технический термин "тот, кто проводит наблюдение"....

    Reported by oasychev on 2014-02-17 13:16:04

  5. Oleg Sychev reporter 
    p.s. самое серьезное, конечно, в 260...

    Reported by oasychev on 2014-02-17 13:16:25

  6. Former user Account Deleted 
    п. 2д) и п. 4) сделано.

    Reported by ushakov.andrey.34 on 2014-02-22 12:28:18 - Status changed: Fixed

  7. Oleg Sychev reporter 
    4) нашел случаи CamelCase в именах функций и переменных; по стандарту не положено. В
правиле исправил - проверьте блок кодечекером, он это пишет.

5) Там в правиле в lib.php TODO осталось - вроде по сути надо доделывать, не оставлять
же мусор при удалении курса и т.д.

    Reported by oasychev on 2014-02-22 21:45:56 - Status changed: InProgress

  8. Oleg Sychev reporter 
    6) формальности
в комментариях к файлам строку 
// This file is part of Moodle - http://moodle.org/
заменить на 
// This file is part of Student Access Control Kit - https://code.google.com/p/oasychev-moodle-plugins/

И далее в этом комментарии заменить Moodle на Student Access Control Kit- кроме строки
про получение GPL.

Также кроме строки @author добавить во все файлы
@copyright 2014 Oleg Sychev, Volgograd State Technical University
(поддержка копирайта под GPL только муть и никакой выгоды, но универ лишний раз прорекламировать
надо)

    Reported by oasychev on 2014-02-22 22:02:20

  9. Former user Account Deleted 
    4,5,6 Сделано

    Reported by ushakov.andrey.34 on 2014-03-03 16:18:34 - Status changed: Fixed

  10. Oleg Sychev reporter 
    В форме начала занятия, раз уж строка для ввода длительности на отдельной строке от
надписи оказалась, может ее по центру выровнять, а то она маленькая и некрасиво как-то
влево прижата.

    Reported by oasychev on 2014-03-04 18:51:41

  11. Former user Account Deleted 
    Я немного менял вывод тела блока, сейчас он выглядит так:
Блок начала занятия http://screencast.com/t/J5xzYqq8DuZ1
Блок начала запланнированного занятия http://screencast.com/t/wobl1NWj
Блок активного занятия http://screencast.com/t/PJA4eFaEMe
Блок студента http://screencast.com/t/DX78grboIi

    Reported by ushakov.andrey.34 on 2014-03-04 19:17:07

  12. Oleg Sychev reporter 
    Похоже мы оба вытолкнули проверку наличия таблицы. Слейте аккуратно изменения - сможете?

Посмотрите на edu.vstu.ru - я сегодня обновлял там (нет только последней проверки наличия
таблицы) - там выглядит по-другому. Возможно зависит еще от экрана и/или браузера...

    Reported by oasychev on 2014-03-04 19:23:52

  13. Former user Account Deleted 
    Скорее всего, это связано с русской версией блока, у меня это выглядит так:
Блок начала занятия http://screencast.com/t/xjrQsMdHxG
Блок начала запланированного занятия http://screencast.com/t/kj9dHKNjZ
Блок активного занятия http://screencast.com/t/5ICvD62pP

    Reported by ushakov.andrey.34 on 2014-03-04 19:50:41

  14. Former user Account Deleted 
    * изменения слил

    Reported by ushakov.andrey.34 on 2014-03-04 19:51:14

  15. Oleg Sychev reporter 
    Вот на вашей картинке "блок начала занятия" - вам не хочется само окошко с цифрами длительности
выровнять по центру? Оно как-то некрасиво влево прижато, а справа куча пустого места...

    Reported by oasychev on 2014-03-05 19:40:17

  16. Former user Account Deleted 
    Проблема в том, что по умолчанию это окошко находится на одной строке со своим лейблом,
 в англ. версии это так, а в русской текст чуть длиннее и оно переносится на следующую
строку. Мы же не сможем сделать выравнивание по центру только если выбран русский язык...
Думаю, нужно либо переносить его на след. строку всегда во всех языках и тогда можно
будет делать выравнивание по центру, либо как-то сократить русский перевод лейбла (в
другой теме, где шрифт может быть чуть крупнее, опять могут быть проблемы).

Для единообразия можно все поля и их лейблы разместить на отдельных строках. А то сейчас
как-то смешанно получается: http://screencast.com/t/PJA4eFaEMe
Как вы считаете?

    Reported by ushakov.andrey.34 on 2014-03-05 20:30:42

  17. Former user Account Deleted 
    В итоге инпут ввода длительности уместился на одной строке со своим лейблом: http://screencast.com/t/jppiYWY28

    Reported by ushakov.andrey.34 on 2014-03-09 09:14:53

  18. Oleg Sychev reporter 
    Вижу, теперь нормально.

    Reported by oasychev on 2014-03-09 21:16:26 - Status changed: Done

  19. Log in to comment
Assignee
–
Type
bug
Priority
major
Status
closed
Component
–
Votes
0
Watchers
0
Jira: the preferred issue tracker for Bitbucket. Join the team!