[drupal-devel] [bug] Node options are still looking for individual variable instead of an array

[Steven]

Issue status update for http://drupal.org/node/20868

 Project:      Drupal
 Version:      4.6.0
 Component:    blogapi.module
 Category:     bug reports
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  Prometheus6
 Updated by:   Steven
 Status:       patch

Commited to 4.6 / HEAD. Made the following code style changes:


'node_options_'.$edit['type'] to 'node_options_'. $edit['type']
no space between . and ", always a space otherwise


$_node_type_default to $node_type_default
no reason to prefix with an underscore




Steven



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

April 19, 2005 - 22:28 : Prometheus6

Attachment: http://drupal.org/files/issues/BLOGAPI_4_6.txt (1.17 KB)

blogapi.module can create (or attempt to..) any node type in 4.6, but
the code to create a new node still assumes the node will be of type
'blog'. In addition, it's still looking for individual variables rather
than the arrays that are stored in 4.6. 


It initializes those fields with calls like


variable_get('node_promote_blog', 1);
This will always return 1 unless you've upgraded to 4.6 from a previous
version and set your blog nodes not to publish by default, and it will
use that value for every type of node.


This patch retrieves the array (and "comment_$node->type") variable for
the appropriate node type and assigns the values accordingly.




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

April 20, 2005 - 20:20 : walkah

ACK! it's true... please apply to HEAD and DRUPAL-4-6.


+1