Merge lp:~toabctl/loco-team-portal/duplicate-venues_fix-528828 into lp:loco-team-portal

[Thomas Bechtold]

[Thomas Bechtold (toabctl)]

Anything wrong with this fix? Why does nobody comment/merge it? Just busy?

[Michael Hall (mhall119)]

It looks good to me. Can we query the database before running the migration, to make sure there aren't currently any duplicates? If there are, it will probably fail to apply the database changes.

review: Approve

[Daniel Holbach (dholbach)]

Has this been merged? What exactly needs to be done? By the IS team?

[Michael Hall (mhall119)]

We just need to make sure that the data we currently have doesn't conflict with the new constraints before IS runs the migration script.

[Daniel Holbach (dholbach)]

What's the easiest way to do that? A test-case? A management command? What needs to be done with that data?

[Michael Hall (mhall119)]

We don't have that many venues, a simple visual check over the ones we do have should be enough.

[Thomas Bechtold (toabctl)]

i think there are no duplicates. would be good to make this schema change before we get more and more venues.

[Michael Hall (mhall119)]

I agree, Jono is pushing more use, so the longer we wait on this, the more venues we'll have to check

[Thomas Bechtold (toabctl)]

@mhall119: so please change status to "Approved" so we can merge this.

[Michael Hall (mhall119)]

Sorry, thought I had done that.

=== added file 'loco_directory/venues/migrations/0002_add_unique_together.py'
--- loco_directory/venues/migrations/0002_add_unique_together.py	1970-01-01 00:00:00 +0000
+++ loco_directory/venues/migrations/0002_add_unique_together.py	2010-02-28 09:34:14 +0000
@@ -0,0 +1,44 @@
+
+from south.db import db
+from django.db import models
+from venues.models import *
+
+class Migration:
+    
+    def forwards(self, orm):
+        
+        # Creating unique_together for [name, country, city] on Venue.
+        db.create_unique('venues_venue', ['name', 'country', 'city'])
+        
+        # Creating unique_together for [longitude, latitude] on Venue.
+        db.create_unique('venues_venue', ['longitude', 'latitude'])
+        
+    
+    
+    def backwards(self, orm):
+        
+        # Deleting unique_together for [longitude, latitude] on Venue.
+        db.delete_unique('venues_venue', ['longitude', 'latitude'])
+        
+        # Deleting unique_together for [name, country, city] on Venue.
+        db.delete_unique('venues_venue', ['name', 'country', 'city'])
+        
+    
+    
+    models = {
+        'venues.venue': {
+            'Meta': {'unique_together': "(('name', 'country', 'city'), ('longitude', 'latitude'))"},
+            'address': ('django.db.models.fields.CharField', [], {'max_length': '150', 'null': 'True', 'blank': 'True'}),
+            'city': ('django.db.models.fields.CharField', [], {'max_length': '150', 'null': 'True', 'blank': 'True'}),
+            'comment': ('django.db.models.fields.TextField', [], {'null': 'True', 'blank': 'True'}),
+            'country': ('django.db.models.fields.CharField', [], {'max_length': '150', 'null': 'True', 'blank': 'True'}),
+            'id': ('django.db.models.fields.AutoField', [], {'primary_key': 'True'}),
+            'latitude': ('django.db.models.fields.FloatField', [], {'null': 'True', 'blank': 'True'}),
+            'longitude': ('django.db.models.fields.FloatField', [], {'null': 'True', 'blank': 'True'}),
+            'map_url': ('django.db.models.fields.URLField', [], {'max_length': '200', 'null': 'True', 'blank': 'True'}),
+            'name': ('django.db.models.fields.CharField', [], {'max_length': '150'}),
+            'venue_url': ('django.db.models.fields.URLField', [], {'max_length': '200', 'null': 'True', 'blank': 'True'})
+        }
+    }
+    
+    complete_apps = ['venues']

=== modified file 'loco_directory/venues/models.py'
--- loco_directory/venues/models.py	2010-01-11 08:35:34 +0000
+++ loco_directory/venues/models.py	2010-02-28 09:34:14 +0000
@@ -23,6 +23,9 @@
     map_url = models.URLField(help_text=_('Map URL for the Venue'), verbose_name=_('URL of the Map'), max_length=200, verify_exists=False, blank=True, null=True)
     comment = models.TextField(help_text=_('Comment about the Venue'), blank=True, null=True)
     
+    class Meta:
+        unique_together = ( ('name', 'country', 'city'), ('longitude', 'latitude'), )
+
     #add manager 
     objects = VenueManager()