comment_nodeapi(), node_nodeapi() not compatible w/ calls in node_types_configure()... homee

Howdy, when node.module calls invoke_nodeapi() with param $op='settings', the 2nd parameter is a string that is the node type module's name.

However, when I look at the source HTML for the form generated at q='admin/node/configure/types/blog', the form contains variable names like 'comment_' and 'node_options_' rather than 'comment_blog' and 'node_options_blog'.

The problem is that the function node_types_configure() passes $type rather than a $node with a $node->type field. The way comment_nodeapi() and node_nodeapi() are coded for param $op='settings' show that those two functions expect an object with a 'type' field (i.e., $node w/ $node->type, rather than just passing $type).

I have a patch here that addresses the issue, but I'm not sure whether you guys want to change the way comment_nodeapi() and node_nodeapi() are written, or whether you want to do it this way (or something similar to this way).

-Ankur