summaryrefslogtreecommitdiff
path: root/packages
diff options
context:
space:
mode:
Diffstat (limited to 'packages')
-rw-r--r--packages/models.py3
-rw-r--r--packages/views/display.py46
2 files changed, 44 insertions, 5 deletions
diff --git a/packages/models.py b/packages/models.py
index 45ff3c08..18a62a3c 100644
--- a/packages/models.py
+++ b/packages/models.py
@@ -297,6 +297,9 @@ class Update(models.Model):
return u'%d:%s-%s' % (self.new_epoch, self.new_pkgver, self.new_pkgrel)
return u'%s-%s' % (self.new_pkgver, self.new_pkgrel)
+ def elsewhere(self):
+ return Package.objects.filter(pkgname=self.pkgname, arch=self.arch)
+
def __unicode__(self):
return u'%s of %s on %s' % (self.get_action_flag_display(),
self.pkgname, self.created)
diff --git a/packages/views/display.py b/packages/views/display.py
index dd57571a..5332dc13 100644
--- a/packages/views/display.py
+++ b/packages/views/display.py
@@ -1,18 +1,21 @@
+import datetime
import json
from string import Template
from urllib import urlencode
from django.http import HttpResponse, Http404
-from django.shortcuts import get_object_or_404, redirect
+from django.shortcuts import get_object_or_404, redirect, render
+from django.utils.timezone import now
from django.views.generic.simple import direct_to_template
from django.views.decorators.vary import vary_on_headers
from main.models import Package, PackageFile, Arch, Repo
from mirrors.utils import get_mirror_url_for_download
+from ..models import Update
from ..utils import get_group_info, PackageJSONEncoder
-def split_package_details(request, name='', repo='', arch=''):
+def split_package_details(request, name, repo, arch):
arch = get_object_or_404(Arch, name=arch)
arches = [ arch ]
arches.extend(Arch.objects.filter(agnostic=True))
@@ -21,10 +24,10 @@ def split_package_details(request, name='', repo='', arch=''):
repo__testing=repo.testing, repo__staging=repo.staging,
arch__in=arches).order_by('pkgname')
if len(pkgs) == 0:
- raise Http404
+ return None
# we have packages, but ensure at least one is in the given repo
if not any(True for pkg in pkgs if pkg.repo == repo):
- raise Http404
+ return None
context = {
'list_title': 'Split Package Details',
'name': name,
@@ -35,6 +38,30 @@ def split_package_details(request, name='', repo='', arch=''):
context)
+CUTOFF = datetime.timedelta(days=60)
+
+
+def recently_removed_package(request, name, repo, arch, cutoff=CUTOFF):
+ '''We're just steps away from raising a 404, but check our packages update
+ table first to see if this package has existed in this repo before. If so,
+ we can show a 410 Gone page and point the requester in the right
+ direction.'''
+ arch = get_object_or_404(Arch, name=arch)
+ arches = [ arch ]
+ arches.extend(Arch.objects.filter(agnostic=True))
+ match = Update.objects.select_related('arch', 'repo').filter(
+ pkgname=name, repo__name__iexact=repo, arch__in=arches)
+ if cutoff is not None:
+ when = now() - cutoff
+ match = match.filter(created__gte=when)
+ try:
+ match = match.latest()
+ return render(request, 'packages/removed.html',
+ {'update': match, }, status=410)
+ except Update.DoesNotExist:
+ return None
+
+
def details(request, name='', repo='', arch=''):
if all([name, repo, arch]):
try:
@@ -54,7 +81,16 @@ def details(request, name='', repo='', arch=''):
repo__name__iexact=repo, arch__agnostic=True)
if len(pkgs) == 1:
return redirect(pkgs[0], permanent=True)
- return split_package_details(request, name, repo, arch)
+ # do we have a split package matching this criteria?
+ ret = split_package_details(request, name, repo, arch)
+ if ret is None:
+ # maybe we have a recently-removed package?
+ ret = recently_removed_package(request, name, repo, arch)
+ if ret is not None:
+ return ret
+ else:
+ # we've tried everything at this point, nothing to see
+ raise Http404
else:
pkg_data = [
('arch', arch.lower()),