testing your calendar, I've found some notices/warnings (only with debug on):
- In lang/en/calendar: line 11. $string['default'] hasn't content.
- In calendar/lib.php; line 1096, the $USER->teacher variable could not exist so it could be changed to if(isset($USER->teacher) && is_array($USER->teacher)) { to avoid the notice.
- In calendar/lib.php: line 549 and 551 throws one notice about undefined $prevyear and $nextyear variables.
- In calendar/lib.php: line 1154, the same about undefined $starttime variable.
- In calendar/view.php: line 556, aprox, the $select variable is empty if the user isn't a teacher or a student (SQL error). This behaviour relies in the calendar_get_default_courses() function. Instead of using some variables as $USER, shouldn't it be better to use some Moodle functions like isadmin() isteacher()...
- In calendar/event.php, with debug=on, some notices are showed *inside* the form fields. Perhaps they should be checked to avoid that effect.
- About day names and/or month names. Is possible to take them from locales instead of using the lang files?
Thanks for your patience, ciao
