Author: astaric
Date: Sun Mar  2 06:52:21 2014
New Revision: 1573264

URL: http://svn.apache.org/r1573264
Log:
Use get to access product in search documents. Refs: #773


Modified:
    
bloodhound/trunk/bloodhound_search/bhsearch/search_resources/milestone_search.py
    
bloodhound/trunk/bloodhound_search/bhsearch/search_resources/ticket_search.py
    bloodhound/trunk/bloodhound_search/bhsearch/search_resources/wiki_search.py
    bloodhound/trunk/bloodhound_search/bhsearch/security.py
    bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py

Modified: 
bloodhound/trunk/bloodhound_search/bhsearch/search_resources/milestone_search.py
URL: 
http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_search/bhsearch/search_resources/milestone_search.py?rev=1573264&r1=1573263&r2=1573264&view=diff
==============================================================================
--- 
bloodhound/trunk/bloodhound_search/bhsearch/search_resources/milestone_search.py
 (original)
+++ 
bloodhound/trunk/bloodhound_search/bhsearch/search_resources/milestone_search.py
 Sun Mar  2 06:52:21 2014
@@ -171,4 +171,8 @@ class MilestoneSearchParticipant(BaseSea
         #TODO: add better milestone rendering
 
         name = res['hilited_name'] or res['name']
-        return tag(u'[', res['product'], u'] Milestone:', name)
+        product = res.get('product')
+        if product:
+            return tag(u'[', product, u'] Milestone:', name)
+        else:
+            return tag(u'Milestone:', name)

Modified: 
bloodhound/trunk/bloodhound_search/bhsearch/search_resources/ticket_search.py
URL: 
http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_search/bhsearch/search_resources/ticket_search.py?rev=1573264&r1=1573263&r2=1573264&view=diff
==============================================================================
--- 
bloodhound/trunk/bloodhound_search/bhsearch/search_resources/ticket_search.py 
(original)
+++ 
bloodhound/trunk/bloodhound_search/bhsearch/search_resources/ticket_search.py 
Sun Mar  2 06:52:21 2014
@@ -248,6 +248,10 @@ class TicketSearchParticipant(BaseSearch
                 stat = res[TicketFields.STATUS]
 
         id = res['hilited_id'] or res['id']
-        id = tag.span('#', id, class_=css_class)
+        id = tag.span(u'#', id, class_=css_class)
         summary = res['hilited_summary'] or res['summary']
-        return tag('[', res['product'], '] ', id, ': ', summary, ' (%s)' % 
stat)
+        product = res.get('product')
+        if product:
+            return tag(u'[', product, u'] ', id, u': ', summary, u' (%s)' % 
stat)
+        else:
+            return tag(id, u': ', summary, u' (%s)' % stat)

Modified: 
bloodhound/trunk/bloodhound_search/bhsearch/search_resources/wiki_search.py
URL: 
http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_search/bhsearch/search_resources/wiki_search.py?rev=1573264&r1=1573263&r2=1573264&view=diff
==============================================================================
--- bloodhound/trunk/bloodhound_search/bhsearch/search_resources/wiki_search.py 
(original)
+++ bloodhound/trunk/bloodhound_search/bhsearch/search_resources/wiki_search.py 
Sun Mar  2 06:52:21 2014
@@ -161,4 +161,8 @@ class WikiSearchParticipant(BaseSearchPa
 
     def format_search_results(self, res):
         title = res['hilited_name'] or res['name']
-        return tag('[', res['product'], '] ', title)
+        product = res.get('product')
+        if product:
+            return tag(u'[', product, u'] ', title)
+        else:
+            return tag(title)

Modified: bloodhound/trunk/bloodhound_search/bhsearch/security.py
URL: 
http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_search/bhsearch/security.py?rev=1573264&r1=1573263&r2=1573264&view=diff
==============================================================================
--- bloodhound/trunk/bloodhound_search/bhsearch/security.py (original)
+++ bloodhound/trunk/bloodhound_search/bhsearch/security.py Sun Mar  2 06:52:21 
2014
@@ -42,7 +42,7 @@ class SecurityPreprocessor(Component):
             self._required_permissions[doc_type] = permission
 
     def check_permission(self, doc, context):
-        product, doctype, id = doc['product'], doc['type'], doc['id']
+        product, doctype, id = doc.get('product'), doc['type'], doc['id']
         username = context.req.authname
         env = self.env
         if product:

Modified: bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py
URL: 
http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py?rev=1573264&r1=1573263&r2=1573264&view=diff
==============================================================================
--- bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py (original)
+++ bloodhound/trunk/bloodhound_search/bhsearch/web_ui.py Sun Mar  2 06:52:21 
2014
@@ -697,7 +697,7 @@ class RequestContext(object):
 
     def _process_doc(self, doc):
         ui_doc = dict(doc)
-        if doc['product']:
+        if doc.get('product'):
             env = ProductEnvironment(self.env, doc['product'])
             product_href = ProductEnvironment.resolve_href(env, self.env)
             # pylint: disable=too-many-function-args


Reply via email to