Re-rolling again.
- Added documentation to new function dblog_allowed_types_element().
- Make both $allowed_types and $known_types keyed by type name for consistency.
This version of the patch includes minor bug fixes. With the previous patch, in cases where dblog module was enabled and no message types are available in the database yet, we were getting php notices and an unfriendly user experience. This patch resolves these issues.
Re-rolling this patch post review for Drupal Gardens by David Rothstein at Acquia.
Removing unnecessary db update to dblog module. Also making syslog UI able to provide defaults independent of watchdog table (which may not exist if Database Logging is not installed).
Re-uploading patch that didn't upload correctly in comment #2.
Fixes minor issues in previous patch. Checkboxes FAPI elements do not have #multiple property. #options were hardcoded for two checkboxes where they shouldn't have been.
This patch includes both the proposed changes described above and the corresponding changes proposed to dblog here, http://drupal.org/node/1295182. It really doesn't make sense to do these separately because they're both form_alter()ing the same System form. From a UX perspective, to do one without or differently than the other is really backward.