hook_nodeapi_X => hook_node_X

Under orders to say "hell yeah".

That's so cool (@webchick).

Amen and pass the sweet potatoes!

Here's what should be added to http://drupal.org/node/224333

<h3 id="replace_hook_nodeapi_op_with_hook_node_op">Replace hook_nodeapi_$op() with hook_node_$op()</h3>
<a href="http://drupal.org/node/383066">(issue)</a>.

Drupal 6.x:
<?php
 /**
 * Implementation of hook_nodeapi_view.
  */
function blog_nodeapi_view($node, $teaser = FALSE) {
?>

Drupal 7.x:
<?php
 /**
 * Implementation of hook_node_view.
  */
function blog_node_view($node, $teaser = FALSE) {
?>

Looks like you also need to add in a bit about node_invoke_nodeapi() being renamed to node_invoke_node()

I advise against this. We need to rename hooks like hook_load() to hook_node_load(), since hook names start with the name of the module that introduces them, and this patch would conflict with that.

@Xano this is already committed

My bad, but my advise still stands. Hook_load(), hook_form(), etc. are confusing and make the API even more sloppy. If we change them to hook_node_load(), etc., we need to think of al alternative for hook_nodeapi_X().

@Xano: *_load(), *_form(), etc. are not really hooks as there are not called by module_invoke_all(). There are a rather dumb implementation of a strategy pattern. We need a better system for D7, for sure (probably based on some DrupalNodeType interface), renaming *_load() to *_node_load() would only add to the confusion.

See #395786: Move node-type callbacks to a proper interface.

With Damien's issue #395786, I'm setting this back to CNW for documentation.

I added this into the docs at http://drupal.org/update/modules/6/7#hook_node_xxx

I attempted to update all previous references (now outdated) to hook_nodeapi_xxx() as well.