I've updated the example module to use the hook_form_alter() instead of hook_nodeapi().

Comments?

CommentFileSizeAuthor
#10 nodeapi_example.module_0.patch10.62 KBdrewish
#1 nodeapi_example.module.patch9.92 KBdrewish
menu.module_create_on_reset_0.patch1.99 KBdrewish

Comments

drewish’s picture

drewish commented
StatusFileSize
new nodeapi_example.module.patch9.92 KB

crap, wrong patch. i'll try it again.

Bèr Kessels’s picture

Bèr Kessels commented

Am I tired, or did you atatch the wrong patch?

Bèr Kessels’s picture

Bèr Kessels commented

heh :) Iam tired and you commited the wrong patch :)

drewish’s picture

drewish commented

hehe, i was just about to write back "both" ;)

chx’s picture

chx commented
Status: Active » Needs work

type is not set always in the form -- form_alter is called for all forms.

drewish’s picture

drewish commented

chx, i thought the comment made it clear that it was only set on forms that dealt with nodes... or are you saying that i'll throw warning/notices because that value isn't set?

chx’s picture

chx commented

yes you will see core form_alters for more

drewish’s picture

drewish commented

chx, I'm sorry I don't understand what your saying. What exactly is the problem with the given code and what should be changed?

chx’s picture

chx commented

open menu.module and check menu_form_alter

drewish’s picture

drewish commented
StatusFileSize
new nodeapi_example.module_0.patch10.62 KB

i took chx's cryptic hint and added a test to see if $form['type'] is set. comments?

zach harkey’s picture

zach harkey commented

+1?

See, I don't know if this works or not. Trying to learn, a task made far more difficult when official documentation that is presented as current is out-of-date.

zach harkey’s picture

zach harkey commented

Seems to work now.

drewish’s picture

drewish commented
Status: Needs work » Fixed

ah, yeah, i'd updated the docs but forgot to close the issue.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)