Closed (fixed)
Project:
Drupal core
Component:
node system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
22 Apr 2005 at 09:17 UTC
Updated:
13 Mar 2006 at 19:02 UTC
Jump to comment: Most recent file
The "node" module isn't marked as one of the required modules, but it is used by a handful of other modules which are required. Specifically, the system and block modules both call the function node_list(), without using it through module_invoke.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | node.module.required.patch.txt | 1.62 KB | robertdouglass |
| #2 | node.module.required.txt | 1.39 KB | robertdouglass |
Comments
Comment #1
Thox commentedDuplicate
Comment #2
robertdouglass commentedI couldn't find the duplicate so I'm reopening this. There are 206 instances of node_list alone in the repository (core+contributions). I cycled through 50 or so of these, and none of them check for the existence of the node module. Our collective usage of the node module indicates that it is, in fact, a required module.
Comment #3
jo1ene commented+1
It makes sense to me. I never really thought about it, but this is a good point. While I'm not aware of any problems with folks disabling the node module, I think it makes sense to cover our collective behinds. The code looks fine and works as expected. It just adds "node" to the required array.
Comment #4
dries commentedAny idea of how hard it would be to fix the dependencies?
Comment #5
chx commentedI know someone who has a nice dependency patch awaiting commit... adding a few depencies after that is trivial :P
Comment #6
killes@www.drop.org commentedSo, what to do with this one? I recall some opposition from Goba to making node.module required. I myself would be in favour.
The patch doesn't apply anymore.
Comment #7
robertdouglass commentedHow fitting that just 5 minutes ago I got burned by this. Please commit... I see no sense in not requiring the node module. Show me a site (are there any?) that doesn't use the node module, and I'll show you a site maintainer who will know how to revert this patch, and I think that we're talking at most about .0001% of all Drupal sites. Yet serious problems do and will continue to arise because unknowing admins can disable what is arguably the central piece of Drupal functionality.
Comment #8
killes@www.drop.org commentedAck, applied.
Comment #9
(not verified) commented