no syntax highlighting when plugin is enabled

FYI. Same error traceback thrown when browsing source.

I can imagine root cause and possible cure:

This plugin's !IRequestFilter method post_process_request extends the data dict for each request unconditionally. There are certainly a lot of requests, that have unexpected data dict content, or None at all, hence the error. I've never seen this before, and code should just be more specific about what requests to react upon, like so:

    def post_process_request(self, req, template, data, content_type):
        if template in ('ticket.html', 'wiki_edit.html'):
            data['keywords'] = self._get_keywords()
        return (template, data, content_type)

I bet this will instantly fix the issue. Anyone able to test this easily? I don't use this plugin myself as well.

Side-note: _get_keywords can hardly get different data without changing trac.ini, that would trigger a reload anyway. If/as soon as the plugin used separate file or wiki space for keyword storage, like was proposed in the TODO, then it could make sense to keep it that way. Or at least re-parse external source(s), for text-file still per request. For wiki pages since Trac 0.12 there is an IWikiChangeListener extension point to allow a more resource conscientious way to re-evaluate only on source change.

So I think this method don't need to be called on each request, but could the list could be prepared once (on class initialization) and get just referenced later on as self.keywords, right?

Of course all that might be overkill for just a few keywords, but I mention it for completeness and encouraging discussion about best-practice. Feel free to disagree or disregard as irrelevant.

I can reproduce the traceback, but not the syntax highlighting issue.

(In [11769]) Refs #9872:

• FIX: Only add to data dictionary if the request is for rendering a page in the ticket or wiki realm.
• Call _get_keywords on initialization and cache the data.
• Minor style and formatting changes.
• Deleted obsolete ClearSilver template.
• Added setup.cfg and set version to 0.3dev.

Thanks to hasienda for the excellent suggestions and guidance.

Hi. Just tried the "fix" and it looks good. Errors gone, and syntax highlighting is working fine. However, it was not parsing the [keywords] section in trac.ini. That is, I only got the default ("patch", "easy"), since section returned was always empty.

So, FWIW, I changed the keyword parsing line as follows and it works now.

--- web_ui.py.orig      2012-07-25 13:43:01.576275692 -0600
+++ web_ui.py.new       2012-07-25 13:42:49.672275608 -0600
@@ -92,7 +92,7 @@
             render_template(req, 'keywords.html', data, 'MarkupTemplate', True)
 
     def _get_keywords(self):
-        section = self.env.config.get('keywords', None)
+        section = self.env.config['keywords']
         if section:
             keywords = []
             for keyword in section:

Replying to asix:

So, FWIW, I changed the keyword parsing line as follows and it works now.

Thanks, I'll take a look now. I was being lazy and hadn't tested that.

(In [11788]) Refs #9872, #8799: Fixed defect introduced in [11771]: keywords would not be parsed from the keywords section in trac.ini. Thanks to asix for testing and reporting this.