Opened 9 years ago

Closed 5 years ago

No documentation on WikiMacros page

Reported by: Owned by: Ryan J Ollos Ryan J Ollos high WikiStatsPlugin normal macro description 0.11

There is no documentation for this plugin on the WikiMacros page for an instance of Trac where the macro is installed. It would be nice to add a short entry describing how to use each macro. See attached screen capture.

Changed 9 years ago by Ryan J Ollos

Screen capture showing no documentation on WikiMacros page

comment:1 Changed 8 years ago by Ryan J Ollos

Component: UserStatsMacro → WikiStatsPlugin

comment:2 Changed 8 years ago by anonymous

Owner: changed from Pedro Paixao to RottenChester

comment:3 Changed 7 years ago by Ryan J Ollos

Owner: changed from RottenChester to Ryan J Ollos normal → high new → assigned

comment:4 Changed 6 years ago by Ryan J Ollos

Status: assigned → new

comment:5 follow-up:  6 Changed 6 years ago by Steffen Hoffmann

For various reasons I'm aggressively rebuilding WikiStatsPlugin functionality inside AccountManagerPlugin (see #9852).

For what it's worth, please take a look at it's embedded macro descriptions. Comments welcome.

comment:6 in reply to:  5 ; follow-up:  7 Changed 6 years ago by Ryan J Ollos

For various reasons I'm aggressively rebuilding WikiStatsPlugin functionality inside AccountManagerPlugin (see #9852).

Thanks for doing that, I think it will be a very nice addition to the AccountManagerPlugin. I'm going to follow though on the adoption for this plugin, fix the remaining open issues and mark it as soon to be deprecated, in order to direct attention to the AccountManagerPlugin.

For what it's worth, please take a look at it's embedded macro descriptions. Comments welcome.

It is looking very good. I've never liked that the WikiStatsPlugin has a macro named Stats, feeling that it is too generic of a name. Perhaps ProjectStats, or TracStats? Off hand, I'd prefer ProjectStats. This would also prevent a collision in the case that both macros are installed. Or are you aiming for backward compatibility with this plugin?

comment:7 in reply to:  6 Changed 6 years ago by Steffen Hoffmann

For various reasons I'm aggressively rebuilding WikiStatsPlugin functionality inside AccountManagerPlugin (see #9852).

Thanks for doing that, I think it will be a very nice addition to the AccountManagerPlugin. I'm going to follow though on the adoption for this plugin, fix the remaining open issues and mark it as soon to be deprecated, in order to direct attention to the AccountManagerPlugin.

Great you like it. But for the other part I hoped to discourage you enough at least to not take #3874. After reading the code I think it's worth to keep the idea. The implementation is a bit too straight-forward, i.e. can't make use of new caching capability in Trac 0.12, and more. I'll try my best to warrant depreciation of this fine plugin in favor of the new AcctMgr wiki macro suite.

It is looking very good. I've never liked that the WikiStatsPlugin has a macro named Stats, feeling that it is too generic of a name. Perhaps ProjectStats, or TracStats? Off hand, I'd prefer ProjectStats. This would also prevent a collision in the case that both macros are installed. Or are you aiming for backward compatibility with this plugin?

Yes, I'd like to take the hurdle of supplying backwards-compatibility for someone with hundreds of existing macro invocations. OTOH, replacing just the name and keeping a compatible parameter set could be an acceptable compromise. I confess, that I liked the old short name for shortness, but it's truly unspecific wording and absence of CamelCase is unsatisfying. I trust you for the choice in general: Stats will become ProjectStats for AcctMgr offerings soon.

Btw, I did look at TracStatsPlugin the first time after you mentioned it here, great stuff. But external site and requirement of a GitHub account has nearly scared me away from creating an issue. Hope we'll gain more on the t-h.o side to convince people to a least to mirror their repositories here again.

comment:8 Changed 6 years ago by Ryan J Ollos

Should we deprecate this plugin, close all of the open tickets and direct users to the AccountManagerPlugin? I think that comment:7 is suggesting this approach.

My thinking is:

• If a user wants this macro but not the account manager features, they can install AccountManagerPlugin and only enable the macros.
• One of the macros depends on AccountManagerPlugin anyway.
• I have too much to maintain and I'm trying to dump things that have no value. Let's drive users towards the AccountManagerPlugin.

Thoughts?

comment:9 Changed 6 years ago by Steffen Hoffmann

I'll try to make WikiMacros inside of AccountManagerPlugin as independent as possible from the other modules and will seek graceful degradation, if possible.

More elaborated statistics certainly involve heavy data processing (see my related comments to #3874). So for me it feels better to move to AccountManager or even TracStatsPlugin for that purpose, than to maintain a somewhat half-hearty and not portable hack.

IMHO depreciation with pointing to alternative and in parts already superior solution would be a reasonable decision.

comment:10 Changed 5 years ago by Ryan J Ollos

Resolution: → wontfix new → closed

This plugin is deprecated, and a notice has been posted to the WikiStatsPlugin page with more details. Please comment here if you have any concerns.

Fine for me.

comment:12 Changed 4 years ago by Ryan J Ollos

Description: modified (diff)

Modify Ticket

Change Properties