[documentation] [task] Update nodeapi_example to 4.7

[drewish]

Issue status update for 
http://drupal.org/node/40936
Post a follow up: 
http://drupal.org/project/comments/add/40936

 Project:      Documentation
 Version:      <none>
 Component:    Developer Guide
 Category:     tasks
 Priority:     normal
 Assigned to:  drewish
 Reported by:  drewish
 Updated by:   drewish
 Status:       patch (code needs work)
 Attachment:   http://drupal.org/files/issues/nodeapi_example.module_0.patch (10.62 KB)

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




drewish



Previous comments:
------------------------------------------------------------------------

Mon, 12 Dec 2005 23:10:28 +0000 : drewish

Attachment: http://drupal.org/files/issues/menu.module_create_on_reset_0.patch (1.99 KB)

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


Comments?




------------------------------------------------------------------------

Mon, 12 Dec 2005 23:11:02 +0000 : drewish

Attachment: http://drupal.org/files/issues/nodeapi_example.module.patch (9.92 KB)

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




------------------------------------------------------------------------

Mon, 12 Dec 2005 23:41:48 +0000 : Bèr Kessels

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




------------------------------------------------------------------------

Mon, 12 Dec 2005 23:42:31 +0000 : Bèr Kessels

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




------------------------------------------------------------------------

Mon, 12 Dec 2005 23:44:14 +0000 : drewish

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




------------------------------------------------------------------------

Tue, 13 Dec 2005 12:49:45 +0000 : chx

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




------------------------------------------------------------------------

Tue, 13 Dec 2005 22:23:06 +0000 : drewish

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?




------------------------------------------------------------------------

Tue, 13 Dec 2005 22:36:44 +0000 : chx

yes you will see core form_alters for more




------------------------------------------------------------------------

Wed, 14 Dec 2005 23:07:13 +0000 : drewish

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?




------------------------------------------------------------------------

Wed, 14 Dec 2005 23:10:46 +0000 : chx

open menu.module and check menu_form_alter