Modify

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#12779 closed defect (fixed)

RSS error - NameError: global name 'Chrome' is not defined

Reported by: kevin.winahradsky@… Owned by: kevin.winahradsky@…
Priority: normal Component: DiscussionPlugin
Severity: normal Keywords:
Cc: Trac Release:

Description

It appears that core.py is missing a from trac.web.chrome import Chrome import which is causing an error when trying to view RSS feeds for the discussions. Adding that in fixes the error for me. I am currently using version 0.11.

The full error

Trac[main] ERROR: Internal Server Error:
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/Trac-1.0.1-py2.7.egg/trac/web/main.py", line 497, in _dispatch_request
    dispatcher.dispatch(req)
  File "/usr/local/lib/python2.7/dist-packages/Trac-1.0.1-py2.7.egg/trac/web/main.py", line 214, in dispatch
    resp = chosen_handler.process_request(req)
  File "build/bdist.linux-x86_64/egg/tracdiscussion/core.py", line 125, in process_request
    req, in_type, context.resource, format, filename=None),
  File "/usr/local/lib/python2.7/dist-packages/Trac-1.0.1-py2.7.egg/trac/mimeview/api.py", line 1015, in send_converted
    content, selector)
  File "/usr/local/lib/python2.7/dist-packages/Trac-1.0.1-py2.7.egg/trac/mimeview/api.py", line 691, in convert_content
    output = converter.convert_content(req, mimetype, content, ck)
  File "build/bdist.linux-x86_64/egg/tracdiscussion/core.py", line 53, in convert_content
    return self._export_rss(req, resource)
  File "build/bdist.linux-x86_64/egg/tracdiscussion/core.py", line 64, in _export_rss
    output = Chrome(self.env).render_template(req, template, data,
NameError: global name 'Chrome' is not defined

Attachments (0)

Change History (3)

comment:1 Changed 8 years ago by Ryan J Ollos

Owner: set to Ryan J Ollos
Status: newaccepted

Issue was introduced in [6310] and not corrected in refactoring [13936].

comment:2 Changed 8 years ago by Ryan J Ollos

Resolution: fixed
Status: acceptedclosed

In 15556:

0.9dev/0.10dev: Fix missing import of Chrome

Patch by kevin.winahradsky@….

Fixes #12779.

comment:3 Changed 8 years ago by Ryan J Ollos

Owner: changed from Ryan J Ollos to kevin.winahradsky@…

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain kevin.winahradsky@….
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.