From 6c31d376c2e4d7115f1455d6dfdff09d6c8cdc6d Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Wed, 13 Dec 2006 09:53:59 +0000
Subject: [PATCH] MDL-5696 fixed 'Checkbox' actually allows 'checkboxes' and
 empty 'options' (radio buttons too)

---
 mod/data/field/checkbox/field.class.php    | 3 +++
 mod/data/field/radiobutton/field.class.php | 3 +++
 2 files changed, 6 insertions(+)

diff --git a/mod/data/field/checkbox/field.class.php b/mod/data/field/checkbox/field.class.php
index a5912ac408..93aa699efc 100755
--- a/mod/data/field/checkbox/field.class.php
+++ b/mod/data/field/checkbox/field.class.php
@@ -46,6 +46,9 @@ class data_field_checkbox extends data_field_base {
         $i = 0;
         foreach (explode("\n", $this->field->param1) as $checkbox) {
             $checkbox = trim($checkbox);
+            if ($checkbox === '') {
+                continue; // skip empty lines
+            }
             $str .= '<input type="checkbox" id="field_'.$this->field->id.'_'.$i.'" name="field_' . $this->field->id . '[]" ';
             $str .= 'value="' . s($checkbox) . '" ';
             
diff --git a/mod/data/field/radiobutton/field.class.php b/mod/data/field/radiobutton/field.class.php
index a208898735..cd1a4005a2 100755
--- a/mod/data/field/radiobutton/field.class.php
+++ b/mod/data/field/radiobutton/field.class.php
@@ -46,6 +46,9 @@ class data_field_radiobutton extends data_field_base {
         $i = 0;
         foreach (explode("\n",$this->field->param1) as $radio) {
             $radio = trim($radio);
+            if ($radio === '') {
+                continue; // skip empty lines
+            }
             $str .= '<input type="radio" id="field_'.$this->field->id.'_'.$i.'" name="field_' . $this->field->id . '" ';
             $str .= 'value="' . s($radio) . '" ';
 
-- 
2.39.5