Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.There is now a required "type" attribute for each filter; the hook documentation should reflect this.
Type is one of the following as near as I can tell:
FILTER_TYPE_HTML_RESTRICTOR HTML tag and attribute restricting filters.
FILTER_TYPE_MARKUP_LANGUAGE Non-HTML markup language filters that generate HTML.
FILTER_TYPE_TRANSFORM_IRREVERSIBLE Irreversible transformation filters.
FILTER_TYPE_TRANSFORM_REVERSIBLE Reversible transformation filters.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 1917746-4-hook_filter_info-type.patch | 1.91 KB | cburschka |
| #2 | 1917746-2-hook_filter_info-type.patch | 1.69 KB | cburschka |
| #1 | 1917746-1-hook_filter_info-type.patch | 1.69 KB | cburschka |
Comments
Comment #1
cburschkaPatch.
Comment #2
cburschkaWhitespace
Comment #3
jhodgdonThanks! Your logic appears to be correct, based on what I can see in the Core implementations of this hook, and the constants defined in filter.module.
Can you make the list of filter types in the parameter into a bullet list that shows what each option does? See
http://drupal.org/node/1354#lists
for nested-list syntax. The definitions of the constants are at the bottom of this page:
http://api.drupal.org/api/drupal/core!modules!filter!filter.module/8
Comment #4
cburschkaThis should do it. (The linebreaks look a bit weird, but I'm guessing that comment blocks are supposed to break at c:80.)
Comment #5
jhodgdonPerfect! You'll get used to the line breaks if you do some more core API docs patches. This and other standards are listed on:
http://drupal.org/coding-standards/docs
And to find more issues to work on, look in the "documentation" component of Drupal Core.
Meanwhile -- thanks for the patch! Committed to 8.x.