TracLink to a PDF in the repository is not redirected

Yes, svn behaves in the same way. I agree that this would be a nice enhancement.

Proposed patch, which is nearly a complete rewrite of this tiny plugin.

• trunk/pdfredirector/pdfredirector.py

  old	new
  13	13	return template, data, content_type
  14	14
  15	15	def pre_process_request(self, req, handler):
  	16
  16	17	path = req.path_info.strip('/').split('/')
  17		if len(path) < 2 or path[0] != 'attachment' or req.args.get('action') == 'delete':
  18		return handler
  19
  20		if not path[-1].lower().endswith('.pdf'):
  21		return handler
  22
  23		filename = req.href(*(['raw-attachment'] + path[1:]))
  24
  25		req.redirect(filename)
  	18
  	19	if len(path) > 1 and path[-1].lower().endswith('.pdf'):
  	20	if path[0] == 'attachment' and not req.args.get('action') == 'delete':
  	21	filepath = req.href(*(['raw-attachment'] + path[1:]))
  	22	req.redirect(filepath)
  	23	elif path[0] == 'browser':
  	24	path[0] = 'export'
  	25	rev = req.args.get('rev', 'HEAD')
  	26	path.insert(1, rev)
  	27	filepath = req.href('/'.join(path))
  	28	req.redirect(filepath)
  26	29
  27
  28
  29
  30
  	30	return handler

(In [11799]) Refs #8066: Redirect TracLink? to a PDF in the repository.

Code review and feedback welcomed and appreciated.

I think it is time to bump the revision and create a tag for this. Anyone r11799 or later that can give some feedback?

(In [12181]) Refs #8066: Bumped version to 0.3. Added tags directory.

(In [12182]) Fixes #8066: Tagged version 0.3.