Browser should show SVN revision properties

[eblot]

Note: This RFE refers to Trac with SVN back-end, but could also be applied to other backend, such as Mercurial back-end.

Trac browser shows file-based properties (for example, svn:ignore).

However, it ignores revision-based properties.
It would be nice if Trac were able to show this kind of properties as well.

[cboos]

Are you sure you want to see those properties in the TracBrowser, not in the TracChangeset view?

[eblot]

You're right, it makes (more) sense to have it in the TracChangeset view.

[cboos]

Initial implementation of the feature

[cboos]

I've uploaded an initial implementation for this. What remains to be done, is a way to associate a plaintext flag as well as an htmlclass (one of time, author, message or changeset) to the property (see the API).

Or perhaps reuse some of the forthcoming mechanisms for rendering node properties (see #1601).

[ttressieres]

do you think the patch could be applied in 0.10.1 or 0.11 Trac version ? I need it to implement the feature in http://trac-hacks.org/ticket/628.

[mgood]

Replying to ttressieres:

do you think the patch could be applied in 0.10.1 or 0.11 Trac version

This is scheduled for 0.11. Upcoming 0.10.x releases will only contain bugfixes.

[cboos]

Yes, I'll commit the patch (for 0.11) but be prepared for possible interface change once #1601 progresses...

[ttressieres]

Replying to cboos:

Yes, I'll commit the patch (for 0.11) but be prepared for possible interface change once #1601 progresses...

Yes, i saw this ticket too, i'll be prepared... thank you.

[cboos]

Patch updated and applied in r4130.

[cboos]

As advertised, the revision properties are now rendered using the IPropertyRenderer from #1601.

Please take a look at the source:sandbox/property-renderers-tmp@4530 (see Last Change from there); I'd be interested in eventually getting some feedback before integrating that work branch in trunk.

[eblot]

I just gave a try. It seems to work fine w/ my custom properties. Thanks!

As a small enhancement, I think the custom properties should be more visible, or at least differenciated from the regular SVN properties. Maybe a different color/style (CSS), and all custom properties grouped as they are now, but separated from the regular properties with a blank line or the like.

I'm thinking about another enhancement - that could be done through another ticket though: it would be nice if it was possible to define a property-to-name mapping in trac.ini. For example, svn:log is shown as Message, svn:author as Author. It would be great to define custom property names in the config file, such as in the following example

[svnprops]
my:version = Version

[cboos]

See also r4618, which introduces the possibility to have even more control over the rendered property, by returning a RenderedProperty instance instance of just text or markup - which is still possible, of course. This is especially useful for changeset properties.

[cboos]

Support for RenderedProperty (in addition to plain text and Markup return values) merged in trunk in r4655.