Modify

Opened 6 years ago

Closed 6 years ago

#3187 closed defect (fixed)

There are still some leftovers from ClearSilver

Reported by: anonymous Owned by: proofek
Priority: high Component: PeerReviewPlugin
Severity: major Keywords: bug
Cc: Trac Release: 0.11

Description (last modified by proofek)

I'm using Trac 0.11rc2 and PeerReview 2.2-dev branch and I don't have ClearSilver installed. When I go to peerReviewSearch I get the following:

Trac detected an internal error: 
AssertionError: HDF dataset not available. Check your clearsilver installation

Python Traceback
Most recent call last: 
File "c:\progra~1\python25\lib\site-packages\trac-0.11rc2-py2.5.egg\trac\web\main.py", line 423, in _dispatch_request 

File "c:\progra~1\python25\lib\site-packages\trac-0.11rc2-py2.5.egg\trac\web\main.py", line 206, in dispatch 

File "c:\progra~1\python25\lib\site-packages\trac-0.11rc2-py2.5.egg\trac\web\api.py", line 297, in display

Code fragment:

def display(self, template, content_type='text/html', status=200):
	"""Render the response using the ClearSilver template given by the
	`template` parameter, which can be either the name of the template file,
	or an already parsed `neo_cs.CS` object.
	"""

	assert self.hdf, 'HDF dataset not available. Check your clearsilver installation'
	if self.args.has_key('hdfdump'):
		# FIXME: the administrator should probably be able to disable HDF
		#        dumps
		self.perm.require('TRAC_ADMIN')
		content_type = 'text/plain'

Local variables:

Name Value 
content_type 'text/html' 
self <Request "GET u'/peerReviewSearch'"> 
status 200 
template 'peerReviewSearch.cs' 

Attachments (0)

Change History (3)

comment:1 follow-up: Changed 6 years ago by proofek

  • Status changed from new to assigned

Yeah, it's been spotted a little bit earlier already. Thanks for raising a ticket for us. A fix should be ready in next few days.

BTW. bear in mind 2.2dev is a development branch so you can expect some stuff be broken from time to time. I won't advise to use it in production env.

Seb

comment:2 in reply to: ↑ 1 Changed 6 years ago by anonymous

Replying to proofek:

Yeah, it's been spotted a little bit earlier already. Thanks for raising a ticket for us. A fix should be ready in next few days.

BTW. bear in mind 2.2dev is a development branch so you can expect some stuff be broken from time to time. I won't advise to use it in production env.

Seb

Thanks a lot! Are you going to merge it back to 0.11 branch, or I would still have to go with 2.2dev?

comment:3 Changed 6 years ago by proofek

  • Description modified (diff)
  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed both in 2.1-toddler and 2.2-dev.

Add Comment

Modify Ticket

Action
as closed The owner will remain proofek.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.