MOre cleanups for notes interface
authormoodler <moodler>
Fri, 31 Aug 2007 05:22:44 +0000 (05:22 +0000)
committermoodler <moodler>
Fri, 31 Aug 2007 05:22:44 +0000 (05:22 +0000)
lang/en_utf8/notes.php
notes/add.php
notes/edit.php
notes/index.php
notes/lib.php

index 3a2d301..2901e13 100755 (executable)
@@ -6,9 +6,10 @@ $string['sitenotes'] = 'Site notes';
 $string['coursenotes'] = 'Course notes';\r
 $string['personalnotes'] = 'Personal notes';\r
 $string['created'] = 'created';\r
-$string['nonotes'] = 'There are no notes.';\r
+$string['nonotes'] = 'There are no notes of this type yet';\r
 $string['notesnotvisible'] = 'You are not allowed to view the notes.';\r
 $string['addnewnote'] = 'Add a new note';\r
+$string['addnewnoteselect'] = 'Select users to write notes about';\r
 $string['groupaddnewnote'] = 'Add a new note for all';\r
 $string['deleteconfirm'] = 'Delete this note?';\r
 $string['content'] = 'Content';\r
@@ -21,4 +22,4 @@ $string['personal'] = 'personal';
 $string['course'] = 'course';\r
 $string['site'] = 'site';\r
 $string['editnote'] = 'Edit note';\r
-?>
\ No newline at end of file
+?>\r
index ab95730..28d5647 100644 (file)
@@ -69,6 +69,7 @@
 
 /// output HTML
     $nav = array();
+    $nav[] = array('name' => get_string('participants'), 'link' => $CFG->wwwroot . '/user/index.php?id=' . $course->id, 'type' => 'misc');
     $nav[] = array('name' => fullname($user), 'link' => $CFG->wwwroot . '/user/view.php?id=' . $user->id. '&amp;course=' . $course->id, 'type' => 'misc');
     $nav[] = array('name' => get_string('notes', 'notes'), 'link' => $CFG->wwwroot . '/notes/index.php?course=' . $course->id . '&amp;user=' . $user->id, 'type' => 'misc');
     $nav[] = array('name' => $strnotes, 'link' => '', 'type' => 'activity');
index 1687bea..ba2238a 100644 (file)
@@ -72,6 +72,7 @@
 
 /// output HTML
     $nav = array();
+    $nav[] = array('name' => get_string('participants'), 'link' => $CFG->wwwroot . '/user/index.php?id=' . $course->id, 'type' => 'misc');
     $nav[] = array('name' => fullname($user), 'link' => $CFG->wwwroot . '/user/view.php?id=' . $user->id. '&amp;course=' . $course->id, 'type' => 'misc');
     $nav[] = array('name' => get_string('notes', 'notes'), 'link' => $CFG->wwwroot . '/notes/index.php?course=' . $course->id . '&amp;user=' . $user->id, 'type' => 'misc');
     $nav[] = array('name' => $strnotes, 'link' => '', 'type' => 'activity');
index b78749a..d20f4a7 100644 (file)
 
     $strnotes = get_string('notes', 'notes');
     $nav = array();
-    $nav[] = array('name' => fullname($user), 'link' => $CFG->wwwroot . '/user/view.php?id=' . $user->id. '&amp;course=' . $course->id, 'type' => 'misc');
+    $nav[] = array('name' => get_string('participants'), 'link' => $CFG->wwwroot . '/user/index.php?id=' . $course->id, 'type' => 'misc');
+    if ($userid) {
+        $nav[] = array('name' => fullname($user), 'link' => $CFG->wwwroot . '/user/view.php?id=' . $user->id. '&amp;course=' . $course->id, 'type' => 'misc');
+    }
     $nav[] = array('name' => $strnotes, 'link' => '', 'type' => 'misc');
 
     print_header($course->shortname . ': ' . $strnotes, $course->fullname, build_navigation($nav));
@@ -72,7 +75,7 @@
         $addid = has_capability('moodle/notes:manage', $context) ? $courseid : 0;
         $view = has_capability('moodle/notes:view', $context);
         note_print_notes('<a name="sitenotes"></a>' . $strsitenotes, $addid, $view, 0, $userid, NOTES_STATE_SITE, 0);
-        note_print_notes('<a name="coursenotes"></a>' . $strcoursenotes, $addid, $view, $courseid, $userid, NOTES_STATE_PUBLIC, 0);
+        note_print_notes('<a name="coursenotes"></a>' . $strcoursenotes. ' ('.$course->fullname.')', $addid, $view, $courseid, $userid, NOTES_STATE_PUBLIC, 0);
         note_print_notes('<a name="personalnotes"></a>' . $strpersonalnotes, $addid, $view, $courseid, $userid, NOTES_STATE_DRAFT, $USER->id);
 
     } else {  // Normal course
index a5fc6bc..4a93842 100644 (file)
@@ -235,7 +235,11 @@ function note_print_notes($header, $addcourseid = 0, $viewnotes = true, $coursei
         echo '<div class="notesgroup">';
     }
     if ($addcourseid) {
-        echo '<p><a href="'. $CFG->wwwroot .'/notes/add.php?course=' . $addcourseid . '&amp;user=' . $userid . '&amp;state=' . $state . '">' . get_string('addnewnote', 'notes') . '</a></p>';
+        if ($userid) {
+           echo '<p><a href="'. $CFG->wwwroot .'/notes/add.php?course=' . $addcourseid . '&amp;user=' . $userid . '&amp;state=' . $state . '">' . get_string('addnewnote', 'notes') . '</a></p>';
+        } else {
+           echo '<p><a href="'. $CFG->wwwroot .'/user/index.php?id=' . $addcourseid. '">' . get_string('addnewnoteselect', 'notes') . '</a></p>';
+        }
     }
     if ($viewnotes) {
         $notes =& note_list($courseid, $userid, $state, $author);