django: fix lookup crash with multiple target fields

fixes #421

Author: timgraham

packages/django-google-spanner/django_spanner/lookups.py

@@ -122,10 +122,10 @@ def cast_param_to_float(self, compiler, connection):
         if isinstance(self.lhs.output_field, DecimalField):
             params[0] = float(params[0])
         # Cast remote field lookups that must be integer but come in as string.
-        elif (hasattr(self.lhs.output_field, 'target_field') and
-                self.lhs.output_field.target_field.rel_db_type(connection) == 'INT64' and
-                isinstance(params[0], str)):
-            params[0] = int(params[0])
+        elif hasattr(self.lhs.output_field, 'get_path_info'):
+            for i, field in enumerate(self.lhs.output_field.get_path_info()[-1].target_fields):
+                if field.rel_db_type(connection) == 'INT64' and isinstance(params[i], str):
+                    params[i] = int(params[i])
     return sql, params
 
 

packages/django-google-spanner/django_test_apps.txt

@@ -33,6 +33,7 @@ file_storage
 file_uploads
 fixtures
 fixtures_model_package
+foreign_object
 from_db_value
 get_earliest_or_latest
 get_object_or_404