Ticket #1835 (closed enhancement: fixed)

Opened 1 year ago

Last modified 1 year ago

[patch] AddCommentMacro updated to 0.11

Reported by: osimons Assigned to: athomas
Priority: normal Component: AddCommentMacro
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

Enclosed is a patch that migrates AddCommentMacro to Trac 0.11.

It maintains the structure and workings of the 0.10 version, and as a working macro it should now be easier to refactor further to use Genshi tag() or even a template for output rendering. However, I have intentionally left this out for now in order to have as few changes as possible.

Some comments on the patch:

  • It depends on a 0.11 update of MacroPostPlugin. I have created a separate ticket with a patch for this - see #1832. A bumped version of 0.2 is made to syncronise, but feel free to find some other means of matching this dependency.
  • I have also bumped the version of AddCommentMacro to 0.3 to signify a change - again, feel free to name the version as you like.
  • There is one new feature in this patch - or rather, and adjustment to 0.11 that is not just migrating code: Like Trac wiki pages, the authname will always default to req.authname if the user is logged in. The username textbox will only show for anonymous users.
  • The macro documentation is improved slightly to add the missing argument and so on.
  • I branched it locally before updates (svn cp 0.10 0.11), so the paths in the diff refers to 0.11. It is originally branched from 0.10 rev 2489.

Attachments

addcommentmacro-011dev-r2502.diff (11.2 kB) - added by osimons on 07/20/07 17:39:42.
Patch for 0.11 version of AddCommentMacro.
addcommentmacro-011dev-r2507.diff (11.1 kB) - added by osimons on 07/22/07 16:16:19.
A to_unicode() had dropped out during initial conversion. New version of patch (replacing 2502)
addcommentmacro-011dev-r2792.diff (12.0 kB) - added by osimons on 11/17/07 15:53:53.
Updated to work with the very recent 0.11 - new context / resource APIs, security, genshi.builder and so on.

Change History

07/20/07 17:39:42 changed by osimons

  • attachment addcommentmacro-011dev-r2502.diff added.

Patch for 0.11 version of AddCommentMacro.

07/22/07 16:16:19 changed by osimons

  • attachment addcommentmacro-011dev-r2507.diff added.

A to_unicode() had dropped out during initial conversion. New version of patch (replacing 2502)

11/17/07 15:53:53 changed by osimons

  • attachment addcommentmacro-011dev-r2792.diff added.

Updated to work with the very recent 0.11 - new context / resource APIs, security, genshi.builder and so on.

11/18/07 01:05:37 changed by osimons

  • status changed from new to closed.
  • resolution set to fixed.

Latest patch applied in [2797]. Closing.


Add/Change #1835 ([patch] AddCommentMacro updated to 0.11)




Change Properties
Action