Closed (fixed)
Project:
Webform
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Jan 2010 at 21:07 UTC
Updated:
4 Feb 2010 at 04:00 UTC
Jump to comment: Most recent file
1. Syntax error in string:
includes/webform.emails.inc
function webform_email_edit_form($form_state, &$node, $email = array()) {
...
case 'subject':
$default_value = _webform_filter_values(webform_variable_get('webform_default_subject'), $node);
$title = t('E-mail subject');
- $description = t('Aany textfield, select, or hidden form element may be selected as the subject for e-mails.');
+ $description = t('Any textfield, select, or hidden form element may be selected as the subject for e-mails.');
2. Slightly inaccurate attribute value (it's allowed, but not recommended):
the same function, a bit lower
// TODO: Allow easy re-use of existing templates.
$form['templates']['#tree'] = TRUE;
$form['templates']['default'] = array(
'#type' => 'textarea',
'#value' => $default_template,
- '#attributes' => array('style' => 'display: none'),
- '#attributes' => array('style' => 'display: none;'),
'#resizable' => FALSE,
);
3. Small mistake in comments:
includes/webform.admin.inc
/**
- * Menu callback for admin/webform/settings.
+ * Menu callback for admin/settings/webform.
*/
function webform_admin_settings() {
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | webform_minor_fixes.patch | 1.79 KB | quicksketch |
Comments
Comment #1
quicksketchThanks! Committed the attached patch containing the above changes.