You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@bloodhound.apache.org by rj...@apache.org on 2014/03/03 08:58:50 UTC

svn commit: r1573476 - /bloodhound/trunk/bloodhound_multiproduct/multiproduct/ticket/query.py

Author: rjollos
Date: Mon Mar  3 07:58:49 2014
New Revision: 1573476

URL: http://svn.apache.org/r1573476
Log:
0.8dev: Do not crash if product names or invalid prefix set in product column. Refs #709.

Patch by Olemis Lang.

Modified:
    bloodhound/trunk/bloodhound_multiproduct/multiproduct/ticket/query.py

Modified: bloodhound/trunk/bloodhound_multiproduct/multiproduct/ticket/query.py
URL: http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_multiproduct/multiproduct/ticket/query.py?rev=1573476&r1=1573475&r2=1573476&view=diff
==============================================================================
--- bloodhound/trunk/bloodhound_multiproduct/multiproduct/ticket/query.py (original)
+++ bloodhound/trunk/bloodhound_multiproduct/multiproduct/ticket/query.py Mon Mar  3 07:58:49 2014
@@ -72,9 +72,13 @@ class ProductQuery(Query):
         return self.cols
 
     def _get_ticket_href(self, prefix, tid):
-        env = lookup_product_env(self.env, prefix)
-        href = resolve_product_href(env, self.env)
-        return href.ticket(tid)
+        try:
+            env = lookup_product_env(self.env, prefix)
+        except LookupError:
+            return '#invalid-product-' + prefix
+        else:
+            href = resolve_product_href(env, self.env)
+            return href.ticket(tid)
 
     def get_href(self, href, id=None, order=None, desc=None, format=None,
                  max=None, page=None):