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?)

CommentFileSizeAuthor
#5 system-block--throttle_0.patch6.78 KBStefan Nagtegaal
#2 system-block--throttle.patch6.53 KBStefan Nagtegaal
#1 system-throttle.patch4.23 KBStefan Nagtegaal
block-throttle.patch3.67 KBStefan Nagtegaal

Comments

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented
Component: block.module » base system
StatusFileSize
new system-throttle.patch4.23 KB

And the second patch for system.module.

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

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented
StatusFileSize
new system-block--throttle.patch6.53 KB

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

Stefan Nagtegaal commented

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

Please, review and comment or apply!

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented

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

Please, review and comment or apply!

Stefan Nagtegaal’s picture

Stefan Nagtegaal commented
StatusFileSize
new system-block--throttle_0.patch6.78 KB

new patch as requested by DRies

dries’s picture

dries commented

Committed to HEAD. Thanks.

Anonymous’s picture

(not verified) commented