Differentiate between no filelist and empty filelist

We had these two cases munged together before; some packages have seen
filelist updates but simply don't have any files ('firefox-i18n' for
example).

Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
Dan McGee 2011-06-20 01:09:20 -05:00
parent 00eb8f426a
commit 92e13e0670
2 changed files with 10 additions and 3 deletions

View File

@ -290,11 +290,14 @@ def files(request, name, repo, arch):
pkg = get_object_or_404(Package, pkg = get_object_or_404(Package,
pkgname=name, repo__name__iexact=repo, arch__name=arch) pkgname=name, repo__name__iexact=repo, arch__name=arch)
fileslist = PackageFile.objects.filter(pkg=pkg).order_by('directory', 'filename') fileslist = PackageFile.objects.filter(pkg=pkg).order_by('directory', 'filename')
context = {
'pkg': pkg,
'files': fileslist,
}
template = 'packages/files.html' template = 'packages/files.html'
if request.is_ajax(): if request.is_ajax():
template = 'packages/files-list.html' template = 'packages/files-list.html'
return direct_to_template(request, template, return direct_to_template(request, template, context)
{'pkg':pkg, 'files':fileslist})
def details_json(request, name, repo, arch): def details_json(request, name, repo, arch):
pkg = get_object_or_404(Package, pkg = get_object_or_404(Package,

View File

@ -2,12 +2,16 @@
<p class="message">Note: This file list was generated from a previous version <p class="message">Note: This file list was generated from a previous version
of the package; it may be out of date.</p> of the package; it may be out of date.</p>
{% endif %} {% endif %}
{% if files.count %} {% if pkg.files_last_update %}
{% if files|length %}
<ul> <ul>
{% for file in files %} {% for file in files %}
<li>{{ file.directory }}{{ file.filename|default:'' }}</li> <li>{{ file.directory }}{{ file.filename|default:'' }}</li>
{% endfor %} {% endfor %}
</ul> </ul>
{% else %} {% else %}
<p class="message">Package has no files.</p>
{% endif %}
{% else %}
<p class="message">No file list available.</p> <p class="message">No file list available.</p>
{% endif %} {% endif %}