WebSVN – Moodle – Diff – /mod/forum/lib.php

         case FEATURE_BACKUP_MOODLE2:          return true;
         case FEATURE_SHOW_DESCRIPTION:        return true;
         case FEATURE_PLAGIARISM:              return true;
         case FEATURE_ADVANCED_GRADING:        return true;
         case FEATURE_MOD_PURPOSE:             return MOD_PURPOSE_COLLABORATION;
+        case FEATURE_CAN_UNINSTALL:
+            return false;
         default: return null;
+    }
         echo "<p>".get_string("noposts", "forum")."</p>";
+    }
+}
-/**
- * @deprecated since Moodle 3.3, when the block_course_overview block was removed.
- */
-function forum_filter_user_groups_discussions() {
-    throw new coding_exception('forum_filter_user_groups_discussions() can not be used any more and is obsolete.');
-}
 /**
  * Returns whether the discussion group is visible by the current user or not.
+ *
  * @since Moodle 2.8, 2.7.1, 2.6.4
  * @param cm_info $cm The discussion course module
     return false;
+}
-/**
- * @deprecated since Moodle 3.3, when the block_course_overview block was removed.
- */
-function forum_print_overview() {
-    throw new coding_exception('forum_print_overview() can not be used any more and is obsolete.');
-}
 /**
  * Given a course and a date, prints a summary of all the new
  * messages posted in the course since that date
+ *
         $groupselect = "";
+    }
     if (empty($forumsort)) {
         $forumsort = forum_get_default_sort_order();
+    }
+    if (!str_contains($forumsort, 'id')) {
+        $forumsort .= ', d.id DESC';
+    }
     if (empty($fullpost)) {
         $postdata = "p.id, p.subject, p.modified, p.discussion, p.userid, p.created";
     } else {
         $postdata = "p.*";
+    }
                    JOIN {forum_posts} p ON p.discussion = d.id
                    JOIN {user} u ON p.userid = u.id
                    $umtable
              WHERE d.forum = ? AND p.parent = 0
                    $timelimit $groupselect $updatedsincesql
-          ORDER BY $forumsort, d.id DESC";
+          ORDER BY $forumsort";
     return $DB->get_records_sql($sql, $params, $limitfrom, $limitnum);
+}
     include_once("$CFG->dirroot/course/lib.php");
     if (! $mod->coursemodule = add_course_module($mod) ) {
         echo $OUTPUT->notification("Could not add a new course module to the course '" . $courseid . "'");
         return false;
+    }
-    $sectionid = course_add_cm_to_section($courseid, $mod->coursemodule, 0);
+    $sectionid = course_add_cm_to_section($courseid, $mod->coursemodule, 0, null, 'forum');
     return $DB->get_record("forum", array("id" => "$forum->id"));
+}
 /**
             $button = new portfolio_add_button();
+        }
         foreach ($files as $file) {
             $filename = $file->get_filename();
             $mimetype = $file->get_mimetype();
-            $iconimage = $OUTPUT->pix_icon(file_file_icon($file), get_mimetype_description($file), 'moodle', array('class' => 'icon'));
+            $iconimage = $OUTPUT->pix_icon(file_file_icon($file),
+                    get_mimetype_description($file),
+                    'moodle',
+                    [
+                            'class' => 'icon',
+                            'style' => 'max-width: 24px; max-height: 24px; vertical-align: middle;',
+                    ]
+            );
             $path = file_encode_url($CFG->wwwroot.'/pluginfile.php', '/'.$context->id.'/mod_forum/attachment/'.$post->id.'/'.$filename);
             if ($type == 'html') {
                 $output .= "<a href=\"$path\">$iconimage</a> ";
 // Use SQL to find the last post for this discussion
     $sql = "SELECT id, userid, modified
               FROM {forum_posts}
              WHERE discussion=?
-             ORDER BY modified DESC";
+             ORDER BY modified DESC, id DESC";
 // Lets go find the last post
     if (($lastposts = $DB->get_records_sql($sql, array($discussionid), 0, 1))) {
  * This function will remove all posts from the specified forum
  * and clean up any related data.
+ *
  * @global object
  * @global object
- * @param $data the data submitted from the reset course.
+ * @param stdClass $data the data submitted from the reset course.
  * @return array status array
  */
 function forum_reset_userdata($data) {
     global $CFG, $DB;
     require_once($CFG->dirroot.'/rating/lib.php');
     $componentstr = get_string('modulenameplural', 'forum');
-    $status = array();
+    $status = [];
-    $params = array($data->courseid);
+    $params = [$data->courseid];
     $removeposts = false;
-    $typesql     = "";
+    $typesql = '';
     if (!empty($data->reset_forum_all)) {
         $removeposts = true;
-        $typesstr    = get_string('resetforumsall', 'forum');
+        $typesstr = get_string('resetforumsall', 'forum');
-        $types       = array();
+        $types = [];
-    } else if (!empty($data->reset_forum_types)){
+    } else if (!empty($data->reset_forum_types)) {
         $removeposts = true;
-        $types       = array();
+        $types = [];
-        $sqltypes    = array();
+        $sqltypes = [];
-        $forum_types_all = forum_get_forum_types_all();
+        $forumtypesall = forum_get_forum_types_all();
         foreach ($data->reset_forum_types as $type) {
-            if (!array_key_exists($type, $forum_types_all)) {
+            if (!array_key_exists($type, $forumtypesall)) {
                 continue;
+            }
-            $types[] = $forum_types_all[$type];
+            $types[] = $forumtypesall[$type];
             $sqltypes[] = $type;
+        }
+    }
     $alldiscussionssql = "SELECT fd.id
                             FROM {forum_discussions} fd, {forum} f
                            WHERE f.course=? AND f.id=fd.forum";
-    $allforumssql      = "SELECT f.id
+    $allforumssql = "SELECT f.id
-                            FROM {forum} f
+                       FROM {forum} f
-                           WHERE f.course=?";
+                      WHERE f.course=?";
-    $allpostssql       = "SELECT fp.id
+    $allpostssql = "SELECT fp.id
-                            FROM {forum_posts} fp, {forum_discussions} fd, {forum} f
+                      FROM {forum_posts} fp, {forum_discussions} fd, {forum} f
-                           WHERE f.course=? AND f.id=fd.forum AND fd.id=fp.discussion";
+                     WHERE f.course=? AND f.id=fd.forum AND fd.id=fp.discussion";
     $forumssql = $forums = $rm = null;
         $forums = $DB->get_records_sql($forumssql, $params);
+    }
     if ($removeposts) {
         $discussionssql = "$alldiscussionssql $typesql";
-        $postssql       = "$allpostssql $typesql";
+        $postssql = "$allpostssql $typesql";
-        // now get rid of all attachments
+        // Now get rid of all attachments.
         $fs = get_file_storage();
         if ($forums) {
-            foreach ($forums as $forumid=>$unused) {
+            foreach ($forums as $forumid => $unused) {
                 if (!$cm = get_coursemodule_from_instance('forum', $forumid)) {
                     continue;
+                }
                 $context = context_module::instance($cm->id);
                 $fs->delete_area_files($context->id, 'mod_forum', 'attachment');
                 $fs->delete_area_files($context->id, 'mod_forum', 'post');
-                //remove ratings
+                // Remove ratings.
                 $ratingdeloptions->contextid = $context->id;
                 $rm->delete_ratings($ratingdeloptions);
                 core_tag_tag::delete_instances('mod_forum', null, $context->id);
             }
+        }
-        // first delete all read flags
+        // First delete all read flags.
         $DB->delete_records_select('forum_read', "forumid IN ($forumssql)", $params);
+        // Remove tracking prefs.
+        $DB->delete_records_select('forum_track_prefs', "forumid IN ($forumssql)", $params);
+        // Remove posts from queue.
-        // remove tracking prefs
+        $DB->delete_records_select('forum_queue', "discussionid IN ($discussionssql)", $params);
+        // All posts - initial posts must be kept in single simple discussion forums.
-        $DB->delete_records_select('forum_track_prefs', "forumid IN ($forumssql)", $params);
+        // First all children.
         $DB->delete_records_select('forum_posts', "discussion IN ($discussionssql) AND parent <> 0", $params);
-        // remove posts from queue
+        // Now the initial posts for non single simple.
-        $DB->delete_records_select('forum_queue', "discussionid IN ($discussionssql)", $params);
+        $DB->delete_records_select(
             'forum_posts',
-        // all posts - initial posts must be kept in single simple discussion forums
+            "discussion IN ($discussionssql AND f.type <> 'single') AND parent = 0",
-        $DB->delete_records_select('forum_posts', "discussion IN ($discussionssql) AND parent <> 0", $params); // first all children
+            $params
-        $DB->delete_records_select('forum_posts', "discussion IN ($discussionssql AND f.type <> 'single') AND parent = 0", $params); // now the initial posts for non single simple
+        );
-        // finally all discussions except single simple forums
+        // Finally all discussions except single simple forums.
         $DB->delete_records_select('forum_discussions', "forum IN ($forumssql AND f.type <> 'single')", $params);
+        // Remove all grades from gradebook.
-        // remove all grades from gradebook
+        if (empty($data->reset_gradebook_grades)) {
+            if (empty($types)) {
+                forum_reset_gradebook($data->courseid);
+            } else {
-        if (empty($data->reset_gradebook_grades)) {
+                foreach ($types as $type) {
-            if (empty($types)) {
+                    forum_reset_gradebook($data->courseid, $type);
                 forum_reset_gradebook($data->courseid);
             } else {
-                foreach ($types as $type) {
+        }
                     forum_reset_gradebook($data->courseid, $type);
         $status[] = [
             'component' => $componentstr,
             'item' => $typesstr,
+            'error' => false,
-        $status[] = array('component'=>$componentstr, 'item'=>$typesstr, 'error'=>false);
+        ];
+    }
-    // remove all ratings in this course's forums
+    // Remove all ratings in this course's forums.
     if (!empty($data->reset_forum_ratings)) {
         if ($forums) {
-            foreach ($forums as $forumid=>$unused) {
+            foreach ($forums as $forumid => $unused) {
                 if (!$cm = get_coursemodule_from_instance('forum', $forumid)) {
                     continue;
                 $context = context_module::instance($cm->id);
                 core_tag_tag::delete_instances('mod_forum', null, $context->id);
+            }
+        }
+        $status[] = [
+            'component' => $componentstr,
+            'item' => get_string('removeallforumtags', 'forum'),
+            'error' => false,
-        $status[] = array('component' => $componentstr, 'item' => get_string('tagsdeleted', 'forum'), 'error' => false);
+        ];
     }
-    // remove all digest settings unconditionally - even for users still enrolled in course.
+    // Remove all digest settings unconditionally - even for users still enrolled in course.
+    if (!empty($data->reset_forum_digests)) {
+        $DB->delete_records_select('forum_digests', "forum IN ($allforumssql)", $params);
-    if (!empty($data->reset_forum_digests)) {
+        $status[] = [
+            'component' => $componentstr,
+            'item' => get_string('resetdigests', 'forum'),
-        $DB->delete_records_select('forum_digests', "forum IN ($allforumssql)", $params);
+            'error' => false,
-        $status[] = array('component' => $componentstr, 'item' => get_string('resetdigests', 'forum'), 'error' => false);
+        ];
+    }
-    // remove all subscriptions unconditionally - even for users still enrolled in course
+    // Remove all subscriptions unconditionally - even for users still enrolled in course.
+    if (!empty($data->reset_forum_subscriptions)) {
+        $DB->delete_records_select('forum_subscriptions', "forum IN ($allforumssql)", $params);
-    if (!empty($data->reset_forum_subscriptions)) {
+        $DB->delete_records_select('forum_discussion_subs', "forum IN ($allforumssql)", $params);
+        $status[] = [
+            'component' => $componentstr,
-        $DB->delete_records_select('forum_subscriptions', "forum IN ($allforumssql)", $params);
+            'item' => get_string('resetsubscriptions', 'forum'),
-        $DB->delete_records_select('forum_discussion_subs', "forum IN ($allforumssql)", $params);
+            'error' => false,
-        $status[] = array('component' => $componentstr, 'item' => get_string('resetsubscriptions', 'forum'), 'error' => false);
+        ];
+    }
+    // Remove all tracking prefs unconditionally - even for users still enrolled in course.
+    if (!empty($data->reset_forum_track_prefs)) {
+        $DB->delete_records_select('forum_track_prefs', "forumid IN ($allforumssql)", $params);
+        $status[] = [
-    // remove all tracking prefs unconditionally - even for users still enrolled in course
+            'component' => $componentstr,
-    if (!empty($data->reset_forum_track_prefs)) {
+            'item' => get_string('resettrackprefs', 'forum'),
-        $DB->delete_records_select('forum_track_prefs', "forumid IN ($allforumssql)", $params);
+            'error' => false,
-        $status[] = array('component'=>$componentstr, 'item'=>get_string('resettrackprefs','forum'), 'error'=>false);
+        ];
+    }
+    // Updating dates - shift may be negative too.
+    if ($data->timeshift) {
+        // Any changes to the list of dates that needs to be rolled should be same during course restore and course reset.
+        // See MDL-9367.
+        shift_course_mod_dates('forum', [
+            'assesstimestart',
+            'assesstimefinish',
+            'duedate',
-    /// updating dates - shift may be negative too
+            'cutoffdate',
+        ], $data->timeshift, $data->courseid);
-    if ($data->timeshift) {
+        $status[] = [
-        // Any changes to the list of dates that needs to be rolled should be same during course restore and course reset.
+            'component' => $componentstr,
-        // See MDL-9367.
+            'item' => get_string('date'),
+ *
  * @param MoodleQuickForm $mform form passed by reference
  */
 function forum_reset_course_form_definition(&$mform) {
     $mform->addElement('header', 'forumheader', get_string('modulenameplural', 'forum'));
+    $mform->addElement('static', 'forumdelete', get_string('delete'));
-    $mform->addElement('checkbox', 'reset_forum_all', get_string('resetforumsall','forum'));
+    $mform->addElement('checkbox', 'reset_forum_digests', get_string('resetdigests', 'forum'));
-    $mform->addElement('select', 'reset_forum_types', get_string('resetforums', 'forum'), forum_get_forum_types_all(), array('multiple' => 'multiple'));
-    $mform->setAdvanced('reset_forum_types');
-    $mform->disabledIf('reset_forum_types', 'reset_forum_all', 'checked');
+    $mform->addElement('checkbox', 'reset_forum_subscriptions', get_string('resetsubscriptions', 'forum'));
+    $mform->addElement('checkbox', 'reset_forum_all', get_string('resetforumsall', 'forum'));
+    $mform->addElement(
+        'select',
+        'reset_forum_types',
-    $mform->addElement('checkbox', 'reset_forum_digests', get_string('resetdigests','forum'));
+        get_string('resetforums', 'forum'),
-    $mform->setAdvanced('reset_forum_digests');
+        forum_get_forum_types_all(),
-    $mform->addElement('checkbox', 'reset_forum_subscriptions', get_string('resetsubscriptions','forum'));
+        ['multiple' => 'multiple'],
-    $mform->setAdvanced('reset_forum_subscriptions');
+    );
+    $mform->hideIf('reset_forum_types', 'reset_forum_all', 'checked');
     $mform->addElement('checkbox', 'reset_forum_track_prefs', get_string('resettrackprefs','forum'));
-    $mform->setAdvanced('reset_forum_track_prefs');
+    $mform->addElement('checkbox', 'reset_forum_track_prefs', get_string('resettrackprefs', 'forum'));
-    $mform->disabledIf('reset_forum_track_prefs', 'reset_forum_all', 'checked');
+    $mform->hideIf('reset_forum_track_prefs', 'reset_forum_all', 'checked');
     $mform->addElement('checkbox', 'reset_forum_ratings', get_string('deleteallratings'));
-    $mform->disabledIf('reset_forum_ratings', 'reset_forum_all', 'checked');
+    $mform->hideIf('reset_forum_ratings', 'reset_forum_all', 'checked');
  *                       a discussion will be returned.
  * @param int $limitfrom The offset of records to return
  * @param int $limitnum The number of records to return
  * @return array An array of forums the user has posted within in the provided courses
  */
-function forum_get_forums_user_posted_in($user, array $courseids = null, $discussionsonly = false, $limitfrom = null, $limitnum = null) {
+function forum_get_forums_user_posted_in($user, ?array $courseids = null, $discussionsonly = false, $limitfrom = null, $limitnum = null) {
     global $DB;
     if (!is_null($courseids)) {
         list($coursewhere, $params) = $DB->get_in_or_equal($courseids, SQL_PARAMS_NAMED, 'courseid');
         $forumsearchwhere[] = "(d.forum $fullidsql)";
+    }
     // Prepare SQL to both count and search.
     // We alias user.id to useridx because we forum_posts already has a userid field and not aliasing this would break
-    // oracle and mssql.
+    // mssql.
     $userfieldsapi = \core_user\fields::for_userpic();
     $userfields = $userfieldsapi->get_sql('u', false, '', 'useridx', false)->selects;
     $countsql = 'SELECT COUNT(*) ';
     $selectsql = 'SELECT p.*, d.forum, d.name AS discussionname, '.$userfields.' ';
+ *
  * @return  array
  */
 function mod_forum_get_fontawesome_icon_map() {
     return [
-        'mod_forum:i/pinned' => 'fa-map-pin',
+        'mod_forum:i/pinned' => 'fa-thumbtack',
         'mod_forum:t/selected' => 'fa-check',
-        'mod_forum:t/subscribed' => 'fa-envelope-o',
-        'mod_forum:t/unsubscribed' => 'fa-envelope-open-o',
         'mod_forum:t/star' => 'fa-star',
+        'mod_forum:t/subscribed' => 'fa-regular fa-envelope',
+        'mod_forum:t/unsubscribed' => 'fa-regular fa-envelope-open',
     ];
+}
 /**
     // This function is called by cron and we need to include the locallib for calls further down.
     require_once($CFG->dirroot . '/mod/forum/locallib.php');
     forum_update_calendar($instance, $cm->id);
+}
+/**
+ * Callback adds navigation to view user posts if the navadduserpostslinks config is on.
+ *
+ * @param navigation_node $usernode User node within navigation
+ * @param stdClass $user User object
+ * @param \core\context\user $usercontext User context
+ * @param stdClass $course Current course
+ * @param \core\context $coursecontext Course context
+ */
+function mod_forum_extend_navigation_user(
+    navigation_node $usernode,
+    stdClass $user,
+    \core\context\user $usercontext,
+    stdClass $course,
+    \core\context $coursecontext,
+): void {
+    global $CFG;
+    if (!empty($CFG->navadduserpostslinks) && $coursecontext instanceof \core\context\system) {
+        $baseargs = ['id' => $user->id];
+        // Add nodes for forum posts and discussions if the user can view either or both
+        // There are no capability checks here as the content of the page is based
+        // purely on the forums the current user has access too.
+        $forumtab = \navigation_node::create(get_string('forumposts', 'forum'));
+        $forumtab->add(
+            get_string('posts', 'forum'),
+            new moodle_url('/mod/forum/user.php', $baseargs),
+        );
+        $forumtab->add(
+            get_string('discussions', 'forum'),
+            new moodle_url('/mod/forum/user.php',
+                array_merge($baseargs, ['mode' => 'discussions']),
+            ),
+        );
+        // We add the forum link either immediately after the 'viewuserdetails' link, or as the first item in the list.
+        foreach ($usernode->children as $child) {
+            if ($child->key === 'viewuserdetails') {
+                continue;
+            }
+            $addbefore = $child;
+            break;
+        }
+        $usernode->add_node($forumtab, $addbefore->key);
+    }

Proyectos de Subversion Moodle

(root)/mod/forum/lib.php – Rev 1 → 1441