| Project: | Wordfilter |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
File: wordfilter.module
Function: wordfilter_filter()
Case: 'settings'
1.)
< '#description' => t('You can define a global list of words to be filtered on the <a href="%url">wordfilter settings page</a>.', array('%url' => url('admin/settings/wordfilter')))
---
> '#description' => t('You can define a global list of words to be filtered on the <a href="@url">wordfilter settings page</a>.', array('@url' => url('admin/settings/wordfilter')))
Replace %url by @url, otherwise the url is emphasized instead of run through check_plain.
2.)
< return drupal_get_form('wordfilter', $form);
---
> return $form;
Just return the $form because the callback function wordfilter() for rendering the form isn't defined in this module.
Comments
#1
Anty chance that these small changes can be implemented in CVS and a 1.0 rolled out anytime soon? please?? :)
#2
Attaching a .patch file to help the maintainer get it committed ;-)
#3
same basic patch, just more in line with the way fapi form elements are written.
#4
should be !url, not %url...
#5
Committed to 5 and HEAD.
#6
Automatically closed -- issue fixed for two weeks with no activity.