1. on the page:

I saw this paragraph:

"Beware of the filter cache when developing your module: it is advised to set your filter to 'no cache' while developing, but be sure to remove it again if it's not needed. You can clear the cache by running the SQL query 'DELETE FROM cache_filter';"

It should be DELETE * FROM cache_filter ... - note the "*" !

Am I right?

and

2. There is a typo in :

"Filters should not use the 'prepare' step for anything other than escaping, because that would short-circuits the control the user has over the order in which filters are applied."

it should say "would short-circuit " - without the s.

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Priority: Normal » Minor
Issue tags: +Novice

This refers to http://api.drupal.org/api/function/hook_filter/6

1) No. Actually, "DELETE FROM cache_filter" is the correct syntax to delete all the rows from the database table.

2) Yes, that is a typo and should be fixed. Changing status to "minor" and marking "Novice".

Just a note that hook_filter() has been overhauled in D7, so this is a D6-only issue.

If someone wants to create a patch for this, they could do a general copy edit of the documentation. The doc is in the Contrib repository in Drupal 6, in file docs/developer/hooks/core.php

rsvelko’s picture

rsvelko commented
Status: Active » Closed (won't fix)

wow, I did not know that docs is in cvs, nice ... Instead of minor, let's mark it with won't fix - nobody has the time to fix typos one by one.... Someone may run a recursive spell checker and do it in one patch ...

Maybe we should call this issue - check spelling of all docs in docs/ ??