If the throttle.module isn't enabled, the 'Throttle' table header and the Throttle checkboxes shouldn't be displayed. Within drupal, we never do this so - in sake of consistency and usability - this patch only displayes the 'throttle'-related fields when the module is enabled.
I also updated the helptext to be certain people know what todo to enable the throttle functionality. (Though maybe this helptext should be inside the throttle.module?)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Stefan Nagtegaal’s picture

Component: block.module » base system
FileSize
4.23 KB

And the second patch for system.module.

Pease review, comment or apply... :-)

Stefan Nagtegaal’s picture

Okay, I fixed a little issue inside the block.module-patch and now unified the two patches into one..

When this patch is applied, the 'Throttle'-fields inside the blocks-overview table and the modules-overview table doesn't get displayed.. Please test, comment or apply this patch...

Stefan Nagtegaal’s picture

Keeping up with CVS and modified the patch conform drupals coding styles..

Please, review and comment or apply!

Stefan Nagtegaal’s picture

Keeping up with CVS and modified the patch conform drupals coding styles..

Please, review and comment or apply!

Stefan Nagtegaal’s picture

new patch as requested by DRies

Dries’s picture

Committed to HEAD. Thanks.

Anonymous’s picture