MDL-46284 tool_messageinbound: Handle checkboxes correctly
authorAndrew Nicols <andrew@nicols.co.uk>
Wed, 8 Oct 2014 07:10:58 +0000 (15:10 +0800)
committerAndrew Nicols <andrew@nicols.co.uk>
Wed, 8 Oct 2014 07:19:17 +0000 (15:19 +0800)
When a checkbox is not checked, it does not submit in the form, therefore
this must be an !empty() test instead of an (int) casting.

This issue is a part of the MDL-47194 Task.
This issue is a part of the MDL-39707 Epic.

admin/tool/messageinbound/index.php

index 8aa16ad..45d8f7f 100644 (file)
@@ -62,11 +62,11 @@ if (empty($classname)) {
         $record->defaultexpiration = (int) $data->defaultexpiration;
 
         if ($handler->can_change_validateaddress()) {
-            $record->validateaddress = (int) $data->validateaddress;
+            $record->validateaddress = !empty($data->validateaddress);
         }
 
         if ($handler->can_change_enabled()) {
-            $record->enabled = (int) $data->enabled;
+            $record->enabled = !empty($data->enabled);
         }
         $DB->update_record('messageinbound_handlers', $record);
         redirect($PAGE->url);