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

  * @license   http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 require_once('../../config.php');
 require_once('lib.php');
-require_once($CFG->libdir.'/completionlib.php');
+require_once($CFG->libdir . '/completionlib.php');
 $reply   = optional_param('reply', 0, PARAM_INT);
 $forum   = optional_param('forum', 0, PARAM_INT);
         if (!isguestuser()) {
             if (!is_enrolled($coursecontext)) {
                 if (enrol_selfenrol_available($course->id)) {
                     $SESSION->wantsurl = qualified_me();
                     $SESSION->enrolcancel = get_local_referer(false);
+                    redirect(
-                    redirect(new moodle_url('/enrol/index.php', array('id' => $course->id,
+                        new moodle_url('/enrol/index.php', array(
+                            'id' => $course->id,
-                        'returnurl' => '/mod/forum/view.php?f=' . $forum->id)),
+                            'returnurl' => '/mod/forum/view.php?f=' . $forum->id
+                        )),
-                        get_string('youneedtoenrol'));
+                        get_string('youneedtoenrol')
+                    );
+                }
+            }
+        }
         throw new \moodle_exception('nopostforum', 'forum');
+    }
     if (!$cm->visible and !has_capability('moodle/course:viewhiddenactivities', $modcontext)) {
         redirect(
-                $urlfactory->get_course_url_from_forum($forumentity),
+            $urlfactory->get_course_url_from_forum($forumentity),
-                get_string('activityiscurrentlyhidden'),
+            get_string('activityiscurrentlyhidden'),
-                null,
+            null,
-                \core\output\notification::NOTIFY_ERROR
+            \core\output\notification::NOTIFY_ERROR
-            );
+        );
+    }
     $post->messagetrust  = 0;
     $post->groupid = $groupid;
     // Unsetting this will allow the correct return URL to be calculated later.
-    unset($SESSION->fromdiscussion);
     unset($SESSION->fromdiscussion);
 } else if (!empty($reply)) {
     // User is writing a new reply.
     if (!$capabilitymanager->can_reply_to_post($USER, $discussionentity, $parententity)) {
         if (!isguestuser()) {
             if (!is_enrolled($coursecontext)) {  // User is a guest here!
                 $SESSION->wantsurl = qualified_me();
                 $SESSION->enrolcancel = get_local_referer(false);
+                redirect(
-                redirect(new moodle_url('/enrol/index.php', array('id' => $course->id,
+                    new moodle_url('/enrol/index.php', array(
+                        'id' => $course->id,
-                    'returnurl' => '/mod/forum/view.php?f=' . $forum->id)),
+                        'returnurl' => '/mod/forum/view.php?f=' . $forum->id
+                    )),
-                    get_string('youneedtoenrol'));
+                    get_string('youneedtoenrol')
+                );
+            }
             // The forum has been locked. Just redirect back to the discussion page.
             if (forum_discussion_is_locked($forum, $discussion)) {
     $post->groupid = ($discussion->groupid == -1) ? 0 : $discussion->groupid;
     $strre = get_string('re', 'forum');
     if (!(substr($post->subject, 0, strlen($strre)) == $strre)) {
-        $post->subject = $strre.' '.$post->subject;
+        $post->subject = $strre . ' ' . $post->subject;
+    }
     // Unsetting this will allow the correct return URL to be calculated later.
-    unset($SESSION->fromdiscussion);
+    // Unsetting this will allow the correct return URL to be calculated later.
     unset($SESSION->fromdiscussion);
 } else if (!empty($edit)) {
     $PAGE->set_cm($cm, $course, $forum);
     if (!($forum->type == 'news' && !$post->parent && $discussion->timestart > time())) {
+        if (((time() - $post->created) > $CFG->maxeditingtime) and
-        if (((time() - $post->created) > $CFG->maxeditingtime) and
+            !has_capability('mod/forum:editanypost', $modcontext)
-            !has_capability('mod/forum:editanypost', $modcontext)) {
+        ) {
             throw new \moodle_exception('maxtimehaspassed', 'forum', '', format_time($CFG->maxeditingtime));
+        }
     }
+    if (($post->userid <> $USER->id) and
-    if (($post->userid <> $USER->id) and
+        !has_capability('mod/forum:editanypost', $modcontext)
-        !has_capability('mod/forum:editanypost', $modcontext)) {
+    ) {
         throw new \moodle_exception('cannoteditposts', 'forum');
+    }
     $post = trusttext_pre_edit($post, 'message', $modcontext);
-    // Unsetting this will allow the correct return URL to be calculated later.
-    unset($SESSION->fromdiscussion);
+    // Unsetting this will allow the correct return URL to be calculated later.
     unset($SESSION->fromdiscussion);
 } else if (!empty($delete)) {
     // User is deleting a post.
     $modcontext = $forumentity->get_context();
     require_login($course, false, $cm);
+    $replycount = $postvault->get_reply_count_for_post_id_in_discussion_id(
+        $USER,
+        $postentity->get_id(),
+        $discussionentity->get_id(),
-    $replycount = $postvault->get_reply_count_for_post_id_in_discussion_id(
+        true
-        $USER, $postentity->get_id(), $discussionentity->get_id(), true);
+    );
                 $e->getMessage(),
                 null,
                 \core\output\notification::NOTIFY_ERROR
             );
+        }
     } else {
         if (!$capabilitymanager->can_delete_post($USER, $discussionentity, $postentity)) {
             redirect(
-                    $urlfactory->get_discussion_view_url_from_discussion($discussionentity),
+                $urlfactory->get_discussion_view_url_from_discussion($discussionentity),
-                    get_string('cannotdeletepost', 'forum'),
+                get_string('cannotdeletepost', 'forum'),
-                    null,
+                null,
-                    \core\output\notification::NOTIFY_ERROR
+                \core\output\notification::NOTIFY_ERROR
-                );
+            );
+        }
         $PAGE->activityheader->disable();
         if ($replycount) {
             if (!has_capability('mod/forum:deleteanypost', $modcontext)) {
                 redirect(
-                        forum_go_back_to($urlfactory->get_view_post_url_from_post($postentity)),
+                    forum_go_back_to($urlfactory->get_view_post_url_from_post($postentity)),
-                        get_string('couldnotdeletereplies', 'forum'),
+                    get_string('couldnotdeletereplies', 'forum'),
-                        null,
+                    null,
-                        \core\output\notification::NOTIFY_ERROR
+                    \core\output\notification::NOTIFY_ERROR
-                    );
+                );
+            }
             echo $OUTPUT->header();
             if (!$PAGE->has_secondary_navigation()) {
+                echo $OUTPUT->heading(format_string($forum->name), 2);
-                echo $OUTPUT->heading(format_string($forum->name), 2);
+            }
             echo $OUTPUT->confirm(
-            echo $OUTPUT->confirm(get_string("deletesureplural", "forum", $replycount + 1),
+                get_string("deletesureplural", "forum", $replycount + 1),
+                "post.php?delete=$delete&confirm=$delete",
-                "post.php?delete=$delete&confirm=$delete",
+                $CFG->wwwroot . '/mod/forum/discuss.php?d=' . $post->discussion . '#p' . $post->id
-                $CFG->wwwroot.'/mod/forum/discuss.php?d='.$post->discussion.'#p'.$post->id);
+            );
             $postentities = [$postentity];
             if (empty($post->edit)) {
                 $postvault = $vaultfactory->get_post_vault();
                 $replies = $postvault->get_replies_to_post(
-                        $USER,
+                    $USER,
-                        $postentity,
+                    $postentity,
-                        // Note: All replies are fetched here as the user has deleteanypost.
+                    // Note: All replies are fetched here as the user has deleteanypost.
-                        true,
+                    true,
-                        'created ASC'
+                    'created ASC'
-                    );
+                );
                 $postentities = array_merge($postentities, $replies);
         } else {
             echo $OUTPUT->header();
             if (!$PAGE->has_secondary_navigation()) {
                 echo $OUTPUT->heading(format_string($forum->name), 2);
+            }
+            echo $OUTPUT->confirm(
-            echo $OUTPUT->confirm(get_string("deletesure", "forum", $replycount),
+                get_string("deletesure", "forum", $replycount),
                 "post.php?delete=$delete&confirm=$delete",
-                $CFG->wwwroot.'/mod/forum/discuss.php?d='.$post->discussion.'#p'.$post->id);
+                $CFG->wwwroot . '/mod/forum/discuss.php?d=' . $post->discussion . '#p' . $post->id
+            );
             $rendererfactory = mod_forum\local\container::get_renderer_factory();
             $postsrenderer = $rendererfactory->get_single_discussion_posts_renderer(null, true);
             echo $postsrenderer->render($USER, [$forumentity], [$discussionentity], [$postentity]);
-        }
+        }
+    }
     echo $OUTPUT->footer();
-    die;
     die;
 } else if (!empty($prune)) {
     // Pruning.
         throw new \moodle_exception('invalidcoursemodule');
+    }
     if (!$postentity->has_parent()) {
         redirect(
-                $urlfactory->get_discussion_view_url_from_discussion($discussionentity),
+            $urlfactory->get_discussion_view_url_from_discussion($discussionentity),
-                get_string('alreadyfirstpost', 'forum'),
+            get_string('alreadyfirstpost', 'forum'),
-                null,
+            null,
-                \core\output\notification::NOTIFY_ERROR
+            \core\output\notification::NOTIFY_ERROR
-            );
+        );
+    }
     if (!$capabilitymanager->can_split_post($USER, $discussionentity, $postentity)) {
         redirect(
-                $urlfactory->get_discussion_view_url_from_discussion($discussionentity),
+            $urlfactory->get_discussion_view_url_from_discussion($discussionentity),
-                get_string('cannotsplit', 'forum'),
+            get_string('cannotsplit', 'forum'),
-                null,
+            null,
-                \core\output\notification::NOTIFY_ERROR
+            \core\output\notification::NOTIFY_ERROR
-            );
+        );
+    }
     $PAGE->set_cm($cm);
         // Display the prune form.
         $course = $DB->get_record('course', array('id' => $forum->course));
         $subjectstr = format_string($post->subject, true);
         $PAGE->navbar->add($subjectstr, new moodle_url('/mod/forum/discuss.php', array('d' => $discussion->id)));
         $PAGE->navbar->add(get_string("prunediscussion", "forum"));
-        $PAGE->set_title(format_string($discussion->name).": ".format_string($post->subject));
+        $PAGE->set_title(format_string($discussion->name) . ": " . format_string($post->subject));
         $PAGE->set_heading($course->fullname);
         echo $OUTPUT->header();
         if (!$PAGE->has_secondary_navigation()) {
             echo $OUTPUT->heading(format_string($forum->name), 2);
+        }
     echo $OUTPUT->footer();
     die;
 } else {
-    throw new \moodle_exception('unknowaction');
     throw new \moodle_exception('unknowaction');
+}
     throw new \moodle_exception('noguest');
+}
 $thresholdwarning = forum_check_throttling($forum, $cm);
 $mformpost = new mod_forum_post_form('post.php', [
-        'course' => $course,
+    'course' => $course,
-        'cm' => $cm,
+    'cm' => $cm,
-        'coursecontext' => $coursecontext,
+    'coursecontext' => $coursecontext,
-        'modcontext' => $modcontext,
+    'modcontext' => $modcontext,
-        'forum' => $forum,
+    'forum' => $forum,
-        'post' => $post,
+    'post' => $post,
-        'subscribe' => \mod_forum\subscriptions::is_subscribed($USER->id, $forum, null, $cm),
+    'subscribe' => \mod_forum\subscriptions::is_subscribed($USER->id, $forum, null, $cm),
-        'thresholdwarning' => $thresholdwarning,
+    'thresholdwarning' => $thresholdwarning,
-        'edit' => $edit,
+    'edit' => $edit,
-        'canreplyprivately' => $canreplyprivately,
+    'canreplyprivately' => $canreplyprivately,
-    ], 'post', '', array('id' => 'mformforum'));
+], 'post', '', array('id' => 'mformforum'));
 $draftitemid = file_get_submitted_draft_itemid('attachments');
 $postid = empty($post->id) ? null : $post->id;
 if ($USER->id != $post->userid) {   // Not the original author, so add a message to the end.
     $data = new stdClass();
     $data->date = userdate($post->created);
     if ($post->messageformat == FORMAT_HTML) {
-        $data->name = '<a href="'.$CFG->wwwroot.'/user/view.php?id='.$USER->id.'&course='.$post->course.'">'.
+        $data->name = '<a href="' . $CFG->wwwroot . '/user/view.php?id=' . $USER->id . '&course=' . $post->course . '">' .
-            fullname($USER).'</a>';
-        $post->message .= '<p><span class="edited">('.get_string('editedby', 'forum', $data).')</span></p>';
+            fullname($USER) . '</a>';
     } else {
-        $data->name = fullname($USER);
-        $post->message .= "\n\n(".get_string('editedby', 'forum', $data).')';
+        $data->name = fullname($USER);
+    }
     unset($data);
+}
         'discussion' => $post->discussion,
         'course' => $course->id,
         'isprivatereply' => $post->isprivatereply ?? false
     ) +
-    $pageparams +
+        $pageparams +
-    (isset($post->format) ? array('format' => $post->format) : array()) +
+        (isset($post->format) ? array('format' => $post->format) : array()) +
-    (isset($discussion->timestart) ? array('timestart' => $discussion->timestart) : array()) +
+        (isset($discussion->timestart) ? array('timestart' => $discussion->timestart) : array()) +
-    (isset($discussion->timeend) ? array('timeend' => $discussion->timeend) : array()) +
+        (isset($discussion->timeend) ? array('timeend' => $discussion->timeend) : array()) +
-    (isset($discussion->pinned) ? array('pinned' => $discussion->pinned) : array()) +
+        (isset($discussion->pinned) ? array('pinned' => $discussion->pinned) : array()) +
         $fromform->id = $fromform->edit;
         $message = '';
         if (!$capabilitymanager->can_edit_post($USER, $discussionentity, $postentity)) {
             redirect(
-                    $urlfactory->get_view_post_url_from_post($postentity),
+                $urlfactory->get_view_post_url_from_post($postentity),
-                    get_string('cannotupdatepost', 'forum'),
+                get_string('cannotupdatepost', 'forum'),
-                    null,
+                null,
-                    \core\output\notification::ERROR
+                \core\output\notification::ERROR
-                );
+            );
+        }
         if (isset($fromform->groupinfo) && $capabilitymanager->can_move_discussions($USER)) {
             // If the user has access to all groups and they are changing the group, then update the post.
             if (empty($fromform->groupinfo)) {
                 $fromform->groupinfo = -1;
+            }
             if (!$capabilitymanager->can_create_discussions($USER, $fromform->groupinfo)) {
                 redirect(
-                        $urlfactory->get_view_post_url_from_post($postentity),
+                    $urlfactory->get_view_post_url_from_post($postentity),
-                        get_string('cannotupdatepost', 'forum'),
+                    get_string('cannotupdatepost', 'forum'),
-                        null,
+                    null,
-                        \core\output\notification::ERROR
+                    \core\output\notification::ERROR
-                    );
+                );
+            }
             forum_go_back_to($discussionurl),
             $message . $subscribemessage,
             null,
             \core\output\notification::NOTIFY_SUCCESS
         );
     } else if ($fromform->discussion) {
         // Adding a new post to an existing discussion
         // Before we add this we must check that the user will not exceed the blocking threshold.
         forum_check_blocking_threshold($thresholdwarning);
             $subscribemessage = forum_post_subscription($fromform, $forum, $discussion);
             if (!empty($fromform->mailnow)) {
                 $message .= get_string("postmailnow", "forum");
             } else {
-                $message .= '<p>'.get_string("postaddedsuccess", "forum") . '</p>';
+                $message .= '<p>' . get_string("postaddedsuccess", "forum") . '</p>';
-                $message .= '<p>'.get_string("postaddedtimeleft", "forum", format_time($CFG->maxeditingtime)) . '</p>';
+                $message .= '<p>' . get_string("postaddedtimeleft", "forum", format_time($CFG->maxeditingtime)) . '</p>';
+            }
             if ($forum->type == 'single') {
             $event->add_record_snapshot('forum_discussions', $discussion);
             $event->trigger();
             // Update completion state.
+            $completion = new completion_info($course);
-            $completion = new completion_info($course);
+            if (
-            if ($completion->is_enabled($cm) &&
+                $completion->is_enabled($cm) &&
+                ($forum->completionreplies || $forum->completionposts)
-                ($forum->completionreplies || $forum->completionposts)) {
+            ) {
                 $completion->update_state($cm, COMPLETION_COMPLETE);
+            }
             redirect(
                 forum_go_back_to($discussionurl),
                 $message . $subscribemessage,
                 null,
-                \core\output\notification::NOTIFY_SUCCESS
-            );
+                \core\output\notification::NOTIFY_SUCCESS
             );
         } else {
             throw new \moodle_exception("couldnotadd", "forum", $errordestination);
-        }
         exit;
         exit;
     } else {
         // Adding a new discussion.
                 $event->trigger();
                 if ($fromform->mailnow) {
                     $message .= get_string("postmailnow", "forum");
                 } else {
-                    $message .= '<p>'.get_string("postaddedsuccess", "forum") . '</p>';
+                    $message .= '<p>' . get_string("postaddedsuccess", "forum") . '</p>';
-                    $message .= '<p>'.get_string("postaddedtimeleft", "forum", format_time($CFG->maxeditingtime)) . '</p>';
+                    $message .= '<p>' . get_string("postaddedtimeleft", "forum", format_time($CFG->maxeditingtime)) . '</p>';
+                }
                 $subscribemessage = forum_post_subscription($fromform, $forum, $discussion);
             } else {
                 throw new \moodle_exception("couldnotadd", "forum", $errordestination);
+            }
+        }
+        // Update completion status.
-        // Update completion status.
+        $completion = new completion_info($course);
+        if (
-        $completion = new completion_info($course);
+            $completion->is_enabled($cm) &&
-        if ($completion->is_enabled($cm) &&
+            ($forum->completiondiscussions || $forum->completionposts)
-            ($forum->completiondiscussions || $forum->completionposts)) {
+        ) {
             $completion->update_state($cm, COMPLETION_COMPLETE);
     $postentities = [$parententity];
     if (empty($post->edit)) {
         if ('qanda' != $forumentity->get_type() || forum_user_can_see_discussion($forum, $discussion, $modcontext)) {
             $replies = $postvault->get_replies_to_post(
-                    $USER,
+                $USER,
-                    $parententity,
+                $parententity,
-                    $capabilitymanager->can_view_any_private_reply($USER),
+                $capabilitymanager->can_view_any_private_reply($USER),
-                    'created ASC'
+                'created ASC'
-                );
+            );
             $postentities = array_merge($postentities, $replies);
+        }
+    }
     echo $postsrenderer->render($USER, [$forumentity], [$discussionentity], $postentities);
+}
 // Call print disclosure for enabled plagiarism plugins.
 if (!empty($CFG->enableplagiarism)) {
-    require_once($CFG->libdir.'/plagiarismlib.php');
+    require_once($CFG->libdir . '/plagiarismlib.php');
     echo plagiarism_print_disclosure($cm->id);
+}

Proyectos de Subversion Moodle

(root)/mod/forum/post.php – Rev 1 → 1222