[Patch] Support for NumberedHeadlinesPlugin

[tiago@…]

When trying to edit a section delimited by the NumberedHeadlinesPlugin, an error such as the following is seen:

The section 1 that you chose could not find.

Not sure if it´s a consistent solution, however I hacked the re in line 99 of web_ui.py and seems to work fine here for both =* and #* headline syntax:

0.11/tracsectionedit/web_ui.py

@@ -96 +96 @@
                 is_code_block = True
             elif is_code_block == True and re.match(r'^\s*}}}\s*$', line):
                 is_code_block = False
-            if is_code_block == False and re.match(r'^\s*(={1,5}) .+ \1(?:\s*#.+)?\s*$', line):
+            if is_code_block == False and re.match(r'^\s*([=#]{1,5}) .+ \1(?:\s*#.+)?\s*$', line):
                 count = count + 1
             if count < int(section):
                 pre.append(line)
@@ -106 +106 @@
                 break
         post = page_list[i:]
         if len(target) == 0:
-            raise TracError(_('The section %(num)d that you chose could not find.', num=int(section)), _('Initialize Error'))
+            raise TracError(_('The section %(num)d that you chose could not be found.', num=int(section)), _('Initialize Error'))
 
         return pre, target, post

Thanks,

[Ryan J Ollos]

I fixed up the patch in the description and tested it out. I'd like to see it applied to the repository if possible, it's a net 3 char change to the regex. I'm happy to go ahead and do this if its okay with the plugin maintainer (have global write access to the repository).

[Ryan J Ollos]

Replying to rjollos:

I fixed up the patch in the description and tested it out. I'd like to see it applied to the repository if possible, it's a net 3 char change to the regex. I'm happy to go ahead and do this if its okay with the plugin maintainer (have global write access to the repository).

Since no response from the author after two weeks and its such a simple patch, I'm going to go ahead and apply it. We can always roll it back if the author has an objection.

[Ryan J Ollos]

(In [9816]) Added support for the NumberedHeadlinesPlugin. Thanks to tiago for the patch. Fixes #5460.

[Ryan J Ollos]

#6712 and #6946 closed as duplicates.

[Ryan J Ollos]

#9302 is probably related.