diff --git a/backup/moodle2/backup_moodleoverflow_activity_task.class.php b/backup/moodle2/backup_moodleoverflow_activity_task.class.php
index e55bb48c44..240538b00e 100644
--- a/backup/moodle2/backup_moodleoverflow_activity_task.class.php
+++ b/backup/moodle2/backup_moodleoverflow_activity_task.class.php
@@ -63,27 +63,27 @@ public static function encode_content_links($content) {
$base = preg_quote($CFG->wwwroot, '/');
// Link to the list of moodleoverflows.
- $search = '/(' . $base . '\/mod\/moodleoverflow\/index.php\?id\=)([0-9]+)/';
+ $search = '/(' . $base . '\/mod\/moodleoverflow\/index.php\?id\=)([0-9]+)/';
$content = preg_replace($search, '$@MOODLEOVERFLOWINDEX*$2@$', $content);
// Link to moodleoverflow view by moduleid.
- $search = '/(' . $base . '\/mod\/moodleoverflow\/view.php\?id\=)([0-9]+)/';
+ $search = '/(' . $base . '\/mod\/moodleoverflow\/view.php\?id\=)([0-9]+)/';
$content = preg_replace($search, '$@MOODLEOVERFLOWVIEWBYID*$2@$', $content);
// Link to moodleoverflow view by forumid.
- $search = "/(" . $base . "\/mod\/forum\/view.php\?f\=)([0-9]+)/";
+ $search = "/(" . $base . "\/mod\/forum\/view.php\?f\=)([0-9]+)/";
$content = preg_replace($search, '$@MOODLEOVERFLOWVIEWBYF*$2@$', $content);
// Link to moodleoverflow discussion with parent syntax.
- $search = "/(" . $base . "\/mod\/forum\/discuss.php\?d\=)([0-9]+)(?:\&|\&)parent\=([0-9]+)/";
+ $search = "/(" . $base . "\/mod\/forum\/discuss.php\?d\=)([0-9]+)(?:\&|\&)parent\=([0-9]+)/";
$content = preg_replace($search, '$@MOODLEOVERFLOWDISCUSSIONVIEWPARENT*$2*$3@$', $content);
// Link to moodleoverflow discussion with relative syntax.
- $search = "/(" . $base . "\/mod\/forum\/discuss.php\?d\=)([0-9]+)\#([0-9]+)/";
+ $search = "/(" . $base . "\/mod\/forum\/discuss.php\?d\=)([0-9]+)\#([0-9]+)/";
$content = preg_replace($search, '$@MOODLEOVERFLOWDISCUSSIONVIEWINSIDE*$2*$3@$', $content);
// Link to moodleoverflow discussion by discussionid.
- $search = "/(" . $base . "\/mod\/forum\/discuss.php\?d\=)([0-9]+)/";
+ $search = "/(" . $base . "\/mod\/forum\/discuss.php\?d\=)([0-9]+)/";
$content = preg_replace($search, '$@MOODLEOVERFLOWDISCUSSIONVIEW*$2@$', $content);
return $content;
diff --git a/backup/moodle2/backup_moodleoverflow_stepslib.php b/backup/moodle2/backup_moodleoverflow_stepslib.php
index cbc0aa1c5e..ed5d83f729 100644
--- a/backup/moodle2/backup_moodleoverflow_stepslib.php
+++ b/backup/moodle2/backup_moodleoverflow_stepslib.php
@@ -51,7 +51,7 @@ protected function define_structure() {
// Define each element separated.
$discussions = new backup_nested_element('discussions');
- $discussion = new backup_nested_element('discussion', array('id'), array(
+ $discussion = new backup_nested_element('discussion', array('id'), array(
'name', 'firstpost', 'userid', 'timemodified', 'usermodified', 'timestart'));
$posts = new backup_nested_element('posts');
diff --git a/backup/moodle2/restore_moodleoverflow_activity_task.class.php b/backup/moodle2/restore_moodleoverflow_activity_task.class.php
index 742719c038..a02b4483a5 100644
--- a/backup/moodle2/restore_moodleoverflow_activity_task.class.php
+++ b/backup/moodle2/restore_moodleoverflow_activity_task.class.php
@@ -95,9 +95,9 @@ public static function define_decode_rules() {
/**
* Define the restore log rules that will be applied
- * by the {@link restore_logs_processor} when restoring
+ * by the {restore_logs_processor} when restoring
* moodleoverflow logs. It must return one array
- * of {@link restore_log_rule} objects
+ * of { restore_log_rule} objects
*/
public static function define_restore_log_rules() {
$rules = array();
@@ -151,9 +151,9 @@ public static function define_restore_log_rules() {
/**
* Define the restore log rules that will be applied
- * by the {@link restore_logs_processor} when restoring
+ * by the { restore_logs_processor} when restoring
* course logs. It must return one array
- * of {@link restore_log_rule} objects
+ * of { restore_log_rule} objects
*
* Note this rules are applied when restoring course logs
* by the restore final task, but are defined here at
diff --git a/backup/moodle2/restore_moodleoverflow_stepslib.php b/backup/moodle2/restore_moodleoverflow_stepslib.php
index a1310ddedd..fabc49a080 100644
--- a/backup/moodle2/restore_moodleoverflow_stepslib.php
+++ b/backup/moodle2/restore_moodleoverflow_stepslib.php
@@ -36,11 +36,11 @@ class restore_moodleoverflow_activity_structure_step extends restore_activity_st
/**
* Defines structure of path elements to be processed during the restore.
*
- * @return array of {@link restore_path_element}
+ * @return array of {restore_path_element}
*/
protected function define_structure() {
- $paths = array();
+ $paths = array();
$userinfo = $this->get_setting_value('userinfo');
$paths[] = new restore_path_element('moodleoverflow', '/activity/moodleoverflow');
@@ -73,7 +73,7 @@ protected function define_structure() {
protected function process_moodleoverflow($data) {
global $DB;
- $data = (object) $data;
+ $data = (object) $data;
$data->course = $this->get_courseid();
if (empty($data->timecreated)) {
@@ -99,15 +99,15 @@ protected function process_moodleoverflow($data) {
protected function process_moodleoverflow_discussion($data) {
global $DB;
- $data = (object) $data;
- $oldid = $data->id;
+ $data = (object) $data;
+ $oldid = $data->id;
$data->course = $this->get_courseid();
$data->moodleoverflow = $this->get_new_parentid('moodleoverflow');
- $data->timemodified = $this->apply_date_offset($data->timemodified);
- $data->timestart = $this->apply_date_offset($data->timestart);
- $data->userid = $this->get_mappingid('user', $data->userid);
- $data->usermodified = $this->get_mappingid('user', $data->usermodified);
+ $data->timemodified = $this->apply_date_offset($data->timemodified);
+ $data->timestart = $this->apply_date_offset($data->timestart);
+ $data->userid = $this->get_mappingid('user', $data->userid);
+ $data->usermodified = $this->get_mappingid('user', $data->usermodified);
$newitemid = $DB->insert_record('moodleoverflow_discussions', $data);
$this->set_mapping('moodleoverflow_discussion', $oldid, $newitemid);
@@ -121,13 +121,13 @@ protected function process_moodleoverflow_discussion($data) {
protected function process_moodleoverflow_post($data) {
global $DB;
- $data = (object) $data;
+ $data = (object) $data;
$oldid = $data->id;
$data->discussion = $this->get_new_parentid('moodleoverflow_discussion');
- $data->created = $this->apply_date_offset($data->created);
- $data->modified = $this->apply_date_offset($data->modified);
- $data->userid = $this->get_mappingid('user', $data->userid);
+ $data->created = $this->apply_date_offset($data->created);
+ $data->modified = $this->apply_date_offset($data->modified);
+ $data->userid = $this->get_mappingid('user', $data->userid);
// If post has parent, map it (it has been already restored).
if (!empty($data->parent)) {
$data->parent = $this->get_mappingid('moodleoverflow_post', $data->parent);
@@ -150,12 +150,12 @@ protected function process_moodleoverflow_post($data) {
protected function process_moodleoverflow_rating($data) {
global $DB;
- $data = (object) $data;
+ $data = (object) $data;
$oldid = $data->id;
- $data->userid = $this->get_mappingid('user', $data->userid);
- $data->postid = $this->get_new_parentid('moodleoverflow_post');
- $data->discussionid = $this->get_new_parentid('moodleoverflow_discussion');
+ $data->userid = $this->get_mappingid('user', $data->userid);
+ $data->postid = $this->get_new_parentid('moodleoverflow_post');
+ $data->discussionid = $this->get_new_parentid('moodleoverflow_discussion');
$data->moodleoverflowid = $this->get_new_parentid('moodleoverflow');
$newitemid = $DB->insert_record('moodleoverflow_ratings', $data);
@@ -170,11 +170,11 @@ protected function process_moodleoverflow_rating($data) {
protected function process_moodleoverflow_subscription($data) {
global $DB;
- $data = (object) $data;
+ $data = (object) $data;
$oldid = $data->id;
$data->moodleoverflow = $this->get_new_parentid('moodleoverflow');
- $data->userid = $this->get_mappingid('user', $data->userid);
+ $data->userid = $this->get_mappingid('user', $data->userid);
$newitemid = $DB->insert_record('moodleoverflow_subscriptions', $data);
$this->set_mapping('moodleoverflow_subscription', $oldid, $newitemid, true);
@@ -189,12 +189,12 @@ protected function process_moodleoverflow_subscription($data) {
protected function process_moodleoverflow_discuss_sub($data) {
global $DB;
- $data = (object) $data;
+ $data = (object) $data;
$oldid = $data->id;
- $data->discussion = $this->get_new_parentid('moodleoverflow_discussion');
+ $data->discussion = $this->get_new_parentid('moodleoverflow_discussion');
$data->moodleoverflow = $this->get_new_parentid('moodleoverflow');
- $data->userid = $this->get_mappingid('user', $data->userid);
+ $data->userid = $this->get_mappingid('user', $data->userid);
$newitemid = $DB->insert_record('moodleoverflow_discuss_subs', $data);
$this->set_mapping('moodleoverflow_discuss_sub', $oldid, $newitemid, true);
@@ -211,9 +211,9 @@ protected function process_moodleoverflow_read($data) {
$data = (object) $data;
$data->moodleoverflowid = $this->get_new_parentid('moodleoverflow');
- $data->discussionid = $this->get_mappingid('moodleoverflow_discussion', $data->discussionid);
- $data->postid = $this->get_mappingid('moodleoverflow_post', $data->postid);
- $data->userid = $this->get_mappingid('user', $data->userid);
+ $data->discussionid = $this->get_mappingid('moodleoverflow_discussion', $data->discussionid);
+ $data->postid = $this->get_mappingid('moodleoverflow_post', $data->postid);
+ $data->userid = $this->get_mappingid('user', $data->userid);
$DB->insert_record('moodleoverflow_read', $data);
}
@@ -229,7 +229,7 @@ protected function process_moodleoverflow_track($data) {
$data = (object) $data;
$data->moodleoverflowid = $this->get_new_parentid('moodleoverflow');
- $data->userid = $this->get_mappingid('user', $data->userid);
+ $data->userid = $this->get_mappingid('user', $data->userid);
$DB->insert_record('moodleoverflow_tracking', $data);
}
diff --git a/classes/event/discussion_created.php b/classes/event/discussion_created.php
index 3dcb9654bb..fd59baee36 100644
--- a/classes/event/discussion_created.php
+++ b/classes/event/discussion_created.php
@@ -44,8 +44,8 @@ class discussion_created extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'c';
- $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
+ $this->data['crud'] = 'c';
+ $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
$this->data['objecttable'] = 'moodleoverflow_discussions';
}
diff --git a/classes/event/discussion_deleted.php b/classes/event/discussion_deleted.php
index cb5abcbb31..d0abdfb380 100644
--- a/classes/event/discussion_deleted.php
+++ b/classes/event/discussion_deleted.php
@@ -45,8 +45,8 @@ class discussion_deleted extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'd';
- $this->data['edulevel'] = self::LEVEL_OTHER;
+ $this->data['crud'] = 'd';
+ $this->data['edulevel'] = self::LEVEL_OTHER;
$this->data['objecttable'] = 'moodleoverflow_discussions';
}
diff --git a/classes/event/discussion_subscription_created.php b/classes/event/discussion_subscription_created.php
index a926fbef13..5dfd3c7bea 100644
--- a/classes/event/discussion_subscription_created.php
+++ b/classes/event/discussion_subscription_created.php
@@ -45,8 +45,8 @@ class discussion_subscription_created extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'c';
- $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
+ $this->data['crud'] = 'c';
+ $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
$this->data['objecttable'] = 'moodleoverflow_discuss_subs';
}
diff --git a/classes/event/discussion_subscription_deleted.php b/classes/event/discussion_subscription_deleted.php
index 43dd4ee203..1644724a7d 100644
--- a/classes/event/discussion_subscription_deleted.php
+++ b/classes/event/discussion_subscription_deleted.php
@@ -45,8 +45,8 @@ class discussion_subscription_deleted extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'd';
- $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
+ $this->data['crud'] = 'd';
+ $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
$this->data['objecttable'] = 'moodleoverflow_discuss_subs';
}
diff --git a/classes/event/discussion_viewed.php b/classes/event/discussion_viewed.php
index 14f7ad6a8b..4f5996aee9 100644
--- a/classes/event/discussion_viewed.php
+++ b/classes/event/discussion_viewed.php
@@ -39,8 +39,8 @@ class discussion_viewed extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'r';
- $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
+ $this->data['crud'] = 'r';
+ $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
$this->data['objecttable'] = 'moodleoverflow_discussions';
}
diff --git a/classes/event/post_created.php b/classes/event/post_created.php
index aafabdfc9b..e001ab415c 100644
--- a/classes/event/post_created.php
+++ b/classes/event/post_created.php
@@ -45,8 +45,8 @@ class post_created extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'c';
- $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
+ $this->data['crud'] = 'c';
+ $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
$this->data['objecttable'] = 'moodleoverflow_posts';
}
diff --git a/classes/event/post_deleted.php b/classes/event/post_deleted.php
index 236f2e461f..9f3ab2bb62 100644
--- a/classes/event/post_deleted.php
+++ b/classes/event/post_deleted.php
@@ -45,8 +45,8 @@ class post_deleted extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'd';
- $this->data['edulevel'] = self::LEVEL_OTHER;
+ $this->data['crud'] = 'd';
+ $this->data['edulevel'] = self::LEVEL_OTHER;
$this->data['objecttable'] = 'moodleoverflow_posts';
}
diff --git a/classes/event/post_updated.php b/classes/event/post_updated.php
index 717d21f22e..2ce7771928 100644
--- a/classes/event/post_updated.php
+++ b/classes/event/post_updated.php
@@ -38,8 +38,8 @@ class post_updated extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'u';
- $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
+ $this->data['crud'] = 'u';
+ $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
$this->data['objecttable'] = 'moodleoverflow_posts';
}
diff --git a/classes/event/rating_created.php b/classes/event/rating_created.php
index f2ea3888f2..d05a2c8314 100644
--- a/classes/event/rating_created.php
+++ b/classes/event/rating_created.php
@@ -39,8 +39,8 @@ class rating_created extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'c';
- $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
+ $this->data['crud'] = 'c';
+ $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
$this->data['objecttable'] = 'moodleoverflow_ratings';
}
diff --git a/classes/event/rating_deleted.php b/classes/event/rating_deleted.php
index 7ced3325a1..b20e75df65 100644
--- a/classes/event/rating_deleted.php
+++ b/classes/event/rating_deleted.php
@@ -39,8 +39,8 @@ class rating_deleted extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'd';
- $this->data['edulevel'] = self::LEVEL_OTHER;
+ $this->data['crud'] = 'd';
+ $this->data['edulevel'] = self::LEVEL_OTHER;
$this->data['objecttable'] = 'moodleoverflow_ratings';
}
diff --git a/classes/event/rating_updated.php b/classes/event/rating_updated.php
index ec44c90ca7..5fd74d66b7 100644
--- a/classes/event/rating_updated.php
+++ b/classes/event/rating_updated.php
@@ -39,8 +39,8 @@ class rating_updated extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'u';
- $this->data['edulevel'] = self::LEVEL_OTHER;
+ $this->data['crud'] = 'u';
+ $this->data['edulevel'] = self::LEVEL_OTHER;
$this->data['objecttable'] = 'moodleoverflow_ratings';
}
diff --git a/classes/event/readtracking_disabled.php b/classes/event/readtracking_disabled.php
index cc9b791d9a..3ff1a02f12 100644
--- a/classes/event/readtracking_disabled.php
+++ b/classes/event/readtracking_disabled.php
@@ -45,7 +45,7 @@ class readtracking_disabled extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'd';
+ $this->data['crud'] = 'd';
$this->data['edulevel'] = self::LEVEL_OTHER;
}
diff --git a/classes/event/readtracking_enabled.php b/classes/event/readtracking_enabled.php
index fe86d762ea..c6d7f146d2 100644
--- a/classes/event/readtracking_enabled.php
+++ b/classes/event/readtracking_enabled.php
@@ -45,7 +45,7 @@ class readtracking_enabled extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'c';
+ $this->data['crud'] = 'c';
$this->data['edulevel'] = self::LEVEL_OTHER;
}
diff --git a/classes/event/subscription_created.php b/classes/event/subscription_created.php
index 0b5fdfe44c..ec858da3bf 100644
--- a/classes/event/subscription_created.php
+++ b/classes/event/subscription_created.php
@@ -44,8 +44,8 @@ class subscription_created extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'c';
- $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
+ $this->data['crud'] = 'c';
+ $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
$this->data['objecttable'] = 'moodleoverflow_subscriptions';
}
diff --git a/classes/event/subscription_deleted.php b/classes/event/subscription_deleted.php
index 8730daf091..8544e75577 100644
--- a/classes/event/subscription_deleted.php
+++ b/classes/event/subscription_deleted.php
@@ -44,8 +44,8 @@ class subscription_deleted extends \core\event\base {
* @return void
*/
protected function init() {
- $this->data['crud'] = 'd';
- $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
+ $this->data['crud'] = 'd';
+ $this->data['edulevel'] = self::LEVEL_PARTICIPATING;
$this->data['objecttable'] = 'forum_subscriptions';
}
diff --git a/classes/observer.php b/classes/observer.php
index cfa96ebcb3..ade917acec 100644
--- a/classes/observer.php
+++ b/classes/observer.php
@@ -87,13 +87,13 @@ public static function role_assigned(\core\event\role_assigned $event) {
$userid = $event->relateduserid;
// Retrieve all moodleoverflows in this course.
- $sql = "SELECT m.id, m.course as course, cm.id AS cmid, m.forcesubscribe
+ $sql = "SELECT m.id, m.course as course, cm.id AS cmid, m.forcesubscribe
FROM {moodleoverflow} m
JOIN {course_modules} cm ON (cm.instance = m.id)
JOIN {modules} mo ON (mo.id = cm.module)
LEFT JOIN {moodleoverflow_subscriptions} ms ON (ms.moodleoverflow = m.id AND ms.userid = :userid)
WHERE m.course = :courseid AND m.forcesubscribe = :initial AND mo.name = 'moodleoverflow' AND ms.id IS NULL";
- $params = array('courseid' => $context->instanceid,
+ $params = array('courseid' => $context->instanceid,
'userid' => $userid,
'initial' => MOODLEOVERFLOW_INITIALSUBSCRIBE);
$moodleoverflows = $DB->get_records_sql($sql, $params);
diff --git a/classes/output/email/renderer.php b/classes/output/email/renderer.php
index 42a855a80a..fdf8ef3f75 100644
--- a/classes/output/email/renderer.php
+++ b/classes/output/email/renderer.php
@@ -63,7 +63,7 @@ public function format_message_text($cm, $post) {
'mod_moodleoverflow', 'post', $post->id);
// Initiate some options.
- $options = new \stdClass();
+ $options = new \stdClass();
$options->para = true;
// Return the message in html.
diff --git a/classes/output/moodleoverflow_email.php b/classes/output/moodleoverflow_email.php
index 66212baa2f..fc83e1cf5c 100644
--- a/classes/output/moodleoverflow_email.php
+++ b/classes/output/moodleoverflow_email.php
@@ -119,14 +119,14 @@ class moodleoverflow_email implements \renderable, \templatable {
* @param bool $canreply whether the user can reply to the post
*/
public function __construct($course, $cm, $moodleoverflow, $discussion, $post, $author, $recipient, $canreply) {
- $this->course = $course;
- $this->cm = $cm;
+ $this->course = $course;
+ $this->cm = $cm;
$this->moodleoverflow = $moodleoverflow;
- $this->discussion = $discussion;
- $this->post = $post;
- $this->author = $author;
- $this->userto = $recipient;
- $this->canreply = $canreply;
+ $this->discussion = $discussion;
+ $this->post = $post;
+ $this->author = $author;
+ $this->userto = $recipient;
+ $this->canreply = $canreply;
}
/**
@@ -155,31 +155,31 @@ public function export_for_template(\renderer_base $renderer, $plaintext = false
protected function export_for_template_text(\mod_moodleoverflow_renderer $renderer) {
return array(
- 'id' => html_entity_decode($this->post->id),
- 'coursename' => html_entity_decode($this->get_coursename()),
- 'courselink' => html_entity_decode($this->get_courselink()),
- 'moodleoverflowname' => html_entity_decode($this->get_moodleoverflowname()),
- 'showdiscussionname' => html_entity_decode($this->has_showdiscussionname()),
- 'discussionname' => html_entity_decode($this->get_discussionname()),
- 'subject' => html_entity_decode($this->get_subject()),
- 'authorfullname' => html_entity_decode($this->get_author_fullname()),
- 'postdate' => html_entity_decode($this->get_postdate()),
- 'firstpost' => $this->is_firstpost(),
- 'canreply' => $this->canreply,
- 'permalink' => $this->get_permalink(),
- 'moodleoverflowindexlink' => $this->get_moodleoverflowindexlink(),
- 'replylink' => $this->get_replylink(),
- 'authorpicture' => $this->get_author_picture(),
+ 'id' => html_entity_decode($this->post->id),
+ 'coursename' => html_entity_decode($this->get_coursename()),
+ 'courselink' => html_entity_decode($this->get_courselink()),
+ 'moodleoverflowname' => html_entity_decode($this->get_moodleoverflowname()),
+ 'showdiscussionname' => html_entity_decode($this->has_showdiscussionname()),
+ 'discussionname' => html_entity_decode($this->get_discussionname()),
+ 'subject' => html_entity_decode($this->get_subject()),
+ 'authorfullname' => html_entity_decode($this->get_author_fullname()),
+ 'postdate' => html_entity_decode($this->get_postdate()),
+ 'firstpost' => $this->is_firstpost(),
+ 'canreply' => $this->canreply,
+ 'permalink' => $this->get_permalink(),
+ 'moodleoverflowindexlink' => $this->get_moodleoverflowindexlink(),
+ 'replylink' => $this->get_replylink(),
+ 'authorpicture' => $this->get_author_picture(),
'unsubscribemoodleoverflowlink' => $this->get_unsubscribemoodleoverflowlink(),
- 'parentpostlink' => $this->get_parentpostlink(),
- 'unsubscribediscussionlink' => $this->get_unsubscribediscussionlink(),
- 'moodleoverflowviewlink' => $this->get_moodleoverflowviewlink(),
- 'discussionlink' => $this->get_discussionlink(),
- 'authorlink' => $this->get_authorlink(),
- 'grouppicture' => $this->get_group_picture(),
+ 'parentpostlink' => $this->get_parentpostlink(),
+ 'unsubscribediscussionlink' => $this->get_unsubscribediscussionlink(),
+ 'moodleoverflowviewlink' => $this->get_moodleoverflowviewlink(),
+ 'discussionlink' => $this->get_discussionlink(),
+ 'authorlink' => $this->get_authorlink(),
+ 'grouppicture' => $this->get_group_picture(),
// Format some components according to the renderer.
- 'message' => html_entity_decode($renderer->format_message_text($this->cm, $this->post)),
+ 'message' => html_entity_decode($renderer->format_message_text($this->cm, $this->post)),
);
}
@@ -192,31 +192,31 @@ protected function export_for_template_text(\mod_moodleoverflow_renderer $render
*/
protected function export_for_template_html(\mod_moodleoverflow_renderer $renderer) {
return array(
- 'id' => $this->post->id,
- 'coursename' => $this->get_coursename(),
- 'courselink' => $this->get_courselink(),
- 'moodleoverflowname' => $this->get_moodleoverflowname(),
- 'showdiscussionname' => $this->has_showdiscussionname(),
- 'discussionname' => $this->get_discussionname(),
- 'subject' => $this->get_subject(),
- 'authorfullname' => $this->get_author_fullname(),
- 'postdate' => $this->get_postdate(),
- 'canreply' => $this->canreply,
- 'permalink' => $this->get_permalink(),
- 'firstpost' => $this->is_firstpost(),
- 'replylink' => $this->get_replylink(),
- 'unsubscribediscussionlink' => $this->get_unsubscribediscussionlink(),
+ 'id' => $this->post->id,
+ 'coursename' => $this->get_coursename(),
+ 'courselink' => $this->get_courselink(),
+ 'moodleoverflowname' => $this->get_moodleoverflowname(),
+ 'showdiscussionname' => $this->has_showdiscussionname(),
+ 'discussionname' => $this->get_discussionname(),
+ 'subject' => $this->get_subject(),
+ 'authorfullname' => $this->get_author_fullname(),
+ 'postdate' => $this->get_postdate(),
+ 'canreply' => $this->canreply,
+ 'permalink' => $this->get_permalink(),
+ 'firstpost' => $this->is_firstpost(),
+ 'replylink' => $this->get_replylink(),
+ 'unsubscribediscussionlink' => $this->get_unsubscribediscussionlink(),
'unsubscribemoodleoverflowlink' => $this->get_unsubscribemoodleoverflowlink(),
- 'parentpostlink' => $this->get_parentpostlink(),
- 'moodleoverflowindexlink' => $this->get_moodleoverflowindexlink(),
- 'moodleoverflowviewlink' => $this->get_moodleoverflowviewlink(),
- 'discussionlink' => $this->get_discussionlink(),
- 'authorlink' => $this->get_authorlink(),
- 'authorpicture' => $this->get_author_picture(),
- 'grouppicture' => $this->get_group_picture(),
+ 'parentpostlink' => $this->get_parentpostlink(),
+ 'moodleoverflowindexlink' => $this->get_moodleoverflowindexlink(),
+ 'moodleoverflowviewlink' => $this->get_moodleoverflowviewlink(),
+ 'discussionlink' => $this->get_discussionlink(),
+ 'authorlink' => $this->get_authorlink(),
+ 'authorpicture' => $this->get_author_picture(),
+ 'grouppicture' => $this->get_group_picture(),
// Format some components according to the renderer.
- 'message' => $renderer->format_message_text($this->cm, $this->post),
+ 'message' => $renderer->format_message_text($this->cm, $this->post),
);
}
@@ -258,8 +258,8 @@ public function get_unsubscribediscussionlink() {
}
// Prepare information.
- $id = $this->moodleoverflow->id;
- $d = $this->discussion->id;
+ $id = $this->moodleoverflow->id;
+ $d = $this->discussion->id;
$url = '/mod/moodleoverflow/subscribe.php';
// Generate a link to unsubscribe from the discussion.
@@ -527,7 +527,7 @@ public function get_authorlink() {
$link = new \moodle_url(
'/user/view.php', array(
- 'id' => $this->post->userid,
+ 'id' => $this->post->userid,
'course' => $this->course->id,
)
);
diff --git a/classes/post_form.php b/classes/post_form.php
index 3738acfe28..5d60710001 100644
--- a/classes/post_form.php
+++ b/classes/post_form.php
@@ -44,8 +44,8 @@ class mod_moodleoverflow_post_form extends moodleform {
public function definition() {
$modform =& $this->_form;
- $post = $this->_customdata['post'];
- $modcontext = $this->_customdata['modulecontext'];
+ $post = $this->_customdata['post'];
+ $modcontext = $this->_customdata['modulecontext'];
$moodleoverflow = $this->_customdata['moodleoverflow'];
// Fill in the data depending on page params later using set_data.
@@ -135,11 +135,11 @@ public static function attachment_options($moodleoverflow) {
$maxbytes = get_user_max_upload_file_size($PAGE->context, $CFG->maxbytes, $COURSE->maxbytes, $moodleoverflow->maxbytes);
return array(
- 'subdirs' => 0,
- 'maxbytes' => $maxbytes,
- 'maxfiles' => $moodleoverflow->maxattachments,
+ 'subdirs' => 0,
+ 'maxbytes' => $maxbytes,
+ 'maxfiles' => $moodleoverflow->maxattachments,
'accepted_types' => '*',
- 'return_types' => FILE_INTERNAL | FILE_CONTROLLED_LINK
+ 'return_types' => FILE_INTERNAL | FILE_CONTROLLED_LINK
);
}
diff --git a/classes/privacy/data_export_helper.php b/classes/privacy/data_export_helper.php
index be1e93d583..750903a5ac 100644
--- a/classes/privacy/data_export_helper.php
+++ b/classes/privacy/data_export_helper.php
@@ -60,8 +60,8 @@ public static function export_discussion_data($userid, array $mappings) {
";
$params = [
'discussionuserid' => $userid,
- 'dmuserid' => $userid,
- 'dsubuserid' => $userid,
+ 'dmuserid' => $userid,
+ 'dsubuserid' => $userid,
];
$params += $forumparams;
@@ -75,9 +75,9 @@ public static function export_discussion_data($userid, array $mappings) {
// Store related metadata for this discussion.
static::export_discussion_subscription_data($context, $discussion);
$discussiondata = (object) [
- 'name' => format_string($discussion->name, true),
- 'timemodified' => transform::datetime($discussion->timemodified),
- 'creator_was_you' => transform::yesno($discussion->userid == $userid),
+ 'name' => format_string($discussion->name, true),
+ 'timemodified' => transform::datetime($discussion->timemodified),
+ 'creator_was_you' => transform::yesno($discussion->userid == $userid),
'last_modifier_was_you' => transform::yesno($discussion->usermodified == $userid)
];
// Store the discussion content.
@@ -122,8 +122,8 @@ public static function export_all_posts($userid, array $mappings) {
GROUP BY mof.id, p.discussion, d.name
";
$params = [
- 'postuserid' => $userid,
- 'readuserid' => $userid,
+ 'postuserid' => $userid,
+ 'readuserid' => $userid,
'ratinguserid' => $userid,
];
$params += $forumparams;
@@ -163,7 +163,7 @@ protected static function export_all_posts_in_discussion($userid, \context $cont
";
$params = [
'discussionid' => $discussionid,
- 'readuserid' => $userid,
+ 'readuserid' => $userid,
'ratinguserid' => $userid
];
@@ -235,15 +235,15 @@ protected static function export_post_data($userid, \context $context, $postarea
// Store related metadata.
static::export_read_data($context, $postarea, $post);
$postdata = (object) [
- 'created' => transform::datetime($post->created),
- 'modified' => transform::datetime($post->modified),
+ 'created' => transform::datetime($post->created),
+ 'modified' => transform::datetime($post->modified),
'author_was_you' => transform::yesno($post->userid == $userid)
];
$postdata->message = writer::with_context($context)->rewrite_pluginfile_urls(
$postarea, 'mod_moodleoverflow', 'attachment', $post->id, $post->message);
$postdata->message = format_text($postdata->message, $post->messageformat, (object) [
- 'para' => false,
+ 'para' => false,
'context' => $context,
]);
@@ -289,16 +289,16 @@ protected static function export_rating_data($postid, $onlyuser, $userid) {
foreach ($ownratings as $rating) {
$userratings[] = (object) [
'firstrated' => $rating->firstrated,
- 'rating' => $rating->rating
+ 'rating' => $rating->rating
];
}
if (!$onlyuser) {
$ratingdata = [
- 'downvotes' => $ratingpost->downvotes,
- 'upvotes' => $ratingpost->upvotes,
+ 'downvotes' => $ratingpost->downvotes,
+ 'upvotes' => $ratingpost->upvotes,
'was_rated_as_helpful' => transform::yesno($ratingpost->ishelpful),
- 'was_rated_as_solved' => transform::yesno($ratingpost->issolved)
+ 'was_rated_as_solved' => transform::yesno($ratingpost->issolved)
];
}
$ratingdata['your_rating'] = (object) $userratings;
@@ -412,14 +412,14 @@ protected static function export_read_data(\context_module $context, array $post
if (null !== $post->firstread) {
$a = (object) [
'firstread' => $post->firstread,
- 'lastread' => $post->lastread,
+ 'lastread' => $post->lastread,
];
writer::with_context($context)->export_metadata(
$postarea,
'postread',
(object) [
'firstread' => $post->firstread,
- 'lastread' => $post->lastread,
+ 'lastread' => $post->lastread,
],
get_string('privacy:postwasread', 'mod_moodleoverflow', $a)
);
diff --git a/classes/privacy/provider.php b/classes/privacy/provider.php
index 479a700dde..80f424b92f 100644
--- a/classes/privacy/provider.php
+++ b/classes/privacy/provider.php
@@ -53,8 +53,8 @@ class provider implements
public static function get_metadata(collection $collection) : collection {
$collection->add_database_table('moodleoverflow_discussions',
[
- 'name' => 'privacy:metadata:moodleoverflow_discussions:name',
- 'userid' => 'privacy:metadata:moodleoverflow_discussions:userid',
+ 'name' => 'privacy:metadata:moodleoverflow_discussions:name',
+ 'userid' => 'privacy:metadata:moodleoverflow_discussions:userid',
'timemodified' => 'privacy:metadata:moodleoverflow_discussions:timemodified',
'usermodified' => 'privacy:metadata:moodleoverflow_discussions:usermodified'
],
@@ -63,34 +63,34 @@ public static function get_metadata(collection $collection) : collection {
$collection->add_database_table('moodleoverflow_posts',
[
'discussion' => 'privacy:metadata:moodleoverflow_posts:discussion',
- 'parent' => 'privacy:metadata:moodleoverflow_posts:parent',
- 'userid' => 'privacy:metadata:moodleoverflow_posts:userid',
- 'created' => 'privacy:metadata:moodleoverflow_posts:created',
- 'modified' => 'privacy:metadata:moodleoverflow_posts:modified',
- 'message' => 'privacy:metadata:moodleoverflow_posts:message'
+ 'parent' => 'privacy:metadata:moodleoverflow_posts:parent',
+ 'userid' => 'privacy:metadata:moodleoverflow_posts:userid',
+ 'created' => 'privacy:metadata:moodleoverflow_posts:created',
+ 'modified' => 'privacy:metadata:moodleoverflow_posts:modified',
+ 'message' => 'privacy:metadata:moodleoverflow_posts:message'
],
'privacy:metadata:moodleoverflow_posts');
$collection->add_database_table('moodleoverflow_read',
[
- 'userid' => 'privacy:metadata:moodleoverflow_read:userid',
+ 'userid' => 'privacy:metadata:moodleoverflow_read:userid',
'discussionid' => 'privacy:metadata:moodleoverflow_read:discussionid',
- 'postid' => 'privacy:metadata:moodleoverflow_read:postid',
- 'firstread' => 'privacy:metadata:moodleoverflow_read:firstread',
- 'lastread' => 'privacy:metadata:moodleoverflow_read:lastread'
+ 'postid' => 'privacy:metadata:moodleoverflow_read:postid',
+ 'firstread' => 'privacy:metadata:moodleoverflow_read:firstread',
+ 'lastread' => 'privacy:metadata:moodleoverflow_read:lastread'
],
'privacy:metadata:moodleoverflow_read');
$collection->add_database_table('moodleoverflow_subscriptions',
[
- 'userid' => 'privacy:metadata:moodleoverflow_subscriptions:userid',
+ 'userid' => 'privacy:metadata:moodleoverflow_subscriptions:userid',
'moodleoverflow' => 'privacy:metadata:moodleoverflow_subscriptions:moodleoverflow'
],
'privacy:metadata:moodleoverflow_subscriptions');
$collection->add_database_table('moodleoverflow_discuss_subs',
[
- 'userid' => 'privacy:metadata:moodleoverflow_discuss_subs:userid',
+ 'userid' => 'privacy:metadata:moodleoverflow_discuss_subs:userid',
'discussion' => 'privacy:metadata:moodleoverflow_discuss_subs:discussion',
'preference' => 'privacy:metadata:moodleoverflow_discuss_subs:preference'
],
@@ -98,17 +98,17 @@ public static function get_metadata(collection $collection) : collection {
$collection->add_database_table('moodleoverflow_ratings',
[
- 'userid' => 'privacy:metadata:moodleoverflow_ratings:userid',
- 'postid' => 'privacy:metadata:moodleoverflow_ratings:postid',
- 'rating' => 'privacy:metadata:moodleoverflow_ratings:rating',
- 'firstrated' => 'privacy:metadata:moodleoverflow_ratings:firstrated',
+ 'userid' => 'privacy:metadata:moodleoverflow_ratings:userid',
+ 'postid' => 'privacy:metadata:moodleoverflow_ratings:postid',
+ 'rating' => 'privacy:metadata:moodleoverflow_ratings:rating',
+ 'firstrated' => 'privacy:metadata:moodleoverflow_ratings:firstrated',
'lastchanged' => 'privacy:metadata:moodleoverflow_ratings:lastchanged'
],
'privacy:metadata:moodleoverflow_ratings');
$collection->add_database_table('moodleoverflow_tracking',
[
- 'userid' => 'privacy:metadata:moodleoverflow_tracking:userid',
+ 'userid' => 'privacy:metadata:moodleoverflow_tracking:userid',
'moodleoverflowid' => 'privacy:metadata:moodleoverflow_tracking:moodleoverflowid'
],
'privacy:metadata:moodleoverflow_tracking');
@@ -164,17 +164,17 @@ public static function get_contexts_for_userid(int $userid) : contextlist {
";
$params = [
- 'modname' => 'moodleoverflow',
+ 'modname' => 'moodleoverflow',
'contextlevel' => CONTEXT_MODULE,
- 'duserid' => $userid,
- 'dmuserid' => $userid,
- 'puserid' => $userid,
- 'ruserid' => $userid,
- 'suserid' => $userid,
- 'dsuserid' => $userid,
- 'rauserid' => $userid,
- 'userid' => $userid,
- 'guserid' => $userid
+ 'duserid' => $userid,
+ 'dmuserid' => $userid,
+ 'puserid' => $userid,
+ 'ruserid' => $userid,
+ 'suserid' => $userid,
+ 'dsuserid' => $userid,
+ 'rauserid' => $userid,
+ 'userid' => $userid,
+ 'guserid' => $userid
];
$contextlist = new \core_privacy\local\request\contextlist();
@@ -220,9 +220,9 @@ public static function export_user_data(approved_contextlist $contextlist) {
";
$params = [
- 'suserid' => $userid,
- 'userid' => $userid,
- 'guserid' => $userid
+ 'suserid' => $userid,
+ 'userid' => $userid,
+ 'guserid' => $userid
];
$params += $contextparams;
@@ -321,28 +321,28 @@ public static function delete_data_for_user(approved_contextlist $contextlist) {
$DB->delete_records('moodleoverflow_read', [
'moodleoverflowid' => $forum->id,
- 'userid' => $userid]);
+ 'userid' => $userid]);
$DB->delete_records('moodleoverflow_subscriptions', [
'moodleoverflow' => $forum->id,
- 'userid' => $userid]);
+ 'userid' => $userid]);
$DB->delete_records('moodleoverflow_discuss_subs', [
'moodleoverflow' => $forum->id,
- 'userid' => $userid]);
+ 'userid' => $userid]);
$DB->delete_records('moodleoverflow_tracking', [
'moodleoverflowid' => $forum->id,
- 'userid' => $userid]);
+ 'userid' => $userid]);
$DB->delete_records('moodleoverflow_grades', [
'moodleoverflowid' => $forum->id,
- 'userid' => $userid]);
+ 'userid' => $userid]);
// Do not delete ratings but reset userid.
$ratingsql = "userid = :userid AND discussionid IN
(SELECT id FROM {moodleoverflow_discussions} WHERE moodleoverflow = :forum)";
$ratingparams = [
- 'forum' => $forum->id,
+ 'forum' => $forum->id,
'userid' => $userid
];
$DB->set_field_select('moodleoverflow_ratings', 'userid', 0, $ratingsql, $ratingparams);
@@ -351,7 +351,7 @@ public static function delete_data_for_user(approved_contextlist $contextlist) {
(SELECT id FROM {moodleoverflow_discussions} WHERE moodleoverflow = :forum)";
$postidsql = "SELECT p.id FROM {moodleoverflow_posts} p WHERE {$postsql}";
$postparams = [
- 'forum' => $forum->id,
+ 'forum' => $forum->id,
'userid' => $userid
];
@@ -388,8 +388,8 @@ public static function get_users_in_context(userlist $userlist) {
return;
}
$params = [
- 'instanceid' => $context->instanceid,
- 'modulename' => 'moodleoverflow',
+ 'instanceid' => $context->instanceid,
+ 'modulename' => 'moodleoverflow',
];
// Discussion authors.
diff --git a/classes/ratings.php b/classes/ratings.php
index 8e20b21b77..6d57d0b302 100644
--- a/classes/ratings.php
+++ b/classes/ratings.php
@@ -86,11 +86,11 @@ public static function moodleoverflow_add_rating($moodleoverflow, $postid, $rati
if (!$canrate) {
// Catch unenrolled users.
- if (!isguestuser() AND !is_enrolled($coursecontext)) {
+ if (!isguestuser() && !is_enrolled($coursecontext)) {
$SESSION->wantsurl = qualified_me();
$SESSION->enrolcancel = get_local_referer(false);
redirect(new \moodle_url('/enrol/index.php', array(
- 'id' => $course->id,
+ 'id' => $course->id,
'returnurl' => '/mod/moodleoverflow/view.php?m' . $moodleoverflow->id
)), get_string('youneedtoenrol'));
}
@@ -248,7 +248,7 @@ public static function moodleoverflow_sort_answers_by_ratings($posts) {
$statusteacher = self::moodleoverflow_discussion_is_solved($discussionid, true);
// The answer that is marked as correct by both is displayed first.
- if ($statusteacher AND $statusstarter) {
+ if ($statusteacher && $statusstarter) {
// Is the same answer correct for both?
if ($statusstarter->postid == $statusteacher->postid) {
@@ -265,7 +265,7 @@ public static function moodleoverflow_sort_answers_by_ratings($posts) {
// If the answers the teacher marks are preferred, and only
// the teacher marked an answer as solved, display it first.
- if ($preferteacher AND $statusteacher) {
+ if ($preferteacher && $statusteacher) {
// Add the post to the new order and delete it from the posts array.
$neworder[] = (int) $statusteacher->postid;
@@ -534,7 +534,7 @@ private static function moodleoverflow_get_reputation_instance($moodleoverflowid
$reputation += get_config('moodleoverflow', 'votescalevote') * $votes->amount;
// Can the reputation of a user be negative?
- if (!$moodleoverflow->allownegativereputation AND $reputation <= 0) {
+ if (!$moodleoverflow->allownegativereputation && $reputation <= 0) {
$reputation = 0;
}
@@ -608,7 +608,7 @@ private static function moodleoverflow_check_old_rating($postid, $userid, $oldra
$rating['normal'] = $DB->get_record_sql($sql, [ $userid, $postid ]);
// Return the rating if it is requested.
- if ($oldrating == RATING_DOWNVOTE OR $oldrating == RATING_UPVOTE) {
+ if ($oldrating == RATING_DOWNVOTE || $oldrating == RATING_UPVOTE) {
return $rating['normal'];
}
@@ -700,7 +700,7 @@ private static function moodleoverflow_remove_rating($postid, $rating, $userid,
// Trigger an event.
$params = array(
'objectid' => $oldrecord->id,
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
);
$event = \mod_moodleoverflow\event\rating_deleted::create($params);
$event->add_record_snapshot('moodleoverflow_ratings', $oldrecord);
@@ -741,7 +741,7 @@ private static function moodleoverflow_add_rating_record($moodleoverflowid, $dis
// Trigger an event.
$params = array(
'objectid' => $recordid,
- 'context' => $mod,
+ 'context' => $mod,
);
$event = \mod_moodleoverflow\event\rating_created::create($params);
$event->trigger();
@@ -772,7 +772,7 @@ private static function moodleoverflow_update_rating_record($postid, $rating, $u
// Trigger an event.
$params = array(
'objectid' => $ratingid,
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
);
$event = \mod_moodleoverflow\event\rating_updated::create($params);
$event->trigger();
@@ -783,7 +783,7 @@ private static function moodleoverflow_update_rating_record($postid, $rating, $u
/**
* Check if a user can rate the post.
*
- * @param object $moodleoverflow
+ * @param object $post
* @param \context_module $modulecontext
* @param null|int $userid
*
@@ -793,7 +793,7 @@ private static function moodleoverflow_user_can_rate($post, $modulecontext, $use
global $USER;
if (!$userid) {
// Guests and non-logged-in users can not rate.
- if (isguestuser() OR !isloggedin()) {
+ if (isguestuser() || !isloggedin()) {
return false;
}
$userid = $USER->id;
diff --git a/classes/readtracking.php b/classes/readtracking.php
index f037f06437..88a4b4a36b 100644
--- a/classes/readtracking.php
+++ b/classes/readtracking.php
@@ -53,7 +53,7 @@ public static function moodleoverflow_can_track_moodleoverflows($moodleoverflow
}
// Guests are not allowed to track moodleoverflows.
- if (isguestuser($USER) OR empty($USER->id)) {
+ if (isguestuser($USER) || empty($USER->id)) {
return false;
}
@@ -69,7 +69,7 @@ public static function moodleoverflow_can_track_moodleoverflows($moodleoverflow
}
// Check the settings of the moodleoverflow instance.
$allowed = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_OPTIONAL);
- $forced = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_FORCED);
+ $forced = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_FORCED);
return ($allowed || $forced);
}
@@ -91,7 +91,7 @@ public static function moodleoverflow_is_tracked($moodleoverflow, $user = null)
}
// Guests cannot track a moodleoverflow.
- if (isguestuser($USER) OR empty($USER->id)) {
+ if (isguestuser($USER) || empty($USER->id)) {
return false;
}
@@ -102,7 +102,7 @@ public static function moodleoverflow_is_tracked($moodleoverflow, $user = null)
// Check the settings of the moodleoverflow instance.
$allowed = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_OPTIONAL);
- $forced = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_FORCED);
+ $forced = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_FORCED);
// Check the preferences of the user.
$userpreference = $DB->get_record('moodleoverflow_tracking',
@@ -238,7 +238,7 @@ public static function moodleoverflow_add_read_record($userid, $postid) {
global $DB;
// Get the current time and the cutoffdate.
- $now = time();
+ $now = time();
$cutoffdate = $now - (get_config('moodleoverflow', 'oldpostdays') * 24 * 3600);
// Check for read records for this user an this post.
@@ -286,28 +286,28 @@ public static function moodleoverflow_delete_read_records($userid = -1, $postid
if ($select != '') {
$select .= ' AND ';
}
- $select .= 'userid = ?';
+ $select .= 'userid = ?';
$params[] = $userid;
}
if ($postid > -1) {
if ($select != '') {
$select .= ' AND ';
}
- $select .= 'postid = ?';
+ $select .= 'postid = ?';
$params[] = $postid;
}
if ($discussionid > -1) {
if ($select != '') {
$select .= ' AND ';
}
- $select .= 'discussionid = ?';
+ $select .= 'discussionid = ?';
$params[] = $discussionid;
}
if ($overflowid > -1) {
if ($select != '') {
$select .= ' AND ';
}
- $select .= 'moodleoverflowid = ?';
+ $select .= 'moodleoverflowid = ?';
$params[] = $overflowid;
}
@@ -371,15 +371,15 @@ public static function moodleoverflow_stop_tracking($moodleoverflowid, $userid =
}
// Check if the user already stopped to track the moodleoverflow.
- $params = array('userid' => $userid, 'moodleoverflowid' => $moodleoverflowid);
+ $params = array('userid' => $userid, 'moodleoverflowid' => $moodleoverflowid);
$isstopped = $DB->record_exists('moodleoverflow_tracking', $params);
// Stop tracking the moodleoverflow if not already stopped.
if (!$isstopped) {
// Create the tracking object.
- $tracking = new \stdClass();
- $tracking->userid = $userid;
+ $tracking = new \stdClass();
+ $tracking->userid = $userid;
$tracking->moodleoverflowid = $moodleoverflowid;
// Insert into the database.
@@ -466,7 +466,6 @@ public static function get_untracked_moodleoverflows($userid, $courseid) {
* Get number of unread posts in a moodleoverflow instance.
*
* @param object $cm
- * @param \stdClass $course The course the moodleoverflow is in
*
* @return int|mixed
*/
@@ -493,7 +492,7 @@ public static function moodleoverflow_count_unread_posts_moodleoverflow($cm) {
// Define a sql-query.
$params = array($USER->id, $cm->instance, $cutoffdate);
- $sql = "SELECT COUNT(p.id)
+ $sql = "SELECT COUNT(p.id)
FROM {moodleoverflow_posts} p
JOIN {moodleoverflow_discussions} d ON p.discussion = d.id
LEFT JOIN {moodleoverflow_read} r ON (r.postid = p.id AND r.userid = ?)
diff --git a/classes/review.php b/classes/review.php
index fa58c71e4a..7d6b417dff 100644
--- a/classes/review.php
+++ b/classes/review.php
@@ -78,6 +78,8 @@ public static function get_short_review_info_for_discussion(int $discussionid) {
}
/**
+ * Get a review post to review.
+ *
* @param int $moodleoverflowid ID of moodleoverflow to look in.
* @param int $afterpostid ID of post after which to look for the first post to review.
* @return string|null
@@ -149,7 +151,7 @@ public static function is_post_in_review_period($post): bool {
/**
* Count outstanding reviews in the moodleoverflow.
*
- * @param $moodleoverflowid
+ * @param int $moodleoverflowid
* @return int
*/
public static function count_outstanding_reviews_in_moodleoverflow($moodleoverflowid): int {
diff --git a/classes/subscriptions.php b/classes/subscriptions.php
index db564e6f84..fc07e9a130 100644
--- a/classes/subscriptions.php
+++ b/classes/subscriptions.php
@@ -94,6 +94,7 @@ class subscriptions {
*
* @param int $userid
* @param object $moodleoverflow
+ * @param object $context
* @param null $discussionid
*
* @return bool
@@ -157,7 +158,7 @@ private static function is_subscribed_to_moodleoverflow($userid, $moodleoverflow
public static function fetch_subscription_cache($moodleoverflowid, $userid) {
// If the cache is already filled, return the result.
- if (isset(self::$moodleoverflowcache[$userid]) AND isset(self::$moodleoverflowcache[$userid][$moodleoverflowid])) {
+ if (isset(self::$moodleoverflowcache[$userid]) && isset(self::$moodleoverflowcache[$userid][$moodleoverflowid])) {
return self::$moodleoverflowcache[$userid][$moodleoverflowid];
}
@@ -165,7 +166,7 @@ public static function fetch_subscription_cache($moodleoverflowid, $userid) {
self::fill_subscription_cache($moodleoverflowid, $userid);
// Catch empty results.
- if (!isset(self::$moodleoverflowcache[$userid]) OR !isset(self::$moodleoverflowcache[$userid][$moodleoverflowid])) {
+ if (!isset(self::$moodleoverflowcache[$userid]) || !isset(self::$moodleoverflowcache[$userid][$moodleoverflowid])) {
return false;
}
@@ -211,7 +212,7 @@ public static function fill_subscription_cache($moodleoverflowid, $userid = null
} else { // The request is not connected to a specific user.
// Request all records.
- $params = array('moodleoverflow' => $moodleoverflowid);
+ $params = array('moodleoverflow' => $moodleoverflowid);
$subscriptions = $DB->get_recordset('moodleoverflow_subscriptions', $params, '', 'id, userid');
// Loop through the records.
@@ -248,7 +249,7 @@ public static function fetch_discussion_subscription($moodleoverflowid, $userid
self::fill_discussion_subscription_cache($moodleoverflowid, $userid);
// Create an array, if there is no record.
- if (!isset(self::$discussioncache[$userid]) OR !isset(self::$discussioncache[$userid][$moodleoverflowid])) {
+ if (!isset(self::$discussioncache[$userid]) || !isset(self::$discussioncache[$userid][$moodleoverflowid])) {
return array();
}
@@ -283,7 +284,7 @@ public static function fill_discussion_subscription_cache($moodleoverflowid, $us
if (!isset(self::$discussioncache[$userid][$moodleoverflowid])) {
// Get all records.
- $params = array('userid' => $userid, 'moodleoverflow' => $moodleoverflowid);
+ $params = array('userid' => $userid, 'moodleoverflow' => $moodleoverflowid);
$subscriptions = $DB->get_recordset('moodleoverflow_discuss_subs', $params,
null, 'id, discussion, preference');
@@ -300,7 +301,7 @@ public static function fill_discussion_subscription_cache($moodleoverflowid, $us
// No user ID is submitted.
// Get all records.
- $params = array('moodleoverflow' => $moodleoverflowid);
+ $params = array('moodleoverflow' => $moodleoverflowid);
$subscriptions = $DB->get_recordset('moodleoverflow_discuss_subs', $params,
null, 'id, userid, discussion, preference');
@@ -362,7 +363,7 @@ public static function subscription_disabled($moodleoverflow) {
public static function is_subscribable($moodleoverflow, $context) {
// Check if the user is an authenticated user.
- $authenticated = (isloggedin() AND !isguestuser());
+ $authenticated = (isloggedin() && !isguestuser());
// Check if subscriptions are disabled for the moodleoverflow.
$disabled = self::subscription_disabled($moodleoverflow);
@@ -372,7 +373,7 @@ public static function is_subscribable($moodleoverflow, $context) {
has_capability('mod/moodleoverflow:allowforcesubscribe', $context);
// Return the result.
- return ($authenticated AND !$forced AND !$disabled);
+ return ($authenticated && !$forced && !$disabled);
}
/**
@@ -429,16 +430,16 @@ public static function get_unsubscribable_moodleoverflows() {
// Find all moodleoverflows from the user's courses that they are subscribed to and which are not set to forced.
// It is possible for users to be subscribed to a moodleoveflow in subscriptions disallowed mode so they must be
// listed here so that they can be unsubscribed from.
- $sql = "SELECT m.id, cm.id as cm, m.course
+ $sql = "SELECT m.id, cm.id as cm, m.course
FROM {moodleoverflow} m
JOIN {course_modules} cm ON cm.instance = m.id
JOIN {modules} mo ON mo.name = :modulename AND mo.id = cm.module
LEFT JOIN {moodleoverflow_subscriptions} ms ON (ms.moodleoverflow = m.id AND ms.userid = :userid)
WHERE m.forcesubscribe <> :forcesubscribe AND ms.id IS NOT NULL AND cm.course $coursesql";
- $params = array('modulename' => 'moodleoverflow',
+ $params = array('modulename' => 'moodleoverflow',
'userid' => $USER->id,
'forcesubscribe' => MOODLEOVERFLOW_FORCESUBSCRIBE);
- $mergedparams = array_merge($courseparams, $params);
+ $mergedparams = array_merge($courseparams, $params);
$moodleoverflows = $DB->get_recordset_sql($sql, $mergedparams);
// Loop through all of the results and add them to an array.
@@ -497,13 +498,13 @@ public static function fill_subscription_cache_for_course($courseid, $userid) {
}
// Fetch a record set for all moodleoverflowids and their subscription id.
- $sql = "SELECT m.id AS moodleoverflowid,s.id AS subscriptionid
+ $sql = "SELECT m.id AS moodleoverflowid,s.id AS subscriptionid
FROM {moodleoverflow} m
LEFT JOIN {moodleoverflow_subscriptions} s ON (s.moodleoverflow = m.id AND s.userid = :userid)
WHERE m.course = :course AND m.forcesubscribe <> :subscriptionforced";
- $params = array(
- 'userid' => $userid,
- 'course' => $courseid,
+ $params = array(
+ 'userid' => $userid,
+ 'course' => $courseid,
'subscriptionforced' => MOODLEOVERFLOW_FORCESUBSCRIBE,
);
$subscriptions = $DB->get_recordset_sql($sql, $params);
@@ -537,7 +538,7 @@ public static function get_subscribed_users($moodleoverflow, $context, $fields =
} else {
$allnames = get_all_user_name_fields(true, 'u');
}
- $fields = "u.id, u.username, $allnames, u.maildisplay, u.mailformat, u.maildigest,
+ $fields = "u.id, u.username, $allnames, u.maildisplay, u.mailformat, u.maildigest,
u.imagealt, u.email, u.emailstop, u.city, u.country, u.lastaccess, u.lastlogin,
u.picture, u.timezone, u.theme, u.lang, u.trackforums, u.mnethostid";
}
@@ -554,9 +555,9 @@ public static function get_subscribed_users($moodleoverflow, $context, $fields =
if ($includediscussions) {
// Determine more params.
- $params['smoodleoverflowid'] = $moodleoverflow->id;
+ $params['smoodleoverflowid'] = $moodleoverflow->id;
$params['dsmoodleoverflowid'] = $moodleoverflow->id;
- $params['unsubscribed'] = self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED;
+ $params['unsubscribed'] = self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED;
// SQL-statement to fetch all needed fields from the database.
$sql = "SELECT $fields
@@ -597,9 +598,9 @@ public static function get_subscribed_users($moodleoverflow, $context, $fields =
unset($results[$CFG->siteguest]);
// Apply the activity module avaiability restrictions.
- $cm = get_coursemodule_from_instance('moodleoverflow', $moodleoverflow->id, $moodleoverflow->course);
+ $cm = get_coursemodule_from_instance('moodleoverflow', $moodleoverflow->id, $moodleoverflow->course);
$modinfo = get_fast_modinfo($moodleoverflow->course);
- $info = new \core_availability\info_module($modinfo->get_cm($cm->id));
+ $info = new \core_availability\info_module($modinfo->get_cm($cm->id));
$results = $info->filter_user_list($results);
// Return all subscribed users.
@@ -655,8 +656,8 @@ public static function subscribe_user($userid, $moodleoverflow, $context, $userr
}
// Create a new subscription object.
- $sub = new \stdClass();
- $sub->userid = $userid;
+ $sub = new \stdClass();
+ $sub->userid = $userid;
$sub->moodleoverflow = $moodleoverflow->id;
// Insert the record into the database.
@@ -667,15 +668,15 @@ public static function subscribe_user($userid, $moodleoverflow, $context, $userr
// Delete all those discussion subscriptions.
$params = array(
- 'userid' => $userid,
+ 'userid' => $userid,
'moodleoverflowid' => $moodleoverflow->id,
- 'preference' => self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED);
- $where = 'userid = :userid AND moodleoverflow = :moodleoverflowid AND preference <> :preference';
+ 'preference' => self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED);
+ $where = 'userid = :userid AND moodleoverflow = :moodleoverflowid AND preference <> :preference';
$DB->delete_records_select('moodleoverflow_discuss_subs', $where, $params);
// Reset the subscription caches for this moodleoverflow.
// We know that there were previously entries and there aren't any more.
- if (isset(self::$discussioncache[$userid]) AND isset(self::$discussioncache[$userid][$moodleoverflow->id])) {
+ if (isset(self::$discussioncache[$userid]) && isset(self::$discussioncache[$userid][$moodleoverflow->id])) {
foreach (self::$discussioncache[$userid][$moodleoverflow->id] as $discussionid => $preference) {
if ($preference != self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED) {
unset(self::$discussioncache[$userid][$moodleoverflow->id][$discussionid]);
@@ -689,12 +690,12 @@ public static function subscribe_user($userid, $moodleoverflow, $context, $userr
// Trigger an subscription created event.
$params = array(
- 'context' => $context,
- 'objectid' => $result,
+ 'context' => $context,
+ 'objectid' => $result,
'relateduserid' => $userid,
- 'other' => array('moodleoverflowid' => $moodleoverflow->id),
+ 'other' => array('moodleoverflowid' => $moodleoverflow->id),
);
- $event = event\subscription_created::create($params);
+ $event = event\subscription_created::create($params);
$event->trigger();
// Return the subscription ID.
@@ -726,14 +727,14 @@ public static function unsubscribe_user($userid, $moodleoverflow, $context, $use
// Delete the discussion subscriptions as well.
$params = array(
- 'userid' => $userid,
+ 'userid' => $userid,
'moodleoverflow' => $moodleoverflow->id,
- 'preference' => self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED,
+ 'preference' => self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED,
);
$DB->delete_records('moodleoverflow_discuss_subs', $params);
// Update the discussion cache.
- if (isset(self::$discussioncache[$userid]) AND isset(self::$discussioncache[$userid][$moodleoverflow->id])) {
+ if (isset(self::$discussioncache[$userid]) && isset(self::$discussioncache[$userid][$moodleoverflow->id])) {
self::$discussioncache[$userid][$moodleoverflow->id] = array();
}
}
@@ -743,12 +744,12 @@ public static function unsubscribe_user($userid, $moodleoverflow, $context, $use
// Trigger an subscription deletion event.
$params = array(
- 'context' => $context,
- 'objectid' => $subscription->id,
+ 'context' => $context,
+ 'objectid' => $subscription->id,
'relateduserid' => $userid,
- 'other' => array('moodleoverflowid' => $moodleoverflow->id),
+ 'other' => array('moodleoverflowid' => $moodleoverflow->id),
);
- $event = event\subscription_deleted::create($params);
+ $event = event\subscription_deleted::create($params);
$event->add_record_snapshot('moodleoverflow_subscriptions', $subscription);
$event->trigger();
}
@@ -770,11 +771,11 @@ public static function subscribe_user_to_discussion($userid, $discussion, $conte
global $DB;
// Check if the user is already subscribed to the discussion.
- $params = array('userid' => $userid, 'discussion' => $discussion->id);
+ $params = array('userid' => $userid, 'discussion' => $discussion->id);
$subscription = $DB->get_record('moodleoverflow_discuss_subs', $params);
// Dont continue if the user is already subscribed.
- if ($subscription AND $subscription->preference != self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED) {
+ if ($subscription && $subscription->preference != self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED) {
return false;
}
@@ -783,7 +784,7 @@ public static function subscribe_user_to_discussion($userid, $discussion, $conte
if ($DB->record_exists('moodleoverflow_subscriptions', $params)) {
// Check if the user is unsubscribed from the discussion.
- if ($subscription AND $subscription->preference == self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED) {
+ if ($subscription && $subscription->preference == self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED) {
// Delete the discussion preference.
$DB->delete_records('moodleoverflow_discuss_subs', array('id' => $subscription->id));
@@ -806,11 +807,11 @@ public static function subscribe_user_to_discussion($userid, $discussion, $conte
} else {
// Else a new record needs to be created.
- $subscription = new \stdClass();
- $subscription->userid = $userid;
+ $subscription = new \stdClass();
+ $subscription->userid = $userid;
$subscription->moodleoverflow = $discussion->moodleoverflow;
- $subscription->discussion = $discussion->id;
- $subscription->preference = time();
+ $subscription->discussion = $discussion->id;
+ $subscription->preference = time();
// Insert the subscription record into the database.
$subscription->id = $DB->insert_record('moodleoverflow_discuss_subs', $subscription);
@@ -820,12 +821,12 @@ public static function subscribe_user_to_discussion($userid, $discussion, $conte
// Create a discussion subscription created event.
$params = array(
- 'context' => $context,
- 'objectid' => $subscription->id,
+ 'context' => $context,
+ 'objectid' => $subscription->id,
'relateduserid' => $userid,
- 'other' => array('moodleoverflowid' => $discussion->moodleoverflow, 'discussion' => $discussion->id),
+ 'other' => array('moodleoverflowid' => $discussion->moodleoverflow, 'discussion' => $discussion->id),
);
- $event = event\discussion_subscription_created::create($params);
+ $event = event\discussion_subscription_created::create($params);
$event->trigger();
// The subscription was successful.
@@ -845,11 +846,11 @@ public static function unsubscribe_user_from_discussion($userid, $discussion, $c
global $DB;
// Check the users subscription preference for this discussion.
- $params = array('userid' => $userid, 'discussion' => $discussion->id);
+ $params = array('userid' => $userid, 'discussion' => $discussion->id);
$subscription = $DB->get_record('moodleoverflow_discuss_subs', $params);
// If the user not already subscribed to the discussion, do not continue.
- if ($subscription AND $subscription->preference == self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED) {
+ if ($subscription && $subscription->preference == self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED) {
return false;
}
@@ -858,7 +859,7 @@ public static function unsubscribe_user_from_discussion($userid, $discussion, $c
if (!$DB->record_exists('moodleoverflow_subscriptions', $params)) {
// Check if the user isn't subscribed to the moodleoverflow.
- if ($subscription AND $subscription->preference != self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED) {
+ if ($subscription && $subscription->preference != self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED) {
// Delete the discussion subscription.
$DB->delete_records('moodleoverflow_discuss_subs', array('id' => $subscription->id));
@@ -885,11 +886,11 @@ public static function unsubscribe_user_from_discussion($userid, $discussion, $c
// There is no record.
// Create a new discussion subscription record.
- $subscription = new \stdClass();
- $subscription->userid = $userid;
+ $subscription = new \stdClass();
+ $subscription->userid = $userid;
$subscription->moodleoverflow = $discussion->moodleoverflow;
- $subscription->discussion = $discussion->id;
- $subscription->preference = self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED;
+ $subscription->discussion = $discussion->id;
+ $subscription->preference = self::MOODLEOVERFLOW_DISCUSSION_UNSUBSCRIBED;
// Insert the discussion subscription record into the database.
$subscription->id = $DB->insert_record('moodleoverflow_discuss_subs', $subscription);
@@ -901,12 +902,12 @@ public static function unsubscribe_user_from_discussion($userid, $discussion, $c
// Trigger an discussion subscription deletetion event.
$params = array(
- 'context' => $context,
- 'objectid' => $subscription->id,
+ 'context' => $context,
+ 'objectid' => $subscription->id,
'relateduserid' => $userid,
- 'other' => array('moodleoverflowid' => $discussion->moodleoverflow, 'discussion' => $discussion->id),
+ 'other' => array('moodleoverflowid' => $discussion->moodleoverflow, 'discussion' => $discussion->id),
);
- $event = event\discussion_subscription_deleted::create($params);
+ $event = event\discussion_subscription_deleted::create($params);
$event->trigger();
// The user was successfully unsubscribed from the discussion.
@@ -930,23 +931,23 @@ public static function moodleoverflow_get_subscribe_link($moodleoverflow, $conte
// Define strings.
$defaultmessages = array(
- 'subscribed' => get_string('unsubscribe', 'moodleoverflow'),
- 'unsubscribed' => get_string('subscribe', 'moodleoverflow'),
+ 'subscribed' => get_string('unsubscribe', 'moodleoverflow'),
+ 'unsubscribed' => get_string('subscribe', 'moodleoverflow'),
'forcesubscribed' => get_string('everyoneissubscribed', 'moodleoverflow'),
- 'cantsubscribe' => get_string('disallowsubscribe', 'moodleoverflow'),
+ 'cantsubscribe' => get_string('disallowsubscribe', 'moodleoverflow'),
);
// Combine strings the submitted messages.
$messages = $messages + $defaultmessages;
// Check whether the user is forced to be subscribed to the moodleoverflow.
- $isforced = self::is_forcesubscribed($moodleoverflow) && has_capability('mod/moodleoverflow:allowforcesubscribe', $context);
+ $isforced = self::is_forcesubscribed($moodleoverflow) && has_capability('mod/moodleoverflow:allowforcesubscribe', $context);
$isdisabled = self::subscription_disabled($moodleoverflow);
// Return messages depending on the subscription state.
if ($isforced) {
return $messages['forcesubscribed'];
- } else if ($isdisabled AND !has_capability('mod/moodleoverflow:managesubscriptions', $context)) {
+ } else if ($isdisabled && !has_capability('mod/moodleoverflow:managesubscriptions', $context)) {
return $messages['cantsubscribe'];
} else {
@@ -960,18 +961,18 @@ public static function moodleoverflow_get_subscribe_link($moodleoverflow, $conte
// Define the text of the link depending on the subscription state.
if ($issubscribed) {
- $linktext = $messages['subscribed'];
+ $linktext = $messages['subscribed'];
$linktitle = get_string('subscribestop', 'moodleoverflow');
} else {
- $linktext = $messages['unsubscribed'];
+ $linktext = $messages['unsubscribed'];
$linktitle = get_string('subscribestart', 'moodleoverflow');
}
// Create an options array.
- $options = array();
- $options['id'] = $moodleoverflow->id;
- $options['sesskey'] = sesskey();
- $options['returnurl'] = 0;
+ $options = array();
+ $options['id'] = $moodleoverflow->id;
+ $options['sesskey'] = sesskey();
+ $options['returnurl'] = 0;
$options['backtoindex'] = 1;
// Return the link to subscribe the user.
@@ -995,7 +996,7 @@ public static function moodleoverflow_post_subscription($fromform, $moodleoverfl
global $USER;
// Check for some basic information.
- $force = self::is_forcesubscribed($moodleoverflow);
+ $force = self::is_forcesubscribed($moodleoverflow);
$disabled = self::subscription_disabled($moodleoverflow);
// Do not continue if the user is already forced to be subscribed.
@@ -1007,10 +1008,10 @@ public static function moodleoverflow_post_subscription($fromform, $moodleoverfl
if ($disabled) {
// If the user is subscribed, unsubscribe him.
- $subscribed = self::is_subscribed($USER->id, $moodleoverflow, $modulecontext);
+ $subscribed = self::is_subscribed($USER->id, $moodleoverflow, $modulecontext);
$coursecontext = \context_course::instance($moodleoverflow->course);
- $canmanage = has_capability('moodle/course:manageactivities', $coursecontext, $USER->id);
- if ($subscribed AND !$canmanage) {
+ $canmanage = has_capability('moodle/course:manageactivities', $coursecontext, $USER->id);
+ if ($subscribed && !$canmanage) {
self::unsubscribe_user($USER->id, $moodleoverflow, $modulecontext);
}
@@ -1043,10 +1044,10 @@ public static function get_discussion_subscription_icon($moodleoverflow, $contex
$status = self::is_subscribed($USER->id, $moodleoverflow, $context, $discussionid);
// Create a link to subscribe or unsubscribe to the discussion.
- $array = array(
- 'sesskey' => sesskey(),
- 'id' => $moodleoverflow->id,
- 'd' => $discussionid,
+ $array = array(
+ 'sesskey' => sesskey(),
+ 'id' => $moodleoverflow->id,
+ 'd' => $discussionid,
'returnurl' => $returnurl,
);
$subscriptionlink = new \moodle_url('/mod/moodleoverflow/subscribe.php', $array);
@@ -1060,11 +1061,11 @@ public static function get_discussion_subscription_icon($moodleoverflow, $contex
// Return the link.
$array = array(
- 'title' => get_string('clicktounsubscribe', 'moodleoverflow'),
- 'class' => 'discussiontoggle iconsmall',
+ 'title' => get_string('clicktounsubscribe', 'moodleoverflow'),
+ 'class' => 'discussiontoggle iconsmall',
'data-moodleoverflowid' => $moodleoverflow->id,
- 'data-discussionid' => $discussionid,
- 'data-includetext' => false,
+ 'data-discussionid' => $discussionid,
+ 'data-includetext' => false,
);
return \html_writer::link($subscriptionlink, $output, $array);
@@ -1076,11 +1077,11 @@ public static function get_discussion_subscription_icon($moodleoverflow, $contex
// Return the link.
$array = array(
- 'title' => get_string('clicktosubscribe', 'moodleoverflow'),
- 'class' => 'discussiontoggle iconsmall',
+ 'title' => get_string('clicktosubscribe', 'moodleoverflow'),
+ 'class' => 'discussiontoggle iconsmall',
'data-moodleoverflowid' => $moodleoverflow->id,
- 'data-discussionid' => $discussionid,
- 'data-includetext' => false,
+ 'data-discussionid' => $discussionid,
+ 'data-includetext' => false,
);
return \html_writer::link($subscriptionlink, $output, $array);
diff --git a/classes/task/send_daily_mail.php b/classes/task/send_daily_mail.php
new file mode 100644
index 0000000000..e2c57a57e2
--- /dev/null
+++ b/classes/task/send_daily_mail.php
@@ -0,0 +1,92 @@
+.
+
+/**
+ * Task schedule configuration for the plugintype_pluginname plugin.
+ *
+ * @package mod_moodleoverflow
+ * @copyright 2023, Tamaro Walter
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+namespace mod_moodleoverflow\task;
+
+defined('MOODLE_INTERNAL') || die();
+/**
+ * This task sends a daily mail of unread posts
+ */
+class send_daily_mail extends \core\task\scheduled_task {
+
+ /**
+ * Return the task's name as shown in admin screens.
+ *
+ * @return string
+ */
+ public function get_name() {
+ return get_string('tasksenddailymail', 'mod_moodleoverflow');
+ }
+
+ /**
+ * Execute the task.
+ */
+ public function execute() {
+ global $DB;
+
+ // Call your own api.
+ $users = $DB->get_records_sql('SELECT DISTINCT userid FROM {moodleoverflow_mail_info}');
+ if (empty($users)) {
+ mtrace('No daily mail to send.');
+ return;
+ }
+ // Go through each user that has unread posts.
+ foreach ($users as $user) {
+ // Sorts the records with "Order by courseid".
+ $userdata = $DB->get_records('moodleoverflow_mail_info', array('userid' => $user->userid), 'courseid, forumid');
+ $mail = array();
+ // Fill the $mail array.
+ foreach ($userdata as $row) {
+ $currentcourse = $DB->get_record('course', array('id' => $row->courseid), 'fullname, id');
+ $currentforum = $DB->get_record('moodleoverflow', array('id' => $row->forumid), 'name, id');
+ $coursemoduleid = get_coursemodule_from_instance('moodleoverflow', $row->forumid);
+ $discussion = $DB->get_record('moodleoverflow_discussions', array('id' => $row->forumdiscussionid), 'name, id');
+ $unreadposts = $row->numberofposts;
+
+ // Build url to the course, forum, and discussion.
+ $linktocourse = new \moodle_url('/course/view.php', array('id' => $currentcourse->id));
+ $linktoforum = new \moodle_url('/mod/moodleoverflow/view.php', array('id' => $coursemoduleid->id));
+ $linktodiscussion = new \moodle_url('/mod/moodleoverflow/discussion.php', array('d' => $discussion->id));
+
+ // Now change the url to a clickable html link.
+ $linktocourse = \html_writer::link($linktocourse->out(), $currentcourse->fullname);
+ $linktoforum = \html_writer::link($linktoforum->out(), $currentforum->name);
+ $linktodiscussion = \html_writer::link($linktodiscussion->out(), $discussion->name);
+
+ // Build a single line string with the digest information and add it to the mailarray.
+ $string = get_string('digestunreadpost', 'mod_moodleoverflow', array('linktocourse' => $linktocourse,
+ 'linktoforum' => $linktoforum,
+ 'linktodiscussion' => $linktodiscussion,
+ 'unreadposts' => $unreadposts));
+ array_push($mail, $string);
+ }
+ // Build the final message and send it to user. Then remove the sent records.
+ $message = implode('
', $mail);
+ $userto = $DB->get_record('user', array('id' => $user->userid));
+ $from = \core_user::get_noreply_user();
+ $subject = get_string('tasksenddailymail', 'mod_moodleoverflow');
+ email_to_user($userto, $from, $subject, $message);
+ $DB->delete_records('moodleoverflow_mail_info', array('userid' => $user->userid));
+ }
+ }
+}
diff --git a/db/access.php b/db/access.php
index 2b13974a3d..232e4076f0 100644
--- a/db/access.php
+++ b/db/access.php
@@ -49,35 +49,35 @@
// Modify capabilities as needed and remove this comment.
$capabilities = array(
'mod/moodleoverflow:addinstance' => array(
- 'riskbitmask' => RISK_XSS,
- 'captype' => 'write',
- 'contextlevel' => CONTEXT_COURSE,
- 'archetypes' => array(
+ 'riskbitmask' => RISK_XSS,
+ 'captype' => 'write',
+ 'contextlevel' => CONTEXT_COURSE,
+ 'archetypes' => array(
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'moodle/course:manageactivities'
),
'mod/moodleoverflow:view' => array(
- 'captype' => 'read',
+ 'captype' => 'read',
'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
+ 'archetypes' => array(
'guest' => CAP_ALLOW,
- 'user' => CAP_ALLOW,
+ 'user' => CAP_ALLOW,
)
),
'mod/moodleoverflow:viewdiscussion' => array(
- 'captype' => 'read',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'frontpage' => CAP_ALLOW,
- 'guest' => CAP_ALLOW,
- 'student' => CAP_ALLOW,
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'read',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'frontpage' => CAP_ALLOW,
+ 'guest' => CAP_ALLOW,
+ 'student' => CAP_ALLOW,
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:viewdiscussion'
),
@@ -86,13 +86,13 @@
'riskbitmask' => RISK_SPAM,
- 'captype' => 'write',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'student' => CAP_ALLOW,
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'write',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'student' => CAP_ALLOW,
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:replypost'
),
@@ -101,13 +101,13 @@
'riskbitmask' => RISK_SPAM,
- 'captype' => 'write',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'student' => CAP_ALLOW,
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'write',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'student' => CAP_ALLOW,
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:startdiscussion'
),
@@ -116,50 +116,50 @@
'riskbitmask' => RISK_SPAM,
- 'captype' => 'write',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'write',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:editanypost'
),
'mod/moodleoverflow:deleteownpost' => array(
- 'captype' => 'read',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'student' => CAP_ALLOW,
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'read',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'student' => CAP_ALLOW,
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:deleteownpost'
),
'mod/moodleoverflow:deleteanypost' => array(
- 'captype' => 'read',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'read',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:deleteanypost'
),
'mod/moodleoverflow:viewanyrating' => array(
- 'riskbitmask' => RISK_PERSONAL,
- 'captype' => 'read',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'teacher' => CAP_ALLOW,
+ 'riskbitmask' => RISK_PERSONAL,
+ 'captype' => 'read',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:viewanyrating'
),
@@ -168,13 +168,13 @@
'riskbitmask' => RISK_SPAM,
- 'captype' => 'write',
+ 'captype' => 'write',
'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'student' => CAP_ALLOW,
- 'teacher' => CAP_ALLOW,
+ 'archetypes' => array(
+ 'student' => CAP_ALLOW,
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
)
),
@@ -182,13 +182,13 @@
'riskbitmask' => RISK_SPAM,
- 'captype' => 'write',
+ 'captype' => 'write',
'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'student' => CAP_PROHIBIT,
- 'teacher' => CAP_ALLOW,
+ 'archetypes' => array(
+ 'student' => CAP_PROHIBIT,
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
)
),
@@ -196,25 +196,25 @@
'riskbitmask' => RISK_SPAM,
- 'captype' => 'read',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'read',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:managesubscriptions'
),
'mod/moodleoverflow:allowforcesubscribe' => array(
- 'captype' => 'read',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'student' => CAP_ALLOW,
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'read',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'student' => CAP_ALLOW,
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'frontpage' => CAP_ALLOW
+ 'frontpage' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:allowforcesubscribe'
),
@@ -223,24 +223,33 @@
'riskbitmask' => RISK_SPAM,
- 'captype' => 'write',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'student' => CAP_ALLOW,
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'write',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'student' => CAP_ALLOW,
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
- 'manager' => CAP_ALLOW
+ 'manager' => CAP_ALLOW
),
'clonepermissionsfrom' => 'mod/forum:createattachment'
),
'mod/moodleoverflow:reviewpost' => array(
- 'captype' => 'write',
- 'contextlevel' => CONTEXT_MODULE,
- 'archetypes' => array(
- 'teacher' => CAP_ALLOW,
+ 'captype' => 'write',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'teacher' => CAP_ALLOW,
'editingteacher' => CAP_ALLOW,
),
),
+ 'mod/moodleoverflow:movetopic' => array(
+ 'captype' => 'write',
+ 'contextlevel' => CONTEXT_MODULE,
+ 'archetypes' => array(
+ 'teacher' => CAP_ALLOW,
+ 'editingteacher' => CAP_ALLOW,
+ 'manager' => CAP_ALLOW
+ ),
+ ),
);
diff --git a/db/events.php b/db/events.php
index a5d05d3463..77c3f0c3ec 100644
--- a/db/events.php
+++ b/db/events.php
@@ -29,19 +29,19 @@
// Delete read records and subscriptions if the user is not anymore enrolled.
array(
'eventname' => '\core\event\user_enrolment_deleted',
- 'callback' => 'mod_moodleoverflow_observer::user_enrolment_deleted',
+ 'callback' => 'mod_moodleoverflow_observer::user_enrolment_deleted',
),
// Subscribe the user to moodleoverflows which force him to when enroling a user.
array(
'eventname' => '\core\event\role_assigned',
- 'callback' => 'mod_moodleoverflow_observer::role_assigned',
+ 'callback' => 'mod_moodleoverflow_observer::role_assigned',
),
// Subscribe the user to moodleoverflows which force him to when creating an instance.
array(
'eventname' => '\core\event\course_module_created',
- 'callback' => 'mod_moodleoverflow_observer::course_module_created',
+ 'callback' => 'mod_moodleoverflow_observer::course_module_created',
),
);
diff --git a/db/install.xml b/db/install.xml
index 2b787b61ad..fb1f1f2b2e 100644
--- a/db/install.xml
+++ b/db/install.xml
@@ -175,5 +175,22 @@
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
\ No newline at end of file
diff --git a/db/services.php b/db/services.php
index e1c585e630..1069519912 100644
--- a/db/services.php
+++ b/db/services.php
@@ -26,28 +26,28 @@
$functions = array(
'mod_moodleoverflow_record_vote' => array(
- 'classname' => 'mod_moodleoverflow_external',
- 'methodname' => 'record_vote',
- 'classpath' => 'mod/moodleoverflow/externallib.php',
- 'description' => 'Records a vote and updates the reputation of a user',
- 'type' => 'write',
- 'ajax' => true,
+ 'classname' => 'mod_moodleoverflow_external',
+ 'methodname' => 'record_vote',
+ 'classpath' => 'mod/moodleoverflow/externallib.php',
+ 'description' => 'Records a vote and updates the reputation of a user',
+ 'type' => 'write',
+ 'ajax' => true,
'capabilities' => 'mod/moodleoverflow:ratepost'
),
'mod_moodleoverflow_review_approve_post' => array(
- 'classname' => 'mod_moodleoverflow_external',
- 'methodname' => 'review_approve_post',
- 'classpath' => 'mod/moodleoverflow/externallib.php',
- 'description' => 'Approves a post',
- 'type' => 'write',
- 'ajax' => true,
+ 'classname' => 'mod_moodleoverflow_external',
+ 'methodname' => 'review_approve_post',
+ 'classpath' => 'mod/moodleoverflow/externallib.php',
+ 'description' => 'Approves a post',
+ 'type' => 'write',
+ 'ajax' => true,
),
'mod_moodleoverflow_review_reject_post' => array(
- 'classname' => 'mod_moodleoverflow_external',
- 'methodname' => 'review_reject_post',
- 'classpath' => 'mod/moodleoverflow/externallib.php',
- 'description' => 'Rejects a post',
- 'type' => 'write',
- 'ajax' => true,
+ 'classname' => 'mod_moodleoverflow_external',
+ 'methodname' => 'review_reject_post',
+ 'classpath' => 'mod/moodleoverflow/externallib.php',
+ 'description' => 'Rejects a post',
+ 'type' => 'write',
+ 'ajax' => true,
),
);
diff --git a/db/tasks.php b/db/tasks.php
index 6246b164be..d7ecba65b4 100644
--- a/db/tasks.php
+++ b/db/tasks.php
@@ -29,23 +29,33 @@
// Deliver mail notification about new posts.
array(
'classname' => 'mod_moodleoverflow\task\send_mails',
- 'blocking' => 0,
- 'minute' => '*',
- 'hour' => '*',
- 'day' => '*',
- 'month' => '*',
+ 'blocking' => 0,
+ 'minute' => '*',
+ 'hour' => '*',
+ 'day' => '*',
+ 'month' => '*',
'dayofweek' => '*'
),
// Clean old read records.
array(
'classname' => 'mod_moodleoverflow\task\clean_readrecords',
- 'blocking' => 0,
- 'minute' => '*',
- 'hour' => '*',
- 'day' => '*',
- 'month' => '*',
+ 'blocking' => 0,
+ 'minute' => '*',
+ 'hour' => '*',
+ 'day' => '*',
+ 'month' => '*',
'dayofweek' => '*'
),
+ // Clean old read records.
+ array(
+ 'classname' => 'mod_moodleoverflow\task\send_daily_mail',
+ 'blocking' => 0,
+ 'minute' => '0',
+ 'hour' => '17',
+ 'day' => '*',
+ 'month' => '*',
+ 'dayofweek' => '*'
+ )
);
diff --git a/db/upgrade.php b/db/upgrade.php
index 8b65345ee4..8c5b7f2356 100644
--- a/db/upgrade.php
+++ b/db/upgrade.php
@@ -246,5 +246,32 @@ function xmldb_moodleoverflow_upgrade($oldversion) {
upgrade_mod_savepoint(true, 2022110700, 'moodleoverflow');
}
+ if ($oldversion < 2023022400) {
+ // Table for information of digest mail.
+ $table = new xmldb_table('moodleoverflow_mail_info');
+
+ // Adding fields to table moodleoverflow_mail_info.
+ $table->add_field('id', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, XMLDB_SEQUENCE, null);
+ $table->add_field('userid', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, null, null);
+ $table->add_field('courseid', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, null, null);
+ $table->add_field('forumid', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, null, null);
+ $table->add_field('forumdiscussionid', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, null, null);
+ $table->add_field('numberofposts', XMLDB_TYPE_INTEGER, '10', null, XMLDB_NOTNULL, null, null);
+
+ $table->add_key('primary', XMLDB_KEY_PRIMARY, array('id'));
+ $table->add_key('userid', XMLDB_KEY_FOREIGN, array('userid'), 'user', array('id'));
+ $table->add_key('courseid', XMLDB_KEY_FOREIGN, array('courseid'), 'course', array('id'));
+ $table->add_key('forumid', XMLDB_KEY_FOREIGN, array('forumid'), 'moodleoverflow', array('id'));
+ $table->add_key('forumdiscussionid', XMLDB_KEY_FOREIGN,
+ array('forumdiscussionid'), 'moodleoverflow_discussions', array('id'));
+
+ // Conditionally launch create table for moodleoverflow_mail_info.
+ if (!$dbman->table_exists($table)) {
+ $dbman->create_table($table);
+ }
+
+ // Moodleoverflow savepoint reached.
+ upgrade_mod_savepoint(true, 2023022400, 'moodleoverflow');
+ }
return true;
}
diff --git a/discussion.php b/discussion.php
index 41b282400d..1989776532 100644
--- a/discussion.php
+++ b/discussion.php
@@ -27,9 +27,9 @@
require_once($CFG->dirroot . '/mod/moodleoverflow/locallib.php');
// Declare optional parameters.
-$d = required_param('d', PARAM_INT); // The ID of the discussion.
-$sesskey = optional_param('sesskey', null, PARAM_TEXT);
-$ratingid = optional_param('r', 0, PARAM_INT);
+$d = required_param('d', PARAM_INT); // The ID of the discussion.
+$sesskey = optional_param('sesskey', null, PARAM_TEXT);
+$ratingid = optional_param('r', 0, PARAM_INT);
$ratedpost = optional_param('rp', 0, PARAM_INT);
// Set the URL that should be used to return to this page.
@@ -88,10 +88,10 @@
// Trigger the discussion viewed event.
$params = array(
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
'objectid' => $discussion->id,
);
-$event = \mod_moodleoverflow\event\discussion_viewed::create($params);
+$event = \mod_moodleoverflow\event\discussion_viewed::create($params);
$event->trigger();
// Unset where the user is coming from.
@@ -122,10 +122,10 @@
$discussion->name = get_string('privacy:anonym_discussion_name', 'mod_moodleoverflow');
}
-$node = $forumnode->add(format_string($discussion->name),
+$node = $forumnode->add(format_string($discussion->name),
new moodle_url('/mod/moodleoverflow/discussion.php', array('d' => $discussion->id)));
$node->display = false;
-if ($node AND ($post->id != $discussion->firstpost)) {
+if ($node && ($post->id != $discussion->firstpost)) {
$node->add(format_string($post->subject), $PAGE->url);
}
@@ -143,7 +143,7 @@
echo $OUTPUT->heading(format_string($discussion->name), 1, 'discussionname');
// Guests and users can not subscribe to a discussion.
-if ((!is_guest($modulecontext, $USER) AND isloggedin() AND $canviewdiscussion)) {
+if ((!is_guest($modulecontext, $USER) && isloggedin() && $canviewdiscussion)) {
echo '';
}
diff --git a/externallib.php b/externallib.php
index 91f8b1f886..ac2831fa69 100644
--- a/externallib.php
+++ b/externallib.php
@@ -47,9 +47,9 @@ public static function record_vote_parameters() {
return new external_function_parameters(
array(
'discussionid' => new external_value(PARAM_INT, 'id of discussion'),
- 'postid' => new external_value(PARAM_INT, 'id of post'),
- 'ratingid' => new external_value(PARAM_INT, 'rating'),
- 'sesskey' => new external_value(PARAM_TEXT, 'session key'),
+ 'postid' => new external_value(PARAM_INT, 'id of post'),
+ 'ratingid' => new external_value(PARAM_INT, 'rating'),
+ 'sesskey' => new external_value(PARAM_TEXT, 'session key'),
)
);
}
@@ -61,10 +61,10 @@ public static function record_vote_parameters() {
public static function record_vote_returns() {
return new external_single_structure(
array(
- 'postrating' => new external_value(PARAM_INT, 'new post rating'),
+ 'postrating' => new external_value(PARAM_INT, 'new post rating'),
'ownerreputation' => new external_value(PARAM_INT, 'new reputation of post owner'),
'raterreputation' => new external_value(PARAM_INT, 'new reputation of rater'),
- 'ownerid' => new external_value(PARAM_INT, 'user id of post owner'),
+ 'ownerid' => new external_value(PARAM_INT, 'user id of post owner'),
)
);
}
@@ -84,9 +84,9 @@ public static function record_vote($discussionid, $postid, $ratingid, $sesskey)
// Parameter validation.
$params = self::validate_parameters(self::record_vote_parameters(), array(
'discussionid' => $discussionid,
- 'postid' => $postid,
- 'ratingid' => $ratingid,
- 'sesskey' => $sesskey,
+ 'postid' => $postid,
+ 'ratingid' => $ratingid,
+ 'sesskey' => $sesskey,
));
$transaction = $DB->start_delegated_transaction();
@@ -127,7 +127,7 @@ public static function record_vote($discussionid, $postid, $ratingid, $sesskey)
$post = moodleoverflow_get_post_full($params['postid']);
$postownerid = $post->userid;
- $rating = \mod_moodleoverflow\ratings::moodleoverflow_get_ratings_by_discussion($discussion->id,
+ $rating = \mod_moodleoverflow\ratings::moodleoverflow_get_ratings_by_discussion($discussion->id,
$params['postid']);
$ownerrating = \mod_moodleoverflow\ratings::moodleoverflow_get_reputation($moodleoverflow->id, $postownerid);
$raterrating = \mod_moodleoverflow\ratings::moodleoverflow_get_reputation($moodleoverflow->id, $USER->id);
@@ -135,10 +135,10 @@ public static function record_vote($discussionid, $postid, $ratingid, $sesskey)
$cannotseeowner = \mod_moodleoverflow\anonymous::is_post_anonymous($discussion, $moodleoverflow, $USER->id) &&
$USER->id != $postownerid;
- $params['postrating'] = $rating->upvotes - $rating->downvotes;
+ $params['postrating'] = $rating->upvotes - $rating->downvotes;
$params['ownerreputation'] = $cannotseeowner ? null : $ownerrating;
$params['raterreputation'] = $raterrating;
- $params['ownerid'] = $cannotseeowner ? null : $postownerid;
+ $params['ownerid'] = $cannotseeowner ? null : $postownerid;
$transaction->allow_commit();
diff --git a/index.php b/index.php
index aa877a3d59..4c3ef65567 100644
--- a/index.php
+++ b/index.php
@@ -28,7 +28,7 @@
require_once($CFG->dirroot . '/course/lib.php');
// Fetch submitted parameters.
-$id = required_param('id', PARAM_INT);
+$id = required_param('id', PARAM_INT);
$subscribe = optional_param('subscribe', null, PARAM_INT);
// Set an url to go back to the page.
@@ -58,35 +58,35 @@
$params = array(
'context' => context_course::instance($course->id)
);
-$event = \mod_moodleoverflow\event\course_module_instance_list_viewed::create($params);
+$event = \mod_moodleoverflow\event\course_module_instance_list_viewed::create($params);
$event->add_record_snapshot('course', $course);
$event->trigger();
// Cache some strings.
-$string = array();
-$string['moodleoverflow'] = get_string('moodleoverflow', 'moodleoverflow');
-$string['moodleoverflows'] = get_string('moodleoverflows', 'moodleoverflow');
-$string['modulenameplural'] = get_string('modulenameplural', 'moodleoverflow');
-$string['description'] = get_string('description');
-$string['discussions'] = get_string('discussions', 'moodleoverflow');
-$string['subscribed'] = get_string('subscribed', 'moodleoverflow');
-$string['unreadposts'] = get_string('unreadposts', 'moodleoverflow');
-$string['tracking'] = get_string('tracking', 'moodleoverflow');
-$string['markallread'] = get_string('markallread', 'moodleoverflow');
-$string['trackmoodleoverflow'] = get_string('trackmoodleoverflow', 'moodleoverflow');
-$string['notrackmoodleoverflow'] = get_string('notrackmoodleoverflow', 'moodleoverflow');
-$string['subscribe'] = get_string('subscribe', 'moodleoverflow');
-$string['unsubscribe'] = get_string('unsubscribe', 'moodleoverflow');
-$string['subscribeenrolledonly'] = get_string('subscribeenrolledonly', 'moodleoverflow');
-$string['allsubscribe'] = get_string('allsubscribe', 'moodleoverflow');
-$string['allunsubscribe'] = get_string('allunsubscribe', 'moodleoverflow');
+$string = array();
+$string['moodleoverflow'] = get_string('moodleoverflow', 'moodleoverflow');
+$string['moodleoverflows'] = get_string('moodleoverflows', 'moodleoverflow');
+$string['modulenameplural'] = get_string('modulenameplural', 'moodleoverflow');
+$string['description'] = get_string('description');
+$string['discussions'] = get_string('discussions', 'moodleoverflow');
+$string['subscribed'] = get_string('subscribed', 'moodleoverflow');
+$string['unreadposts'] = get_string('unreadposts', 'moodleoverflow');
+$string['tracking'] = get_string('tracking', 'moodleoverflow');
+$string['markallread'] = get_string('markallread', 'moodleoverflow');
+$string['trackmoodleoverflow'] = get_string('trackmoodleoverflow', 'moodleoverflow');
+$string['notrackmoodleoverflow'] = get_string('notrackmoodleoverflow', 'moodleoverflow');
+$string['subscribe'] = get_string('subscribe', 'moodleoverflow');
+$string['unsubscribe'] = get_string('unsubscribe', 'moodleoverflow');
+$string['subscribeenrolledonly'] = get_string('subscribeenrolledonly', 'moodleoverflow');
+$string['allsubscribe'] = get_string('allsubscribe', 'moodleoverflow');
+$string['allunsubscribe'] = get_string('allunsubscribe', 'moodleoverflow');
$string['generalmoodleoverflows'] = get_string('generalmoodleoverflows', 'moodleoverflow');
-$string['yes'] = get_string('yes');
-$string['no'] = get_string('no');
+$string['yes'] = get_string('yes');
+$string['no'] = get_string('no');
// Begin to print a table for the general area.
-$generaltable = new html_table();
-$generaltable->head = array($string['moodleoverflow'], $string['description'], $string['discussions']);
+$generaltable = new html_table();
+$generaltable->head = array($string['moodleoverflow'], $string['description'], $string['discussions']);
$generaltable->align = array('left', 'left', 'center');
// Check whether moodleoverflows can be tracked.
@@ -95,11 +95,11 @@
$untracked = \mod_moodleoverflow\readtracking::get_untracked_moodleoverflows($USER->id, $course->id);
// Add information about the unread posts to the table.
- $generaltable->head[] = $string['unreadposts'];
+ $generaltable->head[] = $string['unreadposts'];
$generaltable->align[] = 'center';
// Add information about the tracking to the table.
- $generaltable->head[] = $string['tracking'];
+ $generaltable->head[] = $string['tracking'];
$generaltable->align[] = 'center';
}
@@ -110,13 +110,13 @@
$usesections = course_format_uses_sections($course->format);
// Initiate tables and variables.
-$table = new html_table();
-$generalmoodleoverflows = array();
-$modinfo = get_fast_modinfo($course);
+$table = new html_table();
+$generalmoodleoverflows = array();
+$modinfo = get_fast_modinfo($course);
$showsubscriptioncolumns = false;
// Parse and organize all moodleoverflows.
-$sql = "SELECT m.*
+$sql = "SELECT m.*
FROM {moodleoverflow} m
WHERE m.course = ?";
$moodleoverflows = $DB->get_records_sql($sql, array($course->id));
@@ -125,13 +125,13 @@
foreach ($modinfo->get_instances_of('moodleoverflow') as $moodleoverflowid => $cm) {
// Check whether the user can see the instance.
- if (!$cm->uservisible OR !isset($moodleoverflows[$moodleoverflowid])) {
+ if (!$cm->uservisible || !isset($moodleoverflows[$moodleoverflowid])) {
continue;
}
// Get the current moodleoverflow instance and the context.
$moodleoverflow = $moodleoverflows[$moodleoverflowid];
- $modulecontext = context_module::instance($cm->id);
+ $modulecontext = context_module::instance($cm->id);
// Check whether the user can see the list.
if (!has_capability('mod/moodleoverflow:viewdiscussion', $modulecontext)) {
@@ -139,10 +139,10 @@
}
// Get information about the subscription state.
- $cansubscribe = \mod_moodleoverflow\subscriptions::is_subscribable($moodleoverflow, $modulecontext);
+ $cansubscribe = \mod_moodleoverflow\subscriptions::is_subscribable($moodleoverflow, $modulecontext);
$moodleoverflow->cansubscribe = $cansubscribe || has_capability('mod/moodleoverflow:managesubscriptions', $modulecontext);
$moodleoverflow->issubscribed = \mod_moodleoverflow\subscriptions::is_subscribed($USER->id, $moodleoverflow, $modulecontext);
- $showsubscriptioncolumns = $showsubscriptioncolumns || $moodleoverflow->issubscribed || $moodleoverflow->cansubscribe;
+ $showsubscriptioncolumns = $showsubscriptioncolumns || $moodleoverflow->issubscribed || $moodleoverflow->cansubscribe;
// Add the moodleoverflow to the cache.
$generalmoodleoverflows[$moodleoverflowid] = $moodleoverflow;
@@ -160,10 +160,10 @@
if (!is_null($subscribe)) {
// Catch guests and not subscribable moodleoverflows.
- if (isguestuser() OR !$showsubscriptioncolumns) {
+ if (isguestuser() || !$showsubscriptioncolumns) {
// Redirect the user back.
- $url = new moodle_url('/mod/moodleoverflow/index.php', array('id' => $id));
+ $url = new moodle_url('/mod/moodleoverflow/index.php', array('id' => $id));
$notification = \core\output\notification::NOTIFY_ERROR;
redirect($url, $string['subscribeenrolledonly'], null, $notification);
}
@@ -173,13 +173,13 @@
// Initiate variables.
$moodleoverflow = $moodleoverflows[$moodleoverflowid];
- $modulecontext = context_module::instance($cm->id);
- $cansub = false;
+ $modulecontext = context_module::instance($cm->id);
+ $cansub = false;
// Check capabilities.
- $cap['viewdiscussion'] = has_capability('mod/moodleoverflow:viewdiscussion', $modulecontext);
+ $cap['viewdiscussion'] = has_capability('mod/moodleoverflow:viewdiscussion', $modulecontext);
$cap['managesubscriptions'] = has_capability('mod/moodleoverflow:managesubscriptions', $modulecontext);
- $cap['manageactivities'] = has_capability('moodle/course:manageactivities', $coursecontext, $USER->id);
+ $cap['manageactivities'] = has_capability('moodle/course:manageactivities', $coursecontext, $USER->id);
// Check whether the user can view the discussions.
if ($cap['viewdiscussion']) {
@@ -187,7 +187,7 @@
}
// Check whether the user can manage subscriptions.
- if ($cansub AND $cm->visible == 0 AND !$cap['managesubscriptions']) {
+ if ($cansub && $cm->visible == 0 && !$cap['managesubscriptions']) {
$cansub = false;
}
@@ -197,11 +197,11 @@
if (!$forcesubscribed) {
// Check the current state.
- $subscribed = \mod_moodleoverflow\subscriptions::is_subscribed($USER->id, $moodleoverflow, $modulecontext);
+ $subscribed = \mod_moodleoverflow\subscriptions::is_subscribed($USER->id, $moodleoverflow, $modulecontext);
$subscribable = \mod_moodleoverflow\subscriptions::is_subscribable($moodleoverflow, $modulecontext);
// Check whether to subscribe or unsubscribe the user.
- if ($cap['manageactivities'] OR $subscribable AND $subscribe AND !$subscribed AND $cansub) {
+ if ($cap['manageactivities'] || $subscribable && $subscribe && !$subscribed && $cansub) {
\mod_moodleoverflow\subscriptions::subscribe_user($USER->id, $moodleoverflow, $modulecontext, true);
} else {
\mod_moodleoverflow\subscriptions::unsubscribe_user($USER->id, $moodleoverflow, $modulecontext, true);
@@ -210,11 +210,11 @@
}
// Create an url to return the user back to.
- $url = new moodle_url('/mod/moodleoverflow/index.php', array('id' => $id));
+ $url = new moodle_url('/mod/moodleoverflow/index.php', array('id' => $id));
$returnto = moodleoverflow_go_back_to($url);
// Prepare the message to be displayed.
- $shortname = format_string($course->shortname, true, array('context' => context_course::instance($course->id)));
+ $shortname = format_string($course->shortname, true, array('context' => context_course::instance($course->id)));
$notification = \core\output\notification::NOTIFY_SUCCESS;
// Redirect the user depending on the subscription state.
@@ -232,7 +232,7 @@
foreach ($generalmoodleoverflows as $moodleoverflow) {
// Retrieve the contexts.
- $cm = $modinfo->instances['moodleoverflow'][$moodleoverflow->id];
+ $cm = $modinfo->instances['moodleoverflow'][$moodleoverflow->id];
$modulecontext = context_module::instance($cm->id);
// Count the discussions within the moodleoverflow.
@@ -243,7 +243,7 @@
// Check whether the tracking is disabled.
if ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_OFF) {
- $unreadlink = '-';
+ $unreadlink = '-';
$trackedlink = '-';
} else {
// The moodleoverflow can be tracked.
@@ -275,7 +275,7 @@
// Check whether the moodleoverflow instance can be tracked.
$isforced = $moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_FORCED;
- if ($isforced AND (get_config('moodleoverflow', 'allowforcedreadtracking'))) {
+ if ($isforced && (get_config('moodleoverflow', 'allowforcedreadtracking'))) {
// Tracking is set to forced.
// Define the string.
@@ -292,15 +292,15 @@
// Define the url the button is linked to.
$trackingurlparams = array('id' => $moodleoverflow->id, 'sesskey' => sesskey());
- $trackingurl = new moodle_url('/mod/moodleoverflow/tracking.php', $trackingurlparams);
+ $trackingurl = new moodle_url('/mod/moodleoverflow/tracking.php', $trackingurlparams);
// Check whether the moodleoverflow instance is tracked.
if (!isset($untracked[$moodleoverflow->id])) {
$trackingparam = array('title' => $string['notrackmoodleoverflow']);
- $trackedlink = $OUTPUT->single_button($trackingurl, $string['yes'], 'post', $trackingparam);
+ $trackedlink = $OUTPUT->single_button($trackingurl, $string['yes'], 'post', $trackingparam);
} else {
$trackingparam = array('title' => $string['trackmoodleoverflow']);
- $trackedlink = $OUTPUT->single_button($trackingurl, $string['no'], 'post', $trackingparam);
+ $trackedlink = $OUTPUT->single_button($trackingurl, $string['no'], 'post', $trackingparam);
}
}
}
@@ -308,7 +308,7 @@
// Get information about the moodleoverflow instance.
$moodleoverflow->intro = shorten_text(format_module_intro('moodleoverflow', $moodleoverflow, $cm->id), 300);
- $moodleoverflowname = format_string($moodleoverflow->name, true);
+ $moodleoverflowname = format_string($moodleoverflow->name, true);
// Check if the context module is visible.
if ($cm->visible) {
@@ -320,7 +320,7 @@
// Create links to the moodleoverflow and the discussion.
$moodleoverflowlink = "id\" $style>"
. format_string($moodleoverflow->name, true) . '';
- $discussionlink = "id\" $style>" . $count . "";
+ $discussionlink = "id\" $style>" . $count . "";
// Create rows.
$row = array($moodleoverflowlink, $moodleoverflow->intro, $discussionlink);
@@ -336,10 +336,10 @@
// Set options to create the subscription link.
$suboptions = array(
- 'subscribed' => $string['yes'],
- 'unsubscribed' => $string['no'],
+ 'subscribed' => $string['yes'],
+ 'unsubscribed' => $string['no'],
'forcesubscribed' => $string['yes'],
- 'cantsubscribe' => '-',
+ 'cantsubscribe' => '-',
);
// Add the subscription link to the row.
@@ -359,13 +359,13 @@
echo $OUTPUT->header();
// Show the subscribe all option only to non-guest and enrolled users.
-if (!isguestuser() AND isloggedin() AND $showsubscriptioncolumns) {
+if (!isguestuser() && isloggedin() && $showsubscriptioncolumns) {
// Create a box.
echo $OUTPUT->box_start('subscription');
// Create the subscription link.
- $urlparams = array('id' => $course->id, 'sesskey' => sesskey());
+ $urlparams = array('id' => $course->id, 'sesskey' => sesskey());
$subscriptionlink = new moodle_url('/mod/moodleoverflow/index.php', $urlparams);
// Give the option to subscribe to all.
diff --git a/lang/en/moodleoverflow.php b/lang/en/moodleoverflow.php
index 93a9ccdd0e..c6b5dea5bb 100644
--- a/lang/en/moodleoverflow.php
+++ b/lang/en/moodleoverflow.php
@@ -28,261 +28,263 @@
defined('MOODLE_INTERNAL') || die();
// Default strings.
-$string['modulename'] = 'Moodleoverflow';
-$string['modulenameplural'] = 'Moodleoverflows';
-$string['modulename_help'] = 'The Moodleoverflow module enables participants to use a question-answer forum structure. The forum display is non-chronological as the ordering depends on collaborative voting instead of on time.';
+$string['modulename'] = 'Moodleoverflow';
+$string['modulenameplural'] = 'Moodleoverflows';
+$string['modulename_help'] = 'The Moodleoverflow module enables participants to use a question-answer forum structure. The forum display is non-chronological as the ordering depends on collaborative voting instead of on time.';
$string['moodleoverflowfieldset'] = 'Custom example fieldset';
-$string['moodleoverflowname'] = 'Moodleoverflow name';
-$string['moodleoverflow'] = 'Moodleoverflow';
-$string['pluginadministration'] = 'Moodleoverflow administration';
-$string['pluginname'] = 'Moodleoverflow';
+$string['moodleoverflowname'] = 'Moodleoverflow name';
+$string['moodleoverflow'] = 'Moodleoverflow';
+$string['pluginadministration'] = 'Moodleoverflow administration';
+$string['pluginname'] = 'Moodleoverflow';
// Capabilities.
-$string['moodleoverflow:addinstance'] = 'Add a new Moodleoverflow instance';
+$string['moodleoverflow:addinstance'] = 'Add a new Moodleoverflow instance';
$string['moodleoverflow:allowforcesubscribe'] = 'Allow forced subscription';
-$string['moodleoverflow:createattachment'] = 'Create attachments';
+$string['moodleoverflow:createattachment'] = 'Create attachments';
$string['moodleoverflow:managesubscriptions'] = 'Manage subscriptions';
-$string['moodleoverflow:ratehelpful'] = 'Mark a post as helpful';
-$string['moodleoverflow:ratepost'] = 'Rate a post';
-$string['moodleoverflow:viewanyrating'] = 'View ratings';
-$string['moodleoverflow:deleteanypost'] = 'Delete posts';
-$string['moodleoverflow:deleteownpost'] = 'Delete own posts';
-$string['moodleoverflow:editanypost'] = 'Edit posts';
-$string['moodleoverflow:startdiscussion'] = 'Start a discussion';
-$string['moodleoverflow:replypost'] = 'Reply in discussion';
-$string['moodleoverflow:viewdiscussion'] = 'View discussion';
-$string['moodleoverflow:view'] = 'View discussionlist';
-$string['moodleoverflow:marksolved'] = 'Mark a post as solution';
-$string['moodleoverflow:reviewpost'] = 'Review (approve or reject) posts';
-$string['nowallsubscribed'] = 'All forums in {$a} are subscribed.';
-$string['nowallunsubscribed'] = 'All forums in {$a} are unsubscribed.';
+$string['moodleoverflow:ratehelpful'] = 'Mark a post as helpful';
+$string['moodleoverflow:ratepost'] = 'Rate a post';
+$string['moodleoverflow:viewanyrating'] = 'View ratings';
+$string['moodleoverflow:deleteanypost'] = 'Delete posts';
+$string['moodleoverflow:deleteownpost'] = 'Delete own posts';
+$string['moodleoverflow:editanypost'] = 'Edit posts';
+$string['moodleoverflow:startdiscussion'] = 'Start a discussion';
+$string['moodleoverflow:replypost'] = 'Reply in discussion';
+$string['moodleoverflow:viewdiscussion'] = 'View discussion';
+$string['moodleoverflow:view'] = 'View discussionlist';
+$string['moodleoverflow:marksolved'] = 'Mark a post as solution';
+$string['moodleoverflow:reviewpost'] = 'Review (approve or reject) posts';
+$string['nowallsubscribed'] = 'All forums in {$a} are subscribed.';
+$string['nowallunsubscribed'] = 'All forums in {$a} are unsubscribed.';
+$string['moodleoverflow:movetopic'] = 'Move Topic to another Moddleoverflow Forum';
// Strings for the view.php.
$string['noviewdiscussionspermission'] = 'You do not have the permission to view discussions in this forum';
// Strings for the locallib.php.
-$string['addanewdiscussion'] = 'Add a new discussion topic';
-$string['nodiscussions'] = 'There are no discussion topics yet in this forum.';
-$string['headerdiscussion'] = 'Discussion';
-$string['headerstartedby'] = 'Started by';
-$string['headerreplies'] = 'Replies';
-$string['headerlastpost'] = 'Last published post';
-$string['headerunread'] = 'Unread';
-$string['headervotes'] = 'Votes';
-$string['headerstatus'] = 'Status';
-$string['markallread'] = 'Mark read';
-$string['markallread'] = 'Mark all posts in this discussion read.';
-$string['delete'] = 'Delete';
-$string['parent'] = 'Show parent';
-$string['markread'] = 'Mark read';
-$string['markunread'] = 'Mark unread';
-$string['permalink'] = 'Permalink';
-$string['postbyuser'] = '{$a->post} by {$a->user}';
-$string['bynameondate'] = 'by {$a->name} ({$a->rating}) - {$a->date}';
+$string['addanewdiscussion'] = 'Add a new discussion topic';
+$string['nodiscussions'] = 'There are no discussion topics yet in this forum.';
+$string['headerdiscussion'] = 'Discussion';
+$string['headerstartedby'] = 'Started by';
+$string['headerreplies'] = 'Replies';
+$string['headerlastpost'] = 'Last published post';
+$string['headerunread'] = 'Unread';
+$string['headervotes'] = 'Votes';
+$string['headerstatus'] = 'Status';
+$string['markallread'] = 'Mark read';
+$string['markallread'] = 'Mark all posts in this discussion read.';
+$string['delete'] = 'Delete';
+$string['parent'] = 'Show parent';
+$string['markread'] = 'Mark read';
+$string['markunread'] = 'Mark unread';
+$string['permalink'] = 'Permalink';
+$string['postbyuser'] = '{$a->post} by {$a->user}';
+$string['bynameondate'] = 'by {$a->name} ({$a->rating}) - {$a->date}';
$string['bynameondatenorating'] = 'by {$a->name} - {$a->date}';
-$string['deletesure'] = 'Are you sure you want to delete this post?';
-$string['deletesureplural'] = 'Are you sure you want to delete this post and all replies? ({$a} posts)';
+$string['deletesure'] = 'Are you sure you want to delete this post?';
+$string['deletesureplural'] = 'Are you sure you want to delete this post and all replies? ({$a} posts)';
// Strings for the settings.php.
-$string['configmanydiscussions'] = 'Maximum number of discussions shown in a Moodleoverflow instance per page';
-$string['manydiscussions'] = 'Discussions per page';
-$string['maxattachmentsize'] = 'Maximum attachment size';
-$string['maxattachmentsize_help'] = 'This setting specifies the largest size of file that can be attached to a forum post.';
-$string['configmaxbytes'] = 'Default maximum size for all forum attachments on the site (subject to course limits and other local settings)';
-$string['maxattachments'] = 'Maximum number of attachments';
-$string['maxattachments_help'] = 'This setting specifies the maximum number of files that can be attached to a forum post.';
-$string['configmaxattachments'] = 'Default maximum number of attachments allowed per post.';
-$string['maxeditingtime'] = 'Maximum amount of time during which a post can be edited by its owner (sec)';
-$string['configmaxeditingtime'] = 'Default maximum seconds are 3600 (= one hour). Regarding editing posts, please also consider the "Review possible after" setting for moderated forums.';
-$string['configoldpostdays'] = 'Number of days old any post is considered read.';
-$string['oldpostdays'] = 'Read after days';
-$string['trackingoff'] = 'Off';
-$string['trackingon'] = 'Forced';
-$string['trackingoptional'] = 'Optional';
-$string['trackingtype'] = 'Read tracking';
-$string['configtrackingtype'] = 'Default setting for read tracking.';
-$string['trackmoodleoverflow'] = 'Track unread posts';
+$string['configmanydiscussions'] = 'Maximum number of discussions shown in a Moodleoverflow instance per page';
+$string['manydiscussions'] = 'Discussions per page';
+$string['maxattachmentsize'] = 'Maximum attachment size';
+$string['maxattachmentsize_help'] = 'This setting specifies the largest size of file that can be attached to a forum post.';
+$string['configmaxbytes'] = 'Default maximum size for all forum attachments on the site (subject to course limits and other local settings)';
+$string['maxattachments'] = 'Maximum number of attachments';
+$string['maxattachments_help'] = 'This setting specifies the maximum number of files that can be attached to a forum post.';
+$string['configmaxattachments'] = 'Default maximum number of attachments allowed per post.';
+$string['maxeditingtime'] = 'Maximum amount of time during which a post can be edited by its owner (sec)';
+$string['configmaxeditingtime'] = 'Default maximum seconds are 3600 (= one hour). Regarding editing posts, please also consider the "Review possible after" setting for moderated forums.';
+$string['configoldpostdays'] = 'Number of days old any post is considered read.';
+$string['oldpostdays'] = 'Read after days';
+$string['trackingoff'] = 'Off';
+$string['trackingon'] = 'Forced';
+$string['trackingoptional'] = 'Optional';
+$string['trackingtype'] = 'Read tracking';
+$string['configtrackingtype'] = 'Default setting for read tracking.';
+$string['trackmoodleoverflow'] = 'Track unread posts';
$string['configtrackmoodleoverflow'] = 'Set to \'yes\' if you want to track read/unread for each user.';
-$string['forcedreadtracking'] = 'Allow forced read tracking';
-$string['configforcedreadtracking'] = 'Allows Moodleoverflows to be set to forced read tracking. Will result in decreased performance for some users, particularly on courses with many moodleoverflows and posts. When off, any moodleoverflows previously set to Forced are treated as optional.';
-$string['cleanreadtime'] = 'Mark old posts as read hour';
-$string['configcleanreadtime'] = 'The hour of the day to clean old posts from the \'read\' table.';
-$string['allowdisablerating'] = 'Allow teachers to disable rating and reputation';
-$string['configallowdisablerating'] = 'Set to \'yes\' if you want to give teachers the ability to disable rating and reputation.';
-
-$string['votescalevote'] = 'Reputation: Vote.';
-$string['configvotescalevote'] = 'The amount of reputation voting gives.';
-$string['votescaledownvote'] = 'Reputation: Downvote';
-$string['configvotescaledownvote'] = 'The amount of reputation a downvote for your post gives.';
-$string['votescaleupvote'] = 'Reputation: Upvote';
-$string['configvotescaleupvote'] = 'The amount of reputation an upvote for your post gives.';
-$string['votescalesolved'] = 'Reputation: Solution';
-$string['configvotescalesolved'] = 'The amount of reputation a mark as solution on your post gives.';
-$string['votescalehelpful'] = 'Reputation: Helpful';
-$string['configvotescalehelpful'] = 'The amount of reputation a mark as helpful on your post gives.';
-$string['reputationnotnegative'] = 'Reputation just positive?';
+$string['forcedreadtracking'] = 'Allow forced read tracking';
+$string['configforcedreadtracking'] = 'Allows Moodleoverflows to be set to forced read tracking. Will result in decreased performance for some users, particularly on courses with many moodleoverflows and posts. When off, any moodleoverflows previously set to Forced are treated as optional.';
+$string['cleanreadtime'] = 'Mark old posts as read hour';
+$string['configcleanreadtime'] = 'The hour of the day to clean old posts from the \'read\' table.';
+$string['allowdisablerating'] = 'Allow teachers to disable rating and reputation';
+$string['configallowdisablerating'] = 'Set to \'yes\' if you want to give teachers the ability to disable rating and reputation.';
+
+$string['votescalevote'] = 'Reputation: Vote.';
+$string['configvotescalevote'] = 'The amount of reputation voting gives.';
+$string['votescaledownvote'] = 'Reputation: Downvote';
+$string['configvotescaledownvote'] = 'The amount of reputation a downvote for your post gives.';
+$string['votescaleupvote'] = 'Reputation: Upvote';
+$string['configvotescaleupvote'] = 'The amount of reputation an upvote for your post gives.';
+$string['votescalesolved'] = 'Reputation: Solution';
+$string['configvotescalesolved'] = 'The amount of reputation a mark as solution on your post gives.';
+$string['votescalehelpful'] = 'Reputation: Helpful';
+$string['configvotescalehelpful'] = 'The amount of reputation a mark as helpful on your post gives.';
+$string['reputationnotnegative'] = 'Reputation just positive?';
$string['configreputationnotnegative'] = 'Prohibits the users reputation being negative.';
-$string['allowcoursereputation'] = 'Sum reputation within a course.';
+$string['allowcoursereputation'] = 'Sum reputation within a course.';
$string['configallowcoursereputation'] = 'Allow to sum the reputation of all instances of the current course?';
-$string['maxmailingtime'] = 'Maximal mailing time';
+$string['maxmailingtime'] = 'Maximal mailing time';
$string['configmaxmailingtime'] = 'Posts older than this number of hours will not be mailed to the users. This will help to avoid problems where the cron has not been running for a long time.';
// Strings for the post.php.
$string['invalidmoodleoverflowid'] = 'Forum ID was incorrect';
-$string['invalidparentpostid'] = 'Parent post ID was incorrect';
-$string['notpartofdiscussion'] = 'This post is not part of a discussion!';
-$string['noguestpost'] = 'Sorry, guests are not allowed to post.';
-$string['nopostmoodleoverflow'] = 'Sorry, you are not allowed to post to this forum.';
-$string['yourreply'] = 'Your reply';
-$string['re'] = 'Re:';
-$string['invalidpostid'] = 'Invalid post ID - {$a}';
-$string['cannotfindparentpost'] = 'Could not find top parent of post {$a}';
-$string['edit'] = 'Edit';
-$string['cannotreply'] = 'You cannot reply to this post';
-$string['cannotcreatediscussion'] = 'Could not create new discussion';
-$string['couldnotadd'] = 'Could not add your post due to an unknown error';
-$string['postaddedsuccess'] = 'Your post was successfully added.';
-$string['postaddedtimeleft'] = 'You have {$a} to edit it if you want to make any changes.';
-$string['cannotupdatepost'] = 'You can not update this post';
-$string['couldnotupdate'] = 'Could not update your post due to an unknown error';
-$string['editedpostupdated'] = '{$a}\'s post was updated';
-$string['postupdated'] = 'Your post was updated';
-$string['editedby'] = 'Edited by {$a->name} - original submission {$a->date}';
-$string['cannotdeletepost'] = 'You can\'t delete this post!';
-$string['couldnotdeletereplies'] = 'Sorry, that cannot be deleted as people have already responded to it';
-$string['errorwhiledelete'] = 'An error occurred while deleting record.';
-$string['couldnotdeletereplies'] = 'Sorry, that cannot be deleted as people have already responded to it';
+$string['invalidparentpostid'] = 'Parent post ID was incorrect';
+$string['notpartofdiscussion'] = 'This post is not part of a discussion!';
+$string['noguestpost'] = 'Sorry, guests are not allowed to post.';
+$string['nopostmoodleoverflow'] = 'Sorry, you are not allowed to post to this forum.';
+$string['yourreply'] = 'Your reply';
+$string['re'] = 'Re:';
+$string['invalidpostid'] = 'Invalid post ID - {$a}';
+$string['cannotfindparentpost'] = 'Could not find top parent of post {$a}';
+$string['edit'] = 'Edit';
+$string['cannotreply'] = 'You cannot reply to this post';
+$string['cannotcreatediscussion'] = 'Could not create new discussion';
+$string['couldnotadd'] = 'Could not add your post due to an unknown error';
+$string['postaddedsuccess'] = 'Your post was successfully added.';
+$string['postaddedtimeleft'] = 'You have {$a} to edit it if you want to make any changes.';
+$string['cannotupdatepost'] = 'You can not update this post';
+$string['couldnotupdate'] = 'Could not update your post due to an unknown error';
+$string['editedpostupdated'] = '{$a}\'s post was updated';
+$string['postupdated'] = 'Your post was updated';
+$string['editedby'] = 'Edited by {$a->name} - original submission {$a->date}';
+$string['cannotdeletepost'] = 'You can\'t delete this post!';
+$string['couldnotdeletereplies'] = 'Sorry, that cannot be deleted as people have already responded to it';
+$string['errorwhiledelete'] = 'An error occurred while deleting record.';
+$string['couldnotdeletereplies'] = 'Sorry, that cannot be deleted as people have already responded to it';
// Strings for the classes/mod_form.php.
-$string['subject'] = 'Subject';
-$string['reply'] = 'Comment';
-$string['replyfirst'] = 'Answer';
-$string['message'] = 'Message';
-$string['discussionsubscription'] = 'Discussion subscription';
+$string['subject'] = 'Subject';
+$string['reply'] = 'Comment';
+$string['replyfirst'] = 'Answer';
+$string['message'] = 'Message';
+$string['discussionsubscription'] = 'Discussion subscription';
$string['discussionsubscription_help'] = 'Subscribing to a discussion means you will receive notifications of new posts to that discussion.';
-$string['posttomoodleoverflow'] = 'Post to forum';
+$string['posttomoodleoverflow'] = 'Post to forum';
$string['posts'] = 'Posts';
-$string['erroremptysubject'] = 'Post subject cannot be empty.';
-$string['erroremptymessage'] = 'Post message cannot be empty';
-$string['yournewtopic'] = 'Your new discussion topic';
+$string['erroremptysubject'] = 'Post subject cannot be empty.';
+$string['erroremptymessage'] = 'Post message cannot be empty';
+$string['yournewtopic'] = 'Your new discussion topic';
// Strings for the classes/ratings.php.
-$string['postnotexist'] = 'Requested post does not exist';
-$string['noratemoodleoverflow'] = 'Sorry, you are not allowed to vote in this forum.';
-$string['configallowratingchange'] = 'Can a user change its ratings?';
-$string['allowratingchange'] = 'Allow rating changes';
+$string['postnotexist'] = 'Requested post does not exist';
+$string['noratemoodleoverflow'] = 'Sorry, you are not allowed to vote in this forum.';
+$string['configallowratingchange'] = 'Can a user change its ratings?';
+$string['allowratingchange'] = 'Allow rating changes';
$string['configpreferteachersmark'] = 'The answer marked as solution by a course owner are prioritized over the answer marked as helpful by the starter of the discussion.';
-$string['preferteachersmark'] = 'Prefer course owners\' marks?';
-$string['noratingchangeallowed'] = 'You are not allowed to change your ratings.';
-$string['invalidratingid'] = 'The submitted rating is neither an upvote nor a downvote.';
-$string['notstartuser'] = 'Only the user who started the discussion can mark an answer as helpful.';
-$string['notteacher'] = 'Only course owners can do this.';
-$string['ratingtoold'] = 'Ratings can only be changed within 30 minutes after the first vote. ';
+$string['preferteachersmark'] = 'Prefer course owners\' marks?';
+$string['noratingchangeallowed'] = 'You are not allowed to change your ratings.';
+$string['invalidratingid'] = 'The submitted rating is neither an upvote nor a downvote.';
+$string['notstartuser'] = 'Only the user who started the discussion can mark an answer as helpful.';
+$string['notteacher'] = 'Only course owners can do this.';
+$string['ratingtoold'] = 'Ratings can only be changed within 30 minutes after the first vote. ';
// Strings for the discussion.php.
-$string['invaliddiscussionid'] = 'Discussion ID was incorrect';
-$string['notexists'] = 'Discussion no longer exists';
-$string['discussionname'] = 'Discussion name';
-$string['discussionlocked'] = 'This discussion has been locked so you can no longer reply to it.';
-$string['hiddenmoodleoverflowpost'] = 'Hidden forum post';
+$string['invaliddiscussionid'] = 'Discussion ID was incorrect';
+$string['notexists'] = 'Discussion no longer exists';
+$string['discussionname'] = 'Discussion name';
+$string['discussionlocked'] = 'This discussion has been locked so you can no longer reply to it.';
+$string['hiddenmoodleoverflowpost'] = 'Hidden forum post';
$string['moodleoverflowsubjecthidden'] = 'Subject (hidden)';
-$string['moodleoverflowauthorhidden'] = 'Author (hidden)';
-$string['moodleoverflowbodyhidden'] = 'This post cannot be viewed by you, probably because you have not posted in the discussion, the maximum editing time hasn\'t passed yet, the discussion has not started or the discussion has expired.';
-$string['addanewreply'] = 'Add a new answer';
-$string['ratingfailed'] = 'Rating failed. Try again.';
-$string['rateownpost'] = 'You cannot rate your own post.';
-$string['marksolved'] = 'Mark as solution';
-$string['marknotsolved'] = 'Remove solution mark';
-$string['markhelpful'] = 'Mark as Helpful';
-$string['marknothelpful'] = 'Not Helpful';
-$string['answer'] = '{$a} Answer';
-$string['answers'] = '{$a} Answers';
+$string['moodleoverflowauthorhidden'] = 'Author (hidden)';
+$string['moodleoverflowbodyhidden'] = 'This post cannot be viewed by you, probably because you have not posted in the discussion, the maximum editing time hasn\'t passed yet, the discussion has not started or the discussion has expired.';
+$string['addanewreply'] = 'Add a new answer';
+$string['ratingfailed'] = 'Rating failed. Try again.';
+$string['rateownpost'] = 'You cannot rate your own post.';
+$string['marksolved'] = 'Mark as solution';
+$string['marknotsolved'] = 'Remove solution mark';
+$string['markhelpful'] = 'Mark as Helpful';
+$string['marknothelpful'] = 'Not Helpful';
+$string['answer'] = '{$a} Answer';
+$string['answers'] = '{$a} Answers';
// Strings for the readtracking.php.
-$string['markreadfailed'] = 'A post of the discussion could not be marked as read.';
-$string['markdiscussionreadsuccessful'] = 'The discussion has been marked as read.';
+$string['markreadfailed'] = 'A post of the discussion could not be marked as read.';
+$string['markdiscussionreadsuccessful'] = 'The discussion has been marked as read.';
$string['markmoodleoverflowreadsuccessful'] = 'All posts have been marked as read.';
-$string['noguesttracking'] = 'Sorry, guests are not allowed to set tracking options.';
+$string['noguesttracking'] = 'Sorry, guests are not allowed to set tracking options.';
// OTHER.
-$string['messageprovider:posts'] = 'Notification of new posts';
-$string['unknownerror'] = 'This is not expected to happen.';
-$string['crontask'] = 'Moodleoverflow maintenance jobs';
-$string['taskcleanreadrecords'] = 'Moodleoverflow maintenance job to clean old read records';
-$string['tasksendmails'] = 'Moodleoverflow maintenance job to send mails';
-$string['nopermissiontosubscribe'] = 'You do not have the permission to view subscribers';
-$string['subscribeenrolledonly'] = 'Sorry, only enrolled users are allowed to subscribe to post notifications.';
-$string['everyonecannowchoose'] = 'Everyone can now choose to be subscribed';
-$string['noonecansubscribenow'] = 'Subscriptions are now disallowed';
-$string['invalidforcesubscribe'] = 'Invalid force subscription mode';
-$string['nownotsubscribed'] = '{$a->name} will NOT be notified of new posts in \'{$a->moodleoverflow}\'';
-$string['cannotunsubscribe'] = 'Could not unsubscribe you from that forum';
-$string['discussionnownotsubscribed'] = '{$a->name} will NOT be notified of new posts in \'{$a->discussion}\' of \'{$a->moodleoverflow}\'';
-$string['disallowsubscribe'] = 'Subscriptions not allowed';
-$string['noviewdiscussionspermission'] = 'You do not have the permission to view discussions in this forum';
-$string['nowsubscribed'] = '{$a->name} will be notified of new posts in \'{$a->moodleoverflow}\'';
-$string['discussionnowsubscribed'] = '{$a->name} will be notified of new posts in \'{$a->discussion}\' of \'{$a->moodleoverflow}\'';
-$string['unsubscribe'] = 'Unsubscribe from this forum';
-$string['subscribe'] = 'Subscribe to this forum';
+$string['messageprovider:posts'] = 'Notification of new posts';
+$string['unknownerror'] = 'This is not expected to happen.';
+$string['crontask'] = 'Moodleoverflow maintenance jobs';
+$string['taskcleanreadrecords'] = 'Moodleoverflow maintenance job to clean old read records';
+$string['tasksendmails'] = 'Moodleoverflow maintenance job to send mails';
+$string['tasksenddailymail'] = 'Moodleoverflow job to send a daily mail of unread post';
+$string['nopermissiontosubscribe'] = 'You do not have the permission to view subscribers';
+$string['subscribeenrolledonly'] = 'Sorry, only enrolled users are allowed to subscribe to post notifications.';
+$string['everyonecannowchoose'] = 'Everyone can now choose to be subscribed';
+$string['noonecansubscribenow'] = 'Subscriptions are now disallowed';
+$string['invalidforcesubscribe'] = 'Invalid force subscription mode';
+$string['nownotsubscribed'] = '{$a->name} will NOT be notified of new posts in \'{$a->moodleoverflow}\'';
+$string['cannotunsubscribe'] = 'Could not unsubscribe you from that forum';
+$string['discussionnownotsubscribed'] = '{$a->name} will NOT be notified of new posts in \'{$a->discussion}\' of \'{$a->moodleoverflow}\'';
+$string['disallowsubscribe'] = 'Subscriptions not allowed';
+$string['noviewdiscussionspermission'] = 'You do not have the permission to view discussions in this forum';
+$string['nowsubscribed'] = '{$a->name} will be notified of new posts in \'{$a->moodleoverflow}\'';
+$string['discussionnowsubscribed'] = '{$a->name} will be notified of new posts in \'{$a->discussion}\' of \'{$a->moodleoverflow}\'';
+$string['unsubscribe'] = 'Unsubscribe from this forum';
+$string['subscribe'] = 'Subscribe to this forum';
$string['confirmunsubscribediscussion'] = 'Do you really want to unsubscribe from discussion \'{$a->discussion}\' in moodleoverflow \'{$a->moodleoverflow}\'?';
-$string['confirmunsubscribe'] = 'Do you really want to unsubscribe from moodleoverflow \'{$a}\'?';
-$string['confirmsubscribediscussion'] = 'Do you really want to subscribe to discussion \'{$a->discussion}\' in forum \'{$a->moodleoverflow}\'?';
-$string['confirmsubscribe'] = 'Do you really want to subscribe to forum \'{$a}\'?';
-$string['postmailsubject'] = '{$a->courseshortname}: {$a->subject}';
-$string['smallmessage'] = '{$a->user} posted in {$a->moodleoverflowname}';
-$string['moodleoverflows'] = 'Moodleoverflows';
-$string['postmailinfolink'] = 'This is a copy of a message posted in {$a->coursename}.
+$string['confirmunsubscribe'] = 'Do you really want to unsubscribe from moodleoverflow \'{$a}\'?';
+$string['confirmsubscribediscussion'] = 'Do you really want to subscribe to discussion \'{$a->discussion}\' in forum \'{$a->moodleoverflow}\'?';
+$string['confirmsubscribe'] = 'Do you really want to subscribe to forum \'{$a}\'?';
+$string['postmailsubject'] = '{$a->courseshortname}: {$a->subject}';
+$string['smallmessage'] = '{$a->user} posted in {$a->moodleoverflowname}';
+$string['moodleoverflows'] = 'Moodleoverflows';
+$string['postmailinfolink'] = 'This is a copy of a message posted in {$a->coursename}.
To reply click on this link: {$a->replylink}';
-$string['unsubscribelink'] = 'Unsubscribe from this forum: {$a}';
-$string['unsubscribediscussionlink'] = 'Unsubscribe from this discussion: {$a}';
-$string['postincontext'] = 'See this post in context';
-$string['unsubscribediscussion'] = 'Unsubscribe from this discussion';
-$string['nownottracking'] = '{$a->name} is no longer tracking \'{$a->moodleoverflow}\'.';
-$string['nowtracking'] = '{$a->name} is now tracking \'{$a->moodleoverflow}\'.';
-$string['cannottrack'] = 'Could not stop tracking that forum';
-$string['notrackmoodleoverflow'] = 'Don\'t track unread posts';
-$string['trackmoodleoverflow'] = 'Track unread posts';
-$string['discussions'] = 'Discussions';
-$string['subscribed'] = 'Subscribed';
-$string['unreadposts'] = 'Unread posts';
-$string['unreadpostsnumber'] = '{$a} unread posts';
-$string['unreadpostsone'] = '1 unread post';
-$string['tracking'] = 'Track';
-$string['allsubscribe'] = 'Subscribe to all forums';
-$string['allunsubscribe'] = 'Unsubscribe from all forums';
-$string['generalmoodleoverflows'] = 'Forums in this course';
-$string['subscribestart'] = 'Send me notifications of new posts in this forum';
-$string['subscribestop'] = 'I don\'t want to be notified of new posts in this forum';
-$string['everyoneisnowsubscribed'] = 'Everyone is now subscribed to this forum';
-$string['everyoneissubscribed'] = 'Everyone is subscribed to this forum';
-$string['mailindexlink'] = 'Change your forum preferences: {$a}';
-$string['gotoindex'] = 'Manage preferences';
-$string['areaattachment'] = 'Attachments';
-$string['areapost'] = 'Messages';
+$string['unsubscribelink'] = 'Unsubscribe from this forum: {$a}';
+$string['unsubscribediscussionlink'] = 'Unsubscribe from this discussion: {$a}';
+$string['postincontext'] = 'See this post in context';
+$string['unsubscribediscussion'] = 'Unsubscribe from this discussion';
+$string['nownottracking'] = '{$a->name} is no longer tracking \'{$a->moodleoverflow}\'.';
+$string['nowtracking'] = '{$a->name} is now tracking \'{$a->moodleoverflow}\'.';
+$string['cannottrack'] = 'Could not stop tracking that forum';
+$string['notrackmoodleoverflow'] = 'Don\'t track unread posts';
+$string['trackmoodleoverflow'] = 'Track unread posts';
+$string['discussions'] = 'Discussions';
+$string['subscribed'] = 'Subscribed';
+$string['unreadposts'] = 'Unread posts';
+$string['unreadpostsnumber'] = '{$a} unread posts';
+$string['unreadpostsone'] = '1 unread post';
+$string['tracking'] = 'Track';
+$string['allsubscribe'] = 'Subscribe to all forums';
+$string['allunsubscribe'] = 'Unsubscribe from all forums';
+$string['generalmoodleoverflows'] = 'Forums in this course';
+$string['subscribestart'] = 'Send me notifications of new posts in this forum';
+$string['subscribestop'] = 'I don\'t want to be notified of new posts in this forum';
+$string['everyoneisnowsubscribed'] = 'Everyone is now subscribed to this forum';
+$string['everyoneissubscribed'] = 'Everyone is subscribed to this forum';
+$string['mailindexlink'] = 'Change your forum preferences: {$a}';
+$string['gotoindex'] = 'Manage preferences';
+$string['areaattachment'] = 'Attachments';
+$string['areapost'] = 'Messages';
// EVENTS.
-$string['eventdiscussioncreated'] = 'Discussion created';
-$string['eventdiscussiondeleted'] = 'Discussion deleted';
-$string['eventdiscussionviewed'] = 'Discussion viewed';
-$string['eventratingcreated'] = 'Rating created';
-$string['eventratingupdated'] = 'Rating updated';
-$string['eventratingdeleted'] = 'Rating deleted';
-$string['eventpostcreated'] = 'Post created';
-$string['eventpostupdated'] = 'Post updated';
-$string['eventpostdeleted'] = 'Post deleted';
+$string['eventdiscussioncreated'] = 'Discussion created';
+$string['eventdiscussiondeleted'] = 'Discussion deleted';
+$string['eventdiscussionviewed'] = 'Discussion viewed';
+$string['eventratingcreated'] = 'Rating created';
+$string['eventratingupdated'] = 'Rating updated';
+$string['eventratingdeleted'] = 'Rating deleted';
+$string['eventpostcreated'] = 'Post created';
+$string['eventpostupdated'] = 'Post updated';
+$string['eventpostdeleted'] = 'Post deleted';
$string['eventdiscussionsubscriptioncreated'] = 'Discussion subscription created';
$string['eventdiscussionsubscriptiondeleted'] = 'Discussion subscription deleted';
-$string['eventsubscriptioncreated'] = 'Subscription created';
-$string['eventsubscriptiondeleted'] = 'Subscription deleted';
-$string['eventreadtrackingdisabled'] = 'Read tracking disabled';
-$string['eventreadtrackingenabled'] = 'Read tracking enabled';
+$string['eventsubscriptioncreated'] = 'Subscription created';
+$string['eventsubscriptiondeleted'] = 'Subscription deleted';
+$string['eventreadtrackingdisabled'] = 'Read tracking disabled';
+$string['eventreadtrackingenabled'] = 'Read tracking enabled';
-$string['subscriptiontrackingheader'] = 'Subscription and tracking';
-$string['subscriptionmode'] = 'Subscription mode';
-$string['subscriptionmode_help'] = 'When a participant is subscribed to a forum it means they will receive forum post notifications. There are 4 subscription mode options:
+$string['subscriptiontrackingheader'] = 'Subscription and tracking';
+$string['subscriptionmode'] = 'Subscription mode';
+$string['subscriptionmode_help'] = 'When a participant is subscribed to a forum it means they will receive forum post notifications. There are 4 subscription mode options:
* Optional subscription - Participants can choose whether to be subscribed
* Forced subscription - Everyone is subscribed and cannot unsubscribe
@@ -290,47 +292,47 @@
* Subscription disabled - Subscriptions are not allowed
Note: Any subscription mode changes will only affect users who enrol in the course in the future, and not existing users.';
-$string['subscriptionoptional'] = 'Optional subscription';
-$string['subscriptionforced'] = 'Forced subscription';
-$string['subscriptionauto'] = 'Auto subscription';
-$string['subscriptiondisabled'] = 'Subscription disabled';
-$string['switchtooptional'] = 'If you switch to the optional subscription, all currently subscribed users will be unsubscribed from this forum!';
-$string['switchtoauto'] = 'If you switch to the auto subscription, all enrolled users will be subscribed to this forum!';
-$string['trackingoff'] = 'Off';
-$string['trackingon'] = 'Forced';
-$string['trackingoptional'] = 'Optional';
-$string['trackingtype'] = 'Read tracking';
-$string['trackingtype_help'] = 'Read tracking enables participants to easily check which posts they have not yet seen by highlighting any new posts.
+$string['subscriptionoptional'] = 'Optional subscription';
+$string['subscriptionforced'] = 'Forced subscription';
+$string['subscriptionauto'] = 'Auto subscription';
+$string['subscriptiondisabled'] = 'Subscription disabled';
+$string['switchtooptional'] = 'If you switch to the optional subscription, all currently subscribed users will be unsubscribed from this forum!';
+$string['switchtoauto'] = 'If you switch to the auto subscription, all enrolled users will be subscribed to this forum!';
+$string['trackingoff'] = 'Off';
+$string['trackingon'] = 'Forced';
+$string['trackingoptional'] = 'Optional';
+$string['trackingtype'] = 'Read tracking';
+$string['trackingtype_help'] = 'Read tracking enables participants to easily check which posts they have not yet seen by highlighting any new posts.
If set to optional, tracking is turned on by default but participants can turn tracking off.
If \'Allow forced read tracking\' is enabled in the site administration, then a further option is available - forced. This means that tracking is always on.';
-$string['ratingheading'] = 'Rating and reputation';
-$string['starterrating'] = 'Helpful';
-$string['teacherrating'] = 'Solution';
-$string['ratingpreference'] = 'Display first';
-$string['ratingpreference_help'] = 'Answers can be marked as solution and helpful. This option decides which of these will be pinned as the first answer of the discussion. There are 2 options:
+$string['ratingheading'] = 'Rating and reputation';
+$string['starterrating'] = 'Helpful';
+$string['teacherrating'] = 'Solution';
+$string['ratingpreference'] = 'Display first';
+$string['ratingpreference_help'] = 'Answers can be marked as solution and helpful. This option decides which of these will be pinned as the first answer of the discussion. There are 2 options:
* Heplful - A topic starter\'s helpful mark will be pinned at the top of the discussion
* Solved - A teacher\'s solution mark will be pinned at the top of the discussion';
-$string['allowrating'] = 'Allow post ratings?';
-$string['allowrating_help'] = 'If set to yes, users can up or downvote a post to give the reader an idea of how helpful the post was to other people. If set to no, ratings will be disabled.';
-$string['allowreputation'] = 'Allow user reputation?';
-$string['allowreputation_help'] = 'If set to yes, the users can gain or lose reputation depending on other users voting on their posts. If set to no, user reputation will be disabled.';
-$string['allownegativereputation'] = 'Allow negative reputation?';
+$string['allowrating'] = 'Allow post ratings?';
+$string['allowrating_help'] = 'If set to yes, users can up or downvote a post to give the reader an idea of how helpful the post was to other people. If set to no, ratings will be disabled.';
+$string['allowreputation'] = 'Allow user reputation?';
+$string['allowreputation_help'] = 'If set to yes, the users can gain or lose reputation depending on other users voting on their posts. If set to no, user reputation will be disabled.';
+$string['allownegativereputation'] = 'Allow negative reputation?';
$string['allownegativereputation_help'] = 'If set to yes, the users reputation within a course or within a module can be negative. If set to no, the reputation will stop to decrease at zero.';
-$string['coursewidereputation'] = 'Cross module reputation?';
-$string['coursewidereputation_help'] = 'If set to yes, the users reputations of all moodleoverflow modules in this course will be summed.';
-$string['clicktounsubscribe'] = 'You are subscribed to this discussion. Click to unsubscribe.';
-$string['clicktosubscribe'] = 'You are not subscribed to this discussion. Click to subscribe.';
-$string['attachment'] = 'Attachment';
-$string['attachments'] = 'Attachments';
-$string['attachment_help'] = 'You can optionally attach one or more files to a forum post. If you attach an image, it will be displayed after the message.';
+$string['coursewidereputation'] = 'Cross module reputation?';
+$string['coursewidereputation_help'] = 'If set to yes, the users reputations of all moodleoverflow modules in this course will be summed.';
+$string['clicktounsubscribe'] = 'You are subscribed to this discussion. Click to unsubscribe.';
+$string['clicktosubscribe'] = 'You are not subscribed to this discussion. Click to subscribe.';
+$string['attachment'] = 'Attachment';
+$string['attachments'] = 'Attachments';
+$string['attachment_help'] = 'You can optionally attach one or more files to a forum post. If you attach an image, it will be displayed after the message.';
// Templates.
$string['helpfulanswer'] = 'The question owner accepted this as the best answer.';
-$string['solvedanswer'] = 'This post is marked as solution.';
-$string['bestanswer'] = 'The question owner and a course owner accepted this as the best answer.';
+$string['solvedanswer'] = 'This post is marked as solution.';
+$string['bestanswer'] = 'The question owner and a course owner accepted this as the best answer.';
$string['reputation'] = 'Reputation';
$string['upvote'] = 'Upvote';
$string['upvotenotchangeable'] = 'Upvote (not changeable)';
@@ -338,6 +340,8 @@
$string['downvote'] = 'Downvote';
$string['downvotenotchangeable'] = 'Downvote (not changeable)';
$string['nodownvote'] = 'No downvote';
+$string['movetopicicon'] = 'The icon to move a topic';
+$string['movetopictoforum'] = 'The icon to move a selected topic to another forum';
// Privacy.
$string['privacy:metadata:core_files'] = 'Moodleoverflow stores files which have been uploaded by the user to form part of a forum post.';
@@ -453,3 +457,8 @@
$string['your_post_was_rejected'] = 'Your post was rejected.';
$string['your_post_was_rejected_with_reason'] = 'Your post was rejected with the following reason:';
$string['original_post'] = 'Original post';
+
+
+// Daily mail message.
+$string['digestunreadpost'] = 'Course: {$a->linktocourse} -> {$a->linktoforum}, Topic: {$a->linktodiscussion} has {$a->unreadposts} unread posts.';
+
diff --git a/lib.php b/lib.php
index 757242d05d..ddecdf49a6 100644
--- a/lib.php
+++ b/lib.php
@@ -86,7 +86,7 @@
/**
* Returns the information on whether the module supports a feature.
*
- * See {@link plugin_supports()} for more info.
+ * See {plugin_supports()} for more info.
*
* @param string $feature FEATURE_xx constant for requested feature
*
@@ -180,7 +180,7 @@ function moodleoverflow_update_instance(stdClass $moodleoverflow, mod_moodleover
global $DB;
$moodleoverflow->timemodified = time();
- $moodleoverflow->id = $moodleoverflow->instance;
+ $moodleoverflow->id = $moodleoverflow->instance;
// Get the old record.
$oldmoodleoverflow = $DB->get_record('moodleoverflow', array('id' => $moodleoverflow->id));
@@ -320,7 +320,7 @@ function moodleoverflow_delete_instance($id) {
* @return stdClass|null
*/
function moodleoverflow_user_outline($course, $user, $mod, $moodleoverflow) {
- $return = new stdClass();
+ $return = new stdClass();
$return->time = 0;
$return->info = '';
@@ -359,7 +359,7 @@ function moodleoverflow_get_extra_capabilities() {
* Returns the lists of all browsable file areas within the given module context.
*
* The file area 'intro' for the activity introduction field is added automatically
- * by {@link file_browser::get_file_info_context_module()}
+ * by { file_browser::get_file_info_context_module()}
*
* @param stdClass $course
* @param stdClass $cm
@@ -370,7 +370,7 @@ function moodleoverflow_get_extra_capabilities() {
function moodleoverflow_get_file_areas($course, $cm, $context) {
return array(
'attachment' => get_string('areaattachment', 'mod_moodleoverflow'),
- 'post' => get_string('areapost', 'mod_moodleoverflow'),
+ 'post' => get_string('areapost', 'mod_moodleoverflow'),
);
}
@@ -440,10 +440,10 @@ function moodleoverflow_pluginfile($course, $cm, $context, $filearea, array $arg
return false;
}
- $fs = get_file_storage();
+ $fs = get_file_storage();
$relativepath = implode('/', $args);
- $fullpath = "/$context->id/mod_moodleoverflow/$filearea/$postid/$relativepath";
- if (!$file = $fs->get_file_by_hash(sha1($fullpath)) or $file->is_directory()) {
+ $fullpath = "/$context->id/mod_moodleoverflow/$filearea/$postid/$relativepath";
+ if (!$file = $fs->get_file_by_hash(sha1($fullpath))||$file->is_directory()) {
return false;
}
@@ -451,7 +451,7 @@ function moodleoverflow_pluginfile($course, $cm, $context, $filearea, array $arg
if ($discussion->groupid > 0) {
$groupmode = groups_get_activity_groupmode($cm, $course);
if ($groupmode == SEPARATEGROUPS) {
- if (!groups_is_member($discussion->groupid) and !has_capability('moodle/site:accessallgroups', $context)) {
+ if (!groups_is_member($discussion->groupid) && !has_capability('moodle/site:accessallgroups', $context)) {
return false;
}
}
@@ -484,25 +484,25 @@ function moodleoverflow_extend_settings_navigation(settings_navigation $settings
$moodleoverflow = $DB->get_record('moodleoverflow', array('id' => $PAGE->cm->instance));
// Check if the user can subscribe to the instance.
- $enrolled = is_enrolled($PAGE->cm->context, $USER, '', false);
- $activeenrolled = is_enrolled($PAGE->cm->context, $USER, '', true);
- $canmanage = has_capability('mod/moodleoverflow:managesubscriptions', $PAGE->cm->context);
+ $enrolled = is_enrolled($PAGE->cm->context, $USER, '', false);
+ $activeenrolled = is_enrolled($PAGE->cm->context, $USER, '', true);
+ $canmanage = has_capability('mod/moodleoverflow:managesubscriptions', $PAGE->cm->context);
$forcesubscribed = \mod_moodleoverflow\subscriptions::is_forcesubscribed($moodleoverflow);
- $subscdisabled = \mod_moodleoverflow\subscriptions::subscription_disabled($moodleoverflow);
- $cansubscribe = $activeenrolled && (!$subscdisabled || $canmanage) &&
+ $subscdisabled = \mod_moodleoverflow\subscriptions::subscription_disabled($moodleoverflow);
+ $cansubscribe = $activeenrolled && (!$subscdisabled || $canmanage) &&
!($forcesubscribed && has_capability('mod/moodleoverflow:allowforcesubscribe', $PAGE->cm->context));
- $cantrack = \mod_moodleoverflow\readtracking::moodleoverflow_can_track_moodleoverflows($moodleoverflow);
+ $cantrack = \mod_moodleoverflow\readtracking::moodleoverflow_can_track_moodleoverflows($moodleoverflow);
// Display a link to the index.
- if ($enrolled AND $activeenrolled) {
+ if ($enrolled && $activeenrolled) {
// Generate the text of the link.
$linktext = get_string('gotoindex', 'moodleoverflow');
// Generate the link.
- $url = '/mod/moodleoverflow/index.php';
+ $url = '/mod/moodleoverflow/index.php';
$params = array('id' => $moodleoverflow->course);
- $link = new moodle_url($url, $params);
+ $link = new moodle_url($url, $params);
// Add the link to the menu.
$moodleoverflownode->add($linktext, $link, navigation_node::TYPE_SETTING);
@@ -525,15 +525,15 @@ function moodleoverflow_extend_settings_navigation(settings_navigation $settings
}
// Display a link to enable or disable readtracking.
- if ($enrolled AND $cantrack) {
+ if ($enrolled && $cantrack) {
// Check some basic capabilities.
- $isoptional = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_OPTIONAL);
+ $isoptional = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_OPTIONAL);
$forceallowed = get_config('moodleoverflow', 'allowforcedreadtracking');
- $isforced = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_FORCED);
+ $isforced = ($moodleoverflow->trackingtype == MOODLEOVERFLOW_TRACKING_FORCED);
// Check whether the readtracking state can be changed.
- if ($isoptional OR (!$forceallowed AND $isforced)) {
+ if ($isoptional || (!$forceallowed && $isforced)) {
// Generate the text of the link depending on the current state.
$istracked = \mod_moodleoverflow\readtracking::moodleoverflow_is_tracked($moodleoverflow);
@@ -544,9 +544,9 @@ function moodleoverflow_extend_settings_navigation(settings_navigation $settings
}
// Generate the link.
- $url = '/mod/moodleoverflow/tracking.php';
+ $url = '/mod/moodleoverflow/tracking.php';
$params = array('id' => $moodleoverflow->id, 'sesskey' => sesskey());
- $link = new moodle_url($url, $params);
+ $link = new moodle_url($url, $params);
// Add the link to the menu.
$moodleoverflownode->add($linktext, $link, navigation_node::TYPE_SETTING);
@@ -568,18 +568,18 @@ function moodleoverflow_get_context($moodleoverflowid, $context = null) {
global $PAGE;
// If the context does not exist, find the context.
- if (!$context OR !($context instanceof context_module)) {
+ if (!$context || !($context instanceof context_module)) {
// Try to take current page context to save on DB query.
- if ($PAGE->cm AND $PAGE->cm->modname === 'moodleoverflow' AND $PAGE->cm->instance == $moodleoverflowid
- AND $PAGE->context->contextlevel == CONTEXT_MODULE AND $PAGE->context->instanceid == $PAGE->cm->id
+ if ($PAGE->cm && $PAGE->cm->modname === 'moodleoverflow' && $PAGE->cm->instance == $moodleoverflowid
+ && $PAGE->context->contextlevel == CONTEXT_MODULE && $PAGE->context->instanceid == $PAGE->cm->id
) {
$context = $PAGE->context;
} else {
// Get the context via the coursemodule.
- $cm = get_coursemodule_from_instance('moodleoverflow', $moodleoverflowid);
+ $cm = get_coursemodule_from_instance('moodleoverflow', $moodleoverflowid);
$context = \context_module::instance($cm->id);
}
}
@@ -604,24 +604,24 @@ function moodleoverflow_send_mails() {
$textout = $PAGE->get_renderer('mod_moodleoverflow', 'email', 'textemail');
// Initiate the arrays that are saving the users that are subscribed to posts that needs sending.
- $users = array();
+ $users = array();
$userscount = 0; // Count($users) is slow. This avoids using this.
// Status arrays.
- $mailcount = array();
+ $mailcount = array();
$errorcount = array();
// Cache arrays.
- $discussions = array();
+ $discussions = array();
$moodleoverflows = array();
- $courses = array();
- $coursemodules = array();
+ $courses = array();
+ $coursemodules = array();
$subscribedusers = array();
// Posts older than x days will not be mailed.
// This will avoid problems with the cron not beeing ran for a long time.
- $timenow = time();
- $endtime = $timenow - get_config('moodleoverflow', 'maxeditingtime');
+ $timenow = time();
+ $endtime = $timenow - get_config('moodleoverflow', 'maxeditingtime');
$starttime = $endtime - (get_config('moodleoverflow', 'maxmailingtime') * 60 * 60);
// Retrieve all unmailed posts.
@@ -708,7 +708,7 @@ function moodleoverflow_send_mails() {
$modulecontext = context_module::instance($coursemodules[$moodleoverflowid]->id);
// Retrieve all subscribed users.
- $mid = $moodleoverflows[$moodleoverflowid];
+ $mid = $moodleoverflows[$moodleoverflowid];
$subusers = \mod_moodleoverflow\subscriptions::get_subscribed_users($mid, $modulecontext, 'u.*', true);
if ($subusers) {
@@ -729,35 +729,32 @@ function moodleoverflow_send_mails() {
}
// Initiate the count of the mails send and errors.
- $mailcount[$postid] = 0;
+ $mailcount[$postid] = 0;
$errorcount[$postid] = 0;
}
}
// Send mails to the users with information about the posts.
- if ($users AND $posts) {
-
+ if ($users && $posts) {
// Send one mail to every user.
foreach ($users as $userto) {
-
// Terminate if the process takes more time then two minutes.
core_php_time_limit::raise(120);
// Tracing information.
mtrace('Processing user ' . $userto->id);
-
// Initiate the user caches to save memory.
- $userto = clone($userto);
+ $userto = clone($userto);
$userto->ciewfullnames = array();
- $userto->canpost = array();
- $userto->markposts = array();
+ $userto->canpost = array();
+ $userto->markposts = array();
// Cache the capabilities of the user.
cron_setup_user($userto);
// Reset the caches.
foreach ($coursemodules as $moodleoverflowid => $unused) {
- $coursemodules[$moodleoverflowid]->cache = new stdClass();
+ $coursemodules[$moodleoverflowid]->cache = new stdClass();
$coursemodules[$moodleoverflowid]->cache->caps = array();
unset($coursemodules[$moodleoverflowid]->uservisible);
}
@@ -766,20 +763,48 @@ function moodleoverflow_send_mails() {
foreach ($posts as $postid => $post) {
// Initiate variables for the post.
- $discussion = $discussions[$post->discussion];
+ $discussion = $discussions[$post->discussion];
$moodleoverflow = $moodleoverflows[$discussion->moodleoverflow];
- $course = $courses[$moodleoverflow->course];
+ $course = $courses[$moodleoverflow->course];
$cm =& $coursemodules[$moodleoverflow->id];
+ // Check if user wants a resume.
+ // in this case: make a new dataset in "moodleoverflow_mail_info" to save the posts data.
+ // Dataset from moodleoverflow_mail_info will be send later in a mail.
+ $usermailsetting = $userto->maildigest;
+ if ($usermailsetting != 0) {
+ $dataobject = new stdClass();
+ $dataobject->userid = $userto->id;
+ $dataobject->courseid = $course->id;
+ $dataobject->forumid = $moodleoverflow->id;
+ $dataobject->forumdiscussionid = $discussion->id;
+ $record = $DB->get_record('moodleoverflow_mail_info',
+ array('userid' => $dataobject->userid,
+ 'courseid' => $dataobject->courseid,
+ 'forumid' => $dataobject->forumid,
+ 'forumdiscussionid' => $dataobject->forumdiscussionid),
+ 'numberofposts, id');
+ if (is_object($record)) {
+ $dataset = $record;
+ $dataobject->numberofposts = $dataset->numberofposts + 1;
+ $dataobject->id = $dataset->id;
+ $DB->update_record('moodleoverflow_mail_info', $dataobject);
+ } else {
+ $dataobject->numberofposts = 1;
+ $DB->insert_record('moodleoverflow_mail_info', $dataobject);
+ }
+ continue;
+ }
+
// Check whether the user is subscribed.
if (!isset($subscribedusers[$moodleoverflow->id][$userto->id])) {
continue;
}
// Check whether the user is subscribed to the discussion.
- $iscm = $coursemodules[$moodleoverflow->id];
- $uid = $userto->id;
- $did = $post->discussion;
+ $iscm = $coursemodules[$moodleoverflow->id];
+ $uid = $userto->id;
+ $did = $post->discussion;
$issubscribed = \mod_moodleoverflow\subscriptions::is_subscribed($uid, $moodleoverflow, $modulecontext, $did);
if (!$issubscribed) {
continue;
@@ -787,7 +812,7 @@ function moodleoverflow_send_mails() {
// Check whether the user unsubscribed to the discussion after it was created.
$subnow = \mod_moodleoverflow\subscriptions::fetch_discussion_subscription($moodleoverflow->id, $userto->id);
- if ($subnow AND isset($subnow[$post->discussion]) AND ($subnow[$post->discussion] > $post->created)) {
+ if ($subnow && isset($subnow[$post->discussion]) && ($subnow[$post->discussion] > $post->created)) {
continue;
}
@@ -846,12 +871,12 @@ function moodleoverflow_send_mails() {
// Sent the email.
// Preapare to actually send the post now. Build up the content.
- $cleanname = str_replace('"', "'", strip_tags(format_string($moodleoverflow->name)));
+ $cleanname = str_replace('"', "'", strip_tags(format_string($moodleoverflow->name)));
$coursecontext = context_course::instance($course->id);
- $shortname = format_string($course->shortname, true, array('context' => $coursecontext));
+ $shortname = format_string($course->shortname, true, array('context' => $coursecontext));
// Define a header to make mails easier to track.
- $emailmessageid = generate_email_messageid('moodlemoodleoverflow' . $moodleoverflow->id);
+ $emailmessageid = generate_email_messageid('moodlemoodleoverflow' . $moodleoverflow->id);
$userfrom->customheaders = array(
'List-Id: "' . $cleanname . '" ' . $emailmessageid,
'List-Help: ' . $CFG->wwwroot . '/mod/moodleoverflow/view.php?m=' . $moodleoverflow->id,
@@ -895,22 +920,22 @@ function moodleoverflow_send_mails() {
}
// Retrieve needed variables for the mail.
- $var = new \stdClass();
- $var->subject = $data->get_subject();
+ $var = new \stdClass();
+ $var->subject = $data->get_subject();
$var->moodleoverflowname = $cleanname;
- $var->sitefullname = format_string($site->fullname);
- $var->siteshortname = format_string($site->shortname);
- $var->courseidnumber = $data->get_courseidnumber();
- $var->coursefullname = $data->get_coursefullname();
- $var->courseshortname = $data->get_coursename();
- $postsubject = html_to_text(get_string('postmailsubject', 'moodleoverflow', $var), 0);
- $rootid = generate_email_messageid(hash('sha256', $discussion->firstpost . 'to' . $userto->id));
+ $var->sitefullname = format_string($site->fullname);
+ $var->siteshortname = format_string($site->shortname);
+ $var->courseidnumber = $data->get_courseidnumber();
+ $var->coursefullname = $data->get_coursefullname();
+ $var->courseshortname = $data->get_coursename();
+ $postsubject = html_to_text(get_string('postmailsubject', 'moodleoverflow', $var), 0);
+ $rootid = generate_email_messageid(hash('sha256', $discussion->firstpost . 'to' . $userto->id));
// Check whether the post is a reply.
if ($post->parent) {
// Add a reply header.
- $parentid = generate_email_messageid(hash('sha256', $post->parent . 'to' . $userto->id));
+ $parentid = generate_email_messageid(hash('sha256', $post->parent . 'to' . $userto->id));
$userfrom->customheaders[] = "In-Reply-To: $parentid";
// Comments need a reference to the starting post as well.
@@ -925,34 +950,34 @@ function moodleoverflow_send_mails() {
mtrace('Sending ', '');
// Create the message event.
- $eventdata = new \core\message\message();
- $eventdata->courseid = $course->id;
- $eventdata->component = 'mod_moodleoverflow';
- $eventdata->name = 'posts';
- $eventdata->userfrom = $userfrom;
- $eventdata->userto = $userto;
- $eventdata->subject = $postsubject;
- $eventdata->fullmessage = $textout->render($data);
+ $eventdata = new \core\message\message();
+ $eventdata->courseid = $course->id;
+ $eventdata->component = 'mod_moodleoverflow';
+ $eventdata->name = 'posts';
+ $eventdata->userfrom = $userfrom;
+ $eventdata->userto = $userto;
+ $eventdata->subject = $postsubject;
+ $eventdata->fullmessage = $textout->render($data);
$eventdata->fullmessageformat = FORMAT_PLAIN;
- $eventdata->fullmessagehtml = $htmlout->render($data);
- $eventdata->notification = 1;
+ $eventdata->fullmessagehtml = $htmlout->render($data);
+ $eventdata->notification = 1;
// Initiate another message array.
- $small = new \stdClass();
- $small->user = fullname($userfrom);
- $formatedstring = format_string($moodleoverflow->name, true);
+ $small = new \stdClass();
+ $small->user = fullname($userfrom);
+ $formatedstring = format_string($moodleoverflow->name, true);
$small->moodleoverflowname = "$shortname: " . $formatedstring . ": " . $discussion->name;
- $small->message = $post->message;
+ $small->message = $post->message;
// Make sure the language is correct.
- $usertol = $userto->lang;
+ $usertol = $userto->lang;
$eventdata->smallmessage = get_string_manager()->get_string('smallmessage', 'moodleoverflow', $small, $usertol);
// Generate the url to view the post.
- $url = '/mod/moodleoverflow/discussion.php';
- $params = array('d' => $discussion->id);
- $contexturl = new moodle_url($url, $params, 'p' . $post->id);
- $eventdata->contexturl = $contexturl->out();
+ $url = '/mod/moodleoverflow/discussion.php';
+ $params = array('d' => $discussion->id);
+ $contexturl = new moodle_url($url, $params, 'p' . $post->id);
+ $eventdata->contexturl = $contexturl->out();
$eventdata->contexturlname = $discussion->name;
// Actually send the message.
@@ -1008,9 +1033,9 @@ function moodleoverflow_get_unmailed_posts($starttime, $endtime) {
global $DB;
// Set params for the sql query.
- $params = array();
+ $params = array();
$params['ptimestart'] = $starttime;
- $params['ptimeend'] = $endtime;
+ $params['ptimeend'] = $endtime;
$pendingmail = MOODLEOVERFLOW_MAILED_PENDING;
$reviewsent = MOODLEOVERFLOW_MAILED_REVIEW_SUCCESS;
@@ -1040,11 +1065,11 @@ function moodleoverflow_mark_old_posts_as_mailed($endtime) {
$now = time();
// Define variables for the sql query.
- $params = array();
+ $params = array();
$params['mailedsuccess'] = MOODLEOVERFLOW_MAILED_SUCCESS;
$params['mailedreviewsent'] = MOODLEOVERFLOW_MAILED_REVIEW_SUCCESS;
- $params['now'] = $now;
- $params['endtime'] = $endtime;
+ $params['now'] = $now;
+ $params['endtime'] = $endtime;
$params['mailedpending'] = MOODLEOVERFLOW_MAILED_PENDING;
// Define the sql query.
@@ -1171,7 +1196,7 @@ function moodleoverflow_update_grades($moodleoverflow, $userid, $nullifnone = nu
moodleoverflow_grade_item_update($moodleoverflow, $grades);
- } else if ($userid and $nullifnone) {
+ } else if ($userid && $nullifnone) {
// Insert a grade with rawgrade = null. As described in Gradebook API.
$grade = new stdClass();
@@ -1207,8 +1232,8 @@ function moodleoverflow_grade_item_update($moodleoverflow, $grades=null) {
} else if ($moodleoverflow->grademaxgrade > 0) {
$params['gradetype'] = GRADE_TYPE_VALUE;
- $params['grademax'] = $moodleoverflow->grademaxgrade;
- $params['grademin'] = 0;
+ $params['grademax'] = $moodleoverflow->grademaxgrade;
+ $params['grademin'] = 0;
}
diff --git a/locallib.php b/locallib.php
index 86bce4be4a..3bb8fd54e5 100644
--- a/locallib.php
+++ b/locallib.php
@@ -178,9 +178,16 @@ function moodleoverflow_print_latest_discussions($moodleoverflow, $cm, $page = -
$markallread = null;
}
+ // Check wether the user can move a topic.
+ $canmovetopic = false;
+ if ((!is_guest($context, $USER) && isloggedin()) && has_capability('mod/moodleoverflow:movetopic', $context)) {
+ $canmovetopic = true;
+ }
+
// Check whether the user can subscribe to the discussion.
$cansubtodiscussion = false;
- if ((!is_guest($context, $USER) && isloggedin()) && has_capability('mod/moodleoverflow:viewdiscussion', $context)) {
+ if ((!is_guest($context, $USER) && isloggedin()) && has_capability('mod/moodleoverflow:viewdiscussion', $context)
+ && \mod_moodleoverflow\subscriptions::is_subscribable($moodleoverflow, $context)) {
$cansubtodiscussion = true;
}
@@ -239,7 +246,7 @@ function moodleoverflow_print_latest_discussions($moodleoverflow, $cm, $page = -
// Check if a single post was marked by the question owner and a teacher.
$statusboth = false;
- if ($statusstarter AND $statusteacher) {
+ if ($statusstarter && $statusteacher) {
if ($statusstarter->postid == $statusteacher->postid) {
$statusboth = true;
}
@@ -346,8 +353,11 @@ function moodleoverflow_print_latest_discussions($moodleoverflow, $cm, $page = -
], 'p' . $reviewinfo->first))->out(false);
}
- // Add all created data to an array.
+ // Build linktopopup to move a topic.
+ $linktopopup = $CFG->wwwroot . '/mod/moodleoverflow/view.php?id=' . $cm->id . '&movetopopup=' . $discussion->discussion;
+ $preparedarray[$i]['linktopopup'] = $linktopopup;
+ // Add all created data to an array.
$preparedarray[$i]['statusstarter'] = $statusstarter;
$preparedarray[$i]['statusteacher'] = $statusteacher;
$preparedarray[$i]['statusboth'] = $statusboth;
@@ -370,7 +380,8 @@ function moodleoverflow_print_latest_discussions($moodleoverflow, $cm, $page = -
$mustachedata->istracked = $istracked;
$mustachedata->markallread = $markallread;
$mustachedata->cansubtodiscussion = $cansubtodiscussion;
-
+ $mustachedata->canmovetopic = $canmovetopic;
+ $mustachedata->cannormoveorsub = ((!$canmovetopic) && (!$cansubtodiscussion));
// Print the template.
echo $renderer->render_discussion_list($mustachedata);
@@ -380,6 +391,46 @@ function moodleoverflow_print_latest_discussions($moodleoverflow, $cm, $page = -
}
}
+/**
+ * Prints a popup with a menu of other moodleoverflow in the course.
+ * Menu to move a topic to another moodleoverflow forum.
+ */
+function moodleoverflow_print_forum_list($course, $cm, $movetopopup) {
+ global $CFG, $DB, $PAGE;
+ $forumarray = array(array());
+ $currentforum = $DB->get_record('moodleoverflow_discussions', array('id' => $movetopopup), 'moodleoverflow');
+ $currentdiscussion = $DB->get_record('moodleoverflow_discussions', array('id' => $movetopopup), 'name');
+ $forums = $DB->get_records('moodleoverflow', array('course' => $course->id));
+ $amountforums = count($forums);
+
+ if ($amountforums > 1) {
+ // Write the moodleoverflow-names in an array.
+ $i = 0;
+ foreach ($forums as $forum) {
+ if ($forum->id == $currentforum->moodleoverflow) {
+ continue;
+ } else {
+ $forumarray[$i]['name'] = $forum->name;
+ $movetoforum = $CFG->wwwroot . '/mod/moodleoverflow/view.php?id=' . $cm->id . '&movetopopup='
+ . $movetopopup . '&movetoforum=' . $forum->id;
+ $forumarray[$i]['movetoforum'] = $movetoforum;
+ }
+ $i++;
+ }
+ $amountforums = true;
+ } else {
+ $amountforums = false;
+ }
+
+ // Build popup.
+ $renderer = $PAGE->get_renderer('mod_moodleoverflow');
+ $mustachedata = new stdClass();
+ $mustachedata->hasforums = $amountforums;
+ $mustachedata->forums = $forumarray;
+ $mustachedata->currentdiscussion = $currentdiscussion->name;
+ echo $renderer->render_forum_list($mustachedata);
+}
+
/**
* Returns an array of counts of replies for each discussion.
*
@@ -423,7 +474,7 @@ function moodleoverflow_count_discussion_replies($cm) {
function moodleoverflow_user_can_post_discussion($moodleoverflow, $cm = null, $context = null) {
// Guests an not-logged-in users can not psot.
- if (isguestuser() or !isloggedin()) {
+ if (isguestuser() || !isloggedin()) {
return false;
}
@@ -602,7 +653,7 @@ function moodleoverflow_user_can_see_post($moodleoverflow, $discussion, $post, $
}
// Get the postid if not set.
- if (!isset($post->id) AND isset($post->parent)) {
+ if (!isset($post->id) && isset($post->parent)) {
$post->id = $post->parent;
}
@@ -768,13 +819,13 @@ function moodleoverflow_add_discussion($discussion, $modulecontext, $userid = nu
// Mark the created post as read.
$cantrack = \mod_moodleoverflow\readtracking::moodleoverflow_can_track_moodleoverflows($moodleoverflow);
$istracked = \mod_moodleoverflow\readtracking::moodleoverflow_is_tracked($moodleoverflow);
- if ($cantrack AND $istracked) {
+ if ($cantrack && $istracked) {
\mod_moodleoverflow\readtracking::moodleoverflow_mark_post_read($post->userid, $post);
}
// Trigger event.
$params = array(
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
'objectid' => $post->discussion,
);
@@ -808,13 +859,12 @@ function moodleoverflow_go_back_to($default) {
/**
* Checks whether the user can reply to posts in a discussion.
*
- * @param stdClass $moodleoverflow The moodleoverflow object
- * @param stdClass $user The user object
- * @param object $cm
- * @param stdClass $course The course object
- * @param context_module $modulecontext The modules context
- *
+ * @param object $modulecontext
+ * @param object $posttoreplyto
+ * @param bool $considerreviewstatus
+ * @param int $userid
* @return bool Whether the user can reply
+ * @throws coding_exception
*/
function moodleoverflow_user_can_post($modulecontext, $posttoreplyto, $considerreviewstatus = true, $userid = null) {
global $USER;
@@ -844,7 +894,7 @@ function moodleoverflow_print_discussion($course, $cm, $moodleoverflow, $discuss
global $USER;
// Check if the current is the starter of the discussion.
- $ownpost = (isloggedin() AND ($USER->id == $post->userid));
+ $ownpost = (isloggedin() && ($USER->id == $post->userid));
// Fetch the modulecontext.
$modulecontext = context_module::instance($cm->id);
@@ -1018,25 +1068,28 @@ function moodleoverflow_get_all_discussion_posts($discussionid, $tracking, $modc
return $posts;
}
+
/**
* Prints a moodleoverflow post.
- *
- * @param stdClass $post The post object
- * @param stdClass $discussion The discussion object
- * @param stdClass $moodleoverflow The moodleoverflow object
- * @param object $cm
- * @param stdClass $course The course object
- * @param bool $ownpost Whether the post was submitted by this user
- * @param bool $link Whether there is a link to this post
- * @param string $footer A default footer for posts
- * @param string $highlight A word to highlight in the post
- * @param null $postisread Whether the post has been read
- * @param bool $dummyifcantsee Whether to display an empty dummy
- * @param bool $istracked Whether the discussion is tracked
- * @param bool $iscomment Whether to post is a comment
- * @param int $level Determines the level of a post
- *
- * @return null The output
+ * @param object $post
+ * @param object $discussion
+ * @param object $moodleoverflow
+ * @param object $cm
+ * @param object $course
+ * @param object $ownpost
+ * @param bool $link
+ * @param string $footer
+ * @param string $highlight
+ * @param bool $postisread
+ * @param bool $dummyifcantsee
+ * @param bool $istracked
+ * @param bool $iscomment
+ * @param array $usermapping
+ * @param int $level
+ * @return void|null
+ * @throws coding_exception
+ * @throws dml_exception
+ * @throws moodle_exception
*/
function moodleoverflow_print_post($post, $discussion, $moodleoverflow, $cm, $course,
$ownpost = false, $link = false,
@@ -1175,7 +1228,7 @@ function moodleoverflow_print_post($post, $discussion, $moodleoverflow, $cm, $co
// A teacher can mark an answer as solved.
$cap = $cm->cache->caps['mod/moodleoverflow:marksolved'];
- $canmarksolved = (($iscomment != $post->parent) AND !empty($post->parent) AND $cap);
+ $canmarksolved = (($iscomment != $post->parent) && !empty($post->parent) && $cap);
if ($canmarksolved) {
// When the post is already marked, remove the mark instead.
@@ -1193,9 +1246,9 @@ function moodleoverflow_print_post($post, $discussion, $moodleoverflow, $cm, $co
$age = time() - $post->created;
// Make a link to edit your own post within the given time and not already reviewed.
- if (($ownpost AND ($age < get_config('moodleoverflow', 'maxeditingtime')) &&
+ if (($ownpost && ($age < get_config('moodleoverflow', 'maxeditingtime')) &&
(!review::should_post_be_reviewed($post, $moodleoverflow) || !$post->reviewed))
- OR $cm->cache->caps['mod/moodleoverflow:editanypost']
+ || $cm->cache->caps['mod/moodleoverflow:editanypost']
) {
$editurl = new moodle_url('/mod/moodleoverflow/post.php', array('edit' => $post->id));
$commands[] = array('url' => $editurl, 'text' => $str->edit);
@@ -1205,7 +1258,7 @@ function moodleoverflow_print_post($post, $discussion, $moodleoverflow, $cm, $co
$old = ($age < get_config('moodleoverflow', 'maxeditingtime'));
$capone = $cm->cache->caps['mod/moodleoverflow:deleteownpost'];
$captwo = $cm->cache->caps['mod/moodleoverflow:deleteanypost'];
- if (($ownpost AND $old AND $capone) OR $captwo) {
+ if (($ownpost && $old && $capone) || $captwo) {
$link = '/mod/moodleoverflow/post.php';
$commands[] = array('url' => new moodle_url($link, array('delete' => $post->id)), 'text' => $str->delete);
}
@@ -1240,7 +1293,7 @@ function moodleoverflow_print_post($post, $discussion, $moodleoverflow, $cm, $co
$mustachedata->isread = false;
$mustachedata->isfirstunread = false;
$mustachedata->isfirstpost = false;
- $mustachedata->iscomment = (!empty($post->parent) AND ($iscomment == $post->parent));
+ $mustachedata->iscomment = (!empty($post->parent) && ($iscomment == $post->parent));
// Get the ratings.
$mustachedata->votes = $post->upvotes - $post->downvotes;
@@ -1393,7 +1446,7 @@ function moodleoverflow_print_post($post, $discussion, $moodleoverflow, $cm, $co
$mustachedata->footer = $footer;
// Mark the forum post as read.
- if ($istracked AND !$postisread) {
+ if ($istracked && !$postisread) {
\mod_moodleoverflow\readtracking::moodleoverflow_mark_post_read($USER->id, $post);
}
@@ -1412,6 +1465,7 @@ function moodleoverflow_print_post($post, $discussion, $moodleoverflow, $cm, $co
}
}
+
/**
* Prints all posts of the discussion in a nested form.
*
@@ -1423,11 +1477,14 @@ function moodleoverflow_print_post($post, $discussion, $moodleoverflow, $cm, $co
* @param bool $istracked Whether the user tracks the discussion
* @param array $posts Array of posts within the discussion
* @param bool $iscomment Whether the current post is a comment
- *
- * @return string The html output.
+ * @param array $usermapping
+ * @return string
+ * @throws coding_exception
+ * @throws dml_exception
+ * @throws moodle_exception
*/
function moodleoverflow_print_posts_nested($course, &$cm, $moodleoverflow, $discussion, $parent,
- $istracked, $posts, $iscomment = null, $usermapping = []) {
+ $istracked, $posts, $iscomment = null, $usermapping = []) {
global $USER;
// Prepare the output.
@@ -1568,11 +1625,10 @@ function moodleoverflow_add_attachment($post, $forum, $cm) {
/**
* Adds a new post in an existing discussion.
- *
* @param object $post The post object
- * @param context_module $modulecontext
- *
* @return bool|int The Id of the post if operation was successful
+ * @throws coding_exception
+ * @throws dml_exception
*/
function moodleoverflow_add_new_post($post) {
global $USER, $DB;
@@ -1612,7 +1668,7 @@ function moodleoverflow_add_new_post($post) {
// Mark the created post as read if the user is tracking the discussion.
$cantrack = \mod_moodleoverflow\readtracking::moodleoverflow_can_track_moodleoverflows($moodleoverflow);
$istracked = \mod_moodleoverflow\readtracking::moodleoverflow_is_tracked($moodleoverflow);
- if ($cantrack AND $istracked) {
+ if ($cantrack && $istracked) {
\mod_moodleoverflow\readtracking::moodleoverflow_mark_post_read($post->userid, $post);
}
@@ -1672,7 +1728,7 @@ function moodleoverflow_update_post($newpost) {
// Mark the edited post as read.
$cantrack = \mod_moodleoverflow\readtracking::moodleoverflow_can_track_moodleoverflows($moodleoverflow);
$istracked = \mod_moodleoverflow\readtracking::moodleoverflow_is_tracked($moodleoverflow);
- if ($cantrack AND $istracked) {
+ if ($cantrack && $istracked) {
\mod_moodleoverflow\readtracking::moodleoverflow_mark_post_read($USER->id, $post);
}
@@ -1760,7 +1816,7 @@ function moodleoverflow_delete_post($post, $deletechildren, $cm, $moodleoverflow
// Iterate through all children and delete them.
$childposts = $DB->get_records('moodleoverflow_posts', array('parent' => $post->id));
- if ($deletechildren AND $childposts) {
+ if ($deletechildren && $childposts) {
foreach ($childposts as $childpost) {
moodleoverflow_delete_post($childpost, true, $cm, $moodleoverflow);
}
@@ -1783,10 +1839,10 @@ function moodleoverflow_delete_post($post, $deletechildren, $cm, $moodleoverflow
// Trigger the post deletion event.
$params = array(
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
'objectid' => $post->id,
- 'other' => array(
- 'discussionid' => $post->discussion,
+ 'other' => array(
+ 'discussionid' => $post->discussion,
'moodleoverflowid' => $moodleoverflow->id
)
);
@@ -1940,7 +1996,7 @@ function moodleoverflow_count_discussions($moodleoverflow, $course) {
function moodleoverflow_update_user_grade($moodleoverflow, $postuserrating, $postinguser) {
// Check whether moodleoverflow object has the added params.
- if ($moodleoverflow->grademaxgrade > 0 and $moodleoverflow->gradescalefactor > 0) {
+ if ($moodleoverflow->grademaxgrade > 0 && $moodleoverflow->gradescalefactor > 0) {
moodleoverflow_update_user_grade_on_db($moodleoverflow, $postuserrating, $postinguser);
}
}
@@ -1995,7 +2051,7 @@ function moodleoverflow_update_all_grades_for_cm($moodleoverflowid) {
$moodleoverflow = $DB->get_record('moodleoverflow', array('id' => $moodleoverflowid));
// Check whether moodleoverflow object has the added params.
- if ($moodleoverflow->grademaxgrade > 0 and $moodleoverflow->gradescalefactor > 0) {
+ if ($moodleoverflow->grademaxgrade > 0 && $moodleoverflow->gradescalefactor > 0) {
// Get all users id.
$params = ['moodleoverflowid' => $moodleoverflowid, 'moodleoverflowid2' => $moodleoverflowid];
diff --git a/markposts.php b/markposts.php
index 230600dc04..abae184223 100644
--- a/markposts.php
+++ b/markposts.php
@@ -28,7 +28,7 @@
// Define the parameters.
$moodleoverflowid = required_param('m', PARAM_INT); // The moodleoverflowinstance to mark.
-$discussionid = optional_param('d', 0, PARAM_INT); // The discussion to mark.
+$discussionid = optional_param('d', 0, PARAM_INT); // The discussion to mark.
$returndiscussion = optional_param('return', 0, PARAM_INT); // The page to return to.
// Prepare the array that should be used to return to this page.
@@ -99,7 +99,7 @@
if (!empty($discussionid)) {
// Check if the discussion exists.
- $options = array('id' => $discussionid, 'moodleoverflow' => $moodleoverflow->id);
+ $options = array('id' => $discussionid, 'moodleoverflow' => $moodleoverflow->id);
$discussion = $DB->get_record('moodleoverflow_discussions', $options);
if (!$discussion) {
throw new moodle_exception('invaliddiscussionid', 'moodleoverflow');
@@ -111,13 +111,13 @@
// Display an error, if something failes.
$message = get_string('markreadfailed', 'moodleoverflow');
- $status = \core\output\notification::NOTIFY_ERROR;
+ $status = \core\output\notification::NOTIFY_ERROR;
} else {
// The discussion is successfully marked as read.
$message = get_string('markmoodleoverflowreadsuccessful', 'moodleoverflow');
- $status = \core\output\notification::NOTIFY_SUCCESS;
+ $status = \core\output\notification::NOTIFY_SUCCESS;
}
// Redirect the user.
@@ -131,13 +131,13 @@
// Display an error, if something fails.
$message = get_string('markreadfailed', 'moodleoverflow');
- $status = \core\output\notification::NOTIFY_ERROR;
+ $status = \core\output\notification::NOTIFY_ERROR;
} else {
// All posts of the instance have been marked as read.
$message = get_string('markdiscussionreadsuccessful', 'moodleoverflow');
- $status = \core\output\notification::NOTIFY_SUCCESS;
+ $status = \core\output\notification::NOTIFY_SUCCESS;
}
// Redirect the user back to the view.php.
diff --git a/mod_form.php b/mod_form.php
index c3b6274644..1cb85ae630 100644
--- a/mod_form.php
+++ b/mod_form.php
@@ -116,19 +116,19 @@ public function definition() {
$mform->setDefault('maxbytes', get_config('moodleoverflow', 'maxbytes'));
$choices = array(
- 0 => 0,
- 1 => 1,
- 2 => 2,
- 3 => 3,
- 4 => 4,
- 5 => 5,
- 6 => 6,
- 7 => 7,
- 8 => 8,
- 9 => 9,
- 10 => 10,
- 20 => 20,
- 50 => 50,
+ 0 => 0,
+ 1 => 1,
+ 2 => 2,
+ 3 => 3,
+ 4 => 4,
+ 5 => 5,
+ 6 => 6,
+ 7 => 7,
+ 8 => 8,
+ 9 => 9,
+ 10 => 10,
+ 20 => 20,
+ 50 => 50,
100 => 100
);
$mform->addElement('select', 'maxattachments', get_string('maxattachments', 'moodleoverflow'), $choices);
@@ -163,7 +163,7 @@ public function definition() {
// Choose the default tracking type.
$default = get_config('moodleoverflow', 'trackingtype');
- if ((!get_config('moodleoverflow', 'allowforcedreadtracking')) AND ($default == MOODLEOVERFLOW_TRACKING_FORCED)) {
+ if ((!get_config('moodleoverflow', 'allowforcedreadtracking')) && ($default == MOODLEOVERFLOW_TRACKING_FORCED)) {
$default = MOODLEOVERFLOW_TRACKING_OPTIONAL;
}
$mform->setDefault('trackingtype', $default);
diff --git a/post.php b/post.php
index 84858e016e..214eaec727 100644
--- a/post.php
+++ b/post.php
@@ -52,10 +52,10 @@
// Set the URL that should be used to return to this page.
$PAGE->set_url('/mod/moodleoverflow/post.php', array(
'moodleoverflow' => $moodleoverflow,
- 'reply' => $reply,
- 'edit' => $edit,
- 'delete' => $delete,
- 'confirm' => $confirm,
+ 'reply' => $reply,
+ 'edit' => $edit,
+ 'delete' => $delete,
+ 'confirm' => $confirm,
));
// These params will be passed as hidden variables later in the form.
@@ -65,7 +65,7 @@
$systemcontext = context_system::instance();
// Catch guests.
-if (!isloggedin() OR isguestuser()) {
+if (!isloggedin() || isguestuser()) {
// The user is starting a new discussion in a moodleoverflow instance.
if (!empty($moodleoverflow)) {
@@ -150,12 +150,12 @@
if (!moodleoverflow_user_can_post_discussion($moodleoverflow, $cm, $modulecontext)) {
// Catch unenrolled user.
- if (!isguestuser() AND !is_enrolled($coursecontext)) {
+ if (!isguestuser() && !is_enrolled($coursecontext)) {
if (enrol_selfenrol_available($course->id)) {
$SESSION->wantsurl = qualified_me();
$SESSION->enrolcancel = get_local_referer(false);
redirect(new moodle_url('/enrol/index.php', array(
- 'id' => $course->id,
+ 'id' => $course->id,
'returnurl' => '/mod/moodleoverflow/view.php?m=' . $moodleoverflow->id
)), get_string('youneedtoenrol'));
}
@@ -221,7 +221,7 @@
if (!moodleoverflow_user_can_post($modulecontext, $parent)) {
// Give the user the chance to enroll himself to the course.
- if (!isguestuser() AND !is_enrolled($coursecontext)) {
+ if (!isguestuser() && !is_enrolled($coursecontext)) {
$SESSION->wantsurl = qualified_me();
$SESSION->enrolcancel = get_local_referer(false);
redirect(new moodle_url('/enrol/index.php',
@@ -234,7 +234,7 @@
}
// Make sure the user can post here.
- if (!$cm->visible AND !has_capability('moodle/course:viewhiddenactivities', $modulecontext)) {
+ if (!$cm->visible && !has_capability('moodle/course:viewhiddenactivities', $modulecontext)) {
throw new moodle_exception('activityiscurrentlyhidden');
}
@@ -302,7 +302,7 @@
// Check if the post can be edited.
$beyondtime = ((time() - $post->created) > get_config('moodleoverflow', 'maxeditingtime'));
$alreadyreviewed = review::should_post_be_reviewed($post, $moodleoverflow) && $post->reviewed;
- if (($beyondtime || $alreadyreviewed) AND !has_capability('mod/moodleoverflow:editanypost', $modulecontext)) {
+ if (($beyondtime || $alreadyreviewed) && !has_capability('mod/moodleoverflow:editanypost', $modulecontext)) {
throw new moodle_exception('maxtimehaspassed', 'moodleoverflow', '',
format_time(get_config('moodleoverflow', 'maxeditingtime')));
}
@@ -363,7 +363,7 @@
// Check some capabilities.
$deleteownpost = has_capability('mod/moodleoverflow:deleteownpost', $modulecontext);
$deleteanypost = has_capability('mod/moodleoverflow:deleteanypost', $modulecontext);
- if (!(($post->userid == $USER->id AND $deleteownpost) OR $deleteanypost)) {
+ if (!(($post->userid == $USER->id && $deleteownpost) || $deleteanypost)) {
throw new moodle_exception('cannotdeletepost', 'moodleoverflow');
}
@@ -371,17 +371,17 @@
$replycount = moodleoverflow_count_replies($post, false);
// Has the user confirmed the deletion?
- if (!empty($confirm) AND confirm_sesskey()) {
+ if (!empty($confirm) && confirm_sesskey()) {
// Check if the user has the capability to delete the post.
$timepassed = time() - $post->created;
- if (($timepassed > get_config('moodleoverflow', 'maxeditingtime')) AND !$deleteanypost) {
+ if (($timepassed > get_config('moodleoverflow', 'maxeditingtime')) && !$deleteanypost) {
$url = new moodle_url('/mod/moodleoverflow/discussion.php', array('d' => $post->discussion));
throw new moodle_exception('cannotdeletepost', 'moodleoverflow', moodleoverflow_go_back_to($url));
}
// A normal user cannot delete his post if there are direct replies.
- if ($replycount AND !$deleteanypost) {
+ if ($replycount && !$deleteanypost) {
$url = new moodle_url('/mod/moodleoverflow/discussion.php', array('d' => $post->discussion));
throw new moodle_exception('couldnotdeletereplies', 'moodleoverflow', moodleoverflow_go_back_to($url));
} else {
@@ -394,7 +394,7 @@
// Trigger the discussion deleted event.
$params = array(
'objectid' => $discussion->id,
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
);
$event = \mod_moodleoverflow\event\discussion_deleted::create($params);
@@ -488,13 +488,13 @@
// Prepare the form.
$formarray = array(
- 'course' => $course,
- 'cm' => $cm,
- 'coursecontext' => $coursecontext,
- 'modulecontext' => $modulecontext,
+ 'course' => $course,
+ 'cm' => $cm,
+ 'coursecontext' => $coursecontext,
+ 'modulecontext' => $modulecontext,
'moodleoverflow' => $moodleoverflow,
- 'post' => $post,
- 'edit' => $edit,
+ 'post' => $post,
+ 'edit' => $edit,
);
$mformpost = new mod_moodleoverflow_post_form('post.php', $formarray, 'post', '', array('id' => 'mformmoodleoverflow'));
@@ -543,17 +543,17 @@
$param5 = (isset($discussion->id) ? array('discussion' => $discussion->id) : array());
$mformpost->set_data(array(
'attachments' => $draftitemid,
- 'general' => $heading,
- 'subject' => $subject,
- 'message' => array(
- 'text' => $postmessage,
+ 'general' => $heading,
+ 'subject' => $subject,
+ 'message' => array(
+ 'text' => $postmessage,
'format' => editors_get_preferred_format(),
'itemid' => $postid,
),
- 'userid' => $post->userid,
- 'parent' => $post->parent,
- 'discussion' => $post->discussion,
- 'course' => $course->id
+ 'userid' => $post->userid,
+ 'parent' => $post->parent,
+ 'discussion' => $post->discussion,
+ 'course' => $course->id
) + $pageparams + $param1 + $param2 + $param3 + $param4 + $param5);
// Is it canceled?
@@ -607,7 +607,7 @@
$replypost = has_capability('mod/moodleoverflow:replypost', $modulecontext);
$startdiscussion = has_capability('mod/moodleoverflow:startdiscussion', $modulecontext);
$ownpost = ($realpost->userid == $USER->id);
- if (!(($ownpost AND ($replypost OR $startdiscussion)) OR $editanypost)) {
+ if (!(($ownpost && ($replypost || $startdiscussion)) || $editanypost)) {
throw new moodle_exception('cannotupdatepost', 'moodleoverflow');
}
@@ -636,10 +636,10 @@
// Set some parameters.
$params = array(
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
'objectid' => $fromform->id,
- 'other' => array(
- 'discussionid' => $discussion->id,
+ 'other' => array(
+ 'discussionid' => $discussion->id,
'moodleoverflowid' => $moodleoverflow->id,
));
@@ -688,10 +688,10 @@
// Trigger post created event.
$params = array(
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
'objectid' => $fromform->id,
- 'other' => array(
- 'discussionid' => $discussion->id,
+ 'other' => array(
+ 'discussionid' => $discussion->id,
'moodleoverflowid' => $moodleoverflow->id,
));
$event = \mod_moodleoverflow\event\post_created::create($params);
@@ -732,9 +732,9 @@
} else { // The creation of the new discussion was successful.
$params = array(
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
'objectid' => $discussion->id,
- 'other' => array(
+ 'other' => array(
'moodleoverflowid' => $moodleoverflow->id,
)
);
@@ -743,7 +743,7 @@
// Trigger the discussion created event.
$params = array(
- 'context' => $modulecontext,
+ 'context' => $modulecontext,
'objectid' => $discussion->id,
);
$event = \mod_moodleoverflow\event\discussion_created::create($params);
diff --git a/renderer.php b/renderer.php
index 6790e97726..9fe475d555 100644
--- a/renderer.php
+++ b/renderer.php
@@ -47,6 +47,13 @@ public function render_discussion_list($data) {
return $this->render_from_template('mod_moodleoverflow/discussion_list', $data);
}
+ /**
+ * Display the forum list in the view.php if a discussion needs to be moved to another forum.
+ */
+ public function render_forum_list($data) {
+ return $this->render_from_template('mod_moodleoverflow/forum_list', $data);
+ }
+
/**
* Renders a dummy post for users that cannot see the post.
*
diff --git a/settings.php b/settings.php
index 04b1a6b4d4..02376d7b9f 100644
--- a/settings.php
+++ b/settings.php
@@ -50,10 +50,10 @@
// Default read tracking settings.
- $options = array();
+ $options = array();
$options[MOODLEOVERFLOW_TRACKING_OPTIONAL] = get_string('trackingoptional', 'moodleoverflow');
- $options[MOODLEOVERFLOW_TRACKING_OFF] = get_string('trackingoff', 'moodleoverflow');
- $options[MOODLEOVERFLOW_TRACKING_FORCED] = get_string('trackingon', 'moodleoverflow');
+ $options[MOODLEOVERFLOW_TRACKING_OFF] = get_string('trackingoff', 'moodleoverflow');
+ $options[MOODLEOVERFLOW_TRACKING_FORCED] = get_string('trackingon', 'moodleoverflow');
$settings->add(new admin_setting_configselect('moodleoverflow/trackingtype', get_string('trackingtype', 'moodleoverflow'),
get_string('configtrackingtype', 'moodleoverflow'), MOODLEOVERFLOW_TRACKING_OPTIONAL, $options));
diff --git a/subscribe.php b/subscribe.php
index 339c1a883e..71487cec6d 100644
--- a/subscribe.php
+++ b/subscribe.php
@@ -30,12 +30,12 @@
require_once(dirname(__FILE__) . '/locallib.php');
// Define required and optional params.
-$id = required_param('id', PARAM_INT); // The moodleoverflow to set subscription on.
-$mode = optional_param('mode', null, PARAM_INT); // The moodleoverflow's subscription mode.
-$user = optional_param('user', 0, PARAM_INT); // The userid of the user to subscribe, defaults to $USER.
+$id = required_param('id', PARAM_INT); // The moodleoverflow to set subscription on.
+$mode = optional_param('mode', null, PARAM_INT); // The moodleoverflow's subscription mode.
+$user = optional_param('user', 0, PARAM_INT); // The userid of the user to subscribe, defaults to $USER.
$discussionid = optional_param('d', null, PARAM_INT); // The discussionid to subscribe.
-$sesskey = optional_param('sesskey', null, PARAM_RAW);
-$returnurl = optional_param('returnurl', null, PARAM_RAW);
+$sesskey = optional_param('sesskey', null, PARAM_RAW);
+$returnurl = optional_param('returnurl', null, PARAM_RAW);
// Set the url to return to the same action.
$url = new moodle_url('/mod/moodleoverflow/subscribe.php', array('id' => $id));
@@ -60,20 +60,20 @@
// Get all necessary objects.
$moodleoverflow = $DB->get_record('moodleoverflow', array('id' => $id), '*', MUST_EXIST);
-$course = $DB->get_record('course', array('id' => $moodleoverflow->course), '*', MUST_EXIST);
-$cm = get_coursemodule_from_instance('moodleoverflow', $moodleoverflow->id, $course->id, false, MUST_EXIST);
-$context = context_module::instance($cm->id);
+$course = $DB->get_record('course', array('id' => $moodleoverflow->course), '*', MUST_EXIST);
+$cm = get_coursemodule_from_instance('moodleoverflow', $moodleoverflow->id, $course->id, false, MUST_EXIST);
+$context = context_module::instance($cm->id);
// Define variables.
-$notify = array();
-$notify['success'] = \core\output\notification::NOTIFY_SUCCESS;
-$notify['error'] = \core\output\notification::NOTIFY_ERROR;
-$strings = array();
-$strings['subscribeenrolledonly'] = get_string('subscribeenrolledonly', 'moodleoverflow');
-$strings['everyonecannowchoose'] = get_string('everyonecannowchoose', 'moodleoverflow');
+$notify = array();
+$notify['success'] = \core\output\notification::NOTIFY_SUCCESS;
+$notify['error'] = \core\output\notification::NOTIFY_ERROR;
+$strings = array();
+$strings['subscribeenrolledonly'] = get_string('subscribeenrolledonly', 'moodleoverflow');
+$strings['everyonecannowchoose'] = get_string('everyonecannowchoose', 'moodleoverflow');
$strings['everyoneisnowsubscribed'] = get_string('everyoneisnowsubscribed', 'moodleoverflow');
-$strings['noonecansubscribenow'] = get_string('noonecansubscribenow', 'moodleoverflow');
-$strings['invalidforcesubscribe'] = get_string('invalidforcesubscribe', 'moodleoverflow');
+$strings['noonecansubscribenow'] = get_string('noonecansubscribenow', 'moodleoverflow');
+$strings['invalidforcesubscribe'] = get_string('invalidforcesubscribe', 'moodleoverflow');
// Check if the user was requesting the subscription himself.
if ($user) {
@@ -104,7 +104,7 @@
// Guests, visitors and not enrolled people cannot subscribe.
$isenrolled = is_enrolled($context, $USER, '', true);
-if (is_null($mode) AND !$isenrolled) {
+if (is_null($mode) && !$isenrolled) {
// Prepare the output.
$PAGE->set_title($course->shortname);
@@ -114,7 +114,7 @@
if (isguestuser()) {
echo $OUTPUT->header();
$message = $strings['subscribeenrolledonly'] . '
' . get_string('liketologin');
- $url = new moodle_url('/mod/moodleoverflow/view.php', array('m' => $id));
+ $url = new moodle_url('/mod/moodleoverflow/view.php', array('m' => $id));
echo $OUTPUT->confirm($message, get_login_url(), $url);
echo $OUTPUT->footer;
exit;
@@ -127,14 +127,14 @@
// Create the url to redirect the user back to where he is coming from.
$urlindex = 'index.php?id=' . $course->id;
-$urlview = 'view.php?m=' . $id;
+$urlview = 'view.php?m=' . $id;
$returnto = optional_param('backtoindex', 0, PARAM_INT) ? $urlindex : $urlview;
if ($returnurl) {
$returnto = $returnurl;
}
// Change the general subscription state.
-if (!is_null($mode) AND has_capability('mod/moodleoverflow:managesubscriptions', $context)) {
+if (!is_null($mode) && has_capability('mod/moodleoverflow:managesubscriptions', $context)) {
require_sesskey();
// Set the new mode.
@@ -190,8 +190,8 @@
}
// Create an info object.
-$info = new stdClass();
-$info->name = fullname($user);
+$info = new stdClass();
+$info->name = fullname($user);
$info->moodleoverflow = format_string($moodleoverflow->name);
// Check if the user is subscribed to the moodleoverflow.
@@ -213,9 +213,9 @@
if ($discussionid) {
// Create a new info object.
- $info2 = new stdClass();
+ $info2 = new stdClass();
$info2->moodleoverflow = format_string($moodleoverflow->name);
- $info2->discussion = format_string($discussion->name);
+ $info2->discussion = format_string($discussion->name);
// Create a confirm statement.
$string = get_string('confirmunsubscribediscussion', 'moodleoverflow', $info2);
@@ -262,14 +262,14 @@
// The user needs to be subscribed.
// Check the capabilities.
- $capabilities = array();
+ $capabilities = array();
$capabilities['managesubscriptions'] = has_capability('mod/moodleoverflow:managesubscriptions', $context);
- $capabilities['viewdiscussion'] = has_capability('mod/moodleoverflow:viewdiscussion', $context);
+ $capabilities['viewdiscussion'] = has_capability('mod/moodleoverflow:viewdiscussion', $context);
require_sesskey();
// Check if subscriptionsare allowed.
$disabled = \mod_moodleoverflow\subscriptions::subscription_disabled($moodleoverflow);
- if ($disabled AND !$capabilities['managesubscriptions']) {
+ if ($disabled && !$capabilities['managesubscriptions']) {
throw new moodle_exception('disallowsubscribe', 'moodleoverflow', get_local_referer(false));
}
@@ -293,9 +293,9 @@
if ($discussionid) {
// Create a new info object.
- $info2 = new stdClass();
+ $info2 = new stdClass();
$info2->moodleoverflow = format_string($moodleoverflow->name);
- $info2->discussion = format_string($discussion->name);
+ $info2->discussion = format_string($discussion->name);
// Create a confirm dialog.
$string = get_string('confirmsubscribediscussion', 'moodleoverflow', $info2);
diff --git a/templates/discussion_list.mustache b/templates/discussion_list.mustache
index 4b80db755b..19902f0a78 100644
--- a/templates/discussion_list.mustache
+++ b/templates/discussion_list.mustache
@@ -202,10 +202,20 @@
{{^ lastpostusername }}-{{/ lastpostusername }}
{{ lastpostdate }}
+
+ {{^cannormoveorsub}}
+
+ {{#cansubtodiscussion}}
+ {{{discussionsubicon}}}
+ {{/cansubtodiscussion}}
+ {{#canmovetopic}}
+ {{#pix}} i/arrow-right, core, {{#str}}movetopicicon, moodleoverflow{{/str}} {{/pix}}
+ {{/canmovetopic}}
+ |
+ {{/cannormoveorsub}}
+
+
- {{#cansubtodiscussion}}
- {{{discussionsubicon}}} |
- {{/cansubtodiscussion}}
diff --git a/templates/forum_list.mustache b/templates/forum_list.mustache
new file mode 100644
index 0000000000..886c3f564e
--- /dev/null
+++ b/templates/forum_list.mustache
@@ -0,0 +1,53 @@
+{{!
+ 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 .
+}}
+{{!
+ @template mod_moodleoverflow/discussion_list
+
+ Moodleoverflow discussion_list template.
+ The purpose of this template is to render a list of discussions for the view.php.
+
+ Example context (json):
+ {
+ }
+}}
+{{#hasforums}}
+
+{{/hasforums}}
+{{^hasforums}}
+
+ Unfortunately, there is no other Moodleoverflow where you can move it.
+
+{{/hasforums}}
+
\ No newline at end of file
diff --git a/tests/dailymail_test.php b/tests/dailymail_test.php
new file mode 100644
index 0000000000..b6afe4d54e
--- /dev/null
+++ b/tests/dailymail_test.php
@@ -0,0 +1,211 @@
+.
+
+/**
+ * The module moodleoverflow tests.
+ *
+ * @package mod_moodleoverflow
+ * @copyright 2023 Tamaro Walter
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+namespace mod_moodleoverflow;
+
+use mod_moodleoverflow\task\send_mails;
+use mod_moodleoverflow\task\send_daily_mail;
+
+defined('MOODLE_INTERNAL') || die();
+
+global $CFG;
+require_once($CFG->dirroot . '/mod/moodleoverflow/lib.php');
+
+
+/**
+ * Class mod_moodleoverflow_dailymail_testcase.
+ *
+ * @package mod_moodleoverflow
+ * @copyright 2023 Tamaro Walter
+ * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class dailymail_test extends \advanced_testcase {
+
+ private $sink;
+ private $course;
+ private $user;
+ private $moodleoverflow;
+ private $coursemodule;
+ private $discussion;
+
+ /**
+ * Test setUp.
+ */
+ public function setUp(): void {
+ $this->resetAfterTest();
+ set_config('maxeditingtime', -10, 'moodleoverflow');
+
+ unset_config('noemailever');
+ $this->sink = $this->redirectEmails();
+ $this->preventResetByRollback();
+ $this->redirectMessages();
+ // Create a new course with a moodleoverflow forum.
+ $this->course = $this->getDataGenerator()->create_course();
+ $location = array('course' => $this->course->id, 'forcesubscribe' => MOODLEOVERFLOW_FORCESUBSCRIBE);
+ $this->moodleoverflow = $this->getDataGenerator()->create_module('moodleoverflow', $location);
+ $this->coursemodule = get_coursemodule_from_instance('moodleoverflow', $this->moodleoverflow->id);
+ }
+
+ /**
+ * Test tearDown.
+ */
+ public function tearDown(): void {
+ // Clear all caches.
+ \mod_moodleoverflow\subscriptions::reset_moodleoverflow_cache();
+ \mod_moodleoverflow\subscriptions::reset_discussion_cache();
+ }
+
+
+
+ // Helper functions.
+
+ /**
+ * Function that creates a new user, which adds a new discussion an post to the moodleoverflow.
+ * @param $maildigest The maildigest setting: 0 = off , 1 = on
+ */
+ public function helper_create_user_and_discussion($maildigest) {
+ // Create a user enrolled in the course as student.
+ $this->user = $this->getDataGenerator()->create_user(array('firstname' => 'Tamaro', 'maildigest' => $maildigest));
+ $this->getDataGenerator()->enrol_user($this->user->id, $this->course->id, 'student');
+
+ // Create a new discussion and post within the moodleoverflow.
+ $generator = $this->getDataGenerator()->get_plugin_generator('mod_moodleoverflow');
+ $this->discussion = $generator->post_to_forum($this->moodleoverflow, $this->user);
+ }
+
+ /**
+ * Run the send daily mail task.
+ * @return false|string
+ */
+ private function helper_run_send_daily_mail() {
+ $mailtask = new send_daily_mail();
+ ob_start();
+ $mailtask->execute();
+ $output = ob_get_contents();
+ ob_end_clean();
+ return $output;
+ }
+
+ /**
+ * Run the send mails task.
+ * @return false|string
+ */
+ private function helper_run_send_mails() {
+ $mailtask = new send_mails();
+ ob_start();
+ $mailtask->execute();
+ $output = ob_get_contents();
+ ob_end_clean();
+ return $output;
+ }
+
+
+
+ // Begin of test functions.
+
+ /**
+ * Test if the task send_daily_mail sends a mail to the user.
+ */
+ public function test_mail_delivery() {
+ global $DB;
+
+ // Create user with maildigest = on.
+ $this->helper_create_user_and_discussion('1');
+
+ // Send a mail and test if the mail was sent.
+ $this->helper_run_send_mails();
+ $this->helper_run_send_daily_mail();
+ $messages = $this->sink->count();
+
+ $this->assertEquals(1, $messages);
+ }
+
+
+ /**
+ * Test if the content of the mail matches the supposed content.
+ */
+ public function test_content_of_mail_delivery() {
+ global $DB;
+
+ // Create user with maildigest = on.
+ $this->helper_create_user_and_discussion('1');
+
+ // Send the mails and count the messages.
+ $this->helper_run_send_mails();
+ $this->helper_run_send_daily_mail();
+ $content = $this->sink->get_messages();
+ $message = $content[0]->body;
+ $message = str_replace(["\n\r", "\n", "\r"], '', $message);
+ $messagecount = $this->sink->count();
+
+ // Build the text that the mail should have.
+ // Text structure at get_string('digestunreadpost', moodleoverflow).
+ $linktocourse = 'course->id;
+ $linktoforum = 'coursemodule->id;
+ $linktodiscussion = 'discussion[0]->id;
+
+ // Assemble text.
+ $text = 'Course: ' . $linktocourse . ' -> ' . $linktoforum . ', Topic: '
+ . $linktodiscussion . ' has ' . $messagecount . ' unread posts.';
+
+ $this->assertStringContainsString($linktocourse, $message);
+ $this->assertStringContainsString($linktoforum, $message);
+ $this->assertStringContainsString($linktodiscussion, $message);
+ $this->assertStringContainsString($messagecount, $message);
+ }
+
+
+ /**
+ * Test if the task does not send a mail when maildigest = 0
+ */
+ public function test_mail_not_send() {
+ global $DB;
+ // Creat user with daily_mail = off.
+ $this->helper_create_user_and_discussion('0');
+
+ // Now send the mails and test if no mail was sent.
+ $this->helper_run_send_mails();
+ $this->helper_run_send_daily_mail();
+ $messages = $this->sink->count();
+
+ $this->assertEquals(0, $messages);
+ }
+
+ /**
+ * Test if database is updated after sending a mail
+ */
+ public function test_records_removed() {
+ global $DB;
+ // create user with maildigest = on.
+ $this->helper_create_user_and_discussion('1');
+
+ // Now send the mails.
+ $this->helper_run_send_mails();
+ $this->helper_run_send_daily_mail();
+
+ // Now check the database if the records of the users are deleted.
+ $records = $DB->get_records('moodleoverflow_mail_info', array('userid' => $this->user->id));
+ $this->assertEmpty($records);
+ }
+}
diff --git a/tests/generator/lib.php b/tests/generator/lib.php
index 257ef36951..ace760b258 100644
--- a/tests/generator/lib.php
+++ b/tests/generator/lib.php
@@ -190,6 +190,7 @@ public function create_discussion($record = null, $forum = null) {
* Function to create a dummy post.
*
* @param array|stdClass $record
+ * @param bool $straighttodb
*
* @return stdClass the post object
*/
@@ -377,8 +378,8 @@ public function reply_to_post($parent, $author, $straighttodb = true) {
// Add a post to the discussion.
$record = (object) [
'discussion' => $parent->discussion,
- 'parent' => $parent->id,
- 'userid' => $author->id
+ 'parent' => $parent->id,
+ 'userid' => $author->id
];
return $this->create_post($record, $straighttodb);
}
diff --git a/tests/locallib_test.php b/tests/locallib_test.php
index 560e14fe2e..33d5ae9c49 100644
--- a/tests/locallib_test.php
+++ b/tests/locallib_test.php
@@ -22,6 +22,10 @@
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
+namespace mod_moodleoverflow;
+
+use advanced_testcase;
+
defined('MOODLE_INTERNAL') || die();
require_once(__DIR__ . '/../locallib.php');
@@ -32,7 +36,7 @@
* @copyright 2017 Kennet Winter
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
-class mod_moodleoverflow_locallib_testcase extends advanced_testcase {
+class locallib_test extends advanced_testcase {
public function setUp(): void {
\mod_moodleoverflow\subscriptions::reset_moodleoverflow_cache();
diff --git a/tests/privacy_provider_test.php b/tests/privacy_provider_test.php
index 1ee3bb6c04..ca7e98f1ca 100644
--- a/tests/privacy_provider_test.php
+++ b/tests/privacy_provider_test.php
@@ -20,7 +20,10 @@
* @copyright 2018 Tamara Gunkel
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
+namespace mod_moodleoverflow;
+
defined('MOODLE_INTERNAL') || die();
+
global $CFG;
use core_privacy\local\request\approved_contextlist;
@@ -32,11 +35,12 @@
* Tests for the moodleoverflow implementation of the Privacy Provider API.
*
* @copyright 2018 Tamara Gunkel
+ * @package mod_moodleoverflow
* @group mod_moodleoverflow
* @group mod_moodleoverflow_privacy
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
-class mod_moodleoverflow_privacy_provider_testcase extends \core_privacy\tests\provider_testcase {
+class privacy_provider_test extends \core_privacy\tests\provider_testcase {
/**
* @var \mod_moodleoverflow_generator Plugin generator
*/
@@ -105,10 +109,14 @@ protected function assert_post_data($expected, $actual, $writer) {
);
}
+
/**
* Test that a user who is enrolled in a course, but who has never
* posted and has no other metadata stored will not have any link to
* that context.
+ *
+ * @return void
+ * @throws \coding_exception
*/
public function test_user_has_never_posted() {
// Create a course with moodleoverflow forums.
@@ -291,7 +299,7 @@ public function test_user_has_rated_others() {
$course = $this->getDataGenerator()->create_course();
$forum = $this->getDataGenerator()->create_module('moodleoverflow', [
'course' => $course->id,
- 'scale' => 100,
+ 'scale' => 100,
]);
list($user, $otheruser) = $this->create_users($course, 2);
list($discussion, $post) = $this->generator->post_to_forum($forum, $otheruser);
@@ -334,7 +342,7 @@ public function test_user_has_been_rated() {
$course = $this->getDataGenerator()->create_course();
$forum = $this->getDataGenerator()->create_module('moodleoverflow', [
'course' => $course->id,
- 'scale' => 100,
+ 'scale' => 100,
]);
list($user, $otheruser, $anotheruser) = $this->create_users($course, 3);
list($discussion, $post) = $this->generator->post_to_forum($forum, $user);
@@ -532,7 +540,7 @@ public function test_post_attachment_inclusion() {
list($author, $otheruser) = $this->create_users($course, 2);
$forum = $this->getDataGenerator()->create_module('moodleoverflow', [
'course' => $course->id,
- 'scale' => 100,
+ 'scale' => 100,
]);
$cm = get_coursemodule_from_instance('moodleoverflow', $forum->id);
$context = \context_module::instance($cm->id);
@@ -548,10 +556,10 @@ public function test_post_attachment_inclusion() {
$createdfile = $fs->create_file_from_string([
'contextid' => $context->id,
'component' => 'mod_moodleoverflow',
- 'filearea' => 'attachment',
- 'itemid' => $post->id,
- 'filepath' => '/',
- 'filename' => 'example.jpg',
+ 'filearea' => 'attachment',
+ 'itemid' => $post->id,
+ 'filepath' => '/',
+ 'filename' => 'example.jpg',
],
'image contents (not really)');
@@ -588,7 +596,7 @@ public function test_all_users_deleted_from_context() {
for ($i = 0; $i < 2; $i++) {
$forum = $this->getDataGenerator()->create_module('moodleoverflow', [
'course' => $course->id,
- 'scale' => 100,
+ 'scale' => 100,
]);
$cm = get_coursemodule_from_instance('moodleoverflow', $forum->id);
$context = \context_module::instance($cm->id);
@@ -616,10 +624,10 @@ public function test_all_users_deleted_from_context() {
$fs->create_file_from_string([
'contextid' => $context->id,
'component' => 'mod_moodleoverflow',
- 'filearea' => 'attachment',
- 'itemid' => $post->id,
- 'filepath' => '/',
- 'filename' => 'example.jpg',
+ 'filearea' => 'attachment',
+ 'itemid' => $post->id,
+ 'filepath' => '/',
+ 'filename' => 'example.jpg',
], 'image contents (not really)');
}
}
@@ -725,7 +733,7 @@ public function test_delete_data_for_user() {
for ($i = 0; $i < 2; $i++) {
$forum = $this->getDataGenerator()->create_module('moodleoverflow', [
'course' => $course->id,
- 'scale' => 100,
+ 'scale' => 100,
]);
$cm = get_coursemodule_from_instance('moodleoverflow', $forum->id);
$context = \context_module::instance($cm->id);
@@ -761,10 +769,10 @@ public function test_delete_data_for_user() {
$fs->create_file_from_string([
'contextid' => $context->id,
'component' => 'mod_moodleoverflow',
- 'filearea' => 'attachment',
- 'itemid' => $post->id,
- 'filepath' => '/',
- 'filename' => 'example.jpg',
+ 'filearea' => 'attachment',
+ 'itemid' => $post->id,
+ 'filepath' => '/',
+ 'filename' => 'example.jpg',
], 'image contents (not really)');
}
}
@@ -838,7 +846,7 @@ public function test_delete_data_for_user() {
$this->assertCount(4, $DB->get_records_select('moodleoverflow_posts', "userid = :userid"
. " AND " . $DB->sql_compare_text('message') . " = " . $DB->sql_compare_text(':message')
, [
- 'userid' => 0,
+ 'userid' => 0,
'message' => '',
]));
@@ -891,7 +899,7 @@ protected function create_courses_and_modules($count) {
* course.
* Users are enrolled as students.
*
- * @param stdClass $course The course object
+ * @param \stdClass $course The course object
* @param integer $count The number of users to create
*
* @return array The users created
@@ -910,7 +918,7 @@ protected function create_users($course, $count) {
/**
* Creates and enrols users.
*
- * @param stdClass $course The course to enrol users.
+ * @param \stdClass $course The course to enrol users.
* @param int $count The number of users to create.
*
* @return array The users created
@@ -990,19 +998,19 @@ public function test_delete_data_for_users() {
$fs->create_file_from_string([
'contextid' => $context->id,
'component' => 'mod_moodleoverflow',
- 'filearea' => 'post',
- 'itemid' => $post->id,
- 'filepath' => '/',
- 'filename' => 'example.jpg',
+ 'filearea' => 'post',
+ 'itemid' => $post->id,
+ 'filepath' => '/',
+ 'filename' => 'example.jpg',
], 'image contents (not really)');
// And a fake attachment.
$fs->create_file_from_string([
'contextid' => $context->id,
'component' => 'mod_moodleoverflow',
- 'filearea' => 'attachment',
- 'itemid' => $post->id,
- 'filepath' => '/',
- 'filename' => 'example.jpg',
+ 'filearea' => 'attachment',
+ 'itemid' => $post->id,
+ 'filepath' => '/',
+ 'filename' => 'example.jpg',
], 'image contents (not really)');
}
}
@@ -1418,12 +1426,12 @@ public function test_grades() {
$course = self::getDataGenerator()->create_course();
$forum = self::getDataGenerator()->create_module('moodleoverflow', [
'course' => $course->id,
- 'scale' => 100,
+ 'scale' => 100,
'grademaxgrade' => 50,
'gradescalefactor' => 2
]);
$cm = get_coursemodule_from_instance('moodleoverflow', $forum->id);
- $context = context_module::instance($cm->id);
+ $context = \context_module::instance($cm->id);
list($user, $user2) = $this->create_and_enrol_users($course, 2);
list( , $post) = $this->generator->post_to_forum($forum, $user);
diff --git a/tests/readtracking_test.php b/tests/readtracking_test.php
index ea979137cf..7f50a39f9e 100644
--- a/tests/readtracking_test.php
+++ b/tests/readtracking_test.php
@@ -21,6 +21,9 @@
* @copyright 2017 Kennet Winter
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
+namespace mod_moodleoverflow;
+
+use advanced_testcase;
defined('MOODLE_INTERNAL') || die();
@@ -34,7 +37,7 @@
* @copyright 2017 Kennet Winter
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
-class mod_moodleoverflow_readtracking_testcase extends advanced_testcase {
+class readtracking_test extends advanced_testcase {
/**
* Test the logic in the moodleoverflow_can_track_moodleoverflows() function.
diff --git a/tests/review_test.php b/tests/review_test.php
index 100a9da606..b6e2cb0562 100644
--- a/tests/review_test.php
+++ b/tests/review_test.php
@@ -44,12 +44,31 @@ class review_test extends \advanced_testcase {
/** @var \mod_moodleoverflow_generator $generator */
private $generator;
+ /**
+ * @var \stdClass
+ */
private $teacher;
+ /**
+ * @var \stdClass
+ */
private $student;
+ /**
+ * @var \stdClass
+ */
private $course;
+ /**
+ * @var \phpunit_message_sink
+ */
private $mailsink;
+ /**
+ * @var \phpunit_message_sink
+ */
private $messagesink;
+ /**
+ * set Up testing data.
+ * @return void
+ */
protected function setUp(): void {
$this->resetAfterTest();
@@ -70,6 +89,10 @@ protected function setUp(): void {
$this->messagesink = $this->redirectMessages();
}
+ /**
+ * Closing mailing links.
+ * @return void
+ */
protected function tearDown(): void {
$this->mailsink->clear();
$this->mailsink->close();
@@ -102,8 +125,8 @@ public function test_forum_review_everything() {
$this->assert_matches_properties(['mailed' => MOODLEOVERFLOW_MAILED_SUCCESS, 'reviewed' => 1, 'timereviewed' => null],
$DB->get_record('moodleoverflow_posts', ['id' => $teacherpost->id]));
- $this->assert_matches_properties(['mailed' => MOODLEOVERFLOW_MAILED_REVIEW_SUCCESS, 'reviewed' => 0, 'timereviewed' => null],
- $DB->get_record('moodleoverflow_posts', ['id' => $studentpost->id]));
+ $this->assert_matches_properties(['mailed' => MOODLEOVERFLOW_MAILED_REVIEW_SUCCESS, 'reviewed' => 0,
+ 'timereviewed' => null], $DB->get_record('moodleoverflow_posts', ['id' => $studentpost->id]));
$this->assertEquals(1, $this->mailsink->count()); // Teacher has to approve student message.
$this->assertEquals(2, $this->messagesink->count()); // Student and teacher get notification for student message.
@@ -177,8 +200,8 @@ public function test_forum_review_only_questions() {
$this->assert_matches_properties(['mailed' => MOODLEOVERFLOW_MAILED_SUCCESS, 'reviewed' => 1, 'timereviewed' => null],
$DB->get_record('moodleoverflow_posts', ['id' => $teacherpost->id]));
- $this->assert_matches_properties(['mailed' => MOODLEOVERFLOW_MAILED_REVIEW_SUCCESS, 'reviewed' => 0, 'timereviewed' => null],
- $DB->get_record('moodleoverflow_posts', ['id' => $studentpost->id]));
+ $this->assert_matches_properties(['mailed' => MOODLEOVERFLOW_MAILED_REVIEW_SUCCESS, 'reviewed' => 0,
+ 'timereviewed' => null], $DB->get_record('moodleoverflow_posts', ['id' => $studentpost->id]));
$this->assertEquals(1, $this->mailsink->count()); // Teacher has to approve student message.
$this->assertEquals(2, $this->messagesink->count()); // Student and teacher get notification for student message.
@@ -275,6 +298,10 @@ public function test_forum_review_disallowed() {
$DB->get_record('moodleoverflow_posts', ['id' => $studentanswer2->id]));
}
+ /**
+ * Run the send mails task.
+ * @return false|string
+ */
private function run_send_mails() {
$mailtask = new send_mails();
ob_start();
@@ -285,6 +312,7 @@ private function run_send_mails() {
}
/**
+ * Write own function to check if objects match.
* @param object|array $expected
* @param object|array $actual
*/
diff --git a/tests/subscriptions_test.php b/tests/subscriptions_test.php
index 015cf91b86..f6693e944b 100644
--- a/tests/subscriptions_test.php
+++ b/tests/subscriptions_test.php
@@ -21,6 +21,9 @@
* @copyright 2017 Kennet Winter
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
+namespace mod_moodleoverflow;
+
+use advanced_testcase;
defined('MOODLE_INTERNAL') || die();
@@ -34,7 +37,7 @@
* @copyright 2017 Kennet Winter
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
*/
-class mod_moodleoverflow_subscriptions_testcase extends advanced_testcase {
+class subscriptions_test extends advanced_testcase {
/**
* Test setUp.
@@ -58,7 +61,7 @@ public function tearDown(): void {
* Helper to create the required number of users in the specified course.
* Users are enrolled as students.
*
- * @param stdClass $course The course object
+ * @param \stdClass $course The course object
* @param int $count The number of users to create
*
* @return array The users created
@@ -78,8 +81,8 @@ protected function helper_create_users($course, $count) {
/**
* Crate a new discussion and post within the moodleoverflow.
*
- * @param stdClass $moodleoverflow The moodleoverflow to post in
- * @param stdClass $author The author to post as
+ * @param \stdClass $moodleoverflow The moodleoverflow to post in
+ * @param \stdClass $author The author to post as
*
* @return array Array containing the discussion object and the post object.
*/
@@ -90,7 +93,7 @@ protected function helper_post_to_moodleoverflow($moodleoverflow, $author) {
$generator = $this->getDataGenerator()->get_plugin_generator('mod_moodleoverflow');
// Create a discussion in the moodleoverflow, add a post to that discussion.
- $record = new stdClass();
+ $record = new \stdClass();
$record->course = $moodleoverflow->course;
$record->userid = $author->id;
$record->moodleoverflow = $moodleoverflow->id;
@@ -116,7 +119,7 @@ public function test_subscription_modes() {
$course = $this->getDataGenerator()->create_course();
$options = array('course' => $course->id);
$moodleoverflow = $this->getDataGenerator()->create_module('moodleoverflow', $options);
- $modulecontext = context_module::instance($moodleoverflow->cmid);
+ $modulecontext = \context_module::instance($moodleoverflow->cmid);
// Create a user enrolled in the course as a student.
list ($user) = $this->helper_create_users($course, 1);
@@ -232,10 +235,12 @@ public function test_moodleoverflow_toggle_as_other() {
$discussion->moodleoverflow = $moodleoverflow->id;
// Check that the user is currently not subscribed to the moodleoverflow.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow,
+ $modulecontext));
// Check that the user is unsubscribed from the discussion too.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow,
+ $modulecontext, $discussion->id));
// Check thast we have no records in either on the subscription tables.
$options = array('userid' => $author->id, 'moodleoverflow' => $moodleoverflow->id);
@@ -340,7 +345,7 @@ public function test_moodleoverflow_toggle_as_other() {
$count = $DB->count_records('moodleoverflow_subscriptions', $options);
$this->assertEquals(0, $count);
$count = $DB->count_records('moodleoverflow_discuss_subs', array(
- 'userid' => $author->id,
+ 'userid' => $author->id,
'discussion' => $discussion->id,
));
$this->assertEquals(0, $count);
@@ -351,7 +356,7 @@ public function test_moodleoverflow_toggle_as_other() {
$count = $DB->count_records('moodleoverflow_subscriptions', $options);
$this->assertEquals(0, $count);
$count = $DB->count_records('moodleoverflow_discuss_subs', array(
- 'userid' => $author->id,
+ 'userid' => $author->id,
'discussion' => $discussion->id,
));
$this->assertEquals(1, $count);
@@ -398,7 +403,8 @@ public function test_moodleoverflow_discussion_subscription_moodleoverflow_unsub
unset($post);
// Check that the user is unsubscribed from the discussion too.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $moodleoverflow, $discussion));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow,
+ $moodleoverflow, $discussion));
}
/**
@@ -436,7 +442,8 @@ public function test_moodleoverflow_discussion_subscription_moodleoverflow_subsc
unset($post);
// Check that the user is subscribed to the discussion too.
- $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion));
+ $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion));
}
/**
@@ -479,7 +486,8 @@ public function test_moodleoverflow_discussion_subscription_moodleoverflow_unsub
$this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $moodleoverflow));
// But subscribed to the discussion.
- $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
}
/**
@@ -520,7 +528,8 @@ public function test_moodleoverflow_discussion_subscription_moodleoverflow_subsc
$this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
// But unsubscribed from the discussion.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
}
/**
@@ -556,7 +565,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_subscribed(
$this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
// Check that the user is initially subscribed to that discussion.
- $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// An attempt to subscribe again should result in a falsey return to indicate that no change was made.
$this->assertFalse(\mod_moodleoverflow\subscriptions::subscribe_user_to_discussion($author->id,
@@ -589,7 +599,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_subscribed(
$this->assertEquals(1, $count);
// But unsubscribed from the discussion.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// There should be a record in the discussion subscription tracking table.
$options = array('userid' => $author->id, 'discussion' => $discussion->id);
@@ -608,7 +619,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_subscribed(
$this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
// And is subscribed to the discussion again.
- $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// And one in the moodleoverflow subscription tracking table.
$options = array('userid' => $author->id, 'moodleoverflow' => $moodleoverflow->id);
@@ -627,7 +639,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_subscribed(
$this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
// But unsubscribed from the discussion.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// And one in the moodleoverflow subscription tracking table.
$options = array('userid' => $author->id, 'moodleoverflow' => $moodleoverflow->id);
@@ -647,7 +660,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_subscribed(
$this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
// And is subscribed to the discussion again.
- $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// There should be no record in the discussion subscription tracking table.
$options = array('userid' => $author->id, 'discussion' => $discussion->id);
@@ -663,7 +677,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_subscribed(
\mod_moodleoverflow\subscriptions::unsubscribe_user_from_discussion($author->id, $discussion, $modulecontext);
// But unsubscribed from the discussion.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// Check that the user is still subscribed to the moodleoverflow.
$this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
@@ -679,7 +694,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_subscribed(
$this->assertEquals(1, $count);
// Now unsubscribe the user from the moodleoverflow.
- $this->assertTrue(\mod_moodleoverflow\subscriptions::unsubscribe_user($author->id, $moodleoverflow, $modulecontext, true));
+ $this->assertTrue(\mod_moodleoverflow\subscriptions::unsubscribe_user($author->id, $moodleoverflow, $modulecontext,
+ true));
// This removes both the moodleoverflow, and the moodleoverflow records.
$options = array('userid' => $author->id, 'discussion' => $discussion->id);
@@ -726,7 +742,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_unsubscribe
$discussion->moodleoverflow = $moodleoverflow->id;
// Check that the user is initially unsubscribed to that discussion.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// Then subscribe them to the discussion.
$this->assertTrue(\mod_moodleoverflow\subscriptions::subscribe_user_to_discussion($author->id,
@@ -740,7 +757,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_unsubscribe
$this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
// But subscribed to the discussion.
- $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// There should be a record in the discussion subscription tracking table.
$options = array('userid' => $author->id, 'discussion' => $discussion->id);
@@ -754,7 +772,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_unsubscribe
$this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
// And is unsubscribed from the discussion again.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// There should be no record in the discussion subscription tracking table.
$options = array('userid' => $author->id, 'discussion' => $discussion->id);
@@ -765,7 +784,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_unsubscribe
\mod_moodleoverflow\subscriptions::subscribe_user_to_discussion($author->id, $discussion, $modulecontext);
// And is subscribed to the discussion again.
- $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// Check that the user is still unsubscribed from the moodleoverflow.
$this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
@@ -779,7 +799,8 @@ public function test_moodleoverflow_discussion_toggle_moodleoverflow_unsubscribe
\mod_moodleoverflow\subscriptions::unsubscribe_user_from_discussion($author->id, $discussion, $modulecontext);
// But unsubscribed from the discussion.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// Check that the user is still unsubscribed from the moodleoverflow.
$this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($author->id, $moodleoverflow, $modulecontext));
@@ -890,7 +911,8 @@ public function test_fetch_subscribed_users_discussion_subscriptions() {
// All users should be subscribed.
$subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext);
$this->assertEquals($usercount, count($subscribers));
- $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null, true);
+ $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null,
+ true);
$this->assertEquals($usercount, count($subscribers));
\mod_moodleoverflow\subscriptions::unsubscribe_user_from_discussion($users[0]->id, $discussion, $modulecontext);
@@ -900,11 +922,12 @@ public function test_fetch_subscribed_users_discussion_subscriptions() {
$this->assertEquals($usercount, count($subscribers));
// All users should be subscribed.
- $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null, true);
+ $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null,
+ true);
$this->assertEquals($usercount, count($subscribers));
// Manually insert an extra subscription for one of the users.
- $record = new stdClass();
+ $record = new \stdClass();
$record->userid = $users[2]->id;
$record->moodleoverflow = $moodleoverflow->id;
$record->discussion = $discussion->id;
@@ -914,7 +937,8 @@ public function test_fetch_subscribed_users_discussion_subscriptions() {
// The discussion count should not have changed.
$subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext);
$this->assertEquals($usercount, count($subscribers));
- $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null, true);
+ $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null,
+ true);
$this->assertEquals($usercount, count($subscribers));
// Unsubscribe 2 users.
@@ -926,7 +950,8 @@ public function test_fetch_subscribed_users_discussion_subscriptions() {
// The subscription count should now take into account those users who have been unsubscribed.
$subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext);
$this->assertEquals($usercount - $unsubscribedcount, count($subscribers));
- $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null, true);
+ $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null,
+ true);
$this->assertEquals($usercount - $unsubscribedcount, count($subscribers));
// Now subscribe one of those users back to the discussion.
@@ -936,7 +961,8 @@ public function test_fetch_subscribed_users_discussion_subscriptions() {
}
$subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext);
$this->assertEquals($usercount - $unsubscribedcount, count($subscribers));
- $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null, true);
+ $subscribers = \mod_moodleoverflow\subscriptions::get_subscribed_users($moodleoverflow, $modulecontext, null,
+ true);
$this->assertEquals($usercount - $unsubscribedcount + $subedusers, count($subscribers));
}
@@ -967,7 +993,8 @@ public function test_force_subscribed_to_moodleoverflow() {
// Check that the user is currently subscribed to the moodleoverflow.
$this->assertTrue(\mod_moodleoverflow\subscriptions::is_subscribed($user->id, $moodleoverflow, $context));
- assign_capability('mod/moodleoverflow:allowforcesubscribe', CAP_PROHIBIT, $roleids['student'], $context);
+ assign_capability('mod/moodleoverflow:allowforcesubscribe', CAP_PROHIBIT, $roleids['student'],
+ $context);
$context->mark_dirty();
$this->assertFalse(has_capability('mod/moodleoverflow:allowforcesubscribe', $context, $user->id));
}
@@ -1262,15 +1289,16 @@ public function test_moodleoverflow_subscribe_toggle_as_other_repeat_subscriptio
$this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($user->id, $moodleoverflow, $modulecontext));
// Confirm that the user is unsubscribed from the discussion too.
- $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($user->id, $moodleoverflow, $modulecontext, $discussion->id));
+ $this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribed($user->id, $moodleoverflow, $modulecontext,
+ $discussion->id));
// Confirm that we have no records in either of the subscription tables.
$this->assertEquals(0, $DB->count_records('moodleoverflow_subscriptions', array(
- 'userid' => $user->id,
+ 'userid' => $user->id,
'moodleoverflow' => $moodleoverflow->id,
)));
$this->assertEquals(0, $DB->count_records('moodleoverflow_discuss_subs', array(
- 'userid' => $user->id,
+ 'userid' => $user->id,
'discussion' => $discussion->id,
)));
@@ -1278,11 +1306,11 @@ public function test_moodleoverflow_subscribe_toggle_as_other_repeat_subscriptio
// but not the moodleoverflow discussion subscriptions table.
\mod_moodleoverflow\subscriptions::subscribe_user($user->id, $moodleoverflow, $modulecontext);
$this->assertEquals(1, $DB->count_records('moodleoverflow_subscriptions', array(
- 'userid' => $user->id,
+ 'userid' => $user->id,
'moodleoverflow' => $moodleoverflow->id,
)));
$this->assertEquals(0, $DB->count_records('moodleoverflow_discuss_subs', array(
- 'userid' => $user->id,
+ 'userid' => $user->id,
'discussion' => $discussion->id,
)));
@@ -1306,22 +1334,22 @@ public function test_moodleoverflow_subscribe_toggle_as_other_repeat_subscriptio
\mod_moodleoverflow\subscriptions::unsubscribe_user($user->id, $moodleoverflow, $modulecontext);
$this->assertEquals(0, $DB->count_records('moodleoverflow_subscriptions', array(
- 'userid' => $user->id,
+ 'userid' => $user->id,
'moodleoverflow' => $moodleoverflow->id,
)));
$this->assertEquals(1, $DB->count_records('moodleoverflow_discuss_subs', array(
- 'userid' => $user->id,
+ 'userid' => $user->id,
'discussion' => $discussion->id,
)));
// Subscribing to the discussion should return truthfully because a change was made.
$this->assertTrue(\mod_moodleoverflow\subscriptions::subscribe_user_to_discussion($user->id, $discussion, $modulecontext));
$this->assertEquals(0, $DB->count_records('moodleoverflow_subscriptions', array(
- 'userid' => $user->id,
+ 'userid' => $user->id,
'moodleoverflow' => $moodleoverflow->id,
)));
$this->assertEquals(1, $DB->count_records('moodleoverflow_discuss_subs', array(
- 'userid' => $user->id,
+ 'userid' => $user->id,
'discussion' => $discussion->id,
)));
}
@@ -1378,7 +1406,7 @@ public function test_is_subscribable_logged_out($options) {
$moodleoverflow = $this->getDataGenerator()->create_module('moodleoverflow', $options);
$this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribable($moodleoverflow,
- context_module::instance($moodleoverflow->cmid)));
+ \context_module::instance($moodleoverflow->cmid)));
}
/**
@@ -1402,7 +1430,7 @@ public function test_is_subscribable_is_guest($options) {
$moodleoverflow = $this->getDataGenerator()->create_module('moodleoverflow', $options);
$this->assertFalse(\mod_moodleoverflow\subscriptions::is_subscribable($moodleoverflow,
- context_module::instance($moodleoverflow->cmid)));
+ \context_module::instance($moodleoverflow->cmid)));
}
/**
@@ -1453,6 +1481,6 @@ public function test_is_subscribable_loggedin($options, $expect) {
$this->setUser($user);
$this->assertEquals($expect, \mod_moodleoverflow\subscriptions::is_subscribable($moodleoverflow,
- context_module::instance($moodleoverflow->cmid)));
+ \context_module::instance($moodleoverflow->cmid)));
}
}
diff --git a/tracking.php b/tracking.php
index 47080f7fe4..9011bbe3df 100644
--- a/tracking.php
+++ b/tracking.php
@@ -27,7 +27,7 @@
require_once("locallib.php");
// Get submitted parameters.
-$id = required_param('id', PARAM_INT); // The moodleoverflow to track or untrack.
+$id = required_param('id', PARAM_INT); // The moodleoverflow to track or untrack.
$returnpage = optional_param('returnpage', 'index.php', PARAM_FILE); // The page to return to.
// A session key is needed to change the tracking options.
@@ -52,10 +52,10 @@
require_login($course, false, $cm);
// Set the page to return to.
-$url = '/mod/moodleoverflow/' . $returnpage;
-$params = array('id' => $course->id, 'm' => $moodleoverflow->id);
+$url = '/mod/moodleoverflow/' . $returnpage;
+$params = array('id' => $course->id, 'm' => $moodleoverflow->id);
$returnpageurl = new moodle_url($url, $params);
-$returnto = moodleoverflow_go_back_to($returnpageurl);
+$returnto = moodleoverflow_go_back_to($returnpageurl);
// Check whether the user can track the moodleoverflow instance.
$cantrack = \mod_moodleoverflow\readtracking::moodleoverflow_can_track_moodleoverflows($moodleoverflow);
@@ -67,15 +67,15 @@
}
// Create an info object.
-$info = new stdClass();
-$info->name = fullname($USER);
+$info = new stdClass();
+$info->name = fullname($USER);
$info->moodleoverflow = format_string($moodleoverflow->name);
// Set parameters for an event.
$eventparams = array(
- 'context' => context_module::instance($cm->id),
+ 'context' => context_module::instance($cm->id),
'relateduserid' => $USER->id,
- 'other' => array('moodleoverflowid' => $moodleoverflow->id),
+ 'other' => array('moodleoverflowid' => $moodleoverflow->id),
);
// Check whether the moodleoverflow is tracked.
diff --git a/version.php b/version.php
index 30c0977a12..d630f8cafc 100644
--- a/version.php
+++ b/version.php
@@ -28,7 +28,7 @@
defined('MOODLE_INTERNAL') || die();
$plugin->component = 'mod_moodleoverflow';
-$plugin->version = 2022120600;
+$plugin->version = 2023022406;
$plugin->release = 'v4.1-r1';
$plugin->requires = 2020061500; // Requires Moodle 3.9+.
$plugin->maturity = MATURITY_STABLE;
diff --git a/view.php b/view.php
index f3a9325298..d3200f5afe 100644
--- a/view.php
+++ b/view.php
@@ -31,10 +31,11 @@
require_once($CFG->dirroot.'/mod/moodleoverflow/locallib.php');
// Declare optional parameters.
-$id = optional_param('id', 0, PARAM_INT); // Course Module ID.
-$m = optional_param('m', 0, PARAM_INT); // MoodleOverflow ID.
-$page = optional_param('page', 0, PARAM_INT); // Which page to show.
-
+$id = optional_param('id', 0, PARAM_INT); // Course Module ID.
+$m = optional_param('m', 0, PARAM_INT); // MoodleOverflow ID.
+$page = optional_param('page', 0, PARAM_INT); // Which page to show.
+$movetopopup = optional_param('movetopopup', 0, PARAM_INT); // Which Topic to move.
+$linktoforum = optional_param('movetoforum', 0, PARAM_INT); // Forum to which it is moved.
// Set the parameters.
$params = array();
if ($id) {
@@ -49,13 +50,13 @@
// Check for the course and module.
if ($id) {
- $cm = get_coursemodule_from_id('moodleoverflow', $id, 0, false, MUST_EXIST);
- $course = $DB->get_record('course', array('id' => $cm->course), '*', MUST_EXIST);
- $moodleoverflow = $DB->get_record('moodleoverflow', array('id' => $cm->instance), '*', MUST_EXIST);
+ $cm = get_coursemodule_from_id('moodleoverflow', $id, 0, false, MUST_EXIST);
+ $course = $DB->get_record('course', array('id' => $cm->course), '*', MUST_EXIST);
+ $moodleoverflow = $DB->get_record('moodleoverflow', array('id' => $cm->instance), '*', MUST_EXIST);
} else if ($m) {
- $moodleoverflow = $DB->get_record('moodleoverflow', array('id' => $m), '*', MUST_EXIST);
- $course = $DB->get_record('course', array('id' => $moodleoverflow->course), '*', MUST_EXIST);
- $cm = get_coursemodule_from_instance('moodleoverflow', $moodleoverflow->id, $course->id, false, MUST_EXIST);
+ $moodleoverflow = $DB->get_record('moodleoverflow', array('id' => $m), '*', MUST_EXIST);
+ $course = $DB->get_record('course', array('id' => $moodleoverflow->course), '*', MUST_EXIST);
+ $cm = get_coursemodule_from_instance('moodleoverflow', $moodleoverflow->id, $course->id, false, MUST_EXIST);
} else {
throw new moodle_exception('missingparameter');
}
@@ -114,6 +115,18 @@
}
}
+if ($movetopopup && has_capability('mod/moodleoverflow:movetopic', $context)) {
+ moodleoverflow_print_forum_list($course, $cm, $movetopopup);
+}
+
+if ($linktoforum && $movetopopup && has_capability('mod/moodleoverflow:movetopic', $context)) {
+ // Take the $movetopopup-id and the $linktoforum-id and move the discussion to the forum.
+ $topic = $DB->get_record('moodleoverflow_discussions', array('id' => $movetopopup));
+ $topic->moodleoverflow = $linktoforum;
+ $DB->update_record('moodleoverflow_discussions', $topic);
+ redirect($CFG->wwwroot . '/mod/moodleoverflow/view.php?id=' . $cm->id);
+}
+
// Return here after posting, etc.
$SESSION->fromdiscussion = qualified_me();