The 3rd and 4th arguments for the nodeapi hook should be optional. I get warnings when nodeapi is called from node_load.
Patch coming up.

CommentFileSizeAuthor
#1 millennium-597014.patch822 bytestituomin

Comments

tituomin’s picture

tituomin commented
StatusFileSize
new millennium-597014.patch822 bytes
janusman’s picture

janusman commented

You're right... yikes =)

Need to actually test this, but seems like it will go straight in.

I feel me coming back to this code after what feels like a long hiatus will *surely* unearth more of these pearls of wisdom =) =)

tituomin’s picture

tituomin commented

That's perfectly ok! I love to have some good, working code to play with rather than be waiting for perfectly polished, bug-free code! =)

janusman’s picture

janusman commented
Status: Needs review » Fixed

Committed to 6.x-2.x DEV

tituomin’s picture

tituomin commented
Status: Fixed » Closed (fixed)