diff --git a/mod/wiki/comments.php b/mod/wiki/comments.php index a2699044e88b4..c28cbf4ad4711 100644 --- a/mod/wiki/comments.php +++ b/mod/wiki/comments.php @@ -59,7 +59,7 @@ require_login($course->id, true, $cm); -add_to_log($course->id, 'wiki', 'comments', 'comments.php?id=' . $cm->id, $wiki->id); +add_to_log($course->id, 'wiki', 'view comments', "comments.php?pageid=$pageid", $page->title, $cm->id); /// Print the page header $wikipage = new page_wiki_comments($wiki, $subwiki, $cm); diff --git a/mod/wiki/create.php b/mod/wiki/create.php index 19e571f225965..46939c2130fab 100644 --- a/mod/wiki/create.php +++ b/mod/wiki/create.php @@ -66,8 +66,6 @@ require_login($course->id, true, $cm); -add_to_log($course->id, 'createpage', 'createpage', 'view.php?id=' . $cm->id, $wiki->id); - $wikipage = new page_wiki_create($wiki, $subwiki, $cm); if (!empty($swid)) { @@ -80,22 +78,27 @@ $wikipage->set_uid($uid); } -if (!empty($title)) { - $wikipage->set_title($title); -} else { - $wikipage->set_title(get_string('newpage', 'wiki')); +if (empty($title)) { + $title = get_string('newpage', 'wiki'); } +$wikipage->set_title($title); // set page action, and initialise moodle form $wikipage->set_action($action); switch ($action) { case 'create': - $wikipage->create_page($title); + $newpageid = $wikipage->create_page($title); + $newtitle = $wikipage->get_title(); + add_to_log($course->id, 'wiki', 'add page', "view.php?pageid=$newpageid", "$newtitle", $cm->id); + redirect($CFG->wwwroot . '/mod/wiki/edit.php?pageid=' . $newpageid); break; case 'new': if ((int)$wiki->forceformat == 1 && !empty($title)) { - $wikipage->create_page($title); + $newpageid = $wikipage->create_page($title); + $newtitle = $wikipage->get_title(); + add_to_log($course->id, 'wiki', 'add page', "view.php?pageid=$newpageid", "$newtitle", $cm->id); + redirect($CFG->wwwroot . '/mod/wiki/edit.php?pageid=' . $newpageid); } else { // create link from moodle navigation block without pagetitle $wikipage->print_header(); diff --git a/mod/wiki/diff.php b/mod/wiki/diff.php index cf620960c65af..8c4d4a9fa5847 100644 --- a/mod/wiki/diff.php +++ b/mod/wiki/diff.php @@ -74,6 +74,8 @@ $wikipage->set_page($page); $wikipage->set_comparison($compare, $comparewith); +add_to_log($course->id, "wiki", "view diff", "diff.php?pageid=$pageid&comparewith=$comparewith&compare=$compare", "$page->title", $cm->id); + $wikipage->print_header(); $wikipage->print_content(); diff --git a/mod/wiki/edit.php b/mod/wiki/edit.php index f750718976e4a..410cbc3fc5289 100644 --- a/mod/wiki/edit.php +++ b/mod/wiki/edit.php @@ -77,8 +77,6 @@ $context = get_context_instance(CONTEXT_MODULE, $cm->id); require_capability('mod/wiki:editpage', $context); -add_to_log($course->id, 'wiki', 'edit', "edit.php?id=$cm->id", "$wiki->id"); - if ($option == get_string('save', 'wiki')) { if (!confirm_sesskey()) { print_error(get_string('invalidsesskey', 'wiki')); @@ -87,6 +85,7 @@ $wikipage->set_page($page); $wikipage->set_newcontent($newcontent); $wikipage->set_upload(true); + add_to_log($course->id, 'wiki', 'update', "view.php?pageid=$pageid", "$page->title", $cm->id); } else { if ($option == get_string('preview')) { if (!confirm_sesskey()) { diff --git a/mod/wiki/history.php b/mod/wiki/history.php index 3b8a344c3969a..5ddc2bea033e9 100644 --- a/mod/wiki/history.php +++ b/mod/wiki/history.php @@ -61,7 +61,7 @@ require_login($course->id, true, $cm); $context = get_context_instance(CONTEXT_MODULE, $cm->id); require_capability('mod/wiki:viewpage', $context); -add_to_log($course->id, 'wiki', 'history', 'history.php?id=' . $cm->id, $wiki->id); +add_to_log($course->id, 'wiki', 'view history', "history.php?pageid=$pageid", "$page->title", $cm->id); /// Print the page header $wikipage = new page_wiki_history($wiki, $subwiki, $cm); diff --git a/mod/wiki/index.php b/mod/wiki/index.php index d1db74a22d9cc..9ebb6c16ffbcb 100644 --- a/mod/wiki/index.php +++ b/mod/wiki/index.php @@ -45,7 +45,7 @@ $PAGE->set_pagelayout('incourse'); $context = get_context_instance(CONTEXT_COURSE, $course->id); -add_to_log($course->id, 'wiki', 'view all', "index.php?id=$course->id", ""); +add_to_log($course->id, 'wiki', 'view all', "index.php?id=$id", ""); /// Get all required stringswiki $strwikis = get_string("modulenameplural", "wiki"); diff --git a/mod/wiki/instancecomments.php b/mod/wiki/instancecomments.php index 80aa4277faf22..78043b2a91836 100644 --- a/mod/wiki/instancecomments.php +++ b/mod/wiki/instancecomments.php @@ -82,6 +82,7 @@ } if ($action == 'delete') { + add_to_log($course->id, 'wiki', 'delete comment', "comments.php?pageid=$pageid", "$page->title", $cm->id); $comm->set_action($action, $commentid, 0); } else { if (empty($newcontent)) { @@ -93,9 +94,10 @@ } if ($action == 'edit') { + add_to_log($course->id, 'wiki', 'update comment', "comments.php?pageid=$pageid", "$page->title", $cm->id); $comm->set_action($action, $id, $content); - } else { + add_to_log($course->id, 'wiki', 'add comment', "comments.php?pageid=$pageid", "$page->title", $cm->id); $action = 'add'; $comm->set_action($action, 0, $content); } diff --git a/mod/wiki/lock.php b/mod/wiki/lock.php index 562aabfcf01ce..d82a0f0db1bbf 100644 --- a/mod/wiki/lock.php +++ b/mod/wiki/lock.php @@ -71,8 +71,6 @@ $context = get_context_instance(CONTEXT_MODULE, $cm->id); require_capability('mod/wiki:editpage', $context); -//add_to_log($course->id, "wiki", "lock", "lock.php?id=$cm->id", "$wiki->id"); - $wikipage = new page_wiki_lock($wiki, $subwiki, $cm); $wikipage->set_page($page); diff --git a/mod/wiki/map.php b/mod/wiki/map.php index 341667bfcf66f..cad80f23f017f 100644 --- a/mod/wiki/map.php +++ b/mod/wiki/map.php @@ -62,6 +62,7 @@ /// Finish the page $wikipage = new page_wiki_map($wiki, $subwiki, $cm); +add_to_log($course->id, "wiki", "view map", "map.php?pageid=$pageid", "$page->title", $cm->id); $wikipage->set_view($option); $wikipage->set_page($page); diff --git a/mod/wiki/overridelocks.php b/mod/wiki/overridelocks.php index e4c516499d71c..37bd0bdcd4896 100644 --- a/mod/wiki/overridelocks.php +++ b/mod/wiki/overridelocks.php @@ -67,8 +67,6 @@ $context = get_context_instance(CONTEXT_MODULE, $cm->id); require_capability('mod/wiki:overridelock', $context); -add_to_log($course->id, "wiki", "overridelocks", "overridelocks.php?id=$cm->id", "$wiki->id"); - if (!confirm_sesskey()) { print_error(get_string('invalidsesskey', 'wiki')); } @@ -79,6 +77,7 @@ if (!empty($section)) { $wikipage->set_section($sectioncontent, $section); } +add_to_log($course->id, "wiki", "overridelocks", "view.php?pageid=$pageid", "$page->title", $cm->id); $wikipage->print_header(); diff --git a/mod/wiki/pagelib.php b/mod/wiki/pagelib.php index d14c92b498f24..0a64942fb8d73 100644 --- a/mod/wiki/pagelib.php +++ b/mod/wiki/pagelib.php @@ -220,6 +220,14 @@ function set_title($title) { $PAGE->set_title($this->title); } + /** + * returns title of the wiki page + * @return string + */ + function get_title() { + return $this->title; + } + /** * Method to set current group id * @param int $gid Current group id @@ -935,11 +943,14 @@ function create_page($pagetitle) { $this->subwiki = wiki_get_subwiki($swid); } if ($data) { + $this->set_title($data->pagetitle); $id = wiki_create_page($this->subwiki->id, $data->pagetitle, $data->pageformat, $USER->id); } else { + $this->set_title($pagetitle); $id = wiki_create_page($this->subwiki->id, $pagetitle, $PAGE->activityrecord->defaultformat, $USER->id); } - redirect($CFG->wwwroot . '/mod/wiki/edit.php?pageid=' . $id); + $this->page = $id; + return $id; } } diff --git a/mod/wiki/prettyview.php b/mod/wiki/prettyview.php index 30a19ea1a3b28..0b4ee60355bdd 100644 --- a/mod/wiki/prettyview.php +++ b/mod/wiki/prettyview.php @@ -56,11 +56,10 @@ $context = get_context_instance(CONTEXT_MODULE, $cm->id); require_capability('mod/wiki:viewpage', $context); -add_to_log($course->id, "wiki", "view", "prettyview.php?pageid=$pageid", "$wiki->id"); - $wikipage = new page_wiki_prettyview($wiki, $subwiki, $cm); $wikipage->set_page($page); +add_to_log($course->id, "wiki", "view pretty", "prettyview.php?pageid=$pageid", "$page->title", $cm->id); $wikipage->print_header(); $wikipage->print_content(); diff --git a/mod/wiki/restoreversion.php b/mod/wiki/restoreversion.php index a959b322565ee..3b03fbd2f1104 100644 --- a/mod/wiki/restoreversion.php +++ b/mod/wiki/restoreversion.php @@ -60,8 +60,6 @@ require_login($course->id, true, $cm); -add_to_log($course->id, "restore", "restore", "view.php?id=$cm->id", "$wiki->id"); - if ($confirm) { if (!confirm_sesskey()) { print_error(get_string('invalidsesskey', 'wiki')); @@ -77,6 +75,7 @@ $wikipage->set_versionid($versionid); } +add_to_log($course->id, "wiki", "restore", "view.php?pageid=$pageid", "$page->title : $versionid", $cm->id); $wikipage->print_header(); $wikipage->print_content(); diff --git a/mod/wiki/view.php b/mod/wiki/view.php index c29bfeccadae9..7cd5581b323c8 100644 --- a/mod/wiki/view.php +++ b/mod/wiki/view.php @@ -269,8 +269,6 @@ $context = get_context_instance(CONTEXT_MODULE, $cm->id); require_capability('mod/wiki:viewpage', $context); -add_to_log($course->id, 'wiki', 'view', 'view.php?id=' . $cm->id, $wiki->id); - // Update 'viewed' state if required by completion system require_once($CFG->libdir . '/completionlib.php'); $completion = new completion_info($course); @@ -294,6 +292,14 @@ $wikipage->set_gid($currentgroup); $wikipage->set_page($page); +if($pageid) { + add_to_log($course->id, 'wiki', 'view', "view.php?pageid=$pageid", "$page->title", $cm->id); +} else if($id) { + add_to_log($course->id, 'wiki', 'view', "view.php?id=$id", "$page->title", $cm->id); +} else if($wid && $title) { + add_to_log($course->id, 'wiki', 'view', "view.php?wid=$wid&title=$title", "$page->title", $cm->id); +} + $wikipage->print_header(); $wikipage->print_content(); diff --git a/mod/wiki/viewversion.php b/mod/wiki/viewversion.php index ca420d24f5bf2..ef2977ae017a5 100644 --- a/mod/wiki/viewversion.php +++ b/mod/wiki/viewversion.php @@ -67,6 +67,8 @@ $wikipage->set_page($page); $wikipage->set_versionid($versionid); +add_to_log($course->id, "wiki", "view version", "viewversion.php?pageid=$pageid&versionid=$versionid", "$page->title", $cm->id); + $wikipage->print_header(); $wikipage->print_content();