incubator-bloodhound-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Apache Bloodhound" <>
Subject Re: [Apache Bloodhound] #330: Lazy evaluation of
Date Sun, 27 Jan 2013 12:58:04 GMT
#330: Lazy evaluation of
  Reporter:  peter        |      Owner:  nobody
      Type:  enhancement  |     Status:  closed
  Priority:  minor        |  Milestone:
 Component:  dashboard    |    Version:
Resolution:  fixed        |   Keywords:

Comment (by andrej):

 I can see the similar behavior when rendering an error page on local env
 (rev 1439061).  The error occurs when an exception is trigged during
 request dispatching by handler. For example, use built-in search and set
 page parameter that is out of range e.g.

 You've got a plain text error page indicating that there is an error in
 ... bloodhound_theme.html", line 278, in <Expression
 has no member named "labels"

 As far as I understand the code, the problem is in
 bloodhound_theme/bhtheme/, line 217
     def post_process_request(self, req, template, data, content_type):
         """Post process request filter.
         Removes all trac provided css if required"""

         if template is None and data is None:
             return template, data, content_type
 ...['labels'] = self._get_whitelabelling()

 In case of an exception during request processing, the
 post_process_request method is called with all None parameters, so method
 returns before {{{['labels']}}} is filled.

 Should we move line
 {{{['labels'] = self._get_whitelabelling()}}}
 before the if/return statement?

Ticket URL: <>
Apache Bloodhound <>
The Apache Bloodhound (incubating) issue tracker

View raw message