[drupal-devel] [feature] Use any node type not just modulename-number

chx drupal-devel at drupal.org
Sun Jul 17 19:26:27 UTC 2005


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

 Project:      Drupal
 Version:      cvs
 Component:    node.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  chx
 Reported by:  chx
 Updated by:   chx
 Status:       patch
 Attachment:   http://drupal.org/files/issues/nodelistreverse_2.patch (7.27 KB)

Dries says
"
I'd remove the $pair parameter and fix the calling code to deal with
it. [...] it makes for a simpler api


"
So, here we are.




chx



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

1118791402 : chx

Attachment: http://drupal.org/files/issues/nodelistreverse.patch (1.45 KB)

Sometimes it comes handy to have the name of the module that defined a
node type. I modified a bit node_list so this is possible.




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

1118791571 : chx

Attachment: http://drupal.org/files/issues/nodelistreverse_0.patch (1.16 KB)




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

1118805190 : moshe weitzman

perhaps you missed the already existing node_get_module_name() function?




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

1118813229 : chx

Attachment: http://drupal.org/files/issues/nodelistreverse_1.patch (2.19 KB)

oh. I did. But then again, it artificially constrained the node types to
modulename-number. I have not really known about this contraint, and
see, it now serves as a good example of my patch.




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

1118834307 : killes at www.drop.org

+1


removes more code than it adds.







More information about the drupal-devel mailing list