feature is almost completed, topic can be moved to another forum.

Author: TamaroWalter

Diff for: lang/en/moodleoverflow.php

@@ -341,6 +341,7 @@
 $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.';

Diff for: locallib.php

@@ -355,8 +355,8 @@ function moodleoverflow_print_latest_discussions($moodleoverflow, $cm, $page = -
         // 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;
@@ -390,6 +390,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.
  *

Diff for: renderer.php

@@ -46,6 +46,13 @@ class mod_moodleoverflow_renderer extends plugin_renderer_base {
     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 for: 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 <http://www.gnu.org/licenses/>.
+}}
+{{!
+  @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}}
+    <table cellspacing="0" class="moodleoverflowheaderlist generaltable">
+        <thead>
+            <th class ="header topic" scope="col">
+                {{#currentdiscussion}}
+                    Where do you want to move the topic "{{currentdiscussion}}" ?
+                {{/currentdiscussion}}          
+            </th>
+        </thead>
+        <tbody>
+            {{#forums}}        
+                <tr class="discussion">
+                    <td class="topic starter">
+                        {{name}} 
+                        <a href='{{ movetoforum }}'>{{#pix}} i/arrow-right, core, {{#str}}movetopictoforum, moodleoverflow{{/str}} {{/pix}}</a>
+                    </td>
+                </tr>
+            {{/forums}}
+        <tbody>
+    </table>
+{{/hasforums}}
+{{^hasforums}}
+    <div class="alert alert-info">
+        Unfortunately, there is no other Moodleoverflow where you can move it.
+    </div>
+{{/hasforums}}
+<br>

Diff for: view.php

@@ -34,7 +34,8 @@
 $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);
+$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) {
@@ -114,23 +115,18 @@
     }
 }
 
-if ($movetopopup) {
-    // make a list of all moodleoverflows in the course.
-    $forumarray = array();
-    $currentforum = $DB->get_record('moodleoverflow_discussions', array('id' => $movetopopup), 'moodleoverflow');
-    $forums = $DB->get_records('moodleoverflow', array('course' => $course->id));
-    // write the moodleoverflow-names in an array.
-    foreach ($forums as $forum) {
-        if ($forum->id == $currentforum->moodleoverflow) {
-            continue;
-        } else {
-            array_push($forumarray, $forum->name);
-        }
-    }
-    // build popup
-
+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();