Ticket #3187 (closed defect: fixed)

Opened 6 months ago

Last modified 6 months ago

There are still some leftovers from ClearSilver

Reported by: anonymous Assigned to: 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

Change History

(follow-up: ↓ 2 ) 06/16/08 02:59:23 changed 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

(in reply to: ↑ 1 ) 06/16/08 04:17:05 changed 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?

06/17/08 02:44:31 changed by proofek

  • status changed from assigned to closed.
  • resolution set to fixed.
  • description changed.

Fixed both in 2.1-toddler and 2.2-dev.

Add/Change #3187 (There are still some leftovers from ClearSilver)




Change Properties
Action