Jump to edited section after edit

Let me prefix this by saying I absolutely don't know what I'm doing here, but I hacked around with this enough to get the following patch working.

The changes are:

1. In the Javascript function, I append the name of the heading's anchor to the url fragment.
2. In filter_stream, I followed the pattern of inserting a hidden html element, by adding the anchor name as a hidden element.
3. In pre_process_request, I get the anchor name and use that to construct a redirect, as is done for Trac tickets after a comment is posted. See web_ui.py in Trac 0.12 for reference.

I'm particularly unsure of step (2) Maybe there is a better way? Any help and suggestions would be greatly appreciated!!

Patch against [11180].

• 0.12/tracsectionedit/htdocs/js/tracsectionedit.js

  old	new
  5	5	var href_edit = document.location.href;
  6	6	href_edit = (href_edit.indexOf('?')==-1?'?':href_edit+'&')+"action=edit&section=";
  7	7	return this.filter("*[@id]").each(function(){
  8		$("<a class='anchor'>[<span>edit</span>]</a>").attr("href", href_edit + cnt++).attr("title", title).appendTo(this);
  	8	var anchor = $(this).attr('id')
  	9	$("<a class='anchor'>[<span>edit</span>]</a>").attr("href", href_edit + cnt++ + "&anchor=" + anchor).attr("title", title).appendTo(this);
  9	10	});
  10	11	}
  11	12

• 0.12/tracsectionedit/web_ui.py

  old	new
  49	49	req.args['text'] = "%s%s%s"%(req.args.get('section_pre'),
  50	50	section_text,
  51	51	req.args.get('section_post'))
  	52	fragment = '#' + req.args.get('anchor')
  	53	req.redirect(req.href.wiki(req.args['page']) + fragment)
  52	54	return handler
  53	55
  54	56	def post_process_request(self, req, template, data, content_type):
  …	…
  70	72	section_element = html.input(type='hidden', name='section', id='section', value=req.args.get('section'))
  71	73	pre_element = html.input(type='hidden', name='section_pre', id='section_pre', value=''.join(section_pre))
  72	74	post_element = html.input(type='hidden', name='section_post', id='section_post', value=''.join(section_post))
  	75	anchor_element = html.input(type='hidden', name='anchor', id='anchor', value=''.join(req.args.get('anchor')))
  73	76
  74		section_html = html(section_element, pre_element, post_element)
  	77	section_html = html(section_element, pre_element, post_element,anchor_element)
  75	78	stream = stream | Transformer('//textarea[@name="text"]').empty().append(section_text).before(section_html)
  76	79	stream = stream | Transformer('//div[@id="content"]//h1').append("/%s (section %s)"%(section_text[:section_text.find('\n')].strip(" = \r\n"), req.args['section']))
  77	80	if not self.preview_whole_page:

(In [11192]) Refs #4970: Created a dev branch for version 0.3, which includes the patch from comment:2.

The 0.11 and 0.12 branches will be merged to produce a single code line for this plugin at the time version 0.3 is released.

Well, this patch is no good because it does a redirect in pre_process_request and therefore the change doesn't get saved. It seems like the redirect needs to be done in post_process_request, however, I can't find a way to retrieve req.args['anchor'] in post_process_request, and I don't see a way to pass the value from pre_process_request to post_process_request. I spent about 5 hours on this, so I'm pretty much stuck at this point. It's possible the plugin needs some major redesign in order to implement this feature.

(In [11204]) Refs #4970: Removing 0.12dev branch for now.

I'm stuck on this after about 8 hours of effort and experimentation. Maybe someone will come along and help me out? ;) Otherwise, I'm moving on for now ...