[wp-trac] [WordPress Trac] #22917: Missing User & Site number in Multisite
WordPress Trac
noreply at wordpress.org
Mon Jan 7 18:10:27 UTC 2013
#22917: Missing User & Site number in Multisite
--------------------------+------------------
Reporter: vmaia | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.6
Component: General | Version: 3.5
Severity: normal | Resolution:
Keywords: has-patch |
--------------------------+------------------
Comment (by adamsilverstein):
Replying to [comment:11 wpmuguru]:
> right-now-liv.diff would render the caching on this mostly worthless for
large networks with frequent user/site creation. Someone who has a small
network that wants to have live stats can easily accomplish this with a
small plugin
>
> {{{
> add_action( 'wpmu_new_blog', 'wp_update_network_counts' );
> add_action( 'wpmu_new_user', 'wp_update_network_counts' );
> // etc.
> }}}
>
> recommend wontfix.
>
>
we could add a check for large sites before running the update, but i
agree my patch renders caching useless. still the current behavior is
confusing to novice users, and for small networks without frequent
user/site creation i think the caching is superfluous.
would you accept a patch that, instead of recalculating the entire count,
simply pulled and updated the current count - incremented/decremented
based on the current action? no query would be required, just the option
updated. same locations as my existing changes i believe.
caching is great, but in this case its confusing the end user. it almost
sounds like you are saying the ticket itself should be thrown out, that
the current behavior is fine as is.
do you agree with the ticket idea that the widget should show the live
count, or do you think the caching is more important and things should
stay as they are? could we use wp_is_large_network() to split up the
cases?
--
Ticket URL: <http://core.trac.wordpress.org/ticket/22917#comment:12>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list