From: skodak <skodak>
Date: Wed, 6 Dec 2006 09:57:42 +0000 (+0000)
Subject: MDL-7812 warning in sorting form in data module when no fields defined
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=196da4a45edb007a32c7300be3a38bdfc7c06a7f;p=moodle.git

MDL-7812 warning in sorting form in data module when no fields defined
---

diff --git a/mod/data/field.php b/mod/data/field.php
index e53de75966..dda2bf2abc 100755
--- a/mod/data/field.php
+++ b/mod/data/field.php
@@ -297,32 +297,32 @@
         helpbutton('fields', get_string('addafield','data'), 'data');
         echo '</div>';
 
-        echo '<div class="sortdefault" align="center">';
-        echo '<form name="sortdefault" action="'.$CFG->wwwroot.'/mod/data/field.php" method="get">';
-        echo '<input type="hidden" name="d" value="'.$data->id.'" />';
-        echo '<input type="hidden" name="mode" value="sort" />';
-        echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
-        echo '&nbsp;'.get_string('defaultsortfield','data').':';
-        $fields = get_records('data_fields','dataid',$data->id);
-        echo '<select name="defaultsort"><option value="0">'.get_string('dateentered','data').'</option>';
-        foreach ($fields as $field) {
-            if ($field->id == $data->defaultsort) {
-                echo '<option value="'.$field->id.'" selected="selected">'.$field->name.'</option>';
-            } else {
-                echo '<option value="'.$field->id.'">'.$field->name.'</option>';
+        if ($fields = get_records('data_fields','dataid',$data->id)) {
+            echo '<div class="sortdefault" align="center">';
+            echo '<form name="sortdefault" action="'.$CFG->wwwroot.'/mod/data/field.php" method="get">';
+            echo '<input type="hidden" name="d" value="'.$data->id.'" />';
+            echo '<input type="hidden" name="mode" value="sort" />';
+            echo '<input type="hidden" name="sesskey" value="'.sesskey().'" />';
+            echo '&nbsp;'.get_string('defaultsortfield','data').':';
+            echo '<select name="defaultsort"><option value="0">'.get_string('dateentered','data').'</option>';
+            foreach ($fields as $field) {
+                if ($field->id == $data->defaultsort) {
+                    echo '<option value="'.$field->id.'" selected="selected">'.$field->name.'</option>';
+                } else {
+                    echo '<option value="'.$field->id.'">'.$field->name.'</option>';
+                }
             }
+            echo '</select>';
+            echo '&nbsp;';
+            
+            $options = array(0 => get_string('ascending', 'data'),
+                             1 => get_string('descending', 'data'));
+            choose_from_menu($options, 'defaultsortdir', $data->defaultsortdir, '');
+            echo '<input type="submit" value="'.get_string('go').'" />';
+    
+            echo '</form>';
+            echo '</div>';
         }
-        echo '</select>';
-
-        echo '&nbsp;';
-        
-        $options = array(0 => get_string('ascending', 'data'),
-                         1 => get_string('descending', 'data'));
-        choose_from_menu($options, 'defaultsortdir', $data->defaultsortdir, '');
-        echo '<input type="submit" value="'.get_string('go').'" />';
-
-        echo '</form>';
-        echo '</div>';
 
 
     }