fail on fresh install CentoOS 5.6

• api.py

  old	new
  85	85	if row[0] == 'default':
  86	86	self.default_outcome = row[1].lower()
  87	87	else:
  88		color = row[0].partition('.')[0]
  89		outcome = row[0].partition('.')[2].lower()
  	88	color = row[0].split('.',1)[0]
  	89	outcome = row[0].split('.',1)[-1].lower()
  90	90	caption = row[1]
  91	91
  92	92	if color not in self.outcomes_by_color:
  …	…
  311	311	elif type == 'testplan':
  312	312	req.perm.require('TEST_PLAN_ADMIN')
  313	313
  314		catid = path.rpartition('_TT')[2]
  	314	catid = path.split('_TT')[-1]
  315	315
  316	316	try:
  317	317	# Add the new test plan in the database
  …	…
  345	345	tcIdsList = [tcId]
  346	346
  347	347	try:
  348		catid = path.rpartition('_TT')[2]
  	348	#catid = path.rpartition('_TT')[2]
  	349	catid = path.split('_TT')[-1]
  349	350	tcat = TestCatalog(self.env, catid)
  350	351
  351	352	for tcId in tcIdsList:
  352	353	if tcId is not None and tcId != '':
  353	354	old_pagename = tcId
  354		tc_id = tcId.rpartition('_TC')[2]
  	355	#tc_id = tcId.rpartition('_TC')[2]
  	356	tc_id = tcId.split('_TC')[-1]
  355	357
  356	358	tc = TestCase(self.env, tc_id, tcId)
  357	359	tc.author = author
  …	…
  375	377	req.redirect(req.href.wiki(pagename))
  376	378
  377	379	# Redirect to test catalog, forcing a page refresh by means of a random request parameter
  378		req.redirect(req.href.wiki(pagename.rpartition('_TC')[0], random=str(datetime.now(utc).microsecond)))
  	380	#req.redirect(req.href.wiki(pagename.rpartition('_TC')[0], random=str(datetime.now(utc).microsecond)))
  	381	req.redirect(req.href.wiki('_TC'.join(pagename.split('_TC')[:-1]), random=str(datetime.now(utc).microsecond)))
  379	382
  380	383	elif duplicate and duplicate != '':
  381	384	# Duplicate test case
  382		old_id = tcId.rpartition('_TC')[2]
  	385	#old_id = tcId.rpartition('_TC')[2]
  	386	old_id = tcId.split('_TC')[-1]
  383	387	old_pagename = tcId
  384	388	try:
  385	389	old_tc = TestCase(self.env, old_id, old_pagename)
  …	…
  430	434	req.perm.require('TEST_PLAN_ADMIN')
  431	435
  432	436	planid = req.args.get('planid')
  433		catid = path.rpartition('_TT')[2]
  	437	#catid = path.rpartition('_TT')[2]
  	438	catid = path.split('_TT')[-1]
  434	439
  435	440	self.env.log.debug("About to delete test plan %s on catalog %s" % (planid, catid))

I hacked up the partition/rpartition stuff with split and rsplit. I doing so there are too many places where the path is being analyzed and where the same bits of code exist over and over. These path functions SHOULD be refactored into the util.py module to reduce chance for bugs like the one in get_enclosing_catalog in model.py

else:
            #cat_id = page_name.rpartition('TT')[0].page_name.rpartition('TT')[2].rpartition('_')[0]
            # XXX THIS LOOKS LIKE A BUG. STRING FROM FIRST rpartition WILL NOT HAVE ATTRIBUTE page_name
            cat_id = page_name.rsplit('TT',1)[0].rsplit('TT',1)[-1].rsplit('_',1)[0]

Hi, I guess the problem is your Python version.

Only versions 2.5+ are supported. See also #4846.

Consider that Python 2.4 is not supported by Trac either.

I'll think about using split instead of partition. In the meantime, you can add the following functions to "utils.py":

def partition(str, ch):

        idx = str.find(ch)

        if idx >= 0:
                return (str[0:idx], ch, str[idx+1:])
        else:
                return (str,'','')

def rpartition(str, ch):

        idx = str.rfind(ch)

        if idx >= 0:
                return (str[0:idx], ch, str[idx+1:])
        else:
                return ('','',str)

And replace every occurrence of their use as follows:

somestring.partition('A') ====> partition(somestring, 'A')
somestring.rpartition('A') ====> rpartition(somestring, 'A')

Ciao, Roberto