I know this is minor, but it's my first bug report against CVS (5.0) Drupal.

On the page caching page, the text says:

Enabling the cache will offer a sufficient performance boost.

It seems to me that 'sufficient' isn't really the right word. "Sufficient for what", is the question it raises.

I would suggest that 'significant' is more appropriate.

Minor, perhaps, but I hope that it helps.

CommentFileSizeAuthor
#2 system.module_r1.3701.62 KBAjK

Comments

mbchandar’s picture

mbchandar commented

i agree with you. this could be changed in the modules/system/system.module in line no 36

AjK’s picture

AjK commented
Status: Active » Needs review
StatusFileSize
new system.module_r1.3701.62 KB

Always better if you can roll a patch like the one supplied

moshe weitzman’s picture

moshe weitzman commented
Version: x.y.z » 5.0-beta2
Status: Needs review » Reviewed & tested by the community

applies with fuzz.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Title: 'sufficient' should probably be 'significant' » String freeze: 'sufficient' should probably be 'significant'

Affects the string freeze.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

ChrisKennedy’s picture

ChrisKennedy commented

Just fyi, this text is about to be replaced in http://drupal.org/node/100516 anyway.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)