$op = "prepare" not listed on hook_nodeapi docs [#47476]

Comment #1

dman commented 25 February 2006 at 02:23

Seconded!
I found it mentioned over in the upgrade overview and it seems that as op=validate doesn't take modifications anymore that "prepare" must be where I'm supposed to do that. ... but it doesn't.
The HEAD docs still have the old version. (this is why I hate putting docs in different places than the code, although I know it's neccessary sometimes)

node.module has

/**
 * Apply filters to a node in preparation for theming.
 */
function node_prepare($node, $teaser = FALSE) {

... which is not what I'm looking for at all (I thought).

I'm updating htmltidy.module, and I used to be able to correct users markup on-the-fly as they previewed it. (by validating and fixing the HTML on 'validate')

Do I have to to that in op=form now (feels wrong) or what?

.dan.

Log in or register to post comments

Comment #2

dman commented 25 February 2006 at 04:02

Just some notes (maybe off-topic)
Currently the 4.7 node.module node_form() function does NOT trigger a modules hook_nodeapi($op='form'...) like it used to with $op='form pre'.

node_validate() for example, passes on to
node_invoke_nodeapi($node, 'validate');
... which is to say, it allows any modules to validate the node.

Likewise, most of the $ops normally sprung by hook_node_api()
"prepare", "validate", "submit", "load" and "view" do anyway. But NOT "form".

I believe this is because of the new form_alter() method. I take it modules are supposed to use that to do things they used to do in hook_nodeapi($op='form pre') etc.

... but modifying fields there still doesn't reflect the changes on preview. ... still hunting through the code.