[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