Dynamicity of Trac (Show number of registered users)

[anonymous]

show the number of registered users on top of main page

[Ryan J Ollos]

I think we should provide the number of registered users in the AccountManagerPlugin via the UserStats macro.

[Steffen Hoffmann]

Yes, that fits in there indeed.

[Steffen Hoffmann]

Replying to hasienda:

Yes, that fits in there indeed.

But the priority is not serious. This can easily be postponed behind the acct_mgr-0.4 release.

[Steffen Hoffmann]

Hm, isn't it already there?

UserQuery(count)

will yield exactly the number of known users, see account-/user-related WikiMacros for everyone ("Requires USER_VIEW permission for output in any format other then 'count'."). Thoughts?

[Ryan J Ollos]

Yeah, that works well for me. I think we are covered on the macro side.

It might be nice to show the number of registered users somewhere on the /admin/accounts/users page, but that is a separate matter.

[Steffen Hoffmann]

Replying to rjollos:

Yeah, that works well for me. I think we are covered on the macro side.

Thanks for confirmation.

It might be nice to show the number of registered users somewhere on the /admin/accounts/users page, but that is a separate matter.

Oh, good catch , so why not read this request like that then ('main page' == 'user admin panel')? I'll do that.

[Ryan J Ollos]

Replying to hasienda:

Oh, good catch , so why not read this request like that then ('main page' == 'user admin panel')? I'll do that.

Yeah, the request is a bit ambiguous, but that makes sense. I looked at the user admin panel for a while, but the best place to locate the user count didn't jump out at me. Perhaps to the right of Manage User Accounts, like is done on the report page.

[Steffen Hoffmann]

(In [12496]) AccountManagerPlugin: Add number of accounts into user admin panel, refs #7426.

[Ryan J Ollos]

Patch against r12500 of the trunk.

[Ryan J Ollos]

From my work on ​t:#10743, I know of a different way to accomplish this, which could avoid adding the additional stylesheet: th7426-r12500-1.patch​.

[Ryan J Ollos]

On second look, after the patch is applied the styling of the text is slightly different. I think the patch is an improvement though since the text is slightly larger, so I still favor applying the patch.

[Steffen Hoffmann]

(In [12514]) AccountManagerPlugin: Obsolete additional CSS styles, refs #7426.

Changing to another .class allows to remove report.css, that has been added in [12496]. Thanks to Ryan J Ollos for proposing this simplification.

[Ryan J Ollos]

It appears I may have led you astray here because the trac-count class wasn't added until ​[t 11127], meaning the class isn't available in Trac < 1.0. I think the styling is nice though, and use of the trac-count class will provide consistent styling in Trac >= 1.0. Maybe we should copy the class to acct_mgr.css temporarily, until support for Trac < 1.0 is dropped (AccountManager 0.5 and later?).

[Steffen Hoffmann]

Replying to rjollos:

It appears ... the class isn't available in Trac < 1.0. ... Maybe we should copy the class to acct_mgr.css temporarily, until support for Trac < 1.0 is dropped (AccountManager 0.5 and later?).

Good point. Let's do that.

Enough new stuff has been prepared for another "main" dot-release, so acct_mgr-0.5 will certainly get released before switching the plugin db backend code to Trac 1.0 API, consequently before 0.6dev.

[Steffen Hoffmann]

(In [12532]) AccountManagerPlugin: Supply copy of recent Trac CSS style inside the plugin, refs #7426.

Done for compatibility with Trac < 1.0, even if its a very minor tweak.

[Ryan J Ollos]

Replying to hasienda:

Enough new stuff has been prepared for another "main" dot-release, so acct_mgr-0.5 will certainly get released before switching the plugin db backend code to Trac 1.0 API, consequently before 0.6dev.

Ah, right. I have the development plan straight in my mind, but keep mixing up the numbers when it comes time to type ;)

[Ryan J Ollos]

Oh, and just to avoid confusion down the road, we have everything straight in this ticket, but the code comment says Trac <= 1.0, whereas it should say Trac < 1.0.

[Ryan J Ollos]

See also ​t:#11027, which this ticket inspired.

[Steffen Hoffmann]

(In [12571]) AccountManagerPlugin: Add descriptive tool-tip to 'Toggle account approval', refs #7426 and #8595.

A CSS style comment is now corrected as well.

Thanks to Ryan J Ollos for hints on these minor issues, nevertheless I care and I'm happy to receive code reviews for getting things straight.