euscanwww: don't delete packages when deleting related versions

Signed-off-by: Corentin Chary <corentincj@iksaif.net>

euscanwww/api/handlers.py

@@ -7,6 +7,11 @@ from django.forms.models import model_to_dict
 
 from euscan.models import Version, Package, Herd, Maintainer, EuscanResult, VersionLog
 from euscan.forms import WorldForm, PackagesForm
+from api.emitters import EuscanXMLEmitter
+
+# replace default XMLEmitter with ours
+from piston.emitters import Emitter
+Emitter.register('xml', EuscanXMLEmitter, 'text/xml; charset=utf-8')
 
 def xint(i):
     try:
@@ -75,7 +80,7 @@ class MaintainersHandler(AnonymousBaseHandler):
         maintainers = renameFields(maintainers, [('maintainers__id', 'id'),
                                                  ('maintainers__name', 'name'),
                                                  ('maintainers__email', 'email')])
-        return maintainers
+        return { 'maintainers'  : maintainers }
 
 # /api/1.0/herds
 class HerdsHandler(AnonymousBaseHandler):
@@ -89,7 +94,7 @@ class HerdsHandler(AnonymousBaseHandler):
                                                      n_versions=Sum('n_versions'))
 
         herds = renameFields(herds, [('herds__herd', 'herd')])
-        return herds
+        return { 'herds' : herds }
 
 # /api/1.0/categories
 class CategoriesHandler(AnonymousBaseHandler):
@@ -101,7 +106,7 @@ class CategoriesHandler(AnonymousBaseHandler):
                                          n_overlay=Sum('n_overlay'),
                                          n_versions=Sum('n_versions'))
 
-        return categories
+        return { 'categories' : categories }
 
 # /api/1.0/packages/by-maintainer/
 # /api/1.0/packages/by-category/
@@ -189,4 +194,3 @@ class PackageHandler(AnonymousBaseHandler):
             package['log'] = model_to_dict(log, ['result', 'datetime'])
 
         return package
-

euscanwww/euscan/models.py

@@ -32,11 +32,14 @@ class Package(models.Model):
 
     ' And we also pre-compute last versions '
     last_version_gentoo = models.ForeignKey('Version', blank=True, null=True,
-                                            related_name="last_version_gentoo")
+                                            related_name="last_version_gentoo",
+                                            on_delete=models.SET_NULL)
     last_version_overlay = models.ForeignKey('Version', blank=True, null=True,
-                                             related_name="last_version_overlay")
+                                             related_name="last_version_overlay",
+                                             on_delete=models.SET_NULL)
     last_version_upstream = models.ForeignKey('Version', blank=True, null=True,
-                                              related_name="last_version_upstream")
+                                              related_name="last_version_upstream",
+                                              on_delete=models.SET_NULL)
 
     def __unicode__(self):
         return '%s/%s' % (self.category, self.name)

euscanwww/euscan/views.py

@@ -101,10 +101,25 @@ def overlay(request, overlay):
 
 @render_to('euscan/package.html')
 def package(request, category, package):
+
+    def version_key(version):
+        from distutils.version import StrictVersion, LooseVersion
+
+        version = version.version
+        try:
+            return StrictVersion(version)
+        # in case of abnormal version number, fall back to LooseVersion
+        except ValueError:
+            return LooseVersion(version)
+
     package = get_object_or_404(Package, category=category, name=package)
     package.homepages = package.homepage.split(' ')
-    packaged = Version.objects.filter(package=package, packaged=True).order_by('version', 'revision')
-    upstream = Version.objects.filter(package=package, packaged=False).order_by('version', 'revision')
+    packaged = Version.objects.filter(package=package, packaged=True)
+    upstream = Version.objects.filter(package=package, packaged=False)
+
+    packaged = sorted(packaged, key=version_key)
+    upstream = sorted(upstream, key=version_key)
+
     log = EuscanResult.objects.filter(package=package).order_by('-datetime')[:1]
     log = log[0] if log else None
     vlog = VersionLog.objects.filter(package=package).order_by('-id')