I'm not sure what versions this affects but in at least one branch, the first line of code in location_views.module is this:

define('LOCATION_VIEWS_UNKNOWN', t('unknown'));

This is a disaster for performance because it causes locale_refresh_cache to be called on every page load. This can cause hundreds of queries.

CommentFileSizeAuthor
#1 location_views-DRUPAL-5--netcologne.patch4.71 KBuli

Comments

uli’s picture

uli
he/his
Primary language German
commented
Status: Active » Needs review
StatusFileSize
new location_views-DRUPAL-5--netcologne.patch4.71 KB

The solution is to take the constant out and replace it with t('unknown').

robertdouglass’s picture

robertdouglass commented
Status: Needs review » Closed (duplicate)

http://drupal.org/node/193089

However the patch at 193089 takes the point of view that 'unknown' need not be translated. I'm unsure because I haven't read the code carefully, so decide which patch to use based on some study of the use of the string in the module.