[documentation] [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)
+Status:       fixed

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




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




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

Thu, 15 Dec 2005 21:54:36 +0000 : drewish

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?




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

Mon, 17 Jul 2006 18:39:09 +0000 : Zach Harkey

+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.




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

Fri, 28 Jul 2006 14:23:06 +0000 : Zach Harkey

Seems to work now.