Browse files

Can now specify course by "short name" using name parameter. Useful for

integrating with external interfaces.  Thanks to dopey for the idea.
  • Loading branch information...
1 parent 6f93182 commit 388f8911e3e6f259bf6b3ac924ae5c369af7cf36 martin committed Jul 29, 2002
Showing with 15 additions and 3 deletions.
  1. +15 −3 course/view.php
View
18 course/view.php
@@ -5,13 +5,25 @@
require("../config.php");
require("lib.php");
+ optional_variable($id);
+ optional_variable($name);
- require_login($id);
+ if (!$id and !$name) {
+ error("Must specify course id or short name");
+ }
- if (! $course = get_record("course", "id", $id) ) {
- error("That's an invalid course id");
+ if ($name) {
+ if (! $course = get_record("course", "shortname", $name) ) {
+ error("That's an invalid short course name");
+ }
+ } else {
+ if (! $course = get_record("course", "id", $id) ) {
+ error("That's an invalid course id");
+ }
}
+ require_login($id);
+
add_to_log($course->id, "course", "view", "view.php?id=$course->id", "$course->id");
if ( isteacher($course->id) ) {

0 comments on commit 388f891

Please sign in to comment.