From eefa69ce5ce5ee471392bb8057b5aa617d0015f7 Mon Sep 17 00:00:00 2001
From: mark-nielsen <mark-nielsen>
Date: Sat, 30 Sep 2006 19:24:22 +0000
Subject: [PATCH] Bug Fix: The dependency drop-down was displaying incorrectly.
  Fixed up code to use API to gather Lessons correctly.

---
 mod/lesson/mod.html | 22 +++++++++-------------
 1 file changed, 9 insertions(+), 13 deletions(-)

diff --git a/mod/lesson/mod.html b/mod/lesson/mod.html
index b29d198d66..edf1a8872c 100644
--- a/mod/lesson/mod.html
+++ b/mod/lesson/mod.html
@@ -468,22 +468,18 @@ if ($form->mode == "add") {
     <td align="right"><b><?php  print_string("dependencyon", "lesson"); ?>:</b></td>
     <td align="left">
         <?php
-        $options = array();
-        $options[0] = get_string("none");
-        if (empty($form->instance)) {
-            $select = "course = $course->id";
-        } else {
-            $select = "course = $course->id AND id != $form->instance";
-        }
-        if ($lessons = get_records_select('lesson', $select, 'name ASC')) {
-            foreach($lessons as $lesson) {
-                $options[$lesson->id] = $lesson->name;
+            $options = array();
+            if ($lessons = get_all_instances_in_course('lesson', $course)) {
+                foreach($lessons as $lesson) {
+                    if ($form->instance != $lesson->id) {
+                        $options[$lesson->id] = format_string($lesson->name, true);
+                    }
+                }
             }
-        }
         
-        choose_from_menu($options, "dependency", $form->dependency, "");
+            choose_from_menu($options, 'dependency', $form->dependency, get_string('none'));
+            helpbutton("dependency", get_string("dependency", "lesson"), "lesson");
         ?>
-        <?php helpbutton("dependency", get_string("dependency", "lesson"), "lesson"); ?>
     </td>
 </tr>
 
-- 
2.39.5