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

Anonymous drupal-devel at drupal.org
Tue Feb 8 08:16:09 UTC 2005


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

+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


Anonymous



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

February 7, 2005 - 22: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.


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





More information about the drupal-devel mailing list