Trivial one here. We seem to be standardizing on the more-self-documenting fetchObject() when doing a single fetch, rather than just fetch(). So let's do that in cache.inc, too.

When I hit save, Eclipse also stripped out some trailing whitespace in the file. I figure for a patch this small that still doesn't harm any kittens.

CommentFileSizeAuthor
cache_fetch_object.patch2.28 KBCrell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Needs review » Reviewed & tested by the community

The trailing whitespace shouldn't be there anyway should they? I've noticed other patches have these trailing whitespace changes.

Crell’s picture

Crell CreditAttribution: Crell commented

Yes, trailing whitespace is discouraged. That's why I've recently set Eclipse to strip it automatically.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks!

Anonymous’s picture

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

Automatically closed -- issue fixed for two weeks with no activity.