From 8130b77b40b39ee4df05672822a910833dbbee3a Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Thu, 21 Aug 2003 14:04:04 +0000
Subject: [PATCH] Split get-courses_page from get-courses so they can
 specialise

---
 course/category.php |  4 +---
 course/search.php   |  4 ----
 lib/datalib.php     | 34 ++++++++++++++++++++++++++++++++--
 3 files changed, 33 insertions(+), 9 deletions(-)

diff --git a/course/category.php b/course/category.php
index e9df6bc411..b74127a765 100644
--- a/course/category.php
+++ b/course/category.php
@@ -213,14 +213,12 @@
 
 /// Print out all the courses
 
-    if (!$courses = get_courses($category->id, "c.sortorder ASC", "c.*", $totalcount, $page*$perpage, $perpage)) {
+    if (!$courses = get_course_pages($category->id, "c.sortorder ASC", "c.*", $totalcount, $page*$perpage, $perpage)) {
         print_heading(get_string("nocoursesyet"));
 
     } else {
 
-        echo "<center>";
         print_paging_bar($totalcount, $page, $perpage, "category.php?id=$category->id&perpage=$perpage&");
-        echo "</center>";
 
         $strcourses  = get_string("courses");
         $strselect   = get_string("select");
diff --git a/course/search.php b/course/search.php
index e97ae12f65..5dd76458c4 100644
--- a/course/search.php
+++ b/course/search.php
@@ -66,9 +66,7 @@
 
         print_heading("$strsearchresults: $totalcount");
 
-        echo "<center>";
         print_paging_bar($totalcount, $page, $perpage, "search.php?search=$search&perpage=$perpage&");
-        echo "</center>";
 
         foreach ($courses as $course) {
             $course->fullname = highlight("$search", $course->fullname);
@@ -81,9 +79,7 @@
             print_spacer(5,5);
         }
 
-        echo "<center>";
         print_paging_bar($totalcount, $page, $perpage, "search.php?search=$search&perpage=$perpage&");
-        echo "</center>";
 
     } else {
         print_heading(get_string("nocoursesfound", "", $search));
diff --git a/lib/datalib.php b/lib/datalib.php
index 209544efaa..569ba3bf93 100644
--- a/lib/datalib.php
+++ b/lib/datalib.php
@@ -1108,8 +1108,7 @@ function get_site () {
 }
 
 
-function get_courses($categoryid="all", $sort="c.sortorder ASC", $fields="c.*", 
-                     &$totalcount, $limitfrom="", $limitnum="") {
+function get_courses($categoryid="all", $sort="c.sortorder ASC", $fields="c.*") {
 /// Returns list of courses, for whole site, or category
 
     global $USER, $CFG;
@@ -1132,6 +1131,37 @@ function get_courses($categoryid="all", $sort="c.sortorder ASC", $fields="c.*",
         $visiblecourses = "AND c.visible > 0";
     }
 
+    $selectsql = "{$CFG->prefix}course c $teachertable WHERE $categoryselect $visiblecourses";
+
+    return get_records_sql("SELECT $fields FROM $selectsql $teachergroup ORDER BY $sort");
+}
+
+
+function get_courses_page($categoryid="all", $sort="c.sortorder ASC", $fields="c.*", 
+                          &$totalcount, $limitfrom="", $limitnum="") {
+/// Returns list of courses, for whole site, or category
+/// Similar to get_courses, but allows paging
+
+    global $USER, $CFG;
+
+    $categoryselect = "";
+    if ($categoryid != "all") {
+        $categoryselect = "c.category = '$categoryid'";
+    }
+
+    $teachertable = "";
+    $teachergroup = "";
+    $visiblecourses = "";
+    if (!empty($USER)) {  // May need to check they are a teacher
+        if (!isadmin()) {
+            $visiblecourses = "AND ((c.visible > 0) OR (t.userid = '$USER->id' AND t.course = c.id))";
+            $teachertable = ", {$CFG->prefix}user_teachers t";
+            $teachergroup = "GROUP BY c.id";
+        }
+    } else {
+        $visiblecourses = "AND c.visible > 0";
+    }
+
     if ($limitfrom !== "") {
         switch ($CFG->dbtype) {
             case "mysql":
-- 
2.39.5