[drupal-devel] [bug] Content type options editing broken

[Dries]

 Project:      Drupal
 Version:      cvs
 Component:    node system
 Category:     bug reports
 Priority:     normal
 Assigned to:  Goba
 Reported by:  Goba
 Updated by:   Dries
 Status:       patch

According to the PHPdoc, a $type-string can be used.  If I remember
correctly, this used to work?


Dries



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

February 8, 2005 - 00:06 : Goba

Attachment: http://drupal.org/files/issues/Drupal-fix-node-options-settings.patch (1.38 KB)

The node options settings implemented through the nodeapi 'settings' op
is incorrectly invoked. Only the type is passed on as a string, while
the nodeapi hook expects a node object. Therefore the type property of
the object is not possible to use in the hook, and it is impossible to
set and content type options.
One way to fix it is to use $node as a typename inside the nodeapi
settings op implementation, and the other way is to really pass an
object. A patch for the second option is attached.


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

February 8, 2005 - 10:16 : Anonymous

+1 for this fix, but please change double-quotes to single-quotes before
applying it... BTW: There was a duplicate [1] with a similar (although
slightly less elegant) patch.
[1] http://drupal.org/node/16509


-- 
View: http://drupal.org/node/16960
Edit: http://drupal.org/project/comments/add/16960