This bug is with the location_views.module. Is this the right place for it?

A summary View with a Location: Country as an argument displays an empty string for the argument name in the summary title. The reason is that for location_views_handler_arg_country(), when $op == 'title', $query is the country code (argument value), not an object with a ->country field.

The patch is a one-line change:

diff -u -u -r1.7 location_views.module
--- modules/location/contrib/location_views/location_views.module       6 Jun 2006 00:30:55 -0000       1.7
+++ modules/location/contrib/location_views/location_views.module       27 Jun 2006 17:46:32 -0000
@@ -631,7 +631,7 @@
     case 'link':
       return l($country_name, "$arg/$query->country");
     case 'title':
-      return $country_name;
+      return $all_countries[$query];
   }
 }

Comments

karens’s picture

Project: Location » Location Views
Version: 6.x-3.x-dev »

Moving to location_views issues page. I'm checking this out now...

karens’s picture

Status: Needs review » Fixed

This is fixed. Thanks for the debugging.

bjaspan’s picture

You're welcome!

Barry

Anonymous’s picture

Status: Fixed » Closed (fixed)