Merge lp:~cjohnston/summit/995789 into lp:summit

[Chris Johnston]

Removes code in summitmodel from lpupdate which deletes meetings that have a spec_url which isn't a launchpad blueprint

[Joey Stanford (joey)]

Could do with some drive by comments so others know what's happening in this block. We're deleting the deleting but it's not evident if you look at the code.

[Michael Hall (mhall119)]

Looks fine. I'm not worried about commenting this, as we're going to deprecate the whole thing as soon as we can make a cleaner break from Launchpad

review: Approve

=== modified file 'summit/schedule/models/summitmodel.py'
--- summit/schedule/models/summitmodel.py	2012-05-18 18:57:14 +0000
+++ summit/schedule/models/summitmodel.py	2012-06-11 15:26:24 +0000
@@ -273,13 +273,6 @@
             sprint_info = self._get_sprint_info_from_launchpad(url)
             in_lp |= self.update_from_launchpad_response(sprint_info)
 
-        in_db = set(m for m in self.meeting_set.exclude(spec_url=''))
-
-        for extra in in_db.difference(in_lp):
-            if not len(extra.agenda_set.all()):
-                print "will delete %s" % extra.name
-                extra.delete()
-
         self.last_update = datetime.utcnow()
         self.save()