[drupal-devel] [feature] Deduce the module name from node type
Issue status update for http://drupal.org/node/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.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. chx
Issue status update for http://drupal.org/node/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_0.patch (1.16 KB) chx Previous comments: ------------------------------------------------------------------------ June 15, 2005 - 01:23 : 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.
Issue status update for http://drupal.org/node/25031 Project: Drupal Version: cvs Component: node.module Category: feature requests Priority: normal Assigned to: chx Reported by: chx Updated by: moshe weitzman Status: patch perhaps you missed the already existing node_get_module_name() function? moshe weitzman Previous comments: ------------------------------------------------------------------------ June 14, 2005 - 18:23 : 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. ------------------------------------------------------------------------ June 14, 2005 - 18:26 : chx Attachment: http://drupal.org/files/issues/nodelistreverse_0.patch (1.16 KB)
Issue status update for http://drupal.org/node/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_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. chx Previous comments: ------------------------------------------------------------------------ June 15, 2005 - 01:23 : 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. ------------------------------------------------------------------------ June 15, 2005 - 01:26 : chx Attachment: http://drupal.org/files/issues/nodelistreverse_0.patch (1.16 KB) ------------------------------------------------------------------------ June 15, 2005 - 05:13 : moshe weitzman perhaps you missed the already existing node_get_module_name() function?
Issue status update for http://drupal.org/node/25031 Project: Drupal Version: cvs Component: node.module Category: feature requests Priority: normal Assigned to: chx Reported by: chx Updated by: killes@www.drop.org Status: patch +1 removes more code than it adds. killes@www.drop.org Previous comments: ------------------------------------------------------------------------ June 15, 2005 - 01:23 : 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. ------------------------------------------------------------------------ June 15, 2005 - 01:26 : chx Attachment: http://drupal.org/files/issues/nodelistreverse_0.patch (1.16 KB) ------------------------------------------------------------------------ June 15, 2005 - 05:13 : moshe weitzman perhaps you missed the already existing node_get_module_name() function? ------------------------------------------------------------------------ June 15, 2005 - 07:27 : 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.
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@www.drop.org +1 removes more code than it adds.
participants (3)
-
chx -
killes -
moshe weitzman