Merge branch 'w41_MDL-42199_m26_rev' of https://github.com/skodak/moodle
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Tue, 8 Oct 2013 08:53:03 +0000 (10:53 +0200)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Tue, 8 Oct 2013 08:53:03 +0000 (10:53 +0200)
57 files changed:
admin/tool/behat/tests/behat/manipulate_forms.feature
admin/tool/uploaduser/user_form.php
backup/moodle2/backup_stepslib.php
backup/util/dbops/restore_dbops.class.php
blocks/calendar_month/block_calendar_month.php
blocks/calendar_upcoming/block_calendar_upcoming.php
calendar/classes/type_base.php
calendar/classes/type_factory.php
calendar/delete.php
calendar/event.php
calendar/export.php
calendar/export_execute.php
calendar/lib.php
calendar/preferences.php
calendar/renderer.php
calendar/tests/behat/behat_calendar.php [new file with mode: 0644]
calendar/tests/behat/calendar.feature [new file with mode: 0644]
calendar/tests/calendartype_test.php
calendar/tests/calendartype_test_example.php
calendar/type/gregorian/classes/structure.php
calendar/type/gregorian/version.php
calendar/view.php
grade/edit/tree/grade.php
lang/en/admin.php
lang/en/moodle.php
lang/en/notes.php
lib/adminlib.php
lib/classes/event/note_created.php [new file with mode: 0644]
lib/classes/event/note_deleted.php [new file with mode: 0644]
lib/classes/event/note_updated.php [new file with mode: 0644]
lib/classes/event/notes_viewed.php [new file with mode: 0644]
lib/db/install.xml [changed mode: 0644->0755]
lib/db/upgrade.php
lib/deprecatedlib.php
lib/editorlib.php
lib/form/dateselector.php
lib/form/datetimeselector.php
lib/form/htmleditor.php
lib/moodlelib.php
lib/tests/editorlib_test.php [new file with mode: 0644]
mod/assignment/lib.php
mod/assignment/type/online/assignment.class.php
mod/data/field/textarea/field.class.php
mod/data/templates.php
mod/feedback/edit_item.php
mod/feedback/show_nonrespondents.php
mod/quiz/report/grading/report.php
notes/index.php
notes/lib.php
notes/tests/events_test.php [new file with mode: 0644]
notes/tests/generator/lib.php [new file with mode: 0644]
notes/tests/generator_test.php [new file with mode: 0644]
tag/edit.php
user/editlib.php
user/message.html
user/messageselect.php
version.php

index 2b9ed46..9aa1c88 100644 (file)
@@ -10,10 +10,10 @@ Feature: Forms manipulation
     And I follow "Admin User"
     And I follow "Edit profile"
     When I fill in "First name" with "Field value"
-    And I select "Use standard web forms" from "When editing text"
+    And I select "Plain text area" from "Text editor"
     And I check "Unmask"
     Then the "First name" field should match "Field value" value
-    And the "When editing text" select box should contain "Use standard web forms"
+    And the "Text editor" select box should contain "Plain text area"
     And the "Unmask" checkbox should be checked
     And I uncheck "Unmask"
     And the "Unmask" checkbox should not be checked
index 26876ce..9dfc288 100644 (file)
@@ -237,19 +237,6 @@ class admin_uploaduser_form2 extends moodleform {
         $mform->addElement('select', 'autosubscribe', get_string('autosubscribe'), $choices);
         $mform->setDefault('autosubscribe', 1);
 
-        $editors = editors_get_enabled();
-        if (count($editors) > 1) {
-            $choices = array();
-            $choices['0'] = get_string('texteditor');
-            $choices['1'] = get_string('htmleditor');
-            $mform->addElement('select', 'htmleditor', get_string('textediting'), $choices);
-            $mform->setDefault('htmleditor', 1);
-        } else {
-            $mform->addElement('hidden', 'htmleditor');
-            $mform->setDefault('htmleditor', 1);
-            $mform->setType('htmleditor', PARAM_INT);
-        }
-
         $mform->addElement('text', 'city', get_string('city'), 'maxlength="120" size="25"');
         $mform->setType('city', PARAM_TEXT);
         if (empty($CFG->defaultcity)) {
index 7aaefe6..3877335 100644 (file)
@@ -1191,7 +1191,7 @@ class backup_users_structure_step extends backup_structure_step {
             'confirmed', 'policyagreed', 'deleted',
             'lang', 'theme', 'timezone', 'firstaccess',
             'lastaccess', 'lastlogin', 'currentlogin',
-            'mailformat', 'maildigest', 'maildisplay', 'htmleditor',
+            'mailformat', 'maildigest', 'maildisplay',
             'autosubscribe', 'trackforums', 'timecreated',
             'timemodified', 'trustbitmask');
 
index b1561d8..bdb3775 100644 (file)
@@ -1180,6 +1180,16 @@ abstract class restore_dbops {
                         $status = $DB->insert_record('user_preferences', $preference);
                     }
                 }
+                // Special handling for htmleditor which was converted to a preference.
+                if (isset($user->htmleditor)) {
+                    if ($user->htmleditor == 0) {
+                        $preference = new stdClass();
+                        $preference->userid = $newuserid;
+                        $preference->name = 'htmleditor';
+                        $preference->value = 'textarea';
+                        $status = $DB->insert_record('user_preferences', $preference);
+                    }
+                }
 
                 // Create user files in pool (profile, icon, private) by context
                 restore_dbops::send_files_to_pool($basepath, $restoreid, 'user', 'icon', $recuser->parentitemid, $userid);
index abe2808..41cd98e 100644 (file)
@@ -1,25 +1,71 @@
 <?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
+/**
+ * Handles displaying the calendar block.
+ *
+ * @package    block_calendar_month
+ * @copyright  2004 Eloy Lafuente (stronk7) {@link http://stronk7.com}
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
 class block_calendar_month extends block_base {
-    function init() {
+
+    /**
+     * Initialise the block.
+     */
+    public function init() {
         $this->title = get_string('pluginname', 'block_calendar_month');
     }
 
-    function preferred_width() {
+    /**
+     * Return preferred_width.
+     *
+     * @return int
+     */
+    public function preferred_width() {
         return 210;
     }
 
-    function get_content() {
-        global $USER, $CFG, $SESSION;
-        $cal_m = optional_param( 'cal_m', 0, PARAM_INT );
-        $cal_y = optional_param( 'cal_y', 0, PARAM_INT );
+    /**
+     * Return the content of this block.
+     *
+     * @return stdClass the content
+     */
+    public function get_content() {
+        global $CFG;
+
+        $calm = optional_param('cal_m', 0, PARAM_INT);
+        $caly = optional_param('cal_y', 0, PARAM_INT);
+        $time = optional_param('time', 0, PARAM_INT);
 
         require_once($CFG->dirroot.'/calendar/lib.php');
 
-        if ($this->content !== NULL) {
+        if ($this->content !== null) {
             return $this->content;
         }
 
+        // If a day, month and year were passed then convert it to a timestamp. If these were passed then we can assume
+        // the day, month and year are passed as Gregorian, as no where in core should we be passing these values rather
+        // than the time. This is done for BC.
+        if (!empty($calm) && (!empty($caly))) {
+            $time = make_timestamp($caly, $calm, 1);
+        } else if (empty($time)) {
+            $time = time();
+        }
+
         $this->content = new stdClass;
         $this->content->text = '';
         $this->content->footer = '';
@@ -41,12 +87,12 @@ class block_calendar_month extends block_base {
 
         list($courses, $group, $user) = calendar_set_filters($filtercourse);
         if ($issite) {
-            // For the front page
-            $this->content->text .= calendar_get_mini($courses, $group, $user, $cal_m, $cal_y, 'frontpage', $courseid);
-            // No filters for now
+            // For the front page.
+            $this->content->text .= calendar_get_mini($courses, $group, $user, false, false, 'frontpage', $courseid, $time);
+            // No filters for now.
         } else {
-            // For any other course
-            $this->content->text .= calendar_get_mini($courses, $group, $user, $cal_m, $cal_y, 'course', $courseid);
+            // For any other course.
+            $this->content->text .= calendar_get_mini($courses, $group, $user, false, false, 'course', $courseid, $time);
             $this->content->text .= '<h3 class="eventskey">'.get_string('eventskey', 'calendar').'</h3>';
             $this->content->text .= '<div class="filters calendar_filters">'.calendar_filter_controls($this->page->url).'</div>';
         }
index a387294..335bcc4 100644 (file)
@@ -1,25 +1,53 @@
 <?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
+/**
+ * Handles displaying the calendar upcoming events block.
+ *
+ * @package    block_calendar_upcoming
+ * @copyright  2004 Eloy Lafuente (stronk7) {@link http://stronk7.com}
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
 class block_calendar_upcoming extends block_base {
-    function init() {
+
+    /**
+     * Initialise the block.
+     */
+    public function init() {
         $this->title = get_string('pluginname', 'block_calendar_upcoming');
     }
 
-    function get_content() {
-        global $USER, $CFG, $SESSION;
-        $cal_m = optional_param( 'cal_m', 0, PARAM_INT );
-        $cal_y = optional_param( 'cal_y', 0, PARAM_INT );
+    /**
+     * Return the content of this block.
+     *
+     * @return stdClass the content
+     */
+    public function get_content() {
+        global $CFG;
 
         require_once($CFG->dirroot.'/calendar/lib.php');
 
-        if ($this->content !== NULL) {
+        if ($this->content !== null) {
             return $this->content;
         }
         $this->content = new stdClass;
         $this->content->text = '';
 
         $filtercourse    = array();
-        if (empty($this->instance)) { // Overrides: use no course at all
+        if (empty($this->instance)) { // Overrides: use no course at all.
             $courseshown = false;
             $this->content->footer = '';
 
index 6458220..b470ffc 100644 (file)
@@ -27,6 +27,16 @@ namespace core_calendar;
  */
 abstract class type_base {
 
+    /**
+     * Returns the name of the calendar.
+     *
+     * This is the non-translated name, usually just
+     * the name of the calendar folder.
+     *
+     * @return string the calendar name
+     */
+    public abstract function get_name();
+
     /**
      * Returns a list of all the possible days for all months.
      *
@@ -62,10 +72,76 @@ abstract class type_base {
      */
     public abstract function get_max_year();
 
+    /**
+     * Returns the number of days in a week.
+     *
+     * @return int the number of days
+     */
+    public abstract function get_num_weekdays();
+
+    /**
+     * Returns an indexed list of all the names of the weekdays.
+     *
+     * The list starts with the index 0. Each index, representing a
+     * day, must be an array that contains the indexes 'shortname'
+     * and 'fullname'.
+     *
+     * @return array array of days
+     */
+    public abstract function get_weekdays();
+
+    /**
+     * Returns the index of the starting week day.
+     *
+     * This may vary, for example in the Gregorian calendar, some may consider Monday
+     * as the start of the week, where as others may consider Sunday the start.
+     *
+     * @return int
+     */
+    public abstract function get_starting_weekday();
+
+    /**
+     * Returns the index of the weekday for a specific calendar date.
+     *
+     * @param int $year
+     * @param int $month
+     * @param int $day
+     * @return int
+     */
+    public abstract function get_weekday($year, $month, $day);
+
+    /**
+     * Returns the number of days in a given month.
+     *
+     *
+     * @param int $year
+     * @param int $month
+     * @return int the number of days
+     */
+    public abstract function get_num_days_in_month($year, $month);
+
+    /**
+     * Get the previous month.
+     *
+     * @param int $year
+     * @param int $month
+     * @return array previous month and year
+     */
+    public abstract function get_prev_month($year, $month);
+
+    /**
+     * Get the next month.
+     *
+     * @param int $year
+     * @param int $month
+     * @return array the following month and year
+     */
+    public abstract function get_next_month($year, $month);
+
     /**
      * Returns a formatted string that represents a date in user time.
      *
-     * @param int $date the timestamp in UTC, as obtained from the database
+     * @param int $time the timestamp in UTC, as obtained from the database
      * @param string $format strftime format
      * @param int|float|string $timezone the timezone to use
      *        {@link http://docs.moodle.org/dev/Time_API#Timezone}
@@ -75,7 +151,7 @@ abstract class type_base {
      *        if false then the leading zero is maintained
      * @return string the formatted date/time
      */
-    public abstract function timestamp_to_date_string($date, $format, $timezone, $fixday, $fixhour);
+    public abstract function timestamp_to_date_string($time, $format, $timezone, $fixday, $fixhour);
 
     /**
      * Given a $time timestamp in GMT (seconds since epoch), returns an array that represents
@@ -86,7 +162,7 @@ abstract class type_base {
      *        {@link http://docs.moodle.org/dev/Time_API#Timezone}
      * @return array an array that represents the date in user time
      */
-    public abstract function timestamp_to_date_array($time, $timezone);
+    public abstract function timestamp_to_date_array($time, $timezone = 99);
 
     /**
      * Provided with a day, month, year, hour and minute in the specific
@@ -97,7 +173,7 @@ abstract class type_base {
      * @param int $day
      * @param int $hour
      * @param int $minute
-     * @return array the converted day, month and year.
+     * @return array the converted date
      */
     public abstract function convert_to_gregorian($year, $month, $day, $hour = 0, $minute = 0);
 
@@ -110,7 +186,7 @@ abstract class type_base {
      * @param int $day
      * @param int $hour
      * @param int $minute
-     * @return array the converted day, month and year.
+     * @return array the converted date
      */
     public abstract function convert_from_gregorian($year, $month, $day, $hour = 0, $minute = 0);
 }
index 535dd2b..8a47a84 100644 (file)
@@ -74,7 +74,8 @@ class type_factory {
     public static function get_calendar_type() {
         global $CFG, $USER, $SESSION, $COURSE;
 
-        if (!empty($COURSE->id) and $COURSE->id != SITEID and !empty($COURSE->calendartype)) { // Course calendartype can override all other settings for this page.
+        // Course calendartype can override all other settings for this page.
+        if (!empty($COURSE->id) and $COURSE->id != SITEID and !empty($COURSE->calendartype)) {
             $return = $COURSE->calendartype;
         } else if (!empty($SESSION->calendartype)) { // Session calendartype can override other settings.
             $return = $SESSION->calendartype;
index 4f72df7..4074b1e 100644 (file)
@@ -69,8 +69,7 @@ $event->count_repeats();
 
 // Is used several times, and sometimes with modification if required
 $viewcalendarurl = new moodle_url(CALENDAR_URL.'view.php', array('view'=>'upcoming'));
-$viewcalendarurl->param('cal_y', userdate($event->timestart, '%Y'));
-$viewcalendarurl->param('cal_m', userdate($event->timestart, '%m'));
+$viewcalendarurl->param('time', $event->timestart, '%Y');
 
 // If confirm is set (PARAM_BOOL) then we have confirmation of initention to delete
 if ($confirm) {
index 027527b..bf1c6c2 100644 (file)
@@ -57,26 +57,34 @@ $action = optional_param('action', 'new', PARAM_ALPHA);
 $eventid = optional_param('id', 0, PARAM_INT);
 $courseid = optional_param('courseid', SITEID, PARAM_INT);
 $courseid = optional_param('course', $courseid, PARAM_INT);
-$cal_y = optional_param('cal_y', 0, PARAM_INT);
-$cal_m = optional_param('cal_m', 0, PARAM_INT);
-$cal_d = optional_param('cal_d', 0, PARAM_INT);
+$day = optional_param('cal_d', 0, PARAM_INT);
+$month = optional_param('cal_m', 0, PARAM_INT);
+$year = optional_param('cal_y', 0, PARAM_INT);
+$time = optional_param('time', 0, PARAM_INT);
+
+// If a day, month and year were passed then convert it to a timestamp. If these were passed
+// then we can assume the day, month and year are passed as Gregorian, as no where in core
+// should we be passing these values rather than the time. This is done for BC.
+if (!empty($day) && !empty($month) && !empty($year)) {
+    if (checkdate($month, $day, $year)) {
+        $time = make_timestamp($year, $month, $day);
+    } else {
+        $time = time();
+    }
+} else if (empty($time)) {
+    $time = time();
+}
 
 $url = new moodle_url('/calendar/event.php', array('action' => $action));
+
 if ($eventid != 0) {
     $url->param('id', $eventid);
 }
+
 if ($courseid != SITEID) {
     $url->param('course', $courseid);
 }
-if ($cal_y !== 0) {
-    $url->param('cal_y', $cal_y);
-}
-if ($cal_m !== 0) {
-    $url->param('cal_m', $cal_m);
-}
-if ($cal_d !== 0) {
-    $url->param('cal_d', $cal_d);
-}
+
 $PAGE->set_url($url);
 $PAGE->set_pagelayout('standard');
 
@@ -99,7 +107,7 @@ if ($action === 'delete' && $eventid > 0) {
     redirect($deleteurl);
 }
 
-$calendar = new calendar_information($cal_d, $cal_m, $cal_y);
+$calendar = new calendar_information(0, 0, 0, $time);
 $calendar->prepare_for_view($course, $courses);
 
 $formoptions = new stdClass;
@@ -133,16 +141,7 @@ if ($eventid !== 0) {
             unset($formoptions->eventtypes->groups);
         }
     }
-    if($cal_y && $cal_m && $cal_d && checkdate($cal_m, $cal_d, $cal_y)) {
-        $event->timestart = make_timestamp($cal_y, $cal_m, $cal_d, 0, 0, 0);
-    } else if($cal_y && $cal_m && checkdate($cal_m, 1, $cal_y)) {
-        $now = usergetdate(time());
-        if($cal_y == $now['year'] && $cal_m == $now['mon']) {
-            $event->timestart = make_timestamp($cal_y, $cal_m, $now['mday'], 0, 0, 0);
-        } else {
-            $event->timestart = make_timestamp($cal_y, $cal_m, 1, 0, 0, 0);
-        }
-    }
+    $event->timestart = $time;
     $event = new calendar_event($event);
     if (!calendar_add_event_allowed($event)) {
         print_error('nopermissions');
@@ -168,9 +167,7 @@ if ($data) {
 
     $params = array(
         'view' => 'day',
-        'cal_d' => userdate($event->timestart, '%d'),
-        'cal_m' => userdate($event->timestart, '%m'),
-        'cal_y' => userdate($event->timestart, '%Y'),
+        'time' => $event->timestart,
     );
     $eventurl = new moodle_url('/calendar/view.php', $params);
     if (!empty($event->courseid) && $event->courseid != SITEID) {
index d6a7981..92c2db3 100644 (file)
@@ -56,11 +56,28 @@ if (empty($CFG->enablecalendarexport)) {
 
 $courseid = optional_param('course', SITEID, PARAM_INT);
 $action = optional_param('action', '', PARAM_ALPHA);
-$day  = optional_param('cal_d', 0, PARAM_INT);
-$mon  = optional_param('cal_m', 0, PARAM_INT);
-$yr   = optional_param('cal_y', 0, PARAM_INT);
+$day = optional_param('cal_d', 0, PARAM_INT);
+$mon = optional_param('cal_m', 0, PARAM_INT);
+$year = optional_param('cal_y', 0, PARAM_INT);
+$time = optional_param('time', 0, PARAM_INT);
 $generateurl = optional_param('generateurl', 0, PARAM_BOOL);
 
+// Get the calendar type we are using.
+$calendartype = \core_calendar\type_factory::get_calendar_instance();
+
+// If a day, month and year were passed then convert it to a timestamp. If these were passed
+// then we can assume the day, month and year are passed as Gregorian, as no where in core
+// should we be passing these values rather than the time. This is done for BC.
+if (!empty($day) && !empty($mon) && !empty($year)) {
+    if (checkdate($mon, $day, $year)) {
+        $time = make_timestamp($year, $mon, $day);
+    } else {
+        $time = time();
+    }
+} else if (empty($time)) {
+    $time = time();
+}
+
 if ($courseid != SITEID && !empty($courseid)) {
     $course = $DB->get_record('course', array('id' => $courseid));
     $courses = array($course->id => $course);
@@ -72,36 +89,29 @@ if ($courseid != SITEID && !empty($courseid)) {
 }
 require_course_login($course);
 
-$url = new moodle_url('/calendar/export.php');
+$url = new moodle_url('/calendar/export.php', array('time' => $time));
+
 if ($action !== '') {
     $url->param('action', $action);
 }
-if ($day !== 0) {
-    $url->param('cal_d', $day);
-}
-if ($mon !== 0) {
-    $url->param('cal_m', $mon);
-}
-if ($yr !== 0) {
-    $url->param('cal_y', $yr);
-}
+
 if ($course !== NULL) {
     $url->param('course', $course->id);
 }
 $PAGE->set_url($url);
 
-$calendar = new calendar_information($day, $mon, $yr);
+$calendar = new calendar_information(0, 0, 0, $time);
 $calendar->prepare_for_view($course, $courses);
 
 $pagetitle = get_string('export', 'calendar');
-$now = usergetdate(time());
+$now = $calendartype->timestamp_to_date_array($time);
 
 // Print title and header
 if ($issite) {
     $PAGE->navbar->add($course->shortname, new moodle_url('/course/view.php', array('id'=>$course->id)));
 }
 $link = new moodle_url(CALENDAR_URL.'view.php', array('view'=>'upcoming', 'course'=>$calendar->courseid));
-$PAGE->navbar->add(get_string('calendar', 'calendar'), calendar_get_link_href($link, $now['mday'], $now['mon'], $now['year']));
+$PAGE->navbar->add(get_string('calendar', 'calendar'), calendar_get_link_href($link, 0, 0, 0, $time));
 $PAGE->navbar->add($pagetitle);
 
 $PAGE->set_title($course->shortname.': '.get_string('calendar', 'calendar').': '.$pagetitle);
@@ -125,14 +135,17 @@ switch($action) {
             $weekend = intval($CFG->calendar_weekend);
         }
 
+        // Get the number of days.
+        $numberofdaysinweek = $calendartype->get_num_weekdays();
+
         $authtoken = sha1($USER->id . $DB->get_field('user', 'password', array('id'=>$USER->id)). $CFG->calendar_exportsalt);
         // Let's populate some vars to let "common tasks" be somewhat smart...
-        // If today it's weekend, give the "next week" option
-        $allownextweek  = $weekend & (1 << $now['wday']);
-        // If it's the last week of the month, give the "next month" option
-        $allownextmonth = calendar_days_in_month($now['mon'], $now['year']) - $now['mday'] < 7;
-        // If today it's weekend but tomorrow it isn't, do NOT give the "this week" option
-        $allowthisweek  = !(($weekend & (1 << $now['wday'])) && !($weekend & (1 << (($now['wday'] + 1) % 7))));
+        // If today it's weekend, give the "next week" option.
+        $allownextweek = $weekend & (1 << $now['wday']);
+        // If it's the last week of the month, give the "next month" option.
+        $allownextmonth = calendar_days_in_month($now['mon'], $now['year']) - $now['mday'] < $numberofdaysinweek;
+        // If today it's weekend but tomorrow it isn't, do NOT give the "this week" option.
+        $allowthisweek = !(($weekend & (1 << $now['wday'])) && !($weekend & (1 << (($now['wday'] + 1) % $numberofdaysinweek))));
         echo $renderer->basic_export_form($allowthisweek, $allownextweek, $allownextmonth, $USER->id, $authtoken);
         break;
 }
index 7cd585b..6e76e33 100644 (file)
@@ -31,10 +31,14 @@ if (!$authuserid && !$authusername) {
     die('Invalid authentication');
 }
 
+// Get the calendar type we are using.
+$calendartype = \core_calendar\type_factory::get_calendar_instance();
+
 $what = optional_param('preset_what', 'all', PARAM_ALPHA);
 $time = optional_param('preset_time', 'weeknow', PARAM_ALPHA);
 
-$now = usergetdate(time());
+$now = $calendartype->timestamp_to_date_array(time());
+
 // Let's see if we have sufficient and correct data
 $allowed_what = array('all', 'courses');
 $allowed_time = array('weeknow', 'weeknext', 'monthnow', 'monthnext', 'recentupcoming', 'custom');
@@ -74,20 +78,23 @@ if(!empty($what) && !empty($time)) {
             $groups = false;
         }
 
+        // Store the number of days in the week.
+        $numberofdaysinweek = $calendartype->get_num_weekdays();
+
         switch($time) {
             case 'weeknow':
-                $startweekday  = get_user_preferences('calendar_startwday', calendar_get_starting_weekday());
-                $startmonthday = find_day_in_month($now['mday'] - 6, $startweekday, $now['mon'], $now['year']);
-                $startmonth    = $now['mon'];
-                $startyear     = $now['year'];
+                $startweekday = calendar_get_starting_weekday();
+                $startmonthday = find_day_in_month($now['mday'] - ($numberofdaysinweek - 1), $startweekday, $now['mon'], $now['year']);
+                $startmonth = $now['mon'];
+                $startyear = $now['year'];
                 if($startmonthday > calendar_days_in_month($startmonth, $startyear)) {
                     list($startmonth, $startyear) = calendar_add_month($startmonth, $startyear);
                     $startmonthday = find_day_in_month(1, $startweekday, $startmonth, $startyear);
                 }
                 $timestart = make_timestamp($startyear, $startmonth, $startmonthday);
-                $endmonthday = $startmonthday + 7;
-                $endmonth    = $startmonth;
-                $endyear     = $startyear;
+                $endmonthday = $startmonthday + $numberofdaysinweek;
+                $endmonth = $startmonth;
+                $endyear = $startyear;
                 if($endmonthday > calendar_days_in_month($endmonth, $endyear)) {
                     list($endmonth, $endyear) = calendar_add_month($endmonth, $endyear);
                     $endmonthday = find_day_in_month(1, $startweekday, $endmonth, $endyear);
@@ -95,18 +102,18 @@ if(!empty($what) && !empty($time)) {
                 $timeend = make_timestamp($endyear, $endmonth, $endmonthday) - 1;
             break;
             case 'weeknext':
-                $startweekday  = get_user_preferences('calendar_startwday', calendar_get_starting_weekday());
+                $startweekday = calendar_get_starting_weekday();
                 $startmonthday = find_day_in_month($now['mday'] + 1, $startweekday, $now['mon'], $now['year']);
-                $startmonth    = $now['mon'];
-                $startyear     = $now['year'];
+                $startmonth = $now['mon'];
+                $startyear = $now['year'];
                 if($startmonthday > calendar_days_in_month($startmonth, $startyear)) {
                     list($startmonth, $startyear) = calendar_add_month($startmonth, $startyear);
                     $startmonthday = find_day_in_month(1, $startweekday, $startmonth, $startyear);
                 }
                 $timestart = make_timestamp($startyear, $startmonth, $startmonthday);
-                $endmonthday = $startmonthday + 7;
-                $endmonth    = $startmonth;
-                $endyear     = $startyear;
+                $endmonthday = $startmonthday + $numberofdaysinweek;
+                $endmonth = $startmonth;
+                $endyear = $startyear;
                 if($endmonthday > calendar_days_in_month($endmonth, $endyear)) {
                     list($endmonth, $endyear) = calendar_add_month($endmonth, $endyear);
                     $endmonthday = find_day_in_month(1, $startweekday, $endmonth, $endyear);
@@ -114,13 +121,24 @@ if(!empty($what) && !empty($time)) {
                 $timeend = make_timestamp($endyear, $endmonth, $endmonthday) - 1;
             break;
             case 'monthnow':
-                $timestart = make_timestamp($now['year'], $now['mon'], 1);
-                $timeend   = make_timestamp($now['year'], $now['mon'], calendar_days_in_month($now['mon'], $now['year']), 23, 59, 59);
+                // Convert to gregorian.
+                $gregoriandate = $calendartype->convert_to_gregorian($now['year'], $now['mon'], 1);
+
+                $timestart = make_timestamp($gregoriandate['year'], $gregoriandate['month'], $gregoriandate['day']);
+                $timeend = make_timestamp($gregoriandate['year'], $gregoriandate['month'],
+                    calendar_days_in_month($now['mon'], $now['year']), 23, 59, 59);
             break;
             case 'monthnext':
+                // Get the next month for this calendar.
                 list($nextmonth, $nextyear) = calendar_add_month($now['mon'], $now['year']);
-                $timestart = make_timestamp($nextyear, $nextmonth, 1);
-                $timeend   = make_timestamp($nextyear, $nextmonth, calendar_days_in_month($nextmonth, $nextyear), 23, 59, 59);
+
+                // Convert to gregorian.
+                $gregoriandate = $calendartype->convert_to_gregorian($nextyear, $nextmonth, 1);
+
+                // Create the timestamps.
+                $timestart = make_timestamp($gregoriandate['year'], $gregoriandate['month'], $gregoriandate['day']);
+                $timeend   = make_timestamp($gregoriandate['year'], $gregoriandate['month'],
+                    calendar_days_in_month($nextmonth, $nextyear), 23, 59, 59);
             break;
             case 'recentupcoming':
                 //Events in the last 5 or next 60 days
index ede8280..a805f33 100644 (file)
@@ -126,7 +126,8 @@ define('CALENDAR_SUBSCRIPTION_REMOVE', 2);
  * @return array array of days
  */
 function calendar_get_days() {
-    return array('sunday', 'monday', 'tuesday', 'wednesday', 'thursday', 'friday', 'saturday');
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
+    return $calendartype->get_weekdays();
 }
 
 /**
@@ -158,19 +159,8 @@ function calendar_get_subscription($id) {
  * @return int
  */
 function calendar_get_starting_weekday() {
-    global $CFG;
-
-    if (isset($CFG->calendar_startwday)) {
-        $firstday = $CFG->calendar_startwday;
-    } else {
-        $firstday = get_string('firstdayofweek', 'langconfig');
-    }
-
-    if(!is_numeric($firstday)) {
-        return CALENDAR_DEFAULT_STARTING_WEEKDAY;
-    } else {
-        return intval($firstday) % 7;
-    }
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
+    return $calendartype->get_starting_weekday();
 }
 
 /**
@@ -179,83 +169,96 @@ function calendar_get_starting_weekday() {
  * @param array $courses list of course to list events from
  * @param array $groups list of group
  * @param array $users user's info
- * @param int $cal_month calendar month in numeric, default is set to false
- * @param int $cal_year calendar month in numeric, default is set to false
- * @param string $placement the place/page the calendar is set to appear - passed on the the controls function
- * @param int $courseid id of the course the calendar is displayed on - passed on the the controls function
+ * @param int|bool $calmonth calendar month in numeric, default is set to false
+ * @param int|bool $calyear calendar month in numeric, default is set to false
+ * @param string|bool $placement the place/page the calendar is set to appear - passed on the the controls function
+ * @param int|bool $courseid id of the course the calendar is displayed on - passed on the the controls function
+ * @param int $time the unixtimestamp representing the date we want to view, this is used instead of $calmonth
+ *     and $calyear to support multiple calendars
  * @return string $content return html table for mini calendar
  */
-function calendar_get_mini($courses, $groups, $users, $cal_month = false, $cal_year = false, $placement = false, $courseid = false ) {
-    global $CFG, $USER, $OUTPUT;
+function calendar_get_mini($courses, $groups, $users, $calmonth = false, $calyear = false, $placement = false,
+    $courseid = false, $time = 0) {
+    global $CFG, $OUTPUT;
+
+    // Get the calendar type we are using.
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
 
     $display = new stdClass;
-    $display->minwday = get_user_preferences('calendar_startwday', calendar_get_starting_weekday());
-    $display->maxwday = $display->minwday + 6;
+
+    // Assume we are not displaying this month for now.
+    $display->thismonth = false;
 
     $content = '';
 
-    if(!empty($cal_month) && !empty($cal_year)) {
-        $thisdate = usergetdate(time()); // Date and time the user sees at his location
-        if($cal_month == $thisdate['mon'] && $cal_year == $thisdate['year']) {
-            // Navigated to this month
-            $date = $thisdate;
-            $display->thismonth = true;
+    // Do this check for backwards compatibility. The core should be passing a timestamp rather than month and year.
+    // If a month and year are passed they will be in Gregorian.
+    if (!empty($calmonth) && !empty($calyear)) {
+        // Ensure it is a valid date, else we will just set it to the current timestamp.
+        if (checkdate($calmonth, 1, $calyear)) {
+            $time = make_timestamp($calyear, $calmonth, 1);
         } else {
-            // Navigated to other month, let's do a nice trick and save us a lot of work...
-            if(!checkdate($cal_month, 1, $cal_year)) {
-                $date = array('mday' => 1, 'mon' => $thisdate['mon'], 'year' => $thisdate['year']);
-                $display->thismonth = true;
-            } else {
-                $date = array('mday' => 1, 'mon' => $cal_month, 'year' => $cal_year);
-                $display->thismonth = false;
-            }
+            $time = time();
+        }
+        $date = usergetdate($time);
+        if ($calmonth == $date['mon'] && $calyear == $date['year']) {
+            $display->thismonth = true;
+        }
+        // We can overwrite date now with the date used by the calendar type, if it is not Gregorian, otherwise
+        // there is no need as it is already in Gregorian.
+        if ($calendartype->get_name() != 'gregorian') {
+            $date = $calendartype->timestamp_to_date_array($time);
+        }
+    } else if (!empty($time)) {
+        // Get the specified date in the calendar type being used.
+        $date = $calendartype->timestamp_to_date_array($time);
+        $thisdate = $calendartype->timestamp_to_date_array(time());
+        if ($date['month'] == $thisdate['month'] && $date['year'] == $thisdate['year']) {
+            $display->thismonth = true;
+            // If we are the current month we want to set the date to the current date, not the start of the month.
+            $date = $thisdate;
         }
     } else {
-        $date = usergetdate(time()); // Date and time the user sees at his location
+        // Get the current date in the calendar type being used.
+        $time = time();
+        $date = $calendartype->timestamp_to_date_array($time);
         $display->thismonth = true;
     }
 
-    // Fill in the variables we 're going to use, nice and tidy
-    list($d, $m, $y) = array($date['mday'], $date['mon'], $date['year']); // This is what we want to display
-    $display->maxdays = calendar_days_in_month($m, $y);
+    list($d, $m, $y) = array($date['mday'], $date['mon'], $date['year']); // This is what we want to display.
 
-    if (get_user_timezone_offset() < 99) {
-        // We 'll keep these values as GMT here, and offset them when the time comes to query the db
-        $display->tstart = gmmktime(0, 0, 0, $m, 1, $y); // This is GMT
-        $display->tend = gmmktime(23, 59, 59, $m, $display->maxdays, $y); // GMT
-    } else {
-        // no timezone info specified
-        $display->tstart = mktime(0, 0, 0, $m, 1, $y);
-        $display->tend = mktime(23, 59, 59, $m, $display->maxdays, $y);
-    }
+    // Get Gregorian date.
+    $gregoriandate = $calendartype->convert_to_gregorian($date['year'], $date['mon'], 1);
 
+    // Store the gregorian year and month to be used later.
+    list($gy, $gm, $gd, $gh, $gmin) = array($gregoriandate['year'], $gregoriandate['month'], $gregoriandate['day'],
+        $gregoriandate['hour'], $gregoriandate['minute']);
+
+    // Get the max number of days in this month for this calendar type.
+    $display->maxdays = calendar_days_in_month($m, $y);
+    // Get the starting week day for this month.
     $startwday = dayofweek(1, $m, $y);
+    // Get the days in a week.
+    $daynames = calendar_get_days();
+    // Store the number of days in a week.
+    $numberofdaysinweek = $calendartype->get_num_weekdays();
+
+    // Set the min and max weekday.
+    $display->minwday = calendar_get_starting_weekday();
+    $display->maxwday = $display->minwday + ($numberofdaysinweek - 1);
+
+    // These are used for DB queries, so we want unixtime, so we need to use Gregorian dates.
+    $display->tstart = make_timestamp($gy, $gm, $gd, $gh, $gmin, 0);
+    $display->tend = make_timestamp($gy, $gm, $display->maxdays, 23, 59, 59);
 
     // Align the starting weekday to fall in our display range
     // This is simple, not foolproof.
-    if($startwday < $display->minwday) {
-        $startwday += 7;
+    if ($startwday < $display->minwday) {
+        $startwday += $numberofdaysinweek;
     }
 
-    // TODO: THIS IS TEMPORARY CODE!
-    // [pj] I was just reading through this and realized that I when writing this code I was probably
-    // asking for trouble, as all these time manipulations seem to be unnecessary and a simple
-    // make_timestamp would accomplish the same thing. So here goes a test:
-    //$test_start = make_timestamp($y, $m, 1);
-    //$test_end   = make_timestamp($y, $m, $display->maxdays, 23, 59, 59);
-    //if($test_start != usertime($display->tstart) - dst_offset_on($display->tstart)) {
-        //notify('Failed assertion in calendar/lib.php line 126; display->tstart = '.$display->tstart.', dst_offset = '.dst_offset_on($display->tstart).', usertime = '.usertime($display->tstart).', make_t = '.$test_start);
-    //}
-    //if($test_end != usertime($display->tend) - dst_offset_on($display->tend)) {
-        //notify('Failed assertion in calendar/lib.php line 130; display->tend = '.$display->tend.', dst_offset = '.dst_offset_on($display->tend).', usertime = '.usertime($display->tend).', make_t = '.$test_end);
-    //}
-
-
     // Get the events matching our criteria. Don't forget to offset the timestamps for the user's TZ!
-    $events = calendar_get_events(
-        usertime($display->tstart) - dst_offset_on($display->tstart),
-        usertime($display->tend) - dst_offset_on($display->tend),
-        $users, $groups, $courses);
+    $events = calendar_get_events($display->tstart, $display->tend, $users, $groups, $courses);
 
     // Set event course class for course events
     if (!empty($events)) {
@@ -273,7 +276,6 @@ function calendar_get_mini($courses, $groups, $users, $cal_month = false, $cal_y
     // possibly removing SITEID from $courses, there is only one course left, then clicking on a day in the month
     // will also set the $SESSION->cal_courses_shown variable to that one course. Otherwise, we 'd need to add extra
     // arguments to this function.
-
     $hrefparams = array();
     if(!empty($courses)) {
         $courses = array_diff($courses, array(SITEID));
@@ -283,27 +285,21 @@ function calendar_get_mini($courses, $groups, $users, $cal_month = false, $cal_y
     }
 
     // We want to have easy access by day, since the display is on a per-day basis.
-    // Arguments passed by reference.
-    //calendar_events_by_day($events, $display->tstart, $eventsbyday, $durationbyday, $typesbyday);
     calendar_events_by_day($events, $m, $y, $eventsbyday, $durationbyday, $typesbyday, $courses);
 
-    //Accessibility: added summary and <abbr> elements.
-    $days_title = calendar_get_days();
-
-    $summary = get_string('calendarheading', 'calendar', userdate(make_timestamp($y, $m), get_string('strftimemonthyear')));
-    $content .= '<table class="minicalendar calendartable" summary="'.$summary.'">'; // Begin table
-    if (($placement !== false) && ($courseid  !== false)) {
-        $content .= '<caption>'. calendar_top_controls($placement, array('id' => $courseid, 'm' => $m, 'y' => $y)) .'</caption>';
+    // Accessibility: added summary and <abbr> elements.
+    $summary = get_string('calendarheading', 'calendar', userdate($display->tstart, get_string('strftimemonthyear')));
+    $content .= '<table class="minicalendar calendartable" summary="'.$summary.'">'; // Begin table.
+    if (($placement !== false) && ($courseid !== false)) {
+        $content .= '<caption>'. calendar_top_controls($placement, array('id' => $courseid, 'time' => $time)) .'</caption>';
     }
     $content .= '<tr class="weekdays">'; // Header row: day names
 
-    // Print out the names of the weekdays
-    $days = array('sun', 'mon', 'tue', 'wed', 'thu', 'fri', 'sat');
-    for($i = $display->minwday; $i <= $display->maxwday; ++$i) {
-        // This uses the % operator to get the correct weekday no matter what shift we have
-        // applied to the $display->minwday : $display->maxwday range from the default 0 : 6
-        $content .= '<th scope="col"><abbr title="'. get_string($days_title[$i % 7], 'calendar') .'">'.
-            get_string($days[$i % 7], 'calendar') ."</abbr></th>\n";
+    // Print out the names of the weekdays.
+    for ($i = $display->minwday; $i <= $display->maxwday; ++$i) {
+        $pos = $i % $numberofdaysinweek;
+        $content .= '<th scope="col"><abbr title="'. $daynames[$pos]['fullname'] .'">'.
+            $daynames[$pos]['shortname'] ."</abbr></th>\n";
     }
 
     $content .= '</tr><tr>'; // End of day names; prepare for day numbers
@@ -322,16 +318,17 @@ function calendar_get_mini($courses, $groups, $users, $cal_month = false, $cal_y
     }
 
     // Now display all the calendar
+    $daytime = $display->tstart - DAYSECS;
     for($day = 1; $day <= $display->maxdays; ++$day, ++$dayweek) {
+        $daytime += DAYSECS;
         if($dayweek > $display->maxwday) {
             // We need to change week (table row)
             $content .= '</tr><tr>';
             $dayweek = $display->minwday;
         }
 
-        // Reset vars
-        $cell = '';
-        if ($weekend & (1 << ($dayweek % 7))) {
+        // Reset vars.
+        if ($weekend & (1 << ($dayweek % $numberofdaysinweek))) {
             // Weekend. This is true no matter what the exact range is.
             $class = 'weekend day';
         } else {
@@ -341,9 +338,10 @@ function calendar_get_mini($courses, $groups, $users, $cal_month = false, $cal_y
 
         // Special visual fx if an event is defined
         if(isset($eventsbyday[$day])) {
+
             $class .= ' hasevent';
             $hrefparams['view'] = 'day';
-            $dayhref = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', $hrefparams), $day, $m, $y);
+            $dayhref = calendar_get_link_href(new moodle_url(CALENDAR_URL . 'view.php', $hrefparams), 0, 0, 0, $daytime);
 
             $popupcontent = '';
             foreach($eventsbyday[$day] as $eventid) {
@@ -353,17 +351,17 @@ function calendar_get_mini($courses, $groups, $users, $cal_month = false, $cal_y
                 $event = new calendar_event($events[$eventid]);
                 $popupalt  = '';
                 $component = 'moodle';
-                if(!empty($event->modulename)) {
+                if (!empty($event->modulename)) {
                     $popupicon = 'icon';
                     $popupalt  = $event->modulename;
                     $component = $event->modulename;
-                } else if ($event->courseid == SITEID) {                                // Site event
+                } else if ($event->courseid == SITEID) { // Site event.
                     $popupicon = 'i/siteevent';
-                } else if ($event->courseid != 0 && $event->courseid != SITEID && $event->groupid == 0) {      // Course event
+                } else if ($event->courseid != 0 && $event->courseid != SITEID && $event->groupid == 0) { // Course event.
                     $popupicon = 'i/courseevent';
-                } else if ($event->groupid) {                                      // Group event
+                } else if ($event->groupid) { // Group event.
                     $popupicon = 'i/groupevent';
-                } else if ($event->userid) {                                       // User event
+                } else { // Must be a user event.
                     $popupicon = 'i/userevent';
                 }
 
@@ -823,43 +821,59 @@ function calendar_get_events_by_id($eventids) {
  * @return string $content return available control for the calender in html
  */
 function calendar_top_controls($type, $data) {
-    global $CFG, $PAGE;
+    global $PAGE;
+
+    // Get the calendar type we are using.
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
+
     $content = '';
-    if(!isset($data['d'])) {
-        $data['d'] = 1;
-    }
 
-    // Ensure course id passed if relevant
-    // Required due to changes in view/lib.php mainly (calendar_session_vars())
+    // Ensure course id passed if relevant.
     $courseid = '';
     if (!empty($data['id'])) {
         $courseid = '&amp;course='.$data['id'];
     }
 
-    if(!checkdate($data['m'], $data['d'], $data['y'])) {
+    // If we are passing a month and year then we need to convert this to a timestamp to
+    // support multiple calendars. No where in core should these be passed, this logic
+    // here is for third party plugins that may use this function.
+    if (!empty($data['m']) && !empty($date['y'])) {
+        if (!isset($data['d'])) {
+            $data['d'] = 1;
+        }
+        if (!checkdate($data['m'], $data['d'], $data['y'])) {
+            $time = time();
+        } else {
+            $time = make_timestamp($data['y'], $data['m'], $data['d']);
+        }
+    } else if (!empty($data['time'])) {
+        $time = $data['time'];
+    } else {
         $time = time();
     }
-    else {
-        $time = make_timestamp($data['y'], $data['m'], $data['d']);
-    }
-    $date = usergetdate($time);
 
-    $data['m'] = $date['mon'];
-    $data['y'] = $date['year'];
+    // Get the date for the calendar type.
+    $date = $calendartype->timestamp_to_date_array($time);
+
     $urlbase = $PAGE->url;
 
-    //Accessibility: calendar block controls, replaced <table> with <div>.
-    //$nexttext = link_arrow_right(get_string('monthnext', 'access'), $url='', $accesshide=true);
-    //$prevtext = link_arrow_left(get_string('monthprev', 'access'), $url='', $accesshide=true);
+    // We need to get the previous and next months in certain cases.
+    if ($type == 'frontpage' || $type == 'course' || $type == 'month') {
+        $prevmonth = calendar_sub_month($date['mon'], $date['year']);
+        $prevmonthtime = $calendartype->convert_to_gregorian($prevmonth[1], $prevmonth[0], $date['mday']);
+        $prevmonthtime = make_timestamp($prevmonthtime['year'], $prevmonthtime['month'], $prevmonthtime['day']);
+
+        $nextmonth = calendar_add_month($date['mon'], $date['year']);
+        $nextmonthtime = $calendartype->convert_to_gregorian($nextmonth[1], $nextmonth[0], $date['mday']);
+        $nextmonthtime = make_timestamp($nextmonthtime['year'], $nextmonthtime['month'], $nextmonthtime['day']);
+    }
 
-    switch($type) {
+    switch ($type) {
         case 'frontpage':
-            list($prevmonth, $prevyear) = calendar_sub_month($data['m'], $data['y']);
-            list($nextmonth, $nextyear) = calendar_add_month($data['m'], $data['y']);
-            $nextlink = calendar_get_link_next(get_string('monthnext', 'access'), $urlbase, 0, $nextmonth, $nextyear, true);
-            $prevlink = calendar_get_link_previous(get_string('monthprev', 'access'), $urlbase, 0, $prevmonth, $prevyear, true);
+            $prevlink = calendar_get_link_previous(get_string('monthprev', 'access'), $urlbase, false, false, false, true, $prevmonthtime);
+            $nextlink = calendar_get_link_next(get_string('monthnext', 'access'), $urlbase, false, false, false, true, $nextmonthtime);
+            $calendarlink = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view' => 'month')), false, false, false, $time);
 
-            $calendarlink = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view'=>'month')), 1, $data['m'], $data['y']);
             if (!empty($data['id'])) {
                 $calendarlink->param('course', $data['id']);
             }
@@ -881,12 +895,10 @@ function calendar_top_controls($type, $data) {
 
             break;
         case 'course':
-            list($prevmonth, $prevyear) = calendar_sub_month($data['m'], $data['y']);
-            list($nextmonth, $nextyear) = calendar_add_month($data['m'], $data['y']);
-            $nextlink = calendar_get_link_next(get_string('monthnext', 'access'), $urlbase, 0, $nextmonth, $nextyear, true);
-            $prevlink = calendar_get_link_previous(get_string('monthprev', 'access'), $urlbase, 0, $prevmonth, $prevyear, true);
+            $prevlink = calendar_get_link_previous(get_string('monthprev', 'access'), $urlbase, false, false, false, true, $prevmonthtime);
+            $nextlink = calendar_get_link_next(get_string('monthnext', 'access'), $urlbase, false, false, false, true, $nextmonthtime);
+            $calendarlink = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view' => 'month')), false, false, false, $time);
 
-            $calendarlink = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view'=>'month')), 1, $data['m'], $data['y']);
             if (!empty($data['id'])) {
                 $calendarlink->param('course', $data['id']);
             }
@@ -907,7 +919,7 @@ function calendar_top_controls($type, $data) {
             $content .= html_writer::end_tag('div');
             break;
         case 'upcoming':
-            $calendarlink = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view'=>'upcoming')), 1, $data['m'], $data['y']);
+            $calendarlink = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view' => 'upcoming')), false, false, false, $time);
             if (!empty($data['id'])) {
                 $calendarlink->param('course', $data['id']);
             }
@@ -915,7 +927,7 @@ function calendar_top_controls($type, $data) {
             $content .= html_writer::tag('div', $calendarlink, array('class'=>'centered'));
             break;
         case 'display':
-            $calendarlink = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view'=>'month')), 1, $data['m'], $data['y']);
+            $calendarlink = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view' => 'month')), false, false, false, $time);
             if (!empty($data['id'])) {
                 $calendarlink->param('course', $data['id']);
             }
@@ -923,12 +935,8 @@ function calendar_top_controls($type, $data) {
             $content .= html_writer::tag('h3', $calendarlink);
             break;
         case 'month':
-            list($prevmonth, $prevyear) = calendar_sub_month($data['m'], $data['y']);
-            list($nextmonth, $nextyear) = calendar_add_month($data['m'], $data['y']);
-            $prevdate = make_timestamp($prevyear, $prevmonth, 1);
-            $nextdate = make_timestamp($nextyear, $nextmonth, 1);
-            $prevlink = calendar_get_link_previous(userdate($prevdate, get_string('strftimemonthyear')), 'view.php?view=month'.$courseid.'&amp;', 1, $prevmonth, $prevyear);
-            $nextlink = calendar_get_link_next(userdate($nextdate, get_string('strftimemonthyear')), 'view.php?view=month'.$courseid.'&amp;', 1, $nextmonth, $nextyear);
+            $prevlink = calendar_get_link_previous(userdate($prevmonthtime, get_string('strftimemonthyear')), 'view.php?view=month'.$courseid.'&amp;', false, false, false, false, $prevmonthtime);
+            $nextlink = calendar_get_link_next(userdate($nextmonthtime, get_string('strftimemonthyear')), 'view.php?view=month'.$courseid.'&amp;', false, false, false, false, $nextmonthtime);
 
             if (right_to_left()) {
                 $left = $nextlink;
@@ -946,13 +954,17 @@ function calendar_top_controls($type, $data) {
             break;
         case 'day':
             $days = calendar_get_days();
-            $data['d'] = $date['mday']; // Just for convenience
-            $prevdate = usergetdate(make_timestamp($data['y'], $data['m'], $data['d'] - 1));
-            $nextdate = usergetdate(make_timestamp($data['y'], $data['m'], $data['d'] + 1));
-            $prevname = calendar_wday_name($days[$prevdate['wday']]);
-            $nextname = calendar_wday_name($days[$nextdate['wday']]);
-            $prevlink = calendar_get_link_previous($prevname, 'view.php?view=day'.$courseid.'&amp;', $prevdate['mday'], $prevdate['mon'], $prevdate['year']);
-            $nextlink = calendar_get_link_next($nextname, 'view.php?view=day'.$courseid.'&amp;', $nextdate['mday'], $nextdate['mon'], $nextdate['year']);
+
+            $prevtimestamp = $time - DAYSECS;
+            $nexttimestamp = $time + DAYSECS;
+
+            $prevdate = $calendartype->timestamp_to_date_array($prevtimestamp);
+            $nextdate = $calendartype->timestamp_to_date_array($nexttimestamp);
+
+            $prevname = $days[$prevdate['wday']]['fullname'];
+            $nextname = $days[$nextdate['wday']]['fullname'];
+            $prevlink = calendar_get_link_previous($prevname, 'view.php?view=day'.$courseid.'&amp;', false, false, false, false, $prevtimestamp);
+            $nextlink = calendar_get_link_next($nextname, 'view.php?view=day'.$courseid.'&amp;', false, false, false, false, $nexttimestamp);
 
             if (right_to_left()) {
                 $left = $nextlink;
@@ -1133,24 +1145,33 @@ function calendar_time_representation($time) {
  * @param int $d The number of the day.
  * @param int $m The number of the month.
  * @param int $y The number of the year.
+ * @param int $time the unixtime, used for multiple calendar support. The values $d,
+ *     $m and $y are kept for backwards compatibility.
  * @return moodle_url|null $linkbase
  */
-function calendar_get_link_href($linkbase, $d, $m, $y) {
+function calendar_get_link_href($linkbase, $d, $m, $y, $time = 0) {
     if (empty($linkbase)) {
         return '';
     }
     if (!($linkbase instanceof moodle_url)) {
         $linkbase = new moodle_url($linkbase);
     }
-    if (!empty($d)) {
-        $linkbase->param('cal_d', $d);
-    }
-    if (!empty($m)) {
-        $linkbase->param('cal_m', $m);
-    }
-    if (!empty($y)) {
-        $linkbase->param('cal_y', $y);
+
+    // If a day, month and year were passed then convert it to a timestamp. If these were passed
+    // then we can assume the day, month and year are passed as Gregorian, as no where in core
+    // should we be passing these values rather than the time.
+    if (!empty($d) && !empty($m) && !empty($y)) {
+        if (checkdate($m, $d, $y)) {
+            $time = make_timestamp($y, $m, $d);
+        } else {
+            $time = time();
+        }
+    } else if (empty($time)) {
+        $time = time();
     }
+
+    $linkbase->param('time', $time);
+
     return $linkbase;
 }
 
@@ -1163,10 +1184,12 @@ function calendar_get_link_href($linkbase, $d, $m, $y) {
  * @param int $m The number of the month.
  * @param int $y year The number of the year.
  * @param bool $accesshide Default visible, or hide from all except screenreaders.
+ * @param int $time the unixtime, used for multiple calendar support. The values $d,
+ *     $m and $y are kept for backwards compatibility.
  * @return string HTML string.
  */
-function calendar_get_link_previous($text, $linkbase, $d, $m, $y, $accesshide=false) {
-    $href = calendar_get_link_href(new moodle_url($linkbase), $d, $m, $y);
+function calendar_get_link_previous($text, $linkbase, $d, $m, $y, $accesshide = false, $time = 0) {
+    $href = calendar_get_link_href(new moodle_url($linkbase), $d, $m, $y, $time);
     if (empty($href)) {
         return $text;
     }
@@ -1182,10 +1205,12 @@ function calendar_get_link_previous($text, $linkbase, $d, $m, $y, $accesshide=fa
  * @param int $m The number of the month.
  * @param int $y The number of the year.
  * @param bool $accesshide Default visible, or hide from all except screenreaders.
+ * @param int $time the unixtime, used for multiple calendar support. The values $d,
+ *     $m and $y are kept for backwards compatibility.
  * @return string HTML string.
  */
-function calendar_get_link_next($text, $linkbase, $d, $m, $y, $accesshide=false) {
-    $href = calendar_get_link_href(new moodle_url($linkbase), $d, $m, $y);
+function calendar_get_link_next($text, $linkbase, $d, $m, $y, $accesshide = false, $time = 0) {
+    $href = calendar_get_link_href(new moodle_url($linkbase), $d, $m, $y, $time);
     if (empty($href)) {
         return $text;
     }
@@ -1210,7 +1235,8 @@ function calendar_wday_name($englishname) {
  * @return int
  */
 function calendar_days_in_month($month, $year) {
-   return intval(date('t', mktime(0, 0, 0, $month, 1, $year)));
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
+    return $calendartype->get_num_days_in_month($year, $month);
 }
 
 /**
@@ -1239,8 +1265,7 @@ function calendar_get_block_upcoming($events, $linkhref = NULL, $showcourselink
             $content .= $events[$i]->referer;
         } else {
             if(!empty($linkhref)) {
-                $ed = usergetdate($events[$i]->timestart);
-                $href = calendar_get_link_href(new moodle_url(CALENDAR_URL.$linkhref), $ed['mday'], $ed['mon'], $ed['year']);
+                $href = calendar_get_link_href(new moodle_url(CALENDAR_URL . $linkhref), 0, 0, 0, $events[$i]->timestart);
                 $href->set_anchor('event_'.$events[$i]->id);
                 $content .= html_writer::link($href, $events[$i]->name);
             }
@@ -1262,38 +1287,27 @@ function calendar_get_block_upcoming($events, $linkhref = NULL, $showcourselink
 /**
  * Get the next following month
  *
- * If the current month is December, it will get the first month of the following year.
- *
- *
  * @param int $month the number of the month.
  * @param int $year the number of the year.
  * @return array the following month
  */
 function calendar_add_month($month, $year) {
-    if($month == 12) {
-        return array(1, $year + 1);
-    }
-    else {
-        return array($month + 1, $year);
-    }
+    // Get the calendar type we are using.
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
+    return $calendartype->get_next_month($year, $month);
 }
 
 /**
- * Get the previous month
- *
- * If the current month is January, it will get the last month of the previous year.
+ * Get the previous month.
  *
  * @param int $month the number of the month.
  * @param int $year the number of the year.
  * @return array previous month
  */
 function calendar_sub_month($month, $year) {
-    if($month == 1) {
-        return array(12, $year - 1);
-    }
-    else {
-        return array($month - 1, $year);
-    }
+    // Get the calendar type we are using.
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
+    return $calendartype->get_prev_month($year, $month);
 }
 
 /**
@@ -1309,6 +1323,9 @@ function calendar_sub_month($month, $year) {
  * @return void
  */
 function calendar_events_by_day($events, $month, $year, &$eventsbyday, &$durationbyday, &$typesbyday, &$courses) {
+    // Get the calendar type we are using.
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
+
     $eventsbyday = array();
     $typesbyday = array();
     $durationbyday = array();
@@ -1317,12 +1334,11 @@ function calendar_events_by_day($events, $month, $year, &$eventsbyday, &$duratio
         return;
     }
 
-    foreach($events as $event) {
-
-        $startdate = usergetdate($event->timestart);
+    foreach ($events as $event) {
+        $startdate = $calendartype->timestamp_to_date_array($event->timestart);
         // Set end date = start date if no duration
         if ($event->timeduration) {
-            $enddate   = usergetdate($event->timestart + $event->timeduration - 1);
+            $enddate = $calendartype->timestamp_to_date_array($event->timestart + $event->timeduration - 1);
         } else {
             $enddate = $startdate;
         }
@@ -1637,90 +1653,81 @@ function calendar_preferences_button(stdClass $course) {
  * @param int $showtime determine the show time GMT timestamp
  * @return string $eventtime link/string for event time
  */
-function calendar_format_event_time($event, $now, $linkparams = null, $usecommonwords = true, $showtime=0) {
-    $startdate = usergetdate($event->timestart);
-    $enddate = usergetdate($event->timestart + $event->timeduration);
-    $usermidnightstart = usergetmidnight($event->timestart);
-
-    if($event->timeduration) {
-        // To avoid doing the math if one IF is enough :)
-        $usermidnightend = usergetmidnight($event->timestart + $event->timeduration);
-    }
-    else {
-        $usermidnightend = $usermidnightstart;
-    }
+function calendar_format_event_time($event, $now, $linkparams = null, $usecommonwords = true, $showtime = 0) {
+    $starttime = $event->timestart;
+    $endtime = $event->timestart + $event->timeduration;
 
     if (empty($linkparams) || !is_array($linkparams)) {
         $linkparams = array();
     }
+
     $linkparams['view'] = 'day';
 
     // OK, now to get a meaningful display...
-    // First of all we have to construct a human-readable date/time representation
-
-    if($event->timeduration) {
-        // It has a duration
-        if($usermidnightstart == $usermidnightend ||
-           ($event->timestart == $usermidnightstart) && ($event->timeduration == 86400 || $event->timeduration == 86399) ||
-           ($event->timestart + $event->timeduration <= $usermidnightstart + 86400)) {
-            // But it's all on the same day
-            $timestart = calendar_time_representation($event->timestart);
-            $timeend = calendar_time_representation($event->timestart + $event->timeduration);
-            $time = $timestart.' <strong>&raquo;</strong> '.$timeend;
-
-            if ($event->timestart == $usermidnightstart && ($event->timeduration == 86400 || $event->timeduration == 86399)) {
+    // Check if there is a duration for this event.
+    if ($event->timeduration) {
+        // Get the midnight of the day the event will start.
+        $usermidnightstart = usergetmidnight($starttime);
+        // Get the midnight of the day the event will end.
+        $usermidnightend = usergetmidnight($endtime);
+        // Check if we will still be on the same day.
+        if ($usermidnightstart == $usermidnightend) {
+            // Check if we are running all day.
+            if ($event->timeduration == DAYSECS) {
                 $time = get_string('allday', 'calendar');
+            } else { // Specify the time we will be running this from.
+                $datestart = calendar_time_representation($starttime);
+                $dateend = calendar_time_representation($endtime);
+                $time = $datestart . ' <strong>&raquo;</strong> ' . $dateend;
             }
 
-            // Set printable representation
+            // Set printable representation.
             if (!$showtime) {
                 $day = calendar_day_representation($event->timestart, $now, $usecommonwords);
-                $url = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', $linkparams), $enddate['mday'], $enddate['mon'], $enddate['year']);
-                $eventtime = html_writer::link($url, $day).', '.$time;
+                $url = calendar_get_link_href(new moodle_url(CALENDAR_URL . 'view.php', $linkparams), 0, 0, 0, $endtime);
+                $eventtime = html_writer::link($url, $day) . ', ' . $time;
             } else {
                 $eventtime = $time;
             }
-        } else {
-            // It spans two or more days
-            $daystart = calendar_day_representation($event->timestart, $now, $usecommonwords).', ';
+        } else { // It must spans two or more days.
+            $daystart = calendar_day_representation($event->timestart, $now, $usecommonwords) . ', ';
             if ($showtime == $usermidnightstart) {
                 $daystart = '';
             }
             $timestart = calendar_time_representation($event->timestart);
-            $dayend = calendar_day_representation($event->timestart + $event->timeduration, $now, $usecommonwords).', ';
+            $dayend = calendar_day_representation($event->timestart + $event->timeduration, $now, $usecommonwords) . ', ';
             if ($showtime == $usermidnightend) {
                 $dayend = '';
             }
             $timeend = calendar_time_representation($event->timestart + $event->timeduration);
 
-            // Set printable representation
-            if ($now >= $usermidnightstart && $now < ($usermidnightstart + 86400)) {
-                $url = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', $linkparams), $enddate['mday'], $enddate['mon'], $enddate['year']);
-                $eventtime = $timestart.' <strong>&raquo;</strong> '.html_writer::link($url, $dayend).$timeend;
+            // Set printable representation.
+            if ($now >= $usermidnightstart && $now < ($usermidnightstart + DAYSECS)) {
+                $url = calendar_get_link_href(new moodle_url(CALENDAR_URL . 'view.php', $linkparams), 0, 0, 0, $endtime);
+                $eventtime = $timestart . ' <strong>&raquo;</strong> ' . html_writer::link($url, $dayend) . $timeend;
             } else {
-                $url = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', $linkparams), $enddate['mday'], $enddate['mon'], $enddate['year']);
-                $eventtime  = html_writer::link($url, $daystart).$timestart.' <strong>&raquo;</strong> ';
+                $url = calendar_get_link_href(new moodle_url(CALENDAR_URL . 'view.php', $linkparams), 0, 0, 0, $endtime);
+                $eventtime  = html_writer::link($url, $daystart) . $timestart . ' <strong>&raquo;</strong> ';
 
-                $url = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', $linkparams), $startdate['mday'], $startdate['mon'], $startdate['year']);
-                $eventtime .= html_writer::link($url, $dayend).$timeend;
+                $url = calendar_get_link_href(new moodle_url(CALENDAR_URL . 'view.php', $linkparams),  0, 0, 0, $starttime);
+                $eventtime .= html_writer::link($url, $dayend) . $timeend;
             }
         }
-    } else {
+    } else { // There is no time duration.
         $time = calendar_time_representation($event->timestart);
-
-        // Set printable representation
+        // Set printable representation.
         if (!$showtime) {
             $day = calendar_day_representation($event->timestart, $now, $usecommonwords);
-            $url = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', $linkparams), $startdate['mday'], $startdate['mon'], $startdate['year']);
-            $eventtime = html_writer::link($url, $day).', '.trim($time);
+            $url = calendar_get_link_href(new moodle_url(CALENDAR_URL . 'view.php', $linkparams),  0, 0, 0, $starttime);
+            $eventtime = html_writer::link($url, $day) . ', ' . trim($time);
         } else {
             $eventtime = $time;
         }
     }
 
-    if($event->timestart + $event->timeduration < $now) {
-        // It has expired
-        $eventtime = '<span class="dimmed_text">'.str_replace(' href=', ' class="dimmed" href=', $eventtime).'</span>';
+    // Check if It has expired.
+    if ($event->timestart + $event->timeduration < $now) {
+        $eventtime = '<span class="dimmed_text">' . str_replace(' href=', ' class="dimmed" href=', $eventtime) . '</span>';
     }
 
     return $eventtime;
@@ -2632,14 +2639,14 @@ class calendar_event {
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 class calendar_information {
-    /** @var int The day */
-    public $day;
 
-    /** @var int The month */
-    public $month;
-
-    /** @var int The year */
-    public $year;
+    /**
+     * @var int The timestamp
+     *
+     * Rather than setting the day, month and year we will set a timestamp which will be able
+     * to be used by multiple calendars.
+     */
+    public $time;
 
     /** @var int A course id */
     public $courseid = null;
@@ -2659,26 +2666,34 @@ class calendar_information {
      * @param int $day the number of the day
      * @param int $month the number of the month
      * @param int $year the number of the year
+     * @param int $time the unixtimestamp representing the date we want to view, this is used instead of $calmonth
+     *     and $calyear to support multiple calendars
      */
-    public function __construct($day=0, $month=0, $year=0) {
-
-        $date = usergetdate(time());
-
-        if (empty($day)) {
-            $day = $date['mday'];
-        }
-
-        if (empty($month)) {
-            $month = $date['mon'];
-        }
-
-        if (empty($year)) {
-            $year =  $date['year'];
+    public function __construct($day = 0, $month = 0, $year = 0, $time = 0) {
+        // If a day, month and year were passed then convert it to a timestamp. If these were passed
+        // then we can assume the day, month and year are passed as Gregorian, as no where in core
+        // should we be passing these values rather than the time. This is done for BC.
+        if (!empty($day) || !empty($month) || !empty($year)) {
+            $date = usergetdate(time());
+            if (empty($day)) {
+                $day = $date['mday'];
+            }
+            if (empty($month)) {
+                $month = $date['mon'];
+            }
+            if (empty($year)) {
+                $year =  $date['year'];
+            }
+            if (checkdate($month, $day, $year)) {
+                $this->time = make_timestamp($year, $month, $day);
+            } else {
+                $this->time = time();
+            }
+        } else if (!empty($time)) {
+            $this->time = $time;
+        } else {
+            $this->time = time();
         }
-
-        $this->day = $day;
-        $this->month = $month;
-        $this->year = $year;
     }
 
     /**
@@ -2719,13 +2734,14 @@ class calendar_information {
         }
         return true;
     }
+
     /**
      * Gets todays timestamp for the calendar
      *
      * @return int today timestamp
      */
     public function timestamp_today() {
-        return make_timestamp($this->year, $this->month, $this->day);
+        return $this->time;
     }
     /**
      * Gets tomorrows timestamp for the calendar
@@ -2733,7 +2749,7 @@ class calendar_information {
      * @return int tomorrow timestamp
      */
     public function timestamp_tomorrow() {
-        return make_timestamp($this->year, $this->month, $this->day+1);
+        return $this->time + DAYSECS;
     }
     /**
      * Adds the pretend blocks for the calendar
@@ -2745,7 +2761,7 @@ class calendar_information {
     public function add_sidecalendar_blocks(core_calendar_renderer $renderer, $showfilters=false, $view=null) {
         if ($showfilters) {
             $filters = new block_contents();
-            $filters->content = $renderer->fake_block_filters($this->courseid, $this->day, $this->month, $this->year, $view, $this->courses);
+            $filters->content = $renderer->fake_block_filters($this->courseid, 0, 0, 0, $view, $this->courses);
             $filters->footer = '';
             $filters->title = get_string('eventskey', 'calendar');
             $renderer->add_pretend_calendar_block($filters, BLOCK_POS_RIGHT);
index 165466d..aeb502d 100644 (file)
@@ -32,7 +32,7 @@ if (isset($CFG->calendar_maxevents)) {
 
 $prefs = new stdClass;
 $prefs->timeformat = get_user_preferences('calendar_timeformat', '');
-$prefs->startwday  = get_user_preferences('calendar_startwday', calendar_get_starting_weekday());
+$prefs->startwday  = calendar_get_starting_weekday();
 $prefs->maxevents  = get_user_preferences('calendar_maxevents', $defaultmaxevents);
 $prefs->lookahead  = get_user_preferences('calendar_lookahead', $defaultlookahead);
 $prefs->persistflt = get_user_preferences('calendar_persistflt', 0);
index 4fc9a05..efb4725 100644 (file)
@@ -166,18 +166,27 @@ class core_calendar_renderer extends plugin_renderer_base {
      * @return string
      */
     public function fake_block_threemonths(calendar_information $calendar) {
+        // Get the calendar type we are using.
+        $calendartype = \core_calendar\type_factory::get_calendar_instance();
 
-        list($prevmon, $prevyr) = calendar_sub_month($calendar->month, $calendar->year);
-        list($nextmon, $nextyr) = calendar_add_month($calendar->month, $calendar->year);
+        $date = $calendartype->timestamp_to_date_array($calendar->time);
 
-        $content  = html_writer::start_tag('div', array('class'=>'minicalendarblock'));
-        $content .= calendar_get_mini( $calendar->courses, $calendar->groups, $calendar->users, $prevmon, $prevyr, 'display', $calendar->courseid);
+        $prevmonth = calendar_sub_month($date['mon'], $date['year']);
+        $prevmonthtime = $calendartype->convert_to_gregorian($prevmonth[1], $prevmonth[0], $date['mday']);
+        $prevmonthtime = make_timestamp($prevmonthtime['year'], $prevmonthtime['month'], $prevmonthtime['day']);
+
+        $nextmonth = calendar_add_month($date['mon'], $date['year']);
+        $nextmonthtime = $calendartype->convert_to_gregorian($nextmonth[1], $nextmonth[0], $date['mday']);
+        $nextmonthtime = make_timestamp($nextmonthtime['year'], $nextmonthtime['month'], $nextmonthtime['day']);
+
+        $content  = html_writer::start_tag('div', array('class' => 'minicalendarblock'));
+        $content .= calendar_get_mini($calendar->courses, $calendar->groups, $calendar->users, false, false, 'display', $calendar->courseid, $prevmonthtime);
         $content .= html_writer::end_tag('div');
-        $content .= html_writer::start_tag('div', array('class'=>'minicalendarblock'));
-        $content .= calendar_get_mini($calendar->courses, $calendar->groups, $calendar->users, $calendar->month, $calendar->year, 'display', $calendar->courseid);
+        $content .= html_writer::start_tag('div', array('class' => 'minicalendarblock'));
+        $content .= calendar_get_mini($calendar->courses, $calendar->groups, $calendar->users, false, false, 'display', $calendar->courseid, $calendar->time);
         $content .= html_writer::end_tag('div');
-        $content .= html_writer::start_tag('div', array('class'=>'minicalendarblock'));
-        $content .= calendar_get_mini($calendar->courses, $calendar->groups, $calendar->users, $nextmon, $nextyr, 'display', $calendar->courseid);
+        $content .= html_writer::start_tag('div', array('class' => 'minicalendarblock'));
+        $content .= calendar_get_mini($calendar->courses, $calendar->groups, $calendar->users, false, false, 'display', $calendar->courseid, $nextmonthtime);
         $content .= html_writer::end_tag('div');
         return $content;
     }
@@ -199,24 +208,31 @@ class core_calendar_renderer extends plugin_renderer_base {
      * @param int $day
      * @param int $month
      * @param int $year
+     * @param int $time the unixtime, used for multiple calendar support. The values $day,
+     *     $month and $year are kept for backwards compatibility.
      * @return string
      */
-    protected function add_event_button($courseid, $day=null, $month=null, $year=null) {
+    protected function add_event_button($courseid, $day = null, $month = null, $year = null, $time = null) {
+        // If a day, month and year were passed then convert it to a timestamp. If these were passed
+        // then we can assume the day, month and year are passed as Gregorian, as no where in core
+        // should we be passing these values rather than the time. This is done for BC.
+        if (!empty($day) && !empty($month) && !empty($year)) {
+            if (checkdate($month, $day, $year)) {
+                $time = make_timestamp($year, $month, $day);
+            } else {
+                $time = time();
+            }
+        } else if (empty($time)) {
+            $time = time();
+        }
+
         $output = html_writer::start_tag('div', array('class'=>'buttons'));
-        $output .= html_writer::start_tag('form', array('action'=>CALENDAR_URL.'event.php', 'method'=>'get'));
+        $output .= html_writer::start_tag('form', array('action' => CALENDAR_URL . 'event.php', 'method' => 'get'));
         $output .= html_writer::start_tag('div');
-        $output .= html_writer::empty_tag('input', array('type'=>'hidden', 'name'=>'action', 'value'=>'new'));
-        $output .= html_writer::empty_tag('input', array('type'=>'hidden', 'name'=>'course', 'value'=>$courseid));
-        if ($day !== null) {
-            $output .= html_writer::empty_tag('input', array('type'=>'hidden', 'name'=>'cal_d', 'value'=>$day));
-        }
-        if ($month !== null) {
-            $output .= html_writer::empty_tag('input', array('type'=>'hidden', 'name'=>'cal_m', 'value'=>$month));
-        }
-        if ($year !== null) {
-            $output .= html_writer::empty_tag('input', array('type'=>'hidden', 'name'=>'cal_y', 'value'=>$year));
-        }
-        $output .= html_writer::empty_tag('input', array('type'=>'submit', 'value'=>get_string('newevent', 'calendar')));
+        $output .= html_writer::empty_tag('input', array('type'=>'hidden', 'name' => 'action', 'value' => 'new'));
+        $output .= html_writer::empty_tag('input', array('type'=>'hidden', 'name' => 'course', 'value' => $courseid));
+        $output .= html_writer::empty_tag('input', array('type'=>'hidden', 'name' => 'time', 'value' => $time));
+        $output .= html_writer::empty_tag('input', array('type'=>'submit', 'value' => get_string('newevent', 'calendar')));
         $output .= html_writer::end_tag('div');
         $output .= html_writer::end_tag('form');
         $output .= html_writer::end_tag('div');
@@ -235,18 +251,17 @@ class core_calendar_renderer extends plugin_renderer_base {
             $returnurl = $this->page->url;
         }
 
-        $calendar->checkdate();
         $events = calendar_get_upcoming($calendar->courses, $calendar->groups, $calendar->users, 1, 100, $calendar->timestamp_today());
 
         $output  = html_writer::start_tag('div', array('class'=>'header'));
         if (calendar_user_can_add_event($calendar->course)) {
-            $output .= $this->add_event_button($calendar->course->id, $calendar->day, $calendar->month, $calendar->year);
+            $output .= $this->add_event_button($calendar->course->id, 0, 0, 0, $calendar->time);
         }
         //$output .= html_writer::tag('label', get_string('dayview', 'calendar'), array('for'=>'cal_course_flt_jump'));
         $output .= $this->course_filter_selector($returnurl, get_string('dayview', 'calendar'));
         $output .= html_writer::end_tag('div');
         // Controls
-        $output .= html_writer::tag('div', calendar_top_controls('day', array('id' => $calendar->courseid, 'd' => $calendar->day, 'm' => $calendar->month, 'y' => $calendar->year)), array('class'=>'controls'));
+        $output .= html_writer::tag('div', calendar_top_controls('day', array('id' => $calendar->courseid, 'time' => $calendar->time)), array('class'=>'controls'));
 
         if (empty($events)) {
             // There is nothing to display today.
@@ -376,6 +391,7 @@ class core_calendar_renderer extends plugin_renderer_base {
      * Displays a month in detail
      *
      * @param calendar_information $calendar
+     * @param moodle_url $returnurl the url to return to
      * @return string
      */
     public function show_month_detailed(calendar_information $calendar, moodle_url $returnurl  = null) {
@@ -385,34 +401,52 @@ class core_calendar_renderer extends plugin_renderer_base {
             $returnurl = $this->page->url;
         }
 
-        $date = usergetdate(time());
+        // Get the calendar type we are using.
+        $calendartype = \core_calendar\type_factory::get_calendar_instance();
 
+        // Store the display settings.
         $display = new stdClass;
-        $display->minwday = get_user_preferences('calendar_startwday', calendar_get_starting_weekday());
-        $display->maxwday = $display->minwday + 6;
-        $display->thismonth = ($date['mon'] == $calendar->month);
-        $display->maxdays = calendar_days_in_month($calendar->month, $calendar->year);
-
-        $startwday = 0;
-        if (get_user_timezone_offset() < 99) {
-            // We 'll keep these values as GMT here, and offset them when the time comes to query the db
-            $display->tstart = gmmktime(0, 0, 0, $calendar->month, 1, $calendar->year); // This is GMT
-            $display->tend = gmmktime(23, 59, 59, $calendar->month, $display->maxdays, $calendar->year); // GMT
-            $startwday = gmdate('w', $display->tstart); // $display->tstart is already GMT, so don't use date(): messes with server's TZ
-        } else {
-            // no timezone info specified
-            $display->tstart = mktime(0, 0, 0, $calendar->month, 1, $calendar->year);
-            $display->tend = mktime(23, 59, 59, $calendar->month, $display->maxdays, $calendar->year);
-            $startwday = date('w', $display->tstart); // $display->tstart not necessarily GMT, so use date()
+        $display->thismonth = false;
+
+        // Get the specified date in the calendar type being used.
+        $date = $calendartype->timestamp_to_date_array($calendar->time);
+        $thisdate = $calendartype->timestamp_to_date_array(time());
+        if ($date['mon'] == $thisdate['mon'] && $date['year'] == $thisdate['year']) {
+            $display->thismonth = true;
+            $date = $thisdate;
+            $calendar->time = time();
         }
 
+        // Get Gregorian date.
+        $gregoriandate = $calendartype->convert_to_gregorian($date['year'], $date['mon'], 1);
+
+        // Store the gregorian year and month to be used later.
+        list($gy, $gm, $gd, $gh, $gmin) = array($gregoriandate['year'], $gregoriandate['month'], $gregoriandate['day'],
+            $gregoriandate['hour'], $gregoriandate['minute']);
+
+        // Get the starting week day for this month.
+        $startwday = dayofweek(1, $date['mon'], $date['year']);
+        // Get the days in a week.
+        $daynames = calendar_get_days();
+        // Store the number of days in a week.
+        $numberofdaysinweek = $calendartype->get_num_weekdays();
+
+        $display->minwday = calendar_get_starting_weekday();
+        $display->maxwday = $display->minwday + ($numberofdaysinweek - 1);
+        $display->maxdays = calendar_days_in_month($date['mon'], $date['year']);
+
+        // These are used for DB queries, so we want unixtime, so we need to use Gregorian dates.
+        $display->tstart = make_timestamp($gy, $gm, $gd, $gh, $gmin, 0);
+        $display->tend = make_timestamp($gy, $gm, $display->maxdays, 23, 59, 59);
+
         // Align the starting weekday to fall in our display range
+        // This is simple, not foolproof.
         if ($startwday < $display->minwday) {
-            $startwday += 7;
+            $startwday += $numberofdaysinweek;
         }
 
         // Get events from database
-        $events = calendar_get_events(usertime($display->tstart), usertime($display->tend), $calendar->users, $calendar->groups, $calendar->courses);
+        $events = calendar_get_events($display->tstart, $display->tend, $calendar->users, $calendar->groups, $calendar->courses);
         if (!empty($events)) {
             foreach($events as $eventid => $event) {
                 $event = new calendar_event($event);
@@ -426,61 +460,33 @@ class core_calendar_renderer extends plugin_renderer_base {
         }
 
         // Extract information: events vs. time
-        calendar_events_by_day($events, $calendar->month, $calendar->year, $eventsbyday, $durationbyday, $typesbyday, $calendar->courses);
+        calendar_events_by_day($events, $date['mon'], $date['year'], $eventsbyday, $durationbyday, $typesbyday, $calendar->courses);
 
         $output  = html_writer::start_tag('div', array('class'=>'header'));
         if (calendar_user_can_add_event($calendar->course)) {
-            $output .= $this->add_event_button($calendar->course->id, null, $calendar->month, $calendar->year);
+            $output .= $this->add_event_button($calendar->course->id, 0, 0, 0, $calendar->time);
         }
         $output .= get_string('detailedmonthview', 'calendar').': '.$this->course_filter_selector($returnurl);
         $output .= html_writer::end_tag('div', array('class'=>'header'));
         // Controls
-        $output .= html_writer::tag('div', calendar_top_controls('month', array('id' => $calendar->courseid, 'm' => $calendar->month, 'y' => $calendar->year)), array('class'=>'controls'));
-
-        $days = calendar_get_days();
+        $output .= html_writer::tag('div', calendar_top_controls('month', array('id' => $calendar->courseid, 'time' => $calendar->time)), array('class' => 'controls'));
 
         $table = new html_table();
         $table->attributes = array('class'=>'calendarmonth calendartable');
-        $time = make_timestamp($calendar->year, $calendar->month);
-        $table->summary = get_string('calendarheading', 'calendar', userdate($time, get_string('strftimemonthyear')));
+        $table->summary = get_string('calendarheading', 'calendar', userdate($calendar->time, get_string('strftimemonthyear')));
         $table->data = array();
 
-        $header = new html_table_row();
-        $header->attributes = array('class'=>'weekdays');
-        $header->cells = array();
+        // Get the day names as the header.
+        $header = array();
         for($i = $display->minwday; $i <= $display->maxwday; ++$i) {
-            // This uses the % operator to get the correct weekday no matter what shift we have
-            // applied to the $display->minwday : $display->maxwday range from the default 0 : 6
-            $cell = new html_table_cell(get_string($days[$i % 7], 'calendar'));
-            $cell->header = true;
-            $header->cells[] = $cell;
+            $header[] = $daynames[$i % $numberofdaysinweek]['shortname'];
         }
+        $table->head = $header;
 
         // For the table display. $week is the row; $dayweek is the column.
         $week = 1;
         $dayweek = $startwday;
 
-        // Create an array of all the week days.
-        $wdays = array(0 => '<strong>'. get_string('sun', 'calendar'). '</strong>',
-                       1 => '<strong>'. get_string('mon', 'calendar'). '</strong>',
-                       2 => '<strong>'. get_string('tue', 'calendar'). '</strong>',
-                       3 => '<strong>'. get_string('wed', 'calendar'). '</strong>',
-                       4 => '<strong>'. get_string('thu', 'calendar'). '</strong>',
-                       5 => '<strong>'. get_string('fri', 'calendar'). '</strong>',
-                       6 => '<strong>'. get_string('sat', 'calendar'). '</strong>');
-
-        // Loop only if the day offset is greater than 0.
-        // This loop involves shifting the days around until the desired start day
-        // is at the start of the array.
-        $daycount = 0;
-        while ($display->minwday > $daycount++) {
-            $wdays_end = array_shift($wdays);
-            array_push($wdays, $wdays_end);
-        }
-
-        // Now we set the (modified) array to the table header to be displayed.
-        $table->head = $wdays;
-
         $row = new html_table_row(array());
 
         // Paddding (the first week may have blank days in the beginning)
@@ -496,7 +502,9 @@ class core_calendar_renderer extends plugin_renderer_base {
             $weekend = intval($CFG->calendar_weekend);
         }
 
-        for ($calendar->day = 1; $calendar->day <= $display->maxdays; ++$calendar->day, ++$dayweek) {
+        $daytime = $display->tstart - DAYSECS;
+        for ($day = 1; $day <= $display->maxdays; ++$day, ++$dayweek) {
+            $daytime = $daytime + DAYSECS;
             if($dayweek > $display->maxwday) {
                 // We need to change week (table row)
                 $table->data[] = $row;
@@ -507,36 +515,36 @@ class core_calendar_renderer extends plugin_renderer_base {
 
             // Reset vars
             $cell = new html_table_cell();
-            $dayhref = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view'=>'day', 'course'=>$calendar->courseid)), $calendar->day, $calendar->month, $calendar->year);
+            $dayhref = calendar_get_link_href(new moodle_url(CALENDAR_URL.'view.php', array('view' => 'day', 'course' => $calendar->courseid)), 0, 0, 0, $daytime);
 
             $cellclasses = array();
 
-            if ($weekend & (1 << ($dayweek % 7))) {
+            if ($weekend & (1 << ($dayweek % $numberofdaysinweek))) {
                 // Weekend. This is true no matter what the exact range is.
                 $cellclasses[] = 'weekend';
             }
 
             // Special visual fx if an event is defined
-            if (isset($eventsbyday[$calendar->day])) {
-                if(count($eventsbyday[$calendar->day]) == 1) {
+            if (isset($eventsbyday[$day])) {
+                if(count($eventsbyday[$day]) == 1) {
                     $title = get_string('oneevent', 'calendar');
                 } else {
-                    $title = get_string('manyevents', 'calendar', count($eventsbyday[$calendar->day]));
+                    $title = get_string('manyevents', 'calendar', count($eventsbyday[$day]));
                 }
-                $cell->text = html_writer::tag('div', html_writer::link($dayhref, $calendar->day, array('title'=>$title)), array('class'=>'day'));
+                $cell->text = html_writer::tag('div', html_writer::link($dayhref, $day, array('title'=>$title)), array('class'=>'day'));
             } else {
-                $cell->text = html_writer::tag('div', $calendar->day, array('class'=>'day'));
+                $cell->text = html_writer::tag('div', $day, array('class'=>'day'));
             }
 
             // Special visual fx if an event spans many days
             $durationclass = false;
-            if (isset($typesbyday[$calendar->day]['durationglobal'])) {
+            if (isset($typesbyday[$day]['durationglobal'])) {
                 $durationclass = 'duration_global';
-            } else if (isset($typesbyday[$calendar->day]['durationcourse'])) {
+            } else if (isset($typesbyday[$day]['durationcourse'])) {
                 $durationclass = 'duration_course';
-            } else if (isset($typesbyday[$calendar->day]['durationgroup'])) {
+            } else if (isset($typesbyday[$day]['durationgroup'])) {
                 $durationclass = 'duration_group';
-            } else if (isset($typesbyday[$calendar->day]['durationuser'])) {
+            } else if (isset($typesbyday[$day]['durationuser'])) {
                 $durationclass = 'duration_user';
             }
             if ($durationclass) {
@@ -545,16 +553,16 @@ class core_calendar_renderer extends plugin_renderer_base {
             }
 
             // Special visual fx for today
-            if ($display->thismonth && $calendar->day == $date['mday']) {
+            if ($display->thismonth && $day == $date['mday']) {
                 $cellclasses[] = 'day today';
             } else {
                 $cellclasses[] = 'day nottoday';
             }
             $cell->attributes = array('class'=>join(' ',$cellclasses));
 
-            if (isset($eventsbyday[$calendar->day])) {
+            if (isset($eventsbyday[$day])) {
                 $cell->text .= html_writer::start_tag('ul', array('class'=>'events-new'));
-                foreach($eventsbyday[$calendar->day] as $eventindex) {
+                foreach($eventsbyday[$day] as $eventindex) {
                     // If event has a class set then add it to the event <li> tag
                     $attributes = array();
                     if (!empty($events[$eventindex]->class)) {
@@ -566,9 +574,9 @@ class core_calendar_renderer extends plugin_renderer_base {
                 }
                 $cell->text .= html_writer::end_tag('ul');
             }
-            if (isset($durationbyday[$calendar->day])) {
+            if (isset($durationbyday[$day])) {
                 $cell->text .= html_writer::start_tag('ul', array('class'=>'events-underway'));
-                foreach($durationbyday[$calendar->day] as $eventindex) {
+                foreach($durationbyday[$day] as $eventindex) {
                     $cell->text .= html_writer::tag('li', '['.format_string($events[$eventindex]->name,true).']', array('class'=>'events-underway'));
                 }
                 $cell->text .= html_writer::end_tag('ul');
diff --git a/calendar/tests/behat/behat_calendar.php b/calendar/tests/behat/behat_calendar.php
new file mode 100644 (file)
index 0000000..01dc79b
--- /dev/null
@@ -0,0 +1,62 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+/**
+ * Behat calendar-related steps definitions.
+ *
+ * @package    core_calendar
+ * @category   test
+ * @copyright  2013 Mark Nelson <markn@moodle.com>
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+// NOTE: no MOODLE_INTERNAL used, this file may be required by behat before including /config.php.
+require_once(__DIR__ . '/../../../lib/behat/behat_base.php');
+
+use Behat\Behat\Context\Step\Given as Given;
+use Behat\Gherkin\Node\TableNode as TableNode;
+
+/**
+ * Contains functions used by behat to test functionality.
+ *
+ * @package    core_calendar
+ * @category   test
+ * @copyright  2013 Mark Nelson <markn@moodle.com>
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class behat_calendar extends behat_base {
+
+    /**
+     * Create event.
+     *
+     * @Given /^I create a calendar event with form data:$/
+     * @param TableNode $data
+     * @return array the list of actions to perform
+     */
+    public function i_create_a_calendar_event_with_form_data($data) {
+        // Get the event name.
+        $eventname = $data->getRow(1);
+        $eventname = $eventname[1];
+
+        return array(
+            new Given('I follow "' . get_string('monththis', 'calendar') . '"'),
+            new Given('I click on "' . get_string('newevent', 'calendar') .'" "button"'),
+            new Given('I fill the moodle form with:', $data),
+            new Given('I press "' . get_string('savechanges') . '"'),
+            new Given('I should see "' . $eventname . '"')
+        );
+    }
+}
\ No newline at end of file
diff --git a/calendar/tests/behat/calendar.feature b/calendar/tests/behat/calendar.feature
new file mode 100644 (file)
index 0000000..fc68157
--- /dev/null
@@ -0,0 +1,84 @@
+@core @core_calendar
+Feature: Perform basic calendar functionality
+  In order to ensure the calendar works as expected
+  As an admin
+  I need to create calendar data
+
+  Background:
+    Given the following "users" exists:
+      | username | firstname | lastname | email |
+      | student1 | Student | 1 | student1@asd.com |
+      | student2 | Student | 2 | student2@asd.com |
+    And the following "courses" exists:
+      | fullname | shortname | format |
+      | Course 1 | C1 | topics |
+    And the following "course enrolments" exists:
+      | user | course | role |
+      | student1 | C1 | student |
+    When I log in as "admin"
+    And I follow "Course 1"
+    And I turn editing mode on
+    And I add the "Calendar" block
+
+  Scenario: Create a site event
+    And I create a calendar event with form data:
+      | Type of event | site |
+      | Event title | Really awesome event! |
+      | Description | Come join this awesome event, sucka! |
+    And I log out
+    And I log in as "student1"
+    And I follow "Course 1"
+    And I follow "This month"
+    And I should see "Really awesome event!"
+    And I log out
+    And I log in as "student2"
+    And I follow "This month"
+    And I should see "Really awesome event!"
+
+  Scenario: Create a course event
+    And I create a calendar event with form data:
+      | Type of event | course |
+      | Event title | Really awesome event! |
+      | Description | Come join this awesome event, sucka! |
+    And I log out
+    And I log in as "student1"
+    And I follow "Course 1"
+    And I follow "This month"
+    And I should see "Really awesome event!"
+    And I log out
+    And I log in as "student2"
+    And I follow "This month"
+    And I should not see "Really awesome event!"
+
+  Scenario: Create a user event
+    And I create a calendar event with form data:
+      | Type of event | user |
+      | Event title | Really awesome event! |
+      | Description | Come join this awesome event, sucka! |
+    And I log out
+    And I log in as "student1"
+    And I follow "Course 1"
+    And I follow "This month"
+    And I should not see "Really awesome event!"
+
+  Scenario: Delete an event
+    And I create a calendar event with form data:
+      | Type of event | user |
+      | Event title | Really awesome event! |
+      | Description | Come join this awesome event, sucka! |
+    And I click on "//div[@class='commands']//a[contains(@href, 'delete')]" "xpath_element"
+    And I click on "Delete" "button"
+    And I should not see "Really awesome event!"
+
+  Scenario: Edit an event
+    And I create a calendar event with form data:
+      | Type of event | user |
+      | Event title | Really awesome event! |
+      | Description | Come join this awesome event, sucka! |
+    And I click on "//div[@class='commands']//a[contains(@href, 'edit')]" "xpath_element"
+    And I fill the moodle form with:
+      | Event title | Mediocre event :( |
+      | Description | Wait, this event isn't that great. |
+    And I press "Save changes"
+    And I should see "Mediocre event"
+
index 9f11799..596c838 100644 (file)
@@ -33,6 +33,9 @@ require_once($CFG->dirroot . '/calendar/tests/calendartype_test_example.php');
 require_once($CFG->libdir . '/form/dateselector.php');
 require_once($CFG->libdir . '/form/datetimeselector.php');
 
+// Used to test the calendar/lib.php functions.
+require_once($CFG->dirroot . '/calendar/lib.php');
+
 // Used to test the user datetime profile field.
 require_once($CFG->dirroot . '/user/profile/lib.php');
 require_once($CFG->dirroot . '/user/profile/definelib.php');
@@ -70,8 +73,8 @@ class core_calendar_type_testcase extends advanced_testcase {
         $this->resetAfterTest();
 
         // Test setting it as the 'Test' calendar type.
-        $this->set_calendar_type('test');
-        $this->assertEquals('test', \core_calendar\type_factory::get_calendar_type());
+        $this->set_calendar_type('test_example');
+        $this->assertEquals('test_example', \core_calendar\type_factory::get_calendar_type());
 
         // Test setting it as the 'Gregorian' calendar type.
         $this->set_calendar_type('gregorian');
@@ -90,7 +93,7 @@ class core_calendar_type_testcase extends advanced_testcase {
         $this->core_functions_test('gregorian');
 
         // Test that the core functions reproduce the same results as the test calendar.
-        $this->core_functions_test('test');
+        $this->core_functions_test('test_example');
     }
 
     /**
@@ -120,7 +123,7 @@ class core_calendar_type_testcase extends advanced_testcase {
         $date2['hour'] = 0; // The dateselector element does not have hours.
         $date2['minute'] = 0; // The dateselector element does not have minutes.
         $date2['timestamp'] = 1372896000;
-        $this->convert_dateselector_to_unixtime_test('dateselector', 'test', $date2);
+        $this->convert_dateselector_to_unixtime_test('dateselector', 'test_example', $date2);
 
         $date3 = array();
         $date3['day'] = 4;
@@ -138,12 +141,12 @@ class core_calendar_type_testcase extends advanced_testcase {
         $date4['hour'] = 1;
         $date4['minute'] = 17;
         $date4['timestamp'] = 1372979700;
-        $this->convert_dateselector_to_unixtime_test('datetimeselector', 'test', $date4);
+        $this->convert_dateselector_to_unixtime_test('datetimeselector', 'test_example', $date4);
 
         // The date selector element values are set by using the function usergetdate, here we want to check that
         // the unixtime passed is being successfully converted to the correct values for the calendar type.
         $this->convert_unixtime_to_dateselector_test('gregorian', $date3);
-        $this->convert_unixtime_to_dateselector_test('test', $date4);
+        $this->convert_unixtime_to_dateselector_test('test_example', $date4);
     }
 
     /**
@@ -166,7 +169,7 @@ class core_calendar_type_testcase extends advanced_testcase {
         // the year 1967 should be saved in the DB, as 1/1/1970 converts to 30/10/1967 in Gregorian.
         $date['expectedminyear'] = '1967';
         $date['expectedmaxyear'] = '2010';
-        $this->datetime_field_submission_test('test', $date);
+        $this->datetime_field_submission_test('test_example', $date);
     }
 
     /**
@@ -183,6 +186,17 @@ class core_calendar_type_testcase extends advanced_testcase {
         // Test the userdate function.
         $this->assertEquals($calendar->timestamp_to_date_string($this->user->timecreated, '', 99, true, true),
             userdate($this->user->timecreated));
+
+        // Test the calendar/lib.php functions.
+        $this->assertEquals($calendar->get_weekdays(), calendar_get_days());
+        $this->assertEquals($calendar->get_starting_weekday(), calendar_get_starting_weekday());
+        $this->assertEquals($calendar->get_num_days_in_month('1986', '9'), calendar_days_in_month('1986', '9'));
+        $this->assertEquals($calendar->get_next_month('1986', '9'), calendar_add_month('1986', '9'));
+        $this->assertEquals($calendar->get_prev_month('1986', '9'), calendar_sub_month('1986', '9'));
+
+        // Test the lib/moodle.php functions.
+        $this->assertEquals($calendar->get_num_days_in_month('1986', '9'), days_in_month('1986', '9'));
+        $this->assertEquals($calendar->get_weekday('1986', '9', '16'), dayofweek('16', '9', '1986'));
     }
 
     /**
index 679422e..862cdc7 100644 (file)
@@ -14,7 +14,7 @@
 // You should have received a copy of the GNU General Public License
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
-namespace calendartype_test;
+namespace calendartype_test_example;
 use \core_calendar\type_base;
 
 /**
@@ -29,6 +29,15 @@ use \core_calendar\type_base;
  */
 class structure extends type_base {
 
+    /**
+     * Returns the name of the calendar.
+     *
+     * @return string the calendar name
+     */
+    public function get_name() {
+        return 'test_example';
+    }
+
     /**
      * Returns a list of all the possible days for all months.
      *
@@ -84,10 +93,90 @@ class structure extends type_base {
         return 2050;
     }
 
+    /**
+     * Returns the number of days in a week.
+     *
+     * @return int the number of days
+     */
+    public function get_num_weekdays() {
+        return 7;
+    }
+
+    /**
+     * Returns an indexed list of all the names of the weekdays.
+     *
+     * The list starts with the index 0. Each index, representing a
+     * day, must be an array that contains the indexes 'shortname'
+     * and 'fullname'.
+     *
+     * @return array array of days
+     */
+    public function get_weekdays() {
+        return '';
+    }
+
+    /**
+     * Returns the index of the starting week day.
+     *
+     * @return int
+     */
+    public function get_starting_weekday() {
+        return '';
+    }
+
+    /**
+     * Returns the index of the weekday for a specific calendar date.
+     *
+     * @param int $year
+     * @param int $month
+     * @param int $day
+     * @return int
+     */
+    public function get_weekday($year, $month, $day) {
+        return '';
+    }
+
+    /**
+     * Returns the number of days in a given month.
+     *
+     * @param int $year
+     * @param int $month
+     * @return int the number of days
+     */
+    public function get_num_days_in_month($year, $month) {
+        return '';
+    }
+
+    /**
+     * Get the previous month.
+     *
+     * If the current month is January, it will get the last month of the previous year.
+     *
+     * @param int $year
+     * @param int $month
+     * @return array previous month and year
+     */
+    public function get_prev_month($year, $month) {
+        return '';
+    }
+
+    /**
+     * Get the next month.
+     *
+     * If the current month is December, it will get the first month of the following year.
+     *
+     * @param int $year
+     * @param int $month
+     * @return array the following month and year
+     */
+    public function get_next_month($year, $month) {
+        return '';
+    }
+
     /**
      * Returns a formatted string that represents a date in user time.
      *
-     * @param int $date the timestamp in UTC, as obtained from the database
+     * @param int $time the timestamp in UTC, as obtained from the database
      * @param string $format strftime format
      * @param int|float|string $timezone the timezone to use
      *        {@link http://docs.moodle.org/dev/Time_API#Timezone}
@@ -97,7 +186,7 @@ class structure extends type_base {
      *        if false then the leading zero is maintained
      * @return string the formatted date/time
      */
-    public function timestamp_to_date_string($date, $format, $timezone, $fixday, $fixhour) {
+    public function timestamp_to_date_string($time, $format, $timezone, $fixday, $fixhour) {
         return '';
     }
 
@@ -110,7 +199,7 @@ class structure extends type_base {
      *        {@link http://docs.moodle.org/dev/Time_API#Timezone}
      * @return array an array that represents the date in user time
      */
-    public function timestamp_to_date_array($time, $timezone) {
+    public function timestamp_to_date_array($time, $timezone = 99) {
         $gregoriancalendar = \core_calendar\type_factory::get_calendar_instance('gregorian');
         $date = $gregoriancalendar->timestamp_to_date_array($time, $timezone);
         $newdate = $this->convert_from_gregorian($date['year'], $date['mon'], $date['mday'],
@@ -143,10 +232,10 @@ class structure extends type_base {
         list($year, $month, $day, $hour, $minute) = explode('/', $date);
 
         return array('year' => (int) $year,
-                     'month' => (int) $month,
-                     'day' => (int) $day,
-                     'hour' => (int) $hour,
-                     'minute' => (int) $minute);
+            'month' => (int) $month,
+            'day' => (int) $day,
+            'hour' => (int) $hour,
+            'minute' => (int) $minute);
 
     }
 
@@ -168,9 +257,9 @@ class structure extends type_base {
         list($year, $month, $day, $hour, $minute) = explode('/', $date);
 
         return array('year' => (int) $year,
-                     'month' => (int) $month,
-                     'day' => (int) $day,
-                     'hour' => (int) $hour,
-                     'minute' => (int) $minute);
+            'month' => (int) $month,
+            'day' => (int) $day,
+            'hour' => (int) $hour,
+            'minute' => (int) $minute);
     }
 }
index d209864..cf50cfc 100644 (file)
@@ -26,6 +26,18 @@ use core_calendar\type_base;
  */
 class structure extends type_base {
 
+    /**
+     * Returns the name of the calendar.
+     *
+     * This is the non-translated name, usually just
+     * the name of the folder.
+     *
+     * @return string the calendar name
+     */
+    public function get_name() {
+        return 'gregorian';
+    }
+
     /**
      * Returns a list of all the possible days for all months.
      *
@@ -81,6 +93,140 @@ class structure extends type_base {
         return 2050;
     }
 
+    /**
+     * Returns the number of days in a week.
+     *
+     * @return int the number of days
+     */
+    public function get_num_weekdays() {
+        return 7;
+    }
+
+    /**
+     * Returns an indexed list of all the names of the weekdays.
+     *
+     * The list starts with the index 0. Each index, representing a
+     * day, must be an array that contains the indexes 'shortname'
+     * and 'fullname'.
+     *
+     * @return array array of days
+     */
+    public function get_weekdays() {
+        return array(
+            0 => array(
+                'shortname' => get_string('sun', 'calendar'),
+                'fullname' => get_string('sunday', 'calendar')
+            ),
+            1 => array(
+                'shortname' => get_string('mon', 'calendar'),
+                'fullname' => get_string('monday', 'calendar')
+            ),
+            2 => array(
+                'shortname' => get_string('tue', 'calendar'),
+                'fullname' => get_string('tuesday', 'calendar')
+            ),
+            3 => array(
+                'shortname' => get_string('wed', 'calendar'),
+                'fullname' => get_string('wednesday', 'calendar')
+            ),
+            4 => array(
+                'shortname' => get_string('thu', 'calendar'),
+                'fullname' => get_string('thursday', 'calendar')
+            ),
+            5 => array(
+                'shortname' => get_string('fri', 'calendar'),
+                'fullname' => get_string('friday', 'calendar')
+            ),
+            6 => array(
+                'shortname' => get_string('sat', 'calendar'),
+                'fullname' => get_string('saturday', 'calendar')
+            ),
+        );
+    }
+
+    /**
+     * Returns the index of the starting week day.
+     *
+     * This may vary, for example some may consider Monday as the start of the week,
+     * where as others may consider Sunday the start.
+     *
+     * @return int
+     */
+    public function get_starting_weekday() {
+        global $CFG;
+
+        if (isset($CFG->calendar_startwday)) {
+            $firstday = $CFG->calendar_startwday;
+        } else {
+            $firstday = get_string('firstdayofweek', 'langconfig');
+        }
+
+        if (!is_numeric($firstday)) {
+            $startingweekday = CALENDAR_DEFAULT_STARTING_WEEKDAY;
+        } else {
+            $startingweekday = intval($firstday) % 7;
+        }
+
+        return get_user_preferences('calendar_startwday', $startingweekday);
+    }
+
+    /**
+     * Returns the index of the weekday for a specific calendar date.
+     *
+     * @param int $year
+     * @param int $month
+     * @param int $day
+     * @return int
+     */
+    public function get_weekday($year, $month, $day) {
+        return intval(date('w', mktime(12, 0, 0, $month, $day, $year)));
+    }
+
+    /**
+     * Returns the number of days in a given month.
+     *
+     * @param int $year
+     * @param int $month
+     * @return int the number of days
+     */
+    public function get_num_days_in_month($year, $month) {
+        return intval(date('t', mktime(0, 0, 0, $month, 1, $year)));
+    }
+
+    /**
+     * Get the previous month.
+     *
+     * If the current month is January, it will get the last month of the previous year.
+     *
+     * @param int $year
+     * @param int $month
+     * @return array previous month and year
+     */
+    public function get_prev_month($year, $month) {
+        if ($month == 1) {
+            return array(12, $year - 1);
+        } else {
+            return array($month - 1, $year);
+        }
+    }
+
+    /**
+     * Get the next month.
+     *
+     * If the current month is December, it will get the first month of the following year.
+     *
+     * @param int $year
+     * @param int $month
+     * @return array the following month and year
+     */
+    public function get_next_month($year, $month) {
+        if ($month == 12) {
+            return array(1, $year + 1);
+        } else {
+            return array($month + 1, $year);
+        }
+    }
+
     /**
      * Returns a formatted string that represents a date in user time.
      *
@@ -94,7 +240,7 @@ class structure extends type_base {
      * If parameter fixday = true (default), then take off leading
      * zero from %d, else maintain it.
      *
-     * @param int $date the timestamp in UTC, as obtained from the database
+     * @param int $time the timestamp in UTC, as obtained from the database
      * @param string $format strftime format
      * @param int|float|string $timezone the timezone to use
      *        {@link http://docs.moodle.org/dev/Time_API#Timezone}
@@ -104,7 +250,7 @@ class structure extends type_base {
      *        if false then the leading zero is maintained
      * @return string the formatted date/time
      */
-    public function timestamp_to_date_string($date, $format, $timezone, $fixday, $fixhour) {
+    public function timestamp_to_date_string($time, $format, $timezone, $fixday, $fixhour) {
         global $CFG;
 
         if (empty($format)) {
@@ -134,7 +280,7 @@ class structure extends type_base {
         // Add daylight saving offset for string timezones only, as we can't get dst for
         // float values. if timezone is 99 (user default timezone), then try update dst.
         if ((99 == $timezone) || !is_numeric($timezone)) {
-            $date += dst_offset_on($date, $timezone);
+            $time += dst_offset_on($time, $timezone);
         }
 
         $timezone = get_user_timezone_offset($timezone);
@@ -142,24 +288,24 @@ class structure extends type_base {
         // If we are running under Windows convert to windows encoding and then back to UTF-8
         // (because it's impossible to specify UTF-8 to fetch locale info in Win32).
         if (abs($timezone) > 13) { // Server time.
-            $datestring = date_format_string($date, $format, $timezone);
+            $datestring = date_format_string($time, $format, $timezone);
             if ($fixday) {
-                $daystring  = ltrim(str_replace(array(' 0', ' '), '', strftime(' %d', $date)));
+                $daystring  = ltrim(str_replace(array(' 0', ' '), '', strftime(' %d', $time)));
                 $datestring = str_replace('DD', $daystring, $datestring);
             }
             if ($fixhour) {
-                $hourstring = ltrim(str_replace(array(' 0', ' '), '', strftime(' %I', $date)));
+                $hourstring = ltrim(str_replace(array(' 0', ' '), '', strftime(' %I', $time)));
                 $datestring = str_replace('HH', $hourstring, $datestring);
             }
         } else {
-            $date += (int)($timezone * 3600);
-            $datestring = date_format_string($date, $format, $timezone);
+            $time += (int)($timezone * 3600);
+            $datestring = date_format_string($time, $format, $timezone);
             if ($fixday) {
-                $daystring  = ltrim(str_replace(array(' 0', ' '), '', gmstrftime(' %d', $date)));
+                $daystring  = ltrim(str_replace(array(' 0', ' '), '', gmstrftime(' %d', $time)));
                 $datestring = str_replace('DD', $daystring, $datestring);
             }
             if ($fixhour) {
-                $hourstring = ltrim(str_replace(array(' 0', ' '), '', gmstrftime(' %I', $date)));
+                $hourstring = ltrim(str_replace(array(' 0', ' '), '', gmstrftime(' %I', $time)));
                 $datestring = str_replace('HH', $hourstring, $datestring);
             }
         }
@@ -176,7 +322,7 @@ class structure extends type_base {
      *        dst offset is applied {@link http://docs.moodle.org/dev/Time_API#Timezone}
      * @return array an array that represents the date in user time
      */
-    public function timestamp_to_date_array($time, $timezone) {
+    public function timestamp_to_date_array($time, $timezone = 99) {
         return usergetdate($time, $timezone);
     }
 
@@ -192,7 +338,7 @@ class structure extends type_base {
      * @param int $day
      * @param int $hour
      * @param int $minute
-     * @return array the converted day, month, year, hour and minute.
+     * @return array the converted date
      */
     public function convert_from_gregorian($year, $month, $day, $hour = 0, $minute = 0) {
         $date = array();
@@ -217,7 +363,7 @@ class structure extends type_base {
      * @param int $day
      * @param int $hour
      * @param int $minute
-     * @return array the converted day, month, year, hour and minute.
+     * @return array the converted date
      */
     public function convert_to_gregorian($year, $month, $day, $hour = 0, $minute = 0) {
         $date = array();
index deeac17..cec41a3 100644 (file)
@@ -24,6 +24,6 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$plugin->version   = 2013082300; // The current plugin version (Date: YYYYMMDDXX).
-$plugin->requires  = 2012120300; // Requires this Moodle version.
+$plugin->version   = 2013100200; // The current plugin version (Date: YYYYMMDDXX).
+$plugin->requires  = 2013082300; // Requires this Moodle version.
 $plugin->component = 'calendartype_gregorian'; // Full name of the plugin (used for diagnostics).
index 73e5924..8e6f7d4 100644 (file)
@@ -46,26 +46,36 @@ require_once($CFG->dirroot.'/calendar/lib.php');
 
 $courseid = optional_param('course', SITEID, PARAM_INT);
 $view = optional_param('view', 'upcoming', PARAM_ALPHA);
-$day  = optional_param('cal_d', 0, PARAM_INT);
-$mon  = optional_param('cal_m', 0, PARAM_INT);
-$yr   = optional_param('cal_y', 0, PARAM_INT);
+$day = optional_param('cal_d', 0, PARAM_INT);
+$mon = optional_param('cal_m', 0, PARAM_INT);
+$year = optional_param('cal_y', 0, PARAM_INT);
+$time = optional_param('time', 0, PARAM_INT);
 
 $url = new moodle_url('/calendar/view.php');
+
 if ($courseid != SITEID) {
     $url->param('course', $courseid);
 }
+
 if ($view !== 'upcoming') {
     $url->param('view', $view);
 }
-if ($day !== 0) {
-    $url->param('cal_d', $day);
-}
-if ($mon !== 0) {
-    $url->param('cal_m', $mon);
-}
-if ($yr !== 0) {
-    $url->param('cal_y', $yr);
+
+// If a day, month and year were passed then convert it to a timestamp. If these were passed
+// then we can assume the day, month and year are passed as Gregorian, as no where in core
+// should we be passing these values rather than the time. This is done for BC.
+if (!empty($day) && !empty($mon) && !empty($year)) {
+    if (checkdate($mon, $day, $year)) {
+        $time = make_timestamp($year, $mon, $day);
+    } else {
+        $time = time();
+    }
+} else if (empty($time)) {
+    $time = time();
 }
+
+$url->param('time', $time);
+
 $PAGE->set_url($url);
 
 if ($courseid != SITEID && !empty($courseid)) {
@@ -78,23 +88,16 @@ if ($courseid != SITEID && !empty($courseid)) {
     $courses = calendar_get_default_courses();
     $issite = true;
 }
+
 require_course_login($course);
 
-$calendar = new calendar_information($day, $mon, $yr);
+$calendar = new calendar_information(0, 0, 0, $time);
 $calendar->prepare_for_view($course, $courses);
 
-$now = usergetdate(time());
 $pagetitle = '';
 
 $strcalendar = get_string('calendar', 'calendar');
 
-if (!checkdate($mon, $day, $yr)) {
-    $day = intval($now['mday']);
-    $mon = intval($now['mon']);
-    $yr = intval($now['year']);
-}
-$time = make_timestamp($yr, $mon, $day);
-
 switch($view) {
     case 'day':
         $PAGE->navbar->add(userdate($time, get_string('strftimedate')));
index 9287986..3545a64 100644 (file)
@@ -107,21 +107,17 @@ $mform = new edit_grade_form(null, array('grade_item'=>$grade_item, 'gpr'=>$gpr)
 if ($grade = $DB->get_record('grade_grades', array('itemid' => $grade_item->id, 'userid' => $userid))) {
 
     // always clean existing feedback - grading should not have XSS risk
-    if (can_use_html_editor()) {
-        if (empty($grade->feedback)) {
-            $grade->feedback  = '';
-        } else {
-            $options = new stdClass();
-            $options->smiley  = false;
-            $options->filter  = false;
-            $options->noclean = false;
-            $options->para    = false;
-            $grade->feedback  = format_text($grade->feedback, $grade->feedbackformat, $options);
-        }
-        $grade->feedbackformat = FORMAT_HTML;
+    if (empty($grade->feedback)) {
+        $grade->feedback  = '';
     } else {
-        $grade->feedback       = clean_text($grade->feedback, $grade->feedbackformat);
+        $options = new stdClass();
+        $options->smiley  = false;
+        $options->filter  = false;
+        $options->noclean = false;
+        $options->para    = false;
+        $grade->feedback  = format_text($grade->feedback, $grade->feedbackformat, $options);
     }
+    $grade->feedbackformat = FORMAT_HTML;
 
     $grade->locked      = $grade->locked     > 0 ? 1:0;
     $grade->overridden  = $grade->overridden > 0 ? 1:0;
index 40998f8..07cc2ec 100644 (file)
@@ -1105,7 +1105,6 @@ $string['upgradingversion'] = 'Upgrading to new version';
 $string['upwards'] = 'upwards';
 $string['useblogassociations'] = 'Enable associations';
 $string['useexternalyui'] = 'Use online YUI libraries';
-$string['usehtmleditor'] = 'Use HTML editor';
 $string['user'] = 'User';
 $string['userbulk'] = 'Bulk user actions';
 $string['userlist'] = 'Browse list of users';
index 4447b7b..dcdaa21 100644 (file)
@@ -429,6 +429,7 @@ $string['day'] = 'day';
 $string['days'] = 'days';
 $string['decodinginternallinks'] = 'Decoding internal links';
 $string['default'] = 'Default';
+$string['defaulteditor'] = 'Default editor';
 $string['defaultcoursestudent'] = 'Student';
 $string['defaultcoursestudentdescription'] = 'Students generally have fewer privileges within a course.';
 $string['defaultcoursestudents'] = 'Students';
@@ -1743,7 +1744,7 @@ $string['targetrole'] = 'Target role';
 $string['teacheronly'] = 'for the {$a} only';
 $string['teacherroles'] = '{$a} roles';
 $string['teachers'] = 'Teachers';
-$string['textediting'] = 'When editing text';
+$string['textediting'] = 'Text editor';
 $string['texteditor'] = 'Use standard web forms';
 $string['textformat'] = 'Plain text format';
 $string['thanks'] = 'Thanks';
index 63306f6..84b8489 100644 (file)
@@ -36,6 +36,10 @@ $string['deleteconfirm'] = 'Delete this note?';
 $string['deletenotes'] = 'Delete all notes';
 $string['editnote'] = 'Edit note';
 $string['enablenotes'] = 'Enable notes';
+$string['eventnotecreated'] = 'Note created';
+$string['eventnoteupdate'] = 'Note updated';
+$string['eventnotedeleted'] = 'Note deleted';
+$string['eventnotesviewed'] = 'Notes viewed';
 $string['groupaddnewnote'] = 'Add a common note';
 $string['invalidid'] = 'Invalid note ID specified';
 $string['invaliduserid'] = 'Invalid user id: {$a}';
index 278e802..94d6001 100644 (file)
@@ -3841,8 +3841,7 @@ class admin_setting_special_frontpagedesc extends admin_setting {
     public function output_html($data, $query='') {
         global $CFG;
 
-        $CFG->adminusehtmleditor = can_use_html_editor();
-        $return = '<div class="form-htmlarea">'.print_textarea($CFG->adminusehtmleditor, 15, 60, 0, 0, $this->get_full_name(), $data, 0, true, 'summary') .'</div>';
+        $return = '<div class="form-htmlarea">'.print_textarea(true, 15, 60, 0, 0, $this->get_full_name(), $data, 0, true, 'summary') .'</div>';
 
         return format_admin_setting($this, $this->visiblename, $return, $this->description, false, '', NULL, $query);
     }
diff --git a/lib/classes/event/note_created.php b/lib/classes/event/note_created.php
new file mode 100644 (file)
index 0000000..838ba8c
--- /dev/null
@@ -0,0 +1,87 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+namespace core\event;
+
+defined('MOODLE_INTERNAL') || die();
+
+/**
+ * Event for when a new note entry is added.
+ *
+ * @package    core
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+/**
+ * Class note_created
+ *
+ * Class for event to be triggered when a note is created.
+ *
+ * @package    core
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class note_created extends \core\event\base {
+
+    /**
+     * Set basic properties for the event.
+     */
+    protected function init() {
+        $this->data['objecttable'] = 'post';
+        $this->data['crud'] = 'c';
+        $this->data['level'] = self::LEVEL_OTHER;
+    }
+
+    /**
+     * Returns localised general event name.
+     *
+     * @return string
+     */
+    public static function get_name() {
+        return get_string("eventnotecreated", "core_notes");
+    }
+
+    /**
+     * Returns non-localised event description with id's for admin use only.
+     *
+     * @return string
+     */
+    public function get_description() {
+        return 'Note for user with id "'. $this->relateduserid . '" was created by user with id "' . $this->userid . '"';
+    }
+
+    /**
+     * Returns relevant URL.
+     * @return \moodle_url
+     */
+    public function get_url() {
+        $logurl = new \moodle_url('/note/index.php', array('course' => $this->courseid, 'user' => $this->relateduserid));
+        $logurl->set_anchor('note-' . $this->objectid);
+        return $logurl;
+    }
+
+    /**
+     * replace add_to_log() statement.
+     *
+     * @return array of parameters to be passed to legacy add_to_log() function.
+     */
+    protected function get_legacy_logdata() {
+        $logurl = new \moodle_url('index.php', array('course' => $this->courseid, 'user' => $this->relateduserid));
+        $logurl->set_anchor('note-' . $this->objectid);
+        return array($this->courseid, 'notes', 'add', $logurl, 'add note');
+    }
+}
diff --git a/lib/classes/event/note_deleted.php b/lib/classes/event/note_deleted.php
new file mode 100644 (file)
index 0000000..45249fe
--- /dev/null
@@ -0,0 +1,77 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+namespace core\event;
+
+defined('MOODLE_INTERNAL') || die();
+
+/**
+ * Event for when a new note entry deleted.
+ *
+ * @package    core
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+/**
+ * Class note_deleted
+ *
+ * Class for event to be triggered when a note is deleted.
+ *
+ * @package    core
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class note_deleted extends \core\event\base {
+
+    /**
+     * Set basic properties for the event.
+     */
+    protected function init() {
+        $this->data['objecttable'] = 'post';
+        $this->data['crud'] = 'd';
+        $this->data['level'] = self::LEVEL_OTHER;
+    }
+
+    /**
+     * Returns localised general event name.
+     *
+     * @return string
+     */
+    public static function get_name() {
+        return get_string("eventnotedeleted", "core_notes");
+    }
+
+    /**
+     * Returns non-localised event description with id's for admin use only.
+     *
+     * @return string
+     */
+    public function get_description() {
+        return 'Note for user with id "'. $this->relateduserid . '" was deleted by user with id "' . $this->userid . '"';
+    }
+
+    /**
+     * replace add_to_log() statement.
+     *
+     * @return array of parameters to be passed to legacy add_to_log() function.
+     */
+    protected function get_legacy_logdata() {
+        $logurl = new \moodle_url('index.php', array('course' => $this->courseid, 'user' => $this->relateduserid));
+        $logurl->set_anchor('note-' . $this->objectid);
+        return array($this->courseid, 'notes', 'delete', $logurl, 'delete note');
+    }
+}
diff --git a/lib/classes/event/note_updated.php b/lib/classes/event/note_updated.php
new file mode 100644 (file)
index 0000000..df3ac78
--- /dev/null
@@ -0,0 +1,87 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+namespace core\event;
+
+defined('MOODLE_INTERNAL') || die();
+
+/**
+ * Event for when a new note entry is updated.
+ *
+ * @package    core
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+/**
+ * Class note_updated
+ *
+ * Class for event to be triggered when a note is updated.
+ *
+ * @package    core
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class note_updated extends \core\event\base {
+
+    /**
+     * Set basic properties for the event.
+     */
+    protected function init() {
+        $this->data['objecttable'] = 'post';
+        $this->data['crud'] = 'u';
+        $this->data['level'] = self::LEVEL_OTHER;
+    }
+
+    /**
+     * Returns localised general event name.
+     *
+     * @return string
+     */
+    public static function get_name() {
+        return get_string("eventnoteupdated", "core_notes");
+    }
+
+    /**
+     * Returns non-localised event description with id's for admin use only.
+     *
+     * @return string
+     */
+    public function get_description() {
+        return 'Note for user with id "'. $this->relateduserid . '" was updated by user with id "'. $this->userid . '"';
+    }
+
+    /**
+     * Returns relevant URL.
+     * @return \moodle_url
+     */
+    public function get_url() {
+        $logurl = new \moodle_url('/note/index.php', array('course' => $this->courseid, 'user' => $this->relateduserid));
+        $logurl->set_anchor('note-' . $this->objectid);
+        return $logurl;
+    }
+
+    /**
+     * Replace add_to_log() statement.
+     *
+     * @return array of parameters to be passed to legacy add_to_log() function.
+     */
+    protected function get_legacy_logdata() {
+        $logurl = new \moodle_url('index.php', array('course' => $this->courseid, 'user' => $this->relateduserid));
+        $logurl->set_anchor('note-' . $this->objectid);
+        return array($this->courseid, 'notes', 'update', $logurl, 'update note');
+    }
+}
diff --git a/lib/classes/event/notes_viewed.php b/lib/classes/event/notes_viewed.php
new file mode 100644 (file)
index 0000000..3680fc4
--- /dev/null
@@ -0,0 +1,83 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+namespace core\event;
+
+defined('MOODLE_INTERNAL') || die();
+
+/**
+ * Event for when a new note entry viewed.
+ *
+ * @package    core
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+/**
+ * Class note_viewed
+ *
+ * Class for event to be triggered when a note is viewed.
+ *
+ * @package    core
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class notes_viewed extends \core\event\content_viewed {
+
+    /**
+     * Set basic properties for the event.
+     */
+    protected function init() {
+        $this->data['crud'] = 'r';
+        $this->data['level'] = self::LEVEL_OTHER;
+    }
+
+    /**
+     * Returns localised general event name.
+     *
+     * @return string
+     */
+    public static function get_name() {
+        return get_string("eventnotesviewed", "core_notes");
+    }
+
+    /**
+     * Returns non-localised event description with id's for admin use only.
+     *
+     * @return string
+     */
+    public function get_description() {
+        return 'Note for user with id "'. $this->relateduserid . '" was viewed by user with id "'. $this->userid . '"';
+    }
+
+    /**
+     * Returns relevant URL.
+     * @return \moodle_url
+     */
+    public function get_url() {
+        return new \moodle_url('/note/index.php', array('course' => $this->courseid, 'user' => $this->relateduserid));
+    }
+
+    /**
+     * replace add_to_log() statement.
+     *
+     * @return array of parameters to be passed to legacy add_to_log() function.
+     */
+    protected function get_legacy_logdata() {
+        return array($this->courseid, 'notes', 'view', 'index.php?course=' . $this->courseid.'&amp;user=' . $this->relateduserid,
+            'view notes');
+    }
+}
old mode 100644 (file)
new mode 100755 (executable)
index 9c3152d..f67785f
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8" ?>
-<XMLDB PATH="lib/db" VERSION="20131008" COMMENT="XMLDB file for core Moodle tables"
+<XMLDB PATH="lib/db" VERSION="20131001" COMMENT="XMLDB file for core Moodle tables"
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xsi:noNamespaceSchemaLocation="../../lib/xmldb/xmldb.xsd"
 >
         <FIELD NAME="mailformat" TYPE="int" LENGTH="1" NOTNULL="true" DEFAULT="1" SEQUENCE="false"/>
         <FIELD NAME="maildigest" TYPE="int" LENGTH="1" NOTNULL="true" DEFAULT="0" SEQUENCE="false"/>
         <FIELD NAME="maildisplay" TYPE="int" LENGTH="2" NOTNULL="true" DEFAULT="2" SEQUENCE="false"/>
-        <FIELD NAME="htmleditor" TYPE="int" LENGTH="1" NOTNULL="true" DEFAULT="1" SEQUENCE="false"/>
         <FIELD NAME="autosubscribe" TYPE="int" LENGTH="1" NOTNULL="true" DEFAULT="1" SEQUENCE="false"/>
         <FIELD NAME="trackforums" TYPE="int" LENGTH="1" NOTNULL="true" DEFAULT="0" SEQUENCE="false"/>
         <FIELD NAME="timecreated" TYPE="int" LENGTH="10" NOTNULL="true" DEFAULT="0" SEQUENCE="false"/>
index c1751bb..c14811b 100644 (file)
@@ -2642,5 +2642,23 @@ function xmldb_main_upgrade($oldversion) {
         upgrade_main_savepoint(true, 2013100800.01);
     }
 
+    if ($oldversion < 2013100800.02) {
+        $sql = "INSERT INTO {user_preferences}(userid, name, value)
+                SELECT id, 'htmleditor', 'textarea' FROM {user} u where u.htmleditor = 0";
+        $DB->execute($sql);
+
+        // Define field htmleditor to be dropped from user
+        $table = new xmldb_table('user');
+        $field = new xmldb_field('htmleditor');
+
+        // Conditionally launch drop field requested
+        if ($dbman->field_exists($table, $field)) {
+            $dbman->drop_field($table, $field);
+        }
+
+        // Main savepoint reached.
+        upgrade_main_savepoint(true, 2013100800.02);
+    }
+
     return true;
 }
index 9709302..f089280 100644 (file)
@@ -1483,7 +1483,7 @@ function print_user_picture($user, $courseid, $picture=NULL, $size=0, $return=fa
  * When using this function, you should
  *
  * @global object
- * @param bool $usehtmleditor Enables the use of the htmleditor for this field.
+ * @param bool $unused No longer used.
  * @param int $rows Number of rows to display  (minimum of 10 when $height is non-null)
  * @param int $cols Number of columns to display (minimum of 65 when $width is non-null)
  * @param null $width (Deprecated) Width of the element; if a value is passed, the minimum value for $cols will be 65. Value is otherwise ignored.
@@ -1495,7 +1495,7 @@ function print_user_picture($user, $courseid, $picture=NULL, $size=0, $return=fa
  * @param string $id CSS ID to add to the textarea element.
  * @return string|void depending on the value of $return
  */
-function print_textarea($usehtmleditor, $rows, $cols, $width, $height, $name, $value='', $obsolete=0, $return=false, $id='') {
+function print_textarea($unused, $rows, $cols, $width, $height, $name, $value='', $obsolete=0, $return=false, $id='') {
     /// $width and height are legacy fields and no longer used as pixels like they used to be.
     /// However, you can set them to zero to override the mincols and minrows values below.
 
@@ -1512,29 +1512,19 @@ function print_textarea($usehtmleditor, $rows, $cols, $width, $height, $name, $v
         $id = 'edit-'.$name;
     }
 
-    if ($usehtmleditor) {
-        if ($height && ($rows < $minrows)) {
-            $rows = $minrows;
-        }
-        if ($width && ($cols < $mincols)) {
-            $cols = $mincols;
-        }
+    if ($height && ($rows < $minrows)) {
+        $rows = $minrows;
     }
-
-    if ($usehtmleditor) {
-        editors_head_setup();
-        $editor = editors_get_preferred_editor(FORMAT_HTML);
-        $editor->use_editor($id, array('legacy'=>true));
-    } else {
-        $editorclass = '';
+    if ($width && ($cols < $mincols)) {
+        $cols = $mincols;
     }
 
+    editors_head_setup();
+    $editor = editors_get_preferred_editor(FORMAT_HTML);
+    $editor->use_editor($id, array('legacy'=>true));
+
     $str .= "\n".'<textarea class="form-textarea" id="'. $id .'" name="'. $name .'" rows="'. $rows .'" cols="'. $cols .'" spellcheck="true">'."\n";
-    if ($usehtmleditor) {
-        $str .= htmlspecialchars($value); // needed for editing of cleaned text!
-    } else {
-        $str .= s($value);
-    }
+    $str .= htmlspecialchars($value); // needed for editing of cleaned text!
     $str .= '</textarea>'."\n";
 
     if ($return) {
@@ -4692,3 +4682,15 @@ function badges_get_issued_badge_info($hash) {
     $assertion = new core_badges_assertion($hash);
     return $assertion->get_badge_assertion();
 }
+
+/**
+ * Does the user want and can edit using rich text html editor?
+ * This function does not make sense anymore because a user can directly choose their preferred editor.
+ *
+ * @deprecated since 2.6
+ * @return bool
+ */
+function can_use_html_editor() {
+    debugging('can_use_html_editor has been deprecated please update your code to assume it returns true.', DEBUG_DEVELOPER);
+    return true;
+}
index 15f667b..2cb8b93 100644 (file)
@@ -37,7 +37,14 @@ function editors_get_preferred_editor($format = NULL) {
 
     $enabled = editors_get_enabled();
 
-    $preventhtml = (count($enabled) > 1 and empty($USER->htmleditor));
+    $preference = get_user_preferences('htmleditor', '', $USER);
+
+    if (isset($enabled[$preference])) {
+        // Edit the list of editors so the users preferred editor is first in the list.
+        $editor = $enabled[$preference];
+        unset($enabled[$preference]);
+        array_unshift($enabled, $editor);
+    }
 
     // now find some plugin that supports format and is available
     $editor = false;
@@ -46,27 +53,11 @@ function editors_get_preferred_editor($format = NULL) {
             // bad luck, this editor is not compatible
             continue;
         }
-        if ($preventhtml and $format == FORMAT_HTML and $e->get_preferred_format() == FORMAT_HTML) {
-            // this is really not what we want but we could use it if nothing better found
-            $editor = $e;
-            continue;
-        }
         if (!$supports = $e->get_supported_formats()) {
             // buggy editor!
             continue;
         }
-        if (is_null($format)) {
-            // format does not matter
-            if ($preventhtml and $e->get_preferred_format() == FORMAT_HTML) {
-                // this is really not what we want but we could use it if nothing better found
-                $editor = $e;
-                continue;
-            } else {
-                $editor = $e;
-                break;
-            }
-        }
-        if (in_array($format, $supports)) {
+        if (is_null($format) || in_array($format, $supports)) {
             // editor supports this format, yay!
             $editor = $e;
             break;
@@ -87,22 +78,7 @@ function editors_get_preferred_editor($format = NULL) {
 function editors_get_preferred_format() {
     global $USER;
 
-    $editors = editors_get_enabled();
-    if (count($editors) == 1) {
-        $editor = reset($editors);
-        return $editor->get_preferred_format();
-    }
-
-    foreach ($editors as $editor) {
-        if (empty($USER->htmleditor) and $editor->get_preferred_format() == FORMAT_HTML) {
-            // we do not prefer this one
-            continue;
-        }
-        return $editor->get_preferred_format();
-    }
-
-    // user did not want html editor, but there is no other choice, sorry
-    $editor = reset($editors);
+    $editor = editors_get_preferred_editor();
     return $editor->get_preferred_format();
 }
 
@@ -172,7 +148,7 @@ function editors_head_setup() {
     global $CFG;
 
     if (empty($CFG->texteditors)) {
-        $CFG->texteditors = 'tinymce,textarea';
+        $CFG->texteditors = 'tinymce,atto,textarea';
     }
     $active = explode(',', $CFG->texteditors);
 
@@ -236,29 +212,3 @@ abstract class texteditor {
     public function head_setup() {
     }
 }
-
-//=== TO BE DEPRECATED in 2.1 =====================
-
-/**
- * Does the user want and can edit using rich text html editor?
- * @todo Deprecate: eradicate completely, replace with something else in the future
- * @return bool
- */
-function can_use_html_editor() {
-    global $USER;
-
-    $editors = editors_get_enabled();
-    if (count($editors) > 1) {
-        if (empty($USER->htmleditor)) {
-            return false;
-        }
-    }
-
-    foreach ($editors as $editor) {
-        if ($editor->get_preferred_format() == FORMAT_HTML) {
-            return true;
-        }
-    }
-
-    return false;
-}
index 4e94b09..7676edb 100644 (file)
@@ -99,7 +99,7 @@ class MoodleQuickForm_date_selector extends MoodleQuickForm_group {
             }
         }
         // The YUI2 calendar only supports the gregorian calendar type.
-        if (\core_calendar\type_factory::get_calendar_type() === 'gregorian') {
+        if ($calendartype->get_name() === 'gregorian') {
             form_init_date_js();
         }
     }
@@ -126,7 +126,7 @@ class MoodleQuickForm_date_selector extends MoodleQuickForm_group {
         $this->_elements[] = @MoodleQuickForm::createElement('select', 'month', get_string('month', 'form'), $months, $this->getAttributes(), true);
         $this->_elements[] = @MoodleQuickForm::createElement('select', 'year', get_string('year', 'form'), $years, $this->getAttributes(), true);
         // The YUI2 calendar only supports the gregorian calendar type so only display the calendar image if this is being used.
-        if (\core_calendar\type_factory::get_calendar_type() === 'gregorian') {
+        if ($calendartype->get_name() === 'gregorian') {
             $this->_elements[] = @MoodleQuickForm::createElement('image', 'calendar', $OUTPUT->pix_url('i/calendar', 'moodle'),
                 array('title' => get_string('calendar', 'calendar'), 'class' => 'visibleifjs'));
         }
index e3a908e..f80f356 100644 (file)
@@ -101,7 +101,7 @@ class MoodleQuickForm_date_time_selector extends MoodleQuickForm_group {
             }
         }
         // The YUI2 calendar only supports the gregorian calendar type.
-        if (\core_calendar\type_factory::get_calendar_type() === 'gregorian') {
+        if ($calendartype->get_name() === 'gregorian') {
             form_init_date_js();
         }
     }
@@ -141,7 +141,7 @@ class MoodleQuickForm_date_time_selector extends MoodleQuickForm_group {
             $this->_elements[] = @MoodleQuickForm::createElement('select', 'minute', get_string('minute', 'form'), $minutes, $this->getAttributes(), true);
         }
         // The YUI2 calendar only supports the gregorian calendar type so only display the calendar image if this is being used.
-        if (\core_calendar\type_factory::get_calendar_type() === 'gregorian') {
+        if ($calendartype->get_name() === 'gregorian') {
             $this->_elements[] = @MoodleQuickForm::createElement('image', 'calendar', $OUTPUT->pix_url('i/calendar', 'moodle'),
                 array('title' => get_string('calendar', 'calendar'), 'class' => 'visibleifjs'));
         }
index 2e79dcc..adddc3f 100644 (file)
@@ -42,11 +42,8 @@ class MoodleQuickForm_htmleditor extends MoodleQuickForm_textarea{
     /** @var string defines the type of editor */
     var $_type;
 
-    /** @var bool Does the user want and can edit using rich text html editor */
-    var $_canUseHtmlEditor;
-
     /** @var array default options for html editor, which can be overridden */
-    var $_options=array('canUseHtmlEditor'=>'detect','rows'=>10, 'cols'=>45, 'width'=>0,'height'=>0);
+    var $_options=array('rows'=>10, 'cols'=>45, 'width'=>0,'height'=>0);
 
     /**
      * Constructor
@@ -71,16 +68,7 @@ class MoodleQuickForm_htmleditor extends MoodleQuickForm_textarea{
                 }
             }
         }
-        if ($this->_options['canUseHtmlEditor']=='detect'){
-            $this->_options['canUseHtmlEditor']=can_use_html_editor();
-        }
-        if ($this->_options['canUseHtmlEditor']){
-            $this->_type='htmleditor';
-            //$this->_elementTemplateType='wide';
-        }else{
-            $this->_type='textarea';
-        }
-        $this->_canUseHtmlEditor = $this->_options['canUseHtmlEditor'];
+        $this->_type='htmleditor';
 
         editors_head_setup();
     }
@@ -91,16 +79,11 @@ class MoodleQuickForm_htmleditor extends MoodleQuickForm_textarea{
      * @return string
      */
     function toHtml(){
-        //if ($this->_canUseHtmlEditor && !$this->_flagFrozen){
-        //    $script = '';
-        //} else {
-        //    $script='';
-        //}
         if ($this->_flagFrozen) {
             return $this->getFrozenHtml();
         } else {
             return $this->_getTabs() .
-                    print_textarea($this->_canUseHtmlEditor,
+                    print_textarea(true,
                                     $this->_options['rows'],
                                     $this->_options['cols'],
                                     $this->_options['width'],
index 094c76b..72cad75 100644 (file)
@@ -2685,8 +2685,10 @@ function dst_offset_on($time, $strtimezone = null) {
  * @return int
  */
 function find_day_in_month($startday, $weekday, $month, $year) {
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
 
     $daysinmonth = days_in_month($month, $year);
+    $daysinweek = count($calendartype->get_weekdays());
 
     if ($weekday == -1) {
         // Don't care about weekday, so return:
@@ -2696,46 +2698,40 @@ function find_day_in_month($startday, $weekday, $month, $year) {
     }
 
     // From now on we 're looking for a specific weekday.
-
     // Give "end of month" its actual value, since we know it.
     if ($startday == -1) {
         $startday = -1 * $daysinmonth;
     }
 
     // Starting from day $startday, the sign is the direction.
-
     if ($startday < 1) {
-
         $startday = abs($startday);
-        $lastmonthweekday  = strftime('%w', mktime(12, 0, 0, $month, $daysinmonth, $year));
+        $lastmonthweekday = dayofweek($daysinmonth, $month, $year);
 
         // This is the last such weekday of the month.
         $lastinmonth = $daysinmonth + $weekday - $lastmonthweekday;
         if ($lastinmonth > $daysinmonth) {
-            $lastinmonth -= 7;
+            $lastinmonth -= $daysinweek;
         }
 
         // Find the first such weekday <= $startday.
         while ($lastinmonth > $startday) {
-            $lastinmonth -= 7;
+            $lastinmonth -= $daysinweek;
         }
 
         return $lastinmonth;
-
     } else {
-
-        $indexweekday = strftime('%w', mktime(12, 0, 0, $month, $startday, $year));
+        $indexweekday = dayofweek($startday, $month, $year);
 
         $diff = $weekday - $indexweekday;
         if ($diff < 0) {
-            $diff += 7;
+            $diff += $daysinweek;
         }
 
         // This is the first such weekday of the month equal to or after $startday.
         $firstfromindex = $startday + $diff;
 
         return $firstfromindex;
-
     }
 }
 
@@ -2749,7 +2745,8 @@ function find_day_in_month($startday, $weekday, $month, $year) {
  * @return int
  */
 function days_in_month($month, $year) {
-    return intval(date('t', mktime(12, 0, 0, $month, 1, $year)));
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
+    return $calendartype->get_num_days_in_month($year, $month);
 }
 
 /**
@@ -2763,8 +2760,8 @@ function days_in_month($month, $year) {
  * @return int
  */
 function dayofweek($day, $month, $year) {
-    // I wonder if this is any different from strftime('%w', mktime(12, 0, 0, $month, $daysinmonth, $year, 0));.
-    return intval(date('w', mktime(12, 0, 0, $month, $day, $year)));
+    $calendartype = \core_calendar\type_factory::get_calendar_instance();
+    return $calendartype->get_weekday($year, $month, $day);
 }
 
 // USER AUTHENTICATION AND LOGIN.
diff --git a/lib/tests/editorlib_test.php b/lib/tests/editorlib_test.php
new file mode 100644 (file)
index 0000000..989b5c1
--- /dev/null
@@ -0,0 +1,60 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+/**
+ * Tests editors subsystem.
+ *
+ * @package    core_editors
+ * @subpackage phpunit
+ * @copyright  2013 onwards Martin Dougiamas (http://dougiamas.com)
+ * @author     Damyon Wiese
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+defined('MOODLE_INTERNAL') || die();
+
+
+class core_editorslib_testcase extends advanced_testcase {
+
+    /**
+     * Tests the installation of event handlers from file
+     */
+    public function test_get_preferred_editor() {
+
+        // Fake a user agent.
+        $_SERVER['HTTP_USER_AGENT'] = 'Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_6_5; en-US) AppleWebKit/534.10 (KHTML, like Gecko) Chrome/8.0.552.21     5 Safari/534.10';
+
+        $enabled = editors_get_enabled();
+        // Array assignment is always a clone.
+        $editors = $enabled;
+
+        $first = array_shift($enabled);
+
+        // Get the default editor which should be the first in the list.
+        set_user_preference('htmleditor', '');
+        $preferred = editors_get_preferred_editor();
+        $this->assertEquals($first, $preferred);
+
+        foreach ($editors as $key => $editor) {
+            // User has set a preference for a specific editor.
+            set_user_preference('htmleditor', $key);
+            $preferred = editors_get_preferred_editor();
+            $this->assertEquals($editor, $preferred);
+        }
+    }
+
+}
+
index 690281c..5ca2124 100644 (file)
@@ -68,8 +68,6 @@ class assignment_base {
     var $strlastmodified;
     /** @var string */
     var $pagetitle;
-    /** @var bool */
-    var $usehtmleditor;
     /**
      * @todo document this var
      */
index 8888caa..79d0d6a 100644 (file)
@@ -281,15 +281,9 @@ class assignment_online extends assignment_base {
 
     function preprocess_submission(&$submission) {
         if ($this->assignment->var1 && empty($submission->submissioncomment)) {  // comment inline
-            if ($this->usehtmleditor) {
-                // Convert to html, clean & copy student data to teacher
-                $submission->submissioncomment = format_text($submission->data1, $submission->data2);
-                $submission->format = FORMAT_HTML;
-            } else {
-                // Copy student data to teacher
-                $submission->submissioncomment = $submission->data1;
-                $submission->format = $submission->data2;
-            }
+            // Convert to html, clean & copy student data to teacher
+            $submission->submissioncomment = format_text($submission->data1, $submission->data2);
+            $submission->format = FORMAT_HTML;
         }
     }
 
index 3fc7a71..51ad6fc 100644 (file)
@@ -69,11 +69,7 @@ class data_field_textarea extends data_field_base {
             $text = file_prepare_draft_area($draftitemid, $this->context->id, 'mod_data', 'content', $content->id, $options, $text);
         } else {
             $draftitemid = file_get_unused_draft_itemid();
-            if (can_use_html_editor()) {
-                $format = FORMAT_HTML;
-            } else {
-                $format = FORMAT_PLAIN;
-            }
+            $format = FORMAT_HTML;
         }
 
         // get filepicker info
index f17ce6f..0651c4b 100644 (file)
@@ -192,7 +192,7 @@ echo $OUTPUT->box_start('generalbox boxaligncenter boxwidthwide');
 echo '<table cellpadding="4" cellspacing="0" border="0">';
 
 /// Add the HTML editor(s).
-$usehtmleditor = can_use_html_editor() && ($mode != 'csstemplate') && ($mode != 'jstemplate') && !$disableeditor;
+$usehtmleditor = ($mode != 'csstemplate') && ($mode != 'jstemplate') && !$disableeditor;
 if ($mode == 'listtemplate'){
     // Print the list template header.
     echo '<tr>';
@@ -275,15 +275,13 @@ if ($mode != 'csstemplate' and $mode != 'jstemplate') {
 
     echo '</select>';
     echo '<br /><br /><br /><br /><input type="submit" name="defaultform" value="'.get_string('resettemplate','data').'" />';
-    if (can_use_html_editor()) {
-        echo '<br /><br />';
-        if ($usehtmleditor) {
-            $switcheditor = get_string('editordisable', 'data');
-            echo '<input type="submit" name="switcheditor" value="'.s($switcheditor).'" />';
-        } else {
-            $switcheditor = get_string('editorenable', 'data');
-            echo '<input type="submit" name="useeditor" value="'.s($switcheditor).'" />';
-        }
+    echo '<br /><br />';
+    if ($usehtmleditor) {
+        $switcheditor = get_string('editordisable', 'data');
+        echo '<input type="submit" name="switcheditor" value="'.s($switcheditor).'" />';
+    } else {
+        $switcheditor = get_string('editorenable', 'data');
+        echo '<input type="submit" name="useeditor" value="'.s($switcheditor).'" />';
     }
 } else {
     echo '<br /><br /><br /><br /><input type="submit" name="defaultform" value="'.get_string('resettemplate','data').'" />';
index 20425c4..a15b7ac 100644 (file)
@@ -48,7 +48,6 @@ if ($id !== false) {
 $PAGE->set_url($url);
 
 // set up some general variables
-$usehtmleditor = can_use_html_editor();
 
 
 if (($formdata = data_submitted()) AND !confirm_sesskey()) {
index 4a95584..4f32247 100644 (file)
@@ -268,7 +268,6 @@ if (!$students) {
         echo $OUTPUT->container(html_writer::link($allurl, get_string('showall', '', $matchcount)), array(), 'showall');
     }
     if (has_capability('moodle/course:bulkmessaging', $coursecontext)) {
-        $usehtmleditor = can_use_html_editor();
         echo '<div class="buttons"><br />';
         echo '<input type="button" id="checkall" value="'.get_string('selectall').'" /> ';
         echo '<input type="button" id="checknone" value="'.get_string('deselectall').'" /> ';
@@ -279,14 +278,9 @@ if (!$students) {
         echo '<label for="feedback_subject">'.get_string('subject', 'feedback').'&nbsp;</label>';
         echo '<input type="text" id="feedback_subject" size="50" maxlength="255" name="subject" value="'.$subject.'" />';
         echo '</div>';
-        print_textarea($usehtmleditor, 15, 25, 30, 10, "message", $message);
-        if ($usehtmleditor) {
-            print_string('formathtml');
-            echo '<input type="hidden" name="format" value="'.FORMAT_HTML.'" />';
-        } else {
-            echo '<label for="menuformat" class="accesshide">'. get_string('format') .'</label>';
-            choose_from_menu(format_text_menu(), "format", $format, "");
-        }
+        print_textarea(true, 15, 25, 30, 10, "message", $message);
+        print_string('formathtml');
+        echo '<input type="hidden" name="format" value="'.FORMAT_HTML.'" />';
         echo '<br /><div class="buttons">';
         echo '<input type="submit" name="send_message" value="'.get_string('send', 'feedback').'" />';
         echo '</div>';
index 4173576..f7001c5 100644 (file)
@@ -393,7 +393,6 @@ class quiz_grading_report extends quiz_default_report {
         }
 
         // Display the form with one section for each attempt.
-        $usehtmleditor = can_use_html_editor();
         $sesskey = sesskey();
         $qubaidlist = implode(',', $qubaids);
         echo html_writer::start_tag('form', array('method' => 'post',
index 62c01b9..fc94a78 100644 (file)
@@ -61,12 +61,6 @@ if ($userid) {
 
 /// require login to access notes
 require_login($course);
-add_to_log($courseid, 'notes', 'view', 'index.php?course='.$courseid.'&amp;user='.$userid, 'view notes');
-
-if (empty($CFG->enablenotes)) {
-    print_error('notesdisabled', 'notes');
-}
-
 /// output HTML
 if ($course->id == SITEID) {
     $coursecontext = context_system::instance();   // SYSTEM context
@@ -75,6 +69,19 @@ if ($course->id == SITEID) {
 }
 $systemcontext = context_system::instance();   // SYSTEM context
 
+// Trigger event.
+$event = \core\event\notes_viewed::create(array(
+    'courseid' => $courseid,
+    'relateduserid' => $userid,
+    'context' => $coursecontext,
+    'other' => array('content' => 'notes')
+));
+$event->trigger();
+
+if (empty($CFG->enablenotes)) {
+    print_error('notesdisabled', 'notes');
+}
+
 $strnotes = get_string('notes', 'notes');
 if ($userid) {
     $PAGE->set_context(context_user::instance($user->id));
index ce720e3..1dd5241 100644 (file)
@@ -66,7 +66,7 @@ function note_list($courseid=0, $userid=0, $state = '', $author = 0, $order='las
  * Retrieves a note object based on its id.
  *
  * @param int    $note_id id of the note to retrieve
- * @return note object
+ * @return stdClass object
  */
 function note_load($note_id) {
     global $DB;
@@ -79,13 +79,13 @@ function note_load($note_id) {
  * Saves a note object. The note object is passed by reference and its fields (i.e. id)
  * might change during the save.
  *
- * @param note   $note object to save
+ * @param stdClass   $note object to save
  * @return boolean true if the object was saved; false otherwise
  */
 function note_save(&$note) {
     global $USER, $DB;
 
-    // setup & clean fields
+    // Setup & clean fields.
     $note->module       = 'notes';
     $note->lastmodified = time();
     $note->usermodified = $USER->id;
@@ -95,23 +95,40 @@ function note_save(&$note) {
     if (empty($note->publishstate)) {
         $note->publishstate = NOTES_STATE_PUBLIC;
     }
-    // save data
+    // Save data.
     if (empty($note->id)) {
-        // insert new note
+        // Insert new note.
         $note->created = $note->lastmodified;
         $id = $DB->insert_record('post', $note);
         $note = note_load($id);
-        $logurl = new moodle_url('index.php', array('course'=> $note->courseid, 'user'=>$note->userid));
-        $logurl->set_anchor('note-' . $id);
 
-        add_to_log($note->courseid, 'notes', 'add', $logurl, 'add note');
+        // Trigger event.
+        $event = \core\event\note_created::create(array(
+            'objectid' => $note->id,
+            'courseid' => $note->courseid,
+            'relateduserid' => $note->userid,
+            'userid' => $note->usermodified,
+            'context' => context_course::instance($note->courseid),
+            'other' => array('publishstate' => $note->publishstate)
+        ));
+        $event->add_record_snapshot('post', $note);
+        $event->trigger();
     } else {
-        // update old note
+        // Update old note.
         $DB->update_record('post', $note);
         $note = note_load($note->id);
-        $logurl = new moodle_url('index.php', array('course'=> $note->courseid, 'user'=>$note->userid));
-        $logurl->set_anchor('note-' . $note->id);
-        add_to_log($note->courseid, 'notes', 'update', $logurl , 'update note');
+
+        // Trigger event.
+        $event = \core\event\note_updated::create(array(
+            'objectid' => $note->id,
+            'courseid' => $note->courseid,
+            'relateduserid' => $note->userid,
+            'userid' => $note->usermodified,
+            'context' => context_course::instance($note->courseid),
+            'other' => array('publishstate' => $note->publishstate)
+        ));
+        $event->add_record_snapshot('post', $note);
+        $event->trigger();
     }
     unset($note->module);
     return true;
@@ -127,12 +144,23 @@ function note_delete($note) {
     global $DB;
     if (is_int($note)) {
         $note = note_load($note);
-        debugging('Warning: providing note_delete with a note object would improve performance.',DEBUG_DEVELOPER);
+        debugging('Warning: providing note_delete with a note object would improve performance.', DEBUG_DEVELOPER);
     }
-    $logurl = new moodle_url('index.php', array('course'=> $note->courseid, 'user'=>$note->userid));
-    $logurl->set_anchor('note-' . $note->id);
-    add_to_log($note->courseid, 'notes', 'delete', $logurl, 'delete note');
-    return $DB->delete_records('post', array('id'=>$note->id, 'module'=>'notes'));
+    $return = $DB->delete_records('post', array('id' => $note->id, 'module' => 'notes'));
+
+    // Trigger event.
+    $event = \core\event\note_deleted::create(array(
+        'objectid' => $note->id,
+        'courseid' => $note->courseid,
+        'relateduserid' => $note->userid,
+        'userid' => $note->usermodified,
+        'context' => context_course::instance($note->courseid),
+        'other' => array('publishstate' => $note->publishstate)
+    ));
+    $event->add_record_snapshot('post', $note);
+    $event->trigger();
+
+    return $return;
 }
 
 /**
diff --git a/notes/tests/events_test.php b/notes/tests/events_test.php
new file mode 100644 (file)
index 0000000..c9fed24
--- /dev/null
@@ -0,0 +1,146 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+/**
+ * Tests for notes events.
+ *
+ * @package    core_notes
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later.
+ */
+
+defined('MOODLE_INTERNAL') || die();
+
+/**
+ * Class core_notes_events_testcase
+ *
+ * Class for tests related to notes events.
+ *
+ * @package    core_notes
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later.
+ */
+class core_notes_events_testcase extends advanced_testcase {
+
+    /** @var  stdClass A note object. */
+    private $eventnote;
+
+    public function setUp() {
+
+        $this->resetAfterTest();
+        $this->setAdminUser();
+
+        $course = $this->getDataGenerator()->create_course();
+        $user = $this->getDataGenerator()->create_user();
+        $gen = $this->getDataGenerator()->get_plugin_generator('core_notes');
+        $this->eventnote = $gen->create_instance(array('courseid' => $course->id, 'userid' => $user->id));
+
+    }
+
+    /**
+     * Tests for event note_deleted.
+     */
+    public function test_note_deleted_event() {
+        // Delete a note.
+        $sink = $this->redirectEvents();
+        note_delete($this->eventnote);
+        $events = $sink->get_events();
+        $event = array_pop($events); // Delete note event.
+        $sink->close();
+
+        // Validate event data.
+        $this->assertInstanceOf('\core\event\note_deleted', $event);
+        $this->assertEquals($this->eventnote->id, $event->objectid);
+        $this->assertEquals($this->eventnote->usermodified, $event->userid);
+        $this->assertEquals($this->eventnote->userid, $event->relateduserid);
+        $this->assertEquals('post', $event->objecttable);
+        $this->assertEquals(null, $event->get_url());
+        $this->assertEquals($this->eventnote, $event->get_record_snapshot('post', $event->objectid));
+        $this->assertEquals(NOTES_STATE_SITE, $event->other['publishstate']);
+
+        // Test legacy data.
+        $logurl = new \moodle_url('index.php',
+                array('course' => $this->eventnote->courseid, 'user' => $this->eventnote->userid));
+        $logurl->set_anchor('note-' . $this->eventnote->id);
+        $arr = array($this->eventnote->courseid, 'notes', 'delete', $logurl, 'delete note');
+        $this->assertEventLegacyLogData($arr, $event);
+    }
+
+    /**
+     * Tests for event note_created.
+     */
+    public function test_note_created_event() {
+
+        // Delete a note.
+        $sink = $this->redirectEvents();
+        $note = clone $this->eventnote;
+        unset($note->id);
+        note_save($note);
+        $events = $sink->get_events();
+        $event = array_pop($events); // Delete note event.
+        $sink->close();
+
+        // Validate event data.
+        $this->assertInstanceOf('\core\event\note_created', $event);
+        $this->assertEquals($note->id, $event->objectid);
+        $this->assertEquals($note->usermodified, $event->userid);
+        $this->assertEquals($note->userid, $event->relateduserid);
+        $this->assertEquals('post', $event->objecttable);
+        $this->assertEquals($note, $event->get_record_snapshot('post', $event->objectid));
+        $this->assertEquals(NOTES_STATE_SITE, $event->other['publishstate']);
+
+        // Test legacy data.
+        $logurl = new \moodle_url('index.php',
+            array('course' => $note->courseid, 'user' => $note->userid));
+        $logurl->set_anchor('note-' . $note->id);
+        $arr = array($note->courseid, 'notes', 'add', $logurl, 'add note');
+        $this->assertEventLegacyLogData($arr, $event);
+
+    }
+
+    /**
+     * Tests for event note_updated.
+     */
+    public function test_note_updated_event() {
+
+        // Delete a note.
+        $sink = $this->redirectEvents();
+        $note = clone $this->eventnote;
+        $note->publishstate = NOTES_STATE_DRAFT;
+        note_save($note);
+        $events = $sink->get_events();
+        $event = array_pop($events); // Delete note event.
+        $sink->close();
+
+        // Validate event data.
+        $this->assertInstanceOf('\core\event\note_updated', $event);
+        $this->assertEquals($note->id, $event->objectid);
+        $this->assertEquals($note->usermodified, $event->userid);
+        $this->assertEquals($note->userid, $event->relateduserid);
+        $this->assertEquals('post', $event->objecttable);
+        $this->assertEquals($note, $event->get_record_snapshot('post', $event->objectid));
+        $this->assertEquals(NOTES_STATE_DRAFT, $event->other['publishstate']);
+
+        // Test legacy data.
+        $logurl = new \moodle_url('index.php',
+            array('course' => $note->courseid, 'user' => $note->userid));
+        $logurl->set_anchor('note-' . $note->id);
+        $arr = array($note->courseid, 'notes', 'update', $logurl, 'update note');
+        $this->assertEventLegacyLogData($arr, $event);
+
+    }
+}
+
diff --git a/notes/tests/generator/lib.php b/notes/tests/generator/lib.php
new file mode 100644 (file)
index 0000000..a280a71
--- /dev/null
@@ -0,0 +1,127 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+/**
+ * core_notes data generator.
+ *
+ * @package    core_notes
+ * @category   test
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+defined('MOODLE_INTERNAL') || die();
+
+/**
+ * core_notes data generator class.
+ *
+ * @package    core_notes
+ * @category   test
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class core_notes_generator extends component_generator_base {
+
+    /**
+     * @var number of created instances
+     */
+    protected $instancecount = 0;
+
+    /**
+     * To be called from data reset code only,
+     * do not use in tests.
+     * @return void
+     */
+    public function reset() {
+        $this->instancecount = 0;
+    }
+
+    /**
+     * Create a new note.
+     *
+     * @param array|stdClass $record
+     * @throws coding_exception
+     * @return stdClass activity record with extra cmid field
+     */
+    public function create_instance($record = null) {
+        global $CFG, $USER;
+        require_once("$CFG->dirroot/notes/lib.php");
+
+        $this->instancecount++;
+        $i = $this->instancecount;
+        $record = (object)(array)$record;
+
+        if (empty($record->courseid)) {
+            throw new coding_exception('Module generator requires $record->courseid.');
+        }
+        if (empty($record->userid)) {
+            throw new coding_exception('Module generator requires $record->userid.');
+        }
+        if (!isset($record->module)) {
+            $record->module = 'notes';
+        }
+        if (!isset($record->groupid)) {
+            $record->groupid = 0;
+        }
+        if (!isset($record->moduleid)) {
+            $record->moduleid = 0;
+        }
+        if (!isset($record->coursemoduleid)) {
+            $record->coursemoduleid = 0;
+        }
+        if (!isset($record->subject)) {
+            $record->subject = '';
+        }
+        if (!isset($record->summary)) {
+            $record->summary = null;
+        }
+        if (!isset($record->content)) {
+            $record->content = "This is test generated note - $i .";
+        }
+        if (!isset($record->uniquehash)) {
+            $record->uniquehash = '';
+        }
+        if (!isset($record->rating)) {
+            $record->rating = 0;
+        }
+        if (!isset($record->format)) {
+            $record->format = FORMAT_PLAIN;
+        }
+        if (!isset($record->summaryformat)) {
+            $record->summaryformat = FORMAT_MOODLE;
+        }
+        if (!isset($record->attachment)) {
+            $record->attachment = null;
+        }
+        if (!isset($record->publishstate)) {
+            $record->publishstate = NOTES_STATE_SITE;
+        }
+        if (!isset($record->lastmodified)) {
+            $record->lastmodified = time();
+        }
+        if (!isset($record->created)) {
+            $record->created = time();
+        }
+        if (!isset($record->usermodified)) {
+            $record->usermodified = $USER->id;
+        }
+
+        note_save($record);
+        return $record;
+    }
+
+}
+
diff --git a/notes/tests/generator_test.php b/notes/tests/generator_test.php
new file mode 100644 (file)
index 0000000..cf9df9a
--- /dev/null
@@ -0,0 +1,82 @@
+<?php
+// This file is part of Moodle - http://moodle.org/
+//
+// Moodle is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// Moodle is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU General Public License for more details.
+//
+// You should have received a copy of the GNU General Public License
+// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
+
+/**
+ * Generator tests.
+ *
+ * @package    core_notes
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+defined('MOODLE_INTERNAL') || die();
+
+/**
+ * Generator tests class.
+ *
+ * @package    core_notes
+ * @copyright  2013 Ankit Agarwal
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class core_notes_generator_testcase extends advanced_testcase {
+
+    /** Test create_instance method */
+    public function test_create_instance() {
+        global $DB;
+        $this->resetAfterTest();
+        $this->setAdminUser();
+
+        $course = $this->getDataGenerator()->create_course();
+        $user = $this->getDataGenerator()->create_user();
+        $gen = $this->getDataGenerator()->get_plugin_generator('core_notes');
+
+        $this->assertFalse($DB->record_exists('post', array('courseid' => $course->id)));
+        $note = $gen->create_instance(array('courseid' => $course->id, 'userid' => $user->id));
+        $this->assertEquals(1, $DB->count_records('post', array('courseid' => $course->id, 'userid' => $user->id)));
+        $this->assertTrue($DB->record_exists('post', array('id' => $note->id)));
+
+        $params = array('courseid' => $course->id, 'userid' => $user->id, 'publishstate' => NOTES_STATE_DRAFT);
+        $note = $gen->create_instance($params);
+        $this->assertEquals(2, $DB->count_records('post', array('courseid' => $course->id, 'userid' => $user->id)));
+        $this->assertEquals(NOTES_STATE_DRAFT, $DB->get_field_select('post', 'publishstate', 'id = :id',
+                array('id' => $note->id)));
+    }
+
+    /** Test Exceptions thrown by create_instance method */
+    public function test_create_instance_exceptions() {
+        $this->resetAfterTest();
+
+        $gen = $this->getDataGenerator()->get_plugin_generator('core_notes');
+
+        // Test not setting userid.
+        try {
+            $gen->create_instance(array('courseid' => 2));
+            $this->fail('A note should not be allowed to be created without associcated userid');
+        } catch (coding_exception $e) {
+            $this->assertContains('Module generator requires $record->userid', $e->getMessage());
+        }
+
+        // Test not setting courseid.
+        try {
+            $gen->create_instance(array('userid' => 2));
+            $this->fail('A note should not be allowed to be created without associcated courseid');
+        } catch (coding_exception $e) {
+            $this->assertContains('Module generator requires $record->courseid', $e->getMessage());
+        }
+    }
+
+}
+
index 2e2b74a..433d969 100644 (file)
@@ -60,15 +60,13 @@ $tagname = tag_display_name($tag);
 // set the relatedtags field of the $tag object that will be passed to the form
 $tag->relatedtags = tag_get_related_tags_csv(tag_get_related_tags($tag->id, TAG_RELATED_MANUAL), TAG_RETURN_TEXT);
 
-if (can_use_html_editor()) {
-    $options = new stdClass();
-    $options->smiley = false;
-    $options->filter = false;
-
-    // convert and remove any XSS
-    $tag->description       = format_text($tag->description, $tag->descriptionformat, $options);
-    $tag->descriptionformat = FORMAT_HTML;
-}
+$options = new stdClass();
+$options->smiley = false;
+$options->filter = false;
+
+// convert and remove any XSS
+$tag->description       = format_text($tag->description, $tag->descriptionformat, $options);
+$tag->descriptionformat = FORMAT_HTML;
 
 $errorstring = '';
 
index 158b01d..210dd86 100644 (file)
@@ -226,15 +226,21 @@ function useredit_shared_definition(&$mform, $editoroptions = null, $filemanager
 
     $editors = editors_get_enabled();
     if (count($editors) > 1) {
-        $choices = array();
-        $choices['0'] = get_string('texteditor');
-        $choices['1'] = get_string('htmleditor');
-        $mform->addElement('select', 'htmleditor', get_string('textediting'), $choices);
-        $mform->setDefault('htmleditor', 1);
+        $choices = array('' => get_string('defaulteditor'));
+        $firsteditor = '';
+        foreach (array_keys($editors) as $editor) {
+            if (!$firsteditor) {
+                $firsteditor = $editor;
+            }
+            $choices[$editor] = get_string('pluginname', 'editor_' . $editor);
+        }
+        $mform->addElement('select', 'preference_htmleditor', get_string('textediting'), $choices);
+        $mform->setDefault('preference_htmleditor', '');
     } else {
-        $mform->addElement('hidden', 'htmleditor');
-        $mform->setDefault('htmleditor', 1);
-        $mform->setType('htmleditor', PARAM_INT);
+        // Empty string means use the first chosen text editor.
+        $mform->addElement('hidden', 'preference_htmleditor');
+        $mform->setDefault('preference_htmleditor', '');
+        $mform->setType('preference_htmleditor', PARAM_PLUGIN);
     }
 
     $mform->addElement('text', 'city', get_string('city'), 'maxlength="120" size="21"');
index 1982d2f..98cfbc3 100644 (file)
@@ -10,7 +10,7 @@
      <?php print_string("messagebody"); ?>:
      </b></td>
     <td align="left" rowspan="2">
-    <?php print_textarea($usehtmleditor, 25, 65, 630, 400, "messagebody", $messagebody); ?>
+    <?php print_textarea(true, 25, 65, 630, 400, "messagebody", $messagebody); ?>
     </td>
 </tr>
 
     <td align="right"><label for="menuformat"><b><?php print_string("formattexttype"); ?>:</b></label></td>
     <td>
     <?php
-        if ($usehtmleditor) {   /// Trying this out for a while
-            print_string('formathtml');
-            echo '<input type="hidden" name="format" value="'.FORMAT_HTML.'" />';
-        } else {
-            choose_from_menu(format_text_menu(), "format", $format, "");
-        }
+        print_string('formathtml');
+        echo '<input type="hidden" name="format" value="'.FORMAT_HTML.'" />';
      ?>
     </td>
 </tr>
index a9d6023..877adb8 100644 (file)
@@ -174,7 +174,6 @@ if ((!empty($send) || !empty($preview) || !empty($edit)) && (empty($messagebody)
 
 if (count($SESSION->emailto[$id])) {
     require_sesskey();
-    $usehtmleditor = can_use_html_editor();
     require("message.html");
 }
 
index 9ab9ca2..6d440ae 100644 (file)
@@ -29,7 +29,7 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$version  = 2013100800.01;              // YYYYMMDD      = weekly release date of this DEV branch.
+$version  = 2013100800.02;              // YYYYMMDD      = weekly release date of this DEV branch.
                                         //         RR    = release increments - 00 in DEV branches.
                                         //           .XX = incremental changes.