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 .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from proofek. Next status will be 'closed'.
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.