Closed (won't fix)
Project:
Drupal core
Version:
7.x-dev
Component:
throttle.module
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
14 Sep 2006 at 21:48 UTC
Updated:
17 Apr 2008 at 14:24 UTC
Jump to comment: Most recent file
Comments
Comment #1
nickl commentedRework of merlinofchaos's original patch to move the throttle module settings to throttle.module.
Changes: Checking enabled modules from all modules retrieved by module_rebuild_cache to use .info file descriptions.
Removed '#suffix' from hidden fields as required modules are not displayed.
Added comments.
This patch only has the new additions to throttle.module and does not attempt to remove anything from system.module which will be accomplished by this patch: http://drupal.org/node/81631
Comment #2
nickl commentedRerolled to head since dependencies got committed: http://drupal.org/cvs?commit=41680
Removed throttle functionality from system.module.
Comment #3
chx commentedI would rename the checkboxes table header to "throttled" that describes what happens there.
You need a copy / clever reuse of the dependency system. Or at least prevent throttling of modules depended on.
Comment #4
nickl commentedI started implementing the dependency checking and then remembered a discussion we had when implementing the dependency system.
Excerpt from that discussion: http://drupal.org/node/81631#comment-133827
Correct me if I am wrong but the throttle system only disables certain cpu intensive functionality when webserver load gets too high and does not physically remove the module from the system. If we define the dependency system as modules that cannot work because of methods exposed by other modules then I do not see how throttling a dependency will have much affect on an unthrottled dependent since it still has access to the throttled module's methods for it is still installed on the system and its files are still included by drupal. Please advice...
Rerolled to fresh head...
Send description through t();
Change displayed name to $file->info['name'] instead of filename.
Fix theming to make displayed table look the same as the modules table.
Please nit pick...
Comment #5
chx commentedAlas, throttle removes the module, see module.inc
Comment #6
catchhttp://drupal.org/node/245504