added more clarity to required_by fields

2 files changed, 5 insertions, 8 deletions

diff --git a/main/models.py b/main/models.py
index d4250f3d..fb548e96 100644
--- a/main/models.py
+++ b/main/models.py
@@ -194,20 +194,17 @@ class Package(models.Model):
 
     def get_requiredby(self):
         """
-        Returns a list of tuples(2). 
-
-        Each tuple in the list is as follows: (packageid, packagename)
+        Returns a list of package objects.
         """
         reqs = []
         requiredby = PackageDepend.objects.filter(depname=self.pkgname).filter(
-            Q(pkg__arch=self.arch) | Q(pkg__arch__name__iexact='any')
-            ).order_by('depname')
+            Q(pkg__arch=self.arch) | Q(pkg__arch__name__iexact='any'))
         for req in requiredby:
-            reqs.append((req.pkg.id,req.pkg.pkgname))
+            reqs.append(req.pkg)
         ## sort the resultant list. Django has problems in the orm with
         ## trying to shoehorn the sorting into the reverse foreign key 
         ## reference in the query above. :(
-        reqs.sort(lambda a,b: cmp(a[1],b[1]))
+        reqs.sort(lambda a,b: cmp(a.pkgname,b.pkgname))
         return reqs
 
     def get_depends(self):
diff --git a/templates/packages/details.html b/templates/packages/details.html
index facb697f..7e8c7d36 100644
--- a/templates/packages/details.html
+++ b/templates/packages/details.html
@@ -70,7 +70,7 @@
                         <h4>Required By:</h4>
                         <ul style="font-size:small;list-style:none">
                         {% for req in pkg.get_requiredby %}
-                            <li><a href="/packages/{{ req.0 }}/">{{ req.1 }}</a></li>
+                            <li><a href="/packages/{{ req.id }}/">{{ req.pkgname }}{% ifequal req.repo.name "Testing" %}(testing){% endifequal %}{% ifequal req.repo.name "Unstable" %}(unstable){% endifequal %}</a></li>
                         {% endfor %}
                         </ul>
                     </div>